VS code sign in GitHub waits forever.

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
1.82.2
OS Version:
windows10

Steps to Reproduce:

I tried to link to my github account. the login site is https://github.com/login/oauth/authorize?client_id=01ab8ac9400c4e429b23&redirect_uri=https://vscode.dev/redirect&scope=read:org+user:email&state=vscode%253A%252F%252Fvscode.github-authentication%252Fdid-authenticate%253Fnonce%253D3988647bfd964d53%2526windowId%253D1
And then it just keeps login like that.

What happens when you hit Cancel and follow the next step?

What happens when you hit Cancel and follow the next step?

It just cancel and I am not able to connect to my github. I also try to push my commit. But get error like below:
fatal: An error occurred while sending the request.
fatal: The remote server returned an error: (407) Proxy Authentication Required.
I tried all the stockoverflow solutions but not work.

What happens when you hit Cancel and follow the next step?

And also, when I try to pull the update. I got error like below.
Your configuration specifies to merge with the ref 'refs/heads/main'
from the remote, but no such ref was fetched.
Thank you so much.

@chrmarti assigning you since this is sounding proxy related

If the proxy uses basic auth, you could use the http.proxy user setting to include username and password.

Happening with the Mac M1 also
Version: 1.82.3
Commit: fdb98833154679dbaa7af67a5a29fe19e55c2b73
Date: 2023-10-02T11:09:17.749Z
Electron: 25.8.1
ElectronBuildId: 24153832
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.0.0

@TylerLeonhardt Please help on this

@Shriansh2002 do you use a proxy? If not, open a new issue as that's not related to this.

@Shriansh2002 do you use a proxy? If not, open a new issue as that's not related to this.

How to use proxy for it ?

@Shriansh2002 can you please open a new issue with a screenshot of what you're seeing?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Changed the documentation of the subscribe argument to a more accurat…

The documentation of the subscribe argument in useSyncExternalStore was inaccurate. Now it says the following:
subscribe: A function that takes a single callback argument and subscribes it to the store. When the store changes, it should invoke the provided callback. This alerts React to potential changes in the store, but it will only cause the component to re-render if the snapshot, evaluated subsequently, has also changed. The subscribe function should return a function that removes the subscription when it's no longer needed. This helps prevent unnecessary re-renders and optimizes performance.

@Tinttori, I may be wrong so don't quote me on this, but I always thought "rendering" is the process of running the component functions, and React updates the DOM when the result of rendering is different.


Seeing inconsistent behaviour for different routes

Hi,
I'm encountering odd behaviour when flashing messages for different routes in my Axum server and I'm not quite sure what the issue is.
For example, I have these routes for registering new users (under /register):
pub async fn get_register(flash: IncomingFlashes, Extension(tera): Extension<Tera>) -> Html<String> {
   let mut context = tera::Context::new();
   if flash.len() == 1{
       context.insert("message", flash.iter().nth(0).unwrap().1);
   }

   Html(tera.render("register.html.tera", &context).unwrap())
}

pub async fn post_register(
    form: Form<Register>,
    mut flash: Flash,
    DatabaseConnection(conn): DatabaseConnection,
) -> Redirect {
// Try to register the user
// [......]
   Ok(_) => Redirect::to("/login"),

   Err(e) => {
      flash.error("Unable to register user");
      Redirect::to("/register")
   }
}

