Required parameters in a method are used across HTTP methods executor.

Follow up to https://github.com/microsoft/kiota/issues/3989
Taking a look at the url template
{+baseurl}/groups/{group%2Did}/members/$ref?@id={%40id}{&%24count,%24filter,%24orderby,%24search,%24skip,%24top} generated from the description below.
  '/groups/{group-id}/members/$ref':
    description: Provides operations to manage the collection of group entities.
    get:
      tags:
        - groups.directoryObject
      summary: Get ref of members from groups
      description: 'The members of this group, who can be users, devices, other groups, or service principals. Supports the List members, Add member, and Remove member operations. Nullable. Supports $expand including nested $select. For example, /groups?$filter=startsWith(displayName,''Role'')&$select=id,displayName&$expand=members($select=id,userPrincipalName,displayName).'
      externalDocs:
        description: Find more info here
        url: https://learn.microsoft.com/graph/api/group-list-members?view=graph-rest-1.0
      operationId: groups.ListRefMembers
      parameters:
        - name: ConsistencyLevel
          in: header
          description: 'Indicates the requested consistency level. Documentation URL: https://docs.microsoft.com/graph/aad-advanced-queries'
          schema:
            type: string
          examples:
            example-1:
              description: $search and $count queries require the client to set the ConsistencyLevel HTTP header to 'eventual'.
              value: eventual
        - $ref: '#/components/parameters/top'
        - $ref: '#/components/parameters/skip'
        - $ref: '#/components/parameters/search'
        - $ref: '#/components/parameters/filter'
        - $ref: '#/components/parameters/count'
        - name: $orderby
          in: query
          description: Order items by property values
          style: form
          explode: false
          schema:
            uniqueItems: true
            type: array
            items:
              enum:
                - id
                - id desc
                - deletedDateTime
                - deletedDateTime desc
              type: string
      responses:
        2XX:
          $ref: '#/components/responses/StringCollectionResponse'
        4XX:
          $ref: '#/components/responses/error'
        5XX:
          $ref: '#/components/responses/error'
      x-ms-pageable:
        nextLinkName<EMAIL_ADDRESS>        operationName: listMore
      x-ms-docs-operation-type: operation
    post:
      tags:
        - groups.directoryObject
      summary: Add members
      description: Add a member to a security or Microsoft 365 group through the members navigation property. The following table shows the types of members that can be added to either security groups or Microsoft 365 groups.
      externalDocs:
        description: Find more info here
        url: https://learn.microsoft.com/graph/api/group-post-members?view=graph-rest-1.0
      operationId: groups.CreateRefMembers
      requestBody:
        $ref: '#/components/requestBodies/refPostBody'
      responses:
        '204':
          description: Success
        4XX:
          $ref: '#/components/responses/error'
        5XX:
          $ref: '#/components/responses/error'
      x-ms-docs-operation-type: operation
    delete:
      tags:
        - groups.directoryObject
      summary: Remove member
      description: Remove a member from a group via the members navigation property. You can't remove a member from groups with dynamic memberships.
      externalDocs:
        description: Find more info here
        url: https://learn.microsoft.com/graph/api/group-delete-members?view=graph-rest-1.0
      operationId: groups.DeleteRefMembers
      parameters:
        - name: If-Match
          in: header
          description: ETag
          schema:
            type: string
        - name: '@id'
          in: query
          description: The delete Uri
          required: true
          schema:
            type: string
      responses:
        '204':
          description: Success
        4XX:
          $ref: '#/components/responses/error'
        5XX:
          $ref: '#/components/responses/error'
      x-ms-docs-operation-type: operation
    parameters:
      - name: group-id
        in: path
        description: The unique identifier of group
        required: true
        schema:
          type: string
        x-ms-docs-key-type: group

From the yaml description we conclude that

The @id parameter is only required for the DELETE method
The template is used for all methods thus making a GET request will generate the url https://graph.microsoft.com/v1.0/groups/groupId/members/$ref?@id= which will result in an API error as seen here.

To avoid the issue, the required parameters should be added to the template if they are in all methods or introduce secondary templates for such methods.

Thanks for reporting this issue.
I think this is another instance where the history of the project starts showing up...
When we transitioned from concatenating strings to URI templates, we originally had a single URI template per request builder. Both for simplicity reasons and to avoid duplication of information.
Because at the time we translated all query parameters as optional, differences in query parameters between operations didn't matter.
But this clearly shows to be a design mistake.
Learning from the optimization work in the TypeScript world I've done over the past month, I think we should take this as an opportunity to have defaults and overrides, in order to save sizes when possible.
Currently, most languages besides TypeScript are generated looking this way:
at the constructor level
public MessageItemRequestBuilder(string rawUrl, IRequestAdapter requestAdapter) : base(requestAdapter, "{+baseurl}/users/{user%2Did}/messages/{message%2Did}{?includeHiddenMessages*,%24select,%24expand}", rawUrl) {
        }

At the request generator level
public RequestInformation ToDeleteRequestInformation(Action<RequestConfiguration<DefaultQueryParameters>> requestConfiguration = default) {
            var requestInfo = new RequestInformation(Method.DELETE, UrlTemplate, PathParameters);
            requestInfo.Configure(requestConfiguration);
            requestInfo.Headers.TryAdd("Accept", "application/json");
            return requestInfo;
        }

This UrlTemplate property was assigned by the constructor.
We could still keep the same logic, but have another template in the generator when it differs.
As for the template to use in the constructor, we could use the one that has the most occurrences among operations, if a tie, alphabetically.
Thoughts?

e most occurrences among operations

+1 on the suggestion here. We should do this.
This should result in small changes in the generated code with only the parameter passed to the request generator level changing.


Feature Request: Support for AMD Anti-Lag 2 Integration

Hello,
I recently came across this pull request in the R5SDK project: Mauler125/r5sdk#127, which integrates AMD's Anti-Lag 2 technology.
It appears that integrating AMD Anti-Lag 2 into DXVK could significantly benefit games running on AMD hardware, offering an alternative to NVIDIA Reflex. This could potentially be the first game to leverage such an integration through DXVK.
Is there any possibility of implementing support for AMD Anti-Lag 2 in the foreseeable future?
Thank you for your consideration.

This is not an engine, my friend; the game has to implement it, not the driver. You should close this one.

@cloudishBenne Hi there.
As pointed out above Anti-Lag 2 needs integration in the game for it to work. You are probably thinking of Anti-Lag 1 which is a driver level thing.
Though dxvk and vkd3d-proton would probably have to implement support for using Anti-Lag 2 in games that support it same as with Reflex. The AMD driver in use would need to support it too ofc.

@Blisto91 Yes, that's what i meant, adding support for it equal to what happened to Reflex. R5Reloaded will have it implemented ingame with the next release.
Also thank you for mentioning the additional requirement of a driver side implementation. If someone of you has useful information, what i should mention in a mesa issue, i will open one there.

I am not a dev (tester) so I can't comment on any technical specifics or requirements. From the driver side I imagine it just have to support the VK_AMD_anti_lag extension.
I don't know if the amd ags vendor library in Proton Wine needs to be worked on for support too. Might be likely like with dxvk-nvapi needing to support Reflex.
But either way driver support would probably be a first step.

I am not a dev (tester) so I can't comment on any technical specifics or requirements. From the driver side I imagine it just have to support the VK_AMD_anti_lag extension.
I don't know if the amd ags vendor library in Proton Wine needs to be worked on for support too. Might be likely like with dxvk-nvapi needing to support Reflex. But either way driver support would probably be a first step.

The VK_AMD_anti_lag implementation is in the works, but there is focus on other priorities. It will be addressed eventually.

How do you know it is in the works

How do you know it is in the works

Samuel was discussing the implementation, but he found himself constrained by VK_KHR_shader_float_controls2. Nevertheless, he began drafting it prior to thinking about creating a merge request.

How do you know it is in the works

Samuel was discussing the implementation, but he found himself constrained by VK_KHR_shader_float_controls2. Nevertheless, he began drafting it prior to thinking about creating a merge request.

Thank you for this information!
So this means writing an issue would be redundant i guess?
Do you have a link to his work?

How do you know it is in the works

Samuel was discussing the implementation, but he found himself constrained by VK_KHR_shader_float_controls2. Nevertheless, he began drafting it prior to thinking about creating a merge request.

Thank you for this information!
So this means writing an issue would be redundant i guess?
Do you have a link to his work?

No, there's no merge request open yet. I'll post it when he opens it.


MockShared is not generating classes correctly

Description
In my project, we have an extra source-set between common and the platform called mobile(Main/Test)
We use it to avoid exposing mobile-only classes to JS
Expected Behavior
When using @MockShared("mobileTest", MobileMainClass::class), it should generate the mock for the mobileTest source-set
Actual Behavior
The mock class is not being generated

Steps to Reproduce

Download example project https://github.com/faogustavo/KMockExample
Open the ViewModelTest class
Try to run

Your Environment

Version used: Snapshot

@faogustavo for submitting the issue and the contribution! A fix is merged and will be available in a couple minutes via snapshots. Please confirm it resolves the issue.

@bitPogo everything working in the snapshot! I'm closing this one :)


its duping when i dissassemble

its duping when i dissassemble

(e.g. 1.12.2)
(e.g. <IP_ADDRESS>54)

i know it why now: ive placed a bugged antenna of Galacticraft there but now i can copy items


Fix UI staleness after network issues (#1192)

Cherry pick #1192 into v3.6

required a slight modification, @jakolehm PTAL


Support custom path to user config

Specify custom init.vim through vscode-neovim.neovimInitPath vscode setting.
Also may be will be more useful to use same init.vim and dynamically adjust if running in vscode, see  #87
will be in 0.0.40


Add support for component google-mail

http://camel.apache.org/googlemail.html

Found 3 matching posts for subject:google-mail in Camel - Users within 700 days

No in catalog any more


EntityManager::getReference() and getPartialReference() do not return null

ObjectManagerFindDynamicReturnTypeExtension sets the return type of getReference() and getPartialReference() to nullable. However, these methods never return null.

According to the documentation https://www.doctrine-project.org/api/orm/latest/Doctrine/ORM/EntityManager.html#method_getReference, they do.

I even checked the source code - https://github.com/doctrine/orm/blob/master/lib/Doctrine/ORM/EntityManager.php#L510

Oh, my mistake then. Thanks :)


bug: ion-infinite-scroll stops working in v1.2.1

It worked fine in v1.1.1 and after updating to v1.2.1, it stops working and reports nothing in the browser's console. It seems on-infinite callback doesn't get called.
Anyone else encounters this problem?
    <ion-content style="padding: 0px; background-color: #f4f4f4;">
          <ion-list>
	        <ion-item>
	        </ion-item>
         </ion-list>
	 <ion-infinite-scroll on-infinite="addItems()" ng-if="moreDataCanBeLoaded()" immediate-check="false"></ion-infinite-scroll>
    </ion-content>

