nuxt 3 production build throws errors

Reproduction
http://localhost:3000/test
Describe the bug
Hi @justin-schroeder  @pi0
Thank you for all the amazing work.
I have installed the recent nuxt 3 and formkit beta4/5 in both If I do do
yarn build and then yarn start
It throws errors.
But in development mode yarn dev it works just fine.
To reproduce you can add a formkit to any page and do a yarn build and then yarn start then load the page containing the form.
Let me know If you want to track in another issue
Environment
• OS: [e.g. iOS] LINUX
• Browser [e.g. chrome, safari] ALL
• Version [e.g. 22] irrelevant
Install nuxt 3
install formkit beta4/5
Error: E600
    at error (file:///projectDir/.output/server/node_modules/@formkit/vue/node_modules/@formkit/core/dist/index.mjs:194:11)
    at useInput (file:///projectDir/.output/server/node_modules/@formkit/vue/dist/index.mjs:873:9)
    at setup (file:///projectDir/.output/server/node_modules/@formkit/vue/dist/index.mjs:666:22)
    at callWithErrorHandling (file:///projectDir/.output/server/chunks/index.mjs:1957:22)
    at setupStatefulComponent (file:///projectDir/.output/server/chunks/index.mjs:7517:29)
    at setupComponent (file:///projectDir/.output/server/chunks/index.mjs:7498:11)
    at renderComponentVNode (file:///projectDir/.output/server/chunks/index.mjs:9853:17)
    at Object.ssrRenderComponent (file:///projectDir/.output/server/chunks/index.mjs:10284:12)
    at _sfc_ssrRender$1 (file:///projectDir/.output/server/chunks/app/server.mjs:6093:32)
    at renderComponentSubTree (file:///projectDir/.output/server/chunks/index.mjs:9924:13)
[Vue warn]: Component  is missing template or render function.
[Vue warn]: inject() can only be used inside setup() or functional components.
[Vue warn]: inject() can only be used inside setup() or functional components.

test.vue contents

<template>
  <FormKit
    :type="'text'"
    label="Category Name"
    help="name of the Category to be used"
  />

  <FormKit
    :type="'text'"
    label="slug/handle"
    help="the path that will be used in the URLS"
  />
</template>

Those errors sound like there are multiple versions of vue installed on your project. Are you able to try a fresh copy of nuxt3 + @formkit/nuxt@next
Here is a working copy for reference:
https://stackblitz.com/edit/nuxt3-formkit?file=app.vue

@justin-schroeder
Yes you were looks like one vue version of formkit got also installed, Now it is fine but having an different issue, just tried one form one of the example now I have this error
[Vue warn]: inject() can only be used inside setup() or functional components.
[Vue warn]: inject() can only be used inside setup() or functional components.
Error: Unknown input type "form" ("form_1")
    at error (file:///home/msonowal/sites/project/.output/server/node_modules/@formkit/core/dist/index.mjs:194:11)
    at useInput (file:///home/msonowal/sites/project/.output/server/node_modules/@formkit/vue/dist/index.mjs:873:9)
    at setup (file:///home/msonowal/sites/project/.output/server/node_modules/@formkit/vue/dist/index.mjs:666:22)
    at callWithErrorHandling (file:///home/msonowal/sites/project/.output/server/chunks/index.mjs:1957:22)
    at setupStatefulComponent (file:///home/msonowal/sites/project/.output/server/chunks/index.mjs:7517:29)
    at setupComponent (file:///home/msonowal/sites/project/.output/server/chunks/index.mjs:7498:11)
    at renderComponentVNode (file:///home/msonowal/sites/project/.output/server/chunks/index.mjs:9853:17)
    at Object.ssrRenderComponent (file:///home/msonowal/sites/project/.output/server/chunks/index.mjs:10284:12)
    at file:///home/msonowal/sites/project/.output/server/chunks/app/server.mjs:6410:36
    at renderComponentSubTree (file:///home/msonowal/sites/project/.output/server/chunks/index.mjs:9924:13)
[Vue warn]: Component  is missing template or render function.

template
<template>
  <FormKit
    type="form"
    v-model="formData"
    :form-class="submitted ? 'hide' : 'show'"
    submit-label="Register"
    @submit="submitHandler"
  >

Register!

      You can put any type of element inside a form, not just FormKit inputs
      (although only FormKit inputs are included with the submission).

    <FormKit
      type="text"
      name="name"
      label="Your name"
      placeholder="Jane Doe"
      help="What do people call you?"
      validation="required"
    />
    <FormKit
      type="text"
      name="email"
      label="Your email"
<EMAIL_ADDRESS>      help="What email should we use?"
      validation="required|email"
    />

      <FormKit
        type="password"
        name="password"
        label="Password"
        validation="required|length:6|matches:/[^a-zA-Z]/"
        :validation-messages="{
          matches: 'Please include at least one symbol',
        }"
        placeholder="Your password"
        help="Choose an account password"
      />
      <FormKit
        type="password"
        name="password_confirm"
        label="Confirm password"
        placeholder="Confirm password"
        validation="required|confirm"
        help="Choose an account password"
      />

  </FormKit>

Submission successful!

</template>

This sounds like the same bug honestly. When you get unknown input type for known inputs it generally means you have multiple instances of Vue in your project so the plugin data cannot be located. Hard to address this without looking at your specific instance.

Hi @justin-schroeder  Its just a fresh nuxt 3 project with those installed,
Let me know what you need

@msonowal I just downloaded your zip file and did yarn install and then yarn dev and it worked fine. Although I got some error about tailwind config:
ℹ Merging Tailwind config from ~/tailwind.config.js                                                                                nuxt:tailwindcss 15:05:12
✘ [ERROR] Cannot read directory "..": operation not permitted

ℹ Vite warmed up in 474ms                                                                                15:05:13
ℹ Vite warmed up in 242ms                                                                                15:05:13

 ERROR  3:05:13 PM [vite] error while updating dependencies:                                                                                15:05:13
Error: Build failed with 1 error:
error: Cannot read directory "..": operation not permitted
    at failureErrorWithLog (/Users/justinschroeder/Downloads/formkit-poc/node_modules/esbuild/lib/main.js:1605:15)
    at /Users/justinschroeder/Downloads/formkit-poc/node_modules/esbuild/lib/main.js:1251:28
    at runOnEndCallbacks (/Users/justinschroeder/Downloads/formkit-poc/node_modules/esbuild/lib/main.js:1036:63)
    at buildResponseToResult (/Users/justinschroeder/Downloads/formkit-poc/node_modules/esbuild/lib/main.js:1249:7)
    at /Users/justinschroeder/Downloads/formkit-poc/node_modules/esbuild/lib/main.js:1358:14
    at /Users/justinschroeder/Downloads/formkit-poc/node_modules/esbuild/lib/main.js:668:9
    at handleIncomingPacket (/Users/justinschroeder/Downloads/formkit-poc/node_modules/esbuild/lib/main.js:765:9)
    at Socket.readFromStdout (/Users/justinschroeder/Downloads/formkit-poc/node_modules/esbuild/lib/main.js:635:7)
    at Socket.emit (node:events:520:28)
    at Socket.emit (node:domain:475:12)

It still ran, and FormKit was alive and well. Nuxt 3 is rather unstable right now and the @latest tag is updated on ever push to their main branch — if I was a guessing man, your local cached version of nuxt is out of date...or something like that. Try deleting your node modules and yarn.lock and re-install 🤷♂

Hi @justin-schroeder thank you for your patience,
however I have also tried cleaning everything
yarn cache clear
rm -rf node_modules/
rm -rf .output/
rm -rf .nuxt/
rm yarn.lock

Then Now freshly run which will install and download the latest release from your repos
yarn install
yarn dev

It works, but if you do try to run in production mode (compiled)
yarn build
yarn start

then also it works but in console as I said earlier those errors are still there
[Vue warn]: inject() can only be used inside setup() or functional components.
[Vue warn]: inject() can only be used inside setup() or functional components.
Error: Unknown input type "form" ("form_8")
    at error (file:///home/msonowal/sites/admin/.output/server/node_modules/@formkit/core/dist/index.mjs:236:11)
    at useInput (file:///home/msonowal/sites/admin/.output/server/node_modules/@formkit/vue/dist/index.mjs:878:9)
    at setup (file:///home/msonowal/sites/admin/.output/server/node_modules/@formkit/vue/dist/index.mjs:670:22)
    at callWithErrorHandling (file:///home/msonowal/sites/admin/.output/server/chunks/index.mjs:1957:22)
    at setupStatefulComponent (file:///home/msonowal/sites/admin/.output/server/chunks/index.mjs:7517:29)
    at setupComponent (file:///home/msonowal/sites/admin/.output/server/chunks/index.mjs:7498:11)
    at renderComponentVNode (file:///home/msonowal/sites/admin/.output/server/chunks/index.mjs:9853:17)
    at Object.ssrRenderComponent (file:///home/msonowal/sites/admin/.output/server/chunks/index.mjs:10284:12)
    at file:///home/msonowal/sites/smbtac-admin/.output/server/chunks/app/server.mjs:6610:38
    at renderComponentSubTree (file:///home/msonowal/sites/admin/.output/server/chunks/index.mjs:9924:13)
[Vue warn]: Component  is missing template or render function.

you can see the package.json contents which has @next tag and latest tag for formkit and nuxt and without formkit nuxt build is having no such errors
{
  "private": true,
  "scripts": {
    "dev": "nuxi dev",
    "build": "nuxi build",
    "start": "node .output/server/index.mjs",
    "type-check": "tsc --project tsconfig.json --pretty --noEmit",
    "lint:js": "eslint --ext \".js,.ts,.vue\" --ignore-path .gitignore .",
    "lint": "yarn lint:js",
    "format": "prettier --write .",
    "format-check": "prettier -- --list-different"
  },
  "devDependencies": {
    "@babel/eslint-parser": "^7.17.0",
    "@nuxtjs/eslint-config-typescript": "^8.0.0",
    "@nuxtjs/google-fonts": "^1.3.0",
    "@nuxtjs/tailwindcss": "^5.0.0-4",
    "eslint": "^8.10.0",
    "eslint-config-prettier": "^8.3.0",
    "eslint-plugin-prettier": "^4.0.0",
    "husky": "^7.0.4",
    "lint-staged": "^11.2.3",
    "npm-run-all": "^4.1.5",
    "nuxt3": "latest",
    "prettier": "2.5.1",
    "typescript": "^4.6.2"
  },
  "dependencies": {
    "@formkit/nuxt": "next",
    "@headlessui/vue": "^1.5.0",
    "@heroicons/vue": "^1.0.5",
    "@tailwindcss/forms": "^0.5.0"
  },
  "husky": {
    "hooks": {
      "pre-commit": "CI=1 npm-run-all --parallel type-check && lint-staged"
    }
  },
  "lint-staged": {
    "*.+(js|json|ts|vue)": [
      "yarn run format"
    ]
  }
}

Hmm, lots of variables at play here. What version of yarn are you using?

I just ran a build and start successfully with your code. And I'm almost 100% sure that error means there are multiple versions of vue at play here. It looks like your dependencies:
@formkit/nuxt
@headlessui/vue
@heroicons/vue

all require vue as well. Would you mind posting your yarn.lock?

@justin-schroeder
yarn-lock.zip

@justin-schroeder
any update or clue on this, I am waiting to hear form and do any help as required

@msonowal the yarn.lock looks fine. I'm unable to reproduce this issue and even with your codebase I dont reproduce the issue, so its definitely odd.
Would you mind trying this to isolate that its not another dependency:

Install a fresh copy of nuxt from: npx nuxi init nuxt3-app
cd nuxt3-app && yarn install
yarn add @formkit/vue
Add some formkit inputs to app.vue
yarn build

Does this produce any errors ☝?

Hi @justin-schroeder
yes I have done it as asked and shared you the video also see
and even also tried with  latest "@formkit/nuxt": "next", (beta 6) dev
same outcome.
https://user-images.githubusercontent.com/6334484/157489905-dbe17507-540d-411e-8a30-217b916a185b.mov

@justin-schroeder
thanks will be awaiting for your findings.

I was able to reproduce it — thats good news at least. I'll mark this as a bug now.

@msonowal this looks like it might be a bug in the nuxt nitro server. I'm going to start trying to create reproductions using non-formkit code to confirm, but it looks like the same bug as this: https://github.com/nuxt/framework/issues/2964
In the meantime if you just wrap your FormKit code in <ClientOnly> everything should work:
<ClientOnly>

</ClientOnly>

@msonowal ok, try upgrading @formkit/nuxt to the latest next tag (1.0.0-beta.6-4e39485) I think its resolved thanks to a helpful comment from Daniel Roe over on the nuxt repository.


How to connect to TLS socket?

Expected Behaviour
Be able to connect to a Docker API endpoint with TLS enabled.
Current Behaviour
I have a openfaas/faas-swarm service with the following environment variables set.
DOCKER_API_VERSION: 1.39
DOCKER_HOST: tcp://<IP_ADDRESS>:2376
DOCKER_TLS: 1

I'm expecting faas-swarm to connect to the Docker API endpoint with TLS enabled. But, the logs show it's not.
2019/06/26 17:35:46 Error with Docker server: Get http://<IP_ADDRESS>:2376/v1.39/version: net/http: HTTP/1.x transport connection broken: malformed HTTP response "\x15\x03\x01\x00\x02\x02".
* Are you trying to connect to a TLS-enabled daemon without TLS?

Possible Solution
Unknown.
Steps to Reproduce (for bugs)
Here's a snippet of my service definition for faas-swarm.
  faas-swarm:
    image: openfaas/faas-swarm:0.6.2
    environment:
      read_timeout: 5m
      write_timeout: 5m
      basic_auth: "true"
      secret_mount_path: /run/secrets
      DOCKER_API_VERSION: 1.39
      DOCKER_HOST: tcp://<IP_ADDRESS>:2376
      DOCKER_TLS: 1

Context
I want to be able to connect to a remote Docker swarm manager endpoint that has TLS enabled.
Your Environment

FaaS-CLI version ( Full output from: faas-cli version ): N/A

Docker version docker version (e.g. Docker 17.0.05 ): 18.09.6

Are you using Docker Swarm or Kubernetes (FaaS-netes)? Swarm

Operating System and version (e.g. Linux, Windows, MacOS): Linux

Link to your project or a code example to reproduce issue: N/A

Ah, turns out DOCKER_CERT_PATH has to be set.
https://github.com/moby/moby/blob/master/client/options.go#L26

Welcome back @kinghuang!
It's good to see you back.
Would you like to send a patch or do we not need any changes to support the usecase?
Alex

Thanks, it's good to be back! I haven't had a chance to use OpenFaaS since leaving the U of C. Great to see it's still going strong!
No patch needed for this use case. It turns out the Docker Go client doesn't completely match the Docker CLI for environment variables.

The workshop has come on a lot, and you may enjoy running through the Kubernetes track? You can get a free credit for DigitalOcean's new Kubernetes service too. https://github.com/openfaas/workshop/


More details on when to use different credential types or just some description would be good

[Enter feedback here]
More details on when to use different credential types or just some description would be good.  Currently, there is only the below text
The available credential types are:
AzureCLI
AzurePowerShell
Environment
ManagedIdentity
VisualStudio
VisualStudioCode

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

ID: 3b00e9d8-e683-dad2-1669-50c3b8577553
Version Independent ID: dc63331d-112e-c6ae-1d11-4e6cd407eca8
Content: Module setting for Bicep config - Azure Resource Manager
Content Source: articles/azure-resource-manager/bicep/bicep-config-modules.md
Service: azure-resource-manager
Sub-service: bicep
GitHub Login: @mumian
Microsoft Alias: jgao

@ChrisGibson1982 - Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can review and update accordingly.

@ChrisGibson1982 - thank you for your comments.  I have created a PR with some additional information.  The PR will be published tomorrow.  Please let me know if you need any additional information.
#please-close


How to build an application for window?

Hello, can you help me build the app for window?

The project needs to be updated. I'll try to make an update when I have some spare time.


Dev test: Wave Interference 1.0.0-dev.51

@ariel-phet and @KatieWoe, Wave Interference 1.0.0-dev.51 is ready for the first round of dev testing.  This sim has a publication deadline of Jan 31, 2019 but will be great to complete sooner because other sims share the same deadline.  The dev test should focus on searching for buggy or sim breaking behavior.  This is a PhET Brand sim and does not require PhET-iO testing.
Please document issues in https://github.com/phetsims/wave-interference/issues.  Assign issues to @samreid and link them back to this QA issue.
Per @ariel-phet's direction, I'm labeling top priority and assigning to Katie.

General Dev Test

What to Test

Click every single button.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.

Focus and Special Instructions
In addition to the general QA Dev testing, please test crashing conditions on iPad 2.  How long can you use the sim before it crashes?  Does any particular conditions lead to crashing?  Please test multiple trials and report results at https://github.com/phetsims/wave-interference/issues/128. Per @ariel-phet's direction, take a little bit of time exploring this, but not too long.

General Dev Test Platforms

[x] Latest macOS, Chrome and Safari
[x] Latest iOS, Safari
[x] Windows 10, all browsers
[x] Latest Chrome OS, Chrome

Link(s)

Simulation

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

Reopening to double check on https://github.com/phetsims/wave-interference/issues/128#issuecomment-447537364

https://github.com/phetsims/wave-interference/issues/128#issuecomment-447886331

Thanks @KatieWoe, closing.


Megascans plugin installed, causing plugin to infinitely work , without deleting assets. This happens when project using assets outside "/Game" folder.

Fixed with v1.3


gccgo: problems appending string to string subtype

What version of Go are you using (go version)?
Go, gccgo tip

$ go version
go version devel +06e4097f52 Tue Apr 9 12:08:48 2019 -0400 linux/amd64
$ gccgo --version
...
gccgo (GCC) 9.0.1 20190408 (experimental)

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
linux/amd64
go env Output
$ go env

What did you do?
Compile this program:
https://play.golang.org/p/1g5oPWL7tQ3
What did you expect to see?
Clean compile
What did you see instead?
Compiles and runs ok with the main Go compiler, but when compiled with gccgo I get:

GO111MODULE=off go build -compiler gccgo meas_a.go
./meas_a.go:19:14: error: argument 2 has incompatible type (cannot use type string as type Name)
   19 |  Register(f, n, d)
      |              ^
./meas_a.go:19:17: error: argument 3 has incompatible type (cannot use type string as type Name)
   19 |  Register(f, n, d)
      |                 ^

I am guessing there is a discrepancy in the way the string "+" operator is handled? Haven't yet looked into it in detail.

Related: for this fragment
package main
type N string
type M string
const B N = "B"
const C M = "C"
func main() {
	q := B + C
	println(q)
}

Main go compiler issues an error, gccgo does not.


KISS principle

This part of the code could be nicer.  So I did what you should've done in the first place and looked at what phpredis actually does.
Guess what? incr() and incrBy() do exactly the same thing, only the offset/value parameter is optional for incr().
https://github.com/phpredis/phpredis/blob/develop/redis.c#L1371
So nothing should've been changed in the first place and you did nothing but to waste time for both of us. For the last time - stop submitting pull requests if you don't know what you're doing.

sorry. No next time.

your actions inspire me.  be strict in submitting pull requests.
Thanks.


heatmaps

An example heatmap here
Reference implementations:

https://github.com/OpenGeoscience/geojs/blob/master/src/heatmapFeature.js
https://github.com/pa7/heatmap.js

Maybe the best way for these sorts of non-essential extensions to OpenGlobus would be some sort of (data visualization / UI ) plugin system. Some very useful plugins could then still be included in the core release. Other similar things: legends, arced graphs between points, etc.

In this case I use openlayers and leaflet terminology of controls. For example http://openglobus.org/examples/customControl/customControl.html. The basic idea is to create some "control" embeded with engine instance(scene or renderer) e.g. globe.planet.addControl(new MyConrol(myOptions)); I.e. control is an instance that has access to the whole engine.
However the heatmap is more than just ordinary control but (It's like an windmap) like an aggregate of control which provide visualization parameters and layer which represents visual condition. So it could be named as plugin and placed for example in ./plugin folder with control and layer code.

As far as I'm concerned, the heatmaps should be made with real 3d volumes and bloom effect around it.

hi.
I tried to draw heatmap using canvastile.
but heatmap point is drawn in every tile because of drawTile function.
I will draw heatmap points  only after all tiles are drawn
test-heatmap.zip
.


Add test-lint Makefile rule

For local linting.

Split from https://github.com/up9inc/mizu/pull/749

I understand you took it from the place you did just to make it available before.

nvm, I see now the message you wrote..


cargo bench --no-fail-fast -F=avx2 failed

     Running benches/hashset.rs (target/release/deps/hashset-e0bfa844d705dd83)
Gnuplot not found, using plotters backend
HashSet/u32/Default Hasher
                        time:   [16.380 ns 16.711 ns 17.067 ns]
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild
Benchmarking HashSet/u32/GxHash: Warming up for 3.0000 serror: bench failed, to rerun pass `--bench hashset`

Caused by:
  process didn't exit successfully: `/root/git/gxhash/target/release/deps/hashset-e0bfa844d705dd83 --bench` (signal: 4, SIGILL: illegal instruction)
     Running benches/ilp.rs (target/release/deps/ilp-18f7b97131886606)
Gnuplot not found, using plotters backend
baseline                time:   [134.34 µs 135.20 µs 136.28 µs]
Found 8 outliers among 100 measurements (8.00%)
  5 (5.00%) high mild
  3 (3.00%) high severe

unrolled                time:   [129.67 µs 132.18 µs 136.40 µs]
Found 12 outliers among 100 measurements (12.00%)
  2 (2.00%) high mild
  10 (10.00%) high severe

temp                    time:   [54.313 µs 55.820 µs 57.345 µs]
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild

laned                   time:   [46.377 µs 47.916 µs 49.764 µs]
Found 4 outliers among 100 measurements (4.00%)
  4 (4.00%) high mild

     Running benches/throughput/main.rs (target/release/deps/throughput-ce001a906bb8460b)
gxhash-avx2
error: bench failed, to rerun pass `--bench throughput`

Caused by:
  process didn't exit successfully: `/root/git/gxhash/target/release/deps/throughput-ce001a906bb8460b --bench` (signal: 4, SIGILL: illegal instruction)
     Running benches/throughput_criterion.rs (target/release/deps/throughput_criterion-c233dcb596180928)
Gnuplot not found, using plotters backend
Benchmarking all/gxhash-avx2/4: Warming up for 3.0000 serror: bench failed, to rerun pass `--bench throughput_criterion`

Caused by:
  process didn't exit successfully: `/root/git/gxhash/target/release/deps/throughput_criterion-c233dcb596180928 --bench` (signal: 4, SIGILL: illegal instruction)
error: 3 targets failed:
    `--bench hashset`
    `--bench throughput`
    `--bench throughput_criterion`
~/git/gxhash main                                                                                1m 30s root@u2 13:43:20
❯ cargo bench --no-fail-fast -F=avx2

❯ RUST_BACKTRACE=1 cargo test -Favx2 -- --nocapture
    Finished test [unoptimized + debuginfo] target(s) in 0.12s
     Running unittests src/lib.rs (target/debug/deps/gxhash-db5f87006ff816b7)

running 20 tests
error: test failed, to rerun pass `--lib`

Caused by:
  process didn't exit successfully: `/root/git/gxhash/target/debug/deps/gxhash-db5f87006ff816b7 --nocapture` (signal: 4, SIGILL: illegal instruction)

Hi @3tieto,
On which branch are you? Are you on latest main?
This bug is supposed to be fixed since 2.3.0. There is even a test for it cargo test hasher_handles_empty_inputs --features avx2
If I run cargo bench --no-fail-fast -F=avx2 I do not have any issue on my end

Is is running in a virtualized environment? (I see QEMU)
Also, are other tests passing? (If you comment out hasher_handles_empty_inputs)

i run in my vps https://my.contabo.com/
use rustc 1.76.0-nightly (21cce21d8 2023-12-11)
always crash

i create a tmp user for this error
just
❯ ssh<EMAIL_ADDRESS>password PXeF5q53u9vo
cd ~/gxhash
run test

Thx for the SSH.
The issue is that your machine does not support VAES:

See on my machine I have this flag:

The avx2 check is not sufficient. I thought avx2 implied vaes, but it seems it's not always the case. A check must be added in the code.
https://github.com/ogxd/gxhash/blob/8e109d97538ca95123ba73368fba18bb54d90005/src/gxhash/platform/mod.rs#L5
I'll patch this by adding the vaes check, even though the flag will remained named avx2 (which is misleading now)
This should be enough given that we will be no flag anymore once this work is finished https://github.com/ogxd/gxhash/issues/34

I look forward to the release of hybrid.

Addressed with v3.0.0 (hybrid)


How to use instance group

Thanks for adding instance_group!
I tried to use the new instance_group field. I got this error:
Google Provider:

The following settings shouldn't exist: instance_group

I previously had an older vagrant-google installed. I installed again and removed my box and added it again.
Are there any rules about how instance_group is to be used, like does it require the zone?
The value I'm using is a string, treating it similar to the network field.

@ericmand I haven't rolled a release yet, so I presume the issue is due to that. I'm out today but should be able to run through acceptance and release this weekend.

@ericmand Released and pushed to public, should be working now. Sorry it took a bit longer.

@ericmand Friendly bump. Is everything working for you?

Hi @Temikus I'm using this for deploying alluxio (https://github.com/Alluxio/alluxio/pull/3305). I've added it, and they deploy to instance groups fine. Great work!
Although, maybe it's worth mentioning that later in the Alluxio install process I have issues when I deploy to instance groups that I don't have otherwise. I think it's related to Ansible. Perhaps Ansible doesn't support instance groups.

@ericmand
Instance groups have not been added to libcloud yet, so they will be definitely misssing from Ansible as well.


Speed up mobile nav by keeping tab contents offscreen

@keybase/react-hackers

@chrisnojima I've made the tweaks we discussed. Please TAL?

👍


Revert "Update firefox note in  jpegxl.json" (#6887) and mention pitfall that the flag has no effect in the other builds than Nightly

Reverts Fyrd/caniuse#6887
The flag image.jxl.enabled is still useless even in Firefox 120 stable.

No task to link libjxl even in stable builds is not linked to https://bugzilla.mozilla.org/show_bug.cgi?id=1539075.
This means that there have been no changes related to JPEG XL for stable builds.

@Fyrd This is now ready for merged


The dependencies of pythran are very confusing.

When I compile scipy 1.7.0, an error occured that says "ModuleNotFoundError: No module named 'pythran'". Then I execute pip install pythran, something confusing happened.
17 packages been installed:
pythran decorator beniget ply networkx gast six numpy pandas scipy matplotlib python-dateutil pytz cycler kiwisolver pillow pyparsing
Even the scipy is in the list, and other scientific packages include numpy pandas matplotlib pillow.
First, is it a cyclic dependency for scipy?
Second, pythran is just a compiler to efficiently compile scientific programs, why these packages are installed as its dependencies.
Third, if it is possible to compile scipy without pythran.
environment:
Python: 3.9.2
Pip: 21.1.3
numpy: 1.20.1
Cython: 0.29.21
pybind11: 2.6.2

Yes we have just found this out too when dependency of pythran, networkx changed its dependencies too. We are working on it in #14335

Duplicate of #14335

Yes we have just found this out too when dependency of pythran, networkx changed its dependencies too. We are working on it in #14335 Let's keep track of this issue there.

Ok, well


how to enable fancyzones in  startup

How do you setup up fancyzones with a particular zone setting forever - that is so it starts as such on reboot in the startup apps.

If it doesn't, then we have a bug.

@stevedipaola any chance you can respond?

Hi there I'm having a similar Issue. In my current setup I have 4 separate virtual desktops all with different zones setup to my liking. I have also set the the "Run at Statup" option. However, whenever I open the laptop (both reboot and simple wake from hibernation) all my settings are messed up. Sometimes there are zones set but the incorrect ones and sometimes they just don't show at all. You can find my settings below.

Same error. I have two monitors with the same zones defined and after restart I have to open the app manually.
In my case, after "Sleep" works fine.

@N1x0
we are fixing a bug related to what you described, the fix should be available in 0.18
@xavik0f2
are you also using Virtual Desktops?

Hi @enricogior
I am not sure if this iis what you are asking me:

@xavik0f2
I'll assume you are not using Virtual Desktops.
In any case we will release 0.18 around the 20th of May, it will contain several fixes that may also solve the problem you are experiencing.

closing due to @stevedipaola not following up.
@xavik0f2 we think we fixed your issue in 0.18


Maximum call stack size exceeded on OSX

I'm using gulp-atom-shell, which is in turn using gulp-vinyl-zip, which depends on yazl. I'm running into the following problem (OSX 10.10.2):
RangeError: Maximum call stack size exceeded
    at new Buffer (buffer.js)
    at Function.Buffer.concat (buffer.js:199:16)
    at Entry.getLocalFileHeader (.../node_modules/.../yazl/index.js:302:17)
    at pumpEntries (.../node_modules/.../yazl/index.js:166:33)
    at Entry.doFileDataPump (.../node_modules/.../yazl/index.js:81:7)
    at pumpEntries (.../node_modules/.../yazl/index.js:168:11)
    at Entry.doFileDataPump (.../node_modules/.../yazl/index.js:81:7)
    at pumpEntries (.../node_modules/.../yazl/index.js:168:11)
    at Entry.doFileDataPump (.../node_modules/.../yazl/index.js:81:7)
    at pumpEntries (.../node_modules/.../yazl/index.js:168:11)

As best I can tell, it seems to be due to recursion in pumpEntries() (by calling entry.doFileDataPump(), which in turn (possibly?) calls pumpEntries(), etc).
Maybe it's caused by a very deeply nested file structure, but I'm not sure. I do know that pushing the call to entry.doFileDataPump() to the next frame seems to work, but it feels a little hacky, since i don't actually know why the recursion is occurring.
Thoughts?

Just some further clarification re:

…pushing the call to entry.doFileDataPump() to the next frame seems to work…

changed line 168 to the following:
process.nextTick(entry.doFileDataPump);

It looks like there's a bug where calling addBuffer several hundred times can cause this problem.

I was able to cause a problem that matched your stacktrace, and i implemented a fix for it in version 2.1.1. If this did not solve your problem, please reopen this issue.

:+1: Thanks for the quick fix!


TS throws after changing useSWR typings

Bug report
Description / Observed Behavior
Type '(org: string, repo: string) => Promise<any>' is not assignable to type '((args_0: string) => FetcherResponse<{ name: string; description: string; }>) | ((args_0: any, ...args_1: unknown[]) => FetcherResponse<{ name: string; description: string; }>) | ((args_0: Record<...>) => FetcherResponse<...>) | ((args_0: string | ... 2 more ... | Record<...>) => FetcherResponse<...>) | null'.ts(2769)

This error occurres when i use multiple arguments and generics for data
Repro Steps / Code Example
https://codesandbox.io/s/create-react-app-typescript-forked-yjc49?file=/src/index.tsx
Additional Context
Happening since 1.1.0-beta-3 , even on beta 7

Closed by #1640.


Bump identity.framework versio

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3879979809

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3879979809
Status: success


refactor: Refactoring Round 2

- What I did

Separated lib/ into the following:

sshnp/
sshnpd/
sshrvd/
common/
version.dart -> only file which sits in the root of lib/

Extracted out Arg parsing and at_client creation from SSHNP and SSHNPD classes

N.B. sshnp's is a little bit more complicated due to --config-files, which is a meta parameter
at_client creation has some weird parameters and could probably be cleaner, but it get's the job done, it can be further refined later

Extracted interface of SSHNP and SSHNPD (spec/impl files now)

FIX: fixed a bug where the atSigns would be left unset in sshnpd and sshrvd if the key file path was specified

Extracted sshrvd to a spec / impl / params class similar to sshnpd

- How I did it
- How to verify it
- Description for the changelog
refactor: Refactoring Round 2

It's always the simple things, like not assigning a variable, which break everything...

Nice work Xavier, LGTM. Thank you for the many individual commits, makes the reviewer's job so much easier

Of course, it also makes everyone's lives easier if we uncover a bug or something and need to look back through change history.

If possible, could we merge #258 first? This would be able to do a small test on SSHRVD which was refactored in this PR

Yes, let's do it

wait for the tests on trunk to pass before merging

@JeremyTubongbanua need your review to unblock the merge

@XavierChanth Did you pull latest trunk?

@XavierChanth Did you pull latest trunk?

Thanks for the reminder, will do it now

sshrvd is failing because the option was --keyFile and I changed it to --key-file for consistency... what do we want to do about it?

sshrvd is failing because the option was --keyFile and I changed it to --key-file for consistency... what do we want to do about it?

cc: @gkc @cconstab @JeremyTubongbanua
Ah, I could add "keyFile" as an alias for "key-file" to keep things backward compatible.

I saw the error, let me check that it's not my code given the command you've provided

Oops, I did half the fix in one file, and half the fix in the other file. Both rvd and npd will have the "keyFile" alias

I am very confused right now... it's saying that I'm trying to send a SecureSocket
sshrvd  | Unhandled exception:
sshrvd  | Invalid argument(s): Illegal argument in isolate message: object is unsendable - Library:'dart:io' Class: _SecureSocket@14069316 (see restrictions listed at `SendPort.send()` documentation for more information)
sshrvd  |  <- Instance of 'OutboundConnectionImpl' (from package:at_lookup/src/connection/outbound_connection_impl.dart)
sshrvd  |  <- Instance of 'AtLookupImpl' (from package:at_lookup/src/at_lookup_impl.dart)
sshrvd  |  <- Instance of 'RemoteSecondary' (from package:at_client/src/client/remote_secondary.dart)
sshrvd  |  <- Instance of 'AtClientImpl' (from package:at_client/src/client/at_client_impl.dart)
sshrvd  |  <- Instance of 'SSHRVDImpl' (from package:sshnoports/sshrvd/sshrvd_impl.dart)
sshrvd  |  <- Context num_variables: 1
sshrvd  |  <- Closure: ((int, int, String, String, bool)) => void from Function '_socketConnector@19502927':. (from dart:core)
sshrvd  |
sshrvd  | #0      Isolate._spawnFunction (dart:isolate-patch/isolate_patch.dart:392)
sshrvd  | #1      Isolate.spawn (dart:isolate-patch/isolate_patch.dart:372)
sshrvd  | #2      SSHRVDImpl._spawnSocketConnector (package:sshnoports/sshrvd/sshrvd_impl.dart:179)
sshrvd  | #3      SSHRVDImpl._notificationHandler (package:sshnoports/sshrvd/sshrvd_impl.dart:[133](https://github.com/atsign-foundation/sshnoports/actions/runs/5591810712/jobs/10223412252?pr=257#step:6:133))
sshrvd  | #4      _RootZone.runUnaryGuarded (dart:async/zone.dart:1594)
sshrvd  | #5      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:339)
sshrvd  | #6      _DelayedData.perform (dart:async/stream_impl.dart:515)
sshrvd  | #7      _PendingEvents.handleNext (dart:async/stream_impl.dart:620)
sshrvd  | #8      _PendingEvents.schedule.<anonymous closure> (dart:async/stream_impl.dart:591)
sshrvd  | #9      _microtaskLoop (dart:async/schedule_microtask.dart:40)
sshrvd  | #10     _startMicrotaskLoop (dart:async/schedule_microtask.dart:49)
sshrvd  | #11     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:123)
sshrvd  | #12     _Timer._runTimers (dart:isolate-patch/timer_impl.dart:405)
sshrvd  | #13     _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:429)
sshrvd  | #14     _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:189)

My mistake was that I made the function running on the separate isolate an instance function of the SSHNP class. It must either be a static function or top-level function. When it's an instance function there becomes some shared memory issues with the class that it's a part of.

Ready for a final review @gkc @JeremyTubongbanua


Update editor_setup.rst

Hi there!
Thanks for the PR. As someone who doesn't actually use VSCode, can you add some details about what this PR addresses?
Please see our contributing guide for the standard protocol.


Rerun all Chain notebooks

rerun all notebooks in the folder
Save their results
Commit to your fork
raise a PR from your fork to the main repo's staging branch

issues related to reserializing nested objects


Add an xtrace example

This implements #442: It adds an xtrace/xtruss-like example program.
Instead of being thread-based and spaghetti-coded, this one now uses async (just because I wanted to do something with async (well, and this means I do not have to mess with threads)). Plus, we now have some own code that uses the request parsing that @khuey added, which might not be the worst thing. (Designing an API without trying to use it can allow weird stuff to slip through.)
The sad news is that I am now out of toy projects to do "just because". Next, I'll have to actually fix some issues on x11rb...

There are other requests (i.e. PutImage) that are also extremely verbose so long term you may want a custom printer a la xtrace rather than relying on the Rust Debug implementation.


Added possibility to exclude files from input files inside Xcode

Closing, since #9 has been merged.


OwnedSymbolToken PartialEq Should Be Transitive

OwnedSymbolToken implementation of PartialEq does not appear to be transitive, this should be fixed (or at least tests added to verify see #153).

Originally posted by @almann in https://github.com/amzn/ion-rust/pull/163#issuecomment-817029438

This has been addressed with #181 and merged into main.


Nested items stored in css variables

Environment

## System:
 - OS: macOS 10.15.7
 - CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
 - Memory: 83.83 MB / 16.00 GB
 - Shell: 5.7.1 - /bin/zsh
## Binaries:
 - Node: 14.6.0 - ~/.nvm/versions/node/v14.6.0/bin/node
 - Yarn: 1.22.10 - ~/.nvm/versions/node/v14.6.0/bin/yarn
 - npm: 6.14.11 - ~/Workspace/vista/node_modules/.bin/npm
## npmPackages:
 - babel-plugin-styled-components: ^1.10.7 => 1.12.0
 - styled-components: ^5.1.1 => 5.2.1

Reproduction
https://codesandbox.io/s/misty-shadow-lr4tv?file=/src/App.js
Steps to reproduce
N/A
Expected Behavior
To be able to store nested items in a const using the css imported from styled-components, that I am able to return in a parent div. View option 1 for a clearer understanding of the bug.
Actual Behavior
No error or styling is returned.

I'm not really sure what the issue here is from the description and the sandbox.
The sandbox outputs an error based on something that's not related to styled-components:

 cannot appear as a descendant of .

That's basically a warning that semantic HTML cannot contain <pre> in <p> so <p><pre>... isn't allowed.
Now the issue with referring to ${Component} is usage issue. When you write a css tagged template you get a return value that isn't a string. Instead, it's an intermediate representation that may contain other function interpolations. This means that you can't convert it to a string. So, when I see this:
const Wrapper = styled.div`
  margin-bottom: 10px;

  ${({ isStyle1 }) => {
    return `${isStyle1 ? style1 : style2}`;
    /**
     * This works however:
     * if (isStyle1) {
     *  return style1
     * }
     * return style2
     */
  }}
`;

You're returning a string here and not the css tagged template literal output. That's the difference here basically. Using css allows you to use interpolations, but it's not the same as template literal strings, which can't do the same. If you interpolate css into other strings then you'll see some weird outputs.
On the note of usage questions, please refer with usage questions to Stack Overflow, as you're more likely to get quicker and helpful answers there ❤ Repository issues on GitHub are great for that, issues, things that need to be resolved in the library itself, or can't be resolved by anyone but the maintainers. While I (and many others on other repos) are happy to eventually answer usage questions, styled-components is a busy repo and mostly stable and slow moving towards new releases, so you're not likely to get fast responses here


Collecting Performance Counter From Azure Service Fabric Nodes Using  Azure Application Insights Fails

Need to get Performance counter details like CPU, Memory of Azure Service Fabric nodes using Azure Application insights, to understand what specifically limits the load on the Azure SF cluster.
As per link, modified "ConfigureServices" method in our Startup.cs class.
Note: the code shown below is taken from .Netcore2.1 Web API project deployed on Azure service fabric cluster
  using Microsoft.ApplicationInsights.Extensibility.PerfCounterCollector;
  public IServiceProvider ConfigureServices(IServiceCollection services)
    {

        var aiOptions = new Microsoft.ApplicationInsights.AspNetCore.Extensions.ApplicationInsightsServiceOptions();
        aiOptions.EnableAdaptiveSampling = false;
        aiOptions.EnableQuickPulseMetricStream = true;
        aiOptions.InstrumentationKey = "cxxxxxx4-2xx6-xxx1-axxf-071xxxxxxxb";
        services.AddApplicationInsightsTelemetry(aiOptions);

        // The following configures PerformanceCollectorModule.
        services.ConfigureTelemetryModule<PerformanceCollectorModule>((module) =>
        {
            // the application process name could be "dotnet" for ASP.NET Core self-hosted applications.
            module.Counters.Add(new PerformanceCounterCollectionRequest(
@"\Process([**_dotnet_**])\Page Faults/sec", "DotnetPageFaultsPerfSec")); //not sure "dotnet" is right value here?
        });

        //Add MVC
   services.AddMvcCore(options =>
        {
            options.Filters.Add(new LoggingFilter(_container.GetInstance<IMemoryCache>()));
        });
  }

Also edited .csproj to add the following line
  <PackageReference Include="Microsoft.ApplicationInsights.AspNetCore" Version="2.3.0" />

Still 'PerformanceCounter' table on Azure Application Insights remains empty. What needs done? What I'm missing?

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

ID: 7325660c-6ba7-5955-cadb-e98084e7a624
Version Independent ID: 9ea16a97-de92-6c54-93d5-d18593af5010
Content: Performance counters in Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/performance-counters.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @lgayhardt
Microsoft Alias: lagayhar

Hello @TestUser0811, Thank you for your feedback! We will review and update as appropriate.

@TestUser0811 My sincere apologies for delay in getting back to this thread. I am further requesting our document author @lgayhardt to further review the thread and share any pointers they might have regarding the issue you use and update the document as needed.

Closing out the docs issue as there is nothing actionable on the docs side and identical question was created against the sdk repo with engagement from the engineering team: https://github.com/microsoft/ApplicationInsights-dotnet/issues/2101

#please-close


Rework Print Output Function to Make Better Suited for SPDX Online Tools

@goneall, I noticed that SPDX Online Tools uses the default print functionality. IMO, it's a bit hard to grok as a user since it prints a long list of issues, rather than a more structured output.
Would you be interested if I created a print mode that was optimized for SPDX Online Tools? I could make some suggestions (or you could) in this issue and then I could try to implement it.
Note: Almost any suggestion will require some re-architecting of the codebase. But, TBH, that's on the docket anyway, so that's inevitable and doesn't need to be a constraint while we brainstorm. But once we decide on a new print mode (if we do), I'll also open a ticket about re-architecting and then I can kill two birds with one PR.

@jspeed-meyers - We could either create an online-tools optimized print mode as suggested, or we could change the online tools to use the JSON output and format a specific output in that tool.
I'm not sure which approach is the best - let me know what you think.

@goneall, both could work, certainly.
One limitation of the JSON output for the context of the online-tools is that the fields aren't in an order we choose. For instance, it would be nice if the NTIA-conformance overall check was at the top of the fields. Right now, that's not the case. Additionally, the fields are a little terse from the perspective of a non-expert user. Having a print mode would allow the tool to use more explanatory text to help non-expert users.
So my (weakly held) opinion: is to add an improved print mode.

The other advantage of an improved print mode is it would could also be used in the command line utility.

Ack. Will add an improved print mode to benefit both the online-tools and the command line utility.
Will open a separate issue on re-architecting.

I propose two new print modes:
First one:
Is this SBOM NTIA minimum element conformant? YES

Individual elements                   | Status
_____________________________________________
All component names provided?         | YES
All component versions provided?      | YES
All component identifiers provided?   | YES
All component suppliers provided?     | YES
SBOM author name provided             | YES
SBOM creation timestamp provided      | YES
Dependency relationships provided     | YES

Second one (to be accessed with a -v flag):
Is this SBOM NTIA minimum element conformant? YES

Individual elements                   | Status
_____________________________________________
All component names provided?         | Yes
All component versions provided?      | No
All component identifiers provided?   | No
All component suppliers provided?     | YES
SBOM author name provided             | YES
SBOM creation timestamp provided      | YES
Dependency relationships provided     | YES

Components missing a version: foo, bar, ...

Components missing an identifier: bar, baz, ...

The spdx online-tools version would use, by default, the second one. CLI users would get the first, unless they specified the -v flag.
wdyt, @goneall? Revisions welcome.

@jspeed-meyers - I like the proposed print modes


Optimize the creation and freeing of Rollers in RollerConveyor

This can be a bit laggy when the scale is large.

Closed by: a0427b1760e1ed8a99791feca6ed22f1088c2dbb

Closed by: f9d8ecc0cdfab01ade1e371107cf83abc072e35a


location option ignored

This might be related to #60 - I'm also using vitest.
The following test fails:
import { render, screen} from '@solidjs/testing-library'
import { Route, Router } from "@solidjs/router"

test.only('render location', async () => {
  render(Component, {location: '/foo'})
  expect(await screen.findByText('foo')).toBeTruthy()
})

function Component() {
  return (
    <Router>
      <Route path='/' component={() =>
root
}/>
      <Route path='/foo' component={() =>
foo
}/>
    </Router>
  )
}

If I change the findByText to 'root' it passes. Am I using the API correctly?

Actually, the router is already supplied by the test, so wrapping it in another router could break it.

Does that mean I can't test my top-level <App/> component (which contains a router)?
Also this still fails for me - any advice?
test('render location', async () => {
  render(() => <Route path='/foo' component={() =>
foo
}/>, {location: '/foo'})
  expect(await screen.findByText('foo')).toBeTruthy()
})

Thanks for your help

That is strange, because this is very close to what we do in our tests: https://github.com/solidjs/solid-testing-library/blob/main/src/tests/routes.tsx#L18

I've created a minimal repo with my test above and the test you linked. Both are failing for me
https://github.com/tslocke/solid-router-issue

Thanks for the repro. I'll test it later.

Any update on this? I'm unable to upgrade to the latest router because I can't get tests passing.

Sorry, I updated our testing library and everything worked as expected. Maybe the references to older versions broke something, so I'll release the updated version right away.

OK, I just published 0.8.9 and tested a bit more. Encapsulated routers do not yet work. I'm not sure if I will be able to support them, because we need an in-memory router for that and I don't know if I can inject it by default.

Thanks very much for looking at this. I updated all the deps in that sample repo (and pushed) and the tests still fail for me. Is that expected? I'm not sure what you mean by "encapsulated" routers.

Basically, testing your app.tsx.

OK. Not a problem for me at the moment. But right now I'm stuck on @solidjs/router 0.8 because the new router only supports things like useLocation when the component is inside a route, and as soon as I put things in a route the testing library doesn't render anything, e.g. this fails
test('render location', async () => {
  render(() => <Route path='/foo' component={() =>
foo
}/>, {location: '/foo'})
  expect(await screen.findByText('foo')).toBeTruthy()
})

Repro: https://github.com/tslocke/solid-router-issue

You can wrap useLocation in catchError in order to make the component work outside of the router. I'm sorry, but I cannot possibly support multiple versions of solid-router.

This is a module loading issue (we had a lot of those already). Maybe try making the router an external import.

Can you explain a bit more? I thought external imports were part of the build config and therefore would not affect testing?

vitest internally uses vite to transform the files loaded. However, it has multiple mechanisms to load files - via node modules resolution or its internal resolver. Whenever file transformation fails in vitest, it's usually due to a mix-up between those modes (or in case of solid because it gets resolved twice over the different modes - solid may only be loaded once).

@atk but the problem is only fired with solid-testing-library embedded router.
look at this:
import { createSignal, catchError } from "solid-js";
import { render, fireEvent } from "@solidjs/testing-library";
import { A, Route, Router, useParams } from "@solidjs/router";

describe("location option", () => {
  const Ids = () => (
    <>

Id: {useParams()?.id}

          navigate

    </>
  );
  const App = () => (
    <>
      <Route path="/ids/:id" component={Ids} />
      <Route path="/" component={() =>
Start
} />
    </>
  );

  it("can render the main route with embedded router", async () => {
    const { findByText } = render(() => <App />, { location: "/" });
    expect(await findByText("Start")).not.toBeFalsy();
  });

  it("can render the main route with manually loaded router", async () => {
    const { findByText } = render(() => <Router><App /></Router>);
    expect(await findByText("Start")).not.toBeFalsy();
  });

});

stderr | src/routerino.test.tsx > location option > can render the main route with embedded router
Error attempting to initialize @solidjs/router:
"Unknown file extension ".jsx" for<EMAIL_ADDRESS>
 ❯ src/routerino.test.tsx (2) 1051ms
   ❯ location option (2) 1050ms
     × can render the main route with embedded router 1024ms
     ✓ can render the main route with manually loaded router

In my env adding this conf to vitest seems to resolve the issue without break any other tests
server: {
  deps: {
    inline: ["solid-js", "@solidjs/testing-library", "@solidjs/router"],
  }
}

how this sounds to you? I don’t know if could be anyway to not have to add this config manually, what do u think about?
let me know if I can help in any way.

Did this get fixed? I've finally come back to upgrading the my router dep to 0.14 and I'm not having problems any more. It works even without the deps.inline config from @egtwp

It cannot be fixed in testing-library, but I am preparing a fix in vite-plugin-solid.

I'm no longer seeing that Unknown file extension ".jsx". No idea why. Can't complain : ) But I won't be surprised if it shows up again.
Maybe this issue should be closed, or perhaps renamed to track the issue you are working on in vite-plugin-solid? Up to you.

I can influence the testing config from the vite-plugin-solid, but not from @solidjs/testing-library.


Create an es-latest build

This build will be to support ES2016 environments.

I'd say that the criteria for this is that you should technically be able to load skatejs/es-latest in Safari TP and everything should work. The only blocker for this is dependencies. For example, if you use Component or withComponent, it tries to import ... from 'preact''; which will break it. What's the best way to solve this? cc @justinfagnani (I just saw your issue in Incremental DOM :))

Does @justinfagnani have some super-secret build of Chromium or did he build from source himself? I had to test with this: https://github.com/matthewp/do-modules-work and not in Canary yet.  Safari though!
To answer your question @treshugart, we would need a babel plugin that rewrites the module identifiers to their full paths. So that
import './with-module'; becomes import './with-module.js';
And
import 'preact'; becomes import '../node_modules/preact/preact.js';

There must be a babel plugin for that.

Good article to es2016 raw environments
https://www.contentful.com/blog/2017/04/04/es6-modules-support-lands-in-browsers-is-it-time-to-rethink-bundling/

I added a build in 3984e4f064ea3005d27e907d49d79f038c5226a9. Going to check it against my fritz and see if it works. Also going to give the babel plugin idea a shot.

I can confirm that fritz builds and runs fine with this change. So its just a matter of seeing if we can get this to run in Safari.
Conceptually this babel plugin is very similar: https://github.com/unpkg/babel-plugin-unpkg-rewrite/blob/master/modules/index.js . Except that we'll be resolving to the module's path, instead of using a CDN. So I'll use that a basis.

Got it working locally!  Not quite PR ready.

Exciting!

The one downside is that it needs to copy over preact to es-latest/preact because preact is not web-compatible and must go through the same babel plugin. I filed https://github.com/developit/preact/issues/670 to see if they are open to having a web-compat build too.  If so then we could point skatejs to that during out own build.

This is mostly ready on my end. Waiting to see if there's feedback on https://github.com/developit/preact/issues/670.  If so, I'll create a PR there first.  Otherwise will have to change the build in skatejs so that preact is copied into es-latest/preact.

I wonder if the tool you're building can have two modes:

transform path to point to existing directory
copy target to relative path and rewrite to that

Well, to be clear I'm not building a "tool", just a couple of babel plugins. I don't think we need to create any tool for copying the target.  I think the way the plugins works makes that case easy:
         [
           "modules-map",
            {
              "preact": "preact/dist/preact.esm"
            }
         ],
         "modules-web-compat"

This maps "preact" over to the ESM build that I submitted as a PR to preact.  If that doesn't get accepted for some reason we can just copy preact to the es-latest/ folder, and change that map to point to ./es-latest/preact/dist/preact.esm or whatever is needed. Just an extra npm script probably.

Cool, that looks awesome!

Discussed with @matthewp yesterday via hangout:
It looks like we'll change es to publish ES latest code instead of ES5 + ESM like NWB does by default.

Currently blocked waiting for https://github.com/developit/preact/pull/675 to be merged/released.
@treshugart, I can't remember, did we decide to not use nwb for this build (since we couldn't figure out how to get it not to set default presets/plugins)?

@matthewp I think we should try to use babel-preset-env with NWB. If that doesn't work, we can pass an option that we detect in the NWB config and turn on the ES build with different options and turn off the UMD build.

maybe I'm too biased (a @treshugart  would say :D ) but this config via nwb/babel plugs looks like shooting yourself in the foot.
With TS it's just one config https://github.com/Hotell/typescript-lib-starter/blob/master/tsconfig.json
and one liner in npm scripts https://github.com/Hotell/typescript-lib-starter/blob/master/package.json#L22
just saying [whistling] :) :D

I'm pretty sure TypeScript doesn't replace module specifiers with full paths like described here though.

We'll stick with the Babel stack for now.
This should be fairly trivial to implement by switching out Babel configs. The extra config / code we'd have to write will suck, but I'm not sure using TypeScript and Babel together is any better - two configs, two separate transpilers and push code between the two. At least with this method, everything is still contained within Babel and NWB.
Our prepublish should look something like:
nwb build && nwb build --es

In the config we do something like:
const { es } = yargs().argv;

module.exports = {
  esModules: es,
  babel: es ? esConfig : umdConfig
}

Will be fully fixed up in #1181.

Going to raise a separate issue for making stuff web compat.

Created #1189.


ADOPTERS: elaborate on ovirt/okd integration

Make it a bit clearer how oVirt and okd integrate with KubeVirt
/cc @sandrobonazzola @aburdenthehand
NONE

@dankenigsberg can you please sign off on your commit.
Otherwise the content lgtm. Thanks again

Unassigning because I don't have permissions to approve
@stu-gott PTAL

/approve


ERROR in Directive ɵb in<EMAIL_ADDRESS>has no selector, please add it!

ERROR in Directive ɵb in<EMAIL_ADDRESS>has no selector, please add it!
how to fix it?

what are your agm and angular versions?

"@agm/core": "^3.0.0-beta.0",  and angular "^8.0.3"

what are your agm and angular versions?

"@agm/core": "^3.0.0-beta.0", and angular "^8.0.3"

Hello,
you have to upgrade your Angular version to 10 (or at least 9) or downgrade your AGM version to 1.1.0.

Hello,
you have to upgrade your Angular version to 10 (or at least 9) or downgrade your AGM version to 1.1.0.

Hello, thank you I did that and it works grate


Consuming a C# component in a C++ app crashes at runtime

If you create a C# component and use it from C++, it will crash at runtime on 1709+.
FUSION_E_REF_DEF_MISMATCH 0x80131040  The located assembly's manifest definition does not match the assembly reference

Please fix this, workarounds are hard and not ideal

@Jaiganeshkumaran Are you saying the error message is incorrect?
Or is the "workaround" what it takes to have the C++ reference be compatible?

@orcmid Earlier you can create a C# managed Windows Runtime component and directly reference it from C++ and it will work but now it crashes due to some .NET Core version issue. Some workarounds like modifying the project file for C++ can work sometimes but it's not ideal.

@Jaiganeshkumaran. Could you please provide more info? Do you have a repro case you can share? Which versions of C++ and .NET? Etc.

@Jaiganeshkumaran I've written about this: https://github.com/asklar/WinRTComponent/blob/master/README.md
See also Raymond's post: https://devblogs.microsoft.com/oldnewthing/20200615-00/?p=103868/

@asklar I can't do this workaround every time I create a new project also I'm not that familiar with these project files and it's not a good experience. The SDK should ship proper templates in my opinion that doesn't require workarounds.

Agree that it's not a good experience. This isn't a Windows SDK issue though but a .net native issue, so I suggest you should file a bug against visual studio in the developer community site about it (In VS, go to Help, send feedback)

This issue is out of scope for this repository. Per @asklar's response, please file the bug against VS!


Define additional disks with defined sizes to VMs.

This will be done after creating new data model for Epiphany.
Is your feature request related to a problem? Please describe.
no
Describe the solution you'd like
Attach additional data_disks with specified mount points for each group of virtual machines.
Describe alternatives you've considered
None.
Additional context
None.

There is task in another repo for this in azure. We should probably have similar in aws module and consider it to be closed?

Similar task in aws module created:
https://github.com/epiphany-platform/m-aws-basic-infrastructure/issues/27

@erzetpe do you agree that we close this one?

Actually this task is about disk sizes, and ones mentioned in 2 previous comments are about vm sizes. My mistake. In fact more related issue is this one.

We should think about upgrade process before we start this task. It looks like we need spike to test upgrade process and probably later number of followup tasks.

@toszo @to-bar @erzetpe @plirglo @rafzei I think we need a common discussion and I mention you as involved in support and as you are might be familiar with details.
How I would proceed with this issue (still being a bit out of context):
Split this task into separate ones:

Add support for disks/volumes customization in modules part.
Introduce new parameters in epicli (don't know in module or in standard one) to support installation/applying configuration with these disks, in the same place doing filesystem part configuration: mount, resizing, data movement. Probably this one should be splitted for different supported services where this functionality should be applied: kafka, rabbitmq, etc.
Make upgrade process the same as previously, but if it's necessary to add new disks, perform epicli apply first.

Ok

Lets focus on backup-restore so we can easily apply such changes and restore data.

Created tasks for the first point, see links above. About the 2nd and the 3rd points, I think that there should be an epic created for backup/restore of each component (who will identify/provide the list?) and we should make this task dependent on the whole epic and  tasks https://github.com/epiphany-platform/m-azure-basic-infrastructure/issues/42 and https://github.com/epiphany-platform/m-aws-basic-infrastructure/issues/37. After that grooming label can be removed here, but imo we should keep it open to test in the end of implementation.
@mkyc could you help with that?

We can convert this task to epic, but I'd need some more detailed plan for this epic. AFAIU going with backup-restore way we would need to:

add support for separate disks in infrastructure creating VM's (my opinion is that this should be default and I do not need to configure anything to have separate disk used for "data")
ensure (for each component we have in classic epiphany) backup process works correctly
ensure (for each component as well) that we have either module which is able to restore from backup, or that classic epiphany is able to restore to VM's created with separate disks.

Am I correct?

add support for separate disks in infrastructure creating VM's (my opinion is that this should be default and I do not need to configure anything to have separate disk used for "data")

We discussed that there should be possibility to configure, as each component requires different space amount and probably different mount points.

ensure (for each component we have in classic epiphany) backup process works correctly

IMO that's correct.

ensure (for each component as well) that we have either module which is able to restore from backup, or that classic epiphany is able to restore to VM's created with separate disks.

I think that it's enough to make it possible via classic Epiphany.
@sk4zuzu @to-bar @toszo Could you write your thoughts on that? I remember that last time you participated in discussion.

For me it's clear, we make the backup/restore process work flawlessly for every component and it has to be battle tested with big volumes of data. We have to know how much time it may take and what to do if restore process fails for some reason after let's say 3 hours and there are 5 hours left. During epicli upgrade we create and mount data disks, restore all backups and reconfigure all applications to use the volumes. This deserves a separate epic :+1: :innocent: , it's a lot of work and testing \:heart_eyes:/.

Just reminded myself that we have #1545 epic for backup/restore topics.
Maybe we use this current task (139) only to implement option to add disks to VMs in classic epiphany?

@mkyc agreed. We make this task about adding optional disks to classic Epiphany clusters without moving any data. This mechanism should:

be generic enough to allow defining extra disks per component

This issues should be changed to make sure it includes the best support for the cloud native storage, also as we AWS changes we might want to split this up in AWS and Azure separately.

Closing this as new tasks were created that describe this what needs to be done in current Epiphany more accurate.


Border Sanity

Add check for border sanity, now tank doesn't exit anymore.

...and remember to write Unit Tests :)

Remember that to merge PR all checks on CI must be passing. Here is the latest failure:
https://github.com/wrazik/TankBotFight/pull/11/checks?check_run_id=3284670786
To fix the formatting, you should install clang-format (availible for most IDEs & as standalone application). .clang-tidy file is commited to the repo.
I will add to Readme.md proper information about code formatting in the near future!


iframe with float and other align css techniques

When i use the float or other techniques to align contents right or left inline to the iframe then it doesn't have the expected behaviour. For example I have a video inside the iframe and I need contents outside of the iframe to align to the right inline with the iframe. Whatever technique is used then either the contents will be below the iframe or the video will be shown in half

try some flexbox?  example:
https://codepen.io/sean1138/pen/poaERbN

No I didn't mean that. It shows the button and the paragraph inside the the iframe and video. I mean outside. Side by side.

updated the codepen


ja: Make docs/setup/production-environment/windows/user-guide-windows-containers.md follow v1.18 of the original text

This is a Feature Request

What would you like to be added
Update docs/setup/production-environment/windows/user-guide-windows-containers.md to follow v1.18 of the original (English) text.
Why is this needed
content/ja/docs/setup/production-environment/windows/user-guide-windows-containers.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/tree/dev-1.18-ja.2/content/ja/docs/setup/production-environment/windows/user-guide-windows-containers.md
Original
https://github.com/kubernetes/website/tree/release-1.18/content/en/docs/setup/production-environment/windows/user-guide-windows-containers.md
diff between translated and v1.18
https://gist.github.com/7987e514e817b2d512b416e516588e67
Note
Currently, we use dev-1.18-ja.2 branch. Please open a PR targeting the branch.
ref
https://kubernetes.io/ja/docs/contribute/localization/
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel

/triage accepted

/close


CI: Fix macOS pipeline failure

Changelog / Overview

Update ci.yml to update and upgrade brew if necessary before installing dependencies, and remove the unsupported dvisvgm.86_64-darwin package.

Motivation

CI macOS pipeline failure due to installation of a basictex package that no longer exists.
Exploring this avenue has also revealed that dvisvgm.86_64-darwin is no longer supported, as part of TeX Live 2021 release to support both ARM and Intel architectures. Instead, dvisvgm.universal-darwin will be used for MacTex, supporting macOS 10.14+. The legacy dvisvgm.x86_64-darwinlegacy binary folder will support macOS 10.6+, but it is only available with Unix install-tl.
Explanation for Changes

- name: Install system dependencies (MacOS)
        if: runner.os == 'macOS'
        run: |
          brew update && brew upgrade && brew cleanup
          brew install openssl readline ffmpeg pyenv pyenv-virtualenv
          brew install --cask basictex
          eval "$(/usr/libexec/path_helper -s)"
          sudo tlmgr update --self
          brew install pkg-config libffi pango glib
          sudo tlmgr install --reinstall standalone preview doublestroke relsize fundus-calligra wasysym physics dvisvgm rsfs wasy cm-super
          echo "/Library/TeX/texbin" >> $GITHUB_PATH
          echo "$HOME/.poetry/bin" >> $GITHUB_PATH

Testing Status

Further Comments

Checklist

[X] I have read the Contributing Guidelines
[X] I have written a descriptive PR title (see top of PR template for examples)
[X] I have written a changelog entry for the PR or deem it unnecessary
[X] My new functions/classes either have a docstring or are private
[X] My new functions/classes have tests added and (optional) examples in the docs
[X] My new documentation builds, looks correctly formatted, and adds no additional build warnings

Reviewer Checklist

[x] The PR title is descriptive enough
[x] The PR is labeled correctly
[x] The changelog entry is completed if necessary
[x] Newly added functions/classes either have a docstring or are private
[x] Newly added functions/classes have tests added and (optional) examples in the docs
[x] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings

This is super helpful. Thanks for the hard work and speedy fix.
I'm not familiar with brew; is the update / upgrade / cleanup necessary given we're running this in a CI env, or was it more a temporary fix?

This is super helpful. Thanks for the hard work and speedy fix.
I'm not familiar with brew; is the update / upgrade / cleanup necessary given we're running this in a CI env, or was it more a temporary fix?

It was more of a temporary fix to solve the basictex issue, but it led to finding out about the legacy package. I think we can keep it as redundancy in case a similar issue occurs in the future.

How much time does it add to the build? If we compare it to #1200 's build, it looks like it adds 4 minutes across the board. That is a lot of time for a "might be helpful" change.

How much time does it add to the build? If we compare it to #1200 's build, it looks like it adds 4 minutes across the board. That is a lot of time for a "might be helpful" change.

I do plan on working on caching as described in #824, for dependency-heavy packages such as ffmpeg to make the installation process faster.

That will be helpful. For the time being, though - and assuming it still builds - can you take out that update line? Otherwise it LGTM.

Looks like that line was necessary

How much time does it add to the build? If we compare it to #1200 's build, it looks like it adds 4 minutes across the board. That is a lot of time for a "might be helpful" change.

Sped up the time by a few minutes by only upgrading the brew packages we need.

Amazing, thank you!


垃圾

完全不能用，付款后不退款

Are you kidding me?
I have already said that the plugin and I are not directly related to the service and payments.
And you are not interested in solving the problem, saying from the very beginning that you only need a refund.
You could try out the plugin on the free version, you could help deal with YOUR local problem, but you don't need it.
I will ignore and close your next tickets and messages.
Also I will not provide you with any support as a developer at all.


Flynn product roadmap

Hello,
Flynn is a fantastic product and lots of people love it. But it's development has gone quiet recently and those of us who care can't help but think "Oh no, what's going on? Is this it?"
I saw this firefox product roadmap the other day and it seemed like perhaps flynn could do with one too. Just so we know what's going on. And what's going to happen.
This isn't just curiosity. Products like flynn need people to believe in it.
Thanks!

There are labels on the issues: "roadmap/upcoming" (next on table) "roadmap/current" (right after "roadmap/upcoming"). I think they are stalled as the development of new bigger features is a bit frozen.

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.


how to use --api-token?

I'm trying to automate command line access to the hassio CLI and I can't figure out how to use the --api-token argument.  I get an error without it saying that I need to specify a token, but if I create a long-life-access-token in the UI and use it, I get a "403 Forbidden" message in hassio-cli and this in the system log:
19-03-22 19:05:09 ERROR (MainThread) [hassio.api.security] Invalid token for access /homeassistant/info
I can't find that message anywhere in the Hass code, and I've looked through all the auth sections for what could be wrong.  It looks like the hassio component may require a username/password?  But then I don't know how to specify that to hassio-cli.
Any pointers?

Solved my own problem in several ways, documented here: https://community.home-assistant.io/t/using-long-lived-access-tokens-to-execute-hassio-commands-over-ssh/106878/2

@rccoleman That seems incorrect.
The token needed is an Hass.io API token, not a Home Assistant token.
You can use the remote API developers add-on to get an endpoint & token: https://github.com/home-assistant/hassio-addons-development

Thanks for the link.  Which part do you think is incorrect?  I'm using the last one on a regular basis now.

@rccoleman

While you can use an HA API token for HA related calls, this will not work on ALL the other calls.


Issue with running on custom dataset with --supervise_mode == unsupervised

Hi, thank you so much for the well organized and documented codebase.
I am trying to run your code on a custom dataset and setting the supervise_mode to unsupervised. However, the code seems to get stuck on Generating dataset preview.... Any idea why? When I run the same code with supervise_mode parameter set to malformed_normal, it works fine.
Thank you!
Edit: I found the reason. In datasets\bases.py file, the preview function keeps loading in training data until we have a certain amount of abnormal and normal images. For unsupervised setting, there will not be abnormal images, hence the loop keeps going. I changed that portion of the code and it works now. ;)

Hey.
You're right. Though the preview works in the unsupervised case, it doesn't return early and iterates through the full data loader once. Probably no one ever noticed since typically the datasets for AD are rather small. Still, I pushed a small update that fixes this (6cd3947).
Thanks!


fix: repeat compute in case one field aggregation multiple

fix #90

i review the aggs.Fields() only use in collectors, so i change the filter to collectors.


Automatically message recommendation upon match

What are your plans on this? I have a GIF with ~30% response rate that would be nice to send out programmatically 😆


ssh-tpm-keygen -A returns ECDSA unsupported

version 0.6.0
/etc/ssh # ssh-tpm-keygen --supported
ecdsa bit lengths: 256
rsa bit lengths: 2048

/etc/ssh # ssh-tpm-keygen -A
2024/10/05 17:36:23 INFO Generating new host key algorithm=ECDSA
2024/10/05 17:36:23 unsupported key type

/etc/ssh # ls *.tpm
ssh_tpm_host_rsa_key.tpm

as you see, only RSA key is generated.

Interesting!
Does creating a normal key work?

Interesting!
Does creating a normal key work?

yes, both creating and importing an ecdsa key work as normal.

I didn't change TPMAlgECDSA to TPMAlgECC when clearing up the code in go-tpm-keyfiles. Pushed a fix now that should generate it properly.


Some problems with events: click, mouseup, mousedown, and change for Chrome

Chrome 40.0.2214.111 m
click event
`event.originalEvent.fromElement should = null vs
event.screenX and event.screenY don't seem to match the values clientX/Y, pageX/Y, etc
mousedown and mouseup events
event.originalEvent.__proto__ should =  vs
event.view should =  vs undefined
the following are missing entirely:
event.originalEvent.detail: 0
event.originalEvent.keyIdentifier: "U+xx58"
event.originalEvent.keyLocation: 0
event.originalEvent.layerX: 0
event.originalEvent.layerY: 0
event.originalEvent.location: 0
event.originalEvent.pageX: 0
event.originalEvent.pageY: 0
event.originalEvent.repeat: false
event.originalEvent.view:
change event
I couldn't get the change event to fire at all. It doesn't fire if you type in an input field then click somewhere else:
syn.type($("#b"), "x")
syn.click($("#a"))
Even if one of those events is done manually, the change event doesn't happen. I also noticed that focus is not changed on click - it should be.

syn.type($("#b"), "x")

should be
syn.type($("#b")[0], "x")

That's true.. but its funny, it still types stuff, just doesn't send the change event. When I correct it, it has exactly the same behavior - no change event.

Are you able to repro?

I've not tried.  However, if it's breaking for you, I can show you how to create a test for it.


Remove SDX_HOME and add release tag for sdx-common

…

Codecov Report

Merging #31 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #31   +/-   ##
=======================================
  Coverage   83.01%   83.01%
=======================================
  Files           6        6
  Lines         265      265
=======================================
  Hits          220      220
  Misses         45       45

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 5abc535...5fbb1c8. Read the comment docs.

 (2) Remove erroneous use of SDX_HOME in makefiles

Superseded by #33


Show update button in Smartphone view (Small fix)

About

In Smartphone view, update button is hidden.

This p-r fix it.

Screenshot

master branch
this p-r

With Slack notification

master branch
this p-r

Thank you so much!
I merged this PR to #138 , and this PR will be closed when #138 will be merged to master.


change isDirty by removing from hasMany

Thanks a lot for this project!
I know that removing/adding from hasMany relationships seems to reflect on the .changed() result, but does the same happen with the isDirty property?
user.get('projects').removeObject(firstProject); // remove project1
user.changed() //=> {projects: true }
user.get('isDirty') //=> this returns false, but it should return true

I wonder if I'm doing some configuration wrong or if this isn't supposed to work.
Additional info:
using addObject on a hasMany seems to set isDirty to true.

That was a really great bug you found.
How the hasMany ever worked in the past ( is a real puzzler ) because what I found was it should not have worked at all. Bizzzzzarrreeeee.
But if you get v0.6.1 ( newest release ) you will be back in business.
Let me know if you find any others.

Updating indeed fixed it! Thanks!


Cadl Repository

Where are Cadl files going to be tracked? What will the structure be?

https://github.com/Azure/cadl-azure/issues/1691


[Feature/review] review and confirm with defaults

TBD
TODO delete old review and confirm
TODO skinning
TODO item defaults
TODO CFT summary compact

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@cc9c258). Click here to learn what that means.
The diff coverage is 13.67%.

@@            Coverage Diff             @@
##             develop     #899   +/-   ##
==========================================
  Coverage           ?   30.98%
==========================================
  Files              ?      360
  Lines              ?    18062
  Branches           ?     1629
==========================================
  Hits               ?     5596
  Misses             ?    11808
  Partials           ?      658

Impacted Files
Coverage Δ

...ercadopago/preferences/ReviewScreenPreference.java
27.1% <ø> (ø)

...rcadopago/uicontrollers/ViewControllerFactory.java
0% <ø> (ø)

...pago/paymentresult/components/FooterContainer.java
51.51% <ø> (ø)

...dopago/providers/ReviewAndConfirmProviderImpl.java
4.16% <ø> (ø)

.../paymentresult/components/TotalAmountRenderer.java
0% <ø> (ø)

...cadopago/paymentresult/PaymentResultPresenter.java
40% <ø> (ø)

...adopago/paymentresult/components/BodyRenderer.java
0% <ø> (ø)

.../components/actions/ChangePaymentMethodAction.java
50% <ø> (ø)

...rc/main/java/com/mercadopago/model/Reviewable.java
14.28% <ø> (ø)

...aymentresult/components/PaymentMethodRenderer.java
0% <ø> (ø)

... and 62 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 cc9c258...5982e3c. Read the comment docs.


DTS is not supported

dts audio is not surpported

Not interested in adding more codecs.

dts audio is very common, looking forward to adding，There is no need for a codec anymore.

https://github.com/media-kit/libmpv-darwin-build/blob/main/scripts/ffmpeg/meson.build#L138
provided but not supported

--enable-decoder=dca is required on macOS.
@alexmercerind maybe it's also required on other platforms?

Provided but not supported.

This sounds like a mistake on my end.
Other than this, I'm not really convinced to add more decoders/demuxers to the default selection. It is sufficient for most applications (or even multimedia applications). I took Audio/Video in Chrom(ium|e) as a base & added additional decoders/demuxers/protocols.
I will add the same for Windows & Android.
In even better case (probably enterprise grade projects), one should make their own finer selection of features & focus on optimization. On the other hand, if one is building a media-player, possibly it's better to use a max-ed out version of FFmpeg/mpv (probably with GPL code as well).

I tested with  media_kit_libs_ios_video v0.5.8 , dts still not working

I tested with media_kit_libs_ios_video v0.5.8 , dts still not working

I'll check it, I didn't take the time to check on iOS, only on macOS, maybe the decoder is different on this platform... (Apple...)

Have you tried other platforms?

I tested with media_kit_libs_ios_video v0.5.8 , dts still not working

DTS works fine on iOS.
I used a sample from here: https://filesamples.com/formats/dts
$ ffmpeg -i sample1.dts
ffmpeg version 6.0 Copyright (c) 2000-2023 the FFmpeg developers
Input #0, dts, from 'sample1.dts':
  Duration: 00:02:02.10, start: 0.000000, bitrate: 1411 kb/s
  Stream #0:0: Audio: dts (DTS), 44100 Hz, stereo, fltp, 1411 kb/s

sorry about that, i took wrong SHA256SUM, DTS works fine on iOS.

Apologies for the delay. Getting DTS resolved on Windows & Android.

A new update has been released. Please refer to the "Installation" section of the README.
Thanks!


Fix CustomGrid and update project docs

Fix the CustomGrid implementation to work properly.
Update README to add notes about the "Any CPU" configuration.
Code cleanup.

Tested CustomGrid use.  Looks good.  All ready for v1.0 release.


Rewrite Plugin

Use Quartz Listeners instead of subclassing Factories
Implementation independent of Quartz / Schwartz plugin so it can work with both

merged.


bug(menu): type MenuCloseReason not exported in public api

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
The type MenuCloseReason is not exposed through public api.
It's been around for quite some time now, so I guess it's pretty stable. Why not make it part of the public api, pretty please?
Reproduction
Steps to reproduce:

create a component that contains a <mat-menu>
listen to the (closed) event
use the event type (MenuClosedReason) in the event handler function

import { MenuCloseReason } from '@angular/material/menu';

@NgModule({
  template: `<mat-module (closed)="onMenuClose($event)"></mat-menu>`
})
export class TestComponent {
  onMenuClose(event: MenuCloseReason): void {
  }
}

Expected Behavior
public api of mat-menu should export MenuCloseReason.
Actual Behavior
public api of mat-menu does not export MenuCloseReason.
Environment

Angular: 13.x, 14.x, 15.x
CDK/Material: 13.x, 14.x, 15.x
Browser(s): n/a
Operating System (e.g. Windows, macOS, Ubuntu): Windows

Thank you very much, @crisbeto
Your help is much appreciated 😍
Btw. could you please tell me when and in what version will the fix be available?


Black screen when loading d88 disk

When I try to run games in d88 format with xnp21kai, I get a black screen (after RAM check).
Running games in hdi files goes without any issue.
Note that running the same d88 game with m88 emulator works without issue.
Are there steps to follow to debug ?
I have build NP2kai on Manjaro based on code with commit 82a79e077568bf9deff37e49dac48596cb1163e3
Many thanks

NP2kai is PC98 emulator.
M88 emulator? It's PC88 emulator, different architecture.
How about other PC88 emu, for example quasi88.
Regard.

Thanks for the clarification.
I got confused because I saw that d88 and d98 files are supported.


Session and Socket cannot use adapter 'ioredis'

I have a redis replica cluster using sentinel. I must use ioredis to connect my service. However, sails default redis adapter cannot used. I tried ioredis, but it failed.
How to connect setinels in Sails.js?

Hi @chenweiyj, I haven't tried doing this myself. but if you'd post more information about the error message / stack trace you're seeing, other folks might be able to help, or something might jump out.  You might also check out https://www.google.com/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#q=connect-redis ioredis and https://www.google.com/search?q=socket.io-redis+ioredis&oq=socket.io-redis+ioredis&aqs=chrome..69i57.909j0j7&sourceid=chrome&ie=UTF-8.
@luislobo any chance you've tried doing this before?

No, I never used it like that. How did you configure it?

@chenweiyj @luislobo update: @sgress454 is working on this for Sails v1 (at least on the sockets front)

For posterity:  to use a custom Redis client with sails-hook-sockets, you'll want to use the pubClient and subClient settings (see http://sailsjs.com/documentation/reference/configuration/sails-config-sockets). See https://github.com/balderdashy/sails-hook-sockets/pull/25

@mikermcneil  Can you please provide any example on how to use pubClient, subClient settings. Not sure but i am not following how i should use redis with sentinel support. I tried to search a lot. Any explanation or example would be great.


Support Llama 2 experimentally

Codecov Report

Merging #3 (d8ee2c5) into main (6a868e9) will increase coverage by 2.04%.
The diff coverage is 80.35%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
+ Coverage   87.15%   89.20%   +2.04%
==========================================
  Files          30       31       +1
  Lines        1090     1093       +3
==========================================
+ Hits          950      975      +25
+ Misses        140      118      -22

Files Changed
Coverage Δ

tests/test_dialog.py
99.13% <ø> (-0.11%)
:arrow_down:

pyaskit/code_generator.py
56.71% <33.33%> (ø)

pyaskit/core.py
56.25% <56.25%> (ø)

pyaskit/dialog.py
87.61% <87.50%> (+19.69%)
:arrow_up:

tests/test_llm_openai.py
96.42% <94.73%> (-3.58%)
:arrow_down:

pyaskit/__init__.py
100.00% <100.00%> (ø)

pyaskit/function.py
61.33% <100.00%> (+8.70%)
:arrow_up:

pyaskit/llm_openai.py
100.00% <100.00%> (ø)

tests/test_askit.py
95.83% <100.00%> (+0.59%)
:arrow_up:

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Better Cartesian product support for big data.

Adds two functions:
total_product(): Calculate total permutations of a Cartesian product.
Used to get an integer for iteration over with Cartesian
specific_product()
specific_product(): Extract single permutation of Cartesian product
iteration in an consistent speed regardless of list/set sizes.
This is better then the current intertools.product() generator/yield when dealing with
(big data research) massive amounts of sets/permutations.

@a904guy, I'm leaning toward closing this. If you provide a motivating example (why would I want the 40,000th lexicographic arrangement?) I could be persuaded to change my mind. Also, the terminology seems a bit confusing - "permutation" and "product" are both used in the description.

Simple, it's a fast quick way to pull the 34,921 arrangement without having to compile the entire 40k permutations into memory, so that you can resume processes on large data sets from the last nth piece of data, or that individual pieces can be given out to task managers or job launchers.
As far as the terminology, it can be called whatever, considering product is a different methodology, I'd agree it needs better wording but to say there is no practical reason for a low resource quick way to pluck from a large data set is a bit presumptuous.

Thanks for writing. Your example helps - this is for being able to split up tasks that operate on data sets that result from a cartesian product of inputs. It's not for Project Euler-like puzzles, which is what I had thought of when I read your code.


example/rcnn mistake--KeyError: 'width'

Environment info
Operating System:
Ubuntu16.04.2
Compiler:
gcc5.4.0
Package used (Python/R/Scala/Julia):
Python
MXNet version:
0.9.3
Or if installed from source:
yes
Error Message:
num_images 5011
voc_2007_trainval gt roidb loaded from data/cache/voc_2007_trainval_gt_roidb.pkl
append flipped images to roidb
Traceback (most recent call last):
  File "train_end2end.py", line 184, in <module>
    main()
  File "train_end2end.py", line 181, in main
    lr=args.lr, lr_step=args.lr_step)
  File "train_end2end.py", line 44, in train_net
    for image_set in image_sets]
  File "/home/cv/mx-rcnn/rcnn/utils/load_data.py", line 12, in load_gt_roidb
    roidb = imdb.append_flipped_images(roidb)
  File "/home/cv/mx-rcnn/rcnn/dataset/imdb.py", line 165, in append_flipped_images
    boxes[:, 0] = roi_rec['width'] - oldx2 - 1
KeyError: 'width'

Steps to reproduce
or if you are running standard examples, please provide the commands you have run that lead to the error.

changed to 0.9.1

What have you tried to solve it?
1.NO
2.
3.

@precedenceguo

Could be closed due to duplication in https://github.com/precedenceguo/mx-rcnn/issues/58

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


fixed vpc with terraform; added modify vpc tenancy

Fixed

VPC was missing ownerId attribute which caused terraform to calculate wrong ARN

Added

Added ModifyVpcTenancy API

TF Test

All 21 cases under TestAccAWSVpc_ are working now

Codecov Report

Merging #4116 (03919af) into master (6eeb4b4) will decrease coverage by 0.02%.
The diff coverage is 45.83%.

@@            Coverage Diff             @@
##           master    #4116      +/-   ##
==========================================
- Coverage   94.54%   94.52%   -0.03%
==========================================
  Files         473      473
  Lines       51832    51856      +24
==========================================
+ Hits        49007    49018      +11
- Misses       2825     2838      +13

Flag
Coverage Δ

servertests
55.44% <41.66%> (-0.01%)
:arrow_down:

unittests
94.45% <45.83%> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moto/ec2/responses/vpcs.py
96.68% <28.57%> (-3.32%)
:arrow_down:

moto/ec2/models.py
94.47% <50.00%> (-0.17%)
:arrow_down:

moto/ec2/exceptions.py
98.23% <66.66%> (-0.43%)
:arrow_down:

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 6eeb4b4...03919af. Read the comment docs.

Hi @bblommers , @whummer - PR is ready to merge. Can you please review it?

@whummer , I have updated the code.