The flashing functionality works as expected for these routes (i.e. if there's an error during registration, the user is redirected and the flash message is shown. Upon refreshing the page, the flash message is cleared.
I then have some other routes (under /types/create) which are very similar to the above, but the flash message persists even after refreshing the page (and I can see the 'axum-flash' cookie stored in my browser after refreshing).
pub async fn get_create_type(flash: IncomingFlashes, Extension(tera): Extension<Tera>) -> Html<String>{
    let mut context = tera::Context::new();
    if flash.len() == 1{
        context.insert("message", flash.iter().nth(0).unwrap().1);
    }

    Html(tera.render("new_type.html.tera", &context).unwrap())
}

pub async fn post_create_type(
    form: Form<PostNewType>,
    mut flash: Flash,
    DatabaseConnection(conn): DatabaseConnection,
) -> Redirect {
   // Logic to create a new type
   // If a type with that name exists already, do:
   flash.error("Type already exists");
   Redirect::to("/types/create")
}

In these other routes, I've tried adding extra code to forcibly remove the 'axum-flash' cookie (I also tried using the other 2 types of CookieJar as well), but it still persists:
pub async fn get_create_type(flash: IncomingFlashes, jar: CookieJar, Extension(tera): Extension<Tera>) -> impl IntoResponse{
    let mut context = tera::Context::new();
    if flash.len() == 1{
        context.insert("message", flash.iter().nth(0).unwrap().1);
    }

    // Attempt to remove the 'axum-flash' cookie
    let updated_jar: CookieJar = jar.remove(Cookie::named("axum-flash"));

    (updated_jar, Html(tera.render("new_type.html.tera", &context).unwrap()))
}

From this page (/types/create) with the persistent cookie, if I then navigate back to the /register route and refresh the page, the cookie is cleared.
Can you think of any reason why in some instances the flash cookie is cleared, but in other cases it's not?
Let me know if you need more details on anything

Hm that sounds pretty weird. Not sure what's going on.
Are you able to make a minimal example that I can cargo run?

I've uploaded a minimal example here:
https://github.com/66OJ66/flash_testing
I think the error is related to paths with multiple segments i.e. when creating this example, using /do_a_thing as the route didn't cause any issues, but using /do/a/thing yields the error I mentioned above.
Possibly using SameSite = Strict in the flash cookie is causing this issue?

Hi, are you able to reproduce the example on your end using the code I linked to above?

I haven't had time to look more into it. Did you get it working?

Getting the exact same issue here. My login route is /accounts/login and the flash cookie does not seem to be cleared properly, so the flash message stays there forever. Changing the login route to /login makes it work perfectly.

Update: I've found a consistent workaround for the problem, by manually clearing the cookie:

async fn your_route_here(
    cookies: Cookies,
    flashes: IncomingFlashes,
) -> impl IntoResponse {

    //TODO use `flashes` here

    //Manually delete the cookie
    let mut flash_killer = Cookie::named("axum-flash");
    flash_killer.make_removal();
    flash_killer.set_path("/");
    cookies.add(flash_killer);

    //TODO return your response here

}

You need to use the Cookies extractor, not CookieJar (since the former overrides the latter).
Also ensure you've got the CookieManagerLayer added to your middleware stack, in addition to the axum_flash::layer. Even if you call with_cookie_manager(), the cookie manager will not be available to your own routes if you don't explicitly add it. For example:
 Router::new()
   .layer(
            ServiceBuilder::new()
                .layer(axum_flash::layer(cookie_key).with_cookie_manager())
                .layer(CookieManagerLayer::new())
        );

You need to use the Cookies extractor, not CookieJar (since the former overrides the latter).

You mean CookieJar from axum-extra? Yes those two are probably conflicting. I really should convert this library use axum-extra's CookieJar.

Also ensure you've got the CookieManagerLayer added to your middleware stack, in addition to the axum_flash::layer. Even if you call with_cookie_manager(), the cookie manager will not be available to your own routes if you don't explicitly add it. For example:

That doesn't sound right to me. The CookieManagerLayer is pushed here when you call with_cookie_manager so need need to add it twice. There must be something else going on.

Whoops didn't mean to close this.
I would encourage you to pull this repo down and debug it. I dunno when I'll have time to look into it.

Maybe it has something to do with the order of the layers?

Not sure 🤷

@davidpdrsn Why did you close this issue?

Huh I didn't do that knowingly.

Lol either something happened on github or my computer. When I press command+enter to submit a comment it hits the "Close with comment" button and not the "Comment" button 🤦

Can't extract cookies. Is CookieManagerLayer enabled?

I just got this error and it was caused by two incompatible tower-cookies versions.

@ssendev Thank you, that turned out to be the issue! axum-flash 0.4.0 uses tower-cookies 0.6.0, even though 0.7.0 is available.  I'll update my workaround.

Oh I didn't realize.

I've published axum-flash 0.5 which users tower-cookies 0.7.


Ambassador should warn about configuration that cannot be parsed

I'm not sure if this is a bug or a feature request as I cannot find a reference in official documentation.
I am deploying Ambassador with Helm and I was passing:
ambassadorConfig: |
  apiVersion: ambassador/v1
  kind: Module
  name: ambassador
    config:
      diagnostics:
        enabled: false

instead of:
ambassadorConfig: |
  apiVersion: ambassador/v1
  kind: Module
  name: ambassador
  config:
    diagnostics:
      enabled: false

...
Because of config block was not parsed properly, Ambassador was using default config.
Would it be better practice not to start Ambassador process at all in cases when you actually pass the configuration but the .yaml is not parsed correctly?

Why are you using ambassadorConfig instead of services.annotations?

@nbkrause I had an issue with services.annotations not being considered because of missing ambassador id. I made a small improvement in the documentation as I think it is critical (#1683).
Understanding documentation about Ambassador deployment is even harder if you are going with Helm. 😐
Closing this now, because warnings will appear if one uses services.annotations.


The tags of the blob are not updated after promoting one version

Storage Explorer Version: 1.16.0-dev
Build: 20200917.2
Branch: main
Platform/OS: Windows 10 / CentOS 7.6.1810 (Core) / MacOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Expand one storage account (make sure the 'blob tags' and the 'blob versioning' features are enabled)  -> Blob Containers.
Create a new blob container -> Upload one blob to it -> Add tags and metadatas for the blob.
Click 'Manage History -> Manage Versions' -> Update tags for the blob.
Select one version -> Click 'Promote Version'.
Check the Blob Tags dialog for the blob.

Expect Experience:
The tags of the blob are updated.
Actual Experience:
The tags of the blob are not updated.
More Info:
This issue doesn't reproduce after promoting one blob snapshot.

@JasonYeMSFT  Verified this issue on build 20201009.2.
The current behavior: The tags for the current version shows empty after promoting version.
Details please see the record:
https://msit.microsoftstream.com/video/e868a1ff-0400-9fb2-4566-f1eb0a1092f5

The standard copy api requires us to specify which tags to set instead of copying the tags over.

I will add this support for this when AzCopy supports uploading with tags.

This issue is fixed on build rel/1.16.0 20201030.5. so close it.


Support gitlab

I was looking at GitLab pipelines last night so decided to implement this since it was pretty trivial.
Fixes #121
Test pipeline with sigstore: https://gitlab.com/jku/test-oidc/-/blob/main/.gitlab-ci.yml,
Log from running that: https://gitlab.com/jku/test-oidc/-/jobs/5337879401
Notes:

Called it detect_env_var instead of detect_gitlab since there's nothing really gitlab specific about it
The fact that we now have to require the user to do something extra (configure the variable correctly) is annoying but there's not much we can do except maybe provide fedback to GitLab if they ask
there's an unrelated added test for buildkite
We don't check if the variable actually contains a token. We don't do that in other detectors either but arguably the chance of errors here is somewhat higher...

Called it detect_env_var instead of detect_gitlab since there's nothing really gitlab specific about it

I could be convinced otherwise, but IMO we should make this detect_gitlab and so similar checks for it as we do for other detectors (specifically, sniffing for GITLAB_CI).
My reasoning: each of the detectors is semi-intentionally bound to a single service, even though some services might copy each others' APIs (e.g. FooCI could use the same technique as GitHub Actions). Having an explicit detect_gitlab allows a user to communicate that they're intentionally obtaining a GitLab identity token.
(One idea: maybe we could have detect_env_var and detect_gitlab, with detect_gitlab wrapping the former?)

We don't check if the variable actually contains a token. We don't do that in other detectors either but arguably the chance of errors here is somewhat higher...

I'd be okay with adding this check, both here and elsewhere 🙂

(One idea: maybe we could have detect_env_var and detect_gitlab, with detect_gitlab wrapping the former?)

I like this idea!

IMO we should make this detect_gitlab and so similar checks for it as we do for other detectors (specifically, sniffing for GITLAB_CI).

That makes sense

Changed detect_env_var() to detect_gitlab(), see updated PR description

/gcbrun

Can you also update the first line of the README to include Gitlab?


Using 3rd party libraries when mocking python lambda

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
No response
Amplify CLI Version
5.3.0
What operating system are you using?
Ubuntu 20.04
Amplify Categories
function
Amplify Commands
Not applicable
Describe the bug
I try to mock a python lambda function which uses the numpy library.
I added import numpy
to the generated src/index.py.
I modified the generated Pipfile to include
[packages]
src = {editable = true, path = "./src"}
numpy = "*"

Then I ran:
amplify mock function testampli59bb9ee0
Expected behavior
Local pipenv installs numpy and I can run the lambda using numpy features.
Reproduction steps

amplify init
amplify add function
modify Pipfile to list numpy as a dependency
modify index.py to import numpy
amplify mock function <>

GraphQL schema(s)

# Put schemas below this line

Log output

martin@simba:/tmp/testampli/amplify/backend/function/testampli59bb9ee0$ amplify mock function testampli59bb9ee0
? Provide the path to the event JSON object relative to /tmp/testampli/amplify/backend/function/testampli59bb9ee0 src/event.json
Ensuring latest function changes are built...
Courtesy Notice: Pipenv found itself running within a virtual environment, so it will automatically use that environment, instead of creating its own for any project. You can set PIPENV_IGNORE_VIRTUALENVS=1 to force pipenv to ignore that environment and create its own instead. You can set PIPENV_VERBOSITY=-1 to suppress this warning.
Pipfile.lock not found, creating...
Locking [dev-packages] dependencies...
Locking [packages] dependencies...
Building requirements...
Resolving dependencies...
✔ Success!
Updated Pipfile.lock (bbddef)!
Installing dependencies from Pipfile.lock (bbddef)...
  🐍   ▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉ 1/1 — 00:00:07
Starting execution...
Warning: the environment variable LANG is not set!
We recommend setting this in ~/.profile (or equivalent) for proper expected behavior.
Creating a virtualenv for this project...
Pipfile: /tmp/testampli/amplify/backend/function/testampli59bb9ee0/Pipfile
Using /usr/bin/python3.8 (3.8.10) to create virtualenv...
⠇ Creating virtual environment...created virtual environment CPython3.8.10.final.0-64 in 431ms
  creator CPython3Posix(dest=/home/martin/.local/share/virtualenvs/testampli59bb9ee0-tGTOLLXm, clear=False, no_vcs_ignore=False, global=False)
  seeder FromAppData(download=False, pip=bundle, setuptools=bundle, wheel=bundle, via=copy, app_data_dir=/home/martin/.local/share/virtualenv)
    added seed packages: pip==21.2.2, setuptools==57.4.0, wheel==0.36.2
  activators BashActivator,CShellActivator,FishActivator,PowerShellActivator,PythonActivator

✔ Successfully created virtual environment!
Virtualenv location: /home/martin/.local/share/virtualenvs/testampli59bb9ee0-tGTOLLXm
Traceback (most recent call last):
  File "/home/martin/.amplify/lib/amplify-python-function-runtime-provider/shim/shim.py", line 35, in <module>
    main(sys.argv[1:])
  File "/home/martin/.amplify/lib/amplify-python-function-runtime-provider/shim/shim.py", line 22, in main
    spec.loader.exec_module(handlerModule)
  File "<frozen importlib._bootstrap_external>", line 848, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/tmp/testampli/amplify/backend/function/testampli59bb9ee0/src/index.py", line 2, in <module>
    import numpy
ModuleNotFoundError: No module named 'numpy'
testampli59bb9ee0 failed with the following error:
Error: Command failed with exit code 1: pipenv run python3 /home/martin/.amplify/lib/amplify-python-function-runtime-provider/shim/shim.py /tmp/testampli/amplify/backend/function/testampli59bb9ee0/src/index.py handler
Warning: the environment variable LANG is not set!
We recommend setting this in ~/.profile (or equivalent) for proper expected behavior.
Creating a virtualenv for this project...
Pipfile: /tmp/testampli/amplify/backend/function/testampli59bb9ee0/Pipfile
Using /usr/bin/python3.8 (3.8.10) to create virtualenv...
⠇ Creating virtual environment...created virtual environment CPython3.8.10.final.0-64 in 431ms
  creator CPython3Posix(dest=/home/martin/.local/share/virtualenvs/testampli59bb9ee0-tGTOLLXm, clear=False, no_vcs_ignore=False, global=False)
  seeder FromAppData(download=False, pip=bundle, setuptools=bundle, wheel=bundle, via=copy, app_data_dir=/home/martin/.local/share/virtualenv)
    added seed packages: pip==21.2.2, setuptools==57.4.0, wheel==0.36.2
  activators BashActivator,CShellActivator,FishActivator,PowerShellActivator,PythonActivator

✔ Successfully created virtual environment!
Virtualenv location: /home/martin/.local/share/virtualenvs/testampli59bb9ee0-tGTOLLXm
Traceback (most recent call last):
  File "/home/martin/.amplify/lib/amplify-python-function-runtime-provider/shim/shim.py", line 35, in <module>
    main(sys.argv[1:])
  File "/home/martin/.amplify/lib/amplify-python-function-runtime-provider/shim/shim.py", line 22, in main
    spec.loader.exec_module(handlerModule)
  File "<frozen importlib._bootstrap_external>", line 848, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/tmp/testampli/amplify/backend/function/testampli59bb9ee0/src/index.py", line 2, in <module>
    import numpy
ModuleNotFoundError: No module named 'numpy'
    at makeError (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/execa/lib/error.js:59:11)
    at handlePromise (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/execa/index.js:114:26)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at Object.pythonInvoke (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-python-function-runtime-provider/src/util/invokeUtil.ts:33:22)
    at Object.timeConstrainedInvoker (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-util-mock/src/func/index.ts:67:12)
    at Object.start (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-util-mock/src/func/index.ts:48:20)
    at Object.run (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-util-mock/src/commands/mock/function.ts:14:5)
    at Object.executeAmplifyCommand (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-util-mock/src/amplify-plugin-index.ts:8:3)
    at executePluginModuleCommand (/usr/local/lib/node_modules/@aws-amplify/cli/src/execution-manager.ts:177:3)
    at Object.executeCommand (/usr/local/lib/node_modules/@aws-amplify/cli/src/execution-manager.ts:29:5)
    at Object.run (/usr/local/lib/node_modules/@aws-amplify/cli/src/index.ts:151:5) {
  shortMessage: 'Command failed with exit code 1: pipenv run python3 /home/martin/.amplify/lib/amplify-python-function-runtime-provider/shim/shim.py /tmp/testampli/amplify/backend/function/testampli59bb9ee0/src/index.py handler',
  command: 'pipenv run python3 /home/martin/.amplify/lib/amplify-python-function-runtime-provider/shim/shim.py /tmp/testampli/amplify/backend/function/testampli59bb9ee0/src/index.py handler',
  exitCode: 1,
  signal: undefined,
  signalDescription: undefined,
  stdout: '',
  stderr: 'Warning: the environment variable LANG is not set!\n' +
    'We recommend setting this in ~/.profile (or equivalent) for proper expected behavior.\n' +
    'Creating a virtualenv for this project...\n' +
    'Pipfile: /tmp/testampli/amplify/backend/function/testampli59bb9ee0/Pipfile\n' +
    'Using /usr/bin/python3.8 (3.8.10) to create virtualenv...\n' +
    '\r⠋ Creating virtual environment...\b\r⠙ Creating virtual environment...\b\r⠹ Creating virtual environment...\b\r⠸ Creating virtual environment...\b\r⠼ Creating virtual environment...\b\r⠴ Creating virtual environment...\b\r⠦ Creating virtual environment...\b\r⠧ Creating virtual environment...\b\r⠇ Creating virtual environment...created virtual environment CPython3.8.10.final.0-64 in 431ms\n' +
    '  creator CPython3Posix(dest=/home/martin/.local/share/virtualenvs/testampli59bb9ee0-tGTOLLXm, clear=False, no_vcs_ignore=False, global=False)\n' +
    '  seeder FromAppData(download=False, pip=bundle, setuptools=bundle, wheel=bundle, via=copy, app_data_dir=/home/martin/.local/share/virtualenv)\n' +
    '    added seed packages: pip==21.2.2, setuptools==57.4.0, wheel==0.36.2\n' +
    '  activators BashActivator,CShellActivator,FishActivator,PowerShellActivator,PythonActivator\n' +
    '\n' +
    '\b✔ Successfully created virtual environment! \n' +
    'Virtualenv location: /home/martin/.local/share/virtualenvs/testampli59bb9ee0-tGTOLLXm\n' +
    'Traceback (most recent call last):\n' +
    '  File "/home/martin/.amplify/lib/amplify-python-function-runtime-provider/shim/shim.py", line 35, in <module>\n' +
    '    main(sys.argv[1:])\n' +
    '  File "/home/martin/.amplify/lib/amplify-python-function-runtime-provider/shim/shim.py", line 22, in main\n' +
    '    spec.loader.exec_module(handlerModule)\n' +
    '  File "<frozen importlib._bootstrap_external>", line 848, in exec_module\n' +
    '  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed\n' +
    '  File "/tmp/testampli/amplify/backend/function/testampli59bb9ee0/src/index.py", line 2, in <module>\n' +
    '    import numpy\n' +
    "ModuleNotFoundError: No module named 'numpy'",
  failed: true,
  timedOut: false,
  isCanceled: false,
  killed: false
}
Finished execution.

Additional information
No response

Hi @martinmeinke
I executed the same steps and it is working for me. The one problem I see in error logs is this
Warning: the environment variable LANG is not set!
We recommend setting this in ~/.profile (or equivalent) for proper expected behavior.

Can you try this and again try to mock this function?
Let me know if this works for you?

Explicitly setting
export LANG="en_US.UTF-8"
export LC_ALL="en_US.UTF-8"
export LC_CTYPE="en_US.UTF-8"

in my .zshrc does not solve the problem.

Hi @martinmeinke
One more thing to debug is , When you do amplify mock function , pipenv creates virtual environment to install project dependencies in your local. : /home/martin/.local/share/virtualenvs/testampli59bb9ee0-tGTOLLXm. Can you check on this path under sitePackage if numpy is present?

martin@simba:<>$ ll site-packages/
total 60
drwxrwxr-x 11 martin martin 4096 Aug 12 04:41 ./
drwxrwxr-x  3 martin martin 4096 Aug 10 01:40 ../
drwxrwxr-x  2 martin martin 4096 Aug 10 01:40 _distutils_hack/
-rw-rw-r--  1 martin martin  152 Aug 10 01:40 distutils-precedence.pth
drwxrwxr-x  4 martin martin 4096 Aug 10 01:40 pip/
drwxrwxr-x  2 martin martin 4096 Aug 10 01:40 pip-21.2.2.dist-info/
-rw-rw-r--  1 martin martin    0 Aug 10 01:40 pip-21.2.2.virtualenv
drwxrwxr-x  5 martin martin 4096 Aug 10 01:40 pkg_resources/
drwxrwxr-x  2 martin martin 4096 Aug 12 04:41 __pycache__/
drwxrwxr-x  6 martin martin 4096 Aug 10 01:40 setuptools/
drwxrwxr-x  2 martin martin 4096 Aug 10 01:40 setuptools-57.4.0.dist-info/
-rw-rw-r--  1 martin martin    0 Aug 10 01:40 setuptools-57.4.0.virtualenv
-rw-rw-r--  1 martin martin   18 Aug 10 01:40 _virtualenv.pth
-rw-rw-r--  1 martin martin 5662 Aug 10 01:40 _virtualenv.py
drwxrwxr-x  4 martin martin 4096 Aug 10 01:40 wheel/
drwxrwxr-x  2 martin martin 4096 Aug 10 01:40 wheel-0.36.2.dist-info/
-rw-rw-r--  1 martin martin    0 Aug 10 01:40 wheel-0.36.2.virtualenv

Hey @martinmeinke :wave: just wanted to follow up and see if you are still experiencing this issue with the latest version of the CLI? I am also unable to reproduce following the provided steps.
Here is the output of my amplify mock function execution
> amplify mock function
⚠ Amplify command hooks are disabled in the current execution environment.
⚠ See https://docs.amplify.aws/cli/usage/command-hooks/ for more information.
? Provide the path to the event JSON object relative to /Users/josef/Documents/projects/aws-amplify/reproductions/7924/ampl
ify/backend/function/7924f5e86d1a src/event.json
Ensuring latest function changes are built...
Creating a virtualenv for this project...
Pipfile: /Users/josef/Documents/projects/aws-amplify/reproductions/7924/amplify/backend/function/7924f5e86d1a/Pipfile
Using /usr/bin/python3 (3.8.2) to create virtualenv...
⠹ Creating virtual environment...created virtual environment CPython3.8.2.final.0-64 in 874ms
  creator CPython3macOsFramework(dest=/Users/josef/.local/share/virtualenvs/7924f5e86d1a-q9HUaz1q, clear=False, no_vcs_ignore=False, global=False)
  seeder FromAppData(download=False, pip=bundle, setuptools=bundle, wheel=bundle, via=copy, app_data_dir=/Users/josef/Library/Application Support/virtualenv)
    added seed packages: pip==21.2.4, setuptools==58.1.0, wheel==0.37.0
  activators BashActivator,CShellActivator,FishActivator,NushellActivator,PowerShellActivator,PythonActivator

✔ Successfully created virtual environment!
Virtualenv location: /Users/josef/.local/share/virtualenvs/7924f5e86d1a-q9HUaz1q
Pipfile.lock not found, creating...
Locking [dev-packages] dependencies...
Locking [packages] dependencies...
Building requirements...
Resolving dependencies...
✔ Success!
Updated Pipfile.lock (bbddef)!
Installing dependencies from Pipfile.lock (bbddef)...
  🐍   ▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉ 2/2 — 00:00:12
To activate this project's virtualenv, run pipenv shell.
Alternatively, run a command inside the virtualenv with pipenv run.
Starting execution...

received event:
{'test': 'event'}
{"statusCode": 200, "headers": {"Access-Control-Allow-Headers": "*", "Access-Control-Allow-Origin": "*", "Access-Control-Allow-Methods": "OPTIONS,POST,GET"}, "body": "\"Hello from your new Amplify Python lambda!\""}
Result:
{
  "statusCode": 200,
  "headers": {
    "Access-Control-Allow-Headers": "*",
    "Access-Control-Allow-Origin": "*",
    "Access-Control-Allow-Methods": "OPTIONS,POST,GET"
  },
  "body": "\"Hello from your new Amplify Python lambda!\""
}
Finished execution.


[ApiAuth] Update oidc-client dependency to 1.9.4

Updates the oidc-client dependency to 1.9.4 RTM that was published last Friday.

Failure seems to have been transient networking stuff, should pass on re-run.

No, still doesn't have the final version

This is ready to go to tactics now

Thanks - looks like there are some test issues.  Once those are resolved, I'll send mail.

@Pilchie I'm on it. I'll ping you here.

Looks like @mkArtakMSFT already gave them a heads up.

The migrations need to be regenerated


Normalize logical id of alias part in Lambda roles

Fixes #68
Aliases can now include characters matching [a-zA-Z0-9\-+_]+.
To generate a valid role logical id for the alias, the alias name is normalized by using the following table:
- => Dash
+ => Plus
_ => Uscore

This should work as it is highly unlikely that someone uses one of the replacement strings in real alias names.

Locally ESLint as well as the testsuite passes. Will have to check why Travis fails.

Coverage decreased (-0.1%) to 74.174% when pulling 557d55a297eb4ef625dad1a9fbc38328a5f526df on normalize-role-name into e0215d2694b743567e58a48746129b33af13cc01 on master.

Had to upgrade eslint and the testsuite. However unit tests for the normalized roles are yet missing.

@mbruning24 @arabold - Could you do a test with HyperBrain/serverless-aws-alias#normalize-role-name and check if aliases that include dashes work now?

Coverage decreased (-0.09%) to 74.218% when pulling 4606aeae7668b71936bdc587982a6ac3b043d125 on normalize-role-name into e0215d2694b743567e58a48746129b33af13cc01 on master.

Coverage increased (+0.6%) to 74.894% when pulling 03c823dc9d887708e9ed5a25ac240ab90aef09d4 on normalize-role-name into e0215d2694b743567e58a48746129b33af13cc01 on master.

@HyperBrain sorry it's been a hectic couple weeks. I tested this out. Error looks great BTW!
However, I found a HUGE bummer. API Gateway doesn't allow hyphens in the stage name!  I mean, come one AWS, it's a URL for crying out loud... The regex for that is /a-zA-Z0-9_/, so underscores are allowed by APIG, but ironically not by the regex! Maybe just switch that bad boy from hyphen to underscore and you'll have really done all you can. Hopefully AWS lets hyphens in stage names sometime in the future.

@mbruning24 That's indeed ridiculous. I'll add a light normalization to the stage name creation which only will transform the hyphen as you proposed.
However I have to add that to the docs, because having both aliases my-alias and my_alias would lead to a crash then. Maybe we should even emit a warning in case - or _ are used in an alias.
The hyphen issue on AWS side would imo justify to open a issue/case on their side. Stage names should comply with the official URL rules.

@HyperBrain I've opened a support case with AWS in our company account. For now, I'd be happy with just underscore though, if you wouldn't mind changing that.

No problem. Will do.
Mark Bruning<EMAIL_ADDRESS>schrieb am Fr., 3. Nov. 2017, 8:05
nachm.:

@HyperBrain https://github.com/hyperbrain I've opened a support case
with AWS in our company account. For now, I'd be happy with just underscore
though, if you wouldn't mind changing that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/HyperBrain/serverless-aws-alias/pull/77#issuecomment-341798360,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFRM3nTEB8osjCBkYqc2nGGHelxV50ZSks5sy2PxgaJpZM4QKTQf
.

@mbruning24 Sorry for the delay. Now, for stage names the hyphens are replaced by underscores.
Did you get any valuable feedback from AWS yet?

Coverage increased (+0.6%) to 74.894% when pulling e41fc075af8641358a8348f44b9f39b7438f8a28 on normalize-role-name into e0215d2694b743567e58a48746129b33af13cc01 on master.


Merge Freeze: Stabilize end to end tests on May 19 2015

Individual detailed tracking bugs will be added here shortly.
Further detail on https://groups.google.com/forum/#!topic/kubernetes-dev/gGXraJNAcaw

Since 4AM today, our end-to-end continuous integration tests have failed 9 of 13 runs, for a variety of reasons.  Here is a sample of recent failures:
Shell tests that services.sh passes
Waiting for endpoints to propagate
Endpoints did not propagate in time
Stopping service 'service-8805'
kubectl guestbook should create and stop a working application
/go/src/github.com/GoogleCloudPlatform/kubernetes/_output/dockerized/go/src/github.com/GoogleCloudPlatform/kubernetes/test/e2e/kubectl.go:128
Frontend service did not start serving content in 600 seconds.
ReplicationController should serve a basic image on each replica with a public image
/go/src/github.com/GoogleCloudPlatform/kubernetes/_output/dockerized/go/src/github.com/GoogleCloudPlatform/kubernetes/test/e2e/rc.go:47
Did not get expected responses within the timeout period of 120.00 seconds.
Cluster level logging using Elasticsearch should check that logs from pods on all nodes are ingested into Elasticsearch
/go/src/github.com/GoogleCloudPlatform/kubernetes/_output/dockerized/go/src/github.com/GoogleCloudPlatform/kubernetes/test/e2e/es_cluster_logging.go:46
Expected error:
<*errors.errorString | 0xc208a46200>: {
s: "failed to unmarshal Elasticsearch response: invalid character 'E' looking for beginning of value",
}
failed to unmarshal Elasticsearch response: invalid character 'E' looking for beginning of value
not to have occurred
kubectl update-demo should do a rolling update of a replication controller
/go/src/github.com/GoogleCloudPlatform/kubernetes/_output/dockerized/go/src/github.com/GoogleCloudPlatform/kubernetes/test/e2e/kubectl.go:106
Timed out after 300 seconds waiting for name=update-demo pods to reach valid state

#7972 is a known cause of occasional failures, but not a recent regression.

#7974 is a big change, merged at the time stability dropped significantly, and never had e2e tests run against it.  Busy evaluating whether to roll it back.

Most failures I saw is timeouts. Maybe some regression on our performance?

We have some evidence to suggest that the reboot test is contributing to overall e2e test flakyness:
That merged at 5:27pm PST on Friday.
https://github.com/GoogleCloudPlatform/kubernetes/pull/7580

#7974 is a big change, merged at the time stability dropped significantly, and never had e2e tests run against it. Busy evaluating whether to roll it back.

#7974 is significant, but it is focused exclusively on image pulls.  All the API changes are fully deterministic and optional.  I won't argue that it can't be the cause of the failures, but is there anything more probative?  Image pull failures in particular?  In the absence of ImagePullSecrets, the exact same keyring is used: https://github.com/GoogleCloudPlatform/kubernetes/pull/7974/files#diff-865dd182600da4f2a68f693afb7ca65fR206 .

#6949 is another large change that went in today, though I don't know if it is likely to be causing the problems we're having today.

I ran e2es multiple times yesterday and today on #6949 (but that doesn't mean that it works in conjunction with the other changes that went in at the same time).

/subscribe

FYI: the cluster level logging test (something I spent a lot of time and energy on) was broken by someone else. I take the blame for many things, but not this.

Do we have a status update?  We have multiple PR's all blocked this.

We are actively working on this, and think we are close to resolving the issue.  No more than a few more hours of freeze.

@quinton-hoole many thanks!

Leading candidate is #6949 assigning CIDRs that don't match the forwarding rules. We will probably roll back shortly.

Problem is here: https://github.com/GoogleCloudPlatform/kubernetes/blob/1f4172d23c4e949362ba6c96f5e7b1aa987e1be6/pkg/cloudprovider/gce/gce.go#L574
GCE cloud provider ignores conflict errors.
Problem:

A CIDR is selected.
The forwarding route is created.
The attempt to write the CIDR to the node fails, probably due to the node status having been written, changing the resourceVersion.
The second time around, the opposite CIDR is selected.
Creating the forwarding rule fails with a conflict, which is silently ignored.
Then the wrong CIDR is successfully written to the node.

After that your cluster is pretty hosed.

Then that problem is definitely on me.
#8164 fixes it.

cc/ @fgrzadkowski

To be clear, I'm not advocating merging #8164 to fix this. We should remove the hacky "assume it's correct if it's already there" logic. We may leak routes until we get #8164 in.

We think this will fix it, but give us some time to make sure the tests have stabilized.

Finally our jenkin tests are finished, but still have issues with logging and density test. Better than before.

As for Density test - the test failed because the latency metrics were too high. This is NOT a correctness problem - it's just performance problem.

Might be related to the underlying issue - our scalability test suite is still failing and basically the problem is that the cluster is not starting correctly. The last two runs failed during cluster validation on:
0;31mDetected 91 ready nodes, found 91 nodes out of expected 100. Your cluster may not be working.

/sub

We have managed to stabilize the system sufficiently to lift the merge freeze.  It will take about to a day to systematically merge the PR backlog.
A few end to end tests remain unreliable (see below).  If you see any of these tests fail in your local testing, you can ignore the failures until further notice.  We hope to have these stabilized within the next day or two, at which point I will update this thread.
Thanks for your patience.
Q
Cluster level logging using Elasticsearch should check that logs from pods on all nodes are ingested into Elasticsearch

Cluster health has bad status: red

Services should be able to create a functioning external load balancer

service external-lb-test in namespace e2e-tests-service-0-2c72586e-ab35-4f80-a30c-1c169f8272a3 doesn't have a public IP after 240.00 seconds

Services should correctly serve identically named services in different namespaces on different external IP addresses

service s0 in namespace e2e-tests-service-0-59b2c3ca-71d5-4fcb-9a7b-e92e2909a1e4 doesn't have a public IP after 240.00 seconds

We lifted merge freeze, but there are tons of backlog needed to merge. Please continue reviewing prs, but leave on-call, which is me today to merge, so that we can rate limit the merges and avoid another merge freeze. Thanks!


[cxx-interop][SwiftCompilerSources] Fix conversion between std::string and Swift.String

This fixes a dangling pointer issue when creating a Swift.String from std::string.
Also fixes a warning:
warning: variable 's' was never mutated; consider changing to 'let' constant
    var s = SILBasicBlock_debugDescription(bridged)
    ~~~ ^
    let

rdar://92963081
rdar://93053488

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test Linux


Automator: update<EMAIL_ADDRESS>dependency in<EMAIL_ADDRESS>

Generated by Automator - 2023-02-15T23:03:47+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio_release-1.15
46c69a0b6beb3b17a444be759711087d31f8507b
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio_release-1.15
46c69a0b6beb3b17a444be759711087d31f8507b
link
unknown
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-security-multicluster


docs: Data Balance documentation updates

Updated data balance main documentation to integrate the feedback from the fairness committee in MS. Data Balance is a tool, but it does not completely solves for fairness. In combination with other tools it helps reduce fairness related harms, but there is no way to way a completely unbiased dataset or model.
Updated the notebook to reduce the amount of measure explanations and better introduce the goals of data balance analysis.
AB#1839731

/azp run

Codecov Report

Merging #1540 (9c182df) into master (3bd5bef) will decrease coverage by 52.80%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #1540       +/-   ##
===========================================
- Coverage   83.74%   30.93%   -52.81%
===========================================
  Files         301      272       -29
  Lines       15337    14051     -1286
  Branches      707      707
===========================================
- Hits        12844     4347     -8497
- Misses       2493     9704     +7211

Impacted Files
Coverage Δ

...m/src/main/scala/com/microsoft/lightgbm/SWIG.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

.../org/apache/spark/ml/LimeNamespaceInjections.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

.../microsoft/azure/synapse/ml/io/binary/Binary.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

.../microsoft/azure/synapse/ml/lime/BreezeUtils.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...microsoft/azure/synapse/ml/codegen/Wrappable.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...rosoft/azure/synapse/ml/param/WrappableParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...soft/azure/synapse/ml/core/contracts/Metrics.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...spark/sql/types/injections/MetadataUtilities.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...t/azure/synapse/ml/automl/DefaultHyperparams.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...zure/synapse/ml/lightgbm/TaskTrainingMethods.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 217 more

Continue to review full report at Codecov.

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


⚠ ClamAV has degraded performance

In 5e6d729, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 241 ms

Resolved: ClamAV performance has improved in 4cf52ff.


support for 30-Seconds-of-STL

Any chances the extension could provide Code Snippets from 30-Seconds-Of-STL .

@Bhupesh-V Hey man, I mean sure. Give me a day or two, to write the documentation for contributions regarding new snippets and snippet libraries, and you can branch-out and start adding the snippets.
Just a small info, would this be called the c++ library or stl library in the interface. I want to keep the extension "self-explanatory" to new comers and experienced developers as well. C++ would be ideal name but I assume you gave your repo STL name for a good reason (Standard Template Library). Now I'm not familiar with c++ to much so your input on this would be great.

@petrovicstefanrs :+1:
I guess naming it C++ (in your extension) would be a good idea because people don't easily recognise STL
(Well there was NO specific reason to name it 30-Seconds-Of-STL)
Just ping me when you are done adding CONTRIBUTING guidelines I will make changes in my project accordingly :smiley:

@Bhupesh-V Great, I'll do the documentation in the next day or two and ping you here once I do so you don't have to be on the lookout for it.

Hey @petrovicstefanrs any updates ??

@Bhupesh-V Sorry took me longer that expected, life got in the way. I've finished the documentation for all the currently present snippets. If you want you can start implementing the c++ snippets based on how the current ones are implemented. Just make sure to branch out of dev branch and name your branch feature/cpp_snippets

Ok i have renamed the project to 30-seconds-of-cpp
So now what how would this be integrated with the extension ??


How do I unify the path of the SSH private key in the Windows and Mac versions ?

Because of the SSH private key file path in the configuration file, I had to split the configuration file to synchronize the Windows version and the Mac version, which was a headache.
So, How do I unify the path of the SSH private key in the Windows and Mac versions ?

There's no built-in way to go around this unfortunately - only absolute paths are allowed in the privateKey field. I'll try to either add support for home-relative (~) paths or modify the sync plugin to not overwrite key paths

Set two SSH private key fields(eg. winPrivateKey and linuxPrivateKey) for Windows and MacOS/Linux, and determine which field to use during SSH connection according to the OS type. Such a set of configuration files can be used across platforms~
How about this solution?

Set two SSH private key fields(eg. winPrivateKey and linuxPrivateKey) for Windows and MacOS/Linux, and determine which field to use during SSH connection according to the OS type. Such a set of configuration files can be used across platforms~
How about this solution?

@Eugeny
Add a few lines of code to the following two files to achieve the requirements I mentioned above. After I compile it locally, I can use it. Can it be implemented in the next version?

terminus-ssh/src/api.ts

export interface SSHConnection {
    ......
    winPrivateKey?: string
    linuxPrivateKey?: string
    ......
)

terminus-ssh/src/services/ssh.service.ts

async loadPrivateKeyForSession (session: SSHSession): Promise<string|null> {
    let privateKey: string|null = null
    let privateKeyPath = session.connection.privateKey
    let winPrivateKeyPath = session.connection.winPrivateKey
    let linuxPrivateKeyPath = session.connection.linuxPrivateKey

    if (this.hostApp.platform === Platform.Windows && winPrivateKeyPath) {
        privateKeyPath = winPrivateKeyPath
    } else if (this.hostApp.platform === Platform.macOS && linuxPrivateKeyPath) {
        privateKeyPath = linuxPrivateKeyPath
    }
    ......
}

I've replaced privateKey with a privateKeys path array - the first existing key will be used.


Fix code signing

There’s no need to define code signing for frameworks. When hardcoding iOS Developer, this would prevent this framework from getting signed for Enterprise for example.
Also fixed Unit Tests which didn't build on Xcode 10.1

Thanks @c0diq for your contribution.


Force tls1.2 with HttpUser and FastHttpUser?

Guys,
Is there a way to force tls1.2 with HttpUser and FastHttpUser?
Need for FastHttpUser, but will be interesting to see for HttpUser too.
I'm sure, it's possible.
Googled for more than hour, cannot find anything relevant...
Gotta be something around
geventhttpclient.connectionpool.SSLConnectionPool
and
context = ssl.SSLContext(ssl.PROTOCOL_TLSv1_2)
right?

Hi! For HttpUser, check the documentation for requests (self.client is an instance of requests.Session, maybe you can change it's behaviour by accessing that)
For FastHttpUser your guess is as good as mine. StackOverflow is a better place for questions btw, as we want to keep the github ticket list as small as possible :)


Update documentation

If you haven't already, please read through our contributing guidelines before opening your PR
What is the purpose of this PR?
Updates the ReadMe for the new deepcell script, also rewords some of the explanation of how updating works.
Closes #263 and adds more explicit instructions for what needs to be modified in example notebook

Giving a talk at 9:00 today otherwise I'd wait for your comments Adam :)


[red-knot] Add tests for member lookup on union types

Summary

Write tests for member lookups on union types
Remove TODO comment

part of: #14022
Test Plan
New MD tests

I haven't verified that this was previously untested but I assume you have 😄

Yes. Commenting out one of the two possibly_unbound = true; statements in Type::member will cause one of the new unit tests to fail, while it went unnoticed previously.


[RFC] Rename LineContinationHandler to LineContinuationHandler

Summary

I believe LineContinationHandler is misspelled and meant to be LineContinuationHandler (notice the u).
Detailed Explanation

Drawbacks, Rationale, and Alternatives

Would break backwards-compatibility for anyone using the rustpython-parser crate.
Unresolved Questions

This sounds good! We aren't really too concerned with a stable API for our crates yet, so there shouldn't be a problem with changing the name.


osteocyte dendritic processes

ADD:
abnormal osteocyte dendritic process morphology
DEF: any structural anomaly of the long, slender cytoplasmic processes by which mature osteocytes communicate with one another, receive mechanosensory signals and participate in the regulation of bone turnover; these processes radiate from the osteocyte cell body, run along narrow canaliculi, and are linked to other neighboring osteocytes processes by gap junctions, as well as to cytoplasmic processes of osteoblasts and bone lining cells on the bone surface
SYN: abnormal osteocyte cytoplasmic process morphology
SYN: abnormal osteocyte process morphology
SOURCES:
https://www.hindawi.com/journals/bmri/2015/421746/
PMID:24419319

abnormal osteocyte dendritic process morphology MP:0030487


Typo "@dataset().SinkTableName"→"@dataset().SinkTableName"

bold with escape characters
https://docs.microsoft.com/en-us/azure/data-factory/tutorial-incremental-copy-multiple-tables-portal

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Verified the update.
@hyoshioka0128 Thanks for submitting the fix.


Request to Enhance README.md for Improved Clarity and Usability

Hi @RomeoIndiaJulietUniform , I came across your project and noticed that the README.md could be made more user-friendly and detailed. I'd love to contribute by enhancing the overall structure, adding clearer instructions, and improving readability for both new contributors and users. This could include refining setup steps, usage examples, and adding any missing or useful information.
Please let me know if you're open to this, and I’ll be happy to start working on it right away!

Hii @MadhurSaluja thanks again for looking in the project deeply. I was looking for an updated Readme anyways. I will share you the project structure, pls feel free to refer this links (Twitter link - https://x.com/TraversalTribe/status/1831064932466803178) and (Excali draw link - https://t.co/NTjHOfdP3j) for better Understanding.
This project is an implementation of automating Aquarium ecosystems by controlling and montinoring various Actutators and Sensors through web keeping your pet fishes healthy while staying 1000 of kms.
As of today, the aquarium industry is largely dominated by proprietary software. This project aims to harness the power of modern, highly scalable, open, and robust open-source development to empower the hobbyist aquarist community.
I will assign you this issue, pls create a pull request whenever you are ready.  Happy Coding 🙂
I will raise another issue soon to work on the docs part of the project for the science and engineering behind mainting the aquarium, probably another repo dedicated for the web docs.


%cmi_equationint_% does not work

I want to use this format in conditions:
candiotionName:
type: EQUALS
input: '%cmi_equationint_{bedwars_game_test_minplayers}-{bedwars_game_test_players}%'
value: '0'
but it's not working

Hey, what's the output if you run /papi parse me %cmi_equationint_{bedwars_game_test_minplayers}-{bedwars_game_test_players}% in-game?

Close please. I'm wrong. It works fine. Im sorry.


⚠ ClamAV has degraded performance

In bdeab13, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 139 ms

Resolved: ClamAV performance has improved in b12772f.


Autoplaying on mobile browsers

Is it possible to pass the following props:

{muted autoplay playsinline}

to Reactplayer similar to HTML5 player so it does autplay on mobile browsers.

Or how can add iphone-inline-video lib to React-player

@CookPete Any feedback on this?

Attributes can be added to the HTML5 players like so:
<ReactPlayer
  url='file.mp4'
  fileConfig={{ attributes: { controls: true } }}
/>

However, beware of mobile considerations with regards to autoplaying stuff.


🎡 Analytical Platform Compute EKS

This pull request:

Adds ghcr.io/ministryofjustice/devcontainer-feature/kubernetes to dev container
Adds CloudWatch Log Group for EKS cluster logging
Adds CloudWatch Log Group for AMP logging
Adds EBS and EFS CSI drivers
Creates KMS key for EBS encryption
Updates xvdb of Bottlerocket instances to use gp3 encrypted with KMS
Adds Amazon Managed Prometheus workspace
Adds Kyverno

Signed-off-by: Jacob Woffenden<EMAIL_ADDRESS>

@jacobwoffenden Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance

@jacobwoffenden Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance

@jacobwoffenden Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance

@jacobwoffenden Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance

@jacobwoffenden Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance

@jacobwoffenden Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance

@jacobwoffenden Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance

@jacobwoffenden Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance

@jacobwoffenden Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance


Site no longer active?

@Sjors Is there a reason the site is no longer available? I personally liked it very much.
Resolving / checking DNS doesn't come up with an A (or AAAA) address.
$ host www.pletdecurve.nl
Host www.pletdecurve.nl not found: 3(NXDOMAIN)

Thanks for answering, Sjors.


Add task family version to task_environments_t

We'll use this column to find runs that were performed on a particular version or version range of a task, when analyzing run results for our reports.
Details:

RunQueue populates taskFamilyVersion for viv run
TaskContainerRunner populates taskFamilyVersion for viv task start and viv task test

Documentation: The new column is documented in schema.sql.
Testing:

covered by automated tests

[x] Test starting a run on tasks with and without a version
[x] Test starting a task environment on tasks with and without a version

manual test instructions:

viv task start a task with no top-level version key, check that the task env's version is null
viv task start a task with a top-level version key, check that the task env's version is correct
viv task test a task with no top-level version key, check that the task env's version is null
viv task test a task with a top-level version key, check that the task env's version is correct
viv run an agent on a task with no top-level version key, check that the task env's version is null
viv run an agent on a task with a top-level version key, check that the task env's version is correct

Hmm OK, I should probably revert back to calling it taskVersion then. It's not that misleading and I think it's the only change we need to make in order to support task-specific versions in the future.


Add a page for integrations!

Any tools or systems that integrate with cosign signatures should be listed somewhere here in the repo!

kaniko
distroless
are the ones I know of rn!

This is now on the website.


Transform to unblock java-beta release

https://github.com/microsoftgraph/msgraph-beta-sdk-java/pull/295

I'm curious why the output file shows so much change.


Impl'n: New Celery task infra to process CreateMeasures instances

TP-2000-1147 Impl'n: New Celery task infra to process CreateMeasures instances
Why
Currently, bulk editing occurs within the context of a web worker. Because an edit can take many seconds and even minutes to complete, the confirmation web page presented to a user is bounded at the lower end by the amount of time an edit takes. It’s not the best approach to performing this type of computation for a number of reasons, including:

Generally, users are accustomed to web pages loading in the order of milliseconds or a few seconds in the worst case, so a long delay is frustrating.
Errors may be accumulate through spurious UI interaction with an in-progress creation process.
Proxy server time-outs may occur during edit measure processing - the TAP team and SRE do not have the necessary access permissions to extend the proxy server’s timeout period.

What
This PR is concerned with:

Implementing the Celery infrastructure for processing the task asynchronously.
Prepare the form data in serializable format. It has been established that the best way to do this with Django Form Wizard is to add a serializer and deserializer onto each Form. The serializer returns the user input to the form, the deserializer provides the completed form object.

Charlie's Notes

Migration might be irrelevant
We're duplicating a lot of the serialization code. TODO: Investigate extracting the serialization code into a helper function that can be imported and reused.

Codecov Report
Attention: 91 lines in your changes are missing coverage. Please review.

Comparison is base (6527259) 92.74% compared to head (8264713) 92.50%.
Report is 3 commits behind head on master.

Files
Patch %
Lines

measures/forms.py
36.69%
69 Missing :warning:

measures/views.py
19.04%
17 Missing :warning:

measures/models.py
50.00%
3 Missing :warning:

measures/bulk_handling.py
66.66%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1115      +/-   ##
==========================================
- Coverage   92.74%   92.50%   -0.25%
==========================================
  Files         460      464       +4
  Lines       35231    35461     +230
  Branches     2688     2712      +24
==========================================
+ Hits        32675    32803     +128
- Misses       2029     2129     +100
- Partials      527      529       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


GDScript is not able to infer class member types.

Godot version:
f73374629dd57d3eea6a960859ca223fa2d95fc4
OS/device including version:
Fedora 32
Issue description:
The GDS analyzer is not able to infer the member type, when the variable is initialized to null and assigned after it's used:

If you move the _ready function above the _process function, the error disappear.
Script:
extends Node

var test_var = null

func _process(delta):
	print(test_var.x)

func _ready():
	test_var = Vector3()

You can't use an 'x' value of a variable if you have the variable set to null (NiL). To fix the issue, just set the variable to a Vector value or put the "func _ready(): text_var = Vector3()" above the "func _process(delta): print(test_var.x)". You should also pay attention and read what the errors say, they can help at least a little with most of your problems.
TLDR: Just put the "ready" function above the "process" function, or, just set the variable to a Vector value instead of null.

This issue seems very similar to #49183 which is a bug because null is a valid value for : Object.
The one big difference with this issue is that a primitive type like Vector3 can never be compared to null, so null never makes sense as a value for a Vector3.
If you really wish to insist on assigning null to a Vector3 type, GDScript 2.0 offers a means to escape from the type safety by declaring a variable as : Variant.
I'd be in favor of closing this issue: my assessment is that this is a feature, not a bug: type checking is doing its job of preventing you from mixing null and Vector3 which are incompatible types.
That said, my guess is that lots of users will probably run into this problem when upgrading projects from Godot 3.x - perhaps some good documentation or error messages could help out.


Connect-RSC Throws Error if not Admin

Due to the version check Connect-RSC does upon connection, this will throw an error for any non-admin user, although the connection succeeds. This becomes problematic for any automation.
Note - 'non-admin' is a bit vague...I haven't been able to find a permission set that will allow this action specifically.
PS C:\Users\MyUser> Connect-RSC
Connect-Rsc: One or more errors occurred. (The request generated an error.

Query: query GetRscVersion{
deploymentVersion

__typename }
Variables: null
Operation Name: GetRscVersion

GraphQLResponse<JObject>:
Data: null
Errors:

403 Objects are not authorized
  Locations:
    Line: 2, Column: 1
  Path: deploymentVersion
  Extensions:
{
  "code": 403,
  "trace": {
    "operation": "/api/graphql",
    "traceId": "emCSjcC4/UC3PE53cWkU1A==",
    "spanId": "3COwVjAQPss="
  }
}

HTTP Status Code: 200 OK

)

Can you verify it's an error and not a warning? Connect-Rsc writes a warning when there is a version mismatch
if (_schemaVersion != _serverVersion)
{
	_logger.Warning($"The installed RSC SDK version does not " +
	$"match the server version.");
}


Set up pull request template

It would be useful to set up a pull request template that helps remind absent-minded developers like myself to update the changelog for their PRs when they open them!

+1!

Do you have a format you like @andrrizzi ?

Something like this maybe? But I've no strong opinion about it
- [ ] Add (tests)[https://github.com/openforcefield/openforcefield/tree/master/openforcefield/tests].
 - [ ] Update docstrings/[documentation](https://github.com/openforcefield/openforcefield/tree/master/docs).
 - [ ] Update [changelog](https://github.com/openforcefield/openforcefield/blob/master/docs/releasehistory.rst)

Let's get this implemented ASAP, since it looks like PRs are neglecting to update their changelogs!


Hifi global/base styling and style guide

Implement the "global" styles for pages to build off:

[ ] Text styles
[ ] Colour scheme (subject to change)
[ ] Components

[ ] Button (is there only 1 type?)
[ ] List?
[ ] Others?

I also think we need some style guidelines to avoid either having to put the same style in heaps of different places, but conversely not having to change something in the "global"/component styles and then have layouts broken.
For example:

Do components like Button set their own margin? Or is it 0 and margin is set where it is used?

Any others that would be useful?
Also I had set a few overrides in theme.js eg. for List but I think maybe that was a Bad Idea, since other Mui components might use it with unexpected results.

@darrenvong

@darrenvong

I guess what I want to avoid is the situation where we think "Actually, all text fields should have this style" change the component, and then break the style of some instances

I guess what I want to avoid is the situation where we think "Actually, all text fields should have this style" change the component, and then break the style of some instances

For button, I think it makes sense for it to only manage its internal spacing, so that's probably everything up to its border? This is so that we can port most styles across. If we start fixing margins on it, I bet there will be cases where the original margin we set wouldn't work and then it becomes a hack of styles further down just to override it 😅
For lists, maybe it makes sense for it to manage space between list items and have a few "variants" depending on what the different spacing needs we have?

For button, I think it makes sense for it to only manage its internal spacing, so that's probably everything up to its border? This is so that we can port most styles across. If we start fixing margins on it, I bet there will be cases where the original margin we set wouldn't work and then it becomes a hack of styles further down just to override it 😅
For lists, maybe it makes sense for it to manage space between list items and have a few "variants" depending on what the different spacing needs we have?


Facebook UI  Clone

Project title
Facebook UI  Clone
Project Description
It's Facebook UI  Clone  using HTML,CSS,JS
Stack:
And in order to tick the check box just but x inside them for example - [x] like this. Please delete options that are not relevant.

[X] Html
[X] CSS
[X] JavaScript

/assign


libInput: Add POINTER_MOTION_ABSOLUTE support

Thank you!

@kisvegabor could you cherry-pick this into the 8.3 branch?
Thanks!

Done :slightly_smiling_face:
8cdabe8


Fix the issue 629 and add two testcases

Hi, I have fix the issue #629 and add two testcases for it. I add some code to check the match os parenthesis when compiling the path to throw an exception when a CLOSE_PARENTHESIS loss.

Hi @kallestenflo I noticed merging this broke master. I rebased this branch against ca04684c057a67c84649daa68360715cb479e7d5 and found that the tests fail the same way, though this PR was good back in April based on f3c8fd75a63a2f49f41c81ec0c7f757f48c214c5. Could it be rolled back so it can be amended please?

PS do you need some help setting up CI to catch things like this and make it less of a burden?

there is no need to roll this back, it does the right thing. A regression was introduced between its implementation and merging it, I fixed that here: #763


Philippines page is empty

well its fixed now on my side, its peobably the best to close this issue


Decide the style and composition of colours for our website

I think we could prepare a colour pallet we like or the site we want to base. We could present it on a meeting or adjust it here

My vote goes on:
Purple of Playfulness 💜
@mmisiewiczz What's your vote? 😄

--primary-100:#7E57C2;
--primary-200:#b085f5;
--primary-300:#ffe8ff;
--accent-100:#B085F5;
--accent-200:#4d2c91;
--text-100:#FFFFFF;
--text-200:#e0e0e0;
--bg-100:#2E1A47;
--bg-200:#3f2958;
--bg-300:#594273;

#FFE6E6
#e6cfcf
#E1AFD1
#AD88C6
#7469B6
#514A7F
#B6699C
#CB9EBC
#765C6D
#634E71
#eb3223
#78FC4C


Enhance security by add loadBalancerSourceRanges to lockdown ip address.

Purpose:
When the type is LoadBalancer on the cloud ex: AWS, the Load Balancer ELB/NLB default will allow <IP_ADDRESS>/0 for the public internet.
Add loadBalancerSourceRanges function in service.yaml to help lockdown IP address if needed.

We would like to see this in the chart. Currently, we have to not forget to apply CIDR manually when LB is created, otherwise, it will be huge security breach.

I added unit tests and update PR to the new folder structure.
if there still some things I'm not done properly, please let me know
thank you for review.


[spi_host,dv] Check enable deassertion logic for Dual/Quad segments

Description
As described in this comment and fixed in this PR, the TLT chip_sw_spi_host_tx_rx stimulated a failure when the SIO enable was used to mask the non-transmitting wire sd_en_o[0] in Standard Mode RxOnly segments (original issue).
This TLT generates stimulus that is significantly different to the block level DV environment, and it should be investigated if the same incorrect enable deassertion can be observed when using Dual or Quad mode segments. This will require block level DV enhancements to stimulate any degenerate behaviour.

@antmarzam Maybe you can help @hcallahan-lowrisc on this one?

Sure - Happy to help! :)
I just had a chat with Harry about this, and have now an idea about how to tackle this (typing it here for future reference):
1 - Roll spi_host RTL back before this fix
2 - Write an assertion and ensure it fires when running the test in https://github.com/lowRISC/opentitan/pull/21338#issuecomment-1984908004
3 - Place the assertion in parallel for each of the SPI lanes
4 - Ensure we get stimulus in the block-level TB which fire the assertion (for lane=0)
5 - Update spi_host RTL to the latest and make sure assertion no longer fires.
6 - Generate the stimulus needed to reproduce the same issue for 2 or 4 lanes
7 - Add some FCOV to see these scenarios are hit by the block level TB

For what it's worth, I've managed to spot check a few combinations (1-4-4 and 1-2-2 reads), and the waveforms looked fine.

This can be closed since both DV and RTL changes are in master


Image upload via URL not working in Next Js

There is no option to remove the Image upload via URL.

In next js I am not able to insert any URL in the upload image section.

Has anyone encountered this issue ?
Jodit Version:  4.1.2
Browser: : Chrome
OS: : MacOs
Is React App: : True

I'm facing same issue..
Sometimes its work but most of the time not able to insert any url or alt text..

@hussainmanasa @ravi-wki
It will always work, if you implment properly, first of all dont use config directly, use it though memo,
Here is demo


Configuring email notifications per channel

#578

Closing since this won't really work.

I'm not sure you understood what I wanted to do: I wanted to have a tool to set notifications to OFF because members are complaining about too many notifications and prefer to delete their channel instead of configuring it - and we want them to keep it. Or are you saying that disabling notifications that way does not work?
I would eliminate the part that tried to use "notification_on" because that was something I did not really think about. I could instead print that to turn notifications on you need to use the GUI.
Could you reopen the pull request?

This got somehow out of hand, I'm still struggling with git. Sorry.

I think this merge borked some things... I got a broken db upgrade...

Nevermind, it's probably not related to this merge.


Minor tweaks and fixes for GCE

This PR contains couple of minor tweaks and fixes for GCE code, being still backward compatible with published ref. arch.

Well, depends on when you will have time for the paper, because as it's now, it's not possible to deploy ocp on gce with the current code (because of the ansible 2.2).

If the install is broken then we should merge now, the paper should be updated early next week.


Specify device for init function

You can tell jax to place arrays on a particular device using jax.device_put but the init function returned from hk.Transform appears to perform device placement automatically. Is there a way to specify a device for init? If not I am happy to submit a pull request that adds an optional device keyword to the init function.

Hi @ethanabrooks , you can do this by jitting the init function:
f = hk.transform(f)
devices = jax.local_devices()
params = jax.jit(f.init, device=devices[1])(rng, x)

Many folks will pmap instead of jit to replicate their parameters across local devices.


Npm is not installing correctly and it directs me to here

I've attached my debug.log file. I'm at a loss
npm-debug.txt

I'm getting a whole lot of npm errors

Can you share what you're doing to run into this? Where is the package you're installing? Does typings install usually work for you?

its the 5 minute quickstart with typescript for angular 2
https://angular.io/docs/ts/latest/quickstart.html

Did you do an NPM install?

typings.zip
I too was trying to do the quickstart on Angular2 website.
I have attached the config files from the site.
Below is the npm-debug.txt
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\dev\nodejs\node.exe" "C:\dev\nodejs\node_modules\npm\bin\npm-cli.js" "install"
npm ERR! node v6.2.2
npm ERR! npm  v3.9.5
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>postinstall: typings install
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'typings install'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the angular2-forms package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     typings install

Can anyone here do me a favour - can you tell me their environment variable for NODE_ENV (E.g. echo $NODE_ENV)?

Sorry - I don't have that environment variable defined.

Thanks. There's not much information to go on. Even if the command was failing or Typings was failing, you'd see a different error. Have you tried running Typings separately? What about moving away from postinstall (someone should really log an issue with Angular on the promotion of postinstall here too, it's not great form to use) and try changing postinstall to prepublish?

Can you share any other details of the error also? Is there anything below the "postinstall" line?

I have tried running manually per angularjs instructions( npm run typings install)  but get same error.  In the process of logging issue with them.
[https://github.com/angular/angular/issues]

Thanks. Is there any additional information about the error before you do that? Perhaps there's a line above the NPM errors that you haven't copied across?

Here's the whole error log
0 info it worked if it ends with ok
1 verbose cli [ 'C:\\dev\\nodejs\\node.exe',
1 verbose cli   'C:\\dev\\nodejs\\node_modules\\npm\\bin\\npm-cli.js',
1 verbose cli   'run',
1 verbose cli   'typings',
1 verbose cli   'install' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'pretypings', 'typings', 'posttypings' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for pretypings, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH: C:\dev\nodejs\node_modules\npm\bin\node-gyp-bin;c:\Users\SXK0KMH\git\angular2-forms\node_modules\.bin;C:\dev\nodejs;C:\app\SXK0KMH\product\11.2.0\client_32\bin;C:\windows\system32;C:\windows;C:\windows\System32\Wbem;C:\windows\System32\WindowsPowerShell\v1.0\;C:\Program Files\Git\cmd;C:\Program Files (x86)\scala\bin;C:\dev\nodejs\;C:\Users\SXK0KMH\reveal.js\npm-global\;C:\Users\SXK0KMH\AppData\Roaming\npm;C:\dev\nodejs\npm-global\;S:\proddlls\pbnt;Z:.;Y:.;X:.;S:\proddlls\sql\bin;S:\proddlls\sql\dll;C:\Users\SXK0KMH\AppData\Roaming\npm
10 verbose lifecycle<EMAIL_ADDRESS>CWD: c:\Users\SXK0KMH\git\angular2-forms
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '/d /s /c', 'typings "install"' ]
12 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
13 info lifecycle<EMAIL_ADDRESS>Failed to exec typings script
14 verbose stack Error<EMAIL_ADDRESS>typings: `typings "install"`
14 verbose stack Exit status 1
14 verbose stack     at EventEmitter.<anonymous> (C:\dev\nodejs\node_modules\npm\lib\utils\lifecycle.js:245:16)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at EventEmitter.emit (events.js:191:7)
14 verbose stack     at ChildProcess.<anonymous> (C:\dev\nodejs\node_modules\npm\lib\utils\spawn.js:24:14)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at ChildProcess.emit (events.js:191:7)
14 verbose stack     at maybeClose (internal/child_process.js:852:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:5)
15 verbose pkgid<EMAIL_ADDRESS>16 verbose cwd c:\Users\SXK0KMH\git\angular2-forms
17 error Windows_NT 6.1.7601
18 error argv "C:\\dev\\nodejs\\node.exe" "C:\\dev\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "typings" "install"
19 error node v6.2.2
20 error npm  v3.9.5
21 error code ELIFECYCLE
22 error<EMAIL_ADDRESS>typings: `typings "install"`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>typings script 'typings "install"'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the angular2-forms package,
23 error not with npm itself.
23 error Tell the author that this fails on your system:
23 error     typings "install"
23 error You can get information on how to open an issue for this project with:
23 error     npm bugs angular2-forms
23 error Or if that isn't available, you can get their info via:
23 error     npm owner ls angular2-forms
23 error There is likely additional logging output above.
24 verbose exit [ 1, true ]

Is there anything extra in the CLI? NPM does not seem to copy the errors that were emitted from the CLI into the debug log.

here are cli errors:
typings ERR! message Unable to connect to "https://api.typings.org/entries/dt/node/tags/6.0.0%2B20160613154055"
typings ERR! caused by connect ETIMEDOUT <IP_ADDRESS>:443

typings ERR! cwd c:\Users\SXK0KMH\git\angular2-forms
typings ERR! system Windows_NT 6.1.7601
typings ERR! command "C:\\dev\\nodejs\\node.exe" "c:\\Users\\SXK0KMH\\git\\angular2-forms\\node_modules\\typings\\dist\\bin.js" "install"
typings ERR! node -v v6.2.2
typings ERR! typings -v 1.3.0
typings ERR! code EUNAVAILABLE

typings ERR! If you need help, you may report this error at:
typings ERR!   <https://github.com/typings/typings/issues>

npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\\dev\\nodejs\\node.exe" "C:\\dev\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "typings" "install"
npm ERR! node v6.2.2
npm ERR! npm  v3.9.5
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>typings: `typings "install"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>typings script 'typings "install"'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the angular2-forms package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     typings "install"
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs angular2-forms
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls angular2-forms
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     c:\Users\SXK0KMH\git\angular2-forms\npm-debug.log

Ok, thanks. That's a lot more useful. Are you behind a proxy?

I am but I have the npm config for proxy and https-proxy set

We don't use the NPM config file.

I think that's on my end, no?  I set my config to use my company proxy.

Typings does not use the NPM proxy config file.

So then, how does one specify proxy settings to get to your site?

It's available in the documentation: https://github.com/typings/typings/blob/master/docs/faq.md#configuration.

I had this issue as well. Both the environment variable method and .typingsrc file method worked for me as a fix.
You must have your reasons for not using NPM's configuration variables. But it is a pain for those of us who work behind a corporate proxy to learn each package's configurations steps when they chose another method.
In this case, I read both typings' and rc's documentation before I got it to work, and I will need to change this parameters value when my proxy password expires every three months.

You're welcome to submit a PR enabling the loading of NPM's proxy configuration, however, both tools also accept environment variables that work.


Remove contents of docs/ except for reference, point to new location

I screwed up #27297 and this PR carries that PR. Sorry @johndmulhausen @thaJeztah

Looks like there's some stray .DS_Store files;
ls -la docs/userguide/
total 16
drwxr-xr-x  3 sebastiaan  staff   102 Oct 14 10:07 .
drwxr-xr-x  7 sebastiaan  staff   238 Oct 14 10:07 ..
-rw-r--r--@ 1 sebastiaan  staff  6148 Oct 14 10:08 .DS_Store

ls -la docs/installation/
total 24
drwxr-xr-x  4 sebastiaan  staff   136 Oct 14 10:07 .
drwxr-xr-x  7 sebastiaan  staff   238 Oct 14 10:07 ..
-rw-r--r--@ 1 sebastiaan  staff  8196 Oct 14 10:07 .DS_Store
drwxr-xr-x  3 sebastiaan  staff   102 Oct 14 10:07 images

ls -la docs/installation/images/
total 16
drwxr-xr-x  3 sebastiaan  staff   102 Oct 14 10:07 .
drwxr-xr-x  4 sebastiaan  staff   136 Oct 14 10:07 ..
-rw-r--r--@ 1 sebastiaan  staff  6148 Dec 23  2015 .DS_Store

Oh, there's a couple more :D

Some additional things that need to be removed; there's some leftovers in the Makefile for make docs (we probably need an alternative for that, but not sure how to realize that with the new repo)

https://github.com/docker/docker/blob/master/Makefile#L96-L97
https://github.com/docker/docker/blob/master/Makefile#L1
https://github.com/docker/docker/blob/master/Makefile#L59

The .DS_Store was a false positive. I've removed the lines from the MAKEFILE. PTAL again @thaJeztah

Oops! the .DS_Store files were in my local checkout, and git ignores them so when switching branches, it keeps those :D

I just discussed with @anusha-ragunathan, and we think that the docs/extend directory should be considered reference docs, as it's really close to the code.
Possibly the https://github.com/docker/docker/blob/master/docs/extend/legacy_plugins.md can be considered "docs" (and handled in the docker.github.io repository), and I think the menu.md is no longer needed now that we no longer use hugo (https://raw.githubusercontent.com/docker/docker/master/docs/extend/menu.md)

I think it will be really hard to say that the extend/ directory should be maintained here except for the one file which should be maintained elsewhere, so let's just commit to maintaining extend/ here too. I'll update the PR.

OK, extend/ is back.


2.2.228 version cannot display Chinese normally under ie11, but display normally under chrome

version 2.x can not show chinese normally，but 1.10.xxx can
1.10.xxx can display chinese normally  both ie11 and chrome
2.2.228 version cannot display Chinese normally under ie11, but display normally under chrome
did i miss something？
demo url：https://mozilla.github.io/pdf.js/es5/web/viewer.html
select pdf attach and open it
check in it11 and chrome.pdf

Please note https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#faq-support:

While the library and the default viewer should still work in IE 11/Edge (non-Chromium-based) for the time being, some functionality/features may not be available and the performance will be worse compared to modern browsers. Going forward, only bugs which completely prevent the library and/or the default viewer from running will be accepted.

version 2.x can not show chinese normally，but 1.10.xxx can
1.10.xxx can display chinese normally both ie11 and chrome
2.2.228 version cannot display Chinese normally under ie11, but display normally under chrome
did i miss something？
demo url：https://mozilla.github.io/pdf.js/es5/web/viewer.html
select pdf attach and open it
check in it11 and chrome.pdf

hi, is there some solution to deal with this problem?


how to yield/resume in c

is there a way to yield/resume in c api? (not exactly duk_suspend/duk_resume)
startcoroutine(function () {
    print("in thread context")
    if (show_dialog() == "yes") {
        print("you click yes")
    } else {
        print("you click no")
    }
})
print("in main context")

// pseudo code
duk_context *thread;

duk_ret_t startcoroutine(duk_context *ctx) {
    // ... create a duktape thread
    thread = duk_new_thread(ctx, -1 /* idx of function */);
    duk_resume(thread, 0 /* initial args count */);
    return 0;
}

duk_ret_t show_dialog(duk_context *ctx) {
    // open dialog ui, and wait for user interaction
    ui_show_dialog();
    return duk_yield(ctx);
}

void resume_my_thread(const char *action) {
    duk_push_string(thread, action);
    duk_thread_state state = duk_resume(thread, 1);
    // check state ...
    if (state == STOP) {
        duk_close(thread);
        thread = NULL;
    }
}

// when user click 'yes' button
// ui code
void ui_on_click_yes() {
    resume_my_thread("yes")
}

This is not possible at present, yield/resume only works when the yielding call stack contains pure Ecmascript calls.
The technical issue is that it's difficult to portably reconstitute a native stack transparently. One quite easily ends up implementing an actual stack swap mechanism, which isn't portable.
I can think of at least four approaches to make this possible in the future:

Enable Duktape/C functions to register as being "yield/resume aware" so that they can reconstitute their state on demand without relying on the native stack doing that. This is obviously a burden on the calling code but doesn't require actual stack switching.
Add support to duk_config.h for the application to provide stack switching primitives. This would allow a coroutine to have an actual native stack which we'd swap in and out of with platform specific functions. This is a portability concern in that this capability wouldn't be automatically available unless you went to the trouble of implementing such stack manipulation code.
Use native threads (pthreads etc) as the providers for coroutine native stacks, and add duk_config.h stuff necessary to portably wait and lock the threads. Under the hood one would then in essence use duk_suspend() and duk_resume() to switch threads. This would in effect use the native thread library for the stack switching.
Add built-in support for some specific portable stack manipulation library, as a duk_config.h option. This would still limit portability, but would probably provide the stack switching capability for major platforms with reasonable effort.

There are probably more viable options.

Thank you very much for your detailed explanation, it helps me a lot.
I'm trying to write a unity3d binding for duktape, bring the capability of scripting by javascript/typescript to unity3d.

What about the built-in Promise? It's commented 'disabled until fully functional' in source code, will ti be implemented in the next release of duktape?

Thank you very much for your detailed explanation, it helps me a lot.
I'm trying to write a unity3d binding (https://github.com/ialex32x/duktape-unity) for duktape, bring the capability of scripting by javascript/typescript to unity3d. Is it stable enough to be added to 'projects using duktape'?

How does your project sample code support arrow functions,Duktape does not support it?

@ialex32x Promises will be implemented at some point but I cannot promise a certain time or a release: this is an open source project with a varying amount of available effort (at least for my part). I can add https://github.com/ialex32x/duktape-unity to the project list, it is really up to you :)

https://github.com/svaarala/duktape-wiki/pull/226

@ialex32x Looks like a great project to me :)

@svaarala Thank you :)

Thank you very much for your detailed explanation, it helps me a lot.
I'm trying to write a unity3d binding (https://github.com/ialex32x/duktape-unity) for duktape, bring the capability of scripting by javascript/typescript to unity3d. Is it stable enough to be added to 'projects using duktape'?

How does your project sample code support arrow functions,Duktape does not support it?

Just write in typescript, run as javascript, need to be compiled by tsc.


Fix memory leak

Hello!

Type: bug fix | new feature | code quality | documentation
Link to issue:

In raising this pull request, I confirm the following:

[ ] I have read and understood the Contributing Guidelines
[x] I have checked that another pull request for this purpose does not exist.
[ ] I wrote some tests for this PR.

Small description of change:
Thanks

Thank you!


☁Please optimize the application a little.🙃
Some time main player ui freeze on one track, changing track have long delay.
Thank you in advance!!

Hi, thanks for the report, but I sadly have to say that this project is not maintained anymore. I will place a warning in the README.
I don't discard working on this in some day of my life, but not currently, so sorry.


Update permissions-microsoft-365-security-center.md

highlight the scope, that this article refers to MDO data and experiences (and not all M365D), and adding links to EXO permissions.

Learn Build status updates of commit eaa14fd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/permissions-microsoft-365-security-center.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit ad1a12e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/permissions-microsoft-365-security-center.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit c1a8d96:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/permissions-microsoft-365-security-center.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit dc4e37d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/permissions-microsoft-365-security-center.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit e4f398c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/permissions-microsoft-365-security-center.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 84e2be0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/permissions-microsoft-365-security-center.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit ba645d2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/permissions-microsoft-365-security-center.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 9752eac:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/permissions-microsoft-365-security-center.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Update deployment.md with encoded-pod-template-hash and new examples

This is a follow-up to PR #7632 and Issue #7577
It updates the Deployment documentation with a note on the encoded-pod-template-hash used in ReplicaSet names (kubernetes/kubernetes#51538) and all the examples.

/assign @bradamant3

/unassign @Bradamant3
Woops. My apologies, sorry about the faux pas of assigning before review.

Yup! Should be good to close. Thanks!


AWS EKS: i4i.xlarge machine type reports CPU cost as 0

After deploying #169 I discovered that i4i.xlarge machine types cpu cost is being exported as 0 in us-east-2:

The price should be closer to $0.17/hour: https://instances.vantage.sh/aws/ec2/i4i.large

The root cause is because the i4i is part of the storage optimized family and we don't handle that.

The root cause is from the lookup here: https://github.com/grafana/cloudcost-exporter/blob/ffce31a953d77aba4fe1f5bc6dc642e310d12a18/pkg/aws/eks/pricing_map.go#L147
We do not have any catch all/defaults in the cpu to memory map: https://github.com/grafana/cloudcost-exporter/blob/ffce31a953d77aba4fe1f5bc6dc642e310d12a18/pkg/aws/eks/eks.go#L32-L36

Probably the closest estimate we can get for figuring out the ratio of CPU/instance cost is via GCP's storage optimized instances. The math on it looks a bit like so:
  $.0496531/vCPU/hour * 88 vCPU = $4.369472/hour
+ $.0066553/GiB/hour * 704 GiB = $4.68531/hour
  ---------------------------------------
$4.369472/hour/vCPU / $9.054804000 / hour ~= 0.482558518

This aligns pretty well with our already existing Memory optimized figure. The question remains:
What do we default too if the instance family isn't found?