extension activation fail after upgrade with Error: Cannot find module 'vscode'

Does this issue occur when all extensions are disabled?: Yes/No - it goes away after disable/enableof the extension.

Version: 1.79.2
Commit: 695af097c7bd098fbf017ce3ac85e09bbc5dda06
Date: 2023-06-14T08:58:52.392Z (6 days ago)
Electron: 22.5.7
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 22.3.0

Steps to Reproduce:
I'm not sure, we received several reports like that this week. I just hit it myself. It seems to go like a user upgrades the extension and then restarts VS Code, but new instance cannot activate the extension. As a workaround a user disable, reload, enable extension, reload and it starts working again. Just completely quitting VS Code does not help.
From extension host:
2023-06-21 09:27:09.729 [info] ExtensionService#_doActivateExtension gitpod.gitpod-desktop, startup: false, activationEvent: 'onUri:gitpod.gitpod-desktop'
2023-06-21 09:27:09.765 [error] Activating extension gitpod.gitpod-desktop failed due to an error:
2023-06-21 09:27:09.776 [error] Error: Cannot find module 'vscode'
Require stack:
- /Users/[redacted]/.vscode/extensions/gitpod.gitpod-desktop-0.0.130/out/extension.js
- /Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/loader.js
- /Applications/Visual Studio Code.app/Contents/Resources/app/out/bootstrap-amd.js
- /Applications/Visual Studio Code.app/Contents/Resources/app/out/bootstrap-fork.js
-
    at Module._resolveFilename (node:internal/modules/cjs/loader:1010:15)
    at s._resolveFilename (node:electron/js2c/utility_init:2:2740)
    at Module._load (node:internal/modules/cjs/loader:858:27)
    at f._load (node:electron/js2c/asar_bundle:2:13330)
    at f._load (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:127:30006)
    at E._load (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:127:26701)
    at Module.require (node:internal/modules/cjs/loader:1082:19)
    at g (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/loader.js:4:647)
    at 89496 (/Users/[redacted]/.vscode/extensions/gitpod.gitpod-desktop-0.0.130/out/extension.js:2:1793427)
    at __webpack_require__ (/Users/[redacted]/.vscode/extensions/gitpod.gitpod-desktop-0.0.130/out/extension.js:2:2320025)
    at 50112 (/Users/[redacted]/.vscode/extensions/gitpod.gitpod-desktop-0.0.130/out/extension.js:2:1451662)
    at __webpack_require__ (/Users/[redacted]/.vscode/extensions/gitpod.gitpod-desktop-0.0.130/out/extension.js:2:2320025)
    at /Users/[redacted]/.vscode/extensions/gitpod.gitpod-desktop-0.0.130/out/extension.js:2:2320518
    at Object.<anonymous> (/Users/[redacted]/.vscode/extensions/gitpod.gitpod-desktop-0.0.130/out/extension.js:2:2320581)
    at u._compile (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/loader.js:4:1271)
    at Module._extensions..js (node:internal/modules/cjs/loader:1243:10)
    at Module.load (node:internal/modules/cjs/loader:1058:32)
    at Module._load (node:internal/modules/cjs/loader:893:12)
    at f._load (node:electron/js2c/asar_bundle:2:13330)
    at f._load (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:127:30006)
    at E._load (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:127:26701)
    at Module.require (node:internal/modules/cjs/loader:1082:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Function.i [as __$__nodeRequire] (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/loader.js:5:98)
    at n.sb (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:127:28020)
    at async Promise.all (index 0)

From shared process:
2023-06-21 08:25:56.923 [info] Getting Manifest... gitpod.gitpod-desktop
2023-06-21 08:25:57.069 [info] Installing extension: gitpod.gitpod-desktop
2023-06-21 08:25:59.304 [info] Extension signature is verified: gitpod.gitpod-desktop
2023-06-21 08:25:59.467 [info] Extracted extension to /Users/[redacted]/.vscode/extensions/.7219170f-701f-4290-b3b4-1bb2b000d0a7: gitpod.gitpod-desktop
2023-06-21 08:25:59.489 [info] Renamed to /Users/[redacted]/.vscode/extensions/gitpod.gitpod-desktop-0.0.130
2023-06-21 08:25:59.490 [info] Extracting extension completed. gitpod.gitpod-desktop
2023-06-21 08:25:59.516 [info] Extension installed successfully: gitpod.gitpod-desktop
2023-06-21 08:25:59.519 [info] Marked extension as uninstalled gitpod.gitpod-desktop-0.0.129

Running into this occasionally over the past ~2 weeks when opening a remote ssh connection. Extension host logs:
2023-06-29 11:18:25.794 [error] Activating extension gitpod.gitpod-desktop failed due to an error:
2023-06-29 11:18:25.794 [error] Error: Cannot find module 'vscode'
Require stack:
- /Users/<me>/.vscode/extensions/gitpod.gitpod-desktop-0.0.133/out/extension.js
- /Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/loader.js
- /Applications/Visual Studio Code.app/Contents/Resources/app/out/bootstrap-amd.js
- /Applications/Visual Studio Code.app/Contents/Resources/app/out/bootstrap-fork.js
-

Closing and reopening vs code then connects fine to the remote

@jeanp413 Sorry that I missed the issue for so long, will try to get this with the .2 recovery release.

If we will do a 1.80.2, I will backport the fix there. But so far we don't have a regression important enough to justify a 1.80.2.

This bug has been fixed in the latest release of VS Code Insiders!
@akosyakov, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version 862fa13002d8018c328931ee1edccfd4ca76447d of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!

Friendly ping! Looks like this issue requires some further steps to be verified. Please provide us with the steps necessary to verify this issue.

I've tried a couple things, but I can't get the original issue to reproduce in 1.80. It seems like the reports come from macOS though, so maybe a macOS tester will be able to repro in 1.80. I tried the following:

Install and downgrade Codespaces. Reload.
Upgrade Codespaces and exit.
Open the URI vscode://GitHub.codespaces/init

Hard to repro via steps, but Codespaces also reported seeing this before it was fixed and they seem to no longer be reporting this once the fix went through.

I haven't been able to reproduce on Stable, but at least one Codespaces user has said that the latest Insiders builds don't reproduce the problem anymore, so it looks fixed.

@jkeech Awesome, thanks for chiming in!


[RESTEASY-1880] Parameter annotations with default names

Evolved version of #1467 and #1374
jira: https://issues.jboss.org/browse/RESTEASY-1880
More javadoc && community documentation would be useful
RESTEasyParamBasicTest#customValuesTest always fails, so this test is in ExpectedFailing category. This test needs to be fixed.

Closing this as #1467 is now merged and includes the latest commit here too.


prok.pw down

Hi,
Prok.pw is down, how do you fix it?

I got around prok.pw but forgegradle now..

Hello, that compiling changes need apply patch patch.patch from the folder eclipse-patch, also in dependencies stand modified FG: https://github.com/Bogdan-G/ForgeGradle/commits/FG_1.2 and to him modified https://github.com/Bogdan-G/diff4j/commits/master


Update CONTRIBUTING.md

Revised some wording under the Finding Jobs section - specific to when a contributor qualifies for a bonus and how to submit a proposal.

Details

Revised some wording under the Note section here

Revised some wording in this section

Fixed Issues
NA - no issues fixed, just some rewording in the above sections.

Tests
NA
--->
QA Steps
NA
--->
Tested On
NA
Screenshots
See Above
Web
NA
Mobile Web
NA
Desktop
NA
iOS
NA
Android
NA

@Christinadobrzyn Hi! You will need to sign the CLA assistant so we can merge this.
Just copy this text:
I have read the CLA Document and I hereby sign the CLA
into a new comment here and post it. That should do it 😊

@Christinadobrzyn can you review the changes being made here to see if they're covered?
https://github.com/Expensify/App/pull/4996
If not, can you update that issue with the suggested edits?

Ah yikes! Thanks for catching that Matt!
Yeah I don't want to disrupt the updates that Matt proposed. Because his pull request has many more updates and mine are very minor, I'm going to close this and post a new request once Matt's updates are complete.

Proposed these changes in Matts PR.


generating flow types

Following the examples laid out in the docs, the best practice it seems to put the query with the component, which makes sense to me.
At the same time, if I want to generate flow types, it asks for .graphql files, which means I have to separate out the query to its own files and name them *.graphql.
My question is, how do I put the query with the react component and then generate the flow types?
Thank you

You can also use gql template tags instead of separate .graphql files.

@martijnwalraven  I dont think that works, it says that it can parse javascript files but it throws an error on the first line of javascript:
// @flow
   ^
2: import React, {Component} from 'react'

this is the command I use:
apollo-codegen generate  component/**/*.js  --schema schema.json --target flow
I would like to also point out that the option to add __typename --addTypeName is not recognized.
per help command out here:
apollo-codegen generate [input...]

Options:
  --help                        Show help                              [boolean]
  --version                     Show version number                    [boolean]
  --schema                      Path to GraphQL schema file
                                    [string] [required] [default: "schema.json"]
  --output                      Output directory for the generated files[string]
  --target                      Code generation target language
       [choices: "swift", "json", "ts", "typescript", "flow"] [default: "swift"]
  --passthrough-custom-scalars  Don't attempt to map custom scalars [temporary
                                option]                         [default: false]
  --custom-scalars-prefix       Prefix for custom scalars. (Implies that
                                passthrough-custom-scalars is true if set)
                                                          [string] [default: ""]

@hyusetiawan , what is the output of apollo-codegen --version?

@lewisf  this is the output
0.10.13

@hyusetiawan fyi, 0.10.13 is an old release that doesn't have support for *.js files. You'll want to use apollo-codegen@latest


Global strings showing as undefined

Most Global strings are not getting picked up as defined for me. I saw you were planning on adding them in #9 and there are some commits around it, did something get broken?

The current behavior is that it only adds them to Lua.diagnostics.globals when completing code, it doesn't automatically pick up any existing globals so that has to be done manually. But I think it would be a nice improvement if it did, I should ask sumneko how that should be done.

They should be automatically marked now in the v0.9.1 release


[Question] How retransmits are handled?

Hi Glenn!
I read source, and I can't get the way retransmits are handled.
Let's say we have next situation. A send to B single packet, which is dropped during transmission.
A -[DATA1]-XXXXX (missing)-> B
Because B miss this packet, it will never send ACK to A (or any other thing).
How A should resolve this situation?
Likely there should be some timer, which should retransmit non-ACKed packets after some time.
Like (simplified):
while (reliable_endpoint_pending(endpoint)) {
   sleep(reliable_endpoint_wait_for_ack_timeout(endpoint));
   if (reliable_endpoint_pending(endpoint))
       reliable_endpoint_retransmit(endpoint);
}

