Overheating, Gaming problem

LT and RT are now usable when I use LHLaurini Scrcpy, Tho I still have overheating issues can someone help me with it, and why do the LHLaurini able to use LT and RT tho the 3.0 version are not, I've try emulate a ps4 and the gamepad tester said it's a xbox 360, meant that LHlaurini version are emulated another controller so why can't do that on the 3.0 ver, and can you add another setting to change the encoder from phones to PC because it's getting overheating whenever I gaming
Mi 10
Android 13
Window 10

why do the LHLaurini able to use LT and RT tho the 3.0 version are not

The initial version from LHLaurini uses uinput, while scrcpy uses uhid (https://github.com/Genymobile/scrcpy/pull/2130#issuecomment-2276249226):

The main difference is that it injects events in UHID format to /dev/uhid, while this PR injects events in the UINPUT format to /dev/uinput.
The reason why it is problematic to use uinput is that it requires a call to ioctl(), which cannot be easily performed in Java, so it requires either a native dependency or to add native code, which I would like to avoid in scrcpy.

The gamepad in scrcpy is implemented by #5270:

There was a PR (https://github.com/Genymobile/scrcpy/pull/2130) implementing gamepad support using uinput on the device. However, this required to add a native dependency or to add native code (cf https://github.com/Genymobile/scrcpy/pull/2130#issuecomment-2276249226).
This PR uses HID instead, and provides support for AOA and UHID (like for keyboard and mouse).
It does not add any dependency, and it also works in OTG mode (via AOA), so even without USB debugging.

In addition, as I said in https://github.com/Genymobile/scrcpy/issues/5559#issuecomment-2506676819, the implementation in scrcpy by #5270 uses a generic controller, and it turns out that some games do not behave correctly with a generic controller.
To fix that, we would need to add a feature to simulate a well-known controller: https://github.com/Genymobile/scrcpy/issues/5362#issuecomment-2483497323

Overheating

Running a game and encoding the screen consumes resources, depending on the device hardware it might heat.
You can reduce the resolution/bit rate, or try another encoder/codec (depending on your device): https://github.com/Genymobile/scrcpy/blob/master/doc/video.md

but the LHLaurini version I can't change the encoder, how can I change it

The version you use from LHLaurini was a fork from an old scrcpy version, so it's an old version without all the new features and fixes.
I didn't check, but it migbt be a version where --video-encoder was named --encoder (before audio forwarding was implemented).

I was using scrcpy normally until today (still using scrcpy 3.1 or smth) the phone got overheating, first I thought It's because I tweak the hardware a little bit but then I try to turn off scrcpy it drop from 77 Celcius to 40 Celcius

mostly at 70 degrees sometime it'll jump to 77 and almost 80

I only plug in for like a few sec and It just rapidly jump from 40 to 78

This literally never happens before I've said before that it overheating but not like this, and my phone hardware is kinda good, it can handle genshin impact max setting at 60fps, I've reduce the resolution to 1366 too but it didn't work

idk what's wrong but when I tried to run scrcpy without turn of the screen it do not overheat, but idk why the fps are so low like it's only 10fps or smth

it does get stuck at 11 fps

See #3376.

turn out everything are ok now after I use a replace a new screen on my phone, I've take it off and that's maybe the reason


The configuration is not working as documented.

The configuration says:

If you don't want to show a category, simply remove the specified key from this object.

But removing the key, e.g. marketing from the object simply leads to the cookie banner displaying the default values.
To not show the category, e.g. marketing you have to set the value to false, e.g.
marketing: false
I can offer a PR, but this is just a one-liner.

Thanks for your PR!


Gradleize the plugin

Please describe, why we should Gradleize this plugin, then I will review this proposition.
Anyway, thank you for your engagement.


Error: Cannot find module 'gulp-symdest' when running build.desktop.mac

Ran:
npm run build.desktop.mac
Received error:
Error: Cannot find module 'gulp-symdest'
Simple fix; will PR

closed by: https://github.com/NathanWalker/angular2-seed-advanced/pull/30


Support for ARM64

Greetings,
From the README.md and BUILDING.md, it seems like there is no build support for ARM64 and it would be on a best effort basis.
I tried compiling the libkmsp11 and ran into a failed assertion.
I am wondering whether there's some limitation that would cause the lib to absolutely break on an arm64 system or whether this assertion was a safety net?
Do you have any estimate on how difficult it would be to add support for arm64?
Any suggestions on what I'd need to modify in order to support it would also be greatly appreciated.
Regards.

Hey! -- This assertion is indeed just a safety net.
With a few tweaks I was able to get the library to compile and all the tests to pass on an M1 Mac.  I think I can give it a try on aarch64 Linux sometime next week just to see what happens, but I don't think it should be too much of a lift to get it to work.

Turns out there was another minor patch needed to update one of our dependencies.  With the combination of this patch and the previous one I mentioned, I built and executed all tests successfully on Ubuntu 20.04 arm64.
Moving forward we'll try to get these patches integrated.  Internally we need to put them through CI on all platforms.  Informally, even though we don't officially support arm64, after the next round of dependency updates and the upcoming 1.1 release, I'd be surprised if arm64 breaks.  There's a lot more eyes on it now that developers actually have arm64 hardware with M1.

The last patched worked liked a charm!
I was struggling with the Abseil compilation errors but now everything ran smoothly!
There are two little things that I'd like to point out:

There is a missing character for the package checksum in the last patch you linked. The sha should start with a d. (ie dcf71b9cba8dc0ca9940c4b316a0c796be8fab42b070bb6b7cab62b48f0e66c4 instead of cf71b9cba8dc0ca9940c4b316a0c796be8fab42b070bb6b7cab62b48f0e66c4)
In BUILDING.md it is said that the project can be built using Bazel 4.0 or later. I tried with Bazel 5.0 and ran into some problems regarding. After building using Bazel version to 4.0, everything ran smoothly. This is my first contact with Bazel so it might very well be something I was doing wrong.

Awesome stuff, thanks a ton!


Add GraphiQL API console

Add a new button for "API"
When clicked display the GraphiQL console
Use the existing VARs for the URL and JWT auth
test a few queries work as expected

Instructions/Example here: https://github.com/graphql/graphiql/tree/main/examples/graphiql-webpack

closing this as implemented


feat: Allow GlobalTenantResource to cluster-scoped

Describe the feature
So far, globalTenantResources are namespaced. This feature would allow a globalTenantResource to be cluster-scoped.
What would the new user story look like?
As a cluster-admin, I can create one cluster-scoped resource for each namespace created in a tenant.
Use-case

For each teant namespace, we currently create a calico globalnetworkpolicy. This is something like a netpol, but cluster-scoped.

It seems to me out of scope, honestly.
According to your preferred CD strategy, you can provision a Tenant and along with it additional resources, such as your ClusterScopes resources.
The Capsule domain is related to Multi-Tenant and to the limitations of Kubernetes Namespaces, such as replicating resources across a set of these and orchestrating them.
ClusterScope resources should be provisioned by cluster administrators, and they have already full access to the cluster, thus are no limitations at all.
May I ask you for a further user story with a real use case to understand better your need?

Sure.
When a tenant owner creates a namespace in one of his/her tenants, our current multi-tenancy operator creates some gnp to allow/deny some traffic to/from this namespace.

We're lost in translation, I guess, so I'll try to explain what's wrong according to the Capsule jargon.

Tenants aren't created by Tenant Owners, but rather by the Cluster Administrator
Tenant Owners can just create Namespace resources

I recall @MaxFedotov is using Calico GlobalNetworkPolicy for their tenants, would be great to have his feedback on the discussion.

our current multi-tenancy operator creates some gnp to allow/deny some traffic to/from this namespace

@bsctl @ptx96 can add their usage so far about Capsule in onboarding and offboarding Tenants in a GitOps way.

@h4wkmoon In the Capsule design, the Tenant owner's realm is the group of namespaces being part of the tenant kept togheter by Capsule. The tenant creation as well as other global resources as for example nodes, Storage Classes, GlobalNetworkPolicies, are responsibility of the Cluster Admin. A Tenant owner cannot create global cluster-scoped resources.
As part of tenant onboarding procedure, the cluster admin (or any user with admin capabilities) creates the Tenant Resources as well as the other global resources related to the tenant: Storage, Ingress Classes, ClusterRoles, GlobalNetworkPolicies etc.

@bsctl , I understand that. And for most of the resources I agree, of course. Most of them can be seen as "infrastructure" or "plateform" stuff, and clearly, have nothing to do with capsule.
But what about globalnetworkpolicies that are created specifically for a namespace ?

I recall @MaxFedotov is using Calico GlobalNetworkPolicy for their tenants, would be great to have his feedback on the discussion.

Our level of granularity is the whole tenant. We don't create GlobalNetworkPolicy for each ns, because it seems a bit impractical - the whole tenant is owned by a separate team, so there is no need to segregate their workloads from each other

But what about globalnetworkpolicies that are created specifically for a namespace ?

It's s problem, you're right, and I'm not sure it's responsibility of Capsule to solve it.
I got strong opinions, weakly held, let's try to talk about this on one of the upcoming community meetings finding a consensus and writing down the result of it.

It's s problem, you're right, and I'm not sure it's responsibility of Capsule to solve it.

cannot agree more

@h4wkmoon could you please an example of globalNetworkPolicy you’re going to create for each namespace? Probably you can solve the problem in a different way. Being “global” you do not need to create it for each namespace. A namespaced networkPolicy seems to me more suitable. And Capsule already helps on that.

No need. It looks like my current use-case is not a good one. I'm going to stop wasting everyone's time.
Sorry, and thank you for the fish. :laughing:

I'm going to stop wasting everyone's time

You are not wasting time, we're open to continuing the discussion at one of the next community meetings. Feel free to raise the topic for the next agenda items on the #capsule channel.


Add agent_id to revocation message

The agent_id would be a useful piece of metadata to provide in revocation messages.

wow, i thought for sure that it was in there, but it's not.
However, there is a feature where the tenant can provide whatever metadata they want and it will get passed.  We use this to set the certificate serial number that should be revoked.  See https://github.com/keylime/keylime/blob/68580626b7c2887ff2cbb18ab2a37edba54cf6eb/keylime/tenant.py#L371 and then it gets used here: https://github.com/keylime/keylime/blob/68580626b7c2887ff2cbb18ab2a37edba54cf6eb/keylime/ca_util.py#L394

Interesting, thanks @nabilschear - my only thought here is, would this not require editing the keylime code to add the extra needed values? Or am I missing away a user can parse in the values which are then handed to revocation?

Trying to figure out how that works, so the tenant tells the verifier - include this field in the revocation event. I am confused as it looks like its going into the payload, which gets sent to the agents?

yes, the tenant can say, please include this additional info if this node gets revoked.  Then the agents or anyone else who listens to the revocation message can act on that additional info.  this is kinda the crux of how revocations can be useful.  because you need some information about what to do with the revocation, which is application specific.


[Arctic-188][Flink] Avoiding submitting too much empty snapshots

Issue: #188
Main change:
Flink1.12 and flink1.14 ArcticFileWriter judge whether the WriteResult is empty or not.

@YesOrNo828, could you please add some tests to verify whether not to submit empty snapshots? BTW, I was confused about how to produce this situation for empty results?

@SteNicholas  @zstraw
I added related tests and a configuration to control whether to emit the empty WriteResults.


Test pipeline fails to re-run after input file change pattern ["/*.tsx", "/*.ts"]

What version of Turborepo are you using?
1.5.6
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Windows
Describe the Bug
With this pipeline
    "test": {
      "cache": false,
      "outputs": [],
      "inputs": ["**/*.tsx", "**/*.ts"]
    },

Test is not re-run if *.ts file is changed.
Expected Behavior
Re-run test pipeline
To Reproduce
I am using vitest in the test, but should be any that reproduces this, just use that **/*.ts pattern.

The issue may be the empty outputs: [] folder. Can you try removing that line? (not saying that's the final fix, just curious).
Secondly, we are starting work soon on giving you better insight into how turbo interprets your config (e.g. expanding glob patterns), so hopefully that will help debug situations like this.

I suspect whatever issue you are experiencing here is not triggered by turbo. By specifying cache: false you are forcing turbo to run the test task every single time; no matter what the inputs array looks like.
Reproduction steps:
`npx create-turbo@latest --use-npm issue-2242
diff --git a/apps/web/package.json b/apps/web/package.json
index 37fe4e6..4aaa47e 100644
--- a/apps/web/package.json
+++ b/apps/web/package.json
@@ -6,7 +6,8 @@
     "dev": "next dev",
     "build": "next build",
     "start": "next start",
-    "lint": "next lint"
+    "lint": "next lint",
+    "test": "echo running"
   },
   "dependencies": {
     "next": "12.3.1",
diff --git a/apps/web/pages/index.tsx b/apps/web/pages/index.tsx
index 6ec0887..a00ca70 100644
--- a/apps/web/pages/index.tsx
+++ b/apps/web/pages/index.tsx
@@ -3,7 +3,7 @@ import { Button } from "ui";
 export default function Web() {
   return (

-
Web

+
Webby

       <Button />

   );
diff --git a/apps/web/pages/plain.ts b/apps/web/pages/plain.ts
new file mode 100644
index 0000000..3931d60
--- /dev/null
+++ b/apps/web/pages/plain.ts
@@ -0,0 +1,3 @@
+export default function Web() {
+  return "still here";
+}
diff --git a/package.json b/package.json
index e911460..2832b44 100644
--- a/package.json
+++ b/package.json
@@ -10,7 +10,8 @@
     "build": "turbo run build",
     "dev": "turbo run dev --parallel",
     "lint": "turbo run lint",
-    "format": "prettier --write \"**/*.{ts,tsx,md}\""
+    "format": "prettier --write \"**/*.{ts,tsx,md}\"",
+    "test": "turbo run test"
   },
   "devDependencies": {
     "eslint-config-custom": "*",
diff --git a/turbo.json b/turbo.json
index c2601a4..1781bcb 100644
--- a/turbo.json
+++ b/turbo.json
@@ -5,6 +5,11 @@
       "dependsOn": ["^build"],
       "outputs": ["dist/**", ".next/**"]
     },
+    "test": {
+      "cache": false,
+      "outputs": [],
+      "inputs": ["**/*.tsx", "**/*.ts"]
+    },
     "lint": {
       "outputs": []
     },

Note specifically that the hash is changing between builds as I modify some of the files that are inputs:
~/repos/issue-2242 (main *)$ npm run test

><EMAIL_ADDRESS>test
> turbo run test

• Packages in scope: docs, eslint-config-custom, tsconfig, ui, web
• Running test in 5 packages
• Remote caching disabled
web:test: cache bypass, force executing d99c474dcfb7b4c7
web:test:
web:test: ><EMAIL_ADDRESS>test
web:test: > echo running
web:test:
web:test: running

 Tasks:    1 successful, 1 total
Cached:    0 cached, 1 total
  Time:    233ms

~/repos/issue-2242 (main *)$ npm run test

><EMAIL_ADDRESS>test
> turbo run test

• Packages in scope: docs, eslint-config-custom, tsconfig, ui, web
• Running test in 5 packages
• Remote caching disabled
web:test: cache bypass, force executing 1370ee493b9c0d7c
web:test:
web:test: ><EMAIL_ADDRESS>test
web:test: > echo running
web:test:
web:test: running

 Tasks:    1 successful, 1 total
Cached:    0 cached, 1 total
  Time:    226ms

I'm closing this for now as I have no reason to suspect that there is an issue. If you can get me more information with complete reproduction steps I'll be happy to reopen!

@nathanhammond not using next and I removed outputs:[]
The config is
    "lint": {
      "inputs": ["**/*.tsx", "**/*.ts"]
    },
    "test": {
      "inputs": ["**/*.tsx", "**/*.ts"]
    },

On code changes I only see vite hmr log messages, no re-running of lint or test.

if I had something like vite running in dev mode it would, so turbo has not exited, it would re-run the pipeline. I guess this is a false assertion.

Yes, unfortunately, turbo itself does not implement a "watch" mode. There's some theoretical future in which that may be possible, but for long running tasks that don't exit, the best turbo can do is get out of your way.

The point of inputs and outputs is just to improve the caching, tighten it up so to speak and make a turbo run as fast as possible?

Yes. inputs is a way for turbo to determine whether the cached outputs are still valid. outputs is what is actually stored in the cache.


java.lang.IllegalStateException: Need to call #setCharacterLists first

I have  initialised CharacterList in onCreate() method . It is still giving the following error:
Caused by: java.lang.IllegalStateException: Need to call #setCharacterLists first.
at com.robinhood.ticker.TickerColumnManager.setText(TickerColumnManager.java:71)
at com.robinhood.ticker.TickerView.setText(TickerView.java:299)
at com.robinhood.ticker.TickerView.init(TickerView.java:180)

Are you using the android:text attribute in XML? if so, there is a known issue where it'll crash on initialization.


Training with Image Size 128px

Hi yu4u,
I want to train the network with 128 image size. (40k images)
However, the lost reduce stop at 24 (at 20 epochs) and then it doesn't decrease anymore. I've tried with adam, sgd, rmsprop. But the results are all the same.
Do you have any tips for training the network from scratch? Or is there any pretrained weights for 128 image size?
Thanks for the age-gender-estimation code!

Hi @hazxone,
I didn't have tried training using images with the size of 128x128.
How about changing the learning rate?
I'd like to try it if I have a time...

Hi @yu4u
Yup I've tried changing the learning rate from 1e-3, -2, -1. All end up with the same issue as above. Weirdly, if I trained using image size of 64, they end up nicely at loss 3.25.
(I'm using adience and UTKFace dataset because I need to detect age range from 1-100 since I noticed the pretrained weight doesnt detect age 20 and below.)
Since the network use pure cnn and not face embeddings (like facenet), is it better to use dataset with random different head pose or dataset with all face have been aligned properly?
Ground Truth : 26 F

Ground Truth : 29 F

I'm afraid that the current WideResNet model is not good for this resolution because it assumes the CIFAR-10 dataset (32x32 inputs).
Could you see whether the following modification of wide_resnet.py resolves the issue or not.
before
        pool = AveragePooling2D(pool_size=(8, 8), strides=(1, 1), padding="same")(relu)
        flatten = Flatten()(pool)
        predictions_g = Dense(units=2, kernel_initializer=self._weight_init, use_bias=self._use_bias,
                              kernel_regularizer=l2(self._weight_decay), activation="softmax",
                              name="pred_gender")(flatten)
        predictions_a = Dense(units=101, kernel_initializer=self._weight_init, use_bias=self._use_bias,
                              kernel_regularizer=l2(self._weight_decay), activation="softmax",
                              name="pred_age")(flatten)

after
        flatten = GlobalAveragePooling2D()(relu)
        predictions_g = Dense(units=2, kernel_initializer=self._weight_init, use_bias=self._use_bias,
                              kernel_regularizer=l2(self._weight_decay), activation="softmax",
                              name="pred_gender")(flatten)
        predictions_a = Dense(units=101, kernel_initializer=self._weight_init, use_bias=self._use_bias,
                              kernel_regularizer=l2(self._weight_decay), activation="softmax",
                              name="pred_age")(flatten)

and import GlobalAveragePooling2D as

from keras.layers import Input, Activation, add, Dense, Flatten, Dropout, GlobalAveragePooling2D

Hi,
Update from the modified GlobalAveragePooling2D using 128 image size.
Still no success. Even though the loss drops to 4.73 after 17 epochs, it will only predict 32 F on all faces.
How do I add more size layers to the wideresnet? Now it is 128 --> 64 --> 32 -->prediction.
I want to add 128 --> 64 --> 32 -->16 --> prediction
This is the log file if you are interested.
log (2).zip

@hazxone I'm tring to train on 128 image size for gender only with smaller model of depth 10 and width 4.  I guess this is a bad idea?

But I got another memory error.
Traceback (most recent call last):
  File "create_db.py", line 106, in <module>
    main()
  File "create_db.py", line 65, in main
    output1 = {"image": np.array(out_imgs), "gender": np.array(out_genders), \
numpy.core._exceptions.MemoryError: Unable to allocate array with shape (88451, 128, 128, 3) and data type uint8

Even though I did the following:
sudo su
root@deeplearning-test:/home/ubuntu/pixelation/age-gender-estimation# echo 1 > /proc/sys/vm/overcommit_memory

and
sudo suroot@deeplearning-test:/home/ubuntu/pixelation/age-gender-estimation/data# echo "2" > /proc/sys/vm/overcommit_memory

I reduced size to 64 but still got a memory error although imdb_db.mat was created in data folder.
python create_db.py --output data/imdb_db.mat --db imdb --img_size 64
100%|█████████████████████████████████| 460723/460723 [02:56<00:00, 2608.98it/s]
Traceback (most recent call last):
  File "create_db.py", line 108, in <module>
    main()
  File "create_db.py", line 68, in main
    scipy.io.savemat(output_path, output)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/scipy/io/matlab/mio.py", line 219, in savemat
    MW.put_variables(mdict)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/scipy/io/matlab/mio5.py", line 849, in put_variables
    self._matrix_writer.write_top(var, asbytes(name), is_global)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/scipy/io/matlab/mio5.py", line 590, in write_top
    self.write(arr)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/scipy/io/matlab/mio5.py", line 629, in write
    self.write_numeric(narr)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/scipy/io/matlab/mio5.py", line 655, in write_numeric
    self.write_element(arr)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/scipy/io/matlab/mio5.py", line 496, in write_element
    self.write_regular_element(arr, mdtype, byte_count)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/scipy/io/matlab/mio5.py", line 512, in write_regular_element
    self.write_bytes(arr)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/scipy/io/matlab/mio5.py", line 480, in write_bytes
    self.file_stream.write(arr.tostring(order='F'))
MemoryError


How to run in Macbook air m1

Where to download datasets？
and
How to run it on macbook air m1, do I need to rent a server?

@JayBuendia The extension ops must be compiled and run in CUDA mode.


Ability to export as CSM Template

Would be really cool if we could stand at either a Resource Group or a specific resource and export it as a CSM template.

That would require ARM functionality that I don't believe exists today. If that becomes available and there is an ARM API to create a template from a resource group, it would become easy to expose (and I assume Preview portal would expose it as well). /cc @clamanna

What if you created an export that just follows the same structure that exists in ARMExplorer?  You could click something like 'Recursive' and it would show JSON for the resource group as well as all the underlying services associated with the Resource Group.

I think that theoretically, it would be possible to follow the hierarchy and create an ARM template for all the resources in a Resource Group. But that's non-trivial and I would much prefer for this to happen directly in the ARM engine. I believe this is planned, though I will need to check.

This is a core ARM feature that will come later, and will not be Explorer specific. So closing here.


nil is printed on jdtls attach event

How to recreate

Open neovim
Open java source

Expected:
nvim-lspinstall used to display loading and the percentage
Actuat:
nil printed to the console
https://user-images.githubusercontent.com/18459807/139517785-e4723ffd-4b6e-481e-8219-9f53b53adcc2.mp4

Hey there shouldn't be any differences. I took a look at the nvim-lspinstall definition, and it's pretty much a mirror of nvim-lspconfig. I wonder if jdtls changed their implementation of that specific LSP request

Or actually, this could be due to neovim's LSP handler signature recently changing. Anyways this should probably be filed upstream in the lspconfig repo

Pretty sure it is. Closing in favor of https://github.com/neovim/nvim-lspconfig/pull/1348


ARM_CONTAINER not considered valid Environment->Type value for  AWS::CodeBuild::Project Resource

cfn-lint version 0.33.0
When using a CFN template that contains an AWS::CodeBuild::Project resource whose Environment->Type is set to ARM_CONTAINER cfn-lint exits nonzero with the following error:
➜  cfn-lint --info ../cloudformation/cfn-lint-issue-example.json
2020-06-12 08:09:02,170 - cfnlint.runner - INFO - Run scan of template ../cloudformation/cfn-lint-issue-example.json
E3030 You must specify a valid value for Type (ARM_CONTAINER).
Valid values are ["LINUX_CONTAINER", "WINDOWS_CONTAINER"]
../cloudformation/cfn-lint-issue-example.json:26:11

The full template is:
{
  "Resources": {
    "CodeBuildDockerImageBuilderProject": {
      "Type": "AWS::CodeBuild::Project",
      "Properties": {
        "Artifacts": {
          "Type": "NO_ARTIFACTS"
        },
        "Environment": {
          "ComputeType": "BUILD_GENERAL1_LARGE",
          "EnvironmentVariables": [
            {
              "Name": "AWS_REGION",
              "Value": {
                "Ref": "AWS::Region"
              }
            },
            {
              "Name": "AWS_ACCOUNT_ID",
              "Value": {
                "Ref": "AWS::AccountId"
              }
            }
          ],
          "Image": "aws/codebuild/amazonlinux2-aarch64-standard:1.0",
          "Type": "ARM_CONTAINER",
          "PrivilegedMode": true
        },
        "Name": "build-docker-images-project",
        "Source": {
          "Location": "some-bucket/docker/artifacts.zip",
          "Type": "S3"
        },
        "ServiceRole": "SomeServiceRole"
      }
    }
  }
}

The corresponding documentation lists this as a valid value.
Re-installing the latest version and running cfn-lint -u did not fix it.

As described here, this list will need to be updated:
https://github.com/aws-cloudformation/cfn-python-lint/blob/6cce9222c89056f1546f6fee068ce6dc9dfa394e/src/cfnlint/data/ExtendedSpecs/all/03_value_types/aws_codebuild.json#L46-L52


Sound-Supervisor failing to build

I've tried pushing the balena-sound project to balenaCloud several times using balena CLI but the Sound-Supervisor service keeps failing to build.
[Error]               Service: sound-supervisor
[Error]                 Error: The command '/bin/sh -c npm run build' returned a non-zero code: 2
[Info]              Built on arm06
[Error]             Not deploying release.

One-click deploy also not working. Same error. balena-sound is no longer deployable. Can someone please help with this?
npm
ERR! code
ELIFECYCLE
npm
ERR!
errno 2
npm ERR!
<EMAIL_ADDRESS>build: npm run clean && tsc --project tsconfig.json && cp -R src/ui build/ui && cp -R src/audio-block build/audio-block
npm
ERR!
Exit status 2
npm ERR!
npm
ERR!
Failed at the<EMAIL_ADDRESS>build script.
npm
ERR! This is probably not a problem with npm. There is likely additional logging output above.
The command '/bin/sh -c npm run build' returned a non-zero code: 2

Hmmm strange - I was able to build and push the latest version for my RPi4 without a problem.
What kind of device are you using?

I have a RPi3B+ using RPi3 (64bit OS).

Hmmm - The only difference that I can see is that mine says:
[Info]              Building on arm03
I just downloaded the latest Balena CLI and now it says:
[Info]              Building on arm04
Still builds and deploys, though.
[logging]           [>                                                        ]
[sound-supervisor]   ---> 284c124f41f8
[logging]           [>                                                        ]
[sound-supervisor]  Step 2/15 : WORKDIR /usr/src
[logging]           [>                                                        ]
[sound-supervisor]  Using cache
[logging]           [>                                                        ]
[sound-supervisor]   ---> 4327aeb9024b
[logging]           [>                                                        ]
[sound-supervisor]  Step 3/15 : COPY core/sound-supervisor/package*.json ./
[logging]           [>                                                        ]
[sound-supervisor]  Using cache
[logging]           [>                                                        ]
[sound-supervisor]   ---> 32abe0fd8d6a
[logging]           [>                                                        ]
[sound-supervisor]  Step 4/15 : RUN npm install
[logging]           [>                                                        ]
[sound-supervisor]  Using cache
[logging]           [>                                                        ]
[sound-supervisor]   ---> 0b2f5bbe413f
[logging]           [>                                                        ]
[sound-supervisor]  Step 5/15 : COPY core/sound-supervisor/src ./src
[logging]           [>                                                        ]
[sound-supervisor]  Using cache
[logging]           [>                                                        ]
[sound-supervisor]   ---> a813861981c4
[logging]           [>                                                        ]
[sound-supervisor]  Step 6/15 : COPY core/sound-supervisor/tsconfig.json .
[logging]           [>                                                        ]
[sound-supervisor]  Using cache
[logging]           [>                                                        ]
[sound-supervisor]   ---> 8bc8c07c3357
[logging]           [>                                                        ]
[sound-supervisor]  Step 7/15 : RUN npm run build
[logging]           [>                                                        ]
[sound-supervisor]  Using cache
[logging]           [>                                                        ]
[sound-supervisor]   ---> 5d737bc0738a
[logging]           [>                                                        ]
[sound-supervisor]  Step 8/15 : COPY VERSION .
[logging]           [>                                                        ]
[sound-supervisor]  Using cache
[logging]           [>                                                        ]
[sound-supervisor]   ---> 68981cc0a947
[logging]           [>                                                        ]
[sound-supervisor]  Step 9/15 : FROM balenalib/raspberrypi4-64-alpine-node:12

It looks like it might be an upstream issue with this image, which was updated a couple of days ago.  I'm not versed enough in the Balena build tools to know for sure, but when I forced my sound-supervisor Dockerfile to use that one, I got the same failures.
I would try opening an issue here: https://github.com/balena-io-library/base-images/issues

Thanks - I have logged in the base-images support.

I was able to fix by ignoring the types here in soundAPI.ts
     this.api.get('/audio', asyncHandler(async (_req, res) => res.json(await this.audioBlock.getInfo())))
this.api.get('/audio/volume', asyncHandler(async (_req, res) => res.json(await this.audioBlock.getVolume())))
this.api.post('/audio/volume', asyncHandler(async (req, res) => res.json(await this.audioBlock.setVolume(req.body.volume))))
this.api.get('/audio/sinks', asyncHandler(async (_req, res) => res.json(stringify(await this.audioBlock.getSinks()))))``
    this.api.post('/device/restart', asyncHandler(async (_req, res) => res.json(await restartDevice())))     this.api.post('/device/reboot', asyncHandler(async (_req, res) => res.json(await rebootDevice())))     this.api.post('/device/shutdown', asyncHandler(async (_req, res) => res.json(await shutdownDevice())))

Hey everyone, just created #503 to address this issue. Turns out it's an update to a dependency that broke the types. I'll merge the PR ASAP


Pulls latest graphql-java version, which contains oneOf, and add some tests

Please make sure you consider the following:

[n/a ] Add tests that use __typename in queries
[n/a ] Does this change work with all nadel transformations (rename, type rename, hydration, etc)? Add tests for this.
[n/a] Is it worth using hints for this change in order to be able to enable a percentage rollout?
[n/a ] Do we need to add integration tests for this change in the graphql gateway?
[n/a ] Do we need a pollinator check for this?

@gnawf

Do we need validation to ensure @oneOf also exists in the underlying schema etc?

Yep, that's a fair concern, but I don't think any extra validation would be needed.
The enforcement of the @oneOf spec will be done at the gateway level, so underlying services are guaranteed to receive a valid query, even if the implementation doesn't yet support @oneOf.
Also, @oneOf is never applied in queries, only in schema elements. So its usage can't result in queries that will be invalid in the underlying schema.

Do we need a test with variables?
That's not a bad idea. I'm gonna add a simple one for reference. I want to avoid writing tests to cover all the edge cases, though, as this should be well covered in the graphql-java tests

Is it permitted to specify multiple argument values from variables and only one of them is non-null?
That's not permitted. Exactly one field should be present, and it should have a non-null value.
The RFC has a bunch of examples that describe the expected behaviour in different scenarios: https://github.com/graphql/graphql-spec/pull/948/files (there's a table with the examples)


Wrong wiring diagram

there is a typo in showing how to wire it to the microbit, it should be:
NSS: 16
SCK: 13
MOSI: 15
MISO: 14

THANK YOU  DUDE I WAS SCARED I WAS SCAMMED


Adding led

@willrogers I have updated the alarm sensitivity for the default LED which just monitors the alarm level on a PV, but for user rules the functionality is slightly different. The user defines rules and colours to return in the relevant .json file, much like how this can be done for the "label" widget (mobileDemo.json line 28). Have a look on the widgetTestPage if you want to see both types side by side.

I'm just trying to figure out how the LED widget in CS-Studio does this.

Answer: it's complicated. Could you make this simpler though: one configured colour, then:

if alarm sensitive, change it to red or yellow or white depending on the alarm quality
alternatively set a rule to change the colour property

?
This raises the question of which takes precedence if both are set. The rule, I think, which is probably appropriate.

I think what you mean was basically what it did have before, but it didn't have the alarmSensitive property so I have added that.
Configured colour is green
User rule takes precedence over alarm value, even if alarmSensitive is set to true
If alarmSensitive and no user rule changes to yellow, red, or white (have taken the colours from your previous PR)

OK, let's see how we get on with it.
The other thing I don't understand is the scale prop. Does it scale the size of the LED? Wouldn't you use the widget size to control the size of the LED?

Sounds good, and yes it does, there's a simple reason and it's I didn't know a widget size prop existed. I will have a look into it.

Widgets all have a size, either absolute or relative. I'd expect an LED to fill its container, similar to what I think you were playing with for the SVGs.

Yes looks like the .opi has 'width' tied to each of the LED's, so can use that.


Feature: Support large sandbox content

If the content passed to the GET define API is too large, CodeSandbox will return an HTTP 414 error (URI Too Long).
fetching the POST define API with ?json=1 (the "Define without Render" option) and then using the resulting URL for the iframe would be a workable solution:
https://codesandbox.io/docs/importing#xhr-request
This would require running asynchronous code within the transformer function, which can be achieved by returning a promise:
https://github.com/swimlane/DocSPA/blob/9155c2e07fa3105f4e7858e62e9877cfc4105706/projects/swimlane/docspa-remark-preset/src/plugins/short-codes.ts#L106
More information about async remark plugins: https://github.com/gatsbyjs/gatsby/issues/16403

I may end up implementing this myself, if I need it quick enough.


Feature selection broken

Hi,
It looks like the feature selection option (turning bits of the demo on/of dynamically with GET params) is broken. My old URLs don't work anymore, and the widget lower on the page has disappeared.

Looks like a problem on my end. Some of my old URLs are broken, but some still work, and the widget still appear with the plain URL. Apologies.


Question: How to set up custom Literals?

I know this interface isn't really public at the moment, but I've a question on how to expand numba to support custom literal types.
I want to pass a tuple defining my implementation by value into generated_jit
The following works, but is a cludge:
@numba.generated_jit(nopython=True, nogil=True, cache=True)
def fn(a, b):
    print(a)
    print(b)
    print(b.instance_type.literal_value)

    def impl(a, b):
        return a.sum()

    return impl

a = 1
print(fn(np.ones((10, 10)), types.Literal((a, 2, 3))))
a = 4
print(fn(np.ones((10, 10)), types.Literal((a, 2, 3))))

from pprint import pprint
pprint(fn.signatures)

produces the following:
[(array(float64, 2d, C), typeref[Literal[tuple]((1, 2, 3))]),
 (array(float64, 2d, C), typeref[Literal[tuple]((4, 2, 3))])]

It's a cludge because  according to:
https://github.com/numba/numba/blob/e8ac4951affacd25c63ba2c18d62a3f12ed7e0ba/numba/types/abstract.py#L379-L385
we should use the following to create new literals:
https://github.com/numba/numba/blob/5217ce98ccc10b252885a859a4dcc405084600c2/numba/types/misc.py#L58-L59
Which seems to depend on writing something like the following:
class BobLiteral(types.Literal, types.containers.Tuple):
    def __init__(self, value):
        self._literal_init(value)
        print(self.literal_type)
        types.containers.Tuple.__init__(self,
                                        tuple(type(v) for v in value))

class bob(tuple):
    pass

types.Literal.ctor_map[bob] = BobLiteral

but this would need to registered somehow with:
https://github.com/numba/numba/blob/e8ac4951affacd25c63ba2c18d62a3f12ed7e0ba/numba/datamodel/models.py#L720-L725
Is this possible at the moment? I'm not sure how to go the rest of the way.

This sort of thing seems to work, but is likely dangerous and definitely uses internal APIs!
from numba import types, typeof
from numba import njit, generated_jit
from numba.datamodel.models import UniTupleModel
from numba.datamodel.registry import register_default

import numpy as np

class UniTupleLiteral(types.Literal, types.UniTuple):

    def __init__(self, value):
        self._literal_init(value)
        ty = typeof(value[0])
        for x in value[1:]:
            if typeof(x) != ty:
                raise TypeError("unituple not homogeneous")
        types.UniTuple.__init__(self, ty, len(value))
        self.name = "literal tuple"

types.Literal.ctor_map[tuple] = UniTupleLiteral
register_default(UniTupleLiteral)(UniTupleModel)

@generated_jit
def fn(x):
    print(x)
    ret = np.prod(x.literal_value)
    def impl(x):
        return ret
    return impl

@njit
def call_fn():
    return fn((10, 20, 30))

print(call_fn())

@stuartarchibald any reason for not supporting tuple literals in general? I know it's some work to potentially adapt every place tuples are used, but it would be very useful for several use cases.

@ehsantn I think with some effort it could be done, it may require some sort of work-out-the-type-of-tuple-literal call back, and the cost of walking through large tuples at compile time probably also needs to be considered, but it's no worse than e.g. hash for the same.

@stuartarchibald Cool!
In order to make it less dangerous, perhaps it might be better to restrict it to the following (i.e. force the literal, rather than let the internal typing mechanisms just convert any tuple to a literal)?
from numba.types.misc import literal

@njit
def call_fn():
    return fn(literal(10, 20, 30))


encrypt do not work if value includes \n

TEST1="not work \n"
TEST2='works well \n'

In older versions, TEST1 seems to work well.

hmm, at first test it appears correct:
TEST1="not work \n"
TEST2='works well \n'

before encryption:
$ dotenvx get TEST1
not work

$ dotenvx get TEST2
works well \n

with encryption
$ dotenvx encrypt
$ dotenvx get TEST1
not work

$ dotenvx get TEST2
works well \n

decrypted
$ dotenvx decrypt
$ dotenvx get TEST1
not work

$ dotenvx get TEST2
works well \n

when double quoted \n are expanded, when single quoted they are treated as literals.

when double quoted \n are expanded, when single quoted they are treated as literals.

So, This feature is on purpose? Maybe this differences should be put in docs.
My scenario is to encrypt certificate private key, but encrypt does nothing when I quote the private key with double quotes. In older versions, this seems to work well. So I guess it's a change in recent update.
I'm just making sure if this is a bug or a feature.

effort started here: https://github.com/dotenvx/dotenvx/pull/495
thanks for catching this @bingtsingw 🙏

releasing now as 1.31.1. should be good but if you still run into issues let me know.
there is one caveat here that when you encrypt and then decrypt it is going to expand those newlines. this has always been the way dotenv has worked though - for almost 10 years. so i'm going to keep that expansion here.


ubuntu VirtualBox VM should have flake8 installed in host OS for convenience

@mjstealey I am currently manually installing flake8 at the top level of the Ubuntu vm so that syntax checking doesn't require actually running the HydroShare container. This saves time.

@mjstealey Can this issue be closed?

completed and posted at distribution.hydroshare.org


Add durable-functions preview package to docs

Add reference to the preview version of durable-functions in the docs

Closing in favor of https://github.com/MicrosoftDocs/azure-docs-sdk-node/pull/1551


Actually disallow non-CloudFront traffic, take 2

This has been a messy afternoon. I thought I was good to go in #1189, but then realized I had accidentally started blocking the loaders from sending data to the API. This new approach should make sure loaders use HTTPS and the right domain for the load balancer, plus allow them through since they supply a valid API key.

Here we go again…


[WIP] Adapt Stoa to changes to Transaction, Input, Output

This applied a change in the PR (https://github.com/bpfkorea/boa-sdk-ts/pull/77) of SDK to Stoa.
Some code will be modified after SKD's PR changes.
I will create Branch Flash-layer and merge it there.
Relates to https://github.com/bpfkorea/stoa/issues/222

After the other PRs are merged, I will make a branch called flash-layer.
You can ignore any errors that occur when creating a Docker image.

After the other PRs are merged, I will make a branch called flash-layer.
You can ignore any errors that occur when creating a Docker image.

Rebase has been made to the timestamp of the block.

Rebase has been made to the timestamp of the block.

I checked normal operation as below in tests with Stoa and Agora.
Therefore, I will merge this.
Agora Log
node-0_1  | 2021-01-18 05:43:30,311 Info [agora.node.FullNode] - Accepted transaction: Type : Payment, Inputs (1): 0x3909...c173:0xf500...c643

Stoa Log
[Stoa] 2021-01-18T05:36:02.250Z info Agora endpoint: http://<IP_ADDRESS>:4002
[Stoa] 2021-01-18T05:36:02.251Z info sqlite3 database filename: /Users/mukeunkim/Work/WorkSpace_BPFK/agora-group/michael20/stoa/data/main.db
[Stoa] 2021-01-18T05:36:02.338Z info Connected to Agora, block height is 0
[Stoa] 2021-01-18T05:36:02.382Z info Recovered a block with block height of 0

I checked normal operation as below in tests with Stoa and Agora.
Therefore, I will merge this.
Agora Log
node-0_1  | 2021-01-18 05:43:30,311 Info [agora.node.FullNode] - Accepted transaction: Type : Payment, Inputs (1): 0x3909...c173:0xf500...c643

Stoa Log
[Stoa] 2021-01-18T05:36:02.250Z info Agora endpoint: http://<IP_ADDRESS>:4002
[Stoa] 2021-01-18T05:36:02.251Z info sqlite3 database filename: /Users/mukeunkim/Work/WorkSpace_BPFK/agora-group/michael20/stoa/data/main.db
[Stoa] 2021-01-18T05:36:02.338Z info Connected to Agora, block height is 0
[Stoa] 2021-01-18T05:36:02.382Z info Recovered a block with block height of 0


Cannot read property '0' of null. When Projection is used with ZoomableGroup (Demo included)

Hi,
It seems that using a projection with a <ZoomableGroup> throws Cannot read property '0' of null.
In this example https://codesandbox.io/s/usa-state-labels-map-1icyl you can see that removing the projection makes the map work.
Thank you very much for this super nice package.

Hi @juanpasolano,
That error seems to come from the missing center prop on the ZoomableGroup component. Try adding a valid center (i.e. within the bounds of the Albers USA projection). Something like [-96,38] should work quite well...
That should fix the problem.
Cheers

You are amazing! Thank you so very much!


To Add a Login through Google page to make it easy for users to access instead of old login through username and password

Description:
I propose enhancing the login functionality of the FitBody project by implementing Passport.js for authentication and integrating OAuth2 Google sign-in. This will improve security, usability, and convenience for users.
Proposed Changes:
Integrate Passport.js for authentication.
Implement OAuth2 Google sign-in.
Update the login page UI to accommodate the new authentication methods.
Provide documentation for developers on how to set up and configure the new authentication system.
Expected Benefits:
Enhanced security through standardized authentication mechanisms provided by Passport.js.
Improved user experience with the option for OAuth2 Google sign-in, reducing friction for new users.
Potential for increased user engagement and retention.
References:
Passport.js Documentation
OAuth2 Google Sign-In Documentation
Additional Information:
I have previously implemented similar authentication features on my personal website ((https://secrets-phi.vercel.app/)), showcasing my experience with Passport.js and OAuth2 Google sign-in. I'm excited to contribute these skills to FoodiesWeb and enhance its authentication system.I assure you that while enhancing the login page, I'll ensure the backend logic remains consistent and functional. I'm excited to contribute these skills to Fitbody and enhance its authentication system.

Please assign this issue to me

Looks good @Evadshell , proceed with it !!


Version 1.78 breaks compatibility with Ubuntu 18.04 targets due to GLIBC 2.28 requirement

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

VS Code Version: 1.78.1
OS Version: Host Windows 11
Target OS: Linux Ubuntu 18.04 arm

Steps to Reproduce:

Upgrade VS Code from 1.77.x to 1.78.x
Launch a Terminal on your target
Will now receive error when launching the terminal that GLIBC VERSION 2.28 is not available.  However, 2.27 is the final version of GLIBC6 that was supported on Ubuntu 18.04 LTS.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.78.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

@deepak1556 or @rzhao271 do you know anything abt this?

duplicate of https://github.com/microsoft/vscode/issues/181720

Merging to https://github.com/microsoft/vscode/issues/180798, please refer to the issue for context.


[WIP] First take on making hls async;

Initial test code:
const WalkManifest = require('./walk-manifest.js');

const url = 'https://bitdash-a.akamaihd.net/content/sintel/hls/playlist.m3u8'
let resources = []
WalkManifest(false, '.', url, false, 0,resources,function() {
  console.log(resources);
});

Sorry this PR was made against the wrong fork. It's still a work in progress. I'll close this here and continue on my fork. I'm not sure what the node version for this project is.

only noticed this was closed at the end 😑


Uploaded Status Cancelled only on iOS

I'm trying to upload a file on iOS but it's not working, I get statusCode 500 and Upload Status 5, any idea why is this happening only on iOS? In Android is working fine

I'm having the same problem. However, oddly it works for me in debug mode on a real device and in the simulator. When I make a release build, it stops working and I don't receive any result events, though it enqueues the tasks without any problems. The same web url, data, and file uploads fine in flutter.

@jaysignorello did you find any solution? Please help me out here. The app is live and this is unbearable. Thanks

@waqadArshad yes, I setup the listener to work in the foreground, which seems to work fine while the user is using the app, but certainly isn't ideal given it's not working as intended.


Misleading message when using invalid (?) include field

Problem
When using an invalid (?) include field in Cargo.toml cargo package will fail with "failed to verify package tarball".
Steps

Create a new repo, cargo init tmp && cd tmp
Add include = [ "./src/main.rs" ] to Cargo.toml.

Package (or publish) the create: cargo package

   Packaging tmp v0.1.0 (/private/var/folders/lq/rzq0zxld0bb5mcy4jl1txrf00000gn/T/tmp.1svg10Al/tmp)
   Verifying tmp v0.1.0 (/private/var/folders/lq/rzq0zxld0bb5mcy4jl1txrf00000gn/T/tmp.1svg10Al/tmp)
error: failed to verify package tarball

Caused by:
  failed to read `/private/var/folders/lq/rzq0zxld0bb5mcy4jl1txrf00000gn/T/tmp.1svg10Al/tmp/target/package/tmp-0.1.0/Cargo.toml`

Caused by:
  No such file or directory (os error 2)

Possible Solution(s)
In the actual code I dropped ./ from the include field and it worked, it doesn't seem to be the case with a new crate.
Notes
Output of cargo version: cargo 1.35.0-nightly (0e35bd8af 2019-03-13) (currently can't get a newer version due to clippy being broken on nightly).

I think this is the same as #4660.
@alexcrichton Do you know if there's a reason why include does not automatically add Cargo.toml?  I can sympathize with implicit magic making it hard to understand how something works, but in this case making everyone add it seems a little unnecessary.

@ehuss seems like a reasonable bug in Cargo, agreed we should just include it by default


Bugfix/67 when on imprint change language does not work

Future new pages absolutely need to set the namespace and permalink front matter vars.
Fixes #67

@milgner we should really add an auto-deploy GH action soon. I was testing the website and the bug was still there, cause this merged PR did not deploy a new version automatically.


translated day 3 to english

Hi @rheangocle, thank you so much for submitting the PR, but look like there's some issue with your commit, I don't see any translation file, but do see a lot of unrelated files. Could you pls check if you've committed the translated file or not? Also, pls remove all irrelevant files too.


Plugins/testers

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Closes #1121.
What are the significant changes in functionality due to this change request?
Adds the ability to import Testers for regression testing from within a plugin. As documented, these should be included in the plugin/src/Testers folder.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Job Test qsubs on 6b703a3 : invalidated by @PaulTalbot-INL
fix for qsub conda channels

Tests have passed.

PR Approved.
Check list passed


cleaner: chmod directories too.

They can also be installed with bad permissions so ensure that they are
owned correctly.
TODO: test with homebrew/homebrew-core#6975.
CC @tdsmith

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew tests with your changes locally?

All directories are going to be readonly? That sounds like it's going to break things to me.

All directories are going to be readonly? That sounds like it's going to break things to me.

@ilovezfs Will make them read-write for the owner. Group?

Passing on this as it won't fix the Ansible case and I can't figure out what will as I can't build it locally.


docs/prod install: Make it clear that Zulip should run on its own box

We should change docs/prod-install.md to strongly suggest running Zulip on its own hardware in production.

(We may also want a section for folks who decide to piggyback on to an existing box despite our recommendation, so they know where conflicts may happen with other apps/services, particularly nginx).

I think we may want to make this an explicit step in the docs.  We already warn against this, and people ignore it.

Yeah -- I don't actually see anything about this in docs/prod-install itself, though it may be in other places. We should definitely make this clear up front in that doc.
I'd turn the "installation requirements" section into a numbered step and add that warning there. The requirements doc it links to should also gain a clear mention of this.
Because a dedicated literal physical machine is going to feel pretty retro to a lot of readers in 2017, I'd also mention that of course it could be a machine in a cloud provider like AWS, or you can of course create a VM or container on a shared machine (and just link to generic docs on the web for LXC and KVM or something). You wouldn't do the latter if you're running at scale, but for someone who has a multi-purpose server machine lying around and just wants a small Zulip realm for their team or their friends, it can make a lot of sense.

Yeah, the main place it appears is: https://zulip.org/server.html
Which I suppose we probably want to just make a redirect to prod-install, anyway, so we can clean this up as part of that project.

@gnprice did you end up fixing this in your prod installer docs work?


Creating a better landing page

This repository is dedicated to creating a better landing page that enhances user experience and boosts conversion rates. It features a responsive design that looks great on all devices, SEO optimization for improved search engine visibility, and fast loading times to retain users. With user-friendly navigation and customizable components, this project allows for easy modification to align with your branding. Contributions are welcome, and the project is licensed under the MIT License.

Assign this to me and merge my PR

🚀 Follow me, 💬 Join the WhatsApp Group, and ⭐ Star the Repo to support our journey together!

please assign me this issue .i will solve it diligently and successfully. please consider!!!

hey i want to contribute in this , can you please assign it to me under hacktober fest.


Remove bits-service leftovers

Bits-service is deprecated since 2019: https://lists.cloudfoundry.org/g/cf-dev/message/8660 This commit removes all its leftovers.
Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

A short explanation of the proposed change:

An explanation of the use cases your change solves

Links to any other associated PRs

[ ] I have reviewed the contributing guide

[ ] I have viewed, signed, and submitted the Contributor License Agreement

[ ] I have made this pull request to the main branch

[ ] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

Thank you! I'm excited for this PR.


Startup branch

Started with the src files and a poetry file

It should be mentioned in the README to get the kaggle API key. Also, how to install Poetry if we want that to be our main method of dependency management. (But this can all be done later because we need this merged for further work)


signals: application creation signals

The idea behind the PR is that you can hook into the application creation process. In particular if you would like to e.g. modify the Flask application object. E.g. on Zenodo we need to disable strict slashes on the URL map, which has to happen fore you start adding anything to the URL map. Hence something like this:
from invenio_app.factory import create_app, create_api
from invenio_base.signals import app_created

def disable_strict_slashes(sender, app=None):
    app.url_map.strict_slashes = False

app_created.connect(disable_strict_slashes, sender=create_app)
app_created.connect(disable_strict_slashes, sender=create_api)

Probably needs https://github.com/inveniosoftware/invenio-accounts/pull/246 to pass tests.


Improve perf for multidimensional collections.

Allocating int[] indexes for each element is probably pretty bad. Benchmark and fix if needed.

Specialcased Copy 2D array of immutable.


Expand Cypress test for creating an access code

Overview
We have a test that checks whether a user is sent to the partner admin dashboard on load if they have partner admin authentication - see partner-admin.cy.tsx. However, we don't check whether they successfully can make an access code.
Action Items

[ ] Ensure you have cypress tests running.
[ ] Create branch and get the cypress tests running
[ ] Add the next steps to the test:
[ ] Create partner access code by clicking on the generate code button
[ ] Ensure the success message appears after the code has been generated
[ ] Run cypress tests to ensure your test passes
[ ] Create a PR and tag @kyleecodes or @eleanorreem

Resources/Instructions

Cypress docs

I'd like to work on this.

I was trying to create Badoo and Bumble users to run Cypress tests, but the selection option appears empty on the /admin/dashboard.

How can I get them? Do I need to update them from somewhere?

I was trying to create partner accounts to run Cypress tests. I have seeded the database and made the public user a super admin, but the "Select Partner" dropdown does not show any options. I checked the partner database, and there is data present.

@SureshPradhana thanks for reaching out!
We just updated our database dump, see the updated bloom-backend README and the linked Chayn Tech Wiki guide on Notion for the new directions. This may be due to that. This is a common error whenever the db is not properly seeded and connecting to the frontend.


ROCANA-11643: Network connection events report pid/process name on Linux

Adds pid and process name to network connections on Linux. The inodes for each connection are part of the tcp/udp/raw connection tables, and each entry in /proc has an fd directory which describes all open file descriptors.
So, this reads all inodes from every system process (as permissions allow), caches the open sockets, then joins that with the pid and name for each process.
This is similar to the approach taken by the netstat utility with the -p option.

Thanks @matthew-dailey. Travis-CI is being really slow but this passes on jenkins (Linux and Windows)


No tooltip on hover for the Add Full work item icon in the upper right

Tooltip should read 'Add Work Item' and should go straight up from the icon, with no text wrap.
http://www.patternfly.org/pattern-library/widgets/#tooltip

outdated - closing


Adding documentation

I added a wiki and documented the code.
Please merge asap :)

About merging wiki
https://gist.github.com/larrybotha/10650410


Add SortTabs component on the FE for sorting Search Queries

Closes #605

@aalises, Looks like you have a merge conflict

@tbergq @JoseDuSV Fixed the conflict!


Example script is from the wrong dist

I really have no idea what this means.

I really have no idea what this means.


Fix expansion

Symbols can now refer to macros. Additionally, macroexpand expands
each element of a list until reaching a non-nil expansion, which then
acts as the first element of the list. (I.e. macro? will receive the
first non-nil expansion rather than the unexpanded head of the list.)
There's probably not much chance of this getting merged, but this PR
solves a few remaining expansion corner cases. Performance seems mostly
unaffected on LuaJIT, around 2% slower on Node, and around 4% slower
on Lua.
Using Lua:
> (do (define-symbol def define-global)
      (define-symbol defun def)
      (macroexpand '((target js: 2) defun foo (x) (+ x 1))))
("%global-function" "foo" ("x") ("do" ("+" "x" 1)))

The above example demonstrates two new features:

(target js: 2) expands to nil, causing expansion to proceed using
(defun foo (x) (+ x 1))

defun expands to define-global, and expansion proceeds using
(define-global foo (x) (+ x 1))

To more clearly demonstrate the problem this solves, consider Lumen's
current behavior:
> (macroexpand '((target js: define lua: define-global) foo (x) (+ x 1)))
("define-global" "foo" ("x") ("+" "x" 1))

After merging this PR:
> (macroexpand '((target js: define lua: define-global) foo (x) (+ x 1)))
("%global-function" "foo" ("x") ("do" ("+" "x" 1)))

I think this PR is too complicated, but at least it's a reference point for how to solve those expansion problems. Maybe it can be simplified, though it's hard to avoid performance tradeoffs.

Too complicated.


Fix CodeAnalysis errors

Resolve two code analysis errors with MIEngine found in official builds
DebugEngineHost accidentally didn't have code analysis enabled during the build, which led to one of the bugs. This turns it on.
MICoreUnitTests.csproj was missing the 'Private' element on its reference to the stub definition of DebugEngineHost. Due to msbuild's super annoying reference system, this was dragging the stub definition of DebugEngineHost to the output directory, which was causing build races as that would sometimes overwrite the real version.

@caslan, @chuckries can one of you take a look?

Just one typo, otherwise LGTM


Update cats-helper to 3.10.0

About this PR
📦 Updates com.evolutiongaming:cats-helper from 3.6.0 to 3.10.0
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.evolutiongaming", artifactId = "cats-helper" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.evolutiongaming", artifactId = "cats-helper" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:semver-spec, commit-count:1

Superseded by #272.


Realsense collision box location in model.sdf for turtlebot3_waffle is not on the sensor

In ROS2 branch, the pose values <pose>0.069 -0.047 0.107 0 0 0</pose> for realsense camera collision box in turtlebot3_waffle/model.sdf seems to be wrong.  With these values, in Gazebo the collision box is shown with offset from the actual location.  Please look at the attached image.

I have amended. Please check again.

This issue will be closed since there were no actions for a while. You can reopen this issue to show this issue to the users whenever. Thanks.


[Factorio 2.0.8] Overload doesn't work

Factorio 2.0.8.

If you install several beacons of the same type, the assembler does not turn off, the modules from the beacons simply stop working on it, i.e. overload does not work as intended.
If you install 1 beacon of each type, then they all act on the assembler. Overload doesn't work at all.

for 1, the beacons should stop giving benefits to the machine. i changed the overload to a default off setting (so that the punishment for overlapping beacons is less severe)... maybe that was a stupid idea. for 2, i guess i didnt test that, oops. will look into it

Previously, if an icon was shown when an overload occurred, now, if the modules stopped working, there is no icon. You may not notice this by accident.


get_maching_blocks problem.

Trying to use the library for my application, it works fine when compiling on macOS, but under Windows using the msvc 2019 compiler, I get the following error:
error C2039: 'get_matching_blocks': is not a member of 'rapidfuzz::fuzz::detail'
No matter what I've tried, I cannot get the code to compile.
Any ideas?

Ah yes I did run into this with my python implementation as well and fixed it on my current development branch (I really need to add a CI to the C++ repository, so I do not only catch issues with the Python build).
The lastest master commithttps://github.com/maxbachmann/rapidfuzz-cpp/commit/a75c1da0804cd5a5a2b5e398d2d9403fd3627a7b should fix this issue

Cool.  Thanks for the prompt response!
I just removed the code from my project to get it building again but will undo my commit and add the library back in.  I'll let you know how I get on.

ok, it compiles and works now, however, it emits a load of warnings when compiling. (don't you love MSVC)
FavouritesSortProxyFilterModel.cpp
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz\details/common.hpp(190): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz/fuzz.hpp(52): note: see reference to function template instantiation 'rapidfuzz::common::PatternMatchVector<1>::PatternMatchVector<char>(std::basic_string_view<char,std::char_traits<char>>)' being compiled
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz/fuzz.hpp(51): note: while compiling class template member function 'rapidfuzz::fuzz::CachedRatio<std::basic_string_view<char,std::char_traits<char>>>::CachedRatio(const Sentence1 &)'
        with
        [
            Sentence1=std::basic_string_view<char,std::char_traits<char>>
        ]
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz\fuzz_impl.hpp(64): note: see reference to function template instantiation 'rapidfuzz::fuzz::CachedRatio<std::basic_string_view<char,std::char_traits<char>>>::CachedRatio(const Sentence1 &)' being compiled
        with
        [
            Sentence1=std::basic_string_view<char,std::char_traits<char>>
        ]
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz\fuzz_impl.hpp(64): note: see reference to class template instantiation 'rapidfuzz::fuzz::CachedRatio<std::basic_string_view<char,std::char_traits<char>>>' being compiled
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\components\RouteAnalyser\FavouritesSortProxyFilterModel.cpp(45): note: see reference to function template instantiation 'rapidfuzz::percent rapidfuzz::fuzz::partial_ratio<std::string,std::string,char,char>(const Sentence1 &,const Sentence2 &,rapidfuzz::percent)' being compiled
        with
        [
            Sentence1=std::string,
            Sentence2=std::string
        ]
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz\details/string_metrics/weighted_levenshtein_impl.hpp(138): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz\details/string_metrics/weighted_levenshtein_impl.hpp(293): note: see reference to function template instantiation 'size_t rapidfuzz::string_metric::detail::weighted_levenshtein_bitpal<char,char>(std::basic_string_view<char,std::char_traits<char>>,std::basic_string_view<char,std::char_traits<char>>)' being compiled
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz\details/string_metrics/weighted_levenshtein_impl.hpp(337): note: see reference to function template instantiation 'size_t rapidfuzz::string_metric::detail::weighted_levenshtein<char,char>(std::basic_string_view<char,std::char_traits<char>>,std::basic_string_view<char,std::char_traits<char>>,size_t)' being compiled
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz\string_metric.hpp(259): note: see reference to function template instantiation 'double rapidfuzz::string_metric::detail::normalized_weighted_levenshtein<char,char>(std::basic_string_view<char,std::char_traits<char>>,std::basic_string_view<char,std::char_traits<char>>,const double)' being compiled
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz\fuzz_impl.hpp(24): note: see reference to function template instantiation 'double rapidfuzz::string_metric::normalized_levenshtein<Sentence1,Sentence2>(const Sentence1 &,const Sentence2 &,rapidfuzz::LevenshteinWeightTable,double)' being compiled
        with
        [
            Sentence1=std::basic_string_view<char,std::char_traits<char>>,
            Sentence2=std::basic_string_view<char,std::char_traits<char>>
        ]
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz\fuzz_impl.hpp(38): note: see reference to function template instantiation 'rapidfuzz::percent rapidfuzz::fuzz::ratio<std::basic_string_view<char,std::char_traits<char>>,std::basic_string_view<char,std::char_traits<char>>>(const Sentence1 &,const Sentence2 &,const rapidfuzz::percent)' being compiled
        with
        [
            Sentence1=std::basic_string_view<char,std::char_traits<char>>,
            Sentence2=std::basic_string_view<char,std::char_traits<char>>
        ]
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz\fuzz_impl.hpp(80): note: see reference to function template instantiation 'double rapidfuzz::fuzz::CachedRatio<std::basic_string_view<char,std::char_traits<char>>>::ratio<std::basic_string_view<char,std::char_traits<char>>>(const Sentence2 &,rapidfuzz::percent) const' being compiled
        with
        [
            Sentence2=std::basic_string_view<char,std::char_traits<char>>
        ]
C:\Users\adriancarpenter\Documents\Development\pingnoo\src\libs\3rdparty\rapidfuzz-cpp\rapidfuzz\fuzz_impl.hpp(80): note: see reference to function template instantiation 'double rapidfuzz::fuzz::CachedRatio<std::basic_string_view<char,std::char_traits<char>>>::ratio<std::basic_string_view<char,std::char_traits<char>>>(const Sentence2 &,rapidfuzz::percent) const' being compiled
        with
        [
            Sentence2=std::basic_string_view<char,std::char_traits<char>>
        ]

Yes MSVC tends to be stricter with some of their warnings. I personally do not use Windows and gcc does not emit any warnings at least with -Wall. I often see similar warnings when failing Windows compilations of the Python implementation as well. It would probably help to add a CI build for platforms like Windows, but each time I look at ci builds for CMake in GitHub Actions appear to be a pain. E.g. https://github.com/cristianadam/HelloWorld/blob/master/.github/workflows/build_cmake.yml implements one for all platforms, but does not even take care of things like Catch2, Google Benchmark and Boost builds we need for the tests/benchmarks of RapidFuzz. I opened a issue on this topic, since I planned to do this since a very long time, but just can not get myself to actually implement it. Maybe someone else is faster than me.
Pull Requests to fix these sort of issues are welcome, but as long as they are not tested in the CI I will probably reintroduce similar warnings :disappointed_relieved:

I created a pull request, but master has moved on and the commits I made aren't compatible.
The issue is that you use loops of size_t and then pass the value into other templates that are expecting int.  Under MSVC this throws up a warning.
I fixed the specific issue for my usage case, but it probably needs fixing in more places.

I fixed a couple more of those warnings in MSVC: https://github.com/maxbachmann/rapidfuzz-cpp/commit/758276b8f8f0eff81dcbfc53edeb97fe2aa819f6

I guessed there would be more as I'm only using a couple of functions, I did try to find ones of the same type that I was having appear.

Yes I finally activated '/W4' on my Python Wrapper build, which brought up a few warnings in my code and a few in the code generated by Cython. Slowly fixing them as I encounter them.

Yes I finally activated '/W4' on my Python Wrapper build, which brought up a few warnings in my code and a few in the code generated by Cython. Slowly fixing them as I encounter them.

As an aside, is there any good documentation on making the most of this?  I'm using it very simply and probably not to its full effect.
I'm using it in my app to "fuzzy" search for "bookmarks".  These bookmarks are hierarchical, so as an example:
"Europe/London/Cloudflare"
I split the string by "/" and then I apply the partial match on each segment (3 in this example), if the partial match of any part is above a threshold, then the item is added to the filtered display.


generate auth URI for JWT access token properly

#2911

LGTM, pending tests

I will merge this for now and early next week we will figure out what exactly we should do about JWT auth URI (token audience).


Update Composer dependencies (2024-09-02-00-19)

Visual regression test passed!
View the visual regression test report


Using the simple "test" blocks result display

I'm trying to use the test adaptor as it'd be great to use Expecto more. I don't seem to be able to use the test to create simple cases.
open Expecto

[<Tests>]
let tests =
    testList "samples" [

        testCase "universe exists" <| fun _ ->
            let subject = true
            Expect.isTrue subject "I compute, therefore I am."

        test "A simple test" {
            let subject = "Hello World"
            Expect.equal subject "Hello World" "The strings should equal"
        }

        testCase "After a simple test" <| fun _ ->
            let subject = true
            Expect.isTrue subject "Not seen."
    ]

[<EntryPoint>]
let main args =
  runTestsWithArgs defaultConfig args tests

Leads to the the simple test and subsequent testCase not being displayed.
The tests are being run from the log though:
[2017-11-08 16:56:11 Informational] ------ Run test started ------
[2017-11-08 16:56:12 Informational] Running all tests (99<IP_ADDRESS>)
[2017-11-08 16:56:12 Informational] Creating test host for #####\Testy\Testy\bin\Debug\Testy.exe
[2017-11-08 16:56:12 Informational] Executing tests in assembly #####\Testy\Testy\bin\Debug\Testy.exe
[2017-11-08 16:56:12 Informational] Executing all tests in #####\Testy\Testy\bin\Debug\Testy.exe
[2017-11-08 16:56:12 Informational] All tests: 3
[2017-11-08 16:56:12 Informational] Number of tests included: 3
[2017-11-08 16:56:12 Informational] starting 'samples/universe exists'
[2017-11-08 16:56:12 Informational] starting 'samples/After a simple test'
[2017-11-08 16:56:12 Informational] starting 'samples/A simple test'
[2017-11-08 16:56:12 Informational] summary {results =
  [({name = "samples/universe exists";
     test = Sync <fun:tests@7>;
     state = Normal;
     focusOn = false;
     sequenced = InParallel;}, {result = Passed;
                                count = 1;
                                meanDuration = 3.0;
                                maxDuration = 3.0;
                                totalVariance = 0.0;});
   ({name = "samples/A simple test";
     test = Sync <fun:tests@12-1>;
     state = Normal;
     focusOn = false;
     sequenced = InParallel;}, {result = Passed;
                                count = 1;
                                meanDuration = 5.0;
                                maxDuration = 5.0;
                                totalVariance = 0.0;});
   ({name = "samples/After a simple test";
     test = Sync <fun:tests@16-34>;
     state = Normal;
     focusOn = false;
     sequenced = InParallel;}, {result = Passed;
                                count = 1;
                                meanDuration = 3.0;
                                maxDuration = 3.0;
                                totalVariance = 0.0;})];
 duration = 00:00:00.1374515;
 maxMemory = 0L;
 memoryLimit = 0L;
 timedOut = [];}
[2017-11-08 16:56:12 Error] Test adapter sent back a result for an unknown test case. Ignoring result for 'samples/After a simple test'.
[2017-11-08 16:56:12 Error] Test adapter sent back a result for an unknown test case. Ignoring result for 'samples/A simple test'.
[2017-11-08 16:56:12 Informational] ========== Run test finished: 1 run (0:00:01.021702) ==========

I tried building from master but had the same missing tests:

@cgravill
I trying to reproduce your problem and I have the same issue

But when assertion in single test is different than Expect.equal for example Expect.isTrue/Expect.isEmpty then everything works fine:

So it looks like there is a problem with simple test where assertion is Expecto.equal I will check in a free time if it's a problem with adapter or expecto itself.

Thanks for confirming. It'd be great to run tests with Visual Studio as well as it's easier to select a given test that way than via the command-line.

@cgravill sorry for a late response. I try to discover from where this bug comes from, but when I run those tests one more time, everything works fine..

The same with runTestsWithArgs:

The only thing which I did, since the last attempt was an update of Mono.Cecil library from: 0.10.0-beta5 to 0.10.0-beta7 could you confirm if it also helps you?

In fact this issue seems to be a duplicate of #28 . New version of adapter released to nuget, update of mono.cecil should fix'd the issue.


misc changes for 1.3.0

fix flyway migration Chinese garbled issue
limit hibernate query plan cache
fix import/export api
wording update for yaml/yml namespace support
update ldap sample configurations

Codecov Report

Merging #1963 into master will decrease coverage by 0.08%.
The diff coverage is 50%.

@@             Coverage Diff              @@
##             master    #1963      +/-   ##
============================================
- Coverage     50.24%   50.16%   -0.09%
+ Complexity     1981     1978       -3
============================================
  Files           400      400
  Lines         12342    12342
  Branches       1272     1272
============================================
- Hits           6201     6191      -10
- Misses         5684     5692       +8
- Partials        457      459       +2

Impacted Files
Coverage Δ
Complexity Δ

...llo/portal/controller/ConfigsExportController.java
8.16% <0%> (ø)
1 <0> (ø)
:arrow_down:

...apollo/portal/entity/model/NamespaceTextModel.java
86.95% <100%> (ø)
14 <1> (ø)
:arrow_down:

...work/apollo/biz/message/DatabaseMessageSender.java
57.14% <0%> (-10.21%)
6% <0%> (-2%)

...mework/apollo/portal/component/PortalSettings.java
65.07% <0%> (-4.77%)
5% <0%> (ø)

.../apollo/internals/RemoteConfigLongPollService.java
79.14% <0%> (-1.23%)
27% <0%> (-1%)

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 b45405b...ccbdb86. Read the comment docs.


add wx-work bot template

I'm submitting a...
Related to #85
[ ] Bug Fix
[x] Feature
[ ] Other (Refactoring, Added tests, Documentation, ...)

Checklist

[ ] Commit Messages follow the Conventional Commits pattern

A feature commit message is prefixed "feat:"
A bugfix commit message is prefixed "fix:"

[ ] Tests for the changes have been added

Description
please describe the changes that you are making
for features, please describe how to use the new feature
WeChat Work provides a room bot, using a webhook, it can push notification to WeChat work room. This commit helps us to push GitHub notifications to WeChat work room.
please include a reference to an existing issue, if applicable
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Terrific!
We start supporting WXWorks from today, thanks jiarui!


fixes: multiple fixes

Proposed changes
Based on recent mistakes I'm forcing  if the description is not found for an arg we now return an error
This also adds initial testing
Checklist

[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation (if appropriate)
[x] I have run make fmt and formatted my code

I can see it catching a lot of issues.

I wrote this things I was using it wrong so I can confirm 😛


TC-GR-2.2 Groups Cluster - groups are not removed from GroupTable even after RemoveGroup command gets Success response

Problem - Even after TH removes the group using RemoveGroup command, that particular group exists in GroupTable
Actual Behavior - TH removes the group using RemoveGroup command and the response is Success, then TH reads GroupTable attribute in which the removed group exists.
Expected Behavior - After TH removes the group using RemoveGroup command then that group should not be present in the GroupTable attribute entries.
App used - allclusters app
Platform - Chip-tool - RPI-4, 8GB RAM
DUT         -  RPI-4, 8GB RAM
Network - Ble-wifi
Commit - 425f1fb916ca154deeb75cd82cd5196778aba152
Steps to reproduce -
removegroup - ./chip-tool groups remove-group 0x0001 1 0
viewgroup      - ./chip-tool groups view-group 0x0001 1 0
grouptable     - ./chip-tool groupkeymanagement read group-table 1 0
TH_Logs.txt
DUT_Logs.txt

SDK Spec Review: Removing spec, as we believe we can make the SDK have a reasonable behavior here.

The "spec" label means "SDK is not following the spec".  Which in this case we are not....

This is due to the fact that the GroupTable isn't really a GroupTable but more of a Group/Group Key binding table. So in this case, you're removing a Group and NOT a binding. If I remember correctly the spec doesn't enforce the fact that upon a group removal you need to remove all related bindings (it would make sense to do so).
@rcasallas-silabs @turon

Already had this discussion with @turon here : https://github.com/CHIP-Specifications/chip-test-plans/pull/1006#discussion_r845121983

The problem, I believe, is that Group cluster, and Group Key Management cluster have mutually exclusive requirements. According to section <IP_ADDRESS>. (RemoveGroup Command): The RemoveGroup command allows a client to request that the server removes the membership for the endpoint implementing the server, if any, in a particular group.
This implies that the endpoint to group association must be removed, but point 3 adds:
3. The server removes the requested entry from the Group Table and the status SHALL be SUCCESS.
However, in the main spec, section <IP_ADDRESS>. (GroupInfoMapStruct), each GroupInfo (entries of the Group Table) must be:
GroupId     group-id
Endpoints   list[endpoint-no]
GroupName   string

So, removing one endpoint from the group must NOT remove the entire entry of the Group Table, for this would also remove ALL endpoints from the group.
However, what should happen when the last remaining endpoint is removed from a GroupInfo? Should then the GroupInfo be automatically removed from the Group Table? Is there such requirement?

This seems to be related to having multiple idea of "Group Table".
There is the GroupKeyManagement GroupTable attribute that relates the sum of the groups configured on all endpoints, by group ID. It's a read-only attribute.
There are the Group commands in the Groups cluster, which are per-endpoint and per fabric. The output of ViewGroup is <status, group ID, group Name> for a given group on a given endpoint.
There is also the "Logical concept of a group table" ("Group Table" as is appears in Groups cluster), which is where the prose is complete.
The Group Data Provider has:
    virtual bool HasEndpoint(FabricIndex fabric_index, GroupId group_id, EndpointId endpoint_id)          = 0;
    virtual CHIP_ERROR AddEndpoint(FabricIndex fabric_index, GroupId group_id, EndpointId endpoint_id)    = 0;
    virtual CHIP_ERROR RemoveEndpoint(FabricIndex fabric_index, GroupId group_id, EndpointId endpoint_id) = 0;
    virtual CHIP_ERROR RemoveEndpoint(FabricIndex fabric_index, EndpointId endpoint_id)

This affects the mapping of a group ID for an endpoint.
AddGroup command handler does:
static EmberAfStatus GroupAdd(FabricIndex fabricIndex, EndpointId endpointId, GroupId groupId, const CharSpan & groupName)
{
    VerifyOrReturnError(IsFabricGroupId(groupId), EMBER_ZCL_STATUS_INVALID_VALUE);

    GroupDataProvider * provider = GetGroupDataProvider();
    VerifyOrReturnError(nullptr != provider, EMBER_ZCL_STATUS_NOT_FOUND);

    CHIP_ERROR err = provider->SetGroupInfo(fabricIndex, GroupDataProvider::GroupInfo(groupId, groupName));
    if (CHIP_NO_ERROR == err)
    {
        err = provider->AddEndpoint(fabricIndex, groupId, endpointId);
    }
    if (CHIP_NO_ERROR == err)
    {
        return EMBER_ZCL_STATUS_SUCCESS;
    }

    ChipLogDetail(Zcl, "ERR: Failed to add mapping (end:%d, group:0x%x), err:%" CHIP_ERROR_FORMAT, endpointId, groupId,
                  err.Format());
    return EMBER_ZCL_STATUS_INSUFFICIENT_SPACE;
}

--> Sets group info for a given group ID (which, for instance, renames across all endpoints, since a group has a single name across all endpoints
To me, the whole thing makes perfect sense, but it's clumsily expressed.
For a given fabric:

You AddGroup per endpoint with Groups cluster, which calls both provider->SetGroupInfo and provider->AddEndpoint
You RemoveGroup per endpoint with groups cluster. The GroupKeyManagements Groups attribute will show one less entry in the Endpoints list for a given group ID. If a group is no longer bound to any endpoint, it no longer appears in the GroupKeyManagement Groups list at all.

In terms of actions, in sequence, starting empty, and assuming no name support (omitting names), I would expect:

READ EP0/GroupKeyManagement.GroupTable == []
WRITE EP0/GroupKeyManagement.GroupKeyMap[0] = <Group 222, Keyset 11222>
INVOKE EP2/Groups.AddGroup(222)
READ EP0/GroupKeyManagement.GroupTable == [<GroupId: 222, Endpoints: [2]>]
INVOKE EP5/Groups.AddGroup(222)
READ EP0/GroupKeyManagement.GroupTable == [<GroupId: 222, Endpoints: [2, 5]>]
INVOKE EP2/Groups.RemoveGroup(222)
READ EP0/GroupKeyManagement.GroupTable == [<GroupId: 222, Endpoints: [5]>]
INVOKE EP5/Groups.RemoveGroup(222)
READ EP0/GroupKeyManagement.GroupTable == []

Note how at after step 9, we no longer have the GroupId existing at all.
If after Step 9, it was:
READ EP0/GroupKeyManagement.GroupTable == [<GroupId: 222, Endpoints: []>]
Then we would be in a situation where the GroupTable attribute can always have groups that are basically non-responsive, and help nobody. GroupTable is supposed to reflect the groups added via AddGroup and removed via RemoveGroup

Agreed. Now we should put that on the spec, so the current behavior can be considered a bug, and be fixed.

Need to implement the result of:

https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/5341
https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/5342


Correct typo in spaces-github-actions.md

HF_USERNAME is not included in the space's URL

FULL_SPACE_NAME implied that it contained the user or org like user/space, maybe change FULL_SPACE_NAME to SPACE_NAME then?


Explorer: Blocks Vuex module

Description:

We should create the vuex module for blocks

Subticket of: #1744

most of this data seems to be available from lastHeader already. let's make sure we need another module before we bring it back.

I think it’s better to split it and move it to a new module


Some config files are created under root

Some configuration files are owned by root, some are owned by corresponding role-user (moc, bootnode, etc). They all should probably be owned by role-user.

These permissions are explicitly granted for files. @phahulin, are you sure about that they should be owned by  user?
As from my point of view - yes, the processes that are using these files (for example node.toml) are launched from  user, so we should change the permissions accordingly. But i might have missed something.
It is easy to fix this issue, but we need to be sure, that the new approach is correct.
owner=root group=root mode=0644

@ArseniiPetrovich Yes, let's change it and have them all owned by role-specific user.


Book and Play Edition Number search should not include "n/a"

If I search for Book Edition Number = second-plus, I get a lot of Plays in Collection in the results, I guess because they have "n/a" in the Book Edition Number field?
Search for Book Edition Number -- or Play Edition Number -- needs to exclude all Items with "n/a" in that field.  The same error results in Collections being included in Play Edition Number = second-plus, since they have "n/a" in that field.
So "Second-plus" needs to equal "integers greater than 1", not "anything that isn't = 1" for this reason. Perhaps that's the logic error in the code?

This seems to have been fixed by making Book Edition and Play Edition = 0 in cases of "n/a" -- this might work well for filtering purposes.
BUT: for display purposes, any time there is a zero in one of these fields, it should display as "n/a"

Alternatively you can replace zero with "n/a" in the data itself, so long as the Filter can exclude any items with "n/a" in the field.

The "0" should also not appear in the list of filter choices in the drop-down.
The easiest thing may be to substitute "n/a" for "0" in the data itself, but exclude it from the list of filter choices and have the searches for Book Edition Number and Play Edition Number ignore anything that is not a numeral / integer in the data field?


Jeremr/python databricks setup

Description
Change the databricks installation and preparation scripts to leverage python to be more cross-platform.
This will help get to the point of also making these operations a bit more modular as well.
Related Issues
#570
#564
#567
Checklist:

[X] My code follows the code style of this project, as detailed in our contribution guidelines.
[ ] I have added tests. (no tests for databricks/azure resources yet.)
[x] I have updated the documentation accordingly. (will update README with reviews in shortly; want to have people look at it if possible.)

updated setup.md, so docs are updated.

another general comment, I think it would be handy to create the Recommenders egg with branch / version info if possible: e.g.: Recommenders-v2019.01-staging
thoughts? otherwise it can be very confusing to understand what version of the egg is being used on databricks

another general comment, I think it would be handy to create the Recommenders egg with branch / version info if possible: e.g.: Recommenders-v2019.01-staging

thoughts? otherwise it can be very confusing to understand what version of the egg is being used on databricks
It would be great that we could fix the set-up here for just using the release which is the stable version of the repo.

another general comment, I think it would be handy to create the Recommenders egg with branch / version info if possible: e.g.: Recommenders-v2019.01-staging

this can be done programatically using the version here:
https://github.com/Microsoft/Recommenders/blob/master/reco_utils/__init__.py#L8

another general comment, I think it would be handy to create the Recommenders egg with branch / version info if possible: e.g.: Recommenders-v2019.01-staging
thoughts? otherwise it can be very confusing to understand what version of the egg is being used on databricks

Definitely considered this, and agree that this would be good. The issue that I worry about is two-fold.

Is the un-installation of other prior versions. that requires a bit more overhead if they're not the same name. If the file gets overwritten, then it has to install from that same location. If we start versioning the files (which I agree is a good idea), then we need to do more to check whether a library is already installed and manually uninstall. We could just regex for Recommenders-XXX.egg in the results of a databricks libraries list
I view the ability to install from any branch as a feature, because it's super helpful to be able to test things rapidly. It would definitely have more value if we could also version the egg file as well.

Those are good points, i think being able to overwrite easily is better. One option might be creating a temporary file prior to compression which would allow you to inspect the branch / commit that was used. We could modify reco_utils.init.py to look for this file and if it exists read in the git details. Anyway, I don't want to hold this PR up. We can add this as a future feature.

hey @jreynolds01 is there more work here to be done?

one minute, I'm updating some of the docs

Ok, should be all set once tests pass, I testing recreating the reco_spark.yaml, updating my conda env, uploading the recommenders egg (failed with existing file, works with overwrite flag) and install mmlspark.
nice work @jreynolds01

Thanks Scott and Miguel for updates.  I think it's ready.


please make it possible to hide "tips baloon" when mouse is moved on MenuBar

currently when you hover a mouse over the MenuBar there appears a baloon tip for a short time.
can you please create an option to disable this feature ? thanks

so much needed


Modify 'Header Search Path' for native iOS app to integrate react-nat…

Modify 'Header Search Path' for native iOS app to integrate react-native-webp from react-native package.json.
In file included from /Users/OceanHorn/SourceTree/mmms/node_modules/react-native-webp/DBAWebpImageDecoder.m:1:
In file included from /Users/OceanHorn/SourceTree/mmms/node_modules/react-native-webp/DBAWebpImageDecoder.h:1:
../../node_modules/react-native/Libraries/Image/RCTImageLoader.h:12:9: fatal error: 'React/RCTBridge.h' file not found
#import <React/RCTBridge.h>
        ^~~~~~~~~~~~~~~~~~~
1 error generated.

I also tried to apply this fix but I cannot get this to run again with RN 0.60+. I tried on 0.62.4 (after upgrade) and had it all working on 0.59.10.
Then things stopped working. The exact same issue persists even with a clean 0.62.4 installation.

I found another solution in the meaning time: Using react-native-fast-image for local images.
https://github.com/DylanVann/react-native-fast-image/issues/476#issuecomment-512723403