Refs #31949 -- Made @sensitive_variables and @sensitive_post_parameters work with async functions

Reference to #31949.
This PR makes the sensitive_variables and sensitive_post_parameters decorators able to handle both sync and async views.

@LomaxOnTheRun Unfortunately, it's much more complicated, see 23cbed21876bf02f4600c0dac3a5277db5b2afbb which was reverted 7330408ac36f893a4a11c668ba230b440ec09f41, and a new PR from Jon.

@LomaxOnTheRun  if you can figure out a solution to the problem with my original PR (and my attempt to solve it in the new PR) then I'm all ears!
Basically, write a test case that uses @sensitive_variables on a nested async function and have the nested async function throw. In this case the sensitive variables will not be hidden :/
E.g.:
@sensitive_variables("sauce")
@sensitive_post_parameters("bacon-key", "sausage-key")
async def async_sensitive_function(request):
    # Do not just use plain strings for the variables' values in the code
    # so that the tests don't return false positives when the function's source
    # is displayed in the exception report.
    cooked_eggs = "".join(["s", "c", "r", "a", "m", "b", "l", "e", "d"])  # NOQA
    sauce = "".join(  # NOQA
        ["w", "o", "r", "c", "e", "s", "t", "e", "r", "s", "h", "i", "r", "e"]
    )
    raise Exception

async def async_sensitive_view_nested(request):
    try:
        await async_sensitive_function(request)
    except Exception:
        exc_info = sys.exc_info()
        send_log(request, exc_info)
        return technical_500_response(request, *exc_info)


Enable pook for newer Python versions

Pook now supports 3.11 as of v1.1.1, with CI up to 3.11 since v1.2.0.
Therefore, the check to exclude pook for version 3.11 can be removed:
https://github.com/mindflayer/python-mocket/blob/main/mocket/plugins/pook_mock_engine.py#L3
I'm shortly adding CI for 3.12 as well to ensure compatibility with the latest mocket version for Python 3.12.

@mindflayer I'll open a PR for this later in the week. The next version of pook (v1.2.2) will have tested support up to version 3.12.

I've already prepared a PR for fixing this, let me know when pook is ready for both 3.11 and 3.12.

Here is the new version which allows using pook plugin for Python versions newer than 3.10:
https://pypi.org/project/mocket/3.12.1/

Thanks!


How do I set up an AR camera?

"Play Mode" starts, but the camera does not work. I want a way to set up an AR camera.

are you able to fix this issue?

any  update  plz  on  how  to   use  ARCamara instaid  of  normal  web  camara


Source Request undefined when using the playground from Dialogflow

It happens in the sendResponse_ method.
buildResponseMessages()_ return null because of the undefined platform. It receives the platform name by parsing the request.body.originalDetectIntentRequest.
When using the "Try it now" functionality, it doesn't send anything about the platform.
The platform should be called something like "playground" or "dialogflow"

the Dialogflow simulator (what you called the "playground") doesn't include a platform name in the request, so this would need to be implemented in Dialogflow's backend.  Most platforms include a platform name in the request so, depending on how your agent is setup and how you access your agent a potential workaround is to assume a null platform name means the request is coming from the Dialogflow simulator.
I'll consider adding a playground/dialgoflow platform name in the future and leave this open for further comments.


Handle JSON responses from the server

Summary
Responses from the JSON endpoint were not being handled and now are.  (In practical terms, clock skew adjustment was not being handled previously and now will be.)
Also fixes a bug in the instrumentation internal debugging logs where the verbose option was not being checked correctly.

FYI @djspoons

hooray! clocks everywhere will rejoice (in synchrony)!


Wifi randomly disconnects - rg405m

Hello,
WiFi randomly disconnects and will not auto-reconnect unless i disable wifi and force it to reconnect to my network. It happens very randomly but is annoying bc there is no way to see i am connected while playing games without exiting. if this happens while playing i can no longer earn retro-achievements for whatever game im playing.

This happens on stock as well. I don't know that there is a better wifi driver available for this chipset.

Wanted to add related to this. I have to fully reboot my device occasionally when Wifi doesn't want to reconnect. When trying to access the Wifi settings menu the menu will go 'not responding' and OS will recommend I close or restart it. After rebooting the wifi adapter usually shows as off/disabled, and but responds quickly to turning it back on and detecting my network(s).
Have had this happen 2-3 times, usually as I leave my device on almost all the time (in sleep)... going between 2-3 different Wifi networks I have it paired with.


Where is the unit test part?

This is a very interesting architecture however I think it will be more complete if you can add Unit test and Mock data to the framework.

Hello Thomas,
I apologize that this took longer for me to look at. Many thanks for the suggestion!
If you look at the master branch now, there is a Unit Test project that has basic tests setup.
Thanks again for the feedback!
All the best,
Shawn


Fix mismatched version breaking test

          1 Warning

:warning:
Please provide a changelog summary in the Pull Request description @ohayon

  Generated by :no_entry_sign: Danger


Installation error

Describe the bug
A clear and concise description of what the bug is.
When running npm install the script stalls when trying to install<EMAIL_ADDRESS>I tried npm install -g<EMAIL_ADDRESS>and running npm install again, but the same thing happens.
Additional context
Add any other context about the problem here.
When trying npm run dev here is log:
0 info it worked if it ends with ok
1 verbose cli [ '/usr/bin/node', '/usr/bin/npm', 'run', 'dev' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'predev', 'dev', 'postdev' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
8 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/var/vue/vuesion/node_modules/.bin:/home/ludo/.symfony/bin:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games
9 verbose lifecycle<EMAIL_ADDRESS>CWD: /var/vue/vuesion
10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'vuesion dev -o' ]
11 info lifecycle<EMAIL_ADDRESS>Failed to exec dev script
12 verbose stack Error<EMAIL_ADDRESS>dev: `vuesion dev -o`
12 verbose stack spawn ENOENT
12 verbose stack     at ChildProcess.<anonymous> (/usr/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:48:18)
12 verbose stack     at ChildProcess.emit (events.js:203:13)
12 verbose stack     at maybeClose (internal/child_process.js:1021:16)
12 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:283:5)
13 verbose pkgid<EMAIL_ADDRESS>14 verbose cwd /var/vue/vuesion
15 verbose Linux 4.9.0-9-amd64
16 verbose argv "/usr/bin/node" "/usr/bin/npm" "run" "dev"
17 verbose node v12.6.0
18 verbose npm  v6.10.1
19 error file sh
20 error code ELIFECYCLE
21 error errno ENOENT
22 error syscall spawn
23 error<EMAIL_ADDRESS>dev: `vuesion dev -o`
23 error spawn ENOENT
24 error Failed at the<EMAIL_ADDRESS>dev script.
24 error This is probably not a problem with npm. There is likely additional logging output above.
25 verbose exit [ 1, true ]

Seems a dodgy wireless connection was the problem. Installed from a wired connection and it installs no problem


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #621 (593d3cb) into main (799f936) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #621   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Passing in empty map for traits/properties results in NPE

toMap shouldn't allow readableMap to be nullable here: https://github.com/charlires/react-native-segment-analytics/blob/028875b8ade22406f928fff2322014dfda65159a/android/src/main/java/com/charlires/segmentanalytics/SegmentAnalyticsModule.java#L106
Should short-circuit and return from toTraits and toProperties early if (= readableMap null) OR (empty? readableMap)

@jlo1 Should be fixed by your pull request, Thank you!


[Launcher] Popup still visible when no longer hovering over icon

Describe the bug
When an application is on the leftmost or rightmost side of the bar with multiple windows, the popup for that application shows indefinitely when leaving the module area.
To Reproduce

Hover over leftmost/rightmost applications with multiple windows while the bar is centered
Observe popup staying until ironbar is clicked or another application icon is hovered

Expected behavior
Popup disappears when not hovered over
System information:

Distro: NixOS
Compositor: Hyprland
Ironbar version: https://github.com/JakeStanger/ironbar/commit/f8d8c063008340922b20aa665d8f893ab63e82d7

Configuration
      "center": [
        {
          "type": "launcher"
        }
      ]

I'm aware of this one and it's always been a bit of a head scratcher to solve. I'm hoping the upcoming popup changes for the IPC will make fixing this easier, so should have it sorted soon™

Not sure what I was on before, was a very easy fix...should have it merged shortly.


Not Deployable on Latest Provider