Or am I missing something?

I was looking for the same thing.
Eventually I realized that reliable.io doesn't have a re-transmission strategy, it's left up to the user to implement one in their desired manner.
yojimbo (which uses reliable.io) does just that. I believe it simply re-transmits un-acked packets at a certain rate (100ms by default) until they are acked.
I'm not sure what happens if a packet is never acked. Consider Host A and Peer B:

Host A sends 30 packets per second to Peer B
Host A re-transmits at a rate of 0.1s
Peer B sends back 32 acks also at 30 packets per second
Packet loss is 10%

A packet is included in Peer B's acks for 32/30 seconds or roughly 1s. During which time it can be re-transmitted 1s/0.1s = 10 times.
If this packet for some reason never reaches Peer B within this time window despite the 10 re-transmissions (odds are very low at 0.1^10), it will never again be included in an ack. There are more factors involved that I haven't included, and some network conditions can make this probability much higher - though you wouldn't really expect a realtime game to work with such conditions anyway.
I haven't checked if yojimbo just keeps re-sending this packet forever or if it detects it and disconnects the peer. I think that at one point this packet is removed from the circular "sent packets" buffer, at which point yojimbo detects the situation and disconnects the offending peer.
Again, this is all just my understanding. I might be wrong.

Just FYI, nice interface for the same problem - KCP


fix ValidateStepResultsVariables to validate stepResults only

Changes
fixes https://github.com/tektoncd/pipeline/issues/8255
There is a mixture of validating step result and task result in both ValidateStepResultsVariables
ValidateStepResultsVariables should use step results set of name to only validate StepResults. TaskResults are validated via the function ValidateTaskResultsVariables

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[ ] Has Tests included if any functionality added or changed
[ ] pre-commit Passed
[ ] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including functionality, content, code)
[ ] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

