For Group ... Group By ... not supported

Hi team,
I have tried SELECT it was not supported (I just see another Issue for it) and my second chance was to try the Group statement.
It works on Eclipse on 7.51 but not in Exercism
    aggregated_data = VALUE #(
                        FOR GROUPS grouping_group OF initial_line IN initial_numbers
                        GROUP BY ( group = initial_line-group )
                        ( group = grouping_group-group
                          count = ''
                          sum   = REDUCE #(
                                    INIT subtotal = 0
                                    FOR group_line IN GROUP grouping_group
                                    NEXT subtotal = subtotal + group_line-number )
                        ) ).

Hello Team,
Even I am facing the same issue.
Thanks

@GirodFred can you help pasting a code snippet that passes the syntax check?

can you help pasting a code snippet that passes the syntax check?

Sorry @larshp, I don't understand, you need the code in eclipse or in Exercism ?

just something that can be used for reproducing the issue

just something that can be used for reproducing the issue
also, do you get a parser error, or the wrong result when the code is run on Exercism?

You can use below code to check
 aggregated_data = VALUE aggregated_data( BASE aggregated_data
                                            FOR GROUPS ls_group OF ls_numgrp IN initial_numbers
                                              GROUP BY ( group = ls_numgrp-group  count = GROUP SIZE )
                                                   ASCENDING
                                            ( REDUCE aggregated_data_type(
                                              INIT ls_agr_data = VALUE aggregated_data_type( )
                                               FOR ls_grp_data IN GROUP ls_group
                                              NEXT ls_agr_data = VALUE #(
                                                                      group = ls_group-group
                                                                      count = ls_group-count
                                                                      sum = REDUCE i( INIT lv_sum = 0
                                                                                FOR lv_cal_sum IN GROUP ls_group
                                                                                NEXT lv_sum = lv_sum + lv_cal_sum-number )
                                                                      max = REDUCE i( INIT lv_max = 0
                                                                                FOR lv_max_num IN GROUP ls_group
                                                                                NEXT lv_max = nmax( val1 = lv_max
                                                                                val2 = lv_max_num-number ) )
                                                                      min = REDUCE i( INIT lv_min = 101
                                                                                FOR lv_min_num IN GROUP ls_group
                                                                                NEXT lv_min = nmin( val1 = lv_min
                                                                                val2 = lv_min_num-number ) )
                                                                      average = ls_agr_data-sum / ls_agr_data-count
                                                                     )

                                           ) ) ).

@bpawanchand the code does not pass a syntax check,

the whole local test code is :

CLASS zcl_itab_aggregation DEFINITION
*  PUBLIC
  FINAL
  CREATE PUBLIC .

  PUBLIC SECTION.
    TYPES group TYPE c LENGTH 1.
    TYPES: BEGIN OF initial_numbers_type,
             group  TYPE group,
             number TYPE i,
           END OF initial_numbers_type,
           initial_numbers TYPE STANDARD TABLE OF initial_numbers_type WITH EMPTY KEY.

    TYPES: BEGIN OF aggregated_data_type,
             group   TYPE group,
             count   TYPE i,
             sum     TYPE i,
             min     TYPE i,
             max     TYPE i,
             average TYPE f,
           END OF aggregated_data_type,
           aggregated_data TYPE STANDARD TABLE OF aggregated_data_type WITH EMPTY KEY.

    METHODS perform_aggregation
      IMPORTING
        initial_numbers        TYPE initial_numbers
      RETURNING
        VALUE(aggregated_data) TYPE aggregated_data.
  PROTECTED SECTION.
  PRIVATE SECTION.

ENDCLASS.

CLASS zcl_itab_aggregation IMPLEMENTATION.
  METHOD perform_aggregation.
    " add solution here
    "    select group, count( group ) as count, sum( number ) as sum,
    "           min( number ) as number, max( number ) as max,
    "           average( number ) as average
    "           into corresponding fields of table @aggregated_data
    "          from @initial_numbers
    "           group by group
    "           ##db_feature_mode[itabs_in_from_clause].
    aggregated_data = VALUE #(
                        FOR GROUPS grouping_group OF initial_line IN initial_numbers
                        GROUP BY ( group = initial_line-group )
                        ( group = grouping_group-group
                          count = ''
                          sum   = REDUCE #(
                                    INIT subtotal = 0
                                    FOR group_line IN GROUP grouping_group
                                    NEXT subtotal = subtotal + group_line-number )
                        ) ).

  ENDMETHOD.

ENDCLASS.

@larshp  this was the code snippet which  I wanted to  used in the ITAB aggregation exercise where the whole class is defined with all the needed types and data objects.

The full code is here
CLASS zcl_itab_aggregation DEFINITION
  PUBLIC
  FINAL
  CREATE PUBLIC .

  PUBLIC SECTION.
    TYPES group TYPE c LENGTH 1.
    TYPES: BEGIN OF initial_numbers_type,
             group  TYPE group,
             number TYPE i,
           END OF initial_numbers_type,
           initial_numbers TYPE STANDARD TABLE OF initial_numbers_type WITH EMPTY KEY.

    TYPES: BEGIN OF aggregated_data_type,
             group   TYPE group,
             count   TYPE i,
             sum     TYPE i,
             min     TYPE i,
             max     TYPE i,
             average TYPE f,
           END OF aggregated_data_type,
           aggregated_data TYPE STANDARD TABLE OF aggregated_data_type WITH EMPTY KEY.

    METHODS perform_aggregation
      IMPORTING
        initial_numbers        TYPE initial_numbers
      RETURNING
        VALUE(aggregated_data) TYPE aggregated_data.
  PROTECTED SECTION.
  PRIVATE SECTION.

ENDCLASS.

CLASS zcl_itab_aggregation IMPLEMENTATION.
  METHOD perform_aggregation.
  aggregated_data = VALUE aggregated_data( BASE aggregated_data
                                            FOR GROUPS ls_group OF ls_numgrp IN initial_numbers
                                              GROUP BY ( group = ls_numgrp-group  count = GROUP SIZE )
                                                   ASCENDING
                                            ( REDUCE aggregated_data_type(
                                              INIT ls_agr_data = VALUE aggregated_data_type( )
                                               FOR ls_grp_data IN GROUP ls_group
                                              NEXT ls_agr_data = VALUE #(
                                                                      group = ls_group-group
                                                                      count = ls_group-count
                                                                      sum = REDUCE i( INIT lv_sum = 0
                                                                                FOR lv_cal_sum IN GROUP ls_group
                                                                                NEXT lv_sum = lv_sum + lv_cal_sum-number )
                                                                      max = REDUCE i( INIT lv_max = 0
                                                                                FOR lv_max_num IN GROUP ls_group
                                                                                NEXT lv_max = nmax( val1 = lv_max
                                                                                val2 = lv_max_num-number ) )
                                                                      min = REDUCE i( INIT lv_min = 101
                                                                                FOR lv_min_num IN GROUP ls_group
                                                                                NEXT lv_min = nmin( val1 = lv_min
                                                                                val2 = lv_min_num-number ) )
                                                                      average = ls_agr_data-sum / ls_agr_data-count
                                                                     )

                                           ) ) ).

  ENDMETHOD.

ENDCLASS.

@larshp You can check the below code  by creating a executable program
CLASS zcl_itab_aggregation DEFINITION.

  PUBLIC SECTION.
    TYPES group TYPE c LENGTH 1.
    TYPES: BEGIN OF initial_numbers_type,
             group  TYPE group,
             number TYPE i,
           END OF initial_numbers_type,
           initial_numbers TYPE STANDARD TABLE OF initial_numbers_type WITH EMPTY KEY.

    TYPES: BEGIN OF aggregated_data_type,
             group   TYPE group,
             count   TYPE i,
             sum     TYPE i,
             min     TYPE i,
             max     TYPE i,
             average TYPE f,
           END OF aggregated_data_type,
           aggregated_data TYPE STANDARD TABLE OF aggregated_data_type WITH EMPTY KEY.

    METHODS perform_aggregation
      IMPORTING
        initial_numbers        TYPE initial_numbers
      RETURNING
        VALUE(aggregated_data) TYPE aggregated_data.
  PROTECTED SECTION.
  PRIVATE SECTION.

ENDCLASS.

CLASS zcl_itab_aggregation IMPLEMENTATION.
  METHOD perform_aggregation.
    aggregated_data =  VALUE aggregated_data( BASE aggregated_data
                                              FOR GROUPS ls_group OF ls_numgrp IN initial_numbers
                                                  GROUP BY ( group = ls_numgrp-group  count = GROUP SIZE )
                                                   ASCENDING
                                            ( REDUCE aggregated_data_type(
                                              INIT ls_agr_data = VALUE aggregated_data_type( )
                                               FOR ls_grp_data IN GROUP ls_group
                                              NEXT ls_agr_data = VALUE #(
                                                                      group = ls_group-group
                                                                      count = ls_group-count
                                                                      sum = REDUCE i( INIT lv_sum = 0
                                                                                FOR lv_cal_sum IN GROUP ls_group
                                                                                NEXT lv_sum = lv_sum + lv_cal_sum-number )
                                                                      max = REDUCE i( INIT lv_max = 0
                                                                                FOR lv_max_num IN GROUP ls_group
                                                                                NEXT lv_max = nmax( val1 = lv_max
                                                                                val2 = lv_max_num-number ) )
                                                                      min = REDUCE i( INIT lv_min = 101
                                                                                FOR lv_min_num IN GROUP ls_group
                                                                                NEXT lv_min = nmin( val1 = lv_min
                                                                                val2 = lv_min_num-number ) )
                                                                      average = ls_agr_data-sum / ls_agr_data-count
                                                                     )

                                           ) ) ).

  ENDMETHOD.

ENDCLASS.

START-OF-SELECTION.

  DATA(lo_agr) = NEW zcl_itab_aggregation( ).
  DATA(lt_agr) = VALUE zcl_itab_aggregation=>initial_numbers( ( group = 'A' number = 10  )
*                                                              ( group = 'B' number = 5   )
                                                              ( group = 'A' number = -10   )
*                                                              ( group = 'C' number = 22  )
*                                                              ( group = 'A' number = 13  )
*                                                              ( group = 'C' number = 500 )
                                                             ).

  DATA(lt_agr_data) = lo_agr->perform_aggregation( initial_numbers = lt_agr ).
  BREAK-POINT.

wip, https://github.com/abaplint/abaplint/pull/2686

@GirodFred it looks like your solution does not work?

@bpawanchand try again,

@GirodFred it looks like your solution does not work?