Description
Please provide a clear and concise description of the issue you are encountering, and a reproduction of your configuration (see the examples/* directory for references that you can copy+paste and tailor to match your configs if you are unable to copy your exact configuration). The reproduction MUST be executable by running terraform init && terraform apply without any further changes.
If your request is for a new feature, please use the Feature request template.

[x] ✋ I have searched the open/closed issues and my issue is not listed.

⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Module version [Required]:

Terraform version: 1.3.7

Provider version(s): 5.0.0

Reproduction Code [Required]
module "atlantis" {
  source  = "terraform-aws-modules/atlantis/aws"
  version = "3.28.0"

  atlantis_version = "v0.24.2-debian"

  name               = format("%s-%s", local.deploymentid, local.environment)
  atlantis_fqdn      = "atlantis-ground.company.network"
  vpc_id             = module.vpc.vpc_id
  private_subnet_ids = module.vpc.private_subnets
  public_subnet_ids  = module.vpc.public_subnets

  enable_ephemeral_storage     = true
  ecs_service_platform_version = "LATEST"
  ecs_container_insights       = true
  ecs_task_cpu                 = 1024
  ecs_task_memory              = 2048
  container_memory_reservation = 256

  runtime_platform = {
    operating_system_family = "LINUX"
    cpu_architecture        = "X86_64"
  }

  entrypoint        = ["docker-entrypoint.sh"]
  command           = ["server"]
  working_directory = "/tmp"
  docker_labels = {
    "org.opencontainers.image.title"       = "Atlantis"
    "org.opencontainers.image.description" = "A self-hosted golang application that listens for Terraform pull request events via webhooks."
    "org.opencontainers.image.url"         = "https://github.com/runatlantis/atlantis/pkgs/container/atlantis"
  }
  start_timeout = 30
  stop_timeout  = 30

  readonly_root_filesystem = false # atlantis currently mutable access to root filesystem
  ulimits = [{
    name      = "nofile"
    softLimit = 4096
    hardLimit = 16384
  }]

  user                                  = "1000:1000"
  alb_listener_ssl_policy_default       = "ELBSecurityPolicy-TLS-1-2-2017-01"
  alb_drop_invalid_header_fields        = true
  alb_enable_cross_zone_load_balancing  = true
  allow_unauthenticated_access          = true
  whitelist_unauthenticated_cidr_blocks = ["<IP_ADDRESS>/32", "<IP_ADDRESS>/32", "<IP_ADDRESS>/32"]
  create_route53_record                 = false
  certificate_arn                       = aws_acm_certificate.cert.arn

  alb_authenticate_oidc = {
    issuer                 = "https://auth.company.network/oauth2/default"
    token_endpoint         = "https://auth.company.network/oauth2/default/v1/token"
    user_info_endpoint     = "https://auth.company.network/oauth2/default/v1/userinfo"
    authorization_endpoint = "https://auth.company.network/oauth2/default/v1/authorize"
    client_id              = "0oa4118y2iaRimyth4h7"
    client_secret          = data.aws_ssm_parameter.atlantis_okta_client_secret.value
  }

  extra_container_definitions = [
    {
      name      = "log-router"
      image     = "amazon/aws-for-fluent-bit:latest"
      essential = true

      firelens_configuration = {
        type = "fluentbit"

        logConfiguration = {
          logDriver = "awslogs",
          options = {
            awslogs-group         = "firelens-container",
            awslogs-region        = local.region,
            awslogs-create-group  = true,
            awslogs-stream-prefix = "firelens"
          }
        }
      }
    }
  ]

  custom_environment_variables = [
    # Override server config
    {
      "name" : "ATLANTIS_REPO_CONFIG_JSON",
      "value" : jsonencode(yamldecode(file("${path.module}/server-atlantis.yaml"))),
    },
  ]

  atlantis_gitlab_user       = "gitlab_atlantis"
  atlantis_gitlab_user_token = data.aws_ssm_parameter.atlantis_gitlab_user_token.value
  atlantis_gitlab_hostname   = "gitlab.company.network"
  atlantis_repo_allowlist    = ["gitlab.company.network/company/company_ground_infra"]
  atlantis_log_level         = "debug"

  depends_on = [
    aws_acm_certificate.cert
  ]
}

Expected behavior
Deploys
Actual behavior
running "/home/atlantis/.atlantis/bin/terraform1.3.7 plan -input=false -refresh -out \"/home/atlantis/.atlantis/repos/trex/trex_ground_infra/44/default/prod/us-west-2/ops/atlantis/default.tfplan\"" in "/home/atlantis/.atlantis/repos/trex/trex_ground_infra/44/default/prod/us-west-2/ops/atlantis": exit status 1
Releasing state lock. This may take a few moments...
╷
│ Warning: Argument is deprecated
│
│   with module.atlantis.module.vpc.aws_eip.nat,
│   on .terraform/modules/atlantis.vpc/main.tf line 1068, in resource "aws_eip" "nat":
│ 1068:   vpc = true
│
│ use domain attribute instead
╵
╷
│ Error: Unsupported argument
│
│   on .terraform/modules/atlantis.ecs/main.tf line 6, in resource "aws_ecs_cluster" "this":
│    6:   capacity_providers = var.capacity_providers
│
│ An argument named "capacity_providers" is not expected here.
╵
╷
│ Error: Unsupported block type
│
│   on .terraform/modules/atlantis.ecs/main.tf line 8, in resource "aws_ecs_cluster" "this":
│    8:   dynamic "default_capacity_provider_strategy" {
│
│ Blocks of type "default_capacity_provider_strategy" are not expected here.
╵
╷
│ Error: Unsupported argument
│
│   on .terraform/modules/atlantis.vpc/main.tf line 32, in resource "aws_vpc" "this":
│   32:   enable_classiclink               = var.enable_classiclink
│
│ An argument named "enable_classiclink" is not expected here.
╵
╷
│ Error: Unsupported argument
│
│   on .terraform/modules/atlantis.vpc/main.tf line 33, in resource "aws_vpc" "this":
│   33:   enable_classiclink_dns_support   = var.enable_classiclink_dns_support
│
│ An argument named "enable_classiclink_dns_support" is not expected here.
╵
╷
│ Error: Unsupported argument
│
│   on .terraform/modules/atlantis.vpc/main.tf line 1307, in resource "aws_default_vpc" "this":
│ 1307:   enable_classiclink   = var.default_vpc_enable_classiclink
│
│ An argument named "enable_classiclink" is not expected here.
╵
╷
│ Error: Unsupported argument
│
│   on .terraform/modules/atlantis/main.tf line 593, in data "aws_iam_policy_document" "ecs_task_access_secrets_with_kms":
│  593:   source_json = data.aws_iam_policy_document.ecs_task_access_secrets.json
│
│ An argument named "source_json" is not expected here.
╵

For now, I'm using a lower provider version <5.0:
terraform {
  ...

  required_providers {
    aws = {
      source  = "hashicorp/aws"
      version = ">= 3.69, < 5.0"
    }
  }

  ...
}

Have the same issue. Not a big deal since atlantis lives in its own state and I can use any provider version regardless of the rest of the code

This is def a problem for us, because its preventing the upgrade of our VPC module that this ECS deployment exists in : https://github.com/terraform-aws-modules/terraform-aws-vpc

Still an issue

Still an issue

This issue has been resolved in version 4.0.0 :tada:


cephfs: Adjust with the fallocate API changes

Recent behavioural changes w.r.t fallocate API calls for corresponding changes in our tests which uses mode 0. In addition to skipping those existing tests we try to cover up the time taken for backports to land in released versions causing CI failures in our pre-release jobs. This has been implemented by creating a new test file intended to be controlled and maintained using required build tags. The new test file can be removed once all the expected backports are available with released versions.
fixes #1044

I see that the PR is unlabeled so far. If you're OK with only my review feel fre to put the no-API label on.

I see that the PR is unlabeled so far. If you're OK with only my review feel free to put the no-API label on.

Ah, I seem to forget that from time to time.
Done.


Adding infrared spectra intensity

Hi devs!
I saw the issue https://github.com/pyscf/pyscf/issues/678 and I tried to make an infrared spectra intensity implementation.
Should be able to evaluate RHF/RKS/UHF/UKS IR intensities, as long as hessian is available.
Seems that there is plan to implement more molecular properties. Hope this PR does not collide with existing efforts :joy_cat:

Wow! This is great from my perspective. I just tried it on an optimized acetone molecule and I get what I think are some reasonable looking values. I can also confirm that the vibrational frequencies here are the same as those gotten from a thermo.harmonic_analysis call.
The API I used to generate the following was
from pyscf import gto
from pyscf.prop.infrared.rhf import Infrared

mol = gto.Mole().fromfile('acetone_opt_pyscf.raw')
mf = mol.RHF().run()
mf_ir = Infrared(mf).run()
mf_ir.plot_ir()

which worked great

Thanks so much @ajz34 - that was also an incredibly fast turnaround time if you did that just after seeing my post!!

I should add I'm afraid it's beyond my current computational chemistry expertise to review the code or confirm the accuracies of the caclulations

Wow! This is great from my perspective. I just tried it on an optimized acetone molecule and I get what I think are some reasonable looking values. I can also confirm that the vibrational frequencies here are the same as those gotten from a thermo.harmonic_analysis call.
The API I used to generate the following was
from pyscf import gto
from pyscf.prop.infrared.rhf import Infrared

mol = gto.Mole().fromfile('acetone_opt_pyscf.raw')
mf = mol.RHF().run()
mf_ir = Infrared(mf).run()
mf_ir.plot_ir()

which worked great

Thanks so much @ajz34 - that was also an incredibly fast turnaround time if you did that just after seeing my post!!

Hello
Im interssting to use plot_ir for my work I use VS code and I cant plot my work in in it.
is there any way to do it please

Hello
Im interssting to use plot_ir for my work I use VS code and I cant plot my work in in it.
is there any way to do it please

Great feature. I should merge it early.

Hello Im interssting to use plot_ir for my work I use VS code and I cant plot my work in in it. is there any way to do it please

If you follow example 06, you can change fig.show() to fig.savefig('xxx.png'), and then the figure can be saved.

nice features


Added check for aggregate functions in optimizer rules

Which issue does this PR close?

Closes #12814 .
Rationale for this change

What changes are included in this PR?
The query is invalid due to aggregation functions in where clause.

Are these changes tested?
Added check for aggregation functions.

Are there any user-facing changes?

@jonahgao thank you for the review!

Alright should be good to go @findepi @jonahgao

Thanks @jonathanc-n @findepi


Create Release pipelines from Dev to PPE and PPE to PROD

Depends on https://github.com/Azure/azure-sdk-tools/issues/4742

The existing release pipelines for PPE and PROD worked as expected in the previous release.
Could you clarify what this issue is abou? Is it about creating new ones or a copy of those?

@JonathanCrd the pipelines were not working before so we were looking into it.
Now that Admin app is clean and Wei has been cleaning the pipelines it seems to be working.
We can close the issue


[Sprint 51] - XD-3184 Update spring-xd-yarn servers.yml with settings for HDP <IP_ADDRESS>

LGTM - merging.

Merged.
Master: de057bb4a88c16b8f952e84e65d37a54c0e1e0e6
Cherry-picked to 1.2.x: 9e61558de7a5b32c23155dd040aec5c4a68c9c48


Adopt VuePress v1.0 once it's official released

Is your request for enhancement related to a problem? Please describe.
Current VuePress v0.14.1 which is a little out-dated. Migrating to v1.x will give us more flexibilities to apply plugins to the website. The new plugin feature is introduced by v1.x of VuePress.
This task may require us to revisit some customizations we made related to css styles, versions dropdown, icon links, copy-to-clipboard, etc.
Additional context
Current result of npm view vuepress is:
latest: 0.14.11       next: 1.0.0-alpha.47

We need to wait for the official release to avoid duplicated re-eject work.

PR #672 resolved the issue.


data: bharathkalyans

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Thank you for making the change - I will now merge your PR


Unable to run server using https

I have an app that is running using https. I need to set up e2e testing environment for it.
If I set in the jest-puppeteer.config.js the server option protocol to 'http' the test runner ignores the server started on https://localhost:3000 and terminated by timeout running no tests.
The output is
yarn run v1.9.4
$ NODE_ENV=test jest --useStderr --forceExit --runInBand --no-cache --detectOpenHandles --config config/jest/config.e2e.js
Determining test suites to run...
Jest dev-server output:
[Jest Dev server] $ node --max-old-space-size=4096 ./scripts/start.js
[Jest Dev server] NODE_ENV: test
[Jest Dev server] Starting the development server...
[Jest Dev server] start detecting webpack modules cycles
[Jest Dev server] end detecting webpack modules cycles
[Jest Dev server] start detecting webpack modules cycles
[Jest Dev server] end detecting webpack modules cycles
[Jest Dev server]

Compile done in: 159.054s
[Jest Dev server] Compiled with warnings.
[Jest Dev server] Warning in ./~/ajv/lib/async.js
96:20-33 Critical dependency: the request of a dependency is an expression
    at process._tickCallback (internal/process/next_tick.js:180:9)
Warning in ./~/ajv/lib/async.js
119:15-28 Critical dependency: the request of a dependency is an expression
    at process._tickCallback (internal/process/next_tick.js:180:9)
Warning in ./~/ajv/lib/compile/index.js
13:21-34 Critical dependency: the request of a dependency is an expression
    at process._tickCallback (internal/process/next_tick.js:180:9)
You may use special comments to disable some warnings.
Use // eslint-disable-next-line to ignore the next line.
Use /* eslint-disable */ to ignore all warnings in a file.

Server has taken more than 240000ms to start.

☝ You can set "server.launchTimeout" in jest-puppeteer.config.js
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

If in the jest-puppeteer.config.js I skip the server option protocol or set it to null the test runner starts tests execution right after the call of the server start (not waiting while the app is started). My app is compiled up to 4 minutes. Hence to avoid termination by timeout I removed timeout restriction from page.goto and set the test case timeout to 4 minutes. However once the app compiled I get net::ERR_SPDY_PROTOCOL_ERROR at https://localhost:3000/login error.
The output is
yarn run v1.9.4
$ NODE_ENV=test jest --useStderr --forceExit --runInBand --no-cache --detectOpenHandles --config config/jest/config.e2e.js
Determining test suites to run...
Jest dev-server output:
[Jest Dev server] $ node --max-old-space-size=4096 ./scripts/start.js
[Jest Dev server] NODE_ENV: test
[Jest Dev server] Starting the development server...
[Jest Dev server] start detecting webpack modules cycles
end detecting webpack modules cycles
[Jest Dev server] start detecting webpack modules cycles
[Jest Dev server] end detecting webpack modules cycles
[Jest Dev server]

Compile done in: 157.259s
 FAIL  tests/e2e/home.test.js (157.404s)
  home page
    ✕ contains a particular text (155336ms)

  ● home page › contains a particular text

    net::ERR_SPDY_PROTOCOL_ERROR at https://localhost:3000/login

      at navigate (node_modules/puppeteer/lib/Page.js:622:37)

Test Suites: 1 failed, 1 total
Tests:       1 failed, 1 total
Snapshots:   0 total
Time:        158.343s
Ran all test suites.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Please, note that if I use http (not https) in the app, in the test case and in the jest-puppeteer.config.js everything is fine i.e. the test suite is passed and executed once the app is started but I need to use https.
Test Case
/*global TimeoutLength windowSize*/
import puppeteer from 'puppeteer';
import expect from 'expect-puppeteer';

describe('home page', () => {
  let browser;
  let page;

  beforeAll(async () => {
    jest.setTimeout(TimeoutLength);

    browser = await puppeteer.launch({
      headless: false,
      args: [`--window-size=${windowSize.width},${windowSize.height}`],
      ignoreHTTPSErrors: true
    });
    page = await browser.newPage();
    await page.setViewport(windowSize);
  });

  it('contains a particular text', async () => {
    await page.goto('https://localhost:3000/login', {
      timeout: 0
    });
    await expect(page).toMatch('My Accounts');
  }, TimeoutLength);

  afterAll(() => {
    browser.close();
  });
});

jest-puppeteer.config.js
module.exports = {
  server: {
    command: 'NODE_ENV=test yarn start',
    debug: true,
    launchTimeout: 240000,
    port: 3000,
    protocol: 'http',//or skipped i.e. protocol: null
    usedPortAction: 'kill'
  }
};

Jest config
module.exports = {
  rootDir: '../..',
  testURL: 'http://localhost',
  setupFiles: [
    '<rootDir>/config/jest/setup.js'
  ],
  transform: {
    '^.+\\.(js|jsx)?$': '<rootDir>/config/jest/transform.js'
  },
  moduleDirectories: [
    'node_modules',
    'src'
  ],
  globals: {
    TimeoutLength: 240000,
    windowSize: {
      width: 1200,
      height: 900
    }
  },
  verbose: true,
  preset: 'jest-puppeteer',
  setupTestFrameworkScriptFile: 'expect-puppeteer',
  testRegex: '.*\\.(test)\\.js$',
  testPathIgnorePatterns: [
    'config/babel.test.js'
  ]
};

Looks like there's a healthier alternative to wait-port. https://www.npmjs.com/package/wait-on

Hi @langri-sha
Thank you for your suggestion, look like the wait-on package better than wait-for package.
I will be very happy to replace wait-port to wait-on. What do you thinking @neoziro

Hi @langri-sha, hi @xiaoyuhen. It is OK for me, if it works better, let's use it!

hi @alexander-elgin
after merge this PR and floow this guide
will able to run server using https


bump guava version to 31.0.1

What's the purpose of this PR
Since we no longer support java7, we can bump guava version to get rid of risks(bug、security) problem
Brief changelog

bump guava version to 31.0.1

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Read the Contributing Guide before making this pull request.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit tests to verify the code.
[x] Run mvn clean test to make sure this pull request doesn't break anything.
[x] Update the CHANGES log.

I have read the CLA Document and I hereby sign the CLA

@nobodyiam PTAL

Codecov Report

Merging #4182 (86ef176) into master (9a666c8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #4182   +/-   ##
=========================================
  Coverage     52.58%   52.58%
  Complexity     2621     2621
=========================================
  Files           484      484
  Lines         15201    15201
  Branches       1572     1572
=========================================
  Hits           7994     7994
  Misses         6651     6651
  Partials        556      556

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 9a666c8...86ef176. Read the comment docs.


Move contribute to docker to open source

Move the how to contribute to open source documentation to the opensource project.

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@thaJeztah what's going on here?

"How to contribute to docker" is here now. Let's close this one 👼


[tree widget] small font size updates

made font sizes smaller, alignment fixes for inputs
made the widget ever-so-slightly wider

gave that a shot. also restored font sizes in browser pages


allow jaxmd to read in saved momenta and NH state

For NPT simulations it common to do a short NVT equilibration run first. Therefore we need to be able to read in the momenta of the input structure. Further, if we want to restart a simulation we need to implement reading in the last momenta, too, as well as the NHC state.

will be closed by #208


setx COGNITIVE_SERVICE_KEY "your-key"

[Enter feedback here]

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

ID: 7e866571-0651-bddb-f012-bf3bfe16f179
Version Independent ID: 27429467-3c37-27da-b839-065268c6af23
Content: Create a Cognitive Services resource in the Azure portal - Azure Cognitive Services
Content Source: articles/cognitive-services/cognitive-services-apis-create-account.md
Service: cognitive-services
GitHub Login: @aahill
Microsoft Alias: aahi

``

@DonQsec Thank you for your feedback! We will review and update as appropriate.

@DonQsec Could you provide more details of the issue you are facing the variables?

#please-close
Closing since there is no information about the issue here.


How can I add a new language?

I have downloaded a Ukrainian model and changed archive type to tar.gz, but I couldn't load it in the browser. I have looked into other archives but noticed they have a file of unknown type corresponding to each folder in archive. Is there a way to add a new language?
Thank you.

You can use any vosk model, you just need to set the path correctly when calling createModel. The model files should be in a gzipped tar archive. Can you share what error you get in the console?

You don't have to convert your archive to tar.gz, Vosk's zip archives works fine.

That is true, thank you for correcting me @3NdR1m
@Nata0801 please reopen with specific errors from the console


Checkpoints

Hi,
I wonder if you are going to release the checkpoints you used in the paper. That would be very useful.

Thank you very much for your interest. I assume you are interested in the multi-task checkoints. I could put them on a temporary storage once they are ready, which should be soon. Can you send me an e-mail?


AMBARI-25481. Customize Widget Threshold validation issue

What changes were proposed in this pull request?
Customize Widget Threshold validation issue
How was this patch tested?
manualy
Please review Ambari Contributing Guide before opening a pull request.

retest this please

Fails are not related to current changes. Merging this


Availability zones

This branch provides some new features, made possible by the generalisation and specialisation of the old VimInstance:

Availability Zone listed in the OpenstackVimInstance
KeyPairs listed in the OpenstackVimInstance
Possibility to choose AZ from the GUI at deployment time
Included GenericVImInstance for any other vim instance than OS and Docker
Extended DockerVimInstance with its own DockerNetwork and DockerImage

Still TODO

find appropriate place and way to handle different vim types in the NFVO
extend current specific vim types and fields inside the type

Also go-openbaton is now updated with this branch


Add Musl import, error if unrecognised platform

Add support for Musl
Also error if platform is not recognised.

@finagolfin you may be interested in this.

It was more about the package not compiling for Android anymore when it did before.

It was more about the package not compiling for Android anymore when it did before.

OK, this package should cross-compile fine for Android with the current 5.10 release, but the upcoming 6.0 release uses a new Android overlay, that this package has not been ported to yet.
I have the trivial patch needed for that available locally on my Android phone, just need to clean it up and submit it.

OK, this package should cross-compile fine for Android with the current 5.10 release

Ah that was the problem I found with Musl. It compiled fine, but the actual code caused crashes given the lock primitive defaulted to Int if the platform was unrecognised. https://github.com/apple/swift-async-algorithms/blob/6ae9a051f76b81cc668305ceed5b0e0a7fd93d20/Sources/AsyncAlgorithms/Locking.swift#L28

@swift-ci please test


fix(proto-guard): Use random temp directory

copilot:all

Current dependencies on/for this PR:

main

PR #4785   👈

This stack of pull requests is managed by Graphite.


[Question] Is it possible to expand node without selecting it?

Is it possible to expand a row without selecting it?
@maximilianoforlenza @cancerberoSgx @navarroaxel

Hi @mindaugasnakrosis! Do you want to expand the row when the mouse is hover on it?

Current problems is that expand node and select it, are the same operations currently. Would it be possible to separate them?
What I mean by that is only expanding the node, but not selecting it and other way around, clicking on it, selecting it, but not expanding/closing it.
Basically is it possible to do it like in this gif? This is done using another library.

@RokasVaitkevicius what library is that?

@maximilianoforlenza https://github.com/chenglou/react-treeview
But it is not maintained anymore, so difficult to use.

We need the ability to:

expand the node (one action)
select the row (another action)

ok. Yes it is possible to does. I'll start to work on it.

@maximilianoforlenza amazing! Looking forward to the changes.

@maximilianoforlenza how is it going? Maybe you need some help?

hi @mindaugasnakrosis ! I've published a new release https://github.com/storybookjs/react-treebeard/releases/tag/v4.2.4-beta.0 this is a beta version, please give me your feedback, if is ok I'll fix test and I'll publish stable version.
Now you have a new prop called onSelect this prop, send the node selected. Here is the example https://github.com/storybookjs/react-treebeard/blob/master/example/app.js and you have customStyles prop if you want to change the title color.

At first glance everything looks amazing, we will investigate further and let you know our final feedback.
Thanks a ton!

@maximilianoforlenza everything is perfect! Thanks again, will be closing it now.

@mindaugasnakrosis Thanks! I'll fix the unit test and then I'll publish the stable version.

Is this new version published?  I have version 3.2.4 and onSelect doesn't work.  And I don't see onSelect in the ReadMe, either.

If you want to have it, you have to use version "^4.2.4-beta.0". It has onSelect and onToggle separated and available. I don't know when it will be published and go out of beta.

Thanks for the hint.  But when I tried that version, the "toggle" part of the tree entries disappeared.  When I examine the DOM, the toggle is just plain missing.
I downloaded the current github source and am running with that. It works.


Harmonize LTO support for all platforms

Godot version:
master and 3.2
OS/device including version:
All
Issue description:
Several platform ports have support for Link Time Optimization (LTO) which makes significantly better optimization than non-LTO builds (so more performance, especially for GDScript). But this has been implemented gradually as the LTO technology itself matured, and it's not done consistently.
Nowadays both GCC and LLVM have good LTO support on all platforms, so we should be able to harmonize this build option and make it used consistently in production builds. Not sure about MSVC, needs to be checked (but in general MinGW-GCC builds with LTO will outperform MSVC, at least for the GDScript VM).
Platforms with LTO support currently:
$ rg \\-flto -l
platform/server/detect.py
platform/iphone/detect.py
platform/linuxbsd/detect.py
platform/windows/detect.py
platform/javascript/detect.py

It should be added to macOS.
Clang/LLD workflows as used e.g. for macOS and iOS can use -flto=thin which is much faster than the full LTO, and still provides good results. This should likely be made the default behavior when using such a toolchain.
MSVC LTO should be evaluated to see if it's a relevant option, and if so added to UWP too I guess (at least opt-in).

CC @godotengine/buildsystem

Fixed by #63288.


⚠ Bofrost.de has degraded performance

In b0570a7, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1287 ms

Resolved: Bofrost.de performance has improved in 0956de3 after 36 minutes.


Drawer animation direction is wrong in a rtl page

Current Behavior
The drawer animation direction is wrong in a page where dir="rtl" is specified.
Expected Behavior
The drawer animation should respect the dir attribute.
Steps To Reproduce

Add a drawer.
Set dir="rtl" on any parent of a drawer.
Open/close the drawer.

Link to Reproduction / Stackblitz
https://github.com/LoremFooBar/SkeletonDrawerRtlAnimationIssue
More Information
The workaround I found is to invert the direction of the drawer and invert back the direction of the contents:

  <Drawer width="w-fit">

       {#if $drawerStore.id === "app-sidebar"}
         <Sidebar />
       {/if}

  </Drawer>

Yeah, sadly Skeleton isn't quite as RTL friendly as we'd like right now.
The good news is that work on Version 3 has started and LTR/RTL, css logical properties are very high on the list of things to keep in mind: https://github.com/skeletonlabs/skeleton/issues/2314

Duplicate of: https://github.com/skeletonlabs/skeleton/issues/2314
@LoremFooBar we intend to address LTR/RTL support in Skeleton v3. So this would be considered a feature request for that upcoming version. As such I'm going to mark this as a duplicate for that effort. Thanks!

Great, looking forward to v3. Though I have to say Skeleton's current RTL support is pretty good because that's the only issue I found so far.


Hash Tables in JavaScript

Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
Proposed title of article
Hash Tables in JavaScript
Introduction paragraph(s):
Hash tables are an extremely useful data structure, which allows you to store data using key-value pairs. Hash tables store data in associative arrays.
In JavaScript objects and maps provides an efficient hash table structure. The key inputs run through a hash function. It maps the strings to numbers and numbers correspond to indexes in an array.
This article will make you understand hash tables in JavaScript, how they work, and their implementation
Key takeaways:

What are hash tables?
How hash tables work
Handling collisions
Implementing hash tables

References:
https://www.udemy.com/course/js-algorithms-and-data-structures-masterclass/
https://www.freecodecamp.org/news/how-to-implement-a-simple-hash-table-in-javascript-cb3b9c1f2997/
https://javascript.plainenglish.io/algorithm-in-javascript-hash-table-7b0464d2b81b
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

good topic @JudyNduati - approved


Fix segfault in TopP sampling layer

It's incorrect to call
mBufferManager->copy(runtimeTopP.data(), *mSetupWorkspaceDevice, runtime::MemoryType::kCPU);

and
mBufferManager->copy(runtimeTopK.data(), *mSetupWorkspaceDevice, runtime::MemoryType::kCPU);

because mSetupWorkspaceDevice size is [maxBatchSize] but runtimeTopP/runtimeTopK size is [actualBatchSize]

Hi @akhoroshev , thanks for your contribution to TRT-LLM, our engineer has merged your changes into our code base and make some improvements.
Anyway, we'll cite your name into contribution list.

Closing this out as it's been merged.


Implement DataFrame.info

This PR proposes DataFrame.info.
>>> import databricks.koalas as ks
>>> ks.range(100).info()
<class 'databricks.koalas.frame.DataFrame'>
Index: 100 entries, 0 to 99
Data columns (total 1 columns):
id    100 non-null int64

Resolves #872

Codecov Report

Merging #1124 into master will decrease coverage by 1.61%.
The diff coverage is 93.33%.

@@            Coverage Diff            @@
##           master   #1124      +/-   ##
=========================================
- Coverage   95.11%   93.5%   -1.62%
=========================================
  Files          35      35
  Lines        7019    7033      +14
=========================================
- Hits         6676    6576     -100
- Misses        343     457     +114

Impacted Files
Coverage Δ

databricks/koalas/missing/frame.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/frame.py
96.27% <100%> (-0.55%)
:arrow_down:

databricks/koalas/indexes.py
96.24% <88.88%> (-0.26%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
24.32% <0%> (-72.98%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
76.59% <0%> (-8.52%)
:arrow_down:

databricks/conftest.py
92% <0%> (-6%)
:arrow_down:

databricks/koalas/generic.py
95.76% <0%> (-0.43%)
:arrow_down:

databricks/koalas/plot.py
94.04% <0%> (-0.24%)
:arrow_down:

... and 1 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 0af6c1c...81623bd. Read the comment docs.


Block equations should be numbered

Current behavior
Block equations are not numbered.
Expected behavior
Block equations are numbered in order of appearance in the document.

One number per equation
Number should be visible only in the PDF output (not in the view mode)

Acceptance Criteria

[ ] Block equation numbers are displayed in the Document PDF (and Journal PDF later)

Sorry @frozenhelium, I thought I had an image of this 😆
We should use a parentheses with only the number (without the .................):
Some paragraph text
${E} = mc^2$ .................(1)
Another paragraph
${E} = mc^2$ .................(2)


multiplexing

Hi,
I'm trying to manage different connections through one connection. I haven't seen any clear example or document on how to do so. Does Boost Beast support multiplexing?
If yes please provide a clue on how I can achieve it, if not, please tell me if you have any idea how I can achieve this.
Thanks

Do you mean multiplexing in WebSocket? If so, there is no standardized method for multiplexing messages within a single stream. This task is considered a higher-level responsibility for the user. For instance, if you are working with JSON messages, you might structure them as follows:
{
 "stream_id": "1",
  "payload": "..."
}

Do you know any project that has implemented this?

Do you know any project that has implemented this?

Not that I'm aware of. but it shouldn't be difficult to implement. what is the message format that you are using?

I'm trying to write a WebSocket tunnel that can accept multiple TCP connection and send them through WebSocket.

Your message format should be in binary. A straightforward method of multiplexing involves appending a single byte to the start of each message, indicating the stream's state and ID. You may find the followings useful: https://ckousik.github.io/gsoc/2017/06/16/WebSocket-Multiplexer-Overview.html
https://github.com/xtaci/smux


Fix misleading variable description

Description
changes the description for the variable enable_manage_gitlab_token, since the second part of the description is misleading
Migrations required
NO
Verification
tried to build the infrastructure with this module and believed the wrong part of the description. So, I had chosen the wrong value for this variable.
Documentation
README was updated

@solutionDrive-Alt thanks for taking the time with creating the pull request. I also checked the code again.
I think the name of the variable enable_manage_gitlab_token is ok, but I agree we can come up with a better description. True means the SSM parameter is managed by terraform. False means that the parameter in SSM is still used by the the user-data script but the resource is in SSM is not managed. So once a terraform destoy is applied the token will remain in SSM. In case of true i will be remove.
What doe you think of the following proposal?
Boolean to enable the management of the GitLab token in SSM. If true the token will be stored in SSM however, which means the SSM property is a terraform managed resource. If false the Gitlab token will be stored in the SSM by the user-data script which during creation of the the instance. However the SSM parameter is not managed an will remain in SSM after a terraform delete.

@npalm no problem. I will probably work some more on our forked module. Maybe there will also be some changes this module can benefit from.
Now I understand this variable much better and I agree that your proposal for the description fits better than my just changed description. I will updated this PR later that day.

@solutionDrive-Alt thanks, looking forward to other improvements. I also have a branch prepared to move to TF 0.12. I will keep a branch 0.11 available.


Market creation mobile

Two issues to fix:

UI is displaying 2 columns on mobile, should stack elements as in design
The reporting start time inputs - issue when touch target size it seems, if you tap around the edges of the input it doesn't open the dropdown menu

Figma:
(scroll to the right to see the review step mobile designs also)
https://www.figma.com/file/csVtbWhGCAYe3nVe2LI1UEPM/Timezone-issue?node-id=181%3A1294

I can't follow that mock in a reasonable amount of time because I would need to refactor this whole page pretty substantially to simplify the whole thing -- so refactor in 1.14 but for now ill just flow it into a column as best i can, but i some things will be grouped in a way that's not exactly as the mock has them, just a heads up @cillianhunter0

@Chwy5 put this in v1.13

Looks better, 3 things that would help:
Can we have all elements containers run to the green lines here:

Can we make the text in these two inputs 2px smaller:

Can we eliminate this large gap, the same padding above the "market type" label ideally

Large gap is for scalar or categorical content. Removing the gap by default then the lower content will get pushed down when scalar or categorical is selected.

Removing the gap and pushing content down when needed is preferable
On Thu, Apr 25, 2019 at 11:16 AM Tom Haile<EMAIL_ADDRESS>wrote:

Large gap is for scalar or categorical content. Removing the gap by
default then the lower content will get pushed down when scalar or
categorical is selected.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AugurProject/augur/issues/1845#issuecomment-486613811,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALOMV74P2EJSME76CH7LWSLPSGAGZANCNFSM4HIFCPJQ
.

@Chwy5 these tasks sound like perfect v1.14 tasks. Recommend putting in v1.14

agree, ok to push
On Thu, Apr 25, 2019 at 11:19 AM Tom Haile<EMAIL_ADDRESS>wrote:

@Chwy5 https://github.com/Chwy5 these tasks sound like perfect v1.14
tasks. Recommend putting in v1.14
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AugurProject/augur/issues/1845#issuecomment-486614845,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALOMV77HWDEII2HCJQW5ETDPSGATPANCNFSM4HIFCPJQ
.

Looks more than usable for 1.13 pushing tweaks to 1.14

I can do this in a few min for 1.13, working on it now


re-generate ruby protos and add a test case

The proto generation script does not seem to work properly and will be fixed in a later try.
A compression test case is added to test compression path. The test case passed.
#6745

For reference, tested locally with commands:
server:
bins/opt/interop_server --port=50051 --use_tls=true
client:
ruby src/ruby/pb/test/client.rb --server_host_override foo.test.google.fr --use_tls true --use_test_ca true --test_case client_compressed_unary --server_host <IP_ADDRESS> --server_port 50051

I am glad to have an implementation of this interop test, but this doesn't actually resolve that issue. The point of that issue is to verify that Ruby code behaves correctly when receiving compressed messages, and that test just involves the ruby code sending compressed messages.

Added another case with receive. Is there any way we can check the incoming message is compressed?

Unfortunately, we can't make that check. The way I originally imagined writing this test was as a unit test with a Ruby client and server sending compressed messages to each other.

The last commit was reverted as a unit test is added in #7073.
I will leave this PR to you to see whether you want to close it or not.

Thank you for your help, but I think I will close this one. I think it would be better to implement that interop test once we have a complete implementation of compression in Ruby.


Set up markdown formatter for consistency

We've had a few PRs where different folks have different default formatters from their text editors auto-changing markdown, which leads to messy diffs. We should set up a consistent tool for this to avoid the noise, ideally one that easily integrates into the text editors being used by most maintainers.
I think the UI devs are mostly using vscode, and we use prettier for formatting our UI code, so that might be a good option, but I'm not sure how well that would work for our backend collaborators who are using intellij. Will need to investigate.

we could use mdox. I tried it and it looks ok.
The only thing I dislike is that it inline every paragraph. I would prefer to set a maximum number of character per line. That would be nice.

I have created an issue on the repo to tackle that: https://github.com/bwplotka/mdox/issues/100.
Maybe they will like the idea as well.


get_user_follow fails since new chrome version

Hi,
When scraping followers and followings, I receive the following error (since the update of chrome on my linux pc from version 98.xxx to 99.xxx):
Traceback (most recent call last):
File "~/Scweet/utils.py", line 385, in get_users_follow
log_in(driver, env, wait=wait)
File "~/Scweet/utils.py", line 305, in log_in
email_el = driver.find_element(By.XPATH, email_xpath)
File "/usr/local/lib/python3.8/dist-packages/selenium/webdriver/remote/webdriver.py", line 1244, in find_element
return self.execute(Command.FIND_ELEMENT, {
File "/usr/local/lib/python3.8/dist-packages/selenium/webdriver/remote/webdriver.py", line 424, in execute
self.error_handler.check_response(response)
File "/usr/local/lib/python3.8/dist-packages/selenium/webdriver/remote/errorhandler.py", line 247, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.NoSuchElementException: Message: no such element: Unable to locate element: {"method":"xpath","selector":"//input[@autocomplete="username"]"}
(Session info: headless chrome=99.0.4844.51)
Stacktrace:
#0 0x5613b9b24233
#1 0x5613b987a658
#2 0x5613b98b0941
#3 0x5613b98b0b01
#4 0x5613b98e3a34
#5 0x5613b98ce06d
#6 0x5613b98e17bb
#7 0x5613b98cdf33
#8 0x5613b98a3c8a
#9 0x5613b98a4d55
#10 0x5613b9b54c5d
#11 0x5613b9b6e17e
#12 0x5613b9b569c5
#13 0x5613b9b6ec95
#14 0x5613b9b4a900
#15 0x5613b9b89e28
#16 0x5613b9b89fa8
#17 0x5613b9ba36ad
#18 0x7f7219033609
Thank you for looking into this problem.

I've added Firefox support in the fork here:
https://github.com/rosscg/Scweet/tree/7aadb838eb5ff1db34f0eb1b5af0576c0a8a28bc
It's set as the default in get_users_follow so it can run without changing anything else.

Hi, @rosscg
Thanks for your contribution. I found that you import
from .const import get_username, get_password, get_email, get_geckdriver_path

but .const does not contains get_geckdriver_path.
Could you help me?

@ShoufaChen Yes that's an import I replaced (so shouldn't be there). I've updated the repo so either use the latest on my fork or just remove ", get_geckdriver_path" from that line in utils.py

Thank you @rosscg.

Thank you @rosscg and @ShoufaChen, it's working again.


fix time parse doc

Fixed document but i still ask why not use Time::Location.local by default if time string was not contain timezone?

I did not verify it. 😞
In fact the output is 2016-04-05 00:00:00.0 +08:00 Local now.

i still ask why not use Time::Location.local by default if time string was not contain timezone?

Using the local time zone is not always a sane default and can lead to unexpected errors.
See https://github.com/crystal-lang/crystal/pull/5324#issuecomment-348572486 for details.

Ping, could it merge?

I don't understand this api. Location is optional, but if a location can't be determined it raises? Like, imagine the time comes from user input: sometimes it works, sometimes it raises.
I think the default should be UTC, or else require a location, and maybe have parse_utc, parse_local and parse_with_location.
Just an opinion, though, but I think every other language assumes a location.

Assuming any location (be it UTC, local or anything else) is error prone and such an errors would be hard to detect.
But it could be an improvement to always require a value for location and maybe provide short cuts parse_utc and parse_local. It could still allow a nil value to enforce the parser raises when the formatted string doesn't provide location information. and/or have parse_with_location for that.
It could just be like this:
def self.parse(time : String, pattern : String, location : Location) : Time
  Format.new(pattern, location).parse(time)
end

def self.parse_utc(time : String, pattern : String) : Time
  Format.new(pattern, Location::UTC).parse(time)
end

def self.parse_local(time : String, pattern : String) : Time
  Format.new(pattern, Location.local).parse(time)
end

def self.parse_with_location(time : String, pattern : String) : Time
  Format.new(pattern, nil).parse(time)
end

This can be closed due to #6369. The fix was included there.


kbfs: 1.0.44 -> 2.1.1

Motivation for this change
Upgrade kbfs to the latest release.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg  build kbfs

Success on x86_64-linux (full log)
Attempted: kbfs
Partial log (click to expand)
/nix/store/b9fpzaaswlifdw4qd3389rmciyxw3gi5-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/vendor/golang.org/x/sys/unix/mkall.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/8zkg9ac4s4alzyf4a8kfrig1j73z66dw-bash-4.4-p23/bin/bash"
/nix/store/b9fpzaaswlifdw4qd3389rmciyxw3gi5-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/vendor/go.uber.org/zap/check_license.sh: interpreter directive changed from "/bin/bash -e" to "/nix/store/8zkg9ac4s4alzyf4a8kfrig1j73z66dw-bash-4.4-p23/bin/bash -e"
/nix/store/b9fpzaaswlifdw4qd3389rmciyxw3gi5-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/vendor/github.com/rcrowley/go-metrics/validate.sh: interpreter directive changed from "/bin/bash" to "/nix/store/8zkg9ac4s4alzyf4a8kfrig1j73z66dw-bash-4.4-p23/bin/bash"
/nix/store/b9fpzaaswlifdw4qd3389rmciyxw3gi5-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/vendor/github.com/keybase/go-codec/codec/tests.sh: interpreter directive changed from "/bin/bash" to "/nix/store/8zkg9ac4s4alzyf4a8kfrig1j73z66dw-bash-4.4-p23/bin/bash"
/nix/store/b9fpzaaswlifdw4qd3389rmciyxw3gi5-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/vendor/github.com/keybase/go-codec/codec/prebuild.sh: interpreter directive changed from "/bin/bash" to "/nix/store/8zkg9ac4s4alzyf4a8kfrig1j73z66dw-bash-4.4-p23/bin/bash"
/nix/store/b9fpzaaswlifdw4qd3389rmciyxw3gi5-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/genprotocol/fmt.sh: interpreter directive changed from "/bin/bash" to "/nix/store/8zkg9ac4s4alzyf4a8kfrig1j73z66dw-bash-4.4-p23/bin/bash"
/nix/store/b9fpzaaswlifdw4qd3389rmciyxw3gi5-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/libkbfs/gen_mocks.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/8zkg9ac4s4alzyf4a8kfrig1j73z66dw-bash-4.4-p23/bin/bash"
/nix/store/b9fpzaaswlifdw4qd3389rmciyxw3gi5-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/kbfscodec/gen_mocks.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/8zkg9ac4s4alzyf4a8kfrig1j73z66dw-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/b9fpzaaswlifdw4qd3389rmciyxw3gi5-kbfs-2.1.1...
/nix/store/n1369v8np6p8s8s2xyd1wmsrspnrircm-kbfs-2.1.1-bin

Success on aarch64-linux (full log)
Attempted: kbfs
Partial log (click to expand)
/nix/store/630yvlcdp9c7yg0ixgswc9974h00milj-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/kbfsfuse/kbfsfuse.sh: interpreter directive changed from "/bin/bash" to "/nix/store/p0vy17dp9jk2mvqsxsqnb14s3797lay7-bash-4.4-p23/bin/bash"
/nix/store/630yvlcdp9c7yg0ixgswc9974h00milj-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/libkbfs/gen_mocks.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/p0vy17dp9jk2mvqsxsqnb14s3797lay7-bash-4.4-p23/bin/bash"
/nix/store/630yvlcdp9c7yg0ixgswc9974h00milj-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/vendor/github.com/keybase/go-codec/codec/prebuild.sh: interpreter directive changed from "/bin/bash" to "/nix/store/p0vy17dp9jk2mvqsxsqnb14s3797lay7-bash-4.4-p23/bin/bash"
/nix/store/630yvlcdp9c7yg0ixgswc9974h00milj-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/vendor/github.com/keybase/go-codec/codec/tests.sh: interpreter directive changed from "/bin/bash" to "/nix/store/p0vy17dp9jk2mvqsxsqnb14s3797lay7-bash-4.4-p23/bin/bash"
/nix/store/630yvlcdp9c7yg0ixgswc9974h00milj-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/vendor/github.com/rcrowley/go-metrics/validate.sh: interpreter directive changed from "/bin/bash" to "/nix/store/p0vy17dp9jk2mvqsxsqnb14s3797lay7-bash-4.4-p23/bin/bash"
/nix/store/630yvlcdp9c7yg0ixgswc9974h00milj-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/vendor/go.uber.org/zap/check_license.sh: interpreter directive changed from "/bin/bash -e" to "/nix/store/p0vy17dp9jk2mvqsxsqnb14s3797lay7-bash-4.4-p23/bin/bash -e"
/nix/store/630yvlcdp9c7yg0ixgswc9974h00milj-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/vendor/golang.org/x/sys/unix/mkall.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/p0vy17dp9jk2mvqsxsqnb14s3797lay7-bash-4.4-p23/bin/bash"
/nix/store/630yvlcdp9c7yg0ixgswc9974h00milj-kbfs-2.1.1/share/go/src/github.com/keybase/kbfs/vendor/golang.org/x/sys/unix/mkerrors.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/p0vy17dp9jk2mvqsxsqnb14s3797lay7-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/630yvlcdp9c7yg0ixgswc9974h00milj-kbfs-2.1.1...
/nix/store/pqzxmmm19w64b80s2j8cfzjz4gfabhnq-kbfs-2.1.1-bin

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: kbfs
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Should be merged after #42731.


WFLY-16935: add EnterpiseDB to recongized dialects, throw exception i…

…f dialect cannot be determined
Issue: WFLY-16935

Hello, michpetrov. I'm waiting for one of the admins to verify this patch with /ok-to-test in a comment.

Thanks @michpetrov


Color/Regular inconsitency

The monochrome NotoEmoji-Regular.ttf still uses the old "blob-style" emoji, while NotoColorEmoji.ttf uses the new "Oreo-style" emoji. Besides the different look, the color version has more complete coverage.
Will the monochrome emoji receive an update, or are they considered obsolete?

Hm sorry, looks like I didn't search well enough.
From #35:

Unfortunately, we're not currently budgeted to design full b/w emoji.

So the monochrome are a second-tier, community-supported font.

So the monochrome are a second-tier, community-supported font.

Yep, I'm afraid so.

Thanks for the clarification. 😊


Allow piped information to be appeded to end of prompt.

I want to be able to do

shell operation | aia prompt_id --options
and have the output of the shell operation to be appended to the prompt text.

Also related I want the same thing to happend when I do:

aia prompt_id --options < some_file.txt

I have this working in v0.5.15


Update svgr packages

What are you changing?

updating svgr dev dependancies

Why?

housekeeping

Do we still need @types/svgo? SVGO 3.x includes type definitions so seems like these aren't needed:
https://www.npmjs.com/package/@types/svgo


[Question] - How weed deals with archived and zip files in partial downloading

Hi,
I'm uploaing a 2MB text file in weed , after uploading it return the size of file 2mb but it saved it 300KB,
So weed is archiving and changing my file format to zip,
My question is about partial downloading the file:
When I want to download my file I say : plz give me the file from offset 0 till offset 200000 (The first 200KB of my file) and then
next 200KB up to the end ...

How weed would deal with this issue when I'm downloading the first chunk, it will unzip the whole file and it would give me it partially ? if it's true  where it keeps the unzip file (in ram or a directory on system? )

think about situation that I want to get the first chunk of my file but I deal with an error (I dont know any error eg: connection error) and again I try , each time weed will unzip my file ?

It will unzip the whole file and it would give it partially. in ram.
yes.

2. yes.

@chrislusf
1.How weed understands that it should remove the file from ram ? when I return 200 or it will remove it after each response and for new request it will load it on ram again eg: I'm downloading my file 200 KB by 200KB up to the end continuously , after each partial request weed will remove the file from ram and again for new request it will unzip it in ram ? How weed deals with this issue?

it will remove it after each response and for new request it will load it on ram again

^^

it will remove it after each response and for new request it will load it on ram again

^^
@chrislusf
So for a 300 MB file if I want to give it partially with each time 300kB, weed should 1000 times unzip the file and because requests are continuously it has 2 negative aspect:

it has so overhead for Cpu (for unziping)
I should wait for my file to decompress each time
How weed deals with this 2 issue ? for the first issue how much is the overhead of unzipping?
Can you explain it more?

It is not optimized for this use case.


MERN_NodeJS: 4.1 NPM vs Yarn

Description
Documentation for NPM vs Yarn, and its related subtopics.
Domain
MERN
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Misspell in description inside settings UI

UI misspelling
Small spell in the VSCode UI settings configuration
Line 73 "description": "If autobuild is eanbled" should be "description": "If autobuild is enabled"

Good catch, thank you. We'll fix this together with the next update.


fix: headers are not applied to stream response

I was also thinking about cloning the headers in the other From implementation:
impl From<EdgeResponse> for Response {
    fn from(res: EdgeResponse) -> Self {
        Self {
            // CLONE HERE
            headers: Headers(res.headers() /* .clone() */),
            status_code: res.status(),
            body: match res.body() {
                Some(_) => ResponseBody::Stream(res),
                None => ResponseBody::Empty,
            },
        }
    }
}

A response coming from the cache will not have modifiable headers, but this might sometimes not be necessary, would help a lot with ergonomics though.
Edit: Forgot I opened an issue for that, this fixes #112

I was also thinking about cloning the headers in the other From implementation:
impl From<EdgeResponse> for Response {
    fn from(res: EdgeResponse) -> Self {
        Self {
            // CLONE HERE
            headers: Headers(res.headers() /* .clone() */),
            status_code: res.status(),
            body: match res.body() {
                Some(_) => ResponseBody::Stream(res),
                None => ResponseBody::Empty,
            },
        }
    }
}

I don't really see a point in the cloning, the headers at that point are still owned by JavaScript so we'd just be cloning essentially a reference into the wasm-bindgen object table. If this becomes a problem feel free to open an issue about it.

@zebp the clone does more than just copy the reference:
impl Clone for Headers {
    fn clone(&self) -> Self {
        // Headers constructor doesn't throw an error
        Headers(EdgeHeaders::new_with_headers(&self.0).unwrap())
    }
}

It actually creates a new JS Headers object, which is not immutable anymore (Headers returned from the Cache etc. are readonly).


PDA cartridges should be limited or single-use

Description
All PDA cartridges are infinitely reusable between multiple PDAs
You can equip the entire security department with a single LogProbe cartridge.
Reproduction

Load PDA cartridge into PDA 1
Install app
Eject cartridge
Repeat with other PDAs

Additional context
Cartridges should either be deleted or burnt-out/locked after using a single or multiple charges ("licenses")

This is a good thing.

You just need to set the installation status field of the CartridgeComponent to Readonly.
The field doesn't have a for some reason datafield yet so you'd need to add that:
    [AutoNetworkedField]
    public InstallationStatus InstallationStatus = InstallationStatus.Cartridge;

Aside from it not being a data field, making cartridges not be installable is already supported.
I don't really see a reason why we'd need that for the log probe though.

This is a good thing. As far as I know you can not buy new cartridges, so making them single use would be awful in the case someone loses the only pda on the entire station with the software.

I should clarify some cartridges should be limited-use, not all of them
Also, I've included a LogProbe in a crate PR


Migrate AWS resources (batch 2)

@sinabakh can you please review this if you get a chance?

@aliscott LGTM


CBL-2234 : Implement Property Encryption API

Implemented CBLEncryptable and its related API.
Implemented Replicator config for CBLPropertyEncryptor and CBLProperyDecryptor callback.
Added tests.
Open Issues : CBL-2263, CBL-2264, CBL-2265, CBL-2271

PR Validation (9) on Jenkins : Green on Linux and Windows; Mac Machine was cancelled as the machine was in bad state.


Jquery warns Override

When creating a new project with ox it throws the following error when running:
jquery-exposed-exposed.js? ./ node_modules / expose-loader / dist / cjs.js? exposes = $, jQuery: 5 Uncaught Error: [exposes-loader] The "$" value exists in the global scope, it may not be safe to overwrite it, use the "override" option
my solution for this problem was to remove the first line from the application.js file
require ("expose-loader? exposes = $, jQuery! jquery");

This is an interesting one. It's related with updating our tooling to avoid using JQuery.


Fix some typos

"assignement" -> "assignment"
"indicies" -> "indices"
"RedndantSelf" -> "RedundantSelf"
"correcly" -> "correctly"
"arugment" -> "argument"
"undersoce"-> "underscore"
"regesters" -> "registers"
"whtn" -> "when"
"ergister" -> "register"
"redendant" -> "redundant"
"officense" -> "offense"
"exitted" -> "exited"

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[-] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[-] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[-] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.
[-] If this is a new cop, consider making a corresponding update to the Rails Style Guide.

Interesting. Did you find this byany checking tool?

cspell and my eyes visually detect these typos.
In this case, you can do $ cspell . /**/* and visually check and fix any suspicious parts.

I get it! Thank you!


Screensaver?

Although it's a nice idea, I'd rather not include this functionality here. I'd prefer some other plugin be responsible for screensaver functionality and trigger cellular automata animation.

Thanks!


JAVA Iterators & Essential Methods

Iterators

[ ] iterator()
[ ] listIterator()
[ ] for each style

Essential Methods

[ ] Comparator

[ ] comparable,compareTo(), compare()

[ ] equals(), hashCode(), toString()
[ ] size()
[ ] isEmpty()
[ ] pollFirst() & pollLast()
[ ] poll()

@Kadam-Tushar an additional task would be to monitor all the java issues and check if something is left to add 😄

Okay !


Appearently its not possible to get headers if they are defined multiline

I am trying to communicate with an API which sets multiple Set-Cookie headers but i am able to only read the first one using
        res.headers()
                .get("Set-Cookie")
                .map(|val| val.to_str().unwrap_or_default())
                .unwrap_or_default()
                .to_string(),

Try using get_all(); you should see each instance of the response header as a separate value.


Clublog Ignore Switch

Due to the possibility to add a callsign, you do not own on Clublog, there needs to be a Ignore switch to turn off Clublog Upload for this particular location/callsign

Looks good. Works for creating new/editing locations. Everything is set in db. No errors.
Have not tested the Club Log upload, but it should work when looking at the code.


Entire network re-interviewed on restart, S2 secure devices ignore commands until re-joined (intermittent)

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?
NO, my problem is NOT within Home Assistant or the ZWave JS integration
Is your problem within ZWaveJS2MQTT?
NO, my problem is NOT within ZWaveJS2MQTT
Checklist

[X] I have checked the troubleshooting section and my problem is not described there.

[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
What causes the bug?
Specific root cause unknown, but appears to manifest on process restart.
What do you observe?
Main noticeable symptom is S2 secure nodes ignore all commands. Upon examining the logs, it appears to start with "Restoring the network from cache failed: SyntaxError: Unexpected end of JSON input"
What follows is all nodes are re-interviewed with the appearance of success, but all nodes that use S2 security as opposed to no security will not respond to any commands until a full reset/re-inclusion of the node is performed.
What did you expect to happen?
Network to remain functional when process is restarted.
Steps to reproduce the behavior:

Process restarts. (This time due to cron job that issues "snap restart zwavejs2mqtt" every other day at 3:00 am local time, previous times likely on automatic snap refresh. Cron job was put in place due to lingering stability issues that seem to be fixed upon restart.
Notice nodes that have S2 security fail to respond to any z-wave commands and only respond to physical switch inputs until re-set and re-included. I only have devices with no security and S2 security. All devices were added with the default security options, so devices aquired since S2 was supported ended up with S2 authenticated security. In order to avoid future headaches I rejoined the nodes this time explicitly with no security so that if this happens again it will be largely transparent.
Notice that battery powered nodes need to be woken up to complete their interview before working. I have a battery powered scene controller and a door open/close sensor that would not work at all until manually woken up.

Device information
Manufacturer: Zooz
Model name: Zen 71, Zen 34, XS 41, XS 42
Node ID in your network: 21, 22, 23, 24, 25, 26, 27, 33
How are you using node-zwave-js?

[ ] zwavejs2mqtt Docker image (latest)
[ ] zwavejs2mqtt Docker image (dev)
[ ] zwavejs2mqtt Docker manually built (please specify branches)
[ ] ioBroker.zwave2 adapter (please specify version)
[ ] HomeAssistant zwave_js integration  (please specify version)
[ ] pkg
[ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
[ ] Manually built from GitHub (please specify branch)
[X] Other (please describe)

Which branches or versions?
snap from https://snapcraft.io/zwavejs2mqtt
version: 6.3.1, 6.4.1 was upgraded to during troubleshooting
node-zwave-js branch: 8.10.1, 8.11.1
zwavejs2mqtt branch: 6.3.1, 6.4.1
This same issue has occurred in the past but I did not troubleshoot as extensively then. I believe it was around 6.1 or 6.2 where I had the same issue with S2 devices not functioning, and in 5.9 where I had no secure devices, but the interview process for the nodes had difficulty completing.
Did you change anything?
no
If yes, what did you change?
n/a
Did this work before?
Yes (please describe)
If yes, where did it work?
This issue only happens sporadically. The snap restarts on its own periodically when there is a refreshed version, and lately I have implemented a cron job to restart it every other day due to some observed stability issues. I suspect those issues may be due to the 700 series controller meta issue but this issue is different from that.
Attach Driver Logfile
zwavejs_2022-01-21.log
I apologize this is at log level info and not debug. As this isn't readily reproducible I don't have logs running in debug all the time.

@jmgiaever I think this needs your help. This is a case of the <homeid>.json file getting corrupted because of an improper shutdown.

Hi, I'm actually suspecting that the main issue here is the 700 stick problem.
The last time it happened to me that the cache got corrupted, I spent many days getting it up running again.
Many devices didn't interview properly and some not at all. I had to wait for the network to be "interviewed" and then interviewed one by one - and sometimes several times as entities configurations were still missing.
I also expect the 700 issue to be the reason for instabilities, and not the snap package.
Having a crontab to restart it is not a good idea.
E.g can happen at the same time as a snap refresh and possibly corrupt the cache.
I would recommend doing as I did; convert back to a 500 stick. It's a difference between day and night.
There's a backup and restore option in Z2M under the advanced options.

PS! You can copy the cache from the previous revisions. It's located in the directory
/var/snap/zwavejs2mqtt/<rev>

The subdirectory current within this directory is most likely pointing to the latest/highest revision, that you're on. Try copying the cache from the lower revisions into this directory.
Stop zwavejs2mqtt first.

@jmgiaever it definitly is a corrupted cache file:

"Restoring the network from cache failed: SyntaxError: Unexpected end of JSON input"

Yes, but using a crontab to restart the app doesn't sound like a good solution to fix his 700 instabilities.
If the snap is refreshed at the same time, e.g closing down or just starting up, it can cause the cache to crash?
The snap doesn't touch the cache, except that it's copied as a backup between revisions. But that is synced, so a failed backup where content differs would lead to a failed refresh and roll back.

I have disabled the crontab from your feedback. I had some issues that would be resolved by a restart so had put that in place a couple weeks ago as a band-aid. Crontab was simply:
0 3 */2 * * /usr/bin/snap restart zwavejs2mqtt
I later discovered the 700 series controller issue and have a 500 series stick on order.
Both of the controller sticks are the Zooz ZST-10, the 700 series currently running and a 500 on order that I'll be migrating to. Let me know if there's anything else I can do to help troubleshoot and contribute.
Thanks for hopping on this so fast and thanks for developing such a wonderful framework for the home automation community!

In the upcoming v9 release, the .json cache file will be changed to an appendonly .jsonl file like the other two. I'm expecting this to fix your problems.

Appreciate the update. For what it's worth, I haven't noticed a recurrence of this issue since updating to the 7.17.1 firmware on my ZST10 700 stick. However, it's possible it has happened and I didn't notice it as all devices were re-interviewed successfully. As part of the troubleshooting of this issue I purposely left all my devices with no security.


[Doc Feature][Azure key vault]

Docs Build status updates of commit 5cb51e3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/toolkit/add-resource.md
:white_check_mark:Succeeded
View

msteams-platform/toolkit/provision.md
:white_check_mark:Succeeded
View

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 docs.microsoft.com contributor guidesPost your question in the Docs support channel


[EnivronmentParser] WiP and initial implementation for parsing and mergi...

...ng the travis environment

I cannot merge this

Now you can :)

ace, let's do it.


(maint) Create puppetlabs.testutils namespace

Several tests reuse or redefine several functions that help in
interacting with puppet catalogs, so this commit collects them in a
generic namespace for reuse. Tests that depended on the old functions
are updated to use the new ones.

:clap: :+1:

This is awesome, thanks a ton for doing it.

Rebased and renamed namespace to puppetlabs.puppetserver.testutils.

Looks like this is going to need another rebase. Travis also failed on Java 8 but it looks transient, so I re-ran the build.

Yup...I'll rebase this afternoon.

rebased...running tests locally and eager to see what travis has to say.

(local tests pass for me on this)

oh hey. https://tickets.puppetlabs.com/browse/SERVER-709 looks related to this.

tests passed for me locally.


LocalPCInfo/MicrophoneIsMuted seems to always be true.

Expected Behaviour

I have added a new layer to the desktop profile, I want to use an override setting to enable it when the microphone is muted. I use the value LocalPCInfo/MicrophoneIsMuted
Actual Behaviour

The value LocalPCInfo/MicrophoneIsMuted does not change when microphone is muted or unmuted
Reproduction steps

Create a layer in Desktop profile (don't know if it applies to other profiles)
Create an override condition to enable and disable layer depending on LocalPCInfo/MicrophoneIsMuted.
Tick the "Debug Values" box in the upper right corner to see the value of LocalPCInfo/MicrophoneIsMuted

Frequency

Every time.
Aurora Version:

beta v0.8.1
Previously an Issue?

Don't Know

Check if it is actually muting in windows.

Try to choose your mic in the settings and restart aurora.

If aurora isn't able to get the data from your mic you can loop your mic though https://vb-audio.com/Cable/ getting it's mute state is known to work.


Update manifests/t/Tencent/QQ/<IP_ADDRESS>059/Tencent.QQ.locale.en-US.yaml

Fixes an issue where this package has multiple different monikers
Microsoft Reviewers: Open in CodeFlow

Needs Attention

@wingetbot run

[Policy] reset feedback

[Policy] Needs-Attention

@wingetbot run

@wingetbot run


Add "Juice that makes you Weh" to moths eating Lizard plushies

About the PR

Eating a Lizard Plushie as a moth now also makes you consume 10u Juice that makes you Weh.
Why / Balance

10u was chosen for 2 reasons:

Just finding and eating a plushie shouldn't be as rewarding as having to go through the additional process of juicing the plushie. 10u Wehjuice lasts for 40 seconds, while 30u Wehjuice lasts for 2 minutes. See it as juicing the plushie gives a higher, purer yield.
Each bite removes 5u of the food's reagents. Having 10u fiber, 30u Wehjuice would take a whole 8 bites to get through. 10 fiber, 10u Wehjuice takes 4 bites, which is more manageable.

Technical details

Just edits the SolutionContainer for those plushies. Plushies can't be drawn/injected with a syringe.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

tweak: Moths eating lizard plushies start to Weh!

THE MOTH WEH IS REAL
👍


"Help Page" footer section not properly aligned.

Footer section is misaligned on mobile and tablet devices.
Please assign this issue to me under SWOC'24.

Hi @TeeWrath ! I'm interested in fixing this issue.
Please assign this issue to me.

@sagnik3788 , i guess i have merged this issue's commit in the PR #39 , named "Fixed the footer alignment of help page"
76d15a51840f628fcc1e29ea65e3253767bb86a4


'Market not looking so good'

The bot constantly gives me the sign that the market is not looking so good at the moment so it won't buy anything. Does anyone have the same problem, or is it due to my configurations.

I am not 100% sure but I turned down the trading signal file from 18 out of 26 or whatever down to 10.  I think that helps.  The newer bot also is using the trading signal and pause file.  I think you can edit the pause file as well but did not look at it yet, so that it will not pause.


help configure WHIP obs stream

I apologize for such a simple question. I've read readme 3 times, but I still don't understand how to do what I want to do.
I have a static IP, I know how to forward port through my router. OBS and go2rtc running on same Windows 10 LTSC pc. I want to publish the stream on my IP, on a specific port where only the stream will be, and hide everything else.
My config for now:

go2rtc:stream:
  stream: {}

webrtc:
  candidates:
    - <my IP>:80

OBS:
Service WHIP
Server http://localhost:1984/api/webrtc?dst=stream
And I forward 80 external port to 8555 internal port in my router.

OBS connected to go2rtc and I can watch the stream, but it goes to the 1984 port. Is there any way I can change my configurations to safely publish my stream to the outside, or is this only possible with a reverse proxy?

You need to use reverse proxy like nginx. Or use webtorrent sharing, check go2rtc docs

check go2rtc

I've already read this, but I don't know anything about webRTC or webtorrent, to really understand the doc. I'm a simple guy who wants to share my screen in good quality and low latency with a friend.

Well. Open WebUI, select your stream, goto links page, click to share link, click to copy link, send link to your friend.


How to implement utp protocol based on netty

please help me。thank u

Sorry but this is an issue tracker... please use stack overflow, that said you will need to at least invest a bit more time to try read-up on stuff before asking I would say.


Disable test that has changed the flow

I had a quick look and the only test that is constantly failing is extension/test/e2e/credit-card-3ds-native.spec.js with MasterCard.
The issue is the payment verification page for MasterCard 3D has changed and is now different from before. Therefore our selectors do not work anymore.
After disabling MasterCard tests are working again.
Suggestion: I would remove completely MasterCard from the 3D verification test and leave Visa there. For testing the flow it's enough. Having another credit card do not add much value comparing to the maintenance we have to do. WDYT?

Kudos for finding the issue 👍🏽

Thanks :) so should I remove the MasterCard test? WDYT?

Kudos for finding the issue 👍🏽

Thanks :) so should I remove the MasterCard test? WDYT?

Yes, I think it's flaky and we are covering the main functionality already 👍🏽


Hotel Partnerships

copied from here:
https://github.com/codeforgreenville/trolley-tracker/issues/143#issuecomment-219917602
I went and spoke with Brandon at the Hyatt and they will not have any sort of stand or anything visible. I asked if he had heard of the trolley tracker app, he said no. He said most people think they provide transportation, but they hand out physical trolley maps, but are currently out of them.
I'm not sure we want to get into the 'making/distributing flyers' business. I think getting the workers there to suggest the trolley tracker would be simply a matter of request/training. Show them the app, and ask them to share with fellow workers.
I don't know what a conversation with someone higher up would be like, suggesting they hang a tv with the trolley tracking app on it...with restaurants and points of interest...


Adds OAStackViewProxy to support native UIStackView in iOS 9

For now, it only supports it in code

As mentioned in #2 - I preferred to do the UIView approach.
All properties needed to be declared and the setters pass the property to the correct stack view (native or backwards compatible).
The methods declared in the interface didn't need to be implemented, as the class implements methodSignatureForSelector: and forwardInvocation.

And bear in mind it doesn't support interface builder yet.

That is a nice approach, Thanks!
Do you think you can add a simple test (preferably on a new spec file)?

Hey Omar! I'll try to add it soon.

Please also don't forget to rebase since I fixed the test failing issue in iOS7. Again wonderful addition, I think we can expand on it to have a seamless iOS 9 integration :)

Hey, Any updates on this, I can add the tests and rebase in the weekend if you want.

Hey Omar! I tried to rebase a few days ago, but had some problems (did you change the file tree?). If you could do it, that would be awesome.

Sure I will do that no problem!
Yeah, Sadly I added some files, and moved some code around. Anyways, I will rebase and add some tests.
I will create a pull request with the tests so that you can give me your opinion on wether they cover all your planned scenarios. I will share them in a new branch prior to merge.
Does that sound good to you?
Thanks for the great addition btw!

@natanrolnik maybe you'll suggest me how to use native NSLayoutAnchor API category if API is available? https://github.com/k06a/MissingAnchors/blob/master/Pod/Classes/ABLayoutAnchor/UIView%2BABLayoutAnchor.m

Hm, interesting... I will try to take a look.

Closing as #87 was merged already


new(libsinps): add a new API in sinsp to obtain the events associated with a set of ppm_sc

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area libsinsp
Does this PR require a change in the driver versions?
No
What this PR does / why we need it:
This PR introduces a new API in sinps that can be used to obtain the set of events associated with a set of ppm_sc. The typical use case is when Falco needs to understand if all the events used in the rule are considered in our set of interesting ppm_sc
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
new(libsinps): add a new API in sinsp to obtain the events associated with a set of `ppm_sc`

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 00c06517b339187ac1a208f1dd62b6fc96e9ff7e

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold
Because the CI is broken

/unhold

LGTM label has been added.  Git tree hash: a48fecbd6b629d6d646811a60741e444fff508bf

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold

/unhold


remove exception "ConcurrentModificationError"

"ConcurrentModificationError (Concurrent modification during iteration: Instance of '_HashSet'.)"

How to reproduce this error?

How to reproduce this error?

reproducing when use the public method dropModules.

Please provide an example in which the error is reproduced

Please provide an example in which the error is reproduced

Please provide an example in which the error is reproduced


FadeAnimation.dart needs update

While running flutter run

Stack overflow solution👇
currently the Methods has been changed. MultiTrackTween changed into MultiTween, ControlledAnimation changed into PlayAnimation and a few

has it been resolved for you. The stack solution isnt working. Any suggestions?


Remove rosetta2 hack for arm64 on Mac

We introduced this hack when there were very few JDKs available for the ARM64 variant of Mac.
Subsequently, many versions of Java have become available for ARM64 Mac, making this hack redundant. This PR removes all support for this functionality.

We introduced this hack when there were very few JDKs available for the ARM64 variant of Mac.
Subsequently, many versions of Java have become available for ARM64 Mac, making this hack redundant. This PR removes all support for this functionality.

Hi, maybe it’s too early to remove this option? There are still plenty of java native x86 libraries which does not have an arm port. There is also jdk bugs (e.g. Unsafe#getLongVolatile) in arm where it’s nice to be able to switch over to x86.


Panda_movit tutorial not working

When trying to use the roslaunch panda_moveit_config demo.launch I get a robot with no linked links.

Make sure that franka_ros is in the PATH see the moveit documentation


Lab 6: Step 3 - W Object not found

I'm trying to identify the salary per win in part 3, and I'm getting the below error message:
Error: Problem with mutate() column salary.per.win.mutate.
i salary.per.win.mutate = total.salary/W.
x object 'W' not found
i The error occurred in group 1: teamID = ANA.
Run rlang::last_error() to see where the error occurred.
In Step 2 I saved the merged dataset as Combined, and when I look on the right I can see that the object W is listed in the dataset. I also was able to try to create the object salary.per.win without using the pipe operator and it worked fine, so I know that W is in the dataset Combined
salary.per.win <- sum(Combined$salary.adj) / Combined$W
But when I try to use the mutate function within the pipe, it gives me the object not found error. I've also tried reordering my summarize and mutate functions and the arguments in each to see if the order that they're in matters, but it hasn't gotten me anywhere. Code below:
Combined %>%
group_by(teamID, yearID) %>%
summarize(total.salary = sum(salary.adj),
players = n()) %>%
mutate(salary.per.win.mutate = total.salary/W,
salary.per.100k = total.salary/100000)
What am I missing? @gmcirco

A few issues here:
When you do summarize you will drop all columns not specified in the summarize function. So your group_by %>% summarize function will only retain teamID, yearID,  total.salary, and players. Rather than doing that, you should use mutate to retain all the other columns and then you can sum over them later. This way you will keep the W column in the data as well.