@jkhelil aside from the validation, can we make sure it works ? (a tests from the reproducer in https://github.com/tektoncd/pipeline/issues/8255 that would validate we got the right values).

@jkhelil aside from the validation, can we make sure it works ? (a tests from the reproducer in #8255 that would validate we got the right values).

The task is been created after validation fix, but, yes i will do the hole test to see if we get the right results

@vdemeester
I can see it working with the reproducer scenario
`
➜  tekton tkn tr describe test-ndn9d
Name:              test-ndn9d
Namespace:         default
Task Ref:          collect-data
Service Account:   default
Timeout:           1h0m0s
Labels:
app.kubernetes.io/managed-by=tekton-pipelines
tekton.dev/task=collect-data
Annotations:
pipeline.tekton.dev/release=f1fc5c5
🌡  Status
STARTED         DURATION    STATUS
3 minutes ago   21s         Succeeded
📝 Results
NAME           VALUE
∙ resultsDir   tom
🦶 Steps
NAME             STATUS
∙ collect-data   Completed
∙ reduce         Completed
`
k get pods
Containers:
step-collect-data:
Container ID:  containerd://971efd7afdea2403d40c149e73ee1c534004e9ead4263cf5ae127bc54d1cc083
Image:         quay.io/konflux-ci/release-service-utils:e633d51cd41d73e4b3310face21bb980af7a662f
Image ID:      quay.io/konflux-ci/release-service-utils@sha256:13624fbf75efd88c3220aa19b1423786f4cacfd9abab9042e934ff6114addae0
Port:
Host Port:
Command:
/tekton/bin/entrypoint
Args:
-wait_file
/tekton/downward/ready
-wait_file_content
-post_file
/tekton/run/0/out
-termination_path
/tekton/termination
-step_metadata_dir
/tekton/run/0/status
-step_results
singleComponentMode
-results
resultsDir
-entrypoint
/tekton/scripts/script-0-8v22c
--
State:          Terminated
Reason:       Completed
Message:      [{"key":"resultsDir","value":"tom","type":1},{"key":"singleComponentMode","value":"scott","type":4},{"key":"StartedAt","value":"2024-09-18T11:31:45.793Z","type":3}]
Exit Code:    0
Started:      Wed, 18 Sep 2024 13:31:43 +0200
Finished:     Wed, 18 Sep 2024 13:31:45 +0200
Ready:          False
Restart Count:  0
Environment:
Mounts:
/tekton/artifacts from tekton-internal-artifacts (rw)
/tekton/bin from tekton-internal-bin (ro)
/tekton/creds from tekton-creds-init-home-0 (rw)
/tekton/downward from tekton-internal-downward (ro)
/tekton/home from tekton-internal-home (rw)
/tekton/results from tekton-internal-results (rw)
/tekton/run/0 from tekton-internal-run-0 (rw)
/tekton/run/1 from tekton-internal-run-1 (ro)
/tekton/scripts from tekton-internal-scripts (ro)
/tekton/steps from tekton-internal-steps (ro)
/var/run/secrets/kubernetes.io/serviceaccount from kube-api-access-rdjnz (ro)
/workspace from tekton-internal-workspace (rw)
step-reduce:
Container ID:  containerd://0d791fafaa4f65428d3170626c3a53e63873e908aa431cc53ab7832ac70122f3
Image:         quay.io/konflux-ci/release-service-utils:e633d51cd41d73e4b3310face21bb980af7a662f
Image ID:      quay.io/konflux-ci/release-service-utils@sha256:13624fbf75efd88c3220aa19b1423786f4cacfd9abab9042e934ff6114addae0
Port:
Host Port:
Command:
/tekton/bin/entrypoint
Args:
-wait_file
/tekton/run/0/out
-post_file
/tekton/run/1/out
-termination_path
/tekton/termination
-step_metadata_dir
/tekton/run/1/status
-results
resultsDir
-entrypoint
/tekton/scripts/script-1-cf47m
--
State:          Terminated
Reason:       Completed
Message:      [{"key":"resultsDir","value":"tom","type":1},{"key":"StartedAt","value":"2024-09-18T11:31:45.915Z","type":3}]
Exit Code:    0
Started:      Wed, 18 Sep 2024 13:31:43 +0200
Finished:     Wed, 18 Sep 2024 13:31:47 +0200

/test pull-tekton-pipeline-beta-integration-tests

/test pull-tekton-pipeline-beta-integration-tests

/test pull-tekton-pipeline-go-coverage-df

@vdemeester I can see it working with the reproducer scenario
➜  tekton tkn tr describe test-ndn9d
Name:              test-ndn9d
Namespace:         default
Task Ref:          collect-data
Service Account:   default
Timeout:           1h0m0s
Labels:
 app.kubernetes.io/managed-by=tekton-pipelines
 tekton.dev/task=collect-data
Annotations:
 pipeline.tekton.dev/release=f1fc5c5

🌡  Status

STARTED         DURATION    STATUS
3 minutes ago   21s         Succeeded

📝 Results

 NAME           VALUE
 ∙ resultsDir   tom

🦶 Steps

 NAME             STATUS
 ∙ collect-data   Completed
 ∙ reduce         Completed

k get pods
Containers:
  step-collect-data:
    Container ID:  containerd://971efd7afdea2403d40c149e73ee1c534004e9ead4263cf5ae127bc54d1cc083
    Image:         quay.io/konflux-ci/release-service-utils:e633d51cd41d73e4b3310face21bb980af7a662f
    Image ID:      quay.io/konflux-ci/release-service-utils@sha256:13624fbf75efd88c3220aa19b1423786f4cacfd9abab9042e934ff6114addae0
    Port:          <none>
    Host Port:     <none>
    Command:
      /tekton/bin/entrypoint
    Args:
      -wait_file
      /tekton/downward/ready
      -wait_file_content
      -post_file
      /tekton/run/0/out
      -termination_path
      /tekton/termination
      -step_metadata_dir
      /tekton/run/0/status
      -step_results
      singleComponentMode
      -results
      resultsDir
      -entrypoint
      /tekton/scripts/script-0-8v22c
      --
    State:          Terminated
      Reason:       Completed
      Message:      [{"key":"resultsDir","value":"tom","type":1},{"key":"singleComponentMode","value":"scott","type":4},{"key":"StartedAt","value":"2024-09-18T11:31:45.793Z","type":3}]
      Exit Code:    0
      Started:      Wed, 18 Sep 2024 13:31:43 +0200
      Finished:     Wed, 18 Sep 2024 13:31:45 +0200
    Ready:          False
    Restart Count:  0
    Environment:    <none>
    Mounts:
      /tekton/artifacts from tekton-internal-artifacts (rw)
      /tekton/bin from tekton-internal-bin (ro)
      /tekton/creds from tekton-creds-init-home-0 (rw)
      /tekton/downward from tekton-internal-downward (ro)
      /tekton/home from tekton-internal-home (rw)
      /tekton/results from tekton-internal-results (rw)
      /tekton/run/0 from tekton-internal-run-0 (rw)
      /tekton/run/1 from tekton-internal-run-1 (ro)
      /tekton/scripts from tekton-internal-scripts (ro)
      /tekton/steps from tekton-internal-steps (ro)
      /var/run/secrets/kubernetes.io/serviceaccount from kube-api-access-rdjnz (ro)
      /workspace from tekton-internal-workspace (rw)
  step-reduce:
    Container ID:  containerd://0d791fafaa4f65428d3170626c3a53e63873e908aa431cc53ab7832ac70122f3
    Image:         quay.io/konflux-ci/release-service-utils:e633d51cd41d73e4b3310face21bb980af7a662f
    Image ID:      quay.io/konflux-ci/release-service-utils@sha256:13624fbf75efd88c3220aa19b1423786f4cacfd9abab9042e934ff6114addae0
    Port:          <none>
    Host Port:     <none>
    Command:
      /tekton/bin/entrypoint
    Args:
      -wait_file
      /tekton/run/0/out
      -post_file
      /tekton/run/1/out
      -termination_path
      /tekton/termination
      -step_metadata_dir
      /tekton/run/1/status
      -results
      resultsDir
      -entrypoint
      /tekton/scripts/script-1-cf47m
      --
    State:          Terminated
      Reason:       Completed
      Message:      [{"key":"resultsDir","value":"tom","type":1},{"key":"StartedAt","value":"2024-09-18T11:31:45.915Z","type":3}]
      Exit Code:    0
      Started:      Wed, 18 Sep 2024 13:31:43 +0200
      Finished:     Wed, 18 Sep 2024 13:31:47 +0200

@jkhelil It would be great if this is covered as part of e2e if possible to add


[DataSpreadsheet]: Clicking out of data spreadsheet table doesn't save

Package
Carbon for IBM Products
Description
https://github.com/carbon-design-system/ibm-products/assets/10100397/c58ae0df-b3aa-45d3-a4c1-b613c9237869
I also checked on my local machine and the onDataUpdate does not get triggered in this use case. Expected behavior should be that the callback gets triggered this way user can set their own behavior
Component(s) impacted
Data Spreadsheets
Browser
Chrome
@carbon/ibm-products (previously @carbon/ibm-cloud-cognitive) version
2.40
Severity
Severity 2 = Aspects of design is broken, and impedes users in a significant way, but there is a way to complete their tasks. Affects major functionality, has a workaround.
Product/offering
IBM
CodeSandbox or Stackblitz example
https://carbon-for-ibm-products.netlify.app/?path=/story/ibm-products-components-data-spreadsheet-dataspreadsheet--data-spreadsheet&globals=viewport:basic
Steps to reproduce the issue (if applicable)
No response
Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

Will take a look at this sometime tomorrow. @matthewgallo I don't think this is an enhancement but rather it is a bug. Not sure why we wouldn't want to save what user inserted text if they click out of the table. For example, if I have a table that is 1 column and 1 row, there is no way for me to save this data since I can't click anywhere else.


Update affiliation and institutional partners

I have moved to AWS.  Per our definition of institution partners, I have removed JP Morgan Chase.  Several Jupyter contributors are currently employed by JP Morgan Chase, but no current Steering Council members.

Thanks Brian.
On Wed, Oct 23, 2019, 5:52 PM Brian E. Granger<EMAIL_ADDRESS>wrote:

Merged #353 https://github.com/jupyter/jupyter.github.io/pull/353 into
master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jupyter/jupyter.github.io/pull/353?email_source=notifications&email_token=AFIVZQCB5YTEYQZ5XCVKVZDQQDWTJA5CNFSM4JENGMY2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUNA4W2A#event-2738998120,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFIVZQGQH2JBXHIXOD3XZBTQQDWTJANCNFSM4JENGMYQ
.


fix: remove sudo from Clean Up action

Fixing the error:
Run sudo rm -rf ci/tasks/smoketest-settings
sudo: you do not exist in the passwd database
Error: Process completed with exit code 1.

was included in #3109, closing.


[vm/ffi] Allow subclassing Pointer?

dart:ffi provides two ways to instantiate Pointer objects: allocate and fromAddress.
Structs in dart:ffi are modeled as subtypes of Pointer<Void>. For this reason the signature of fromAddress is:
T fromAddress<T extends Pointer>(int ptr);

allocate does not support allocating structs with malloc, as we do not have a way (yet) to know what the size of a struct is. Because of this allocate always returns a Pointer, not a subtype:
Pointer<T> allocate<T extends NativeType>({int count: 1});

With these signatures, allocate can be a factory, static method or top level function, while fromAddress can only be a static method or top level function (as the static return type can be a sub type of Pointer).
We might be able to support allocate of structs in the future, if we have a predefined way to calculate/define the size of structs.
For reference, the current boilerplate of structs:
@ffi.struct
class Coordinate extends ffi.Pointer<ffi.Void> {
  @ffi.Double()
  double x;

  @ffi.Double()
  double y;

  @ffi.Pointer()
  Coordinate next;

  // generated by @ffi.struct annotation
  external static int sizeOf();

  Coordinate offsetBy(int offsetInBytes) =>
      super.offsetBy(offsetInBytes).cast();

  Coordinate elementAt(int index) => offsetBy(sizeOf() * index);

  static Coordinate allocate({int count: 1}) =>
      ffi.allocate<ffi.Uint8>(count: count * sizeOf()).cast();

  /// Allocate a new [Coordinate] in C memory and populate its fields.
  factory Coordinate(double x, double y, Coordinate next) {
    Coordinate result = Coordinate.allocate()
      ..x = x
      ..y = y
      ..next = next;
    return result;
  }
}

Do we prefer a factory constructor for allocate?
Or do we want to keep allocate and fromAddress as similar as possible and change the signature of allocate to support allocating structs as well?
And if so, a top level function or a static method for both?

Subclassing pointer requires us to invent values (boxing) of a type without calling a constructor -- places strange restrictions on subclasses, which might be hard to apply completely
Subclassing pointer is not as necessary if generalized typedefs are introduced
If Pointers cannot be subclassed, we don't need to use a boxed representation for the pointer address
However, it allows more ergonomic native signatures to be written, e.g.:

class Statement extends Pointer<Void> {
  // ...
}

Statement prepare(String query);

Quite often the Dart api does not coincide with Pointer-wrapper-classes
class Result extends IterableBase<Row> implements ClosableIterable<Row> {
  final StatementPointer _statement;
  // ...

https://github.com/dart-lang/sdk/blob/master/samples/ffi/sqlite/lib/src/database.dart#L122-L125
This would then have to be:
class StatementPointer {
  Pointer<Void> pointer;
}

No we will not allow subtyping, the API will be:
class Statement extends NativeType { }

Pointer<Statement> p;

Will class Foo extends Void {} be allowed? Or is it limited to only non-zero-sized native types?

I think we're planning to only allow extending Struct for now. However, structs can have size 0 if there are no fields.

When this is implemented, we should ensure that the following code doesn't crash:
import 'dart:ffi';
import 'dart:async';

class DbOptionsPtr extends Pointer<Void> {}

main(args) async {
  DbOptionsPtr opts = fromAddress(0);
  await Isolate.spawn(print, opts);
}

@sjindel-google Any thoughts on what we do with CString = Pointer<Uint8>?
See our use of CString in the SQLite sample: samples/ffi/sqlite/lib/src/bindings/signatures.dart and samples/ffi/sqlite/lib/src/bindings/bindings.dart.
Conceptually I would say:
CString = char* = Pointer<Uint8>.

With generalized typedefs and extension methods we could write these things as extension methods:
typedef CString = Pointer<Uint8>
extension CStringMethods on CString {
  // ..
}

You've proposed the following in the CL disallowing subclassing Pointer:
class CString extends Struct {}
Pointer<CString> = Pointer<Uint8>

This seems to equate CString to char instead of char*.
Alternatively we could require manual wrapping, but that does not enable us to write CString in function signatures.
class CString {
  final Pointer<Uint8> chars;
}

My feeling is that typedef/extension method is "obviously" the right solution.
What I proposed (intended to be a temporary workaround) is actually:
class CString extends Struct<CString> {
  Uint8 char;
}
Pointer<CString> = Pointer<Uint8>

I'm not an expert on the commonalities between inline C arrays and char pointers, but the sqlite3 api just mentions char*
If we want to keep the mental load of writing signatures and bindings low, we should maybe opt for Pointer<Char>, where we use a class Char extends Struct. The use sites would need to interact primarily through the Pointer instead of the ref:
Pointer<Char> myString;
myString.load<Char>().someMethod();

If we add .ref, this would instead look like:
Pointer<Utf8> myString;
myString.ref.toString();


Update setup_env_ubuntu.sh to use configure-hypervisor.sh script

:warning: IMPOTANT: This issue should not be actioned until https://github.com/clearcontainers/packaging/issues/46 has landed (post beta).

We need to change setup_env_ubuntu.sh so that rather than hard-coding the hypervisor options, it instead runs the script below to generate the correct build options:

https://github.com/clearcontainers/packaging/blob/master/scripts/configure-hypervisor.sh

To avoid all repos that go get github.com/clearcontainers/tests from also having to go get github.com/clearcontainers/packaging, setup_env_ubuntu.sh should be modified to just download the script itself:
curl -LO https://raw.githubusercontent.com/clearcontainers/packaging/master/scripts/configure-hypervisor.sh

See https://github.com/clearcontainers/packaging/issues/46

But we don't want to build/install qemu binary from sources, this was temporary, right ? The idea is to update the packages and our test scripts rely on qemu-lite or qemu-cc packages.


Display during boot sequence

Would it be possible to see the EZ-WBC 1.6 splashscreen instead of raspbian boot sequence displayed lines ?

At some point during the boot it should show the EZWBC logo. This is the earliest point possible. In future i will optimize the time this logo is shown. It will timeout after 10 seconds now, if the boot is not yet complete you will see the text again. This is the same behavior as the normal EZWBC image

I think that's important to see the splashscreen as long as possible for the final 1.6, because we will soon have a nice logo. That's important that this major version should show clearly the new logo, symbol of the revival of the project.

Agreed, put it in the 1.6 project pipeline. Will embed it a.s.a.p.

Thanks ! But now you need to sleep ...

@RespawnDespair
I will sent send you soon the new splashscreen with the new logo. It will be nice for the final 1.6 !!

@RespawnDespair :
I don't where to find the original splashscreen file !
Do you know where it is ?

Yes, please do, i will implement it with image loading, it should look very nice while booting!
(In future in will make boot splash more interactive, but that will be one version later i think)

@RespawnDespair :
I don't where to find the original splashscreen file !
Do you know where it is ?

It's implemented in the status app now, there is no image or anything, just text being sent to wbc_status.

Oh sorry, I believed it was an image ...

@Yes21 Not yet, soon it will be 😄

@RespawnDespair OK I prepare the image. Do you think that .png 1280x720 is suitable ?

@RespawnDespair : Here is the new splashscreen image.

I've made little changes to the splashscreen :

You can find here how @seeul8er did for DroneBridge : https://github.com/seeul8er/DroneBridge/tree/nightly/splash

While displaying a jpg or png image is certainly possible, there is the problem with the fixed resolution. In 1.6RC6, the splashscreen is a truetype font which is always drawn with the same size, regardless of resolution.
With an image you'll have the problem that a logo image made for a 1920x1080 screen won't fit on a 1280x720 or 640x480 screen and a 640x480 logo won't look good on a 1920x1080 screen.
I'd suggest looking into the openvg documentation and see if it's possible to scale jpgs or pngs down on the fly (and then use a logo image with a high resolution).
Another option would be to make jpgs/pngs with different resolutions and display the logo according to the detected resolution.
Or convert the Logo to a truetype font and display that (similar to the symbols in the OSD).

@rodizio1 Good to see you're back!
In the 'windowsave' branch there is the option to scale (or better stretch) the image.
I have implemented this in the wifibroadcast-splash repository.
It looks great as long as the aspect ratio of the screen does not vary to wildly.

As this is now implemented i'm closing the issue.
I'd like to expand on this idea, but for now we have other issues.

Latest splashscreen with new version number.


Dockerize snorkel!

@netj further details here?  Feel free to reassign, just thought you could provide a bit of detail here first

There are actually two aspects: using Docker for production and for development.
DeepDive started prototyping a few scripts for development with Docker, also used by Travis:

https://github.com/HazyResearch/deepdive/tree/master/DockerBuild
https://github.com/HazyResearch/deepdive/blob/master/.travis.yml
The workflow is documented: https://github.com/HazyResearch/deepdive/blob/master/doc/developer.md#-containerized-builds-and-tests

DeepDive doesn't have a good example of Docker for production yet, but it'll be a separate Dockerfile that installs runtime dependencies (instead of build-time) and the built code.  In Snorkel's case, we'd just set up a sane Python environment, pip install all requirements.txt, then start the notebook.  Snorkel users don't have to mess with the setup, and just docker run HazyResearch/snorkel and jump to their browser. (Caveat: Docker is the only thing they have to install, but it requires admin rights.)
Happy to take a first stab at the latter!

Closing for now


Inccorect peer key logging

If ledger does not contain current peer keys, there is nothing info about that
Would be nice to write smth like "None of the peers have your key, make sure you have configured it properly"

Haven't noticed, postgres have error messages, it is fine


Minor Nit --  Readme Image has Azure misspelled

In the Readme file, user cloud - Azure is misspelled.

Fixed in https://github.com/openclarity/functionclarity/pull/55


Description to run this on docker

What's the best way to run this on Docker? Can you specify what the Dockerfile should look like? I've tried running from an ubuntu base image but it keeps terminating with an error for version mismatch of ruby instance and other times issue with installing MySQL server
# Use the official Ubuntu image as the base
FROM ubuntu:latest

# Install MySQL Server
RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y mysql-server

# Install dependencies
RUN apt-get update && apt-get install -y git curl build-essential mysql-client redis-server

RUN apt-get install -y ruby-full=3.2.3

# Clone the repo
RUN git clone https://github.com/seven1m/bible_api
WORKDIR /bible_api
RUN git submodule update --init

# Install Ruby dependencies
RUN gem install bundler
RUN bundle config --local deployment true
RUN bundle install

...

I'm sorry, but I cannot help with this. I'm not very good at Docker myself. 😄


90° Display font

How to display 90° fonts in the left tab bar

Currently floem doesn't have text rotation so for now this isn't possible

You'd have to use an SVG to make that work (just to unblock you right now)


Test branch deployments should get deleted after branch is deleted

Bug Report

[x] I’ve read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I’ve read and agree to the Code of Conduct.
[x] I’ve searched for any related issues and avoided creating a duplicate issue.

What happened
When we add a -test to the end of a branch name, the branch is automatically deployed. However, once we delete that branch, the deployments still exist. I wonder if we can automatically delete these deployments, once the branch is deleted?
For example: these exist, but the branches have been deleted
design-system-components-featureexplicit-label-test.apps.y.cld.gov.au
design-system-components-featureexplicit-labels-2-test.apps.y.cld.gov.au
designsystem-featureroadmap-test.apps.y.cld.gov.au

After some quick research there really isn't a simple way to do this. The easiest way is probably to run a script with a CircleCI cron task: https://circleci.com/docs/2.0/configuration-reference/#cron

I've got access to cf spaces so I can just remove these deployments every couple of weeks

Nice

yea easy solution for now...


Windows Admin Center fails to load after installation of RSAT tools

Installed Admin Center on a Windows 2016 server, all fine.  Then RSAT were installed via the Roles/Features and now Admin Center fails to load its webpage.
The service still runs, and there have been no firewall changes at play either...
Nothing in the EventLog either.

Hi Corey,
Can you assign all future Windows Admin Center doc issues to Jeff Woolslayer?
Thanks!
Jason
From: Corey Plett<EMAIL_ADDRESS>Sent: Monday, October 8, 2018 8:17 AM
To: MicrosoftDocs/windowsserverdocs<EMAIL_ADDRESS>Cc: Jason Gerend<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windowsserverdocs] Windows Admin Center fails to load after installation of RSAT tools (#1604)
Assigned #1604https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs%2Fissues%2F1604&data=02|01|jgerend%40microsoft.com|1894bc84d4e342ee0bcd08d62d310970|72f988bf86f141af91ab2d7cd011db47|1|0|636746085961732796&sdata=nKS7kpmCHA2PvYx4tbarb%2FTfxG4buruOMfgtpzBGa6E%3D&reserved=0 to @JasonGerendhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJasonGerend&data=02|01|jgerend%40microsoft.com|1894bc84d4e342ee0bcd08d62d310970|72f988bf86f141af91ab2d7cd011db47|1|0|636746085961742805&sdata=ekTDLIRGjU8MwvKUhtr%2FQMBef1GD9oZ445r2ck%2BeXt4%3D&reserved=0.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs%2Fissues%2F1604%23event-1890602672&data=02|01|jgerend%40microsoft.com|1894bc84d4e342ee0bcd08d62d310970|72f988bf86f141af91ab2d7cd011db47|1|0|636746085961742805&sdata=sE%2FQkgUUFMwFHnNjmk5N8tH66zgsa0b2ZzCGGUWQt44%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHij1w3ZoCbFNtAfStnfysVPvQ8Ra2U6ks5ui2xSgaJpZM4XM7M6&data=02|01|jgerend%40microsoft.com|1894bc84d4e342ee0bcd08d62d310970|72f988bf86f141af91ab2d7cd011db47|1|0|636746085961752813&sdata=rNM3Y9XCgEg2Rhvw0yWkRVdmIq6qNTv%2Bj14uEI4PjW8%3D&reserved=0.

Sure. Will do.

From: Jason Gerend<EMAIL_ADDRESS>Sent: Monday, October 8, 2018 11:58:04 AM
To: MicrosoftDocs/windowsserverdocs
Cc: Subscribed
Subject: Re: [MicrosoftDocs/windowsserverdocs] Windows Admin Center fails to load after installation of RSAT tools (#1604)
Hi Corey,
Can you assign all future Windows Admin Center doc issues to Jeff Woolslayer?
Thanks!
Jason
From: Corey Plett<EMAIL_ADDRESS>Sent: Monday, October 8, 2018 8:17 AM
To: MicrosoftDocs/windowsserverdocs<EMAIL_ADDRESS>Cc: Jason Gerend<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windowsserverdocs] Windows Admin Center fails to load after installation of RSAT tools (#1604)
Assigned #1604https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs%2Fissues%2F1604&data=02|01|jgerend%40microsoft.com|1894bc84d4e342ee0bcd08d62d310970|72f988bf86f141af91ab2d7cd011db47|1|0|636746085961732796&sdata=nKS7kpmCHA2PvYx4tbarb%2FTfxG4buruOMfgtpzBGa6E%3D&reserved=0 to @JasonGerendhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJasonGerend&data=02|01|jgerend%40microsoft.com|1894bc84d4e342ee0bcd08d62d310970|72f988bf86f141af91ab2d7cd011db47|1|0|636746085961742805&sdata=ekTDLIRGjU8MwvKUhtr%2FQMBef1GD9oZ445r2ck%2BeXt4%3D&reserved=0.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs%2Fissues%2F1604%23event-1890602672&data=02|01|jgerend%40microsoft.com|1894bc84d4e342ee0bcd08d62d310970|72f988bf86f141af91ab2d7cd011db47|1|0|636746085961742805&sdata=sE%2FQkgUUFMwFHnNjmk5N8tH66zgsa0b2ZzCGGUWQt44%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHij1w3ZoCbFNtAfStnfysVPvQ8Ra2U6ks5ui2xSgaJpZM4XM7M6&data=02|01|jgerend%40microsoft.com|1894bc84d4e342ee0bcd08d62d310970|72f988bf86f141af91ab2d7cd011db47|1|0|636746085961752813&sdata=rNM3Y9XCgEg2Rhvw0yWkRVdmIq6qNTv%2Bj14uEI4PjW8%3D&reserved=0.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs%2Fissues%2F1604%23issuecomment-427943696&data=02|01|coreyp%40microsoft.com|d37f5dcb65694941647508d62d4ffa2c|72f988bf86f141af91ab2d7cd011db47|1|0|636746218858152163&sdata=xCyjXIol4uhEJ4Iih85vl67s9jJaVoEEUXpv9VBd7F0%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJbp4frIcw79t557pVMXhJHcS_fKk9BOks5ui6A8gaJpZM4XM7M6&data=02|01|coreyp%40microsoft.com|d37f5dcb65694941647508d62d4ffa2c|72f988bf86f141af91ab2d7cd011db47|1|0|636746218858162163&sdata=ai%2FeYZru9FGTMnxvr6vQN9TvlM4K%2F3F7o44vLmH9Ztk%3D&reserved=0.

Hi all, Any thoughts?
-thx

You mentioned that you checked to make sure the service was running, could you please review the other troubleshooting  steps? Even though you did not make any changes to the firewall, it would be valuable to make sure that nothing happened to the rule.
There are two locations in the event logs to look:
Windows Logs \ Application and Application and Service Logs \ Microsoft-ServerManagementExperience
When you are trying to access the gateway open the browsers development tools (F12), look in the console tab as well as the network traffic tab. Reply here with any errors.
Finally, check the output of the following commands (replace 443 with the port your specificed, if applicable):
netsh http show urlacl url=https://+:443/
netsh http show sslcert ipport=<IP_ADDRESS>:443
netstat -a


Submit button still around after registering web hook for new module

This submit button is still here on the deno.land/add_module page after I added a module. Is that intentional? Am I supposed to click it again?

Not the case anymore due to redesign


Better error message for vhost and location.

It would be nice to see what location or vhost for debugging.

FWIW this PR would have saved me a very significant number of hours of debugging due to the default location block added to vhosts.

Hi --
There's been no further activity on this since the 'needs rebase' and 'needs tests' labels were set. If we don't have any further comments (or a fix) in the next month, this PR will be closed.
The change does seem useful; if someone is interested in updating it or providing tests, you can get help in #voxpupuli on IRC or on the Puppet Community Slack in #voxpupuli or #testing.

Replaced with #928


Migration tp Flutter 3.0.0

After upgrading flutter to 3.0.0 I have an error when running desktop:
example:
Launching lib\main.dart on Windows in debug mode...
Building Windows application...
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_navigation/src/extension_navigation.dart(357,24): warning GBC99161D: Operand of null-aware operation '!' has type 'SchedulerBinding' which excludes null. [C:\Users\prozhar\Flutter\_projects\quest_creator\build\windows\flutter\flutter_assemble.vcxproj]
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_navigation/src/extension_navigation.dart(468,24): warning GBC99161D: Operand of null-aware operation '!' has type 'SchedulerBinding' which excludes null. [C:\Users\prozhar\Flutter\_projects\quest_creator\build\windows\flutter\flutter_assemble.vcxproj]
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_navigation/src/snackbar/snackbar.dart(452,22): warning GBC99161D: Operand of null-aware operation '!' has type 'SchedulerBinding' which excludes null. [C:\Users\prozhar\Flutter\_projects\quest_creator\build\windows\flutter\flutter_assemble.vcxproj]
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_state_manager/src/rx_flutter/rx_disposable.dart(20,22): warning GBC99161D: Operand of null-aware operation '?.' has type 'SchedulerBinding' which excludes null. [C:\Users\prozhar\Flutter\_projects\quest_creator\build\windows\flutter\flutter_assemble.vcxproj]
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_state_manager/src/rx_flutter/rx_notifier.dart(130,22): warning GBC99161D: Operand of null-aware operation '?.' has type 'SchedulerBinding' which excludes null. [C:\Users\prozhar\Flutter\_projects\quest_creator\build\windows\flutter\flutter_assemble.vcxproj]
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_state_manager/src/simple/get_controllers.dart(90,20): warning GBC99161D: Operand of null-aware operation '!' has type 'WidgetsBinding' which excludes null. [C:\Users\prozhar\Flutter\_projects\quest_creator\build\windows\flutter\flutter_assemble.vcxproj]
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_state_manager/src/simple/get_controllers.dart(96,20): warning GBC99161D: Operand of null-aware operation '!' has type 'WidgetsBinding' which excludes null. [C:\Users\prozhar\Flutter\_projects\quest_creator\build\windows\flutter\flutter_assemble.vcxproj]
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_navigation/src/router_report.dart(53,22): warning GBC99161D: Operand of null-aware operation '!' has type 'WidgetsBinding' which excludes null. [C:\Users\prozhar\Flutter\_projects\quest_creator\build\windows\flutter\flutter_assemble.vcxproj]
Debug service listening on ws://<IP_ADDRESS>:59093/vix8PknOt28=/ws
Syncing files to device Windows...
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_navigation/src/snackbar/snackbar.dart:452:22: Warning: Operand of null-aware operation '!' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../flutter/packages/flutter/lib/src/scheduler/binding.dart').
    SchedulerBinding.instance!.addPostFrameCallback(
                     ^
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_navigation/src/router_report.dart:53:22: Warning: Operand of null-aware operation '!' has type 'WidgetsBinding' which excludes null.
 - 'WidgetsBinding' is from 'package:flutter/src/widgets/binding.dart' ('../../flutter/packages/flutter/lib/src/widgets/binding.dart').
      WidgetsBinding.instance!.addPostFrameCallback((_) {
                     ^
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_state_manager/src/simple/get_controllers.dart:90:20: Warning: Operand of null-aware operation '!' has type 'WidgetsBinding' which excludes null.
 - 'WidgetsBinding' is from 'package:flutter/src/widgets/binding.dart' ('../../flutter/packages/flutter/lib/src/widgets/binding.dart').
    WidgetsBinding.instance!.addObserver(this);
                   ^
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_state_manager/src/simple/get_controllers.dart:96:20: Warning: Operand of null-aware operation '!' has type 'WidgetsBinding' which excludes null.
 - 'WidgetsBinding' is from 'package:flutter/src/widgets/binding.dart' ('../../flutter/packages/flutter/lib/src/widgets/binding.dart').
    WidgetsBinding.instance!.removeObserver(this);
                   ^
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_state_manager/src/rx_flutter/rx_notifier.dart:130:22: Warning: Operand of null-aware operation '?.' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../flutter/packages/flutter/lib/src/scheduler/binding.dart').
    SchedulerBinding.instance?.addPostFrameCallback((_) => onReady());
                     ^
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_state_manager/src/rx_flutter/rx_disposable.dart:20:22: Warning: Operand of null-aware operation '?.' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../flutter/packages/flutter/lib/src/scheduler/binding.dart').
    SchedulerBinding.instance?.addPostFrameCallback((_) => onReady());
                     ^
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_navigation/src/extension_navigation.dart:357:24: Warning: Operand of null-aware operation '!' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../flutter/packages/flutter/lib/src/scheduler/binding.dart').
      SchedulerBinding.instance!.addPostFrameCallback((_) {
                       ^
../../flutter/.pub-cache/hosted/pub.dartlang.org/get-4.6.1/lib/get_navigation/src/extension_navigation.dart:468:24: Warning: Operand of null-aware operation '!' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../flutter/packages/flutter/lib/src/scheduler/binding.dart').
      SchedulerBinding.instance!.addPostFrameCallback((_) {
                       ^

Screenshots

Flutter Version:
3.0.0
Getx Version:
4.6.1
Describe on which device you found the bug:
Windows desktop

same issue for me

same issue here need to fix it fast !

I will do this in the next few hours

@jonataslaw Obrigado mano pelo seu trabalho, você é o cara!!

Also with flutter 2.13.0

same issue here
for those who want to work with getx anyway

flutter downgrade v2.10
works
https://docs.flutter.dev/development/tools/sdk/releases?tab=linux

It will work with 3.0.0. It's just a warning.

same issue for me

t will work with 3.0.0. It's just a warning.

It break completly the app

t will work with 3.0.0. It's just a warning.

It break completly the app

New version released 4.62 😊

anyone face with this issue? Response to preflight request doesn't pass access control check: Redirect is not allowed for a preflight request. before it's alright

You can use it for a temporary fix before PR https://github.com/jonataslaw/getx/pull/2360
https://github.com/arjundevlucid/getx


ci: 👷 run code_checks in parallel

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ec1911b). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master        #7   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         5
  Lines             ?       199
  Branches          ?        34
==========================================
  Hits              ?       199
  Misses            ?         0
  Partials          ?         0

Continue to review full report at Codecov.

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


Windows Phone 10 is not detected

Windows Phone 10 is not correctly detected by this plugin.
isMobile.windows.device returns false
Mobile device model: Nokia Lumia 550
Operating system: Windows 10 Mobile
Browser: Microsoft Edge 38.14393.67.0

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Language translations incorrect

I'm not completely proficient at Spanish, but when I switch to the Spanish language, there are words that are definitely either wrong or not translated. See the screenshot below. "Yesterday" is not translated, and the word "memsaje" should be "mensaje" for "message". (@YagoGG If you have time, simply change this line: https://github.com/zulip/zulip-mobile/blob/55ab0bf94e82d04eb708b1066ccc8b0fa99ca1e8/src/i18n/translations/messages_es.json#L62).
Other issues (in the 2nd screenshot) are that "edited 2 days ago" is still in English. I understand that all the topics, channels, and names cannot and should not be translated but the system text should.
Neither "Yesterday" nor "edited _ days ago" are marked for translation. "Yesterday" (as well as "Today") is found here: https://github.com/zulip/zulip-mobile/blob/55ab0bf94e82d04eb708b1066ccc8b0fa99ca1e8/src/utils/tests/date-test.js#L47-L53. "edited _ days ago" is found here: https://github.com/zulip/zulip-mobile/blob/55ab0bf94e82d04eb708b1066ccc8b0fa99ca1e8/src/message/MessageTags.js#L58
This same issues and others happen when I switch to other languages. For example, if I switch to Chinese (in the 3rd screenshot), the entire word "message" is still in English. In this screenshot, I'm holding down on the screen while taking the screenshot to demonstrate that I'm in the Chinese language yet the word "Message" (partly hidden by the left slide-in) is displayed in English. From the mentors I learned that these types of translations (where one language gets translated and another isn't) are due to individual languages not having phrases in its file (https://github.com/zulip/zulip-mobile/tree/master/src/i18n/translations) for certain English phrases.
To get to these issues, simply log in to Zulip, click languages, and select any language. Then go back into the app and look for the words that are not translated or translated incorrectly.

Thank you for the detailed report, @thejwzhang! A couple things:

I don't consider "memsaje" -> "mensaje" an actual a bug in the software, but it's still a nice find! Mentioning @YagoGG, if he has time to update this :)

if Spanish word for "message" is translated, but the Chinese word isn't, it means that the word is correctly tagged, but simply has not been translated yet. That's annoying, but not a bug :)

"Yesterday" and "edited x days ago" not being translated are good finds. It is quite likely that these have not been tagged for translation, even if they should be. However, it is also possible that they simply have not been translated yet. Can you check if these strings have been marked for translation (should check in the zulip-mobile repo)? If not, you should add that information to the issue description, preferably with a link to the source line, and we can approve this :)

As Robert said, thank you for such a nice report @thejwzhang!

I have just fixed the "memsaje" typo, good catch! It should appear correctly in the next release of the app.
Regarding the strings that aren't translated, it's mostly what @roberthoenig mentioned: due to some technical difficulties, about a 30% of the mobile app strings aren't translated (I'm only talking about Spanish, don't know about other languages). The best thing you can do in those cases is to check the source and make sure that they have been tagged for translation.

Thanks, @roberthoenig @YagoGG
Apart from missing translations and occasional incorrect translations, there are two other issues that we can do more on the development side. So pinging @jainkuniya and @kunall17 to help too ;)
The first is easy: just missed strings to translate. I think there aren't many but will be great to find and fix them.
The second is more tricky: the stuff that is 'calculated'. For example, the '2 days ago' is calculated by an external function and is not directly translatable.

Closing as no longer relevant.


Cell Distance Explorer custom tool icon

Cell Distance Explorer custom tool icon
Katy requests a custom icon for the Cell Distance Explorer. She is looking for an icon that shows a measurement occurring between two cells. Currently, we are using the VCCF digital object icon, in the Cell Distance Explorer design file. We need Ushma to create another custom icon for this tool.

Design Organization
The following tasks are to be completed by Monday, June 3rd, 2024:
@u-p-design

Role: Illustrator, designer, artist
Tasks:

[x]  Design Cell Distance Explorer icon options for Katy
[x]  Iterate on Katy's feedback as needed
[x]  Receive written approval from Katy on the Cell Distance Explorer icon design
[x]  Add/organize the approved icon to the tools icon component in the design system

@katyb

Role: Sole decision maker, approval provider
Task:

Provide iterative feedback to Ushma to implement your vision
Send written approval to Ushma for the Cell Distance Explorer icon

@LibbyUX

Role: UX Designer, design system manager
Tasks:

Organize needs
Communicate needs to all parties
Add the approved icon from the design system to the Cell Distance Explorer file
Communicate the change to the development team

Resources

Design file component: Tool Icons
Design file: Cell Distance Explorer Landing Page (1920x1080)

Libby's quick iterations

I have contacted development about this change via Slack. Thank you team!


[WIP] Calendar frontend

hi dan

hi judtin

Thanks for the review. There were some weird versioning issues with linting I think so we disabled it, there's a separate PR up to resolve that.

Merging before resolving some nits to launch before start of sem, will correct in future PR soon(TM)


docs and tests of this repository

Hello,
after debugging and researching for https://github.com/ionos-cloud/module-ansible/pull/59 for some hours I've just found the tests in https://github.com/ionos-cloud/module-ansible/blob/master/tests/k8s-cluster-nodepool-test.yml , which seems do be the better documentation for me :+1: :laughing:
Would it be a solution to match these folders - either with symlinks or running include_tasks or something like that? So there would be a single source of truth ;)

Hello! Thank you for raising this issue!
Indeed, a single source of truth will add more clarity to the documentation. We're now working on some refactoring on the documentation part and we'll take care of this too. Will keep you updated here with the status of this enhancement.
Have a nice day!


FIRST/LAST should remove when transformed to NONE

e.g.
(setval FIRST NONE [1 2 3])
;; => [2 3]

Implemented in master.


Turnstile Analytics page Help very confusing.

Which Cloudflare product does this pertain to?
Turnstile
Existing documentation URL(s)
https://developers.cloudflare.com/turnstile/reference/turnstile-analytics/#turnstile-analytics
What changes are you suggesting?
The terminology/wording you are using is confusing.  When I am looking at the Turnstile Analytics page I want to know the number of events that Turnstile was active for e.g., 35 checkouts.  Then for this number how many challenges were issued and how many were solves with API or user or failed.  The number of events (35 checkouts) is either not shown or not described adequately
Additional information
no

Hi @GlennPresentsofMind ! Thank you for bringing this to our attention. Closing this out as this not something that documentation can address since we are not in charge of the Cloudflare dashboard. I’ve passed your issue along internally and the team is working on improving Turnstile Analytics.


Add spec to cover post-EU-Exit search dates

if the searched-for date is past BREXIT_DATE, and the current date is past BREXIT_DATE, then the searched-for date should show

Asana: [TARIFF0520] Quey/Bug from Daniel Clarke - re date setting, https://app.asana.com/0/1199180449660757/1198723048518973/f

@mcumcu is it still a Draft?

@mcumcu  @matthewford  can this be merged?

Yes, this can be merged. It handles the case wherethe searched-for date is past BREXIT_DATE, and the current date is past BREXIT_DATE. i.e., the search should work "normally" on 1 JAN 2021 and after.


DeclareStrictTypesFixer - Remove fix CS fix logic from fixer.

With the DeclareEqualNormalizeFixer (https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/2067 :+1: ) in place the CS fixing part of the declare strict fixer can (and should) be removed to make it more atomic.

Thank you @SpacePossum.


Issue with npm. package.json version of node-forge should be >=0.6.10

getPublicKeyFingerprint method of forge.pki. is not available before version 0.6.10. An error is thrown in ./lib/credentials/APNKey.js:14

It seems to be an issue with npm. The version that I fetch with "npm install" has a "node-forge": "0.6.x " dependency and that seems to be the real issue :/

Did you successfully resolve this?

Yeah, I cleaned the npm cache, deleted apn folder in node-modules and "npm install apn" successfuly installed the correct version of node-forge.


go get https://github.com/user/repo should work

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

$ go version
go version go1.12.1 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
macOS / amd64 (this applies to all platforms) (and I don't like giving out my local user name)
What did you do?
go get https://github.com/cloudflare/mitmengine
What did you expect to see?
Nothing, just having the dependency added.
What did you see instead?
An error:
package https:/github.com/cloudflare/mitmengine: https:/github.com/cloudflare/mitmengine: invalid import path: malformed import path "https:/github.com/cloudflare/mitmengine": invalid char ':'

I'm not sure what's the rationale behind this (it actually goes through the git protocol probably), but as an user I expect to be able to copy/paste from Chrome and have it work without removing the https:// everytime.
Thanks.

Please read go help packages - Go packages are defined by import paths, which do not include http nor https schemes.

Yes, I'm not saying this isn't how it's meant to be working right now, I'm saying this is how I expect it to work as a new user of Go.

Is there something in the go get documentation that made you think there should be an https:// prefix in front of the import path?

Anyway, I don't expect to change anyone's opinion, just pointing this out. Maybe you don't experience this, maybe you do. I think I'm far from the only person to expect https:// http:// git:// protocols to work when writing an url (even if in that case it represents an import path).

Thanks for providing additional information and context. It's helpful to finding out if there's something we can do to make an improvement.

I'm closing this as I don't think this is going to change and telling me to read the documentation is a bit disrespectful. To be honest that's exactly the reaction I expected.

I'm sorry that you feel that the replies here were disrespectful.  I know that was not the intent.
Unfortunately it's not really possible to use a complex system like Go without reading any documentation.  That is not a goal of ours.

Agree with OP this should work.

At a minimum go could suggest a corrected command

Complex systems often use dependencies, and sometimes those dependencies are poorly or improperly documented.
I ran across this issue when fighting the exact same problem (due to poor documentation in a guide I was following), so clearly there are at least two of us, and probably many more. The fact that the error drops a forward slash makes one immediately think there's a typo, or a broken redirect, or an escaping issue. My first step was to crawl through Go files in our repository trying to figure out where the missing slash was in the codebase, and it took me at least 10-15 minutes to figure out what ended up being a very simple mistake.
Providing a useful error (rather than one that ends up being unintentionally misleading, through no fault of the Go maintainers) would involve a minimum level of effort and would be appreciated by many folks in the future who will run across this same problem.
It may not be a goal of yours that folks use Go without reading the documentation, but the fact of the matter is that users will do so, as mentioned in the case of the issue creator's tutorial. Usability can help or hinder adoption of a platform, so I think it's at least worth considering, though, like all of us, I'm sure you have many competing priorities. Thanks for hearing us out :)


Data transfer: value transformers

We need to allow to configure column value transformations. For export, import and direct data migration.
Transformation like:

convert number to string
truncate string
call some SQL function in INSERT statement
etc

Duplicate of #12889


Aggregation UDF

We should convert PySpark aggregation UDF to an aggregation expression in the logical plan. This would allow the aggregation logical plan node to work correctly.

This is done in #214.


fix: update dependencies to their latest version & shifted project to…

Updated the packages in generator as well as base template
Also now projects will use webpack 5 instead of webpack 4

@vinodloha
Feedback Incorporated.
please have a look.


AbstractByteBuf.ensureWritable(...) should check if buffer was released

Motivation:
AbstractByteBuf.ensureWritable(...) should check if buffer was released and if so throw an IllegalReferenceCountException
Modifications:
Ensure we throw in all cases.
Result:
More consistent and correct behaviour

Cherry-picked into 4.1 (d125adec38a0fa64011e5d77c5e4eadad69d4368) and 4.0 (44a7475002fd9834de44224e5ec05ad9a5f489d2)


explicit tests comparing pawpyseed's all-electron WF and VASP's internal calculations

We've been trying to reproduce VASP's internal calculations of optical transition matrix elements - that's not what we really want, but just to convince ourselves that we have the right wavefunctions.  We've been unable to do so, so far, and we'd like to eliminate the possibility that the issue is a bug in pawpyseed.

Are there any explicit tests of the pawpyseed output wavefunctions that show that they give the same results as VASP's internal matrix element (of any sort) calculations?   I think something like that would help us figure out if we are using the wavefunctions correctly.

Would you expect that pawpyseed's wavefunctions (if printed at sufficient spatial resolution) would be able to reproduce VASP's own optical transition matrix elements, or is there something subtle about pseudo vs. PAW non-pseudized vs. actual all electron wavefunctions that would preclude that?

I just did a test with LAECHG=T comparing the content of AECCAR2 and the norm of the pawpyseed wavefunction for a single electron system (1 Li atom in a cell), and it looks correct.  Might still be nice to have that as a unit test.

Sorry for the lack of responsiveness this week; I will push to PyPI tonight and answer the other issue you posted ASAP.

Not really, only the overlap operator (which is tested pretty thoroughly in core.tests.test_core.py). The MomentumMatrix class has some basic unit tests to make sure that frequency components of the density are correct by calculating <nk|exp(iGR)|nk> with some nonzero G (https://github.com/kylebystrom/pawpyseed/blob/6f5aa0b8ca8c28a0221e5256afeb939c3344560b/pawpyseed/core/tests/test_momentum.py#L77). You are right that this would be good though. I also have a test to check electron count normalization (https://github.com/kylebystrom/pawpyseed/blob/6f5aa0b8ca8c28a0221e5256afeb939c3344560b/pawpyseed/core/tests/test_core.py#L481). It is correct, but only to about 2e-3 (see Part 2 for why that is). Unfortunately, I have not done optical matrix element calculations before and do not currently have a VASP license, so I cannot do that on my own currently. I will possibly have a VASP license again in the near future though.

Not quite. The main issue here is that the projector functions used to transform between pseudo wavefunctions and the AE wavefunction are not complete, and the formula used in the PAW formalism for expectation values and matrix elements is only exact in the complete basis set limit of the projectors. Therefore, a matrix element calculated as $\int d^3r \phi_i(r) \hat{O} \phi_j(r)$ will always have some numerical error compared to $<i|\hat{O}|j>$ in the PAW formalism, no matter how dense the grid is. This is also why the unit tests to check that the frequency components of the density are correct only goes over a few wavenumbers; the incompleteness of the partial wave basis set causes issues at higher frequencies (unless there's a significant bug I missed, which is possible, but I have proofread that function a few times). Sorry for the confusion on this issue; I should find a place in the docs to make this more clear so that people are not overly trusting of this info.

Thanks for the thorough response. I appreciate the help, and will now have to spend some time thinking about your comments and the PAW formalism.  I'll let you know if I have any followup questions.


Create SQS service for the S3 Bucket

Develop Amazon Simple Queue Service (SQS) to react to new content uploaded into the S3 bucket.

Finished in https://github.com/hivereview/article-hosting-infrastructure/commit/e1a8e9c7261c9bcee49d860ab888624ffd6ac15e


[FEAT] Using with Apple Silicon M1 / M2 / M3 chipsets

How can I use zep with power of Apple M chipsets? What should I wrote in getzep/zep-nlp-server/config.yaml in embeddings.device section?
How can I use already downloaded all-MiniLM-L6-v2 model if I have already download it in some specific foler?

log_level: info
server:
  port: 5557
embeddings:
  # Modifying the device to cuda requires a container with CUDA installed
  # and torch with CUDA support
  device: cpu
  messages:
    enabled: true
    model: all-MiniLM-L6-v2
  documents:
    enabled: true
    model: all-MiniLM-L6-v2
    # This is the recommended, moderate-memory model for document embeddings
#    model: multi-qa-mpnet-base-dot-v1
nlp:
  spacy_model: en_core_web_sm

You can set device to mps. Make sure you run zep-nlp directly from the command line with:
make run
When running from the command line, HF models are stored in the model cache and won't be downloaded again on subsequent runs.

@danielchalef I try to doing this way, there is messages:
zep_nlp  | 2023-11-09 09:44:30,577 - zep-nlp - INFO - Configured for MPS
zep_nlp  | 2023-11-09 09:44:30,577 - zep-nlp - WARNING - Configured for MPS but MPS not available, using CPU

Here is the full log:
(base) andrey@m2 current % docker compose up zep_nlp
WARN[0000] Found orphan containers ([llm_redis]) for this project. If you removed or renamed this service in your compose file, you can run this command with the --remove-orphans flag to clean it up.
[+] Running 1/1
 ✔ Container zep_nlp  Recreated                                                                                0.2s
Attaching to zep_nlp
zep_nlp  | 2023-11-09 09:44:30,577 - uvicorn.error - INFO - Started server process [1]
zep_nlp  | 2023-11-09 09:44:30,577 - uvicorn.error - INFO - Waiting for application startup.
zep_nlp  | 2023-11-09 09:44:30,577 - zep-nlp - INFO - Configured for MPS
zep_nlp  | 2023-11-09 09:44:30,577 - zep-nlp - WARNING - Configured for MPS but MPS not available, using CPU
zep_nlp  | 2023-11-09 09:44:30,577 - zep-nlp - INFO - Downloading model all-MiniLM-L6-v2 if not present. This may take a while.
Downloading (…)e9125/.gitattributes: 100%|██████████| 1.18k/1.18k [00:00<00:00, 6.10MB/s]
Downloading (…)_Pooling/config.json: 100%|██████████| 190/190 [00:00<00:00, 913kB/s]
Downloading (…)7e55de9125/README.md: 100%|██████████| 10.6k/10.6k [00:00<00:00, 13.8MB/s]
Downloading (…)55de9125/config.json: 100%|██████████| 612/612 [00:00<00:00, 1.91MB/s]
Downloading (…)ce_transformers.json: 100%|██████████| 116/116 [00:00<00:00, 327kB/s]
Downloading (…)125/data_config.json: 100%|██████████| 39.3k/39.3k [00:00<00:00, 572kB/s]
Downloading pytorch_model.bin: 100%|██████████| 90.9M/90.9M [00:03<00:00, 24.6MB/s]
Downloading (…)nce_bert_config.json: 100%|██████████| 53.0/53.0 [00:00<00:00, 188kB/s]
Downloading (…)cial_tokens_map.json: 100%|██████████| 112/112 [00:00<00:00, 233kB/s]
Downloading (…)e9125/tokenizer.json: 100%|██████████| 466k/466k [00:00<00:00, 1.21MB/s]
Downloading (…)okenizer_config.json: 100%|██████████| 350/350 [00:00<00:00, 1.03MB/s]
Downloading (…)9125/train_script.py: 100%|██████████| 13.2k/13.2k [00:00<00:00, 13.6MB/s]
Downloading (…)7e55de9125/vocab.txt: 100%|██████████| 232k/232k [00:00<00:00, 1.67MB/s]
Downloading (…)5de9125/modules.json: 100%|██████████| 349/349 [00:00<00:00, 1.26MB/s]
zep_nlp  | 2023-11-09 09:44:42,780 - zep-nlp - INFO - Loaded model all-MiniLM-L6-v2 on device cpu.
zep_nlp  | 2023-11-09 09:44:42,780 - zep-nlp - INFO - Loading spacy model en_core_web_sm. If the model is not already downloaded, this may take a while.
zep_nlp  | 2023-11-09 09:44:43,004 - zep-nlp - INFO - Loaded spacy model en_core_web_sm.
zep_nlp  | 2023-11-09 09:44:43,004 - uvicorn.error - INFO - Application startup complete.
zep_nlp  | 2023-11-09 09:44:43,004 - uvicorn.error - INFO - Uvicorn running on http://<IP_ADDRESS>:5557 (Press CTRL+C to quit)

What I'm doing wrong?

Zep relies on PyTorch for embedding inference. On MacOS, Zep's NLP server runs in a Linux ARM64 container. PyTorch is not optimized to run on Linux ARM64 and does not have access to MacBook M-series acceleration hardware. You'll need to clone the zep-nlp repo and run it directly from the MacOS terminal.
git clone https://github.com/getzep/zep-nlp-server.git
cd zep-nlp-server
poetry install
make run

Closing as solved.


Add config option for s3 key

First off, thanks a lot for the package, I'm playing around with it and excited to roll it out gradually.
This is a QoL feature request; it would be great to specify a key inside the S3 bucket for the shared website. I currently have an s3 website hosting the dbt documentation and would like to use the same for elementary, just specifying a subfolder.
The current workaround of setting up a post-run hook after running edr monitor report works perfectly of course, but that would be a slight improvement.

Hi @askvinni,
Thank you for the compliments!
I'm not sure I understand the request all the way.
We have a edr monitor send-report command that uploads directly to S3, here is the documentation.
Does this answer the use case or are there improvements you think we should add?


Best Practice on Switching Environments

Add Best Practice on Switching Environments to README

Don't forget that Apickli has nothing to do with Apigee...


[PL-46739] Add important change info to delegate RN

Add important change info to delegate RN
For reviewers, a preview is available:
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screenshot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65e634c9d94524008484d788--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65e63a25d0044700a058129e--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65e645181341d60d842e79cd--harness-developer.netlify.app


fix: ColorScale can miss events because render does not return a Promise

fixes: #981
Describe your changes
Render should return a promise, otherwise, the Figure will not wait before sending events, causing the ColorAxis to miss them, which in turn causes a layout issue.
Testing performed
This is a timing issue, which only happens when there are many browser tabs open, or other forms of delay. This cannot be unit tested as it. It is however not observed to occur with this change.

@maartenbreddels this appears to require a rebase.
Also, there is the new sign-off thing that we need to do in the last commit of a PR.


move scalar-handling logic into possibly_convert_objects

[x] User visible changes (including notable bug fixes) are documented in whats-new.rst

This seems an undisputed change up to now if we count @spencerkclark in who suggested that change in #9618. Getting this in soon.

Thanks @spencerkclark and no worries!

Mypy errors are unrelated, getting this in now.


Fix grad getting enabled for parameters on which explicitly disabled in case of multiple optimizers

What does this PR do?
In case of multiple optimizers, the bug causes grad to be enabled for all the parameters associated to an optimizer. This is erroneous in case there are any parameters which are explicitly kept non-trainable.
Fixed it by storing the original grad setting for all the parameters, and then setting it accordingly when that optimizer is called.
Co-authored by : @nisheethlahoti

Fixes #5292 <- this links related issue to this PR
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes [if needed]?
[ ] Did you write any new necessary tests [no need for typos, docs]?
[x] Did you verify new and existing tests pass locally with your changes?
[x] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified
[ ] Check that target branch and milestone are aligned!

Did you have fun?
Make sure you had fun coding 🙃

Dear @jain-anshul @nisheethlahoti,
Thanks a lot for your work ! I gave me the base to resolve the bug.
I solved in this PR: https://github.com/PyTorchLightning/pytorch-lightning/pull/5574
Closing this one for now.
Best,
T.C

Dear @jain-anshul @nisheethlahoti,
Thanks a lot for your work ! I gave me the base to resolve the bug.
I solved in this PR: https://github.com/PyTorchLightning/pytorch-lightning/pull/5574
Closing this one for now.
Best,
T.C


api key is not able to generate

Describe the bug
I am trying to generate api key from https://cicd.blstsecurity.com/ ,but its keep loading .
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

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

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hey @kpreety !
We appreciate you contacting us about this. We just fixed it; please get in touch with us if you run across any new issues.
<EMAIL_ADDRESS>


HLL coverage improvements

HLL coverage improvements
HyperLogLog line coverage is a bit low because some conditions during estimation are not so easy to trigger. After exposing the HLL internals it was easier to find test cases using data already available for the current tests.
This PR improves the coverage and also simplifies some error checking inside the HLL.

[ ] Is it mergable?
[ ] Did it pass the tests?
[ ] If it introduces new functionality in scripts/ is it tested?
Check for code coverage.
[ ] Is it well formatted? Look at make pep8, make diff_pylint_report,
make cppcheck, and make doc output. Use make format and manual
fixing as needed.
[ ] Did it change the command-line interface? Only additions are allowed
without a major version increment. Changing file formats also requires a
major version number increment.
[ ] Is it documented in the ChangeLog?
[ ] Was a spellchecker run on the source code and documentation after
changes were made?

Ready for review @mr-c @ctb @camillescott

@luizirber As soon as this is made mergable and https://github.com/ged-lab/khmer/pull/738#discussion_r23560245 is address I think it will be ready to go

Jenkins, test this please

[ ] Is it mergeable?
[ ] Did it pass the tests?
[ ] If it introduces new functionality in scripts/ is it tested?
Check for code coverage with make clean diff-cover
[ ] Is it well formatted? Look at make pep8, make diff_pylint_report,
make cppcheck, and make doc output. Use make format and manual
fixing as needed.
[ ] Did it change the command-line interface? Only additions are allowed
without a major version increment. Changing file formats also requires a
major version number increment.
[ ] Is it documented in the ChangeLog?
http://en.wikipedia.org/wiki/Changelog#Format
[ ] Was a spellchecker run on the source code and documentation after
changes were made?

Jenkins, test this please

Ready for review @mr-c @ctb @camillescott

Can you eliminate that warning and cover those last two bits with tests?
Otherwise, it LGTM; I love the output of python -c 'import khmer; help(khmer.HLLCounter)'!

jenkins, Test this please

Jenkins, test this please

@mr-c fixed remaining issues


[No QA] Changed App Store Icon

Details

We were getting a deployment error due to a transparent icon, and we believe we've replaced the culprit 🤞
Fixed Issues

Slack thread: https://expensify.slack.com/archives/C01GTK53T8Q/p1626293141260600
Tests

QA Steps

Tested On

[ ] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

:warning: :warning: Heads up! This pull request has the CP Staging label. :warning: :warning:
Merging it will cause it to be immediately deployed to staging, even if the open StagingDeployCash deploy checklist is locked.

🚀 Cherry-picked to staging         in version: 1.0.77-7🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production         in version: 1.0.79-4🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Mobile styling review

Review site for mobile functionality

Finished via https://github.com/lbryio/lbry.tech/commit/d876bdac34f31400e38193b7bed2b38853ff7869


Boston House Price Prediction

ML-Crate Repository (Proposing new issue)
🔴 Project Title : Boston House Price Prediction
🔴 Aim : The Boston House Price Prediction project aims to predict the median value of owner-occupied homes in the Boston area.
🔴Dataset : (https://lib.stat.cmu.edu/datasets/boston)
🔴 Approach : 1. Project Setup and Planning
Define the scope and goals of the project.
Gather all necessary libraries and tools.
Understand the dataset and its features.
2. Data Collection and Understanding
Load the Dataset:
Use sklearn.datasets.load_boston to load the Boston housing dataset.
Convert the dataset to a Pandas DataFrame for easier manipulation and analysis.
Initial Exploration:
Display the first few rows of the dataset to understand its structure.
Get a summary of the dataset to see the number of features, target variable, and data types.
3. Exploratory Data Analysis (EDA)
Visualize Data:
Use histograms to visualize the distribution of features and the target variable.
Correlation Analysis:
Create a correlation matrix heatmap to understand the relationships between features and the target variable.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name :  Payal Sinha
GitHub Profile Link : https://github.com/payalsinha12
Participant ID (If not, then put NA) :
Approach for this Project : The Boston House Price Prediction project aims to predict the median value of owner-occupied homes in the Boston area.
What is your participant role? (Mention the Open Source Program name. Eg. HRSoC, GSSoC, GSOC etc.) SSoc

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@das-ankur please assign my this issue .

Hi @payalsinha12 this problem statement is already solved. Closing this issue as duplicate.


Fix Fisher's exact plot for non-boolean columns

The switch from benefit to Response, combined with my with_condition1_fraction addition, didn't play well together; needed to use a condition1_mask rather than the same of the column.
@arahuja

Coverage decreased (-0.3%) to 58.801% when pulling 20267ccadd20b02eaf4e0346bcdb640938ea2978 on fisher_fixes into 3ceb6bb92aea59704f9757ffe3f38e36c7074296 on master.

ah, ok, I see the issue here, looks good, I'd say the alternative is to use the count_table to compute the means, but either works

@arahuja good call, sounds simpler, though I'll leave it as is for now.


Analogic for daterange

This PR is to resolves Fred's request in PR https://github.com/Avaiga/taipy/pull/1991
cc: @FredLL-Avaiga

sorry Fred, I pushed wrong commit previously (it included my local files) but now it's clean =)

I think it's the same culprit Missing 'checks: write' permission as in https://github.com/Avaiga/taipy/pull/1991

I think it's the same culprit Missing 'checks: write' permission as in #1991

that's not what I saw

Ohh got it, fixed now

please update your branch to get ridd of the linter issue

I've updated your branch
If the tests are OK, we'll merge it


Interoperability: RTX

DECT station Snom M900 with DSP module A900 and DECT phone Snom M70 (both firmware 06.50) provide a horrible quality.
The cause is unknown, especially because there is nothing in the Syslog of the DECT station nor any difference on the RTP layer in Wireshark visible. My transcoding module was tested with several versions and variants of the BroadVoice32 library, between 64-bit machines, 32-bit machines, and a mixture of 64-bit and 32-bit. If anyone has an idea, please, comment.
The results should be the same for all RTX8663-based DECT stations.

I changed the library to the untouched official variant, to the latest version 1.2. Baresip, Linphone, and X-lite work. Still, Snom M70 sounds horrible.
Originally, without my patch for the makefile, the library contains a process tool:
cd ./BroadVoice32/FloatingPoint/Linux/
make
../process/BroadVoice32 enc /var/lib/asterisk/sounds/en/demo-echotest.sln16 ~/output.bv32
xxd -g 1 -c 16 ~/output.bv32

For example, if you use the sample configs of Asterisk, then extension 600 is an Echo Test. Capture the created RTP packets in Wireshark. Compare with the above encoding result. The bits in my tests matched 100%, indicating that my module works correctly.
The next step would be going through the test cases ….


Example Doesn't Work

If I simply change the UpiIndiApps.PhonePe to UpiIndiaApps.PayTm then the app simply broke down and exit.

Try writing UpiIndiaApps.PayTM.

I have done that but still no use. App crashes anytime I click on the pay button.
I think you should use some kind of exception handling in case any error happens.

I just downloaded PayTM and the example did actually crashed. It happens because PayTM is not ready to accept the payment i.e. I wasn't logged in. After registration it works fine. I will try to add some exception handling in the plugin for this issue. Thanks for letting me know about this issue.

I have also tested the other options. TrueCallerUpi is also crashing when button clicked and the airtelpaymentupi shows error after opening the app.

Well I have updated the plugin. It was working fine in my case. I have tested it with all the apps (some ready to accept while some not). In case you see error again, write the error in comment.


Cant scroll in char selection modal when selecting character for teammates

Don't think this is a "fix ASAP" angle. users can still use the filters to narrow down the search


Routing - use @key in NavLink loop warning (instead of NavLink href)

I believe the @key directive attribute should be emphasized whenever there's a loop in the rendering process. Moreover, in this scenario, it could also serve to demonstrate the difference between using @c and @current. Specifically, using <NavLink href="@c" ...> might not be very clear for numeric values of c and could lead to confusion for readers.

Internal previews

📄 File
🔗 Preview link

aspnetcore/blazor/fundamentals/routing.md
ASP.NET Core Blazor routing and navigation

whenever there's a loop in the rendering process

Not really ... @key shouldn't be required and isn't recommended when the rendered content is static because it isn't performant. Also, introducing it here is a bit distracting from the example because it introduces an additional concept that must be explained.
Also, I don't agree with removing the href because it demonstrates that the content of the NavLink that's at stake, not the C# value of c (@c). However, I propose a different change to make the href assignment relevant ...
- href="@c"
+ href="product-number/@c"

... with ...
-  @c
+  Product #@current

... or in the second case with foreach ...
-  @c
+  Product #@c

... and possibly a slight content addition mentioning that the nav menu lists a handful of product links.
WRT my understanding on the C# value of c ... I would like to ask Mackinnon to put an 👁 on this coverage.
Mackinnon ... I don't think we should have @key here for the reason that I gave, but let me know if you disagree and why. If we include it, we're going to need to say why it's here and cross-link coverage on it.
Second ... IIRC/AFAIK, but I could be wrong ... the value of c problem to overcome is in the content of the NavLink, not in general use of c otherwise. I think that because when this was discussed with the PU and the content was placed we were careful to design the example to show c (@c) in use outside of the content (i.e., in the href) to demonstrate no ill effects. I don't do "random" 😄 ... I might be wrong about something 🙈 ... but I definitely don't do random things 😆.

You're right; for static SSR, the @key doesn't make much sense since there are no updates happening (or are there with enhanced forms or enhanced navigation?).
On the other hand, the importance of @key for interactive scenarios, in my opinion, is not sufficiently emphasized in the documentation, and in practice, I repeatedly encounter its underestimation (for example, with the argument "it only gains a little more performance sometimes"), even though @key is not just about performance and is absolutely necessary in many scenarios. See, for example:

https://github.com/dotnet/aspnetcore/issues/22355
https://github.com/havit/Havit.Blazor/issues/524

The documentation for @key states:

Typically, it makes sense to use @key whenever a list is rendered (for example, in a foreach block) and a suitable value exists to define the @key.

...so it seemed good to me to use @key in a for/foreach example to spread best practices.
(BTW. as far as I know, in React, a key is required in loops, and I have a feeling that Blazor will gradually move in that direction too, :-D I guess we will see at least code analyzer for this suggestion soon.)

Yeah ... but I didn't say that. I merely said that it wouldn't be performant in this example. I defer to Mackinnon's wisdom on just how far @key should be pushed/recommended.
Mackinnon ... let's focus just on this example/this article for the moment ... because .......................
WRT the wider issue of how @key is described and demonstrated ... @hakenr, open an issue from that article. More examples can be added to the article, and additional cross-links can be added around the repo to that article where it might be relevant. Mackinnion might leave comments on your new issue for when it's reached. This is just a patch PR, so let's not "work" the problem from here. Patch PRs are only for patches, and we require an issue for any significant work.
Open the issue from this one .......
https://learn.microsoft.com/en-us/aspnet/core/blazor/components/element-component-model-relationships?view=aspnetcore-8.0
Note that working on it will be a lower-priority task. There's still a large amount of 8.0 churn right now, so I think working on that would be pushed off at least a month (or three! 😆 ... probably three actually 🙈😆 I'm sorry to say) because the focus now is on shoring up all of the 8.0 scenarios that need critical work 🏃♂🏃♂🏃♂🏃♂🏃♂🏃♂.

I agree with @guardrex's assessment here - the content in the loop is effectively static in the sense that the same 10 items will always be rendered. If items between 0 and 9 could be dynamically removed, then @key might be useful so that the right state gets preserved. But this example doesn't demonstrate that, so readers might scratch their heads as to why @key is necessary. I think we should leave @key out and keep things how they are.