My solution wasn't finish, as the first one (with SELECT) was not reconized by the system, I don't want to write a whole code if there is another error.
And I have another problem today (for another Exercism) ;)

@bpawanchand try again,

@larshp  it worked. All the test cases passed. Thanks

closing, feel free to open a new issue if(when) you find more bugs


Time trigger function suddenly stops firing

Issue Transfer
This issue has been transferred from the Azure SDK for .NET repository, #26710.
Please be aware that @tkachuk2222 is the author of the original issue and include them for any questions or replies.
Details
Library name and version
Microsoft.Azure.Functions.Worker 1.5.1
Describe the bug
I have an azure function (hosted on ASP P1v3) that has near 20 time trigger functions and near 20 service bus functions. From time to time timer trigger functions stops firing.
What can I do?
Option 'Always on' is set to 'On'
Expected behavior
Timer trigger azure functions firing
Actual behavior
Timer trigger azure functions not firing
Reproduction Steps
I don't know how to reproduce. I just have azure functions with service bug trigger, timer trigger and http trigger
there are next schedules:
0 */15 * * * *
0 1 * * * *
0 */15 * * * *
0 */5 * * * *
0 */8 * * * *
0 */6 * * * *
0 0 * * * *
0 */1 * * * *
*/1 * * * *
*/1 * * * *
0 10 0 * * *
0 1 0 * * *
0 0 */2 * * *
0 0 6 * * *
0/40 * * * * *
0 */1 * * * *
0 0 1 * * *
0 */5 * * * *
0/5 * * * *
0 0 2 * * *
Environment
App Service Plan P1v3: 1
OS Windows
Azure function runtime version: 3
.NET 5.0
Worker runtime: dotnet-isolated
Microsoft.Azure.Functions.Worker.Extensions.Time 4.1.0
Microsoft.Azure.Functions.Worker.Sdk 1.0.4
Microsoft.Azure.Functions.Worker 1.5.1

This issue may be related to things like the ScaleController or Functions host -- it's highly recommended that issues like this are resolved via a support ticket as we'll need more in-depth information about timelines and resources.