set config as:
.config(function($stateProvider, $urlRouterProvider, $ionicConfigProvider, $httpProvider) {
$ionicConfigProvider.scrolling.jsScrolling(true);
v1.1.1 default is true
v1.2.1 default is false

@pengpengmz this does fix my "problem". Thank you very much!


Add Class Diagrams to Wiki Page

Class Diagrams should be added to Wiki Page.

Sidebar nav should be arranged accordingly.

An out-link should be given to original source.

Created.
TO-DO:

[ ] Add Class Diagrams to page
[ ] Add original resource
[ ] Nav-link

Completed.


fix: mark attributes of sensitive blocks as sensitive

Recursively mark attributes of sensitive blocks as sensitive due to an issue in Terraform

Tests https://github.com/aiven/terraform-provider-aiven/actions/runs/10471136785


fix: compatibility of bind_mount_volumes, docker_volumes and efs_volumes

what

Sets the expected data format of bind_mount_volumes
Explicitly marks the docker_volumes optional fields

why

fixes an issue that prevents when using a combination of efs_volumes, docker_volumes and bind_mount_volumes (see below)
allows omitting optional parameters from the docker_volumes objects

Example configuration

module "test_ecs_service" {
  source  = "cloudposse/ecs-alb-service-task/aws"
  version = "0.67.1"

  # Omitted non-relevant parts here
  # [...]

  docker_volumes = [
    {
      host_path = null               # This should not be mandatory
      name = "temp",
      docker_volume_configuration = [{
        autoprovision = null         # This should not be mandatory
        labels = {}                  # This should not be mandatory
        scope  = "task"
        driver = "local"
        driver_opts = {
          "device" = "tmpfs"
          "o"      = "size=1g,uid=82"
          "type"   = "tmpfs"
        }
      }]
    }
  ]

  efs_volumes    = [
    {
      name      = "files"
      host_path = null
      efs_volume_configuration = [
        {
          file_system_id          = module.efs.id
          root_directory          = "/"
          transit_encryption      = "ENABLED"
          transit_encryption_port = null
          authorization_config = [{
            iam             = "ENABLED"
            access_point_id = module.efs.access_point_id
          }]
        }
      ]
    }
  ]
}

Encountered error:
 Error: Error in function call

   on .terraform/modules/test_ecs_service/main.tf line 11, in locals:
   11:   volumes = concat(var.docker_volumes, var.efs_volumes, var.fsx_volumes, var.bind_mount_volumes)
     ├────────────────
     │ while calling concat(seqs...)
     │ var.bind_mount_volumes is empty list of dynamic
     │ var.docker_volumes is list of object with 1 element
     │ var.efs_volumes is list of object with 2 elements
     │ var.fsx_volumes is empty list of object

references
N/A

@esolitos Thank you for this PR. However, this requires Terraform 1.3, and we do not want to lock out older versions just yet without a very strong reason.
I believe this PR also does not solve the problem, which is that docker_volumes, efs_volumes, fsx_volumes, and bind_mount_volumes are different Terraform types and cannot be combined into a list. Fixing that will take a more substantial rewrite.

/terratest


[Feature Request]: Make JSON encoding consistent with Celestia-Core

Implementation ideas
Background
Section 9 of rfc 8359 allows JSON parsers to silently truncate numeric types to arbitrary ranges and precisions. Since this "feature" is actually implemented in production JSON parsers, JSON integer types are unreliable.
To circumvent this issue, Tendermint serializes some int and all int64 fields as strings (see here), and relies on application-level logic to convert those strings to appropriate integer types.
As a fork of Tendermint, Celestia-core inherits this behavior. So, for example, a serialized block header looks like this:
{
	"header": {
		"version": {
		        "block": "11"
		},
		"chain_id": "arabica-2",
		"height": "45963",
		"time": "2022-11-29T00:14:29.836769661Z",
		"last_block_id": {
		"hash": "3B5F0613A6CB7C60BEB1DDADE9461475E5603122293E2E07E770DFE91F2A6D6C",
		"parts": {
			"total": 1,
			"hash": "8FA8BFE8AD2A3C69A888470445E254D1266D286939FBA679539D93380616471B"
		}
		},
		...
	},
}

On the other hand, the celestia-node /header/ RPC endpoint uses standard JSON encoding for the tendermint header, representing numeric types as JSON numbers.
{
	"header": {
		"version": {
		        "block": 11
		},
		"chain_id": "arabica-2",
		"height": 45963,
		"time": "2022-11-29T00:14:29.836769661Z",
		"last_block_id": {
		"hash": "3B5F0613A6CB7C60BEB1DDADE9461475E5603122293E2E07E770DFE91F2A6D6C",
		"parts": {
			"total": 1,
			"hash": "8FA8BFE8AD2A3C69A888470445E254D1266D286939FBA679539D93380616471B"
		}
		},
		...
	},
}

Since celestia-node inherits its types from celestia-core, this divergence was likely introduced in the custom marshalling logic here
Steps to Reproduce
Compile Celestia Node version v0.5.0-rc5:
$ celestia version
Semantic version: v0.5.0-rc5
Commit: 039facd94104c741f8fa995e80f1321776ae4ad3
Build Date: Mon Nov 28 17:05:56 CST 2022
System version: amd64/darwin
Golang version: go1.19

Start the Celestia node on Arabica testnet:
celestia light start --core.ip https://limani.celestia-devops.dev --gateway --gateway.port 26659

Wait a few minutes for the node to sync, then fetch a header:
curl -X GET http://localhost:26659/header/300

Compare the output with the output of celestia-core at the same height
curl -X POST -H "Content-type: application/json" 'http://limani.celestia-devops.dev:26657/blockchain?minHeight=300&maxHeight=300'

Impact
This inconsistency prevents the use of standard Tendermint libraries including tendermint-rs with celestia-node.

Hello @preston-evans98 and thanks for reporting this diversion.
I looked at the both attached json samples, but could not find any difference between them. Are they correct?

If you look closely, you should see that the first one has "height": "45963" (notice the quotes around the number) while the second has "height": 45963. The same is true of version.block field.

Nice catch @preston-evans98, thanks for opening this issue
The custom logic was only meant to change the encoding of one type (crypto.PubKey) to use amino, as the default JSON output cannot be unmarshaled back otherwise.
Will look into fixing the other fields

@distractedm1nd Any progress on this?

@distractedm1nd I'm still seeing this problem on the latest testnet
$ curl -X GET localhost:26659/header/300
{"validator_set": {...} "header":{"version":{"block":11},"chain_id":"mocha","height":300,"time":"2022-12-08T21:20:52.05495665Z", ...}}

$ ./celestia version
Semantic version: v0.6.4


fix: remove node-ip dependency

Description of changes
Remove dependency on poorly maintained "ip" package, which has an open security advisory https://github.com/indutny/node-ip/issues/150
Implement an equivalent getLocalIpAddress() function that returns the first internal IP address. This is used by clients of the appsync-sumulator to connect to.
Issue #, if available
https://github.com/aws-amplify/amplify-cli/issues/13890
Description of how you validated changes
Ran yarn lerna run --scope @aws-amplify/amplify-appsync-simulator test
Checklist

[x] PR description included
[x] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thank you for contributing!


Data availability

Hello,
First of all, thanks for making this exciting work open-source. I want to test it. Do you know if the data used for training open? If so, where can I find it?
I have been checking this data source from swissSURFACE but the resolution doesn't seem enough for tree segmentation.
https://www.swisstopo.admin.ch/en/geodata/height/surface3d.html#musterdaten
Is this the data source that you have been using?
Thanks beforehand,
Carlos

Thanks @caviri for your interest in our work!
We used this point clouds dataset, which is released by the Canton of Geneva (my employer) under Open Data terms. I can help you getting that dataset, if you need it. As far as ground-truth data is concerned, we used a curated subset of this other dataset, plus some other data which were collected on purpose for this project.
Best,
Alessandro

Thank you a lot for the answer @acerioni. I just found time to check this now. This is a side project, and I only have a little time to dedicate to it. Nevertheless,  I will try to replicate the pipeline by using the point cloud dataset available on the first link you shared. Thank you again.


Allow customizing gdscript port

Language server
gdscript
Requested feature
The gdscript configuration hard-codes 6008 (https://github.com/neovim/nvim-lspconfig/blob/master/lua/lspconfig/server_configurations/gdscript.lua), while Godot allows for customizing the port (see screenshot).

Since Godot 3 and 4 uses different ports, it would be great to allow customizing the port.
Other clients which have this feature
Godot's VS Code plugin has an option to change the port.

Also it seems CoC has it, however I'm not experiences using it. https://github.com/habamax/vim-godot#setting-up-cocnvim-for-godot-322

#2313 work with a port env variable named GDScript_Port .

That would definitely help. Is there a way to pass it to the setup function too?

hmm currently can do that . but a little trouble :) so maybe we can support it in future.


🐛(frontend) problem router on upload document

Purpose
After uploading a document, the router didn't have the correct formating to be able to extract the params.
This PR fixes that.

Please merge this to fix e2e tests, @AntoLC  :pray:

I added lot of tests to the LTI routes:
https://github.com/openfun/marsha/pull/2306/files#diff-3b7495a0ed4a27f6e5943f95e75578ebc63cad5fdee3a952f5190ea81e392737


"justifyContent: space-evenly" gets prefixed to an invalid result

The result of
{
  justifyContent: 'space-evenly'
}

Gets turned into
{
  WebkitBoxPack: "space-evenly",
  justifyContent: "space-evenly"
}

"-webkit-box-pack" doesn't allow the value of "space-evenly": https://developer.mozilla.org/en-US/docs/Web/CSS/box-pack

Hmm. As the value doesn't exist in general, what would you use as a fallback value? We can add justify as well. Otherwise it simply doesn't matter as the browser won't be able to do anything with that invalid value.

Personally, I think it should be dropped as a "prefixed" style since -webkit-box-pack is not directly compatible with justify-content.
It's not a huge deal, I just noticed it when Webstorm marked it as an error in my generated stylesheet.


docs: Example of reading an octet stream

Hello!
Loving the abstraction so far, great job!
Was wondering if we could add an example that reads an octet-stream similar to this in the cli repo using net/http?
I'm trying a couple of different approaches, but I'm not sure what's the best way to define the response to facilitate reading bytes?
Let me know if I can help in any way as well 😄
Edit: I'm using the rest client for reference, and looking at https://github.com/cli/go-gh/blob/c41a1271d97cf42ba0dcf94289da729cdb1554eb/internal/api/rest_client.go#L48 I'm not sure this is possible currently?
Using Get or Do I get:
invalid character '\x1f' looking for beginning of value

@alejandrovelez7 That is a good question. In the latest release (v0.0.3) there isn't a good way to read an octet-stream with the RESTClient and you would need to use the raw HTTPClient. Having said that, in trunk, there is a new function Request on the RESTClient that will return the http response without doing any reading or parsing. Using that function it should be easy to adapt the code you linked in the cli repo to read an octet-stream.
https://github.com/cli/go-gh/blob/91ca4ef447d4db5075be14acdac2759710983b26/internal/api/rest_client.go#L51
Hopefully that answers your question. Let me know if anything is unclear!


Cannot cancel world saving

I tried cancelling SaveWorldEvent, but it saves anyways.
After researching, I noticed that the events are posted, but it is not checked whether it was cancelled:
https://github.com/SpongePowered/SpongeCommon/blob/10c4e2f31c5bacd0b6e4b9018e528ca0cb9dd3fe/src/main/java/org/spongepowered/common/mixin/core/world/MixinWorldServer.java#L890.
Also line 899.
I also looked in SpongeForge and SpongeVanilla, this is the only occurrence.
This should be irrelevant, but I reproduced it on spongevanilla-1.11.2-6.0.0-BETA-208.
This should be an easy fix.
Thanks!

Feel like submitting a pull request?

This should be closed considering the PR has been merged.


Compare API Gateways

[x] #226

Articles:

How to choose the right API Gateway for your platform: Comparison of Kong, Tyk, Apigee, and alternatives

I'm using the Spring Cloud Gateway.


BUG - deployment to ABAP fails with 502 error from Axios

Related Feature
No related fuatures/issues found
Description
Deployment to ABAP using deploy-to-abap task fails with following error:
debug abap-deploy-task *** Retrieving application *** from /sap/opu/odata/UI5/ABAP_REPOSITORY_SRV, AxiosError: Request failed with status code 502

We're connecting to ABAP system through the gateway, deployment happens through fiori deploy --config ui5.yaml --yes command
Full log of command is attached to an issue (all private parts are being masked with asterisk
deploy-to-abap-logs.txt
).
Expected results
Application is deployed to ABAP system protected behind proxy
Actual results
Error being thrown
Version/Components/Environment
@sap/ux-ui5-tooling version: 1.14.1
Add any other context about the problem here
OS:

[ ] Mac OS
[x] Windows
[ ] Other

Root Cause Analysis
Problem
{describe the problem}
Fix
{describe the fix}
Why was it missed
{Some explanation why this issue might have been missed during normal development/testing cycle}
How can we avoid this
{if we don’t want to see this type of issues anymore what we should do to prevent}

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Hi @tavnabisoft
Can you please run the following command;
curl -L -u ‘username:password’ -X GET -i -H 'X-Csrf-Token:Fetch' http://your-site-url/sap/bc/adt/discovery > output.txt

Please update the username, password and site URL details to reflect your environment.
This is a good test to validate, independent of our tooling if its an issue with the ABAP target system and/or any local proxy/firewall as the HTTP error 502 indicates an upstream issue.
Can you also confirm any details related to your ABAP system? Any additional details about your system would be appreciated too i.e. on-premise | cloud.
Thanks.
John

Hello @longieirl  ,
Thanks for quick response! I've attached output files for both commands.
output-tsk1.txt
output-tsk2.txt
ABAP system is on-premise. Here is some additional info that may be useful:

SAP_BASIS
752
0012
SAPK-75212INSAPBASIS

SAP_ABA
752
0012
SAPK-75212INSAPABA

SAP_GWFND
752
0012
SAPK-75212INSAPGWFND

SAP_UI
757
0003
SAPK-75703INSAPUI

There is a workaround for mentioned issue is by running in parallel ui5 serve command which refers to ui config with fiori-tools-proxy middleware which proxies requests to abap system. Here is the config of ui5.yaml file:
specVersion: "3.0"
metadata:
  name: ***
type: application

server:
  customMiddleware:
    # credentials are defined via environment variables vars: https://www.npmjs.com/package/@sap/ux-ui5-tooling#providing-credentials
    - name: fiori-tools-proxy
      afterMiddleware: compression
      configuration:
        ignoreCertError: false
        backend:
          - path: /sap/opu/odata
            url: https://***.com
          - path: /sap/public/bc
            url: https://***.com

Let me know if you need any further details.
Best Regards,
Slava

@tavnabisoft so you are running the command fiori deploy --config ui5.yaml --yes which is reading the ui5.yaml as the config. Can you share this config?
Also, when you ran the curl commands? Were they run with ui5-serve running in the background?

@longieirl we've run mentioned curl commands without running ui5 serve command in the background. So requests are executed with no issues without proxying by ui5 middleware.
Here is config which we're using for deployment to abap. If we're running deployment with ui5 serve in background, url of abap system is changed to http://localhost:8080:
specVersion: "3.0"
metadata:
  name: test-app
type: application

builder:
  resources:
    excludes:
      - /test/**
      - /localService/**
      - /index.html

  customTasks:

    - name: ui5-tooling-transpile-task
      afterTask: escapeNonAsciiCharacters
      configuration:
        debug: false
        removeConsoleStatements: true
        transformAsyncToPromise: true

    - name: deploy-to-abap
      afterTask: generateCachebusterInfo
      configuration:
        target:
          url: https://***.com
          auth: basic
        app:
          name: /TESTAPP
          description: "Test app"
          package: /TEST
          transport: ABC1324567
        credentials:
          username: env:SAP_GATEWAY_AUTH_USR
          password: env:SAP_GATEWAY_AUTH_PSW

Morning @tavnabisoft, a couple of things to note,
The curl commands are picking up the proxy Fortinet-Proxy so I'm wondering if the issue is related to this. The ui5 command must be picking up the proxy as well. When I look at the deploy-to-abap-logs.txt it shows the host as proxy.***.com, is this correct? Is that the expected proxy?
It might be worth reviewing the npm proxy settings to ensure the proxy is picked up;
https://ga.support.sap.com/dtp/viewer/index.html#/tree/3046/actions/45995:48363:53593:53499
Also, when you run ui5 serve, it should indicate in the logs if a proxy is defined? Can you confirm this?
FYI, you can append the following exclude to the same level as app and target;
        exclude:
          - /test/

You also have testMode: true configured but this is not in your config. By default it should be false. I'm wondering where this is being picked up?

Hi @longieirl

The ui5 command must be picking up the proxy as well. When I look at the deploy-to-abap-logs.txt it shows the host as proxy.***.com, is this correct? Is that the expected proxy?

Yes, it is an expected proxy, the url has been masked. I've attached additional log for details, urls should be there:
abap-deploy-logs-3.txt

It might be worth reviewing the npm proxy settings to ensure the proxy is picked up; https://ga.support.sap.com/dtp/viewer/index.html#/tree/3046/actions/45995:48363:53593:53499

Checked => proxy urls are set up

Also, when you run ui5 serve, it should indicate in the logs if a proxy is defined? Can you confirm this?

Yes, according to the logs proxy config is being picked. I've attached logs of ui5 serve command for reference:
ui5-serve-logs.txt

FYI, you can append the following exclude to the same level as app and target;

Thanks for notice, we will adjust that :)

You also have testMode: true configured but this is not in your config. By default it should be false. I'm wondering where this is being picked up?

Yes, I've executed the task in test mode firstly (was defined in config of the deploy-to-abap task). The newly provided logs abap-deploy-logs-3.txt were executed with test mode being turned off

Hi @tavnabisoft,
Thanks again for the logs;
A couple of more questions!

Have you configured a SAP Fiori system using the URL and client as defined in your ui5 yaml configuration? For example, if you open VSCode and select SAP Fiori on the left navigation bar, is there is a system created matching the same host and client?
Can you please create a new scipt in your package.json or manipulate the existing script, for example;

# Windows
set DEBUG=* && fiori deploy --config ui5.yaml --yes
# Linux/Mac
DEBUG=* && fiori deploy --config ui5.yaml --yes

You may need to tweak the fiori command to ensure it matches your deployment settings. Please attach the generated log, again, you may need to obfuscate any sensitive data.
This should hopefully output all the axios HTTP requests which might provide more insight to why the request is failing.
Thanks.
John

Hello @longieirl,

System was added but test connection returns following error:

2024-09-04 12:55:06.600 [error] The V2 request failed with error: Request failed with status code 502.
2024-09-04 12:55:06.601 [error] The V4 request failed with error: Request failed with status code 502.

Please find attached logs of mentioned script: debug-all-deploy-log.txt

Thank you again for the latest scripts.
I've a feeling the ui5 serve is working since that pulls in as many environment variables as it can find i.e.
        process.env.FIORI_TOOLS_PROXY ||
        process.env.npm_config_proxy ||
        process.env.npm_config_https_proxy ||
        process.env.http_proxy ||
        process.env.HTTP_PROXY ||
        process.env.https_proxy ||
        process.env.HTTPS_PROXY ||

But I'm wondering what the npm proxy settings are; if you run the following command, it will output the the variables used when npm is executed;
npm config list

Does this have a proxy setting? Can you please verify they are correct.
You can edit this file by running the following command, it will default to your system editor i.e. vi;
npm config edit

You can set them manually;
npm config set proxy http://<username><password>@proxy-server-url>:<port>
npm config set https-proxy http://<username><password>@proxy-server-url>:<port>

Once you have made any modifications, you can re-run the deploy command again.
Thanks.

Ok! Thank you for the investigation, its appreciated. Let me do some more analysis and I will hopefully come up with a plan of action to get this resolved.

Soo I've been playing with the following docker setup;
---
services:
  proxy:
    image: ubuntu/squid:latest
    container_name: squidproxy
    restart: unless-stopped
    ports:
      - 3128:3128

Then setting the npm proxy using;
npm config set proxy http://localhost:3128
npm config set https-proxy http://localhost:3128

And I'm able to deploy as expected, in the squid proxy logs, I can see the GET and POST requests.
I need to tweak the settings to validate passing credentials to see if that is an issue.

Hello @longieirl ,
Thanks for investigation, but I believe that difference in original case and your scenario is that request should go to https target url over the http proxy. Axios does not support this scenario out the box and require usage of https-proxy-agent library to execute such requests. You can see details in Stackoverflow questiion and Github Issue of axios.

Morning,
I've managed to hack a squid proxy with SSL certs and I'm getting a 503 when using axios. The same request using a curl with the proxy flag, is working as expected.
info abap-deploy-task ZTESTJL Creating archive with UI5 build result.
info abap-deploy-task ZTESTJL Archive created.
info abap-deploy-task ZTESTJL Starting to deploy.
error abap-deploy-task ZTESTJL Deployment has failed.
error abap-deploy-task ZTESTJL Change logging level to debug your issue
error abap-deploy-task ZTESTJL  (see examples https://github.com/SAP/open-ux-tools/tree/main/packages/deploy-tooling#configuration-with-logging-enabled)
error builder:custom deploy-to-abap Request failed with status code 503
Command deploy failed with error : Request failed with status code 503

I will try to address this issue over the coming days.
Thanks again.
John

Hello @longieirl ,
Do you have updates on the topic?

Hello @longieirl , Thanks for investigation, but I believe that difference in original case and your scenario is that request should go to https target url over the http proxy. Axios does not support this scenario out the box and require usage of https-proxy-agent library to execute such requests. You can see details in Stackoverflow questiion and Github Issue of axios.

I noticed the same behavior with the abap deploy task some time ago. All other middleware are working fine, but they don't use @sap-ux/axios-extension and use a different http lib.
The following didn't work for me

HTTP over HTTP proxy doesn't work (I wasn't able to reach the system - I tested this only a few minutes)
HTTPs over HTTP proxy doesn't work
ignoreCertErrors was ignored

I was able to fix 2 and 3 with the following patch and just replaced the whole @sap-ux/axios-extension module in my project via npm@overrides.  I assume it will only work for HTTPs requests, because httpAgent isn't patched for HTTP yet.
Note: I also had to patch https-proxy-agent dirty because rejectUnauthorized was ignored (see TooTallNate/proxy-agents/issues#89)
diff --git a/packages/axios-extension/package.json b/packages/axios-extension/package.json
index 71411a758..db43a0e2f 100644
--- a/packages/axios-extension/package.json
+++ b/packages/axios-extension/package.json
@@ -1,6 +1,6 @@
 {
     "name": "@sap-ux/axios-extension",
-    "version": "1.16.2",
+    "version": "1.16.2-patched",
     "description": "Extension of the Axios module adding convenience methods to interact with SAP systems especially with OData services.",
     "repository": {
         "type": "git",
@@ -31,8 +31,10 @@
         "axios": "1.6.8",
         "detect-content-type": "1.2.0",
         "fast-xml-parser": "4.4.1",
+        "https-proxy-agent": "7.0.5",
         "lodash": "4.17.21",
         "open": "7.0.3",
+        "proxy-from-env": "1.1.0",
         "qs": "6.11.0",
         "xpath": "0.0.33",
         "@xmldom/xmldom": "0.8.10"
diff --git a/packages/axios-extension/src/factory.ts b/packages/axios-extension/src/factory.ts
index 886fe8aa0..ee8d5edbd 100644
--- a/packages/axios-extension/src/factory.ts
+++ b/packages/axios-extension/src/factory.ts
@@ -1,4 +1,8 @@
+/* eslint-disable jsdoc/require-jsdoc */
 import type { AxiosRequestConfig, InternalAxiosRequestConfig } from 'axios';
+import type { HttpsProxyAgentOptions } from 'https-proxy-agent';
+import { HttpsProxyAgent } from 'https-proxy-agent';
+import { getProxyForUrl } from 'proxy-from-env';
 import cloneDeep from 'lodash/cloneDeep';
 import type { Destination } from '@sap-ux/btp-utils';
 import {
@@ -22,6 +26,19 @@ import { AbapServiceProvider } from './abap';
 import { inspect } from 'util';
 import { TlsPatch } from './base/patchTls';

+// eslint-disable-next-line jsdoc/require-jsdoc
+export class PatchedHttpsProxyAgent<Uri extends string> extends HttpsProxyAgent<Uri> {
+    private readonly extraOptions: any;
+
+    constructor(proxy: Uri | URL, opts?: HttpsProxyAgentOptions<Uri>) {
+        super(proxy, opts);
+        this.extraOptions = opts;
+    }
+
+    async connect(req: any, opts: any) {
+        return super.connect(req, Object.assign({}, this.extraOptions, opts));
+    }
+}
 type Class<T> = new (...args: any[]) => T;

 /**
@@ -36,9 +53,24 @@ function createInstance<T extends ServiceProvider>(
     config: AxiosRequestConfig & Partial<ProviderConfiguration>
 ): T {
     const providerConfig: AxiosRequestConfig & Partial<ProviderConfiguration> = cloneDeep(config);
-    providerConfig.httpsAgent = new HttpsAgent({
-        rejectUnauthorized: !providerConfig.ignoreCertErrors
-    });
+
+    if (config.baseURL) {
+        const proxyUrl = getProxyForUrl(config.baseURL);
+        if (proxyUrl) {
+            // axios doesn't handle proxies correctly
+            providerConfig.httpsAgent = new PatchedHttpsProxyAgent(proxyUrl, {
+                rejectUnauthorized: !providerConfig.ignoreCertErrors
+            });
+            providerConfig.proxy = false; // disable proxy handling of axios
+        }
+    }
+
+    if (!providerConfig.httpsAgent) {
+        providerConfig.httpsAgent = new HttpsAgent({
+            rejectUnauthorized: !providerConfig.ignoreCertErrors
+        });
+    }
+
     delete providerConfig.ignoreCertErrors;
     providerConfig.withCredentials = providerConfig?.auth && Object.keys(providerConfig.auth).length > 0;

Hello @zinserjan ,
Thanks for proposed solution, however the plain overriding is not working for us as we're using @sap/ux-ui5-tooling package and it contains all dependencies (including axios-extension) bundled & minimized into single js file: @sap\ux-ui5-tooling\dist\cli\index.js. Replacing just axios-extension module in this file is not easy as all naming/minimization is bound with all other sub-modules of ux-tooling. Also, open-ux-tools does not provide command to bundle all dependencies into single source.
What package did you use for workaround?
@longieirl Is there a way to create single bundle file with all sub-modules, so it can substitute @sap\ux-ui5-tooling\dist\cli\index.js in node modules?

@tavnabisoft Thankfully, I don't use  @sap/ux-ui5-tooling in this project. Generators didn't work in this case and that's why I just did the setup with the new ui5 cli v4 on my own. Nearly every function of @sap/ux-ui5-tooling can be replaced with a package of this repo. And my setup is anyway pretty custom due to odata2ts and prettier:
  "dependencies": {
    "@odata2ts/converter-v2-to-v4": "^0.4.0",
    "@odata2ts/http-client-jquery": "^0.8.2",
    "@odata2ts/odata-service": "^0.19.3"
  },
  "devDependencies": {
    "@odata2ts/odata2ts": "^0.36.2",
    "@prettier/plugin-xml": "^3.4.1",
    "@sap-ux/backend-proxy-middleware": "^0.8.4",
    "@sap-ux/deploy-tooling": "^0.15.7",
    "@sap-ux/eslint-plugin-fiori-tools": "^0.5.0",
    "@sap-ux/preview-middleware": "^0.16.35",
    "@sap-ux/reload-middleware": "^0.2.0",
    "@sap-ux/ui5-middleware-fe-mockserver": "^2.2.71",
    "@sap-ux/ui5-proxy-middleware": "^1.4.6",
    "@sapui5/types": "~1.120.14",
    "@ui5/builder": "^4.0.1",
    "@ui5/cli": "^4.0.3",
    "eslint": "^8.57.0",
    "prettier": "^3.3.3",
    "prettier-plugin-imports": "^4.2.11",
    "prettier-plugin-packagejson": "^2.5.1",
    "prettier-plugin-properties": "^0.3.0",
    "typescript": "^5.1.6",
    "ui5-tooling-modules": "^3.9.1",
    "ui5-tooling-transpile": "^3.4.7"
  },
  "overrides": {
    "@sap-ux/axios-extension": "file:.packages/sap-ux-axios-extension-1.16.2-patched.tgz"
  }


scrolling does not work

though there are scrollbars, they have no effect

#scrollBy: seems to need negated coordinates
maybe we should spike a window with scroll pane and scroller and test this functionality


audio_constraints doesn't work

I try to pass audio constraints to
    media_stream_constraints={
        "video": False,
        "audio": {
            "sampleRate": 16000,
            "sampleSize": 16,
            "noiseSuppression": True,
            "channelCount": 1,
        },
    },

but sample_rate is still 48000, why
audio_frames = webrtc_ctx.audio_receiver.get_frames(timeout=1)
...
for i, audio_frame in enumerate(audio_frames):
    print(f' frame sample rate {audio_frame.sample_rate}')
# get 48000

"noiseSuppression": True, # works by default, I turn it off later because I think it denoise too much
as for sampleRate, I guess it is because audio track establish with 48k? so I can't change it latter ?

Hi,
Regarding sampleRate, it might be unchangeable.
This article, though it's bit old, says sampleRate is fixed in all browsers.
Thank you very much for the informative figure about the experiment on echoCancellation and noiseSuppression!


Enable local consumption of OUI

Description
This change allows OUI to self-clean while being installed during development when linked to as a dependency.
Also:

Fixed linting of sass files

Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[x] All tests pass

[x] yarn lint
[x] yarn test-unit

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Is there a cmd to run during development to get this script triggered?

Is there a cmd to run during development to get this script triggered?

In the package.json of the project that wants to use this, instead of the version pattern, file:<absolute path> would be used:
"@opensearch-project/oui": "file:/Users/miki/projects/oui"


Missing parts.

I tried to follow with your steps and a few issues. It looks like you missed some steps.
TLDR: The main issue is that when doing everything step by step (fixing missing steps) at the end, when I need to train with "local/chain/tuning/run_tdnn_1j.sh" it says that final.mdl is missing and cannot proceed.

First of all in your description you are missing spk2utt file creation and because of this "utils/validate_data_dir.sh data/train" reports and error about missing file. So you are missing a step of creating utt2spk and later generating spk2utt file.
You do not create feats.scp and other files that are generated with "./steps/makemfcc ..."
Your forgot to create cmvn.scp with "steps/compute_cmvn_stats.sh ..."

Regarding missing files, it is my mistake because when I say, in the Data Creation, "By following this official kaldi guide" I thought the reader did the steps in that guide on his own (like creating all those necessary files). Thanks for pointing it out to me because now I know that I have to be more specific.
As for the main issue, can you post here the logs? Because I remember that the final.mdl file is created during training, so it's strange that it says this file is missing.
if it helps you, I only had one error with the final.mdl file. The problem was that the file was present and therefore had to be removed to continue the training (basically the opposite of what you told me).

The final.mdl is there now as I supplied all the required files. It was just when I followed this step by step errors got lost in a long list of messages. The I read all the messages one by one and fixed all errors and missing files.
But now I get this:
...
...
...
LOG (ivector-extractor-est[5.5.0~1539-72c1]:UpdateProjections():ivector-extractor.cc:1431) Overall objective function improvement for M (mean projections) was 0 per frame over 10166 frames.
ASSERTION_FAILED (ivector-extractor-est[5.5.0~1539-72c1]:UpdateVariances():ivector-extractor.cc:1486) Assertion failed: (var_floor_count > 0.0)

[ Stack-Trace: ]
/opt/kaldi/src/lib/libkaldi-base.so(kaldi::MessageLogger::LogMessage() const+0xa39) [0x7f84141ced3d]
/opt/kaldi/src/lib/libkaldi-base.so(kaldi::KaldiAssertFailure_(char const*, char const*, int, char const*)+0x59) [0x7f84141cf991]
/opt/kaldi/src/lib/libkaldi-ivector.so(kaldi::IvectorExtractorStats::UpdateVariances(kaldi::IvectorExtractorEstimationOptions const&, kaldi::IvectorExtractor*) const+0x449) [0x7f8414433d6d]
/opt/kaldi/src/lib/libkaldi-ivector.so(kaldi::IvectorExtractorStats::Update(kaldi::IvectorExtractorEstimationOptions const&, kaldi::IvectorExtractor*) const+0x1eb) [0x7f84144397b3]
ivector-extractor-est(main+0x76d) [0x403b33]
/usr/lib/libc.so.6(__libc_start_main+0xd5) [0x7f840cb5ab25]
ivector-extractor-est(_start+0x2e) [0x4032ee]

Could this be due to low amount of data? I am using only 85 utterances with only digits.

Yes, for training 85 utterances is very few. For example my lowest number of sentences used was about 10000. Also in the dataset that downloads the recipe by default (http://www.openslr.org/resources/31/, http://www.openslr.org/resources/11/) about 200000 sentences are used.
I don't know how much it is and if there is a minimum.
Try to find a dataset in the language you want and then add your utterances to it (you can also check the link in the guide).
But unfortunately you don't always find a dataset prepared for training with kaldi and you have to organise the files manually.


SVG export Errors after Upgrading vite-plugin-svgr from 3.x to 4.x

Issue with Upgrading vite-plugin-svgr from Version 3.3.0 to 4.2.0: SVG Import Errors
Description of the Issue
After updating vite-plugin-svgr from version 3.3.0 to 4.2.0,
I am encountering an issue with importing SVG icons.
Previously, I exported all my SVG icons from an index.jsx file in my assets folder,
which allowed for easy use with only one import and auto completion in my .tsx files.
Code Example
Here is how icons were exported in version 3.3.0:
// assets/images/index.jsx
export { ReactComponent as IconDownload } from "assets/images/icons/icon_download.svg";
export { ReactComponent as IconEye } from "assets/images/icons/icon_eye.svg";
export { ReactComponent as IconPause } from "assets/images/icons/icon_pause.svg";
export { ReactComponent as IconPlay } from "assets/images/icons/icon_play.svg";

And this is how I imported these icons in my .tsx components:
import { IconDownload, IconEye, IconPause, IconPlay } from "assets/images";

Encountered Problem
After updating to version 4.2.0, I am getting an import error in the index.jsx file, where I only do exports:
Uncaught SyntaxError: ambiguous import: IconDownload index.jsx:2:10
Attempts to Resolve
I have spent approximately a day trying to resolve this issue.
I have reviewed the documentation and searched for similar cases online,
but I haven't found anything that exactly matches my problem.
export MyIcon from "../MyIcon.svg";  // with or without `?react`

Request for Assistance
Could you please guide me towards a solution or provide insights on what might be causing this issue?
I don't want to lost my one line import with auto completion.
Thank you in advance for your help and guidance.

Hello, had a same problem and wasted lots of time searching for possible solutions.
Using:

"vite": "^5.0.0",
"vite-plugin-svgr": "^4.2.0".

What really helped me is removing old index.d.ts for .svg files:
declare module '*.svg' {
  import * as React from 'react'

  export const ReactComponent: React.FunctionComponent<
    React.ComponentProps<'svg'> & { title?: string }
  >
  export default ReactComponent
}

Next step is changing index file with exports from:
export { ReactComponent as Icon } from './icon.svg';

to:
export { default as Icon } from './icon.svg?react';

I believe it's not the best solution, but at least it works for me (imports work fine too).

Not working

Hey, my workaround was to use @svgr/rollup
vite.config.js
import svgr from "@svgr/rollup";
...
export default defineConfig({
...
plugins: [
    svgr({ dimensions: false, svgo: false, typescript: true }),
    ...
  ],
  ...
  });

I let the custom.d.ts
declare module "*.svg" {
  import * as React from "react";

  export const ReactComponent: React.FunctionComponent<
    React.SVGProps<SVGSVGElement> & { title?: string }
  >;

  export default ReactComponent;
}

And my index.tsx as usual
export { ReactComponent as IconDownload } from "assets/images/icons/icon_download.svg";
export { ReactComponent as IconEye } from "assets/images/icons/icon_eye.svg";
export { ReactComponent as IconPause } from "assets/images/icons/icon_pause.svg";
export { ReactComponent as IconPlay } from "assets/images/icons/icon_play.svg";

export IconPlay from "assets/images/icons/icon_play.svg?react";
worked for me. It returns react component

export IconPlay from "assets/images/icons/icon_play.svg?react";
worked for me. It returns react component

It works but if you are using jest and react testing library for testing. It doesn't find the module (file) with the path ends with ?react and the test case fails.

This worked for me in combination with the above mentioned custom.d.ts: https://github.com/pd4d10/vite-plugin-svgr/issues/107#issuecomment-1969337293

Doesn't work for me, even in combination of the above mentioned custom.d.ts changes.
export IconPlay from "assets/images/icons/icon_play.svg?react";

worked for me. It returns react component

It's pretty pathetic that we have to abandon this library just to get working SVGs in Vite ... but it looks like that's the only option: since the update to this package, it's become (literally) unusable :(

Under the hood vite-plugin-svgr use "@svgr/core", "@rollup/pluginutils"
all this come from the react-svgr team
So please use directly their rollup plugin described here: https://react-svgr.com/docs/rollup/
To make no difference between vite-plugin-svgr and @rollup/svgr use it like follow in the vite.config.js:
vite.config.js
import svgr from "@svgr/rollup";
...
export default defineConfig({
...
plugins: [
    svgr({ dimensions: false, svgo: false, typescript: true }),
    ...
  ],
  ...
  });

and like before:
I let the custom.d.ts
declare module "*.svg" {
  import * as React from "react";

  export const ReactComponent: React.FunctionComponent<
    React.SVGProps<SVGSVGElement> & { title?: string }
  >;

  export default ReactComponent;
}

And my index.tsx as usual
export { ReactComponent as IconDownload } from "assets/images/icons/icon_download.svg";
export { ReactComponent as IconEye } from "assets/images/icons/icon_eye.svg";
export { ReactComponent as IconPause } from "assets/images/icons/icon_pause.svg";
export { ReactComponent as IconPlay } from "assets/images/icons/icon_play.svg";


Some short steps on how to use, please

Some short steps with instructions on how to use your exporter would be much appreciated.
Thank you!
Regards,
Dan

Calling with --help gives some hints:
./vsphere_exporter-master --help
Usage of ./vsphere_exporter-master:
-config.file string
vsphere_exporter configuration file name.
-log.level value
Only log messages with the given severity or above. Valid levels: [debug, info, warn, error, fatal, panic].
-web.listen-address string
Address on which to expose metrics and web interface. (default ":9155")
-web.telemetry-path string
Path under which to expose Prometheus metrics. (default "/metrics")


Option to hide resource attributes that are <computed>

Would be nice to hide the <compute> attributes

Tracking in #108.


Login error alert

Description:
Added alert to login view, when invalid login data were provided in form.

Codecov Report

Merging #740 into master will decrease coverage by 0.12%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #740      +/-   ##
==========================================
- Coverage   91.57%   91.44%   -0.13%
==========================================
  Files          90       90
  Lines        2671     2678       +7
  Branches        5        6       +1
==========================================
+ Hits         2446     2449       +3
- Misses        218      222       +4
  Partials        7        7

Impacted Files
Coverage Δ

frontend/src/app/login/login.component.ts
66.66% <50%> (-11.12%)
:arrow_down:

frontend/src/app/auth.service.ts
29.62% <50%> (-1.14%)
:arrow_down:

Continue to review full report at Codecov.

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


Error: ZB_ESP_MAC: MAC RX buffer is full! (TZ-202)

Hello!
When i receive many Zigbee packets from devices at the same time it leads to the RX buffer overflow. Sometimes it even leads to the core panic.
How can i manage buffer sizes on the MAC layer?
ESP32-C6

Hi,
The Rx buffer overflow issue should have been fixed in the latest  esp-zigee-sdk version. Could you please update it and try again?
If the problem still persists, please feel free to contact us with the log information that is helpful for us to fix problem.


[system-probe] Print kernel and platform versions when there are missing kernel functions

What does this PR do?
When system-probe can't run due to kernel version + eBPF support, we were only logging the missing required functions. This PR adds the following two pieces of information to that log message:

kernel version
platform version

Motivation
The existing error message does not provide enough context.
Additional Notes
Before:

After:

With Error:

Without error:


[web][textfield] [touch screen] TextField with a virtual keyboard makes white space at bottom

I made a simple app with TextField and run on iPad:Safari, ChromeOS:Chrome, Windows: Edge.

This is a screenshot on Edge. Chrome is same.
On Safari no white space appered but the TextFIeld disappered under the virtual keyboard.
How should I do ?

code sample
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
        title: 'Flutter Demo',
        theme: ThemeData(primarySwatch: Colors.blue),
        home: MyHomePage(title: 'Flutter Demo Home Page'));
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  final _focus = FocusNode();
  var _resizeToAvoidBottomInset = false;

  @override
  Widget build(BuildContext context) => Scaffold(
      resizeToAvoidBottomInset: _resizeToAvoidBottomInset,
      appBar: AppBar(title: Text(widget.title)),
      backgroundColor: Colors.grey,
      body: Padding(
          padding: const EdgeInsets.all(16),
          child: Stack(children: [
            GestureDetector(
                onTap: () {
                  if (_focus.hasFocus) _focus.unfocus();
                },
                child: SingleChildScrollView(
                    child: Column(
                        mainAxisAlignment: MainAxisAlignment.center,
                        children: [
                      const SizedBox(height: 1000),
                      Container(
                          color: Colors.white,
                          child: TextFormField(
                            focusNode: _focus,
                            decoration: InputDecoration(
                                contentPadding:
                                    const EdgeInsets.symmetric(horizontal: 8)),
                          ))
                    ]))),
            Align(
                alignment: Alignment.topLeft,
                child: MaterialButton(
                    color: Colors.blue,
                    textColor: Colors.white,
                    onPressed: () => setState(() =>
                        _resizeToAvoidBottomInset = !_resizeToAvoidBottomInset),
                    child: Text(
                        'resizeToAvoidBottomInset: $_resizeToAvoidBottomInset')))
          ])));
}

If the TextField's position is not under the virtual keyboard, no white space appear.

Hi @Isonaga-Compass, thanks for filing the issue.
Can you please provide the output of flutter doctor -v?
Thank you

[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5.2 20G95 darwin-x64, locale ja-JP)
• Flutter version 2.2.3 at /Users/manabu/fvm/versions/2.2.3
• Framework revision f4abaa0735 (10 weeks ago), 2021-07-01 12:46:11 -0700
• Engine revision 241c87ad80
• Dart version 2.13.4
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at /Users/manabu/Library/Android/sdk
• Platform android-30, build-tools 30.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.5.1, Build version 12E507
• CocoaPods version 1.10.1
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2020.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
[✓] VS Code (version 1.60.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.26.0
[✓] Connected device (1 available)
• Chrome (web) • chrome • web-javascript • Google Chrome 93.0.4577.63

@Isonaga-Compass, there's a new version of stable out at the moment, can you please upgrade and see if you still experience the issue?
Thank you

No change...
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.5.0, on macOS 11.5.2 20G95 darwin-x64, locale ja-JP)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.60.0)

Hi @Isonaga-Compass, I noticed in your code that you have Container with a white colour wrapped around your TextField.
Am I correct in assuming that you want your TextField to have a white background?

Event I made it Colors.green, no change.
TextField is under the  white space.

@Isonaga-Compass, do you see the whitespace when you set the resizeToAvoidBottomInset to true?

Always shown true or false

@Isonaga-Compass, does the device you're experiencing this issue on have a touch display?

yes.
iPad, Chromebook, Windows tablet.

Labelling as I do not have the device to replicate this issue

not able to reproduce on windows surface laptop

@Isonaga-Compass Can you please provide the details about the tablet model this is happening on?


Resolve symlink of the matlab executable when finding install path

Fixing https://github.com/mathworks/MATLAB-extension-for-vscode/issues/42

Can you contact me to sign a CLA? Thank you for the PR!

The CLA has been signed. Please evaluate the PR. Thank you for the contribution!

Hi @MoetaYuko, I am trying this out but I think this is not behaving as desired. When I try this, I am seeing that line 356 is executed before the fs.realpath callback is executed, resulting in a null return every time.
Is this what you are seeing as well?

Hi @MoetaYuko, I am trying this out but I think this is not behaving as desired. When I try this, I am seeing that line 356 is executed before the fs.realpath callback is executed, resulting in a null return every time.
Is this what you are seeing as well?

Nope, it simply works as expected on my side.

I didn't manually set matlabInstallPath so that the "else" block on line 338 is entered, then the "try" block.
I have a symlink /usr/bin/matlab -> /opt/MATLAB/R2023a/bin/matlab where /usr/bin is in $PATH obviously. Before this PR binPath was assigned /usr/bin, but now it's /opt/MATLAB/R2023a/bin which is expected.

I think this may be working for you somewhat by luck. Since you are using the non-promise version of fs.realpath, I don't believe the await has any effect. If it resolves fast enough, the callback function may be executed before the try is exited, however this is not guaranteed.

I think this may be working for you somewhat by luck. Since you are using the non-promise version of fs.realpath, I don't believe the await has any effect. If it resolves fast enough, the callback function may be executed before the try is exited, however this is not guaranteed.

Indeed. You reminded me that async shouldn't be used. Now I switched to the promise version so it should be working.

Thanks for making that change!
For me, realpath is resolving to /usr/local/MATLAB/R2023b/bin/matlab, which is expected. However, for this to work, I believe that binPath needs to point to /usr/local/MATLAB/R2023b/bin/.

Oops, my bad, this revision must be okay...


Add full version info (with build number) to RELEASE file

Hello,
It turns out that release file at the root of the installation lacks build number info.
Currently we have only Major.Minor.Security, and not +Build, although +Build can be important (https://github.com/AdoptOpenJDK/openjdk-support/issues/163).
For instance, if I want to determine full release version, like 11.0.8+10, I currently have to query windows registry to get that '+10' (in case of windows) or run java -version and parse its output (which is kinda slow if I'm going multiple installations) in case installation was extracted from ZIP, and not MSI.
I suggest adding something like SEMVER = "11.0.8+10" to release file so It will be possible to get all version info from there.
Additionally, it would be great to have something about hotspot/openj9 (and openj9 heap size) in release as well without having to dig through the filesystem (https://github.com/AdoptOpenJDK/openjdk-support/issues/153).

The contents of the release file is almost entirely decided within the OpenJDK being built, this is why jdk8u/11u+ differ greatly in their contents, so adding new values might not be as clean as initially expected.
Currently --with-vendor-version-string / IMPLEMENTOR_VERSION is set to always be AdoptOpenJDK and could be easily changed to contain the full version+build number. However, this parameter only exists for jdk9+ so jdk8u would still need a separate patch.
Solution following this is to add a new section to the build scripts that adds values to the release file, this way https://github.com/AdoptOpenJDK/openjdk-support/issues/153, https://github.com/AdoptOpenJDK/openjdk-support/issues/163, and any future requests could be accommodated. Tagging @sxa on if this is a good/bad idea and for any info of why --with-vendor-version-string is set to a static AdoptOpenJDK.

Closing as https://github.com/AdoptOpenJDK/openjdk-build/pull/2049 has been merged.


workflow code issue

set-output  will be disable in workflow code

I can try to solve this problem.

fix #2824


Add note about programmatically creating categories

Thanks for contributing to Wagtail! 🎉
Before submitting, please review the contributor guidelines https://docs.wagtail.io/en/latest/contributing/index.html and check the following:

Do the tests still pass? (https://docs.wagtail.io/en/latest/contributing/developing.html#testing)
Does the code comply with the style guide? (Run make lint from the Wagtail root)
For Python changes: Have you added tests to cover the new/fixed behaviour?
For front-end changes: Did you test on all of Wagtail’s supported browsers? Please list the exact versions you tested.
For new features: Has the documentation been updated accordingly?

Thanks for the suggestion @PresidentNick! However, I don't think the tutorial is the right place for this - adding categories programmatically is a fairly specialist task that won't come up for the majority of users, and given that the tutorial is already introducing a lot of new concepts, we don't want to overwhelm new users with information that doesn't apply to them.
Working with Wagtail data programmatically is also a big topic to cover in full - a user wanting to do this would need to understand the appropriate Django ORM methods, but depending on what they're trying to achieve, they might also need to know about the distinction between saving and publishing, working with the page tree, management commands, data migrations and many other things. Pointing them to the documentation for just one aspect of this information is liable to leave them more lost than they were before. Perhaps this calls for a new sub-section of the documentation?

Those are all good points I hadn't considered! I've been working on a django management command that loads data into a wagtail instance and I am doing lots of programmatic creation so I could include these notes on a sub-section like you said. What exactly does a subsection look like in the documentation? Do you have a recommendation of where I should put it?

I think it would belong under 'advanced topics' - creating a new .rst page within docs/advanced_topics and adding an entry to index.rst would do it.


spurious failed test in travis

tried to reproduce without success actually..
======================================================================
FAIL: Tests warning for an invalid bulk size.
Traceback (most recent call last):
File "/home/travis/build/savoirfairelinux/influxdb-python/tests/influxdb/influxdb08/helper_test.py", line 171, in testWarnBulkSizeZero
.format(WarnBulkSizeZero))
nose.proxy.AssertionError: 5 != 1 : <class 'tests.influxdb.influxdb08.helper_test.WarnBulkSizeZero'> call should have generated one warning.
-------------------- >> begin captured logging << --------------------
requests.packages.urllib3.connectionpool: INFO: Starting new HTTP connection (1): host
requests.packages.urllib3.connectionpool: INFO: Starting new HTTP connection (2): host
requests.packages.urllib3.connectionpool: INFO: Starting new HTTP connection (3): host
--------------------- >> end captured logging << ---------------------
Ran 172 tests in 41.485s
FAILED (SKIP=28, failures=1)
ERROR: InvocationError: '/home/travis/build/savoirfairelinux/influxdb-python/.tox/pypy3/bin/nosetests -v'

I'm trying to investigate this ..

Looks like this no longer happens.
Also, I have fixed most flaky tests issues in 78e5adad441767361bcc1a85393140a984394f1a


Add support for GetRandomBytes in Key Client

For .NET we'll be slipping this to August. I will try to get it in sooner, though.


Update settings page

please complete by 4/21 Thurs
check in with meredith upon completion

update page to match figma
separate pages for admin and MT
add functionality to change name/phone#/pass & connect to backend

/cib settings-page

Branch 118-settings-page created!


How to select features when needing to perform extra computation on the selected features before feeding into SVM?

The Feature Selection example is good, but how to select features when needing to perform extra computation on the selected features before feeding into SVM?
Here is an example: X are features with a shape [num_example, 1000] and y are labels with a shape [num_example,1]. I want to select 100 features from 1000 features. The selected features can form a new training data X' with a shape [num_example, 100]. I want an SVM model to fit on cos(X'[num_example, 0:50], X'[num_example, 50:100], axis=1) and labels y.
How should I do this?

Hi @imxtx, if I understand correctly, you want to use only the subset of selected features, you can achieve this with different approaches
One is to directly call the predict method or your evolved_estimator, to which you already passed the SVM, so you don't need to fit it again
evolved_estimator.fit(X, y)

# Predict only with the subset of selected features
y_predict_ga = evolved_estimator.predict(X_test)

The second option, if you want to create another model using the selected features, is to use the transform method, which will take the X and return only X' with the subset of selected features
new_X = evolved_estimator.transform(X_test)

You can also inspect which features were selected and which ones weren't with this method
evolved_estimator.support_

I hope it helps

Thank you @rodrigo-arenas ! BTW, I created pull request, can you check it?


[FancyZones Editor] New UX for the FZ editor.

Summary of the Pull Request
This PR introduces a brand new UX for the FZ editor created by @niels9001
New features:

Ability to duplicate layouts
New UX that simplifies the creation and selection of (new) layouts.
Theming support (dark/light and all high contrast themes).
UI can now be fully controlled with tab/arrow keys
Monitor selection now supports keyboard (for selection)
Accessibility improvements.
Other minor enhancements (e.g. scrollable list).

Editor

New layout dialog

Edit template layout

Edit custom layout

Light theme

Quality Checklist

[x] Linked issue: #891 #8160 #7073 #7071 #7048 #6780 #6098 #3672 #3095 #2802 #2588 #2208 #738 #326 #8315 #6867 #8564 #8148
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Amazing work, @SeraphimaZ :)!!!
I'll close #8148.

Amazing work, @SeraphimaZ :)!!!
I'll close #8148.

@niels9001 @SeraphimaZ
you did an amazing work!!! 🥇

@niels9001 @SeraphimaZ
you did an amazing work!!! 🥇

I agree with @enricogior. Thank you for this nice UI.

I agree with @enricogior. Thank you for this nice UI.

@enricogior
Can we do layout based review too or do we only review the functional things to release it with 0.31?

@enricogior
Can we do layout based review too or do we only review the functional things to release it with 0.31?

Looking good! 😁
May I make a few suggestions?
@ Editor: top part - I see you left away the resolution of the monitors. Is that for a specific reason? I believe I've seen that people like to see it.
@ New layout dialog - Top title mentions "type", but immediately below that there is an input for the name. Maybe "New layout" for the title and "Name" and "Type" labels would be a bit better?
@ Edit (template) layout - Maybe a NumberBox for "Number of zones" and "Show space" and "Distance to highlight" would be better. I would mention "px" with the last two aswel. Also we could combine the controls for Show space around zones?

Looking good! 😁
May I make a few suggestions?
@ Editor: top part - I see you left away the resolution of the monitors. Is that for a specific reason? I believe I've seen that people like to see it.
@ New layout dialog - Top title mentions "type", but immediately below that there is an input for the name. Maybe "New layout" for the title and "Name" and "Type" labels would be a bit better?
@ Edit (template) layout - Maybe a NumberBox for "Number of zones" and "Show space" and "Distance to highlight" would be better. I would mention "px" with the last two aswel. Also we could combine the controls for Show space around zones?

Looking good! 😁
May I make a few suggestions?
@ Editor: top part - I see you left away the resolution of the monitors. Is that for a specific reason? I believe I've seen that people like to see it.
This has been moved into a tooltip, so hovering over / on focus would show the tooltip. Let's look at the feedback to see if it needs to be visible at all times.

@ New layout dialog - Top title mentions "type", but immediately below that there is an input for the name. Maybe "New layout" for the title and "Name" and "Type" labels would be a bit better?

Agree. I think the dialog title should just be "Create new layout".

@ Edit (template) layout - Maybe a NumberBox for "Number of zones" and "Show space" and "Distance to highlight" would be better. I would mention "px" with the last two aswel. Also we could combine the controls for Show space around zones?

The edit dialog probably needs some (minor) moving around of controls. Could you open an issue for that so it's track-able :)?

Looking good! 😁
May I make a few suggestions?
@ Editor: top part - I see you left away the resolution of the monitors. Is that for a specific reason? I believe I've seen that people like to see it.
This has been moved into a tooltip, so hovering over / on focus would show the tooltip. Let's look at the feedback to see if it needs to be visible at all times.

@ New layout dialog - Top title mentions "type", but immediately below that there is an input for the name. Maybe "New layout" for the title and "Name" and "Type" labels would be a bit better?

Agree. I think the dialog title should just be "Create new layout".

@ Edit (template) layout - Maybe a NumberBox for "Number of zones" and "Show space" and "Distance to highlight" would be better. I would mention "px" with the last two aswel. Also we could combine the controls for Show space around zones?

The edit dialog probably needs some (minor) moving around of controls. Could you open an issue for that so it's track-able :)?

@enricogior, the installer build got stuck on a heart beat, i'm good to go if you are.  I kicked off another one but it is only 1 minute into the actual compile.

@enricogior, the installer build got stuck on a heart beat, i'm good to go if you are.  I kicked off another one but it is only 1 minute into the actual compile.


RPC/CLI: Allow abandonchannel to be used in regular build

An often requested feature is to use the abandonchannel API in regular
builds and not only dev builds to get rid of stuck channels that had
their funding transaction invalidated.
The initial reason for putting the call behind the build flag was a
safety concern to make sure nobody uses this on active channels by
accident.

Change looks good, only question from my side is whether it would be better to handle this in funding manager and watch for invalidation rather than surfacing a somewhat risky call (even if people do --know_what_im_doing?

I agree, this is already being tracked in https://github.com/lightningnetwork/lnd/issues/386. This serves as a temporary solution until that is implemented, and we should probably revert this change after.


Newsletters: add 94 (2020-04-22)

[x] Action items, news, releases/RCs, and (most) merged PRs @harding
[x] Update releases/RCs on Tuesday afternoon @harding
[x] Client and services update @bitschmidty
[x] Bitcoin Core #17595 @dongcarl

Made most requested edits (thanks for the suggestions!) and a couple extra small edits to @dongcarl and @bitschmidty write-ups (thanks!).  Also add "payjoin" to the style guide (if anyone objects to that addition, either let me know or drop it from this PR yourself and we can discuss it further in a separate PR).

ACK 84f0e761c95464074ef101d24a20f7c1f098dec8. Thanks Dave!
+1 for payjoin in the style guide :)

ACK 84f0e76. Excited for Rust-Lightning coverage ⚡!

ACK 479626ee666fa5ee5f0c8c697e10b0594f2cb615
squashed

Thanks to the authors @harding and @dongcarl this week. Great newsletter guys. Thanks for the review @adamjonas @jnewbery @jonatack ! 🍻


SAMOA-11: Incorrect String comparison in Clustering.java

Fixes SAMOA-11

+1
@abifet could you take a look at it and commit it if it looks ok?

+1
It looks fine for me.

@abifet Could u merge this with master, these PRs have been around even before the move to Apache incubator. Same with PR# 15


Guild.getMember(User) returns null only with JDABuilder().createDefault() or .createLight()

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version.
[x] I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Bug Report
The getMember(User) method returns null when JDABuilder is created with the Default or Light Caching settings and even after setting the MemberCachePolicy.
Expected Behavior
When using the now deprecated new JDABuilder(); to create my JDA objects, Guild.getMember(User) works fine and returns the Member object of the related user.
Code Example or Reproduction Steps
JDABuilder builder = JDABuilder.createDefault(api_key); builder.setMemberCachePolicy(MemberCachePolicy.ONLINE);
Code for JDABuilder or DefaultShardManagerBuilder Used
s.a.
Exception or Error
Object is null which causes a NPE later on.

This is intended behavior. As the docs mention, MemberCachePolicy.ONLINE requires GatewayIntent.GUILD_PRESENCES to function. So you have to do builder.enableIntents(GatewayIntent.GUILD_PRESENCES). Note that MemberCachePolicy.ONLINE will not be supported in future versions due to breaking changes in API v8. In the future, you have to use guild.retrieveMembersByIds(...) to load presences at runtime.


Out of date

The options and screenshots in this documentation are out of date and no longer reflect the options available in the drop downs for Problem Type and Sub types.

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

ID: 2c3ad66f-14bd-bd67-40d2-9b6f29cd633c
Version Independent ID: 922adbe4-fefb-e895-d7dd-42279d24ff66
Content: Region or SKU series unavailable - Azure
Content Source: articles/azure-portal/supportability/sku-series-unavailable.md
Service: azure-supportability
GitHub Login: @stevendotwang
Microsoft Alias: xingwan

Hello @GarethOates  - really appreciate the suggestion. We will review with the content owner to see how best to improve and update the document based on your feedback.
We will now close this issue. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.
@stevendotwang - FYI

Update: @stevendotwang working internally and would update the page with appropriate content. ETA - 8/15.

Update: This is the support document that is current and maintained by our support team. Whoever have issues, please follow it. We will work on our end to update our process to point customers to this support doc instead.


Getting crash block invoke

Here is the details of crash
0x100e46d38 __30-[SCSocket WSConnectedHandler]_block_invoke (SCSocket.m:327)
3  SocketCluster_ios_client       0x100e4741c -[SCSocket WSMessageHandler:] (SCSocket.m:422)
4  SocketRocket                   0x100e62f48 __30-[SRWebSocket _handleMessage:]_block_invoke (SRWebSocket.m:839)
Line number can change by 2 or 3 numbers.

Can you reproduce this crash?
It's caused by data sent from server.
422 line is   id responseData =[dictionary objectForKey:@"data"] ;
I think that dictionary in null

Yes i reproduced the crash, actually it was due to the lost reference of the SocketClusterDelegate when any event is received. And also i was able to fix it by changing the delegate type to WEAK.
@property (weak, nonatomic,nullable) id delegate.


IE11 Support

Does the IE11 support mentioned at What You Need to Get Started apply to this repo?
<!DOCTYPE html>

    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Contentstack IE11</title>

hi there!

errors in IE11 with:
SCRIPT5007: Unable to set property 'Headers' of undefined or null reference
contentstack-v3.11.0.min.js (9,6715)

This error does not appear in v3.10.1.  Is this change between v3.10.1 and v3.11.0 meant to be node-fetch for the client-side?
Changing to isomorphic-fetch/whatwg-fetch appears to make it pull in the correct client-side fetch polyfill.

Thanks @mbeck-sans for raising issue, We will check this on our side, and update you once we resolve issue.


contrib: Remove unnecessary Windows restarts

During minion init, a script is created which will run at startup
and Windows gets rebooted.
Since ovnkube runs as a service, this is not required anymore.
This commit removes the Windows reboot and starts directly the
ovn-kubernetes-node service which will run minion init. The restart
after docker installation is also removed since the path is
already updated.
Signed-off-by: Alin-Gheorghe Balutoiu<EMAIL_ADDRESS>Signed-off-by: Alin Gabriel Serdean<EMAIL_ADDRESS>Co-authored-by: Alin Gabriel Serdean<EMAIL_ADDRESS>

Closed in favor of #450


Catkin_make fail with absl and hash problem

catkin_make failed with sophus, fmt, absl problem, need to check the version of them. THANKS!

The error seems form absl? Any idea to fix that?

I'm using 20220623, which is the one installed via mamba install abseil-cpp

I'm using 20220623, which is the one installed via mamba install abseil-cpp
You can also look here https://github.com/versatran01/dsol/blob/main/.github/workflows/build.yaml
I also just pushed a fix to make it compile with fmt v9.

Thanks for updating. I still got same problem(same warning), after I followed the command in build.yaml, and rebuild it.
Any idea why this error happens? Always stuck with summary.cpp, which uses flat_hash_map.h.

I have no idea why this happens based on your screenshot.

Problem solved. If the error shows "undefined reference to absl::lts_xxxxxxx", try rm all absl stuff and re cmake with"-DABSL_PROPAGATE_CXX_STD", it should works.


Submit the package to Stackage

Hi! Since your package has the potential to improve code quality so much, etc., etc., I'm wondering if you considered submitting it to
https://github.com/fpco/stackage
I'd do it myself, but I think they require maintainers to do that. Thanks!

I see it's done. Thank you!


Adding blobmoji support

Despite my best attempt, I couldn't get the preferences to properly split into 4 instead of 3, as the proportions of the 4th button look off.
Also, I couldn't get the submodule added properly. Here's the git of blobmoji - I based the implementation off of your noto, as blobmoji is itself a fork of Google's noto.
https://github.com/C1710/blobmoji

Blobmoji is in the latest version, but it's no longer in python and gtk.


GUI rework

Hi, Alexofp!
I'm Godot dev, and I'm fully changed GUI handle for game. Check whats in your satisfaction)
Some warning: my changes can barly touch non-UI related parts of the game.
Used Godot 3.5.
And, yea... Sorry for all-modding files in github. I'm just work on this project without git before trying make a commit.
CHANGELOG:

Main Menu now works as in-game scene, as GUI node reworked to handle ALL project menus by system of tab containers.

GUI scene now is a project’s main scene on start. All functions from old UI was moved/remaked to work with tab containers and whole game.

Project now runs on last 3.5 Godot version.

GUI :

Almost in all menus their elements get some “space” between each other or readjusted in align/position.

Text in menus get polished too – in some centered, in some get bolded for more attention for reader.

Project use Global font (res://UI/FontResources/GlobalDefaultFont.tres) by default (not from GlobalTheme in root node as before).

RichText’s don’t align their text to their edge now.

Save/Load menus now handle showing saves a bit effective.

As bugfix because all menus works at once: Stage3D’s from Character Creation and from Player Panel now in different places in 3D-word by x coordinate.

FIX: Option pages button now work properly. (Before they compute pages from buttons count, instead from their max index.)

FIX: fixed unnecessary queue_free() func's call in Output text for added UItextbox and customUItextbox’s, what causes freezes after freeing complex customUItextbox.

And many other small tweaks, what I may forgot…
Code tip:
Don’t use if (something != null): sentences, use instead is_instance_valid() function for objects, and checking with empty analog for non-objects (like strings or arrays). For another Godot coder it’s pretty handy to know with what type of data he works in your code.
Here is why it’s unsafe for release builds too: from 3.3 Godot changelog.
Some requests?..:

Fully remember state of prev. scene will be pretty handy and makes possible remove In-game menu and pass saving functionality to MainMenu. Maybe it isn’t needed, but I’m too afraid for shadowly losing game data by just loading MainMenu scene and then reload previous.

Move to 3.5 Godot if you not! Changing custom_varibles in Control’s from Editor is great (and other new features cool too, but I better appreciate this one)! Project from 3.4.4 will be work on 3.5.

Annnnnnnnd... Thanks for cool game, Dev!)
I think you hear from other players what's character visualization is a super-cool feature for rpg games like TITS or Lilith Throne) But, man... God, looks like you read my mind!..
And runs on Godot with I develop my own game too! Hm... Let's peek a bit what's inside...
Wow.
I. Need. Rework Main Menu. QUICKLY!..
Well... That's how it started)
P.S. Sorry for my English - I'm not a English-native speaker.

Yeah, again. Thanks for trying to improve the code. I'm sorry if I sounded a bit mean but please try to make smaller changes. One thing at a time, you know. One bugfix, one feature. Avoid major reworks please, at least for now.
Should also look into how to work with git, you should create a separate branch for each pull request instead of using your main branch ^^. I should have a dev branch too but I'm lazy. I will close this as the idea of changing the main menu into a game scene is very flawed, sorry.

I understand it too. It's a bit too hard handle one-big rework for whole game aspect)
But think about it - If Main Menu was a game scene too, it has a potential to allow moders make whole new game on yours!) This sounds at least super-cool)
At least I learn many things about coop-develop and using GitHub) ...And how to make game with working mod support in Godot! I think about it, but doesn't know from where and how I can start)
Hm... Maybe you leave some warning in README for future contributes about small changes? Just in case for silent big-makers like me and let other know what you open for little help.
Well... I have some ideas to add to game, not so big, but UI related. And maybe I can pass some parts from this rework too - just cosmetic changes in scenes what I've done, lightly touches code.

Mods do that already, they can override any file, including the starting scene ^^. Menu should stay a separate object, trust me. Yeah, sorry, I do indeed need some info about the code style and how to contribute. People can contribute big things but its better to start with something easy, especially if its your fist time using github ^^.
Cosmetic changes are welcomed, make sure to attach a screenshot of how they look when submitting the pull request.
Nah, there is no place for mods yet. The modding update is not even out yet, I'm busy writing some content and tutorials for it


Bug Fixes and DG Changes

Codecov Report
Base: 72.53% // Head: 72.53% // No change to project coverage :thumbsup:

Coverage data is based on head (7d8c798) compared to base (a640aa6).
Patch coverage: 50.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #224   +/-   ##
=========================================
  Coverage     72.53%   72.53%
  Complexity      794      794
=========================================
  Files           117      117
  Lines          2370     2370
  Branches        301      301
=========================================
  Hits           1719     1719
  Misses          565      565
  Partials         86       86

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/ui/PersonViewPanel.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/CommandHistory.java
85.00% <100.00%> (ø)

.../java/seedu/address/storage/JsonAdaptedPerson.java
75.00% <0.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Strip extra newline from the production config

With e09a56d we are getting a failing spec because of an extra newline
in the production config. Removing comments later will take care of the
extra newlines, and I think may have made e09a56d unnecessary.

Huh.
Yeah sure LGTM.


Unable to update mission

When this extension is enabled in 2.7.4, I am unable to save any updates to the Mission via the Edit Mission page.

v1.2.0 released with support for Nova 2.7.5+ and PHP 8.


Publish to Maven Central before the docs

Made it a minor, so we can trigger another minor release for the Maven Central artifacts.

/canary


Excessive Storage Utilisation - Tiles are cached at duplicate locations, never get pruned

Spawned from https://github.com/streetcomplete/StreetComplete/issues/3417 .
In a nutshell
It looks like the map tiles are cached both at the location the user of this library specified and at another location at the same time. The issues are:

that the tiles get cached at that other location at all, this duplicates the cache space requirement
that the tiles in that other location seemingly never get pruned

My cache configuration
// = sdcard/Android/data/de.westnordost.streetcomplete/cache/tile_cache
val cacheDir = File(context.externalCacheDir, "tile_cache")

val cache = Cache(cacheDir , 50 * 1000L * 1000L) // prune after reaching 50 MB

val cacheControl = CacheControl.Builder()
    .maxAge(12, TimeUnit.HOURS) // do not re-download tiles younger than 12 hours
    .maxStale(14, TimeUnit.DAYS) // prune tiles older than 14 days
    .build()

val httpHandler = object : DefaultHttpHandler(OkHttpClient.Builder().cache(cache)) {
    override fun configureRequest(url: HttpUrl, builder: Request.Builder) {
        builder.cacheControl(cacheControl)
    }
}

tangramMapView.initMap(httpHandler )

The OkHttpCache actually prunes tiles in sdcard/Android/data/de.westnordost.streetcomplete/cache/tile_cache correctly when this directory exceeds a size of 50MB. I tested this.
What is wrong? Observations

Reported cache size:

Actual size of external cache directory:

There are no other cache files in neither of

/sdcard/Android/data/de.westnordost.streetcomplete/cache/
/data/data/de.westnordost.streetcomplete/cache/

When panning the map and thus downloading new tiles, the reported cache size grows at pretty much at 2x the rate as the size of the external cache directory

When comparing the total free storage space on the phone before and after clearing the cache for the app, it is clear that the reported cache size by Android is not a display error: In this case, indeed 57 MB storage space has been freed.
However, the size of the directories mentioned in point 3 were reduced by only 22MB.

Thus, I conclude that there must be a third directory somewhere where the tiles are cached and that is never pruned that is outside of the directories mentioned in point 3 and used by tangram-es to store duplicates of the downloaded tiles.
Android is able to attribute this directory to the app and able to clear it but I have not found where on the sdcard it is supposed to be.
Used versions

tangram-es 0.16.2
Android (different versions, different devices)

Good information to know! This makes me think that it would be possible to implement a more space-efficient cache by combining tile responses into a single file - like a sqlite database. I don't expect to do this in Tangram ES but it might be possible for a client application.

Hi there, yes file system is not adapted to tile storing, you should use SQLite based on the MBTiles specification

@Joxit indeed MBTiles is what I had in mind when I mentioned an sqlite database. It's a tempting idea, but I think it would be pretty complicated for a couple reasons:

In addition to storing tile data, we also need to respect cache control headers in the client request and in the server response. Currently this is handled by OkHTTP. Implementing this again would be non-trivial work and would probably not be as good as OkHTTP's implementation!
Tangram ES doesn't just request tiles, it can also request scene files, scene bundles, or image resources over HTTP. These other resources don't fit in the MBTiles schema, so we would need multiple caches for different types of resources.

This doesn't mean it can't be done! Tangram ES allows a client application to replace the entire HTTP implementation, so you could certainly create a tile cache based on MBTiles if you wanted to.

Hmm...

To create a tile cache based on MBTiles just so to store the cache more efficiently seems something no user of a map render library would do. So it seems this is something that should be done by the render library.
However, for web-maps (tangram, maplibre-gl-js), the render library obviously does not manage the cache by itself but the browser does that. So why should the library on native manage the cache itself and on the web, not?

I am not sure if browsers put all the cached files in some kind of giant indexed table so they don't have this problem. I'd also be curious how MapLibre handles this.

Seems like the ideal setup would be an independent library which handles this. That way it could be shared between renderers, and only included when needed.

Last I remember we did add some code to do offline Rendering via mbtiles. If needed we should be able to use that tile source class generically for all tile source cache?

Yep, there is code in Tangram ES currently for caching vector tile sources in a local mbtiles database. However in its current form it isn't a substitute for HTTP caching. The existing mbtiles cache will naively store all tiles forever, regardless of the cache-control headers that accompanied the tile response.
I looked into maplibre to see what approach is used there. It does seem to use a local sqlite database for tile caching, complete with parsing and logic for response headers. The maplibre caching code is in the shared native library, for reuse across platforms.
A similar approach would make sense for Tangram ES. Tile caching could be implemented in the shared core library using sqlite, with options like the maximum cache size exposed to the client SDKs. I would support this approach, I think it could lead to a better user experience for Tangram ES. But the scope of this work is beyond my current availability (for the time being, at least).
It would be nice if the caching code from maplibre could be shared in an independent library, but unfortunately it seems very tightly coupled with the internals of their renderer.


issue with "sadd" within event listener

I have simple code :
var events = require('events'),util = require('util');
	var testingredis = function(){
		var redis = require("redis"), client = redis.createClient();
		var self = this;
		self.on('new_surname', function() {
			if(client.sadd('surname', 'smith')===true){
					console.log('surname added');
				}else{
					console.log("surname exists");
			}
		});
		setInterval(function(){
			self.emit('new_surname');
		},1000);
	}

testingredis.prototype = new events.EventEmitter;
new testingredis();

but "client.sadd('surname', 'smith')" is returning TRUE on every event call. I always see "surname added" message, but it should be showed only once. Is it nodejs or redis issue?

It is neither a Node.js, nor a Redis issue. Please read the documentation and also read about async functions in Javascript. This is fundamental for programming with JS.