server not found in app.py

After the major updating, the statement server = app. server is deleted which makes the Heroku deployment failed

I updated the app.py with the required extra line


Commands to enable SSTP and OpenVPN in the latest (compiled) version of SoftEther

SSTP Enable and OpenVPN Enable are not in the vpncmd commands list when I compiled the latest version from this repo from source (Following the BUILD_UNIX instructions). I am assuming they were just moved into another command and if so what command?
To reproduce follow the BUILD_UNIX instructions on Ubuntu 20.04 and start the vpn server as root. Use vpncmd as root as well. For now I went back to using the PPA but I would love to try the new things recently merged.

When you're talking about latest version, specify the commit hash or tag.
It doesn't reproduce to me. Mine is 5.01.9674

Also show me the output of the commands.

They were replaced in #1177.

Yeah I notice that command and tried to use it  but when I ran
ProtoOptionsGet SSTP
Nothing showed up and then I ran
ProtoOptionsSet
For the protocol I entered SSTP, for the option I put in enabled, and for the attribute I put true and I was told the command completed successfully. But even though it says it was completed successfully it did not work.

I think you're using an up-to-date vpncmd, but vpnserver is not updated.

I literally cloned the source code from master yesterday and compiled it so I would assume the client, server, vpncmd, and vpnbridge would all be the correct version. Also when I use the older GUI it does not work.

UPDATE: After looking in my client logs (VPN Client Pro, Android) it seems that SSL socket closed by remote side seems to be the real error. So I guess your enable command wasn't the real issue.

Based on my testing, it seems to be an error with the SSL or TLS. I am now using Davide's WireGuard branch and I am having no problems with SSTP. There must of been some change with SSL or TLS in between  Davide's WireGuard branch and SoftEther's Master branch.

Here is the server log related to the issue.
2020-11-20 04:41:49.546 On the TCP Listener (Port 443), a Client (IP address 172.x.x.3, Host name "ue.tmodns.net", Port number 61946) has connected.
2020-11-20 04:41:49.546 For the client (IP address: 172.x.x.3, host name: "ue.tmodns.net", port number: 61946), connection "CID-29" has been created.
2020-11-20 04:41:49.626 SSL communication for connection "CID-29" has been started. The encryption algorithm name is "TLS_AES_256_GCM_SHA384".
2020-11-20 04:42:08.534 Connection "CID-29" terminated by the cause "Connection has been disconnected." (code 3).
2020-11-20 04:42:08.534 Connection "CID-29" has been terminated.
2020-11-20 04:42:08.534 The connection with the client (IP address 172.x.x.3, Port number 61946) has been disconnected.

After doing some digging, I have a feeling this has to do with the changes merged in #1235 I'm not even close to an experienced developer so you might want to investigate deeper yourself. The reason I think this is the problem is because of changes to the OpenSSL security.

Had the same problem, so I compiled SE-VPN with debugging enabled and attached gdb to it (server is a Debian stable amd64).
While trying to connect via SSTP (with Windows 10 Enterprise) it crashed on line 513 of Cedar/Proto.c:
impl = container->Impl;
Changing that line to what I guess it should be makes SSTP work for me again. But I'm not a programmer and this is sensitive code, so I really don't want to provide my possibly broken "fix" here :)
Could someone with insight to the code please have a look at this?

@sl077 please provide your "fix" so I can test it. If it works well then we can talk to @davidebeatrici and see if it can be made into a PR.

@andrewfer000 sure, I just replaced that line at Cedar/Proto.c:513 by
container = tmp;

@sl077 Seems like this fixes the SSTP error. I can confirm it works on Android (VPN Client Pro) and Windows 10 but it seems that this breaks L2TP/IPsec (and likley the other PPP based protocols) but the other protocols such as OpenVPN and SoftEther are working fine.

@andrewfer000 are you sure about L2TP/IPsec? I'm currently connected (on Windows Server 2019) via L2TP/IPsec+PSK to my Test-SE-VPN (running on Debian Buster amd64) patched like this (and actually a few unrelated IPv6 related patches I'm currently testing) and for me it seems to work fine?!?

It might just be Davide's proto-wireguard branch.  I guess I'll just have to wait for Wireguard and use Master. I am assuming if Davide updates his branch with the latest commits it might fix this.

Nope, Still have the same issue in Master

I think it would be really strange if this patch breaks L2TP/IPsec, as the affected code seems to switch between different possible protocols on shared ports like 443. But afaik L2TP/IPsec is working with a completely different set of protocols unrelated to this?

Looks like that was the issue. I was using my Domain and it failed. I will re-test on Davide's Wireguard branch and see if it works. Thanks.

2020-12-07 23:43:29.327 IPsec Client 1 (73...214:500 -> <IP_ADDRESS>:500): A new IPsec client is created.
2020-12-07 23:43:29.327 IPsec IKE Session (IKE SA) 1 (Client: 1) (73...214:500 -> <IP_ADDRESS>:500): A new IKE SA (Main Mode) is created. Initiator Cookie: 0x217F83FCB6DA5BF1, Responder Cookie: 0x15CAC2459EA386C0, DH Group: MODP 2048 (Group 14), Hash Algorithm: SHA-2-256, Cipher Algorithm: AES-CBC, Cipher Key Size: 256 bits, Lifetime:<PHONE_NUMBER> Kbytes or 10800 seconds
2020-12-07 23:43:29.437 IPsec Client 1 (73...214:1100 -> <IP_ADDRESS>:4500): The port number information of this client is updated.
2020-12-07 23:43:29.437 IPsec Client 1 (73...214:1100 -> <IP_ADDRESS>:4500):
2020-12-07 23:43:29.437 IPsec IKE Session (IKE SA) 1 (Client: 1) (73...214:1100 -> <IP_ADDRESS>:4500): This IKE SA is established between the server and the client.
2020-12-07 23:43:29.478 IPsec IKE Session (IKE SA) 1 (Client: 1) (73...214:1100 -> <IP_ADDRESS>:4500): The client initiates a QuickMode negotiation.
2020-12-07 23:43:29.478 IPsec ESP Session (IPsec SA) 1 (Client: 1) (73...214:1100 -> <IP_ADDRESS>:4500): A new IPsec SA (Direction: Client -> Server) is created. SPI: 0x9CB47287, DH Group: (null), Hash Algorithm: SHA-1, Cipher Algorithm: AES-CBC, Cipher Key Size: 256 bits, Lifetime:<PHONE_NUMBER> Kbytes or 3600 seconds
2020-12-07 23:43:29.478 IPsec ESP Session (IPsec SA) 1 (Client: 1) (73...214:1100 -> <IP_ADDRESS>:4500): A new IPsec SA (Direction: Server -> Client) is created. SPI: 0xC126BEF2, DH Group: (null), Hash Algorithm: SHA-1, Cipher Algorithm: AES-CBC, Cipher Key Size: 256 bits, Lifetime:<PHONE_NUMBER> Kbytes or 3600 seconds
2020-12-07 23:43:29.518 IPsec ESP Session (IPsec SA) 1 (Client: 1) (73...214:1100 -> <IP_ADDRESS>:4500): This IPsec SA is established between the server and the client.
2020-12-07 23:43:29.568 IPsec Client 1 (73...214:1100 -> <IP_ADDRESS>:4500): The L2TP Server Module is started.
2020-12-07 23:43:29.639 L2TP PPP Session [73...214:1701]: A new PPP session (Upper protocol: L2TP) is started. IP Address of PPP Client: 73...214 (Hostname: "owner-Zephyrus-G"), Port Number of PPP Client: 1701, IP Address of PPP Server: <IP_ADDRESS>, Port Number of PPP Server: 1701, Client Software Name: "L2TP VPN Client - xelerance.com", IPv4 TCP MSS (Max Segment Size): 1314 bytes
2020-12-07 23:43:30.674 IPsec ESP Session (IPsec SA) 1 (Client: 1) (73...214:1100 -> <IP_ADDRESS>:4500): This IPsec SA is deleted.
2020-12-07 23:43:30.674 IPsec IKE Session (IKE SA) 1 (Client: 1) (73...214:1100 -> <IP_ADDRESS>:4500): This IKE SA is deleted.
2020-12-07 23:43:30.674 IPsec ESP Session (IPsec SA) 1 (Client: 1) (73...214:1100 -> <IP_ADDRESS>:4500): This IPsec SA is deleted.
2020-12-07 23:43:31.036 L2TP PPP Session [73...214:1701]: The PPP session is disconnected because the upper-layer protocol "L2TP" has been disconnected.
2020-12-07 23:43:31.036 L2TP PPP Session [73...214:1701]: The PPP session is disconnected.

I get logs like this, if my PPP authentication/security settings aren't valid. Did you check, that MS-CHAP v2 is available for this?
After creating a new VPN connection on Windows, I usually have to go to the network adapter settings in the control center and select that checkbox manually in the security tab of the VPN network adapter. On the other hand, I have to do this for SSTP connections as well, don't know why...
(Note that the given labels are guesswork by me, since I have a german Windows and I am basically only using it for gaming).

@sl077 Thank you very much! That's indeed the proper fix, feel free to create a pull request.
I apologize for the bug, it was introduced in 8685fe0da1a24aeed27b728e4a4f3e0393672951.
The reason why I didn't catch it is because I generally test changes with the native protocol, WireGuard and OpenVPN.
SSTP is the only protocol that requires direct specification, because the session is initiated through the HTTPS server.
As for L2TP/IPsec: it's handled separately right now. The objective is to handle all protocols in Proto, in the future.

@sl077 Yes I do have MS-CHAPv2 checked off on Linux and Windows (they are default for me at least) and I think it's strange that Android has no issue but Linux and Android does. For all I know this is caused because I am using the same config file from an older version of SoftEther. Also, great job on fixing the bug I look forward to seeing your PR.

Glad if I could help. But can any of you guys please create that PR or just include that line in some other patch roundup? I'm not using git for more than "clone" and guess you wouldn't see this fixed in 2020 otherwise...

Could you send me your email address?
I would like to set you as author for the commit.

@davidebeatrici it's my user name gmx.de. But this is really not necessary, it was a single edited line and I'm already happy, if it solves someones problems here.


Feature to navigate line-wise without preview window?

Is your feature request related to a problem? Please describe.
I'd like to navigate the search results without opening the preview window.
Describe the solution you'd like
Navigate line-wise without opening the preview window.

I don't follow, you can always navigate the results in any cases, which is a very fundamental function. And the preview window is a plus, not all the providers has the preview feature.

Closing, feel free to reopen if you still have some questions about this.

@liuchengxu it'll be nice to have an option to disable the automatic preview window, and only open it with a keyboard mapping on-demand. I like scrolling a file or grep results without the extra delay of preview window, and only toggle it on-demand. Let me know what you think..

No delay if you are using the Rust binary, it's all async.

Supported by 2afffaa using let g:clap_open_preview = 'never'


[ISSUE #1009]🚀Support client Broadcasting consume

Which Issue(s) This PR Fixes(Closes)

Fixes #1009
Brief Description

How Did You Test This Change?

🚀Thanks for your contribution🎉. CodeRabbit(AI) will review your code first🔥

🚀Thanks for your contribution🎉. CodeRabbit(AI) will review your code first🔥


Feature superset migration

search by author
can't edit name in metadata
versions!!!!!

@kyraSclark @pojojojo21 when committing changes for an asset the "selected version" dropdown does not update to match – is this intentional?

Updated file to be named store.json, including user config (access key) for other DCC integrations

Merged previous electron-store implementation with JSON store

Tweaked frontend version select UI a bit


[Dev] Use pre-commit for cmake-format

Describe the enhancement requested
We can use https://github.com/cheshirekow/cmake-format-precommit for this.
Component(s)
Developer Tools

Issue resolved by pull request 40545
https://github.com/apache/arrow/pull/40545


moved default event log to its own log

Weather or not this is preferable is up to you

It looks good, did you test it?

yes, although to create the log you will have to install the service


Implement Land Obstacle Class

Purpose

Just as we did for Boat obstacles, we need to implement a class for land obstacles.
Description

A Land class will be added to the obstacles.py. After implementation, we can decide if the Land and Boat classes are similar enough that  it would make sense to combine them.
There are also some design choices to make:

Since we care more about the depth of the water surrounding land obstacles, rather than the actual coastline, depth/bathymetric charts will be used as the data source
Although the current plan is to not take any inside passages, it would still be nice to make Sailbot capable of this, so we'll have to make sure the land data is pretty accurate
** For production though, we'll probably want to use a simplified land obstacle where the coastline is all connected as a continuous barrier, like the grey line in the figure

Resources

Raye Local Pathfinding

Adding this to visualize the current plan for data flow.
This is a rough sketch of how I plan to convert raw bathymetric data from GEBCO to a Land Obstacle.


docs: looks -> logs fix

Thank you, the same typo was present in the README.md so I went a head and fixed it


Group txn

Add function to support group transactions for - transferring microAlgo's signed by logic.
update example

We can also merge transferMicroAlgosLsig with transferMicroAlgosLsigAtomic. The difference would be in case of a single txn, we will assign a groupId for that transaction also. So, a single transaction now would mean - a group of 1 transaction.


[docs] Fix Tailwind CSS integration docs

[x] I have followed (at least) the PR section of the contributing guide.

I have several projects that use tailwind with MUI, removing @tailwind/base causes some problems when using tailwind, for instance when we use translate-y-12, it won't work without the @tailwind/base.
Tailwind CSS provide better alternative to remove all of its "preflight" base styles from its library. I have included the documentation in the link and explain in the 2nd point.
https://tailwindcss.com/docs/preflight#disabling-preflight
https://deploy-preview-32512--material-ui.netlify.app/material-ui/guides/interoperability/#tailwind-css

Thankyou for the detailed explanation on your review! I understand now why we shouldn't use [here] :)

I updated examples as well :)


How does App X work?

Overview
There are a handful of apps in this repository, each with different owners, deployment strategies, permissions, environments, etc. Most of this is documented, in one way or another, in the project subdirectories READMEs or code comments. The goal of this thread is to collect high-level info about each project (not full deployment instructions or READMEs). The below template contains questions which each app owner could answer to provide a useful starting point for looking at all the apps in one place and seeing where we might want to unify or standardize approaches:
Template
[App Name]
Owner(s): rcebulko, rsimha, ...
Directory: /my-app
Deployed on: [GCE | GAE | ???]
Project ID: amp-github-app-name
URL: [None | amp-app-link.appspot.com]
README link: README
Portability: [Can only work with amphtml | Could be modified to be used on other repos | is repository-agnostic/configurable]
Incoming requests:

GitHub Webhooks
Admin pages

Outgoing requests:

GitHub API
Travis API

Database: [Yes | No]
Authentication [No | GitHub OAuth | Basic Auth]
Language/Framework: Probot
If Probot:

Smee channel: http://smee.io/12345
GitHub user: ampprojectbot
GitHub App permissions (see https://github.com/settings/apps/my-app-name/permissions):

Checks: read & write (update/create check-runs on PRs)
Contents: read-only (read commits, branches, PRs, releases, etc)
Pull requests: read & write (read PR info, read & write comments)

Webhooks:

Pull request: update check
Pull request review comment: unused
Pull request review: update check
Status: monitor status changes
Label: unused

Personal access token name (from https://github.com/settings/tokens): ampprojectbot (owners bot, maybe other things)
Required permissions [no idea if these are accurate, just examples]:

public_repo: Needed to read PR metadata, ...
read: org: Needed to list members of organization teams
repo: status: Needed to update check-runs, leave comments

Package management: [Yarn | npm | idk]
Testing framework: Jest
Test command: npm run test
Tests run on presubmit: [Yes | No]
Deployment workflow: [Manual | Automated]
Can this be run locally?: [Yes | No]
If so:

Command to run locally: npm run dev
Env. vars required to run locally:

GITHUB_ACCESS_TOKEN: Access token with at least public_repo permission
GITHUB_REPO_DIR: Path to local copy of ampproject/amphtml repository (ex. /tmp/amphtml)
GITHUB_REPO: Name of repository (ie. ampproject/amphtml or your fork)

Public environment (ie. config): [Shell script &| .env &| app.yaml &| GCE/GAE metadata &| deploy command arguments &| config file(s)]
Private environment (ie. secrets, tokens, keys): [Shell script &| .env &| app.yaml &| GCE/GAE metadata &| deploy command arguments &| config file(s)]
App uses a private-key.pem file for authentication: [Yes | No]
Plan
I'm going to start posting for the apps I've created/worked on, and then will start trying to do the same for the other apps with as much info as I can find in the repo.
/cc @ampproject/wg-infra let me know if you think of anything that might be missing. Ideally we can identify some pieces of this to standardize, if not with actual infrastructure then at least with more standard processes for things like keeping track of permissions, which apps use which tokens (since you can't see which token is which in the GitHub UI, making it risky to ever delete any), manage deployments, testing, etc.

Owners Bot
Owner(s): rcebulko, erwinmombay
Directory: /owners
Deployed on: GCE
Project ID: amp-owners-bot
URL: No public URL
README link: README
Incoming requests:

GitHub Webhooks
Outgoing requests:
GitHub API

Database: No
Authentication: No
Language/Framework: Probot

Smee channel: http://smee.io/PfdLoWYIFFhgHA
GitHub user: ampprojectbot
GitHub app permissions:

Checks: read & write (update/create check-runs on PRs)
Contents: read-only (read commits, branches, PRs, releases, etc)
Issues: read-only (read comments, assignees, labels, etc)
Pull requests: read & write (read PR info, read & write comments)
Commit statuses: read & write (update/create check runs for commits) [deprecated API]
Members: read-only (list members of organization teams)

Webhooks:

Check suite: unused [TODO: verify and unsubscribe]
Pull request: update check
Pull request review comment: unused  [TODO: verify and unsubscribe]
Pull request review: update check
Status: unused  [TODO: verify and unsubscribe]
Label: unused  [TODO: verify and unsubscribe]
Membership: update local team record
Team: update local team record

Personal access token name (from https://github.com/settings/tokens): ampprojectbot (owners bot, maybe other things) (used for admin pages that don't have OAuth from the Probot webhook)
Required permissions:

public_repo: Needed to read PR info, reviews, comments
read: org: Needed to list members of organization teams
repo: status: unknown

Package management: npm
Testing framework: Jest
Test command: npm run test
Tests run on presubmit: Yes
Deployment workflow: Manual (push to cloud repo, reset GCE instance)
Can this be run locally?: Yes (for accessing status/info pages)
If so:

Command to run locally: npm run start
Env. vars required to run locally:

GITHUB_ACCESS_TOKEN: Access token with at least public_repo and read: org permissions
GITHUB_REPO_DIR: Path to local copy of ampproject/amphtml repository (ex. /tmp/amphtml)
GITHUB_REPO: Name of repository (ie. ampproject/amphtml or your fork)

Public environment (ie. config): Shell script, GCE/GAE metadata, deploy command arguments
Private environment (ie. secrets, tokens, keys): Shell script, GCE/GAE metadata, deploy command arguments
App uses a private-key.pem file for authentication: Yes

Project Metrics
Owner(s): rcebulko
Directory: /project-metrics
Deployed on: GAE
Project ID: amp-project-metrics
URL: https://amp-project-metrics.appspot.com
README link: README
Incoming requests:

Metrics display page
Badge API (used by (amphtml README)[https://github.com/ampproject/amphtml/blob/master/README.md])
Metric History API (used by (amphtml METRICS doc)[https://github.com/ampproject/amphtml/blob/master/METRICS.md])
Outgoing requests:
GitHub API
CodeCov API
Travis API

Database: PostgreSQL (CloudSQL)
Authentication: No
Language/Framework: Python/Flask
GitHub:

Personal access token name (from https://github.com/settings/tokens): ampproject/amp-github-apps/project-metrics
Required permissions:

public_repo: Needed to read PR info, commits, releases
Travis API token: required for collecting information on builds
CodeCov API token: required for computing code coverage metric

Package management: pip
Testing framework: unittest
Test command: python -m unittest discover -s . -p '*_test.py'
Tests run on presubmit: No
Deployment workflow: Manual (gcloud app deploy app.yaml)
Can this be run locally?: No (depends on AppEngine-provided imports)
Public environment (ie. config): env.yaml
Private environment (ie. secrets, tokens, keys): env.yaml
App uses a private-key.pem file for authentication: No

Bundle-size Bot
Owner(s): danielrozenberg
Directory: /bundle-size
Deployed on: GAE
Project ID: amp-bundle-size-bot
URL: https://amp-bundle-size-bot.appspot.com
README link: README
Portability: can only work with amphtml
Incoming requests:

GitHub Webhooks
Travis API calls
Outgoing requests:
GitHub API

Database: Yes
Authentication: No
Language/Framework: Probot

Smee channel: ???
GitHub user: ampprojectbot
GitHub app permissions:

Checks: read & write (update/create check-runs on PRs)
Pull requests: read & write (read PR info, read & write comments)
Members: read-only (list members of organization teams) [TODO: verify or remove]

Webhooks:

Check suite: re-run bundle size check when requested
Check run: re-run bundle size check when requested
Pull request: re-run check on new/updated PRs
Pull request review: unknown [TODO: verify or unsubscribe]

Personal access token name (from https://github.com/settings/tokens): None

Package management: npm
Testing framework: Jest
Test command: npm run test
Tests run on presubmit: Yes
Deployment workflow: ???
Can this be run locally?: Yes

Command to run locally: npm run dev
Env. vars required to run locally: ???

Public environment (ie. config): app.yaml
Private environment (ie. secrets, tokens, keys): db-config.js, .envApp uses aprivate-key.pem` file for authentication: Yes

PR Deploy Bot
Owner(s): estherkim
Directory: /pr-deploy
Deployed on: GAE
Project ID: amp-pr-deploy-bot
URL: https://amp-pr-deploy-bot.appspot.com
README link: README
Portability: is repository-agnostic/configurable
Incoming requests:

GitHub Webhooks
Deployed instances (in Cloud Storage bucket)

Outgoing requests:

GitHub API

Database: No
Authentication: No
Language/Framework: TypeScript + Probot

Smee channel: ???
GitHub user: ampprojectbot
GitHub App permissions (see https://github.com/settings/apps/my-app-name/permissions):

Checks: read & write (update/create check-runs on PRs)
Pull requests: read-only (read PR info)

Webhooks:
Check run: re-run deploy when requested
Pull request: re-run deploy on new/updated PRs
Personal access token name (from https://github.com/settings/tokens): None

Package management: npm
Testing framework: Jest
Test command: npm run test
Tests run on presubmit: Yes
Deployment workflow: ???
Can this be run locally?: Yes

Command to run locally: npm run dev
Env. vars required to run locally: ???

Public environment (ie. config): .env
Private environment (ie. secrets, tokens, keys): .env
App uses a private-key.pem file for authentication: Yes
@estherkim Please verify/fill in the ??? blanks if possible

Test Status Bot
Owner(s): danielrozenberg
Directory: /test-status
Deployed on: GAE
Project ID: amp-test-status-bot
URL: https://amp-test-status-bot.appspot.com/
README link: README
Portability: is repository-agnostic/configurable
Incoming requests:

GitHub Webhooks
Travis API calls
Build cop rotation API calls
Web UI (for build cop)

Outgoing requests:

GitHub API

Database: Yes
Authentication: GitHub OAuth
Custom lint config: No
Language/Framework: Probot

Smee channel: ???
GitHub user: ampprojectbot
GitHub App permissions:

Checks: read & write (update/create check-runs on PRs)
Pull requests: read-only (read PR info)

Webhooks:

Pull request: re-run check on new/updated PRs

Personal access token name (from https://github.com/settings/tokens): None

Package management: npm
Testing framework: Jest
Test command: npm run test
Tests run on presubmit: ???
Deployment workflow: Manual (npm run deploy)
Can this be run locally?: [Yes | No]
If so:

Command to run locally: npm run dev
Env. vars required to run locally:

GITHUB_ACCESS_TOKEN: Access token with at least public_repo permission
GITHUB_REPO_DIR: Path to local copy of ampproject/amphtml repository (ex. /tmp/amphtml)
GITHUB_REPO: Name of repository (ie. ampproject/amphtml or your fork)

Public environment (ie. config): [Shell script &| .env &| app.yaml &| GCE/GAE metadata &| deploy command arguments &| config file(s)]
Private environment (ie. secrets, tokens, keys): [Shell script &| .env &| app.yaml &| GCE/GAE metadata &| deploy command arguments &| config file(s)]
App uses a private-key.pem file for authentication: [Yes | No]
Plan
I'm going to start posting for the apps I've created/worked on, and then will start trying to do the same for the other apps with as much info as I can find in the repo.
/cc @ampproject/wg-infra let me know if you think of anything that might be missing. Ideally we can identify some pieces of this to standardize, if not with actual infrastructure then at least with more standard processes for things like keeping track of permissions, which apps use which tokens (since you can't see which token is which in the GitHub UI, making it risky to ever delete any), manage deployments, testing, etc.

Just based on what I've gathered looking through the existing apps and READMEs and environment files and configurations, a few free-floating thoughts in no particular order:

A couple apps have custom ESLint configs; it may be possible to cut some of them down/out so linting is consistent throughout
Use of databases and authentication so far is pretty sparse; probably not worth unifying/creating any shared structure there
Since most of our apps use Probot, there's a common shared set of required environment variables (mostly private keys/secrets), and then a handful of application-specific environment variables; likewise, many apps provide their own instructions on how to create a Probot app, set up a Smee channel, request permissions/webhook subscriptions, etc., and each with varying levels of detail

It may be worth creating separate probot.env and app.env where the former contains the standard keys required by all probot apps (APP_ID, WEBHOOK_SECRET, PRIVATE_KEY, etc.)
It may be worth separating out instructions for creating a GitHub app, setting up a Smee channel, requesting permissions, subscribing to events, and generating the private key into the repository-level README; this will double as a bootstrap guide for those starting new apps

Most apps share similar npm scripts (start, test), and some have dev and deploy, which are extremely useful; it might be good to have a standard set so the build/test/deploy workflow is constant across all (which could make some automation easier)
What is our strategy for managing private keys, tokens, secrets, etc? Where does this info live, besides local copies of .env?
Some apps select the repository to use from an environment variable (which is great!)

The name of the environment variable is different from app to app; it may be worth picking one name/convention and sticking with it
Future apps should, if possible, be designed around the repository as an environment variable
Portable apps should mention that fact in the README, so other interested teams know they can be easily repurposed

The GitHub UI does not show what uses each access token, and you can't view them once they've been created; it's probably worth creating separate access tokens on a per-app basis named after the app (like ampproject/amp-github-apps/project-metrics created by erwinmombay) rather than named after purpose/consumer (like ampproject-bot-process-github-issues or ampprojectbot (owners bot, maybe other things) which is likely shared by multiple apps and may have unnecessary permissions, but can't be safely altered because we don't know)
We should probably try to write new apps in TypeScript, like pr-deploy; this would both make it a bit cleaner (less JSDoc and more valuable type-checking) and would also gradually get us more comfortable in the hybrid JS-TS environment, which is what we may be pushing towards in amphtml
It would be nice for the repository-level README to list each of the apps, a quick description of what it does, and possibly a link to the README

Additional thought: Project Metrics doesn't really fit this repo. It's not a GitHub app, it's a separate Python web app that should probably live in its own repository. If that project were to be moved out, all other projects here are Probot GitHub apps.

Relevant to #410
/cc @ampproject/wg-infra

A couple apps have custom ESLint configs; it may be possible to cut some of them down/out so linting is consistent throughout

This is definitely worth doing. Eslint lets you add an .eslintrc at the root with repo-wide settings, and separate .eslintrc files in each directory that contain rules specific to that directory.

It would be nice for the repository-level README to list each of the apps, a quick description of what it does, and possibly a link to the README

+1!

Project Metrics doesn't really fit this repo. It's not a GitHub app, it's a separate Python web app that should probably live in its own repository. If that project were to be moved out, all other projects here are Probot GitHub apps.

Any ideas for where you'd move it?
Another suggestion for standardization (if it's not already done): Pick one workflow for package management (yarn vs. npm). We use yarn for amphtml because npm would generate different package-lock.json files on different OSs, causing the file to bounce back and forth as people made changes.

@rsimha Re: last item, all of the apps are currently using npm. I think we've been lucky in that we all use Linux

Re: second item, I think project-metrics should probably just live in its own mini-repo? WDYT? Not sure of any obvious other repo that should hold it.

Archiving this.


skysurvey footprint

The footprint is quite restrictive in terms of options. Circle and Rectangles should be allowed. https://github.com/skyportal/skyportal/blob/main/skyportal/handlers/api/instrument.py#L164
Also I think either a single Polygon or a list of Polygons (as required by ZTF) should also be allowed.
if type(verts_or_polygon) == geometry.Polygon: # polygon
    as_polygon = True
    fra, fdec = np.asarray(verts_or_polygon.exterior.xy)
else:
    as_polygon = False
    fra, fdec = np.asarray(verts_or_polygon)

On a similar note, I think PolygonSkyRegion should also be allowed. Getting the vertices is relatively straightforward: https://github.com/skyportal/skyportal/blob/main/skyportal/handlers/api/instrument.py#L1119

I 100% agree.
Working on it.

@mcoughlin This function is part of ztffields.
I'll add the issue there (keeping this one open until solved)

https://github.com/MickaelRigault/ztffields/issues/1


Generate all Mals localizations using a preferred language

Suggestion: Make it so that there is an option to track all changes made in the selected mals files regardless of language, and then on merge, generate a file of a preffered language containing the modified or added entries from the selected mals. Or another way to do it could be to generate every language with the entry of the preffered langauge.

For changelog generation, track all the mod changes regardless of this. When merging, however, generate every Mals localization, using the preferred language if the language is not tracked as a change.
Examples

If a mod only edits USen, generate every Mals archive with the changes from USen.
If a mod edits USen and EUfr, and the preferred language is USfr, generate every Mals archive with the changes from EUfr.

Process

First, use the preferred localization if it exists.
Otherwise, match the language of the preferred localization (over the region)
If there is still no match, use the first archive.


Add Increase/Decrease allowance to prevent sandwich attacks

This note from our security audit was never pushed back to the casper labs erc20
https://github.com/Rengo-Labs/CasperLabs-Crates/blob/6e2e449d11c097df318e6660667dc85c8bc12aae/casper_erc20_new/src/lib.rs#L207
https://github.com/Rengo-Labs/CasperLabs-Crates/blob/6e2e449d11c097df318e6660667dc85c8bc12aae/casper_erc20_new/src/lib.rs#L223
Per QSP-7
https://certificate.quantstamp.com/full/rengo-labs

spoke with David Tai about the sandwich attack thingy
turns out it's "just" a double spend attack their resolution is breaking the API, so no immidiate solution from me, but will be addressed in the "rework"


Improve RI scraper by removing extraneous slash in URL, sparing redirects

Closes #637

Seems abandoned. Closing.


Use production decentralized identifier format

This library is currently returning the decentralized identifier in a legacy format.
We need to update this library and the authentication format to use the current version:
did:stack:v0:${address}-${name_index}

There's work on this in this branch:
https://github.com/blockstack/blockstack.js/tree/feature/update-did
Because of our requirement that Blockstack IDs work before names confirm, @kantai @jcnelson and I decided we needed to switch our production did format to the public key.

Adding a note to help with prioritization. In an effort to embrace true decentralization, meaning interoperability across different platforms, exposing the updated did that resolves using the DIF's universal resolver, would allow a user better ownership of their identity. Right now, users likely don't even know they have an identity that could, in theory, be used outside of Blockstack apps. But they should know. Each app dev can help in revealing that, which Graphite plans to do, but that did should probably be something available outside of any one app.

I have an update on this.  The current stable version of Blockstack Core (v19) has a /v1/dids endpoint that will let you resolve your DID to your public key and profile.  Also, the /v1/names endpoint has been updated to return the name's DID along with the other WHOIS-like information.  I also wrote up the specification for our DID method for the W3C working group here.
The core.blockstack.org node will be updated to v19 shortly.  I also want to make a "blockstack-did.js" library for doing client-side DID resolution at some point.

The core.blockstack.org node will be updated to v19 shortly. I also want to make a "blockstack-did.js" library for doing client-side DID resolution at some point.

I know I'm picking up a tangent here, but I think if we're going to be splitting up our javascript libraries, I think we should do it with a high-level plan like the one discussed here (but actually settle on such a plan):
https://forum.blockstack.org/t/proposal-split-blockstack-js-into-multiple-modules/5749
We should think about splitting up libraries based on who should be responsible for maintaining that particular library. A problem with splitting up libraries too much, is that smaller libraries would increasingly trend towards single-author and same-tester libraries, which leads to a whole slew of other problems --- APIs that don't match their usage, unintended integration dependencies, tests that don't match usage, unclear contribution standards, etc. I'm guilty of this myself, and I think we should be wary of trending in that direction.


[Bug]: The game crash before getting in the main menu (slime rancher 2)

All of the following criteria must be met

[X] All Requirements must be installed.
[X] Changed the title so that it doesn't just says "[Bug]: "
[X] I have searched the GitHub issues for my bug, even in the closed issues.

All of the following are optional to answer

[X] Tried reinstalling the Game.
[X] Tried reinstalling MelonLoader.
[X] Tried restarting PC.
[X] Was able to see the Start Screen.

Describe the issue.
Hello when I open the game, touch a button and try to go into the main menu, the game crash please help.
Latest.log
Did you attach your log file?
Yes, I attached my log file to the text box above.

This is a known issue with some MelonLoader versions on Slime Rancher 2.
Please retest this issue with MelonLoader v0.6.5.


Cannot undo edit from CodeAction

monaco-editor version: 0.17.1
Browser: Chrome/Firefox/Edge Dev
OS: Windows 10
If you register a CodeActionProvider that applies an edit operation, this edit operation is apparently not pushed to the undo stack.

Run the code below in the playground
Hover over the marker
Apply the provided code action
Press Ctrl+Z to undo

Nothing happens. If you edit the text before applying the code action and undo, the edit from before is undone instead of the edit from the code action.

Note that this example hard codes the creation of the marker data, which normally comes directly from a linter such as ESLint. Since the marker data is not recomputed, a small text decoration remains after applying the code action. This does not happen when the marker data is recomputed.

Playground:
const markerRange = new monaco.Range(2, 8, 2, 22);

monaco.languages.registerCodeActionProvider("javascript", {
    provideCodeActions(model, range, context, token) {
        /** @type {monaco.languages.CodeAction[]} */
        const codeActions = [];
        for (const marker of context.markers) {
            /** @type {monaco.languages.TextEdit} */
            const fix = {
                range: markerRange,
                text: '"Hello world!"'
            };
            codeActions.push({
                title: `Fix: ${marker.message}`,
                diagnostics: [marker],
                edit: {
                    edits: [{
                        edits: [fix],
                        resource: model.uri,
                    }],
                },
                kind: "quickfix" // not sure if necessary / what value should be used
            });
        }
        return codeActions;
    }
});

const editor = monaco.editor.create(document.getElementById("container"), {
    value: "function hello() {\n\talert('Hello world!');\n}",
    language: "javascript",
    lightbulb: { enabled: true },
});

monaco.editor.setModelMarkers(editor.getModel(), "ESLint", [{
    message: "Strings must use doublequote.",
    startLineNumber: markerRange.startLineNumber,
    startColumn: markerRange.startColumn,
    endLineNumber: markerRange.endLineNumber,
    endColumn: markerRange.endColumn,
    source: "ESLint",
    severity: monaco.MarkerSeverity.Warning
}]);

Any known workaround for this issue?

@rkistner, if you're still looking for a workaround:
In https://github.com/microsoft/vscode/blob/6ee4811eed9a20487b28996522cdbe664c852dfa/src/vs/editor/standalone/browser/simpleServices.ts#L651 if I replace
applyEdits(edits.map(edit => EditOperation.replaceMove(Range.lift(edit.range), edit.text)));

with
model.pushEditOperations([], edit.map((e) => EditOperation.replaceMove(Range.lift(e.range), e.text)), () => []);
model.pushStackElement();

the undo/redo functionality seems to be working again (tested with monaco-editor-core 0.17.0). I have not thoroughly investigated if this quick fix breaks anything else, but maybe it works for your purposes.
Since this issue is already open since August, I expect it can't be this simple. If @alexandrudima has any insight on this, that would be helpful.

I took a stab at taking your example @gins3000 and getting it upstreamed 👍

Thanks a bunch @orta !


[FLINK-17958][core] Fix MathUtils#divideRoundUp bug for handling zero or negative values.

What is the purpose of the change
This PR fix the problem that MathUtils.divideRoundUp(0, x) returns 1 when x > 1. This problem causes Flink Kubernetes deployment keeps requesting for new pods even there's no pending resource requests.
Brief change log

Update MathUtils#divideRoundUp to properly handle zero and negative values.
Add test cases for MathUtils#divideRoundUp

Verifying this change

Add test cases in MathUtilsTest
Manually verified that problem of improper requesting pod on K8s is fixed

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Thanks a lot the quick fix. I have verified in a K8s cluster and it could fix the constantly allocating taskmanagers issue.


Access-Control-Allow-Origin Problem

I host apprtc & coturn on the same Virtual Server. But i got Access-Control-Allow-Origin Problem when access the room server with chrome 46.

3.029: Initializing; server= undefined.
3.040: Initializing; room=undefined.
getUserMedia() is deprecated on insecure origins, and support will be removed in the future. You should consider switching your application to a secure origin, such as HTTPS. See https://goo.gl/rStTGz for more details.
47656.011: Opening signaling channel.
XMLHttpRequest cannot load http://<IP_ADDRESS>:3478/v1alpha/iceconfig?key=none. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://<IP_ADDRESS>:8080' is therefore not allowed access.
47656.759: ICE server request error: Status=0, response=

How can i solve this problem?
Much appreciate!

OK. I solved this using Reverse Proxy of Nginx


Unclear code in check around supporting -p  and -l flags

Hi there. I recently forked your project. I all looks really good. I am going through your source-code to see if it is something that will be actually useful for my use-case. I found these lines: https://github.com/ubersan/pylic/blob/main/pylic/cli/commands/check.py#L89-L98 which deal with handling the -p and -l arguments. This block is very confusing and not intuitive, with the if not all and then 3 different if clauses with various conditions.
Would you mind explaining the rationale behind this block? It would be greatly appreciated! I can add a comment in the upstream code once you've explained it to me.

Hi @DamianBarabonkovQC
Just quickly glanced at it and indeed, this code block is very hard to understand! Best to refactor or at least comment after we uncovered its use again😄
I will find some time today to check if/what these lines do and then come back to you.
Are you planning to contribute/extend or do you have different use cases in mind that are currently not supported?
cheers!

Thanks! I am currently modifying the project to pull license info from a conda environment directory rather than pip distributions.

okay, this is something that exists locally only? dont know muxh about conda, habe not used it much and not for long time. Is thisnsomething that's required often?

There are two main ways to install a Python package, from PyPI and from a conda repository. Every package installed by conda has a metadata file which includes all of the good stuff, including license -- yes, locally.
Conda is relatively popular. My company uses it over venv since you can install non-Python libraries as well. It's a generic package manager with a strong leaning towards Python.

I see, thanks for the explanation! Would be nice then if we could already here also incorporate the license info in the metadata file. I'd see this as a very valuable extension to the current (very pypi-centric) feature set! Would you consider contributing this?

Possibly, however first we'd need to open-source the code that I am writing, which will take time. Also, for the internal solution, I removed all PyPI support since we do not need it. So contributing would require having a plan on how to support both.

yes we'll need to see the requirements and how this could be integrated. re the code, sorry it was very busy the last few days, I hope to come back to you today

No rush!

I believe that the line of code in question https://github.com/ubersan/pylic/blob/main/pylic/cli/commands/check.py#L89-L98 should be:
if not any(
    [
        unnecessary_ignore_packages,
        bad_unsafe_packages,
        missing_unsafe_packages,
        unsafe_licenses.found,
    ]
):

Notice the any instead of an all. Essentially, as I try to understand it, this code block should only be entered, where the CLI option flags might override a return 1 with a return 0 if something more serious such as bad_unsafe_packages or missing_unsafe_packages does not exist. If any one of those exist, we should return 1 regardless.
The described logic is very much an any operation instead of an all operation.

sooo, finally found an hour to dig into this. First of all, the code is hugely complicated and of course there are bugs. I found a few examples where the return code is wrong. Yes you're wrong, this code should basically just deal with the scenario where there were extra/unnecessary packages or licenses listed, which would normally lead to an error (return code 1), but instead we'll allow it in case the corresponding option is set.
The snippet can be rewritten into something like this:
extra_packages_declared_and_allowed = (
    len(unnecessary_unsafe_packages) > 0 or len(unnecessary_ignore_packages) > 0
) and "allow_extra_packages" in options
if extra_packages_declared_and_allowed and not any(
    [
        bad_unsafe_packages,
        missing_unsafe_packages,
        unsafe_licenses.found,
        unnecessary_safe_licenses,
    ]
):
    console_writer.write_all_licenses_ok()
    return 0

extra_licenses_declared_and_allowed = len(unnecessary_safe_licenses) > 0 and "allow_extra_licenses" in options
if extra_licenses_declared_and_allowed and not any(
    [
        unnecessary_ignore_packages,
        bad_unsafe_packages,
        missing_unsafe_packages,
        unsafe_licenses.found,
        unnecessary_unsafe_packages,
    ]
):
    console_writer.write_all_licenses_ok()
    return 0

if (
    extra_licenses_declared_and_allowed
    and extra_packages_declared_and_allowed
    and not any([bad_unsafe_packages, missing_unsafe_packages, unsafe_licenses.found])
):
    console_writer.write_all_licenses_ok()
    return 0

which already is much easier to understand and it's clearer what is happening. Basically we test if the exception case is applicable and no other un-silenced error is present. This cross-checking makes it a bit harder, as we e.g. only check extra packages and then return 0, but it has to be extra packages and no other error. The original version also tried to accomplish this in a "smart" way, but it fully fails, for the case where extra packages are correctly applicable and ignored, but a bad license is detected, yet it still returns 0:
def test_temp(
    check_command: CheckCommand,
    package: str,
    license: str,
    version: str,
    read_config: MagicMock,
    read_all_installed_licenses_metadata: MagicMock,
) -> None:
    unsafe_packages = [f"{package}", "some-other-package-124"]
    ignore_packages = unsafe_packages
    installed_licenses = [
        {"package": f"{package}", "license": license, "version": version},
        {"package": "some-other-package", "license": "THIS-IS-IGNORED", "version": version},
    ]
    safe_licenses = [license]
    read_config.return_value = AppConfig(safe_licenses, unsafe_packages, ignore_packages)
    read_all_installed_licenses_metadata.return_value = installed_licenses
    return_code = check_command.handle(["allow_extra_packages"])
    assert return_code == 1 # This fails, it returns 0

Switching to not any would mean that it's required that all entries have to be false (i.e. empty lists - also this is quite implicit), not sure if that would improve things.
We have to fix this soon and rewrite this so it's much easier to understand.
Not sure if the extended version above fixes everything, at least the failing test now would pass. But we need to spend a bit more time on this and ensure all edge cases are covered.
Hope this helps for now.
cheers!

Thanks so much for looking into this!
So how I understood unsafe_packages is that it is a "blacklist" of sorts. So if a package is present there, even if its license is deemed safe, it will error. The ignore_packages is a "whitelist" of sorts. So if a package is present there, even if its license is deemed unsafe, it will pass.
In your test case, you write:
unsafe_packages = [f"{package}", "some-other-package-124"]
ignore_packages = unsafe_packages

Which does not make sense in my understanding of the code.
The final version I came up with was:
            # If any of these more serious errors are not encountered, the return may
            # still be `0` depending on the command line options passed
            if not any(
                [
                    bad_unsafe_packages,
                    missing_unsafe_packages,
                    unsafe_licenses.found,
                ]
            ):
                # We are allowing a extra packages and there are extra packages
                extra_packages_declared_and_allowed = (
                    len(unnecessary_unsafe_packages) + len(unnecessary_ignore_packages)
                    > 0
                    and "allow_extra_packages" in options
                )
             .......

This makes sense to me. If no packages are unsafe, have unknown licenses or are unsafe_licenses.found, there is a chance that the return could be 0 if the appropriate exception is applied. Otherwise, if any of those cases is True, always return `.

No problem, thanks for helping to uncover the problems! :sweat_smile:
So unsafe_packages indeed could mean what you think, but it actually means: (see the readme) "If you rely on a package that does not come with a license you have to explicitly list it as such.". Perhaps a better name might be unlicensed_packages; that would most probably be easier to understand.
If a package with a license is marked as unsafe_package, then this will result in an error (-> will be in the list of bad_unsafe_packages).
The problem with not any If I understand your proposal correctly, is that then all of the list entries have to map to False, ie. be non-empty lists. Example with taking only two elements:
e1 | e2 | any | not any
T  | T  |  T  |   F
F  | T  |  T  |   F
F  | F  |  F  |   T

with not all also the mixed/cross cases are included
e1 | e2 | all | not all
T  | T  |  T  |   F
F  | T  |  F  |   T
F  | F  |  F  |   T

So in the not all case only if there are no problems, the extra checks would be skipped. But then of course all the possible combinations have to be checked.
Does this make sense to you?

Thanks for the detailed answer once again.
To be entirely honest, I never understood the README when it said "If you rely on a package that does not come with a license you have to explicitly list it as such." regarding the unsafe packages. Regardless, it appeared to me that when a package was listed there and was installed, it would error if it was encountered. Therefore I treated that as a blacklist of packages. Please elaborate on the case that that is not true/correct?
Otherwise, let's assume my understanding where unsafe -> blacklist, ignore -> whitelist.
We have 3 conditions in the if statement in my code (one less than you because I made unnecessary_ignore_packages silent if allow_extra_packages is true).
bad_unsafe_packages,
missing_unsafe_packages,
unsafe_licenses.found,

My goal, as described in my comment, is that these conditions are considered "serious" errors -- when there is an unsafe package detected, when a package has unknown as a license and when a package exists with an unsafe license. There is no case where if at least one of them is non-empty that the program should return 0. It should always return 1 then.
Now, referring to your truth table for not any.
e1 | e2 | any | not any
T  | T  |  T  |   F
F  | T  |  T  |   F
F  | F  |  F  |   T

I only want to enter the if statement and potentially return 0 if a correct option flag was passed. Well then not any is perfect because it is T only if all of the values are F. As soon as only one of them is T, then it is F and the program will not enter the if statement and error. That is exactly the behavior I described above that I want.
For not all, that simply is not the logic:
e1 | e2 | all | not all
T  | T  |  T  |   F
F  | T  |  F  |   T
F  | F  |  F  |   T

As a matter of fact, when say only one of the errors bad_unsafe_packages, missing_unsafe_packages and unsafe_licenses.found is failing, the if statement will still be entered and potentially return 0. This is NOT what I want. If one of these errors is failing, I want it to always return 1.
Do you follow now the logic for why I believe not any is the correct choice, not not all.

To be entirely honest, I never understood the README

That's so great to know! We need to improve the API and explanations in the readme and cli helpers!
The reasoning behind the unsafe_packages feature is that if a package is installed that does not provide a license (which can happen on pypi, but I guess is much more common in private/company package indexes), it has to be explicitly marked as such.

Otherwise, let's assume my understanding where unsafe -> blacklist, ignore -> whitelist.

The setup with the unsafe/safe packages/licenses should be such, that there is no need for an explicit blacklist, because any license and or package (without license) is automatically on the blacklist, as anything which is not on one of the whitelists and installed will lead to an error.

that there is no need for an explicit blacklist

What if a Python package has an incorrect license. The PyPI info was not updated when the LICENSE was changed. We'd need to blacklist this specific package, even though the program thinks that its license is "safe".

The reasoning behind the unsafe_packages feature is that if a package is installed that does not provide a license (which can happen on pypi, but I guess is much more common in private/company package indexes), it has to be explicitly marked as such.

How is the behavior then different when it is marked as unsafe vs when it is not. Both times, it errors, no?

that there is no need for an explicit blacklist

What if a Python package has an incorrect license? The PyPI info was not updated when the LICENSE was changed. We'd need to blacklist this specific package, even though the program thinks that its license is "safe".

Do you mean if the e.g. pypi license classifiers or metadata are different to what LICENSE is actually checked into the repo? So far we trust the license data we get.

The reasoning behind the unsafe_packages feature is that if a package is installed that does not provide a license (which can happen on pypi, but I guess is much more common in private/company package indexes), it has to be explicitly marked as such.

How is the behavior then different when it is marked as unsafe vs when it is not. Both times, it errors, no?

If you install a-cool-private-repo-package that does not ship with a license then:

no marking the package as unsafe: pylic check fails with an error ("Found unsafe packages: ...")
marking the package as unsafe: pylic check succeeds without an error or warning. The reasoning is that the developer acknowledges that the package is safe to use - despite not providing a license.

An example of such an unsafe package is e.g. (pyNakadi)[https://pypi.org/project/pyNakadi/]. It has a checked in MIT license, but does not expose it via classifier or metadata.
pylic check then returns:
Found unsafe packages:
  pyNakadi (<IP_ADDRESS>)

I understand finally your whole point regarding unsafe_packages. I don't think that the behavior of what we want is exactly achieved by your definition of unsafe. In my code, I feel like retaining unsafe -> blacklist (we cannot trust the license info in the metadata). ignore -> whitelist.
What are your thoughts on the not any vs not all case in my longer post above? Regardless of how you define unsafe, I think my comments still apply.

sorry busy days ...
about your longer post above and the not any logic. I think I do understand what you mean. I think we also do talk about the same thing - at least we both know that the current logic is flawed :slightly_smiling_face:

My goal, as described in my comment, is that these conditions are considered "serious" errors -- when there is an unsafe package detected, when a package has unknown as a license and when a package exists with an unsafe license. There is no case where if at least one of them is non-empty that the program should return 0. It should always return 1 then.

Yes, but just bare in mind that we're already in the error/return code 1 branch and we only want to exceptionally instead return 0.

My goal, as described in my comment, is that these conditions are considered "serious" errors -- when there is an unsafe package detected, when a package has unknown as a license and when a package exists with an unsafe license. There is no case where if at least one of them is non-empty that the program should return 0. It should always return 1 then.

I agree, but wouldn't this then imply any, i.e. if any(list of all the potential serious problems):

I only want to enter the if statement and potentially return 0 if a correct option flag was passed.

Yes that's what I tried here

I only want to enter the if statement and potentially return 0 if a correct option flag was passed. Well then not any is perfect because it is T only if all of the values are F. As soon as only one of them is T, then it is F and the program will not enter the if statement and error. That is exactly the behavior I described above that I want.

Ah okay I think I now understand what you mean. So you want something along the lines of
if not any([packages_are_problem, licenses_are_a_problem, ...]):
  return 0

? Not sure if this would be understandable. Also I think you'd have to wrap this in if (package_option_given or license_option_given): and also make sure that other problems such as unnecessary_ignore_packages are not still failing the check.
I think the easiest would be to have this (same as in the code linked above):
if any([...]):
  ...

  if package option given and only packages are wrong:
    return 0

 if license option given and only licenses are wrong:
    return 0

 if package and license options are given and only packages and licenses are wrong:
    return 0

  return 1

I think this would cover all the cases and is easier to understand and read compared to these rather unusual boolean transformations.
wdyt?

Hey Sandro. Unrelated to the topic, I am letting you know that I am on holiday for 2 weeks. So I won't be able to run stuff until I get back. Hope that is fine.


SUPPORT PROD : Enhancement in Delay Condonation

Describe the bug
If the cause of action is 5th of Nov , the system should consider from next day i.e 6th of nov to 5th nov (It should include the last date as well.
Program team have already provided the sign off .
In case any doubt reach out to Abhishek

Hi @Susmitabe , Tested in UAT env, working as expected,
Attaching jam video for reference:
https://jam.dev/c/a8ec4b50-e801-4f41-bbca-ca365e581f97
cc: @rajeshcherukumalli


When logging in via SSH, it automatically elevates privileges to the Administrator account

Windows Terminal version
1.18.1462.0
Windows build number
Win32NT             10.0.22621.0 Microsoft Windows NT 10.0.22621.0
Other Software
Both the local and remote computers are running Windows 11, and they are using the same version of Windows Terminal Preview. The PowerShell and OpenSSH versions are also the same.
OpenSSH_for_Windows_8.6p1, LibreSSL 3.4.3
powershell 7.3.7
theme https://starship.rs

Steps to reproduce

Both the local and remote computers are logged in using the same Microsoft account.
When I connect to the remote computer using OpenSSH, it will be elevated to the Administrator privilege.
I use the following command to check if I have elevated privileges:

([Security.Principal.WindowsPrincipal] [Security.Principal.WindowsIdentity]::GetCurrent()).IsInRole([Security.Principal.WindowsBuiltInRole]::Administrator)

Expected Behavior

Actual Behavior
The command should not be elevated.

Unfortunately we cannot do anything about this, as this is an issue with OpenSSH. It has been reported here and subsequently been closed as "by design": https://github.com/PowerShell/Win32-OpenSSH/issues/1652
I suspect this might change once #146 is resolved.


Upgrade dependencies to allow React 16.0.0

https://reactjs.org/blog/2017/09/26/react-v16.0.html

Thanks! :-)


feat (networking): [akv] update Key Vault to publicNetworkAccess = disabled

closing this feat for now as this will incur in major changes from docs to get the cluster deployed.


Milestone Tag is inconsistent with kubernetes/kubernetes

In this repo, we are using milestone tags formatted as "1.9".  However, in kubernetes/kubernetes, milestone tags are formatted "v1.9".  This breaks cross-repo searches.  We should fix to match kubernetes/kubernetes.

I renamed the milestones here to add the "v". Good catch!


No docs on how to actually use History

From Query instances its detailed that the history can be returned. However, the history object is a JArray. There's no details on how we should process this correctly. It also makes no sense to return a JArray when the history is derived from a table with a schema. If its due to it 'changing' in the future, then that's even worse, our code will just break!. Either way, an example on how we're supposed to process this would be helpful.

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

ID: 6c17b738-aa32-4cbb-3d82-bda2bc032487
Version Independent ID: a34e246c-7c65-461f-4417-3c47eb4672e2
Content: Manage instances in Durable Functions - Azure
Content Source: articles/azure-functions/durable/durable-functions-instance-management.md
Service: azure-functions
GitHub Login: @cgillum
Microsoft Alias: azfuncdf

@cgillum Appreciate your inputs in this.

@dotnetprofessional I have assigned this to content author for further review and update as appropriate.


Watching on resource returns update without updating status

This might be some internal Kubernetes api but making the change on the resource kicks in watch - that brings error from resource before it is being updated. Technically we could reset all states to unknown at the beginning of the processing but this will still not reduce flicker. Logging for future consideration

Resource exist
We update it to get update
Watches gets notified by update
Operator updates status
Second update

Can we avoid 3.

I've noticed this as well, I did a little investigation and it seems like it is some weird internal thing. We'll take it up after summit

@wtrocki I believe this was fixed by changing the refresh property in the console?


R200 fails with librealsense after firmware upgrade

RS200 / Windows 10 / VS2013
I've been having a few problems getting the R200 to work properly with LibRealsense. Even in the examples the camera would sometimes fail, throwing exceptions in the lib and requiring to be replugged to make it work again. It generally works, the CPPCapture sample shows the colour, depth and infrared images, but it doesn't feel stable at all. The same problems occur in the integration I've done with our software.
So I installed the latest DCM - Build Number: <IP_ADDRESS>76 - and let it upgrade the firmware. And then the problems really started.. in LibRealsense it either doesn't recognise the camera at all, recognises the streams but hangs on the first attempt to get a buffer, or (best of all) bluescreens the PC when it tries to get a buffer.
Worse, it seems like that upgrade is a one way street. I can't find any way to uninstall the DCM and roll back to an older one, or any way to get back to the state I had before when the camera at least did something.
Any advice much appreciated. We're integrating the cameras with our software (to go alongside Kinect1 and Kinect2) - but this is currently looking like a dead end.

I don't think it's a problem with the firmware. I also updated it yesterday with DCM (build <IP_ADDRESS>76). I tested some of the example apps (e.g. the cpp-capture app) on both Windows 8.1 with VS15 (Community) and on Ubuntu 14.04 and had very few crashes, I don't remember actually if there were any crashes at all. And the camera is always properly recognized.
The only example that I found not to be too stable yet is the cpp-config-ui program which always crashes on both operating systems when I try to change the R200_AUTO_EXPOSURE_MEAN_INTENSITY_SET_POINT, I will try to have a closer look at what's actually the problem there. The RealSense SDK also works fine most of the time.
Could you already test your camera on another computer to figure out if it's rather a problem with the computer or with the camera?

If it was a problem with either the computer or the camera, why did it only break after I installed the new DCM?

Hi @mattswoboda -- sorry for all your troubles! In general, debugging our cameras comes down to 2 or 3 things. Very rarely it's a problem with the software in this repo, and even more rarely is it a H/W issue.

Make sure it's on a compatible system (where compatibility is a function of supported USB host controller chipset that provides enough power)
Driver installation issues (In 99% of cases this means Ubuntu/Linux)
Firmware. We don't track DCM version number (... and that ecosystem is largely a disaster because of all the reasons you mention related to the installation/upgrade process).

Please shoot me an email<EMAIL_ADDRESS>and I'll send you the direct F/W update binary for Windows. That'll be a good place to start. There definitely should be a mechanism to uninstall the DCM, so definitely do that as well!

@ddiakopoulos : Mail sent - thanks!

Got it. Will follow up soon... need to find the installer binary.

What became of this issue?  Was the problem resolved?

Not yet - I still need the firmware file to flash the device. Any help gratefully received.

Hello,
If the issue is software related, we are no longer updating this software as we have moved to our latest generation, the D400 series. If you are in your warranty, a replacement can be provided.
Jesus G.
Intel Customer Support


Unable to save printer detalis

Hello,
I try to edit a printer detalis (I need to chagne its IP address) and I can't click on the "Save" button - it's grayed out.
Under hostname section, it says "Enter a valid URL", altough the URL is correct.

I tried to delete the app and reinstalled, didn't help.

Hmm, I wonder why it says that URL is not valid. Does it have an empty space at the end? I will try it locally and see if I can reproduce it

ok. I was able to reproduce it by doing the following:

Enter an invalid URL
Click on Scan and select an octoprint instance
Even though URL is completed .... message of invalid URL remains

If you tap on the URL and then tap on the printer name or API key or any other field it will remove the invalid URL message and will enable the button. I will fix this for the next release.
Thanks for reporting this issue.
Gaston

Hello,
Did this issue got fixed? I’m still having this bug on v3.23 (build 1).


Player Attacks

Expanding upon the different moves for the 3 instruments that the player can choose from at the beginning.
Document: https://docs.google.com/document/d/1M9rU9qWbARQ4KiU2RGAFJp31dVz8iWQoDYE-EPu7LRo/edit?usp=sharing

Created Drumroll move effect
Timer lasts for 5 seconds giving player opportunity to press space bar until time runs up
Damage is multiplied by space bar presses
Had to change project settings to allow both new and old input systems (will fix later when I have time to learn how to use new input system)
TODO: more moves
TODO: connect them to the musician

completed all Drummer moves


chore: decouple amplify cli core from grapqhl core

Description of changes
This PR:

bumps category api dependencies
pins versions (no carat) due to wrong resolutions to cdkv2 tag otherwise.
removes transformer core from amplify cli core dependency list

Issue #, if available

Description of how you validated changes
Checklist

[x] PR description included
[x] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

schema-iterative-update-4 requires change from dev to bump timeout.
schema-auth-5b verified manually that it's passing in us-west-2


It annoyed me that there is no support for the new-ish (3 years now?) Windows Unix sockets in libuv. I have to admit that I don't use Windows much - I kind of hate it and everything about it's API - but that's more or less exactly why I hoped there would be support for Unix sockets. It's nice to be able to simply use the same code, no changes, etc. Why else would Microsoft add them, after all? (IIRC, they were added to stable Windows in 2020, so it's also possible that the devs were at home with nothing better to do; see also their new pre-processor of that year).
Actually adding them properly would take some major changes that would probably break everything, not that I haven't tried (see paragraph 1). But there's one simple thing that I think wouldn't break anything. As mentioned in the title, that's to add the GUID for Unix sockets to the short list in win/poll.c so the "fast poll" mechanism will work. It therefore must be opened manually beforehand and configured as a uv_poll_t. Previously this did work but defaulted to using select.
I don't have any tests for this at the moment. I'm not sure whether there's even the slightest interest in doing this, so I'll just make this basic request for now. I can, however, offer the supremely useless vote of confidence in saying "it worked for me".
Note: I got the GUID value by debugging the libuv code that compares them.
Note 2: I wish the official style used uppercase hexadecimal (with lowercase 0x). Oh well.

Thanks @roflcopter4!

Just fyi, this somehow broke ABI on 32bit Windows, making cmake and automoc crash. Rebuilding against a newer libuv fixes things though, so likely this can be ignored at this point.
See https://github.com/msys2/MINGW-packages/pull/13364#issuecomment-1264585152

Oh, hum. That's regrettable. I'm really not sure how this PR could cause ABI breakage but I see you've bisected it to this PR. I assume reverting would break ABI again?

I see it, and can look closer tomorrow

Apparently what we do here is a bunch of complicated nonsense, where it makes several poll handles, when only one is needed. This seems to have been fixed in the wepoll code (which was originally derived from the libuv code), but not here: https://github.com/piscisaureus/wepoll/commit/dc895c6decbadf6b9626b829304365ba802f3a35

I thought the code looked pretty inscrutable, but I figured that either I must be too stupid to understand (usually a good assumption) or it's just Windows being Windows (also a good assumption).


Support (and default to) parenthesis for block expressions

This changes allow to use parenthesis ( ) to encapsulate
expressions in addition to braces { } used to define blocks.
The main use-case is for arithmetic and boolean expressions for which
developers are used to using parenthesis. For example:
{ 14 + 42 } * 1337

can now be written as:
( 14 + 42 ) * 1337

This may sound straightforward at first but wasn't necessarily trivial
in Aiken given that (a) everything is an expression, (b) whitespaces
do not generally matter and (c) there's no symbol indicating the end
of a 'statement' (because there's no statement).
Thus, we have to properly disambiguate between:
let foo = bar(14 + 42)

and
let foo = bar
(14 + 42)

Before this commit, the latter would be interpreted as a function call
and would lead to a somewhat puzzling error. Now, the newline serves
as a delimiting symbol. The trade-off being that for a function call,
the left parenthesis has to be on the same line as the function name
identifier -- which is a fair trade off. So this is still allowed:
let foo = bar(
  14 + 42
)

As there's very little ambiguity about it.
This fixes #236 and would seemingly allow us to get rid of the leading
# in front of tuples.

Very clever


How to use pyglet with asyncio

Specifically, how to use it with websockets

Hi,
pyglet is not an asyncio library, and contains it's own event loop. The two are not directly compatible. That said, it's possible to integrate with asyncio code using threads and events, but it's not something we support.
You might want to try a Sans-IO style library. Maybe this would work for you (I haven't tried it myself) https://github.com/python-hyper/wsproto

Closing due to lack of response, but please reopen the ticket if you still have questions.


bitcoin_flutter is incompatible with bip39

dependencies:
flutter:
sdk: flutter
web3dart: ^2.3.1
bip39: ^1.0.6
bitcoin_flutter: ^2.0.2
Running "flutter pub get" in t_wallet...
Because every version of bitcoin_flutter depends on hex ^0.1.2 and bip39 1.0.6 depends on hex ^0.2.0, bitcoin_flutter is incompatible with bip39 1.0.6.
And because no versions of bip39 match >1.0.6 <2.0.0, bitcoin_flutter is incompatible with bip39 ^1.0.6.
So, because t_wallet depends on both bip39 ^1.0.6 and bitcoin_flutter ^2.0.2, version solving failed.
pub get failed (1; So, because t_wallet depends on both bip39 ^1.0.6 and bitcoin_flutter ^2.0.2, version solving failed.)
Process finished with exit code 1

Also have this problem. Please, fix this.

I cloned this repository into the root of my project. In pubspec.yaml, I import the package from the local path.
bitcoin_flutter:
    path: ./bitcoin_flutter

Ditto.   Need to upgrade to hex ^0.2.0


ConsenSys teku

Question
I am running ConsenSys teku it fails to sync throwing the error bellow. Is it related to the library or teku/I am asking here since it says to report the bug here. Sorry if i am at the wrong address
Context
otna@ethValidator:~/teku/teku-22.9.0$ sudo docker run --network="host" -p 9000:9000/tcp -p 9000:9000/udp -p 30304:30303/tcp -p 30304:30303/udp -p 5051:5051 --user 1000:1000 --mount type=bind,source=/home/otna/.local/share/teku,target=/var/lib/teku consensys/teku:22.9.0-jdk16 --data-base-path=/var/lib/teku --ee-endpoint=http://localhost:8551 --ee-jwt-secret-file=/var/lib/teku/token.txt
WARNING: Published ports are discarded when using host network mode
2022-09-15 21:29:00,804 main INFO Configuring logging for destination: console and file
2022-09-15 21:29:00,809 main INFO Logging file location: /var/lib/teku/logs/teku.log
2022-09-15 21:29:00,809 main INFO Logging includes events: true
2022-09-15 21:29:00,809 main INFO Logging includes validator duties: true
2022-09-15 21:29:00,809 main INFO Logging includes color: true
2022-09-15 21:29:00,863 main INFO Include P2P warnings set to: false
21:29:01.224 INFO  - Teku version: teku/v22.9.0/linux-x86_64/-eclipsefoundation-openjdk64bitservervm-java-16
21:29:01.227 INFO  - This software is licensed under the Apache License, Version 2.0 (the "License"); you may not use this software except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
21:29:01.420 INFO  - Storing beacon chain data in: /var/lib/teku/beacon
21:29:01.422 INFO  - Loading 0 validator keys...
21:29:01.424 INFO  - Loaded 0 Validators:
21:29:01.444 INFO  - Using optimized BLST library
21:29:01.565 INFO  - Initializing storage
21:29:02.196 INFO  - Storage initialization complete
21:29:03.048 INFO  - Loading generated p2p private key from: generated-node-key
21:29:03.270 INFO  - PreGenesis Local ENR: enr:-Iu4QER380mIeK6bmZYIRDYblM1Z7SGcwBWdw8LRN0MGfvIdaTTu7bqnmMU9-eLGcAEuLgebBAMiqcQCZK8R_-NwfABZhGV0aDKQ9aX9QgAAAAD__________4JpZIJ2NIlzZWNwMjU2azGhAsYIJXwNlu4HItcK4CyvxKwNVXfDOn8u3MB1bxFj7yim
21:29:03.434 INFO  - Listening for connections on: /ip4/<IP_ADDRESS>/tcp/9000/p2p/16Uiu2HAm8khufnK6dQLfuN1uye4rf4ShTd8NcvdnP2uCZJm5amnV
21:29:03.472 INFO  - Syncing     *** Target slot: 4704443, Head slot: 296625, Remaining slots: 4407818, Connected peers: 0
21:29:03.506 INFO  - Local ENR: enr:-Iu4QPo5uKkNi53vLw3rx8MDbuZfcNVHdliWjH8UWipf56E_JPUUYCdDlaZy_XKNqgzrLwSx5Hzknn6TWHbKYax1nIxahGV0aDKQSibFiwIAAAD__________4JpZIJ2NIlzZWNwMjU2azGhAsYIJXwNlu4HItcK4CyvxKwNVXfDOn8u3MB1bxFj7yim
21:29:03.543 INFO  - Loading deposits up to Eth1 block 0
21:29:11.459 INFO  - Syncing     *** Target slot: 4704444, Head slot: 296625, Remaining slots: 4407819, Connected peers: 0
21:29:23.458 INFO  - Syncing     *** Target slot: 4704445, Head slot: 296625, Remaining slots: 4407820, Connected peers: 0
21:29:35.454 INFO  - Syncing     *** Target slot: 4704446, Head slot: 296625, Remaining slots: 4407821, Connected peers: 0
21:29:46.219 INFO  - Syncing started
21:29:47.454 INFO  - Syncing     *** Target slot: 4704447, Head slot: 296625, Remaining slots: 4407822, Connected peers: 1
21:29:47.954 INFO  - Epoch Event *** Epoch: 147014, Justified checkpoint: 9268, Finalized checkpoint: 9267, Finalized root: 744151ad281ee9b3690c69524fff49d53562111892bf8fb5cd21347945297da4
21:29:59.507 INFO  - Syncing     *** Target slot: 4704448, Head slot: 296922, Remaining slots: 4407526, Connected peers: 1
21:30:03.482 INFO  - Execution Client is online
21:30:11.486 INFO  - Syncing     *** Target slot: 4704449, Head slot: 296944, Remaining slots: 4407505, Connected peers: 1
21:30:23.494 INFO  - Syncing     *** Target slot: 4704450, Head slot: 297214, Remaining slots: 4407236, Connected peers: 4
21:30:35.578 INFO  - Syncing     *** Target slot: 4704451, Head slot: 297591, Remaining slots: 4406860, Connected peers: 4
21:30:47.493 INFO  - Syncing     *** Target slot: 4704452, Head slot: 297972, Remaining slots: 4406480, Connected peers: 2
21:30:59.522 INFO  - Syncing     *** Target slot: 4704453, Head slot: 298094, Remaining slots: 4406359, Connected peers: 2
21:31:11.454 INFO  - Syncing     *** Target slot: 4704454, Head slot: 298094, Remaining slots: 4406360, Connected peers: 2
21:31:23.508 INFO  - Syncing     *** Target slot: 4704455, Head slot: 298303, Remaining slots: 4406152, Connected peers: 5
21:31:35.504 INFO  - Syncing     *** Target slot: 4704456, Head slot: 298494, Remaining slots: 4405962, Connected peers: 4
21:31:47.501 INFO  - Syncing     *** Target slot: 4704457, Head slot: 298534, Remaining slots: 4405923, Connected peers: 2
21:31:59.508 INFO  - Syncing     *** Target slot: 4704458, Head slot: 298909, Remaining slots: 4405549, Connected peers: 2
21:32:11.492 INFO  - Syncing     *** Target slot: 4704459, Head slot: 298944, Remaining slots: 4405515, Connected peers: 7
21:32:23.511 INFO  - Syncing     *** Target slot: 4704460, Head slot: 299136, Remaining slots: 4405324, Connected peers: 7
21:32:35.494 INFO  - Syncing     *** Target slot: 4704461, Head slot: 299520, Remaining slots: 4404941, Connected peers: 10
21:32:47.495 INFO  - Syncing     *** Target slot: 4704462, Head slot: 299906, Remaining slots: 4404556, Connected peers: 9
21:32:59.501 INFO  - Syncing     *** Target slot: 4704463, Head slot: 300274, Remaining slots: 4404189, Connected peers: 9

A fatal error has been detected by the Java Runtime Environment:

SIGSEGV (0xb) at pc=0x00007f27a55e9ce0, pid=1, tid=50

JRE version: OpenJDK Runtime Environment Temurin-16.0.2+7 (16.0.2+7) (build 16.0.2+7)
Java VM: OpenJDK 64-Bit Server VM Temurin-16.0.2+7 (16.0.2+7, mixed mode, tiered, compressed oops, compressed class ptrs, g1 gc, linux-amd64)
Problematic frame:
V  [libjvm.so+0x76dce0]  G1ParScanThreadState::trim_queue_to_threshold(unsigned int)+0x1a0

Core dump will be written. Default location: Core dumps may be processed with "/usr/share/apport/apport -p%p -s%s -c%c -d%d -P%P -u%u -g%g -- %E" (or dumping to /opt/teku/core.1)

An error report file with more information is saved as:
/opt/teku/hs_err_pid1.log

If you would like to submit a bug report, please visit:
https://github.com/adoptium/adoptium-support/issues

[error occurred during error reporting (), id 0xb, SIGSEGV (0xb) at pc=0x00007f27a64a2941]

Your operating system and platform:

@AntoBrajkovic Java 16 is no longer supported, you can try our latest <IP_ADDRESS>

tried it, still fails
context
otna@ethValidator:~$ sudo docker run --network="host" -p 9000:9000/tcp -p 9000:9000/udp -p 30304:30303/tcp -p 30304:30303/udp -p 5051:5051 --user 1000:1000 --mount type=bind,source=/home/otna/.local/share/teku,target=/var/lib/teku consensys/teku:latest --data-base-path=/var/lib/teku --ee-endpoint=http://localhost:8551 --ee-jwt-secret-file=/var/lib/teku/token.txt
[sudo] password for otna:
WARNING: Published ports are discarded when using host network mode
2022-09-16 13:18:01,734 main INFO Configuring logging for destination: console and file
2022-09-16 13:18:01,744 main INFO Logging file location: /var/lib/teku/logs/teku.log
2022-09-16 13:18:01,744 main INFO Logging includes events: true
2022-09-16 13:18:01,744 main INFO Logging includes validator duties: true
2022-09-16 13:18:01,745 main INFO Logging includes color: true
2022-09-16 13:18:01,804 main INFO Include P2P warnings set to: false
13:18:02.162 INFO  - Teku version: teku/v22.9.0/linux-x86_64/-eclipseadoptium-openjdk64bitservervm-java-17
13:18:02.167 INFO  - This software is licensed under the Apache License, Version 2.0 (the "License"); you may not use this software except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
13:18:02.361 INFO  - Storing beacon chain data in: /var/lib/teku/beacon
13:18:02.362 INFO  - Loading 0 validator keys...
13:18:02.365 INFO  - Loaded 0 Validators:
13:18:02.387 INFO  - Using optimized BLST library
13:18:02.677 INFO  - Initializing storage
13:18:03.413 INFO  - Storage initialization complete
13:18:04.326 INFO  - Loading generated p2p private key from: generated-node-key
13:18:04.515 INFO  - PreGenesis Local ENR: enr:-Iu4QG0rfdTWX9nSZNdE7hjGyY_GdVTi6uJQucVfJsE3Ix_zTzwvvkfH2ir830zhXZed1O8S1eHXM-yxUc7a1kSmvmtnhGV0aDKQ9aX9QgAAAAD__________4JpZIJ2NIlzZWNwMjU2azGhAsYIJXwNlu4HItcK4CyvxKwNVXfDOn8u3MB1bxFj7yim
13:18:04.670 INFO  - Listening for connections on: /ip4/<IP_ADDRESS>/tcp/9000/p2p/16Uiu2HAm8khufnK6dQLfuN1uye4rf4ShTd8NcvdnP2uCZJm5amnV
13:18:04.702 INFO  - Syncing     *** Target slot: 4709188, Head slot: 369687, Remaining slots: 4339501, Connected peers: 0
13:18:04.741 INFO  - Local ENR: enr:-Iu4QEA1nigA4mRPV5HoTz2-_RtJgEw3ImbLJ7UGtRlcD913XzwlVDr9ljQ0HBehiqPjCd1p93MwMVgk0aZo8NXJTr1ohGV0aDKQSibFiwIAAAD__________4JpZIJ2NIlzZWNwMjU2azGhAsYIJXwNlu4HItcK4CyvxKwNVXfDOn8u3MB1bxFj7yim
13:18:04.925 INFO  - Loading deposits up to Eth1 block 0
13:18:11.192 INFO  - Syncing     *** Target slot: 4709189, Head slot: 369687, Remaining slots: 4339502, Connected peers: 0
13:18:23.189 INFO  - Syncing     *** Target slot: 4709190, Head slot: 369687, Remaining slots: 4339503, Connected peers: 0
13:18:35.186 INFO  - Syncing     *** Target slot: 4709191, Head slot: 369687, Remaining slots: 4339504, Connected peers: 0
13:18:47.060 INFO  - Syncing started
13:18:47.186 INFO  - Syncing     *** Target slot: 4709192, Head slot: 369687, Remaining slots: 4339505, Connected peers: 1
13:18:59.231 INFO  - Syncing     *** Target slot: 4709193, Head slot: 369950, Remaining slots: 4339243, Connected peers: 4
13:19:04.696 INFO  - Execution Client is online
13:19:11.237 INFO  - Syncing     *** Target slot: 4709194, Head slot: 370259, Remaining slots: 4338935, Connected peers: 3
13:19:23.230 INFO  - Syncing     *** Target slot: 4709195, Head slot: 370559, Remaining slots: 4338636, Connected peers: 3
13:19:35.228 INFO  - Syncing     *** Target slot: 4709196, Head slot: 370840, Remaining slots: 4338356, Connected peers: 5
13:19:47.232 INFO  - Syncing     *** Target slot: 4709197, Head slot: 371032, Remaining slots: 4338165, Connected peers: 5
13:19:59.185 INFO  - Syncing     *** Target slot: 4709198, Head slot: 371032, Remaining slots: 4338166, Connected peers: 4
13:20:11.236 INFO  - Syncing     *** Target slot: 4709199, Head slot: 371190, Remaining slots: 4338009, Connected peers: 3
13:20:23.235 INFO  - Syncing     *** Target slot: 4709200, Head slot: 371515, Remaining slots: 4337685, Connected peers: 2
13:20:35.228 INFO  - Syncing     *** Target slot: 4709201, Head slot: 371839, Remaining slots: 4337362, Connected peers: 5
13:20:47.238 INFO  - Syncing     *** Target slot: 4709202, Head slot: 372162, Remaining slots: 4337040, Connected peers: 6
13:20:59.260 INFO  - Syncing     *** Target slot: 4709203, Head slot: 372332, Remaining slots: 4336871, Connected peers: 5
13:21:11.230 INFO  - Syncing     *** Target slot: 4709204, Head slot: 372447, Remaining slots: 4336757, Connected peers: 4
13:21:23.394 INFO  - Syncing     *** Target slot: 4709205, Head slot: 372768, Remaining slots: 4336437, Connected peers: 4
13:21:35.234 INFO  - Syncing     *** Target slot: 4709206, Head slot: 373087, Remaining slots: 4336119, Connected peers: 5
13:21:47.258 INFO  - Syncing     *** Target slot: 4709207, Head slot: 373408, Remaining slots: 4335799, Connected peers: 7
13:21:59.227 INFO  - Syncing     *** Target slot: 4709208, Head slot: 373728, Remaining slots: 4335480, Connected peers: 7
13:22:11.228 INFO  - Syncing     *** Target slot: 4709209, Head slot: 374056, Remaining slots: 4335153, Connected peers: 6
13:22:23.229 INFO  - Syncing     *** Target slot: 4709210, Head slot: 374370, Remaining slots: 4334840, Connected peers: 5
13:22:35.237 INFO  - Syncing     *** Target slot: 4709211, Head slot: 374695, Remaining slots: 4334516, Connected peers: 9
13:22:47.230 INFO  - Syncing     *** Target slot: 4709212, Head slot: 375020, Remaining slots: 4334192, Connected peers: 8
13:22:59.230 INFO  - Syncing     *** Target slot: 4709213, Head slot: 375331, Remaining slots: 4333882, Connected peers: 9
13:23:11.232 INFO  - Syncing     *** Target slot: 4709214, Head slot: 375669, Remaining slots: 4333545, Connected peers: 9
13:23:23.228 INFO  - Syncing     *** Target slot: 4709215, Head slot: 375985, Remaining slots: 4333230, Connected peers: 8
13:23:35.230 INFO  - Syncing     *** Target slot: 4709216, Head slot: 376303, Remaining slots: 4332913, Connected peers: 7
13:23:47.264 INFO  - Syncing     *** Target slot: 4709217, Head slot: 376630, Remaining slots: 4332587, Connected peers: 12
13:23:59.276 INFO  - Syncing     *** Target slot: 4709218, Head slot: 376959, Remaining slots: 4332259, Connected peers: 11
13:24:11.229 INFO  - Syncing     *** Target slot: 4709219, Head slot: 377282, Remaining slots: 4331937, Connected peers: 8
13:24:23.230 INFO  - Syncing     *** Target slot: 4709220, Head slot: 377623, Remaining slots: 4331597, Connected peers: 8
13:24:35.227 INFO  - Syncing     *** Target slot: 4709221, Head slot: 377955, Remaining slots: 4331266, Connected peers: 8
13:24:47.239 INFO  - Syncing     *** Target slot: 4709222, Head slot: 378290, Remaining slots: 4330932, Connected peers: 10
13:24:59.228 INFO  - Syncing     *** Target slot: 4709223, Head slot: 378621, Remaining slots: 4330602, Connected peers: 9
13:25:11.229 INFO  - Syncing     *** Target slot: 4709224, Head slot: 378943, Remaining slots: 4330281, Connected peers: 9
13:25:23.227 INFO  - Syncing     *** Target slot: 4709225, Head slot: 379231, Remaining slots: 4329994, Connected peers: 7
13:25:35.240 INFO  - Syncing     *** Target slot: 4709226, Head slot: 379447, Remaining slots: 4329779, Connected peers: 14
13:25:47.228 INFO  - Syncing     *** Target slot: 4709227, Head slot: 379776, Remaining slots: 4329451, Connected peers: 13
13:25:59.239 INFO  - Syncing     *** Target slot: 4709228, Head slot: 380116, Remaining slots: 4329112, Connected peers: 10
13:26:11.231 INFO  - Syncing     *** Target slot: 4709229, Head slot: 380447, Remaining slots: 4328782, Connected peers: 9
13:26:23.228 INFO  - Syncing     *** Target slot: 4709230, Head slot: 380774, Remaining slots: 4328456, Connected peers: 8
13:26:35.290 INFO  - Syncing     *** Target slot: 4709231, Head slot: 381110, Remaining slots: 4328121, Connected peers: 4
13:26:47.227 INFO  - Syncing     *** Target slot: 4709232, Head slot: 381282, Remaining slots: 4327950, Connected peers: 7
13:26:59.419 INFO  - Syncing     *** Target slot: 4709233, Head slot: 381500, Remaining slots: 4327733, Connected peers: 6
13:27:11.240 INFO  - Syncing     *** Target slot: 4709234, Head slot: 381700, Remaining slots: 4327534, Connected peers: 3
13:27:23.235 INFO  - Syncing     *** Target slot: 4709235, Head slot: 381956, Remaining slots: 4327279, Connected peers: 3
13:27:35.275 INFO  - Syncing     *** Target slot: 4709236, Head slot: 382232, Remaining slots: 4327004, Connected peers: 7
13:27:47.186 INFO  - Syncing     *** Target slot: 4709237, Head slot: 382247, Remaining slots: 4326990, Connected peers: 7
13:27:59.284 INFO  - Syncing     *** Target slot: 4709238, Head slot: 382582, Remaining slots: 4326656, Connected peers: 7
13:28:11.225 INFO  - Syncing     *** Target slot: 4709239, Head slot: 382911, Remaining slots: 4326328, Connected peers: 7
13:28:23.227 INFO  - Syncing     *** Target slot: 4709240, Head slot: 383136, Remaining slots: 4326104, Connected peers: 5
13:28:35.225 INFO  - Syncing     *** Target slot: 4709241, Head slot: 383478, Remaining slots: 4325763, Connected peers: 3
13:28:47.225 INFO  - Syncing     *** Target slot: 4709242, Head slot: 383582, Remaining slots: 4325660, Connected peers: 6
13:28:59.185 INFO  - Syncing     *** Target slot: 4709243, Head slot: 383582, Remaining slots: 4325661, Connected peers: 6
13:29:11.228 INFO  - Syncing     *** Target slot: 4709244, Head slot: 383675, Remaining slots: 4325569, Connected peers: 5
13:29:23.266 INFO  - Syncing     *** Target slot: 4709245, Head slot: 383821, Remaining slots: 4325424, Connected peers: 2
13:29:35.227 INFO  - Syncing     *** Target slot: 4709246, Head slot: 384098, Remaining slots: 4325148, Connected peers: 2
13:29:47.228 INFO  - Syncing     *** Target slot: 4709247, Head slot: 384332, Remaining slots: 4324915, Connected peers: 3
13:29:59.186 INFO  - Syncing     *** Target slot: 4709248, Head slot: 384332, Remaining slots: 4324916, Connected peers: 1
13:30:11.186 INFO  - Syncing     *** Target slot: 4709249, Head slot: 384332, Remaining slots: 4324917, Connected peers: 1
13:30:23.186 INFO  - Syncing     *** Target slot: 4709250, Head slot: 384332, Remaining slots: 4324918, Connected peers: 10
13:30:35.231 INFO  - Syncing     *** Target slot: 4709251, Head slot: 384479, Remaining slots: 4324772, Connected peers: 9

A fatal error has been detected by the Java Runtime Environment:

SIGSEGV (0xb) at pc=0x00007f294a1fbe06, pid=1, tid=125

JRE version: OpenJDK Runtime Environment Temurin-<IP_ADDRESS>+1 (<IP_ADDRESS>+1) (build <IP_ADDRESS>+1)
Java VM: OpenJDK 64-Bit Server VM Temurin-<IP_ADDRESS>+1 (<IP_ADDRESS>+1, mixed mode, tiered, compressed oops, compressed class ptrs, g1 gc, linux-amd64)
Problematic frame:
J 5360 c2 tech.pegasys.teku.spec.logic.common.statetransition.epoch.status.AbstractValidatorStatusFactory$$Lambda$758+0x0000000800572dc8.apply(Ljava/lang/Object;)Ljava/lang/Object; (20 bytes) @ 0x00007f294a1fbe06 [0x00007f294a1fbc60+0x00000000000001a6]

Core dump will be written. Default location: Core dumps may be processed with "/usr/share/apport/apport -p%p -s%s -c%c -d%d -P%P -u%u -g%g -- %E" (or dumping to /opt/teku/core.1)

An error report file with more information is saved as:
/opt/teku/hs_err_pid1.log

If you would like to submit a bug report, please visit:
https://github.com/adoptium/adoptium-support/issues

[error occurred during error reporting (), id 0xb, SIGSEGV (0xb) at pc=0x00007f295bd38941]

@AntoBrajkovic are you able to send us / upload the /opt/teku/hs_err_pid1.log log file?  Is this a fairly easy crash to reproduce?

https://bugs.openjdk.org/browse/JDK-8274365

I got the same error with Trino w/JDK 19.0.1
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007f9af83cc369, pid=1, tid=214
#
# JRE version: OpenJDK Runtime Environment Temurin-17.0.5+8 (17.0.5+8) (build 17.0.5+8)
# Java VM: OpenJDK 64-Bit Server VM Temurin-17.0.5+8 (17.0.5+8, mixed mode, sharing, tiered, compressed class ptrs, g1 gc, linux-amd64)
# Problematic frame:
# V  [libjvm.so+0x733369]  G1ParScanThreadState::trim_queue_to_threshold(unsigned int)+0x989
#
# Core dump will be written. Default location: /var/cores/core-%e-sig%s-user%u-group%g-pid1-time%t
#
# An error report file with more information is saved as:
# /data/trino/hs_err_pid1.log
#
# If you would like to submit a bug report, please visit:
#   https://github.com/adoptium/adoptium-support/issues
#

[error occurred during error reporting (), id 0xb, SIGSEGV (0xb) at pc=0x00007f9af9000898]

@metalshanked Can you send the core dump and/or log file?


Error on page

I believe this is reversed.  If you are asking to install the 64 bit version on a 64 bit OS, then the default landing folder should be %ProgramFiles% and not %ProgramFiles(x86)%.  It is at least confusing right?
From page:
Copy
msiexec /i <path_to_msi> /l*v <install_logfile_name> ALLUSER=1 ALLUSERS=1
This process installs Teams to the %ProgramFiles(x86)% folder on a 64-bit operating system and to the %ProgramFiles% folder on a 32-bit operating system. At this point, the golden image setup is complete. Installing Teams per-machine is required for non-persistent setups._

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

ID: fa9bcf61-0f3b-5705-959d-dc6f97239f98
Version Independent ID: 8a313955-4459-23ad-5c75-9f49482ec5a9
Content: Use Microsoft Teams on Azure Virtual Desktop - Azure
Content Source: articles/virtual-desktop/teams-on-avd.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@CRuD117
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@Heidilohr
Can you please check and add your comments on this doc update request as applicable.

Hi @CRuD117 - no, this is correct. the 'Program Files (x86)' folder is only available on 64-bit versions of Windows in order to differentiate between programs that are 64-bit vs 32-bit. 64-bit programs go to 'Program Files', 32-bit programs go to 'Program Files (x86)'. 'Program Files (x86)' doesn't exist on a 32-bit version of Windows. This has been the case since the first 64-bit versions of Windows.
Installing Teams_windows_x64.msi results in it being installed to C:\Program Files (x86)\Teams Installer, meaning the article matches what happens.

Whether the 64-bit Teams installer should install to the 'Program Files (x86)' folder is another question, but you should provide that feedback at https://aka.ms/TeamsFeedback instead.
#please-close


Standard cell page in the docs is broken

Expected Behavior
The page @ https://gf180mcu-pdk.readthedocs.io/en/latest/digital/standard_cells/standard_cells.html should list the standard cell libraries.
Actual Behavior
The standard cell libraries are missing currently. See below;

Version - GlobalFoundries GF180MCU PDK 0.0.0-75-g331e678 documentation

I think this is being caused by the submodules: recursive: false option added @
https://github.com/google/gf180mcu-pdk/pull/80/files#diff-03efc769b870804394632e45d7885272b44c16939517fb31c9d7c614d2ffae57R29

I think this is being caused by the submodules: recursive: false option added @

Looks like false is the default https://docs.readthedocs.io/en/stable/config-file/v2.html#submodules-recursive, so I don't think that's the culprid (and also https://github.com/google/open-source-pdks doesn't seem to have any nested submodules).
https://github.com/google/gf180mcu-pdk/pull/80/commits/60550f39ec47e36bf13eab7bd1b48d5e5490aa11 introduced a explicit list of submodules which could cause readthedocs to only clone those, @cbalint13 can you comment if that was intentional?
Switching to:
submodules:
  include: all

should allow all submodules to get cloned.

60550f3 introduced a explicit list of submodules which could cause readthedocs to only clone those, @cbalint13 can you comment if that was intentional?

@proppy
I picked up only that one module without beeing aware that before all was picked behaviour.

Goal was to make the after pdf generation working and we upgrade to v2  too.
The v2 upgrade puzzled us in a way, see:  https://github.com/google/gf180mcu-pdk/pull/80#issuecomment-1270494197 .

@cbalint13 thanks for the background, can you take a look at #83 and verify it doesn't affect the PDF support you contributed?

@cbalint13 thanks for the background, can you take a look at #83 and verify it doesn't affect the PDF support you contributed?

@proppy
PDF are never generated on the open PR only after push is merged (that was also our puzzle).


ios: getUserAgent not same as real user agent !

hello
I get user agent
Mozilla/5.0 (iPhone; CPU iPhone OS 10_1 like Mac OS X) AppleWebKit/602.2.14 (KHTML, like Gecko) Mobile/14B72
when using your DeviceInfo.getUserAgent()
but the real useragent which posted to Apache is
AppName/1 CFNetwork/808.1.4 Darwin/16.3.0
can I get the real one ?

I believe this is related to how we get the user agent. Whether it's using native networking primitives within react-native, or using webviews. We get the WebView, this is covered elsewhere but if you want the react-native primitive response you'd need a new API. Either way, resolving this is very stale and it's covered elsewhere


Training the model does not converge  0mAP 0 NDS score

When i train the model on the mini or trainval dataset using the simple detr3d_resnet101_gridmask config ,  i get around 0 map and 0 nds in the end, the model does not learn anything as the training process goes, but evaluating the model using your trained weights i can get the correct results, I am using nuscenes mini v1.0 dataset with mmdetection 1.0.0rc2 mmcv 1.6.0 and pytorch 1.9.1 with cuda 11.1

have you solve this problem？