[devops:bot] update deps in nix/sources.json (from: rhino-core@9980f93

update nix refernces for rhino-core and|or nixpkgs to match https://github.com/rhinofi/rhino-core/blob/9980f93f426c0f25e29a6351c47ad3f549a8c710/nix/sources.json (branch: dev)

superseded by #102


The flops and parameters of NASFPN is not matched

Hi, table.1  in [1] show that  NAS-FPN R-50 (7 @ 256)  FLops and parameters is  281.3B , 60.3 M.  However, I use get_flops.py to get 140.42 GF and 68.02M when using NasFPN config.
Refer:
[1] https://arxiv.org/pdf/1904.07392.pdf

@RangiLyu  Thanks for your reply. But how to explain the difference in parameters?


Implement bulk write of activations in the Invoker.

Writing activations in the Invoker is not time critical and can be greatly optimized by batching the activations to reduce the number of connections used towards the database. That enables a lot of optimizations on the database side and is in general a good practice.
Signed-off-by: Christian Bickel<EMAIL_ADDRESS>

If use original deal method, with mutliple action request, the invoker's controller<n>_logs.log exist below error
[PUT] 'lambda-exp_activations' internal error, failure: 'DB request queue is full.' [marker:database_saveDocument_error:12941:22]

So i am very expected that this pr can be merged to improve the invoker/couchdb performance.

Should consider applying to sequence activations written in the controller as well.

We did indeed. The first version of this PR included a latency penalty in the batching, which is why we started implementation in the invoker, where the penalty doesn't matter as much.
We're currently investigating an even deeper integration in the CouchDB client itself. If that doesn't work out, we'd "just" bring the put based implementation.
Want to wait for that and put it in this PR or are you fine with a second one improving on this?

PG3 1156 🔵


Fix typo in withRouter

I noticed that the HoC was renamed from withRoute to withRouter some time ago, but the displayName was not updated accordingly. This PR fixes this issue.

Thanks @bjrnt
Looks good to me.


Can you tell me what should be done?

Can you tell me what should be done?

Just open the executable.


Easier execution via docker image

Hello,
I am trying to implement pre-commit-terraform tflint fmt and checkov but I am facing issues when trying to execute it when running everything on docker image.
To be a bit more specific I just build image with dependencies and created .git/hooks/pre-push script manually:
# Dockerfile
(...)
WORKDIR /pre-commit
CMD ["pre-commit", "run", "--hook-stage", "push"]

# .git/hooks/pre-push
(...)
docker run -t -v "$(pwd)":/pre-commit --name "$NAME" my-image:latest

This is how my .pre-commit-config.yamls looks like:
# .pre-commit-config.yaml
  repos:
  - repo: https://github.com/antonbabenko/pre-commit-terraform
    rev: v1.70.0
    hooks:
      - id: terraform_tflint
        stages:
          - push
      - id: terraform_checkov
        stages:
          - push

This surprisingly works very well when trying same solution for commit stage, but it sees no diff when its set push and its getting always "skipped". I saw there is a dedicated docker image but if I am correct there is no easy way for implementation in pre-commit-config.yaml to do the same but via docker image to get rid off dependencies.
How could pre-commit-terraform help solve your problem?
It would be great to have possibility to have hooks preconfigured with docker image and just run following without worrying about dependencies:
#.pre-commit-config.yamls
  repos:
  - repo: https://github.com/antonbabenko/pre-commit-terraform
    rev: v1.70.0
    hooks:
      - id: terraform_tflint_docker
      - id: terraform_checkov_docker

but it sees no diff when its set push and its getting always "skipped".

That is expected, because there is no git diff on pre-push stage. And next is redundant:
        stages:
          - push

https://pre-commit.com/#hooks-stages
You need manually detect differ files (GHA example) if you'd like to run pre-commit only on that files, otherwise, use pre-commit run -a

It would be great to have possibility to have hooks preconfigured with docker image and just run following without worrying about dependencies:

That already can be done via
TAG=latest
docker run -v $(pwd):/lint -w /lint ghcr.io/antonbabenko/pre-commit-terraform:$TAG run

or if you need specific versions use --build-arg's as specified in 1. Install dependencies -> Docker
Also, it should not be a problem to create a shell or git alias to run the needed docker run command.
Running all that stuff as many separate containers will slow down hook executions compared to a usage of OS-native env, different teams prefer to use different versions of each tool (and sometimes, their dependencies) so to do what you ask good (and w/o big maintenance effort) will need some time.
Anyway, glad to review your PR that will implement that.
P.S. That may resolve #397 as not needed, if anyone with Mac will switch to docker

#.pre-commit-config.yamls
repos:

repo: https://github.com/antonbabenko/pre-commit-terraform
rev: v1.70.0
hooks:

id: terraform_tflint_docker
id: terraform_checkov_docker

I would also love to have the above functionality and not have to install any dependencies on my CI/CD pipeline.
Below, there is a list of repos that already offer pre-commit hooks using Docker:

https://github.com/igorshubovych/markdownlint-cli/blob/master/.pre-commit-hooks.yaml#L15
https://github.com/igorshubovych/markdownlint-cli/blob/master/.pre-commit-hooks.yaml#L22
https://github.com/zricethezav/gitleaks/blob/master/.pre-commit-hooks.yaml#L7

Note: TF_PLUGIN_CACHE_DIR should be exported and set inside most hooks. That may be related to other envs too.
Also, in case of TF_PLUGIN_CACHE_DIR usage, terraform init will need to run inside the container OR mount to the same path that is used outside. That need to have the right symlinks to cached dir in .terraform, if .terrafrom was generated outside the image.
Example
on host machine
➜ pwd
/home/vm/code/Oslo/modules/aws-environment/.terraform/providers/registry.terraform.io/hashicorp/random/3.4.3

➜ ls -lah
lrwxrwxrwx 1 vm vm   91 Oct  3 17:59 linux_amd64 -> /home/vm/.terraform.d/plugin-cache/registry.terraform.io/hashicorp/random/3.4.3/linux_amd64

in container:

```bash
bash-5.1# pwd
/lint/modules/aws-environment/.terraform/providers/registry.terraform.io/hashicorp/random/3.4.3
bash-5.1# ls -lah
lrwxrwxrwx    1 root     root          68 Oct  3 14:55 linux_amd64 -> /tf_plugins/registry.terraform.io/hashicorp/random/3.4.3/linux_amd64

So, work command is
TAG=latest
docker run \
    -e "USERID=$(id -u):$(id -g)" \
    -v "$TF_PLUGIN_CACHE_DIR:$TF_PLUGIN_CACHE_DIR" -e TF_PLUGIN_CACHE_DIR="$TF_PLUGIN_CACHE_DIR" \
    -v $(pwd):/lint -w /lint \
    ghcr.io/antonbabenko/pre-commit-terraform:$TAG run -a

Looks like https://github.com/antonbabenko/pre-commit-terraform/issues/622#issuecomment-1945725749 could be related to this issue
WORKDIR $PRE_COMMIT_CACHE

COPY .pre-commit-config.yaml .

RUN \
  git init . && \
  pre-commit install --install-hooks \
  && chmod -R a+rwX "$PRE_COMMIT_HOME"

Also, it can be done in slightly different way:
https://github.com/StyraInc/regal/blob/4d7cbe19ff5dacc51e957f8811d30e681ddc7ea9/.pre-commit-hooks.yaml#L15-L20
Could be implemented after: https://github.com/antonbabenko/pre-commit-terraform/pull/644


go/types,cmd/compile: seg fault with generics, channels and closures

This bug is re-producable on Go Playground here

What version of Go are you using (go version)?

$ go version go1.18rc1 darwin/amd64

Does this issue reproduce with the latest release?
Yes, go1.18.rc1
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/kevinstrong/Library/Caches/go-build"
GOENV="/Users/kevinstrong/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/kevinstrong/go/pkg/mod"
GONOPROXY="github.com/merit"
GONOSUMDB="github.com/merit"
GOOS="darwin"
GOPATH="/Users/kevinstrong/go"
GOPRIVATE="github.com/merit"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/kevinstrong/sdk/go1.18rc1"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/kevinstrong/sdk/go1.18rc1/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.18rc1"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/kevinstrong/code/go-sandbox/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/hz/nb5q5m612zl1dkwtzj_jyvvm0000gn/T/go-build3831904386=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

Here is link to go playground
It is the same as the link at the top of this post.
What did you expect to see?
The value "test" printed.
What did you see instead?
A segmentation fault.

CC @griesemer

Looks like we're capturing the dictionary for use in the go'd closure incorrectly. Instead of putting the dictionary in the closure, we're putting the address of a slot on the stack that contains the dictionary. Of course, that slot is an address on the stack of the caller, which might be long gone by the time the goroutine runs.
Maybe forcing the dictionary to always be captured by value instead of by address would fix it. I thought we already did that, but maybe not.

This seems to be an incorrect compilation of a valid program, so marking as a release blocker.
CC @danscales

@findleyr No, this is definitely backend stuff.


BlockingUnaryCall  crushes on Windows

Hello!
I'm trying to make gRPC work under windows. Both server and client are on C++.
I have successfully compiled protoc, all plugins and gRPC libraries. I have generated C++ classes for my protocol. I have successfully implemented server and client then I compiled them. But when I run applications, the client crushes with Access Violation exception.
I managed to determine where this exception occurs. Method BlockingUnaryCall from grpc\include\grpc++\impl\client_unary_call.h has the following body:
CompletionQueue cq;
  Call call(channel->CreateCall(method, context, &cq));
  CallOpSet<CallOpSendInitialMetadata, CallOpSendMessage,
            CallOpRecvInitialMetadata, CallOpRecvMessage<OutputMessage>,
            CallOpClientSendClose, CallOpClientRecvStatus> ops;
  Status status = ops.SendMessage(request);
  if (!status.ok()) {
    return status;
  }
  ops.SendInitialMetadata(context->send_initial_metadata_);
  ops.RecvInitialMetadata(context);
  ops.RecvMessage(result);
  ops.ClientSendClose();
  ops.ClientRecvStatus(context, &status);
  call.PerformOps(&ops);
  GPR_ASSERT((cq.Pluck(&ops) && ops.got_message) || !status.ok());
  return status;

This method calls ops.SendInitialMetadata which manipulates context->send_initial_metadata_ field. But this filed is not valid! It says it has enormous size, something like 19500360. As a result, it tries to accesses elements that do not exist.
But I didn't manage to understand why this happens. When I create the context it is valid, but inside that function it is not, moreover it is not changed until the call to ops.SendInitialMetadata.
Have anyone faced this problem?

see
https://github.com/grpc/grpc/issues/3274
and
https://github.com/grpc/grpc/issues/2671
best practice currently is to remove _USE_32BIT_TIME_T from the grpc
libraries if you use std::chrono or to add it to your project if you don't
On Wed, Oct 14, 2015 at 7:04 AM, Ivan<EMAIL_ADDRESS>wrote:

Hello!
I'm trying to make gRPC work under windows. Both server and client are on
C++.
I have successfully compiled protoc, all plugins and gRPC libraries. I
have generated C++ classes for my protocol. I have successfully implemented
server and client then I compiled them. But when I run applications, the
client crushes with Access Violation exception.
I managed to determine where this exception occurs. Method
BlockingUnaryCall from grpc\include\grpc++\impl\client_unary_call.h has
the following body:
CompletionQueue cq;
Call call(channel->CreateCall(method, context, &cq));
CallOpSet<CallOpSendInitialMetadata, CallOpSendMessage,
CallOpRecvInitialMetadata, CallOpRecvMessage,
CallOpClientSendClose, CallOpClientRecvStatus> ops;
Status status = ops.SendMessage(request);
if (!status.ok()) {
return status;
}
ops.SendInitialMetadata(context->send_initial_metadata_);
ops.RecvInitialMetadata(context);
ops.RecvMessage(result);
ops.ClientSendClose();
ops.ClientRecvStatus(context, &status);
call.PerformOps(&ops);
GPR_ASSERT((cq.Pluck(&ops) && ops.got_message) || !status.ok());
return status;
This method calls ops.SendInitialMetadata which manipulates
context->send_initial_metadata_ field. But this filed is not valid! It says
it has enormous size, something like 19500360. As a result, it tries to
accesses elements that do not exist.
But I didn't manage to understand why this happens. When I create the
context it is valid, but inside that function it is not, moreover it is not
changed until the call to ops.SendInitialMetadata.
Have anyone faced this problem?
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/issues/3821.

#3278 has some updated windows documentation as well, which addresses this problem (waiting on merge/comment)

Thanks for help! Removing _USE_32BIT_TIME_T from all of the gRPC libraries didn't solve the problem, because I'm not using std::chrono.
Problem was solved by adding this define to my client and server projects. Thanks!

we are not longer using _USE_32BIT_TIME_T
https://github.com/grpc/grpc/pull/4315


fix: #2360. Add validation for project name.

Project name should not contain special characters like space. So we can exit when no suitable name provided.

Thanks for the PR, but validate-npm-package-name pulls in a number of extra transitive dependencies which should ideally be avoided.
Also, we should allow user to create directories with any name, we just need to make sure the name used in package.json is valid so that npm/yarn doesn't complain. see 1dbf2462

@yyx990803 Apologies, validate... was my suggestion, but how exactly does it pull in a number of transitive deps? The only item in the dep stack is builtins. http://npm.anvaka.com/#/view/2d/validate-npm-package-name
Fair call on allowing any directory name, though https://github.com/vitejs/vite/commit/1dbf24627d0fed4d6936f53fb84bc94b79372652 is not a valid solution as npm/yarn still won't appreciate users using fs as their package name. The regex also allows me to make my package name .example, yet this too is invalid.
I'd say it's probably beneficial to allow the lib which covers all edge cases to do this. Simpler than refining that regex and offloads the responsibility.


fix(bch-js): Updated to v4.2.5 to fix issue with hydrateUtxos()

:tada: This PR is included in version 1.24.7 :tada:
Your semantic-release bot :package::rocket:


Fix 'TextEdit's line wrapping being highlighted incorrectly

Fixes #22867.

@Paulb23 I actually can't, as it also can be "0".

It should be fine, as indent_px is only explicitly set to 0 when this condition will not be ran.

@Paulb23 Changes made.

Thanks!


bug fixes of both issues of #838

Bug 1 is fixed by converting a single String in dc.subject to an array of one String so that the loop iterates not over the characters of the String but over the String(s) of the array.
Bug 2 is fixed by joining the full path to the filename without its file extension.

Thank you for your addition!
There as a bigger MetadataLoader.ts change with one the latest PR that conflicts with the PR. Can you please resolve the conflict?

There as a bigger MetadataLoader.ts change with one the latest PR that conflicts with the PR. Can you please resolve the conflict?

Hi @bpatrik, all merge conflicts due to PR #829 have been resolved.

Can you please rebase again?
There was a bug in the MetadataLoader over the week that got fixed now.

Hi @bpatrik, all merge conflicts due to the bugfix of PR https://github.com/bpatrik/pigallery2/pull/829 have been resolved again for a second time.

Thanks!
The sidecar tests are failing, can you please take a look?

The sidecar tests are failing, can you please take a look?

Not for me 😄
When I resolved the merge conflicts due to PR  #829, some sidecar tests also failed because of a different creationDate. However, my code does not touch creationDate at all. It seems that the MetadataLoader.ts changes done in #829 read the creationDate inconsistently.
How to proceed? I can't reproduce the failing tests, see screenshot.

My guess those files do not have a creation date in metadata and app falls back to the filesystem's date, so the test are flaky.
Did you happen to touch those files?
@grasdk do you have any idea?
I take a look later but I won't be at computer during the next few weeks.

I'll take a look tonight CET.
@kagahd did you try removing the node modules dir or git clone your code into a new empty dir and rebuild everything from scratch? This would have an affect on the environment similar to what the automated test that fails has.
Also: what is your timezone? I added timezone support..if metadata does not have timezone, local timezone is assumed. Your timezone might not be the same as the build server's timezone, which may cause irreproducible test results between your environment and the build server's. You can mitigate that by.expmicitly adfing timezone offset tonyour test data.
/Regards

@bpatrik was right. In the absense of reading creation date from metadata, file date was read. Since the test files were created at different times on the server and on @kagahd 's test environment, there was a failure.
I made a not-too-pretty pull request for @kagahd 's branch: https://github.com/kagahd/pigallery2/pull/1 that fixes the immediate problem. I think the test data that this PR brings is worth it.
Also I think that these tests will benefit the other pull request for metadata: https://github.com/bpatrik/pigallery2/pull/841

Thanks @grasdk for the PR that I merged already.
@bpatrik , this PR should now be ready to merge.

Thank you for both of you!


Possible bug in Safari when calling callback() method

The below code does not work in Safari 8.0.4.  I'm using HistoryLocation for my routes:
statics: {
    willTransitionTo: function (transition, params, query, callback) {
      checkAuthAndStoreItSomewhere().then(function (auth) {
        if (auth == null)
          transition.redirect('login');
      }).then(callback);
    }
}

It will not continue to the route - but, when I call callback() manually without waiting for anything to return from the promise - it works just fine, like this:
 statics: {
    willTransitionTo: function (transition, params, query, callback) {
     //this will work
      callback();
    }
}

Maybe this has something to do with HistoryLocation?  I'm doing a little more digging and testing to see what I can find and provide more information.

We could be clearer on the documentation, but you need to call callback even after you redirect.
(Maybe we should call it automatically)
statics: {
    willTransitionTo: function (transition, params, query, next) {
      checkAuthAndStoreItSomewhere().then(function (auth) {
        if (auth == null)
          transition.redirect('login');
          next();
      }).then(next);
    }
}

Thanks @ryanflorence - figured this out a while after I had posted the question.  Would there be any reason to not call callback() automatically?
Thanks for the hard work here

@ryanflorence the code you posted doesn't seem right; why invoke callback twice?
I don't see anything wrong with the original code assuming that the promise resolves in time. So my guess would be that checkAuthAndStoreItSomewhere returns a rejected promise or never resolves, which would cause callback to never be invoked.

@taurose - The promise does resolve correctly, but callback was simply not doing anything.

@taurose and @ryanflorence - Even after I call the callback() method after redirect - it's still not working.  Not sure what to do about it at this point..Perhaps I can find another work-around for now (maybe do a hard refresh with window.location.href).  Any other ideas?  If I could show you the real code it might help debug
Let me know how I could help to resolve this.  Thanks again!

we had the same problem and definitely need to callback after the transitionTo().  stumped as to why this wouldn't automatically do that.


how to 3D visualization

Thank you for your open source project.
At present, I am segmenting the mitochondria of scanning electron microscope, and then forming mrc based on the segmented image for 3D visualization.
But there is no suitable visualization tool found at present, I see [picture] (https://github.com/sanketx/mitochondria_segmentation/blob/main/media/cover_1920x1080.jpg) in your project，How is this visualized？
Looking forward to your reply！

I used Chimera X to perform the visualizations. I used it to load up the tomograms and the corresponding segmentation masks, along with some custom scripts to color the granules. Images from multiple mitochondria were composited using Canva to create the final image you linked.


Is it possible to replace the default ctrl+shift+z with ctrl+z?

Since in other scenarios I used ctrl+z as undo, it will be more convenient to give a feature to that.

Thanks for sharing your issue! I tested it on both macOS and Windows 11 and confirmed that Ctrl+Z is used to undo in Beaver Notes. Could you please provide more information so I can better understand your problem? Thanks!

Thanks for sharing your issue! I tested it on both macOS and Windows 11 and confirmed that Ctrl+Z is used to undo in Beaver Notes. Could you please provide more information so I can better understand your problem? Thanks!

Oh it is fixed now, thanks !


Error when running SQL commands- ORA-01034: ORACLE not available

Hi,
I just tried to use this oracle image but I am getting the following error for any command that I issue.

ORA-01034: ORACLE not available

Following are the steps I did:

docker pull wnameless/docker-oracle-xe-11g
docker run -d --name oracledb -p 41960:22 -p 49161:1521 wnameless/oracle-xe-11g
ssh root@localhost -p 41960

On providing the password, it logs in and then to the SQL prompt as well but can't do anything since then. The screen also shows Connected to an idle instance.!
Could someone help me out with this immediately?

Thanks very much!

I just tried and everything was fine on my laptop.
Did you perform anything else after starting the container?

No.. I did not do anything else after starting the container. Do you think it would be the problem with my laptop? I could not figure it out.
However docker pull mysql is working fine. Can't figure out what is wrong and where!

Hey.. Just a quick doubt! Is it necessary that we should have oracle installed in the host system to get this up and running?

Nope, but it's only an express version. Keep it in mind.
abrahamjaison01<EMAIL_ADDRESS>於 2016年6月17日週五 17:54 寫道：

Hey.. Just a quick doubt! Is it necessary that we should have oracle
installed in the host system to get this up and running?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/wnameless/docker-oracle-xe-11g/issues/25#issuecomment-226729387,
or mute the thread
https://github.com/notifications/unsubscribe/ACY08FO-y9Fqx53i_VF0uHJ2yi3VaTlPks5qMm7bgaJpZM4IukEc
.


SpriteSortMode.Immediate control

Xelmish uses SpriteSortBatch.Immediate for its sort mode, which allows the user to do things like switch the sampler state from one texture to the next, i.e. if they want pixel textures drawn using pointclamp, and text drawn using some form of antialiasing.
Despite its name, immediate isn't really immediate in most cases: it does batching. Specifically, it batches by texture, so its basically efficient if you group all draw calls by texture.
This is the way I do most games anyway: all tiles share the same texture, terrain or units tend to be subsets of a single texture etc and again drawn all at once, so its not too much of an issue.
But it might be worth POCing out some sort of alternative: maybe view methods return a construct like (viewable list * sortMode * samplerState) list, then for each Xelmish can create an configure an individual spriteBatch? Hmm.
Reference:

SpriteBatch and SpriteSortMode
SpriteSortMode.Immediate in XNA Game Studio 4.0

Closing this, as the current solution (explicitly setting sampler state on immediate) along with some good view hygiene (don't jump between textures) seems sufficient


Query assignments with side effects

I only just discovered that calling most methods on the existing Query modifies/mutates the query instance instead of returning new instance. So even if no left-side assignment is made the query is modified.
For instance
var qb = query.Query;
var countQuery = qb.AsCount();

both qb and countQuery are the same after applying qb.AsCount().
I am trying to manipulate queries based on certain conditions and would like to keep the original query for later.
I found Clone() method, is that the only way to go?

Yes this is by design, as you have mentioned .Clone() is what you want here.

What about immutable query, would you consider supporting that?

There is no plan for this, unless there is a valid reason to do this, check this thread #176
anyway, we are open to discussing more this.

As the other guy pointed out EF is doing it in immutable way also the methods even though they are not extension methods they look similar to extension methods and for those one typically assumes they return new instances.
I'm not sure whether memory consumption would go through the roof (I could be wrong though).
The way I found out about this is when I ran into issues when I wasn't doing the assignment and the original query mutated anyway.
I don't particularly mind the Clone() but it does introduce certain noise imo.

I would also vote for an immutable pattern. I'm not sure why you say that memory consumption would go up, since on each assignment the old object would fall out of scope (and eventually be garbage collected).

This remains open and I'd hope plans would be revealed if this is planned for any version or maybe the next major version?
I'd imagine going from mutated queries to immutable ones is a breaking change, isn't it?

Making the query immutable requires major rewriting, and honestly I don't see the clear benefit for it, the query.Clone() provide a handy way to do that, when needed.
So I will close this one, since there is no plan for it at the time being.


Adding rate functionality for MusicPlayer based sequencers

This change breaks the SamplerDemo project.  Looks like the SamplerDemo example could be updated relatively easily to use setRate instead, but I kind of like use properties if possible.

Ah ok, I'll update to use the property setter. The reason I didn't was because there are other properties, such as tempo and length, that have a separate method for the setter so I thought that was the style you guys were going for.

If that's the style, you can leave it as you did it and update the example project.  I guess we have evolved a bit since then but haven't updated this code.

If you wanted to make everything property setters, I'm not going to complain, but that would be a separate pull request I guess.

I'll probably just update the Demo for now, then I'll make a separate PR if I get round to refactoring the property setters. Cheers!


this example not working while when i am changing image

Please let me know to solve how to color image.

Now it will work on the image with all size without code change.


Increase timeout for QueryGranularityTest.testDeadLock()

From stack traces, it seems that the reason for #4393, frequent failure of QueryGranularityTest.testDeadLock() is just that 10 seconds is not enough to perform this test, it runs for 5-6 seconds on average on my powerful dev machine with SSD, so it could easily be more than 10 seconds on Travis. Increasing timeout to 60 seconds.

👍


created manual deploy development branch for APT dev stack

Merge request as feature branch.
We will use this branch to review process and manually deploy a strictly development stack for APT backend. In this example, opensearch is merged in as a feature branch. Delete after merging, etc.

@naomatheus please just use feature_add_opensearch_dev_stack as your dev stack branch, no need for [nasa-apt-api-dev](https://github.com/NASA-IMPACT/nasa-apt/tree/nasa-apt-api-dev)


Move DrugChemical synonyms into subdirectory

We currently create a DrugChemicalConflated.txt synonyms file using the DrugChemical conflation information to combine cliques. However, we still include synonym names for each individual chemical output (i.e. SmallMolecule.txt, Drug.txt, and so on). This PR moves those files into a subdirectory, so we still create those files, but they are in a separate directory so a user won't get confused and accidentally use both DrugChemicalConflated.txt and the redundant synonyms in the individual chemical output files.

This is harder to implement than I expected, because it would require modifying write_compendium() so it can write compendia and synonym files into different locations, which will be a lot of work. Closing branch.


Fixes for a couple of component examples / fixtures

I have spotted a couple of issues with the current fixtures whilst using them in govuk-react-jsx. This PR is to fix them...

on the fieldset params example in the checkboxes component, the legend param was not nested underneath the fieldset param.

On the select component, value is not marked as required underneath the items / individual options. If not supplied, the component will render every option with an empty value which I think doesn't make much sense. I have therefore marked it as required, and fixed two of the hidden examples to match (All the other examples already had value specified. This was causing problems in my react port since React cannot have two <option> elements with the same underlying value, even if that value is blank.

Ah, I spy a failing snapshot, will sort after lunch

I've marked this as a draft for a second as I've got more changes coming...

That should be all the changes now so this is ready for review. I've got govuk-react-jsx passing 100% against the govuk-frontend fixtures now, with the changes made in this pull request hacked into my library as overrides.

Thank you for raising this PR @andymantell ! I think on the whole we're happy with these changes and it's great to see the examples being made more consistent!
However, we're not sure about marking value as a required option for selects. At the moment, not providing a value means you end up with an empty string as a value - this is valid HTML. It's possible that in the future we may want to revisit the macro behaviour so that not passing value actually results in what you would perhaps expect (no value on the option), and therefore marking it as required would be correct. But this would be a breaking change, so not something we're in a position to look at yet.
Would you be happy with updating this PR to remove the change that marks value as required? If so, I'll also ask our tech writer to take a look to see if we can update the documentation for that option to make it clearer that you get an empty string if not provided.

Ok that's fair enough. I'll remove the required option. I suppose even though I would consider that output with duplicate empty values to be somewhat weird, there may be some use case that I haven't envisaged...
From my perspective, so long as the actual examples contain values that are unique and non empty then that works.

@vanitabarrett That's done now


Feature/upgrade angular 11

Fixes PXBLUE-2027 & #246 .

Changes proposed in this Pull Request:

Upgrade components to Angular 11

Just update Compatibility table

So I think we are good here, we just need to create a new branch off of dev for release/v5.0.0 and make this a PR into that branch instead of directly into dev.

Or you could just rename this branch to release.v5.0.0 since it's already up to date with dev.

Closing this PR in favor of branch release/v5.0.0.


Adds the is_repair context for the recipe formed event

Also updated it to use the modern determination system + modern switch.
Requested by LG_Legacy

Should both be fixed now.


Jobname.yml constantly reapears when restarting server.

Hi, I don't if it is a bug are not but when I try to delete some insertyourjobname.yml (because I don't want it to my server) it always reapears when restarting my server.
I have tried to find in config file a parameter or an array of job name list to remove these but nothing.
Jobs version: Jobs<IP_ADDRESS>
Server Type (Spigot/Paper/etc): purpur-1.19-1685
Server Version (using /ver): 1.19
Relevant plugins (Delete if this isn't needed): Chunky, CoreProtect, DeathMessage, EssentialsX, EssentialXChat, EssentialsXDiscord, EssentialsXSpawn, FastAsyncWorldEdit, HolographicDisplays, Lands, LuckPerms, Multiverse-Core, Multiverse NetherPortals, PlayerKits, PsudoCommands, QuickShop, Sentinel, SuperVanish, TimeIsMoney, Vault, Worldguard, XConomy.

Out of curiosity, are you stopping the server, deleting the file, then restarting? Or are you deleting the file while the server is running?

Generally speaking example job files are only created if there isint sub folder with name "jobs" in any other case it gets ignored even if you keep that folder completely empty.


Check in our generated code, so we can go get it

As we begin reusing parts of torus in other tools and utilities,
checking in the generated code makes more sense. This way, consumers of
parts of torus don't have to worry about building the generation
tooling, or installing go-bindata, or manually calling make, etc.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


make native NuvlaBox MQTT data FIWARE compliant

won't do, because now MQTT data is equal to Nuvla telemetry, with he same format

won't do, because now MQTT data is equal to Nuvla telemetry, with he same format


Module url not viewing page

Hi,
I have installed the module package according to your instructions but after creating a "test" module, im unable to view it while accessing the path at http://mydomain.com/test
The page says: the url or page cannot be found.
Can you let me know what im missing?
Thanks
PS1 - I can see the "test" folder being created under "modules" folder.
PS2 - I have also included psr4 under "autoload" in composer.json and ran the last composer command aswell.

Hi,
Did you create a route for this url in this test module?

There is already a route created when I have created the "test" module. Is that not enough?

Should be yes, did you check the routes service provider? Is it called? Check the controller, etc.

Im unable to understand. Should I upload the .zip file?


Limit parallelization of test_vm_execute_and_finalize

Motivation
Aims to partially fix https://github.com/AleoNet/snarkVM/issues/2494
This test will sometimes OOM. Limiting parallellization should prevent that.
Additionally, on M2 Max the test ran almost 10% faster due to reduced parallelization.
If the Foundation decides to use a smaller or bigger machine, the existing 10 can easily be changed.
Test Plan
Ran the test locally

Nevermind, we can limit parallelization using -- --test-threads=, which I'll implemented here: https://github.com/AleoNet/snarkVM/pull/2523


cities switcher

todos:

[x] : default cities
[x] : switch between cities
[x] : cache api calls (backend)

an overview:


Demo no longer works

The demo no longer works, which I believe is due to the renaming of handnsontable js and css file to  remove th jQuery in its name, after they made it to work independent of jQuery. But even after correcting the js and css files, its still not working.
Now, I'm getting a ReferenceError: Handsontable is not defined, at the following code:
Handsontable._TextCell = {
  renderer: Handsontable.TextRenderer,
  editor: Handsontable.TextEditor
};

+1


Redo sharpdx math replacement

Tested against the issues presented, and all seems to be just dandy. im sure you'll want to test on your end before pulling this in though.
I wouldn't call this a complete wipe of sharpDX math, but it's certainly a lot. next id like to make int vectors uniform and have some stuff in progress for that

Thanks a lot for that great work!


bump elastic version

Bumping elastic to the version we actually use.
Also give us some query parameters that I would like to use.

@alexwlchan given it's a minor release change, would you consider this to be minor given we trust ES to follow semver definitions?

I think minor is fine, although we're not strict about semver in this repo tbh.


Fix for TIKA-2683 contributed by karanjeets

@kkrugler - Can you please share your thoughts?

If it was possible to have a smaller, generic HTML file for the test then that would be better. I'm almost certain Apache projects are OK with CCA Share Alike but I'd have to double-check.

@kkrugler - Thanks a lot for your quick review. I will update the PR with suggested changes and will try to create a smaller/generic HTML file for the test.

Thanks @karanjeets!

thanks for the review and commit @kkrugler and @karanjeets! 👍


Negative scaling and coordinates lead to misposition

Hey there, I hope you're having a good new year @RazrFalcon! 🍾
There is a difference in behavior between resvg and Chrome rendering when negative scaling and coordinates are involved.
See this cryptic SVG with world Hello being mirrored on the sides of the image. Chrome renders it correctly on the top image (almost what I need), resvg is completely off on the bottom image.
SVG code (apologies for very long base64 image line):
Check out links inside <g clip-path="url(#area)"> that refer the image. You can see that I just change x, y coordinates and transform(scale()) values.
I'm trying to wrap my head around what is wrong with either my understanding or resvg rendering and/or transform property, but it's hard to think in this 4D chess format. Would you kindly clarify what could cause the difference? Thank you.
<?xml version="1.0" encoding="UTF-8"?>
<svg:svg xmlns="http://www.w3.org/2000/svg" xmlns:svg="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" height="3300" id="canvas" preserveAspectRatio="xMidYMid meet" viewBox="0 0 3900 3300" width="3900">

	<svg:defs>
		<clipPath id="area">
			<rect width="14740.1574803149" height="12472.4409448819" />
		</clipPath>

		<svg:image overflow="visible" width="118.110236220472in" height="94.4881889763782in" id="template_image" xlink:href="data:image/png;........."

Here you go:
external_image.svg.zip

Some weird nested SVG placement issue is happening. No idea why. Probably some transformations order bug.
Inkscape and librsvg are also broken in the same way. Browsers are fine.

Makes sense. Will try to find way around...
The problem is, I have correct result rendered withlibrsvg (at least with rsvg-convert), it doesn't fail when image is wrapped in another SVG like this:
<svg id="placeholder_front">
  <svg preserveAspectRatio="xMinYMin meet" viewBox="0 0 1 1" width="123.01059890567664%" height="4200%" x="50.0%" y="50.0%" id="560e62174272ec52c33fa4e13bdc98f51" overflow="visible" class="artwork">
    <g transform="rotate(37.000000)">
      <image xlink:href="560e62174272ec52c33fa4e13bdc98f5.png" width="100%" height="100%" x="-50%" y="-50%" />
    </g>
  </svg>
</svg>

Also, see I raised that in this issue.
2.40.16_working.zip
It works on rsvg-convert version 2.40.16, but fails on rsvg-convert version 2.50.7 😵
Obviously, the problem here is more in abusing SVG format than implementation failure, but still it's a bummer.
I guess will try to find a way around using resvg. I can change SVG to some extent (e.g., I stripped SVG wrapper to reference only <image> using <use>, but nevertheless have to achieve the same visuals.

I'm not sure what results are you trying to achieve, but you're using a very obscure methods. For example, preserveAspectRatio on image with SVG content would not work in Chrome and Safari.
A general rule of thumb when it comes to SVG - never write SVG by hand. It would 100% fail in some app or another. Even in resvg test suite, browsers are barely reaching 90% coverage.
I will take a look into when I will have time. Hopefully it would be an easy fix.


mx.symbol.crop problem

I am going to use the previous mxnet model, using the mx.symbol.crop interface, but now the interface has been deprecated, I want to know some specific details used in this interface.

Could you please detailed information of the code you try to use and the action you would like to achieve?


Publish artifacts on maven central with github actions

The artifacts are now released with github actions. Therefore, preparing a release with maven is sufficient to build and deploy the release.
mvn release:prepare -DautoVersionSubmodules=true -DgenerateBackupPoms=false


[Fix]Add circle ci

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Add circle ci
Modification
Please briefly describe what modification is made in this PR.
BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

delete github ci in this PR


Multiple issues in JobSet CI

CI system has been borked this week, making it difficult to get code checked in... repeatedly running into the following issues:

failing to pull golang:1.20 image from GCR (seems like an intermittent prow or gcr issue that self-resolves but has been happening a lot lately. example: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_jobset/211/pull-jobset-test-e2e-main-1-26/1674124616177553408)
webhook connection refused (example: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_jobset/209/pull-jobset-test-e2e-main-1-26/1674119214257934336)
jobset object was modified, please retry with latest object" error (example: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_jobset/211/pull-jobset-test-integration-main/1674124616152387584)
"pod deleted unexpectedly" (never seen this one before - example: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_jobset/209/pull-jobset-test-e2e-main-1-24/1674119214148882432)
both e2e "pod ping tests" timeout

cc @ahg-g @kannon92

Probably, https://github.com/kubernetes/test-infra/pull/29961 fixed the first issue.

Regarding the second issue, adding logging to e2e like kueue would be good And then we can investigate the root cause. I guess that the jobset controller takes a bit long time to start. So I believe that we need to increase resource requests in the e2e env.

Regarding the third issue, it seems that we updated the jobset without getting created jobset after creating the jobset.
The issue is typically controller issue. So I would suggest getting jobset after https://github.com/kubernetes-sigs/jobset/blob/2903be85cb40273bff9bbaa19fb6b102d0db1d17/test/integration/controller/jobset_controller_test.go#L88.

Regarding the second issue, adding logging to e2e like kueue would be good And then we can investigate the root cause. I guess that the jobset controller takes a bit long time to start. So I believe that we need to increase resource requests in the e2e env.

Good idea, I just submitted kubernetes/test-infra#29966 to bump the CPU/memory resources to align with what Kueue presubmits use.

Also, adding --wait flag to kind would be helpful.
-$KIND create cluster --name $KIND_CLUSTER_NAME --image $E2E_KIND_VERSION
+$KIND create cluster --name $KIND_CLUSTER_NAME --image $E2E_KIND_VERSION --wait 1m

https://github.com/kubernetes-sigs/jobset/blob/main/hack/e2e-test.sh#L15

Regarding the second issue, adding logging to e2e like kueue would be good And then we can investigate the root cause. I guess that the jobset controller takes a bit long time to start. So I believe that we need to increase resource requests in the e2e env.

Good idea, I just submitted kubernetes/test-infra#29966 to bump the CPU/memory resources to align with what Kueue presubmits use.

Also, adding --wait flag to kind would be helpful.
-$KIND create cluster --name $KIND_CLUSTER_NAME --image $E2E_KIND_VERSION
+$KIND create cluster --name $KIND_CLUSTER_NAME --image $E2E_KIND_VERSION --wait 1m

https://github.com/kubernetes-sigs/jobset/blob/main/hack/e2e-test.sh#L15

Oh wow, I didn't know the kind cluster creation would finish/unblock before the control plane was ready... Just created https://github.com/kubernetes-sigs/jobset/pull/213 with your suggestion

webhook connection refused (example: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_jobset/209/pull-jobset-test-e2e-main-1-26/1674119214257934336)

May be we should use Eventually in the jobset create calls to wait for the jobset controller to come up: here  and here

Adding the logs would be great too, it will tell us why the jobset controller didn't get scheduled: https://github.com/kubernetes-sigs/kueue/blob/cc58315b5c572fd872e354bfafaea5582c04a60c/hack/e2e-test.sh#L27-L40

jobset object was modified, please retry with latest object" error (example: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_jobset/211/pull-jobset-test-integration-main/1674124616152387584)

For this one, this update needs to be wrapped in an Eventually clause with the following sequence:

js = client.Get(js.name)
2 update.fn(js)
client.Update(js)

similar to https://github.com/kubernetes-sigs/jobset/blob/7534695e63d49cdb9893f5e38eecb210266c8719/test/integration/controller/jobset_controller_test.go#L708

jobset object was modified, please retry with latest object" error (example: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_jobset/211/pull-jobset-test-integration-main/1674124616152387584)

For this one, this update needs to be wrapped in an Eventually clause with the following sequence:

js = client.Get(js.name)
2 update.fn(js)
client.Update(js)

similar to
https://github.com/kubernetes-sigs/jobset/blob/7534695e63d49cdb9893f5e38eecb210266c8719/test/integration/controller/jobset_controller_test.go#L708

That's right.

I'll open up a PR to make our e2e tests look similar to kueue (add logs) and I'll open up a PR to add eventually for updates as @ahg-g pointed out.

we addressed all issues mentioned here, right?

"pod deleted unexpectedly" (never seen this one before - example: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_jobset/209/pull-jobset-test-e2e-main-1-24/1674119214148882432)

Yes, everything except the "pod deleted unexpectedly" (example: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_jobset/209/pull-jobset-test-e2e-main-1-24/1674119214148882432) but that appears to have been a transient issue or perhaps was fixed by bumping the CPU/memory resources for the e2e tests.  We can close this now.


[jaxrs] Minor TCK fixes to improve implementation compatibility

The JAX-RS TCK makes heavy use of String concatenation to build URIs, but it isn't very careful about introducing extra / characters. Extra / characters can cause failures in some JAX-RS clients so we should use the built in API methods to avoid these problema
Signed-off-by: Tim Ward<EMAIL_ADDRESS>

Is the leading / in the SSETestCase path in line 62 correct? I believe we have to remove it here as well

I'll remove it, but the definition of the path method requires it to handle leading/trailing / characters properly. Previously we were concatenating String objects which was the source of the problem.

looks good


environment question

[ INFO] [1650269147.989569531, 34523.438000000]: waitForService: Service [/gazebo/set_physics_properties] is now available.
[ INFO] [1650269147.999950504, 34523.524000000]: Physics dynamic reconfigure ready.
DEVICE :  cuda
[spawn_urdf-3] process has finished cleanly
log file: /home/wanbaoliang/.ros/log/13227772-beed-11ec-8a00-63681b92ee34/spawn_urdf-3*.log
ACTOR:
PolicyNet_1225(
(fc_1): Linear(in_features=4, out_features=128, bias=True)
(fc_2): Linear(in_features=128, out_features=128, bias=True)
(fc_3): Linear(in_features=128, out_features=64, bias=True)
(fc_mu): Linear(in_features=64, out_features=2, bias=True)
(fc_std): Linear(in_features=64, out_features=2, bias=True)
)
The program doesn't run here and the gazebo model doesn't move. What's the reason?
version: noetic  gazebo11 ptyorch10.0

Thank you for interests in my project
Which launch file did you use?
This command,
& roslaunch myrobot mobile_robot_SAC.launch
might work.
The Gazebo world maybe need some times to be run depending on your computer performance.
Actually this project need some update, such as delete unnecessary code, requirements list, and how to use.
I will update this repository as soon as possible.


global: Docker and dependencies

Added fixes for dependencies causing conflicts in setup.py
closes https://github.com/inveniosoftware/invenio-app-ils/issues/1131 https://github.com/inveniosoftware/invenio-app-ils/issues/1130

Just for curiosity. How did you determine the correct setup.py versions? Trial and error o something more rational?

I would say it is a dance between trial and error followed by going through the packages and looking for conflicting dependencies


Ensure template follows version documentation

If the dependency/tool's version uses the 4th segment, update the package's version to use the current date (YYYYMMDD) in the 4th segment. We had this problem with https://github.com/mandiant/VM-Packages/pull/347 This PR should allows us to use the integration next time.

cool! thanks!


[Settings] Add mint - need to be able to directly type in

Need to be able to type in directly. I was successful in pasting in from the clipboard, but would be nice to be able to enter directly. You also need to prepend https:// if the user hasn't type that in.

Input added in v0.1.2-beta.10


Sorry, no response was returned.

Type: Bug
Sorry, no response was returned.
Extension version: 0.23.1
VS Code version: Code 1.96.0 (Universal) (138f619c86f1199955d53b4166bef66ef252935c, 2024-12-11T02:29:09.626Z)
OS version: Darwin arm64 24.2.0
Modes:

I can not reproduce this. Can you reproduce with latest version of VS Code Insiders and latest pre-release of Copilot Chat extension?
If you can reproduce, can you please provide reproducible steps?
Thank you

Have this issue right now after working on a project a few hours.
Extension version: 1.253.0
Version: 1.95.3 (system setup)
Commit: f1a4fb101478ce6ec82fe9627c43efbf9e98c813
Date: 2024-11-13T14:50:04.152Z
Electron: 32.2.1
ElectronBuildId: 10427718
Chromium: 128.0.6613.186
Node.js: 20.18.0
V8: 12.8.374.38-electron.0
OS: Windows_NT x64 10.0.22631
upgraded copoilot ( not status is restart extensions - so maybe not ?) and was asked to restart vscode. after that copoilt chats again

What are the reproducible steps? Can you also provide a screenshot?

Not after the reboot... but before everytime ask a q it would say, sorry
cant. It started out of the blue while working on the code maybe a 1 or 2
since last q. No obvious reason
tors. 19. dec. 2024 11.04 skrev Isidor Nikolic @.***>:

What are the reproducible steps? Can you also provide a screenshot?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-copilot-release/issues/3352#issuecomment-2553301119,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACR3TKPBS7TNXGPUQBAYQJ32GKK2JAVCNFSM6AAAAABT2WQNY2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJTGMYDCMJRHE
.
You are receiving this because you commented.Message ID:
@.***>

Ok. For now I will go ahead and close this. If you find reproducible steps I am happy to reopen this issue

I just hit it again now, after being AFK for four hours. But has it just
happens when asking a Q it is hard to come by a way to reproduce. changed
model from Claude to o1-mini and it started working again...
Den tors. 19. dec. 2024 kl. 12.40 skrev Isidor Nikolic <
@.***>:

Ok. For now I will go ahead and close this. If you find reproducible steps
I am happy to reopen this issue
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-copilot-release/issues/3352#issuecomment-2553495129,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACR3TKLFSD2UTQPY2VJMEDD2GKWBFAVCNFSM6AAAAABT2WQNY2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJTGQ4TKMJSHE
.
You are receiving this because you commented.Message ID:
@.***>

again I had the issue with Claude. Selected "Done" this time and startede a
new copilot session. There has to be a timeout or session breaking
Den tors. 19. dec. 2024 kl. 20.50 skrev Michael Ryom @.***

:

Changing back to the claude model gives me the same error. Again dont know
how to trigger it, but once it there a reboot or changing model is the only
way to use copilot
Den tors. 19. dec. 2024 kl. 20.16 skrev Michael Ryom <
@.***>:

I just hit it again now, after being AFK for four hours. But has it just
happens when asking a Q it is hard to come by a way to reproduce. changed
model from Claude to o1-mini and it started working again...
Den tors. 19. dec. 2024 kl. 12.40 skrev Isidor Nikolic <
@.***>:

Ok. For now I will go ahead and close this. If you find reproducible
steps I am happy to reopen this issue
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-copilot-release/issues/3352#issuecomment-2553495129,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACR3TKLFSD2UTQPY2VJMEDD2GKWBFAVCNFSM6AAAAABT2WQNY2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJTGQ4TKMJSHE
.
You are receiving this because you commented.Message ID:
@.***>

Thank you for your feedback. There is a service side issue that we are aware of with Claude. I know the team is investigating, and I expect a fix to land soon.


Removed from Chrome Web Store

I know it's technically an Opera extension, but it WAS published in the Chrome Web Store and it worked very well in Chrome, I've been using it for years.
However, as of a few days ago, the page for the web store returns a 404: https://chrome.google.com/webstore/detail/restman/ihgpcfpkpmdcghlnaofdmjkoemnlijdi
And according to Chrome Stats it was removed 6-10-22:
https://chrome-stats.com/d/ihgpcfpkpmdcghlnaofdmjkoemnlijdi/download
Any reason it was removed? I recommend this extension to coworkers all the time since it's faster than Postman for simple stuff

Edge told me it was marked as Malware

Sorry guys, the extension was removed from the Chrome Web Store (it seems not having an icon it's enough to get you delisted :shrug: ), I'm sorting out the details, will let you know once it's fixed.

It's back, please let me know if you still face issues. Thanks for reporting!


add fix for metricsProperties when HasPrometheusConfigFile is true.

Fix the bug where Monitor.Prometheus.ConfigFile is set. the configMap which contains spark metrics.properties & prometheus config file will not be created & mounted, so the metricsProperties will be useless.
Add a new property metricsPropertiesFile, this will overwrite the value spark.metrics.conf in spark.properties. give a better way to specify spark metrics configuration when it is large.

@liyinan926 Please review.
When I am running the build commands such as:
go generate
controller-gen ...
make build-api-docs
There are auto-generate code for previous PRs such as lifeCycle, TerminationGracePeriod
Are those not included intentionally?

The authors of the previous PRs must have forgotten to run these. Please include them in this PR.

@liyinan926 Please review again, thanks


removed edit and remove icons for isVersionOf relation

@pkdash Can you give a quick review on this PR? Only one conditional statement is added on template page so that edit and remove icon buttons will not be displayed for isVersionOf relation since it should not be editable by owners but can only be created programmatically by the system.

+1. Merge after Jenkins checks are successful.


Dynamic react routes from json

Utterly amazed no concrete, definitive, unobtusive documentation on the topic.
Working with this small sample:
const routesArray = [
{
root: { paths: '/', component: '../components/pages/welcome-page' },
AboutUs: { paths: 'About-Us', component: '../components/pages/about-us' },
Blog: { paths: 'Blog', component: '../components/pages/blog' },
ContactUs: { paths: 'Contact-Us', component: '../components/pages/contact-us' },
NotFound: { paths: '*', component: '../components/pages/not-found' }
}
];
const getRoutes = () => {
??????????????
};
export default (
< Route component={ require('../components/APP') }>
{ getRoutes }
< /Route>
);
I need to find a way to make this work or react-router offers me no benefit for larger projects.
Moreover, there are times, when circumstances dictate that I cannot rely on isomorphic routing and must use 100% client-side.
Any ideas, someone?

Per the readme, the issue tracker is for bug reports and feature requests only. Please use Stack Overflow or one of our chat channels if you need help.


More optimizer updates, add MARS, LaProp, add Adopt fix and more

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Add docs for Image and Process.

Merge activity

Dec 20, 3:56 PM EST: A user started a stack merge that includes this pull request via Graphite.


ARM support

I want to use this on my Raspberry Pi, but it doesn't support ARM. Should I download the old one while I wait for ARM support?

The old one doesn't work, it says error 400
I really hope you add ARM support

Ill just use fclone instead


FizzBuzz exercise - update

Added new controls on passed values into the FizzBuzz method:

Updated the test file with new test cases
Updated the assignment into readme.md

OK!


I can't compile

Hi.
Some time ago I downloaded your project but I was busy and I didn't test it.
Now I've downloaded your current version but I can't compile. Lots of errors in voice.h
The old versión compiles ok.
errors.txt
I dont know if this is a "boards" versión issue,. I've tested with three different "boards" versions.
This project is amazing. I'm thinking about building an e-drum module.
Thank you.

@zircothc Sorry for that, it seems that versions mixed somehow.
I am working on the next iteration, so hopefully in a couple of days there'll be a new version.
I got rid of multiple functions for different bit depths, cleaned some code, but the sampler still suffers of clicks on overrunning max polyphony.

@zircothc  I have just uploaded a more or less stable version, please, try

Hey thank you.
I'll try in a moment.
;)

Hi.
Compiles OK.
And...wow... it sounds amazing.
Thank you @copych


[fix] bug in rlsbb.py that occurred when the entry had no urls field

Motivation for changes:
A bug occurred in rlsbb.py when an entry had no urls field.
Addressed issues:

Fixes a bug I just recently found.

thanks :-)


Support dynamically bridged Terraform providers in the Registry

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
Pulumi released dynamically bridged TF providers, which allows Pulumi users to consume any Terraform provider quickly. The registry should host documentation for dynamically bridged providers.
This is a tracking issue. To implement, we will need:

A list of TF providers who's dynamically bridged docs we want to host. This could be the top 100 providers we see used, or an entire registry.
A safe environment to run pulumi package gen-sdk terraform-provider <provider>.
A mechanism to serve these providers to the registry and listen for upstream updates.

Affected area/feature

An additional (or alternative) simple short-term nice feature could be to just generate the markdown docs in the local package and let you run a simple local web sever that serves them up.


pre-configuration based assignment

Assign instances based on a pre-configured instance assignment map. The assignment will always respect the mirrored servers in the pre-configured map. E.g.
  uplift from 15 instances in 3 replicas to 21 instance in 3 replicas
  21 instances in 4 pools
  Pre-configured partitioning:
          RG1    RG2    RG3
    Host  0      1      2
    Host  3      4      5
    Host  6      7      8
    Host  9      10     11
    Host  12     13     14
    Host  15     16     17
    Host  18     19     20

  Existing configured partitioning:
          RG1    RG2    RG3
    Host  0      6      2
    Host  12     7      14
    Host  1      4      5
    Host  3      13     8
    Host  9      10     11

  Final assignment for this table:
          RG1    RG2    RG3
    Host  0      1      2
    Host  12     13     14
    Host  3      4      5
    Host  6      7      8
    Host  9      10     11
    Host  15     16     17
    Host  18     19     20

In the above assignment, the mirrored servers (row) in the pre-configured partitioning will always be respected when converging the existing assignment to the final assignment, meanwhile, we preserve the existing assignment as much as we can in this process.
Tested using randomized testing scheme with 10000000 iterations. No errors.

Codecov Report

Merging #11578 (f135864) into master (9e0752c) will decrease coverage by 48.43%.
Report is 71 commits behind head on master.
The diff coverage is 63.50%.

@@              Coverage Diff              @@
##             master   #11578       +/-   ##
=============================================
- Coverage     62.98%   14.56%   -48.43%
+ Complexity     1099      201      -898
=============================================
  Files          2302     2327       +25
  Lines        124040   125029      +989
  Branches      18903    19109      +206
=============================================
- Hits          78131    18208    -59923
- Misses        40362   105282    +64920
+ Partials       5547     1539     -4008

Flag
Coverage Δ

integration
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration1
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration2
0.00% <0.00%> (ø)

java-11
14.56% <63.50%> (-48.41%)
:arrow_down:

java-17
14.55% <63.50%> (-48.31%)
:arrow_down:

java-20
14.56% <63.50%> (-48.30%)
:arrow_down:

temurin
14.56% <63.50%> (-48.43%)
:arrow_down:

unittests
14.55% <63.50%> (-48.43%)
:arrow_down:

unittests1
?

unittests2
14.55% <63.50%> (+0.08%)
:arrow_up:

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

Files Changed
Coverage Δ

...mmon/assignment/InstanceAssignmentConfigUtils.java
0.00% <0.00%> (ø)

...ntroller/helix/core/rebalance/TableRebalancer.java
66.12% <0.00%> (-1.80%)
:arrow_down:

...fig/table/assignment/InstanceAssignmentConfig.java
0.00% <0.00%> (-89.48%)
:arrow_down:

...ntroller/helix/core/PinotHelixResourceManager.java
62.94% <20.00%> (-0.27%)
:arrow_down:

...ources/PinotInstanceAssignmentRestletResource.java
65.34% <26.31%> (-4.13%)
:arrow_down:

...ent/instance/InstancePartitionSelectorFactory.java
42.85% <50.00%> (+2.85%)
:arrow_up:

...stance/PreConfiguredInstancePartitionSelector.java
85.60% <85.60%> (ø)

.../assignment/instance/InstanceAssignmentDriver.java
100.00% <100.00%> (ø)

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

What is the difference between this and #8989?

Do you have a design doc for this? Trying to understand what we want to achieve in this PR

Do you have a design doc for this? Trying to understand what we want to achieve in this PR

Let's meet to share our context on where we are coming from and the overall goal here. We will setup a meeting

@jasperjiaguo  Capturing our offline discussion here:
The configuration for enabling this feature works as follows:

PropertyStore will have a tenant-level instance assignment mapping. The name of the ZNode is the tenant name (with instancePartitionType i.e offline,consuming,realtime)
TableConfig.instanceAssignmentConfig will contain "partitionSelector" that denotes whether we have to use MIRROR_SERVER_SET_PARTITION_SELECTOR or not.
We additionally overload  instancePartitionsMap in tableConfig with detail about the Znode where the tenant-level instance assignment mapping (1) is stored.

There seems to be a few things we can simplify here:
(1) Remove the additional level of redirection i.e populating instancePartitionsMap can be avoided:

Each table belongs to a specific server tenant
Each tenant has a dedicated Zk node in PropertyStore that stores the instance assignment mapping.
So, given a table, we should automatically be able to figure out the corresponding the tenant-level instance assignment mapping from Zk.

(or)
(2) If we need this extra detail, we should probably avoid overloading the already existing tableConfig.instancePartitionsMap with this mapping detail (even though they are mutually exclusive) and have a new field for it
@jasperjiaguo  is going to look at the possibility for this. If the change is non-trivial, it's better left as a TODO.

this PR broken ColocatedJoinQuickStart can we take a look CC @jasperjiaguo @siddharthteotia

should be fixed by https://github.com/apache/pinot/pull/11915

should be fixed by #11915

Thanks for the fix!


Software unsigned, false positive for antivirus

Using ESET, but recently because of no signature the software gets quarantined and needs an exception to allow download and usage

not really sure what to do about that

Allot of windows software does not have signature since its a time consuming and a cost to get it. Windows defender does not have false positive on this software. Your antivirus seems over aggressive.


how to used Form for Views Page

Hi
for example , we need a contact us form.
how to used this in a single webpage !?!?
not dialog box or something. in a standard view page.
can you help me!?

Can't you just write this as a standard view using regular ASP and not really using Serenity at all? That would just be standard MVC?
I don't think this is related to Serenity at all.

There are several sample pages from AdminLTE using ordinary MVC. If you want to do it in Serenity way, there are signup, change password etc. pages.


Monthly release (7.8.0)

@ansasaki is everything ok with the versioning of the configuration for this release?

Wait, do not merge this right now: maybe it added incompatibility with python 3.6
I'll investigate more and will report back.

@maugustosilva Sorry, apparently it was a false alarm.  I think you can proceed with the new release.
The problem in my case was an older python-asn1-modules which is missing the newly used rfc4108. Anyway, we should implement compatibility tests with a matrix of supported python versions and include in the CI.
About bumping the minimum python version, what are the benefits?

@ansasaki Ok, I will leave the discussion about bumping the minimum python version for a future release, we are (well, I am :-) ) "late" with the release as is. Thanks.

Just need an approval from someone now @mpeters @THS-on @stefanberger @aplanas @sergio-correia @ansasaki


build: switch from glide to dep for dependency management

This PR aims to replace lnd's dependency management glide with dep.
dep allows one to automatically generate the manifest files from other dependency management systems, like glide. When doing this, I noticed that not all of the specific requirements we had when using glide where set, like specific commit hashes for packages. Instead, dep's manifest file, Gopkg.toml, was manually modified to ensure all the version requirements were in place.
For more info on dep's manifest file, see https://golang.github.io/dep/docs/Gopkg.toml.html.
Fixes #794.

Can you rebase this? Thanks!
Note that we've switched over from bolt to bbolt.

Pushed out a new version. Packages that specified a version number in the previous glide.yaml have now been converted to the commit hash of that version. Also, packages that used the master branch have been converted to the hash of the latest commit of the master branch as of today.


Create a path class in VisIt to handle path strings internally always as unix-style

Describe the bug
I tried to open a file on Windows and the obvious failed and I had to replace all the slashes with double slashes. This seems really unintuitive. Here is what I tried.

Desktop

OS and version: Windows
VisIt Version: 3.1.1

This is the case for most everything we do with paths via python on Windows.

I believe Python itself has best practices and modules for dealing with this in an OS/file system agnostic sort of way.
Can we maybe either define/document a best practice in the use of existing python modules to deal with file system paths in an OS agnostic way or maybe create a path utility python function that allows VisIt python code to be easily (maybe more easily than using existing python module functionality).

It just seems like there are a lot of gotchas to using VisIt on Windows and this is another example that we should try to make better, or at a minimum document. Even when using the GUI this type of stuff sometimes leaks in because we are using Python under the covers.

gotchas to using VisIt on Windows

FWIW...I think these are less specific to VisIt and more related to Python and OS portability in Python itself.
IMHO, this is a common issue in Python. So much so, Python does have modules and best practices for handling it. For example, use the os.path module. But, if one is not accustomed to using those modules in their python code to handle strings related to file system paths, it is a bit of a shock to realize it is necessary.

I guess its concievable we could add logic wherever a user passes a string representing a path to a VisIt CLI function to do the right thing with that string in terms of interpreting things like C: and slash characters.  I dunno how involved that would be and how good a result we could achieve doing that.
In the VisIt test suite, we handle this the pythonic way (well, @cyrush did when he overhauled our scripting to go 100% python)

Yes, that might be a good idea.
Another thing about my specific use case, I thought I was using the windows path separator and since the path was quoted, spaces should be ok.
Using os.path.join() would probably also have done it, but seems harder than it should be.

Problem with Windows-style separators are the fact that they are escape characters, hence the need to use '\' in strings.
Python 3 has pathlib which should be helpful.
Overall, I think VisIt should also use some sort of Path class so that handling filenames/paths and os-specific conversions can all happen in one place.  Store them with forward-slashes so we don't have to worry about the need to escape the path-separator, and have a method to retrieve a windows-style path when needed.

As long as Windows users can write paths naturally for their world and Unix users for their world...I think this makes sense.
That said, I think there are basic problems for Windows case to be natural...
>>> f="This\is\a\path"
>>> f
'This\\is\x07\\path'
>>> f="This\is\one\path"
>>> f
'This\\is\\one\\path'

In the first case, one of the backslashes is interpreted as an escape character on a. In the second case, all backslashes are interperted as backslash characters. So, without creating our own Path object (maybe that is what @biagas was suggesting), I don't see how we fix this problem, apart from the others already mentioned.

Do python raw strings help:
f=r'This\is\a\path
?

Oh, good idea...and yes they do...
>>> f=r"this\is\a\path"
>>> f
'this\\is\\a\\path'

And, BTW...I neglected to mention the bad behavior (e.g. without raw string) occurs in both Python 2 and Python 3. But, the raw string works in both.

We should create a path class that stores paths internally using unix style and presents them in the other manner if running on Windows. This would be a lot of work but worth it. Qt has a class that solves this problem, which we could use as basis for our class.

C++ std 17 introduced filesystem::path that may prove useful.
https://en.cppreference.com/w/cpp/filesystem/path

If its not already, we should document the python r"/this is a /path" tip for python strings.

Inventory of places in VisIt strings can represent paths...

.visit files
GUI where user either enters text or use file system browsing widget to specify a path
CLI where user specifies file paths
Metadata associated with various database formats

Silo's multi-block objects and maybe some of Silo's DBOPT_XXX options
Conduit metadata
BOV metadata
HDF5 multi-data sets and refs objects (not sure we have any of these)
Mili json bootstrap
Testing scripts where we test textual output from VisIt (special case of CLI)

I also worry about wide characters and non-UTF8 characters (which is apparently supported in Linux)


增加获取获取所有生产者组的客户端连接信息，对应issues信息：　https://github.com/apache/rocketmq/i…

ssues/2939
https://github.com/apache/rocketmq/issues/2939
Make sure set the target branch to develop
What is the purpose of the change
add producerConnectionAll command

Coverage increased (+0.1%) to 53.984% when pulling c326cdd142c067447d4fbab0faeec7c2bcdbed3d on dingwpmz:develop into 3e1d5015c6a0aa8b2fc1a5653f1e42088f2d1fc5 on apache:develop.

Codecov Report

Merging #2940 (c326cdd) into develop (f77d7fe) will increase coverage by 1.31%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop    #2940      +/-   ##
=============================================
+ Coverage      47.10%   48.42%   +1.31%
+ Complexity      3488     3192     -296
=============================================
  Files            309      420     +111
  Lines          29192    26443    -2749
  Branches        4178     3511     -667
=============================================
- Hits           13751    12804     -947
+ Misses         13583    11982    -1601
+ Partials        1858     1657     -201

Impacted Files
Coverage Δ
Complexity Δ

...ocketmq/broker/processor/AdminBrokerProcessor.java
7.68% <0.00%> (-0.28%)
13.00 <0.00> (ø)

...g/apache/rocketmq/common/protocol/RequestCode.java
0.00% <ø> (ø)
0.00 <0.00> (?)

...mq/common/protocol/body/ProducerConnectionMap.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

...der/GetAllProducerConnectionListRequestHeader.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

.../broker/subscription/SubscriptionGroupManager.java
54.11% <0.00%> (-16.48%)
8.00% <0.00%> (-3.00%)

...org/apache/rocketmq/store/ha/WaitNotifyObject.java
71.15% <0.00%> (-7.70%)
10.00% <0.00%> (-1.00%)

.../apache/rocketmq/remoting/common/RemotingUtil.java
12.62% <0.00%> (-1.21%)
3.00% <0.00%> (ø%)

...n/java/org/apache/rocketmq/store/ha/HAService.java
71.47% <0.00%> (-0.66%)
17.00% <0.00%> (ø%)

.../apache/rocketmq/logging/inner/LoggingBuilder.java
63.92% <0.00%> (-0.64%)
3.00% <0.00%> (ø%)

...ache/rocketmq/store/config/MessageStoreConfig.java
59.44% <0.00%> (-0.14%)
80.00% <0.00%> (+1.00%)
:arrow_down:

... and 305 more

Continue to review full report at Codecov.

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


How to get start time of the day?

Is there any way to achieve this in a nicer way:
today_start = arrow.Arrow.fromdate(arrow.utcnow().date())

@zero-master This here is the Apache Arrow project which is known in Python as pyarrow. I thunk you confused it with the arrow python package.


Design a cluster registry

We need to develop our cluster registration story. Today in skv2, a registered cluster is represented by a secret of type solo.io/kubeconfig, and a cluster name is the metadata.name on that secret. This simplistic approach introduces issues such as cluster name duplication across namespaces. Let's review KubeFed, Service Mesh Hub, Fleet, and other multicluster projects and determine how we want to represent clusters at the solo-kit level of our products.

we should store namespace.name but make it opt-in


Add support for fullnameOverride to Chart

Add support for fullnameOverride in the kong.fullname define in _helpers.tpl

Is this something that aligns with guidelines for charts from the maintainers of Helm?

https://github.com/helm/charts/blob/master/stable/ambassador/templates/_helpers.tpl#L14
This is typically the condition I have found to support this.  It avoids duplication of named entries for ...metadata.name in the event that you do not use multiple release names and maybe has other uses than the one I just highlighted.

I see. @rainest how do you feel about adding yet another knob for this?

Yeah that's reasonable to add--just today ran into someone confused as to why resources got named kong-kong-whatever when they named their release kong. I'd recommend still having some sort of indicator showing the purpose of a release (e.g. "prod", "test", etc. for the environment type; "dmz", "internal" for the network zone; etc.), but those can often be handled via other means.
Marked "good first issue" as this should be simple to add if someone in the community is interested. We should create it if we don't see any community submissions, though it's not an immediate priority.

I used Release Name for identification across all cluster about my applications and using fullname on Kong chart don't remove the prefix of Release.Name.
In https://github.com/Kong/charts/blob/main/charts/kong/templates/_helpers.tpl#L11-L18
Would be great if we have a way to fully override the name of resources that will be created in some release.

Closing this as stale as there has not been activity in nearly a year. If you're still interested in this functionality please feel free to comment and we'll be happy to re-open this and re-assess.

re-open this for related FTI: https://konghq.atlassian.net/browse/FTI-4075


Loading events are not announced to the user

Describe the bug
For cases where there is an async loading behavior, wizard page changes and datagrid expandable rows for example, screen readers will not know that the icon has changed from the expandable option to a loading indicator.
How to reproduce

Using a screen reader, try a wizard with async behavior
Tab to the Next button and activate it
Notice that the screen reader does not announce any status messages about the loading of the page. But sighted users see an infinite spinner animation letting them know the page is still loading.

Expected behavior
Use an empty and polite live region with aria-busy set to false like as seen below.

Then after the Next button is activated, it should be populated with the loading message as seen below.

Loading ...

This should be repeated for long wait times.
Recommendation
Use an empty and polite live region with aria-busy set to false like as seen below.
Versions
App

Angular: [e.g. 6]
Node: [e.g. 8.10.0]
Clarity: [e.g. 0.12.5]

Device:

Type: [e.g. MacBook]
OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional notes
WCAG 2.1 Guideline:
4.1.3 Status Messages
(Level AA)
In content implemented using markup languages, status messages can be programmatically determined through role or properties such that they can be presented to the user by assistive technologies without receiving focus.

Ideally, we need to fix this for both Datagrids, Wizards, and Tree View. There is also the general recommendation we should make in our documentation on the progress bars/spinners about a11y details related to how this pattern works for apps to apply it themselves.


[BUG] Tiles pop in and out, and range is very low

Describe the bug
When using the mod (latest version, latest version of MFS), with no other mods, with a 1gbps internet connection and a powerful computer, the tiles pop in and out when in move.
Even if I remain static and just move the camera, the tiles will pop in and out (some tiles that I have previously downloaded will disappear)
To Reproduce
Steps to reproduce the behavior:
My ingame LOD is set to 200
Screenshots

Desktop (please complete the following information):

OS: W10
MOD Version 2.2.7
MSFS Version, MS Store or Stream : MS store

Additional context
The console shows no error, it downloads the tiles properly and fast

It may be caused by the "Rolling Cache" enabled in MSFS data configuration. Please delete the rolling data.

It may be caused by the "Rolling Cache" enabled in MSFS data configuration. Please delete the rolling data.

I tried this, but it doesn't fix the issue

Could you try to record a screen capture video and share to me (maybe youtube)

After disabling MFS cache, clearing it, AND uninstalling your program and reinstalling it, the problem is gone ! Thanks


Issue template Updated ✨

Closes
Closes #1524

Description
I have Updated the Issue Template that automatically added gssoc23 label on the issue
Checklist:

[x] I have linked the PR to the correct issue.
[x] I have read the Contribution Guidelines
[x] I have built and tested the changes, and they do not break or show any errors.

Hey @Vaishnavi-Patil2211  Please Review The PR And Merge it kindly give it Level3


Check uncoverml revrand gridsearch traceback

Details here: https://github.com/NICTA/uncover-ml/issues/130

@dsteinberg based on https://github.com/NICTA/uncover-ml/issues/130, consider using a different default convergence criterion for StandardLinearModel.

Resolved.


Fix README

I was going through the repository and saw the formatting was off.
I reran cargo-readme, seems a few edits were not put public.

Add missing quotes
Update README

Sorry, I completely missed your pr before.
These fixes look good. Thanks for contributing.