May I ask if KEDA can take the initiative to consume the event itself?

Proposal
May I ask if KEDA can take the initiative to consume the event itself? For example, when a producer inserts a message into MQ or Redis, does a component of KEDA automatically pop an event? In my scenario, I don't want to do roll polling with pollingInterval, and I just want something to trigger scaling up, without a real consumer(or KEDA itself). May I ask if there is a way?
thanks~
Use-Case
No response
Is this a feature you are interested in implementing yourself?
Maybe
Anything else?
No response

Hello,
Do you mean if KEDA can work as consumer to detect the queue length instead of polling every X seconds? Which is the problem with the polling?
I think that I'm missing something 😞

Hello, Do you mean if KEDA can work as consumer to detect the queue length instead of polling every X seconds? Which is the problem with the polling?
I think that I'm missing something 😞

Hello, my project requires the ability to scale up and down elastically.
I have researched keda and knative, both of which can meet my needs, but the cost of using knative is bigger then keda. I particularly like keda because it is very simple and easy to use.
However, in my scenario, when the Pod is scaled down to 0, I hope to wake it up simply by making an HTTP GET request. Since the http-addon in the community is currently only in beta version, so I exposed an API in my backend service to the frontend, and this API sends a message to the redis list. Then my keda polls the redis list every 5 seconds.
This can wake up the Pod, but I don't have a real consumer to consume the redis list. Now I can only set a timer in the backend to actively clean up the redis list every short period of time.
Although this way meets my demand, but it increases a lot of development costs. I have to develop a program to consume message queue(my scenario is redis list). So I would like to ask if it is possible to provide the feature to actively consume the queue with keda.
Actually, I have tried to use the external mode to directly consume the Redis list.

type MyRedisScaler struct {
	pb.ExternalScalerServer
	redis *redis.Client
}

func NewScaler() *MyRedisScaler {
	return &MyRedisScaler{
		redis:                redisClient.Client(),
		ExternalScalerServer: pb.UnimplementedExternalScalerServer{},
	}
}

func (scaler *MyRedisScaler) IsActive(ctx context.Context, scaledObject *pb.ScaledObjectRef) (*pb.IsActiveResponse, error) {
	fmt.Println("1111111111111111111")
	listName, ok := scaledObject.ScalerMetadata["listName"]
	if !ok {
		return nil, fmt.Errorf("listName not found in metadata")
	}

	elements, err := scaler.redis.LRange(ctx, listName, 0, -1).Result()
	if err != nil {
		return nil, err
	}

	_, err = scaler.redis.Del(ctx, listName).Result()
	if err != nil {
		return nil, err
	}

	fmt.Println("1111111111111111111 list length: ", len(elements))

	return &pb.IsActiveResponse{
		Result: len(elements) > 0,
	}, nil
}

func (scaler *MyRedisScaler) StreamIsActive(_ *pb.ScaledObjectRef, sias pb.ExternalScaler_StreamIsActiveServer) error {
	return errors.New("external push is not supported")
}

func (scaler *MyRedisScaler) GetMetricSpec(ctx context.Context, scaledObject *pb.ScaledObjectRef) (*pb.GetMetricSpecResponse, error) {
	fmt.Println("2222222222222222222")
	listLength, ok := scaledObject.ScalerMetadata["listLength"]
	fmt.Println("2222222222222222222 listLength: ", listLength)
	if !ok {
		return nil, fmt.Errorf("listLength not found in metadata")
	}

	listLengthInt64, err := strconv.ParseInt(listLength, 10, 64)
	if err != nil {
		return nil, err
	}
	return &pb.GetMetricSpecResponse{
		MetricSpecs: []*pb.MetricSpec{
			{
				MetricName: "listLength",
				TargetSize: listLengthInt64,
			},
		},
	}, nil
}

func (scaler *MyRedisScaler) GetMetrics(ctx context.Context, metricRequest *pb.GetMetricsRequest) (*pb.GetMetricsResponse, error) {
	fmt.Println("3333333333333333333")
	listName, ok := metricRequest.ScaledObjectRef.ScalerMetadata["listName"]
	if !ok {
		return nil, fmt.Errorf("listName not found in metadata")
	}

	length, err := scaler.redis.LLen(ctx, listName).Result()
	if err != nil {
		fmt.Println("error: ", err)
		return nil, err
	}

	fmt.Println("3333333333333333333 length: ", length)

	return &pb.GetMetricsResponse{
		MetricValues: []*pb.MetricValue{
			{
				MetricName:  "listLength",
				MetricValue: length,
			},
		},
	}, nil
}

I placed the consumption logic in IsActive function:
	_, err = scaler.redis.Del(ctx, listName).Result()

However, I found that although the Pod can be ran normally after being woken up, the messages in the list are consumed immediately in IsActive, which leads to the HPA target field being always 0/1(avg) when KEDA updates the HPA resources:

As a result, only theminReplicaCount number of replicas can be created and the maxReplicaCount number of replicas can never be created. So, can KEDA add a feature that allows KEDA to actively consume messages of a specified length in the queue when started one Pod?

Hello

So I would like to ask if it is possible to provide the feature to actively consume the queue with keda.

Sadly, this is not possible and we don't have plans to support it as KEDA doesn't modify in any manner the messages. I can suggest different options to achieve your goals:

Giving a try to the HTTP add-on:  The community engagement is growing and the traction is increasing, so it's a good option to take into account, even more in this case where you don't expect huge loads (because you don't need a proxy in the middle handling thousands of requests)
Use an external scaler or REST api (gRPC and REST are supported by external scaler and metrics API scaler) but receiving the request directly there and not using a Redis in the middle. You could store last GET request received and based on the timestamp of it, response active or not to KEDA

However, I found that although the Pod can be ran normally after being woken up, the messages in the list are consumed immediately in IsActive, which leads to the HPA target field being always 0/1(avg) when KEDA updates the HPA resources:

I think that this doesn't matter if you want to scale 0->1->0 as the HPA works only during 1->N->1. The scaling from/to 0 is managed by KEDA operator and the cooldown period since the metric returns 0 and KEDA scales to 0 is customizable using the ScaledObject directly

Hello

So I would like to ask if it is possible to provide the feature to actively consume the queue with keda.

Sadly, this is not possible and we don't have plans to support it as KEDA doesn't modify in any manner the messages. I can suggest different options to achieve your goals:

Giving a try to the HTTP add-on:  The community engagement is growing and the traction is increasing, so it's a good option to take into account, even more in this case where you don't expect huge loads (because you don't need a proxy in the middle handling thousands of requests)
Use an external scaler or REST api (gRPC and REST are supported by external scaler and metrics API scaler) but receiving the request directly there and not using a Redis in the middle. You could store last GET request received and based on the timestamp of it, response active or not to KEDA

However, I found that although the Pod can be ran normally after being woken up, the messages in the list are consumed immediately in IsActive, which leads to the HPA target field being always 0/1(avg) when KEDA updates the HPA resources:

I think that this doesn't matter if you want to scale 0->1->0 as the HPA works only during 1->N->1. The scaling from/to 0 is managed by KEDA operator and the cooldown period since the metric returns 0 and KEDA scales to 0 is customizable using the ScaledObject directly

@JorTurFer Thank you for your advice.
I'd love to use the HTTP add-on. it really fits my scenario. In fact, if it were my own project, I would choose the HTTP add-on directly. However, I am developing a company project, so there is a little worry that my boss might blame me for using a beta version. Haha. May I ask if there are any plans to release the production version of the HTTP add-on?
And then, I want to ask a question about updating the HPA with KEDA. I have provided a video to show my case. If you have some free time, could you please take a look at this short video? I would be extremely grateful.
https://github.com/user-attachments/assets/1fad8f00-c017-4930-9682-dbf19a46985b
In this video, I started a gRPC server in my local for running a external mode KEDA ScaledObject:
apiVersion: keda.sh/v1alpha1
kind: ScaledObject
metadata:
  name: scaledobject-nettools-11
  namespace: ding-test
spec:
  scaleTargetRef:
    name: nettools
  pollingInterval: 10
  cooldownPeriod: 20
  idleReplicaCount: 0
  # initialCooldownPeriod: 10
  minReplicaCount: 1
  maxReplicaCount: 3
  triggers:
    - type: external
      metadata:
        scalerAddress: <IP_ADDRESS>:13190
        listName: nettools-test:keda:queue
        listLength: "1"
        databaseIndex: "6" # optional

I developed a simple redis scaler:
package scaler

import (
	"context"
	"errors"
	"fmt"
	"strconv"

	pb "git.dp.tech/lebesgue/launching/externalscaler"
	redisClient "git.dp.tech/lebesgue/launching/pkg"
	"github.com/go-redis/redis/v8"
)

type MyRedisScaler struct {
	pb.ExternalScalerServer
	redis *redis.Client
}

func NewScaler() *MyRedisScaler {
	return &MyRedisScaler{
		redis:                redisClient.Client(),
		ExternalScalerServer: pb.UnimplementedExternalScalerServer{},
	}
}

func (scaler *MyRedisScaler) IsActive(ctx context.Context, scaledObject *pb.ScaledObjectRef) (*pb.IsActiveResponse, error) {
	fmt.Println("1111111111111111111")
	listName, ok := scaledObject.ScalerMetadata["listName"]
	if !ok {
		return nil, fmt.Errorf("listName not found in metadata")
	}

	elements, err := scaler.redis.LRange(ctx, listName, 0, -1).Result()
	if err != nil {
		return nil, err
	}

	_, err = scaler.redis.Del(ctx, listName).Result()
	if err != nil {
		return nil, err
	}

	fmt.Println("1111111111111111111 list length: ", len(elements))

	return &pb.IsActiveResponse{
		Result: len(elements) > 0,
	}, nil
}

func (scaler *MyRedisScaler) StreamIsActive(_ *pb.ScaledObjectRef, sias pb.ExternalScaler_StreamIsActiveServer) error {
	return errors.New("external push is not supported")
}

func (scaler *MyRedisScaler) GetMetricSpec(ctx context.Context, scaledObject *pb.ScaledObjectRef) (*pb.GetMetricSpecResponse, error) {
	fmt.Println("2222222222222222222")
	listLength, ok := scaledObject.ScalerMetadata["listLength"]
	fmt.Println("2222222222222222222 listLength: ", listLength)
	if !ok {
		return nil, fmt.Errorf("listLength not found in metadata")
	}

	listLengthInt64, err := strconv.ParseInt(listLength, 10, 64)
	if err != nil {
		return nil, err
	}
	return &pb.GetMetricSpecResponse{
		MetricSpecs: []*pb.MetricSpec{
			{
				MetricName: "listLength",
				TargetSize: listLengthInt64,
			},
		},
	}, nil
}

func (scaler *MyRedisScaler) GetMetrics(ctx context.Context, metricRequest *pb.GetMetricsRequest) (*pb.GetMetricsResponse, error) {
	fmt.Println("3333333333333333333")
	listName, ok := metricRequest.ScaledObjectRef.ScalerMetadata["listName"]
	if !ok {
		return nil, fmt.Errorf("listName not found in metadata")
	}

	length, err := scaler.redis.LLen(ctx, listName).Result()
	if err != nil {
		fmt.Println("error: ", err)
		return nil, err
	}

	fmt.Println("3333333333333333333 length: ", length)

	return &pb.GetMetricsResponse{
		MetricValues: []*pb.MetricValue{
			{
				MetricName:  "listLength",
				MetricValue: length,
			},
		},
	}, nil
}

And I have a deployment which is controlled by this KEDA ScaledObject. At first, my redis-list didn't have any messages, so the Pod was scaled down to 0.:

Then I pushed 5 message into redis-list:

Thereafter, my external scaler was triggered by the order GetMetrics -> IsActive. So I read redis in GetMetrics and got 5 messages and returned by MetricValues:

Afterwards, my IsActive was invoked, I got 5 messages from redis-list. Then I deleted the redis-list by list name:

Subsequently, My Pod was woke up:

But in my ScaledObject config, I set the minReplicaCount was 1, maxReplicaCount was 3. And I pushed  5 messages into redis-list, furthermore when KEDA operator trigger my external scaler's GetMetrics method, I return the MetricValue was 5.
I have told KEDA my MetricName was listLength and TargetSize was 1 in GetMetricSpec method:

And after I sent 5 messages into redis-list, the GetMetrics method returned metric listLength was 5.
So according to my understanding, I thought KEDA would update HPA's target to 5. And Pods should be scale up to 3. But I found the HPA targets was 0/1(avg):

I don't know if it's my misunderstanding of KEDA? I originally thought that as soon as the current metrics value was returned in GetMetrics, HPA would be updated to my current metric value. But it's different from what I thought. So I want to ask if the current behavior meets expectations? When will KEDA trigger the update of HPA?
Thank you very much~ 🙏🙏🙏

Hello,

May I ask if there are any plans to release the production version of the HTTP add-on?

The OSS add-on GA roadmap is here -> https://github.com/kedacore/http-add-on/issues/911
I know that there are vendors like Kedify which offers enterprise support and GA HTTP Add-on, but I don't have any other details, just what I read from their blog.
About your scaler, you're deleting the items from Redis during is active, so GetMetrics probably is returning 0. IsActive will be used to scale from 0 to 1 the workload but then, you need to expose the current value via GetMetrics to scale from 1 to N.
Due to this is why I suggested using a timestamp to change the value

Use an external scaler or REST api (gRPC and REST are supported by external scaler and metrics API scaler) but receiving the request directly there and not using a Redis in the middle. You could store last GET request received and based on the timestamp of it, response active or not to KEDA

Hello,

May I ask if there are any plans to release the production version of the HTTP add-on?

The OSS add-on GA roadmap is here -> kedacore/http-add-on#911 I know that there are vendors like Kedify which offers enterprise support and GA HTTP Add-on, but I don't have any other details, just what I read from their blog.
About your scaler, you're deleting the items from Redis during is active, so GetMetrics probably is returning 0. IsActive will be used to scale from 0 to 1 the workload but then, you need to expose the current value via GetMetrics to scale from 1 to N. Due to this is why I suggested using a timestamp to change the value

Use an external scaler or REST api (gRPC and REST are supported by external scaler and metrics API scaler) but receiving the request directly there and not using a Redis in the middle. You could store last GET request received and based on the timestamp of it, response active or not to KEDA

I have a rough idea of what I should do. Thank you for your patient explanation. 👍

You're welcome! just let me know if you have any doubt about it 😄


Docs: Improve types and default values

Feature Request
Problem description
Currently we're very inconsistent in display of default values for properties on the DocSite.
For example,
Boolean values default to false, but this isn't clear
Some enumerations have default value in practise, but this isn't shown i.e. hAlign in Flex
Other enumerations don't and behaviour changes when property is present or not. i.e. gap in Flex.  Probably gap should have 'none' in enumeration and default to that.
Proposed solution
Review all properties and check if defaults are sensible and reflected on DocSite correctly.
Maybe create rules.  Boolean defaults to false, number to 0, enumeration to first value.
MVP

#1277

closing as the duplicate is closed


clean up tpr-storage unit tests

This is a follow up to #714 and #715 with some cleanup for both. Most of the cleanup is for what was in #715. I was inspired by @arschles testing approach in #714 where he manipulated the fake core apiserver client's in-memory storage directly to satisfy preconditions for tests. I had used an alternative approach of a longer narrative in a single TestCreateAndRead() test wherein the resources required to exist to test Get() and List() where guaranteed to exist by testing Create() first. Needless to say, I like @arschles approach better, so here I am emulating it.

All CI has passed, merging


Remove use of Properties for URL query parsing, fixes #6403

Fixes #6403
Changes proposed in this pull request:

Deprecate all methods in ParsingUtilities that return java.util.Properties.
Introduce a new method, static public Map<String, String> parseParameters(HttpServletRequest request), as a replacement for parseUrlParameters. This new method utilizes the request.getParameterMap() functionality for its implementation.
Modify all uses of parseUrlParameters to use the new method.
Adjust all necessary code to accommodate the new return type (Map<String, String>).
Deprecate all public methods ImportingUtilities that use java.util.Properties and make new versions of them using Map<String, String> instead.

When I run the tests locally with ./refine test, all tests pass, any idea why the tests fail on the CI server @tfmorris ?

The tests pass locally for me as well, but a re-run of the CI failed as
well, so it's not just a flaky test. Let me investigate...

Message ID: @.***>

Three of the Database tests fail when run in my IDE as well, so looking
into what's different among the CI, ./refine test, and IntelliJ
environments.

Message ID: @.***>

OK, I'm not sure why the test isn't failing locally (perhaps some difference in the test runner environment?), but the failure is real. I think what's needed is to update the mocks to implement HttpServletRequest.getParameterMap() rather than just HttpServletRequest.getParameter() as they do now
https://github.com/OpenRefine/OpenRefine/blob/cdbdf362952914f7284dc182084293f12bc9f3b9/extensions/database/tests/src/com/google/refine/extension/database/DatabaseImportControllerTest.java#L171-L178

The local test suite does not have all tests of the database extension enabled, because those require having a running SQL server (of different flavors).
You can see in the CI configuration how they are enabled:
https://github.com/OpenRefine/OpenRefine/blob/e6dbcfcd2619dc5a8328daf7d1de522a8369e276/.github/workflows/pull_request_server.yml#L99-L110

Thank you for your responses, but unfortunately, I am not sure how I should move forward with this? I have tried, but I don't know how I should replicate the failures locally so it becomes very hard for me to test.
One small difference between the old URL parser and the new parser is that the old parser took the last value if the key was provided multiple times. However, @tfmorris explained in the issue that parameters could not be repeated. Therefore it shouldn't be a problem to return the first value like this  return request.getParameterMap().entrySet().stream()                 .collect(Collectors.toMap(Map.Entry::getKey, entry -> entry.getValue()[0])); ?

@EliasStihl My apologies. The situation is definitely sub-optimal. The method I used to recreate the failure was to directly run the tests in DatabaseImportControllerTest using my IDE's test runner. I was using IntelliJ, but Eclipse should work as well. Without MySQL installed they won't all succeed, but they should give you enough insight to see what needs to be fixed.
The problem is not with your code, but with the test itself. It mocks getParameter(), but does not mock getParameterMap() which is what the new query parameter parser needs. Because there is other code which still depends on getParameter() they actually both need to be mocked and return the same data, so I'd suggest one or more helper methods be added to the test code.
We apparently don't have the database test environment setup documented for developers, which I will work on now, but you should be able to get away with just MySQL. Basically, you want to:

Install MySQL, then run the two commands that @wetneb mentioned above:
mysql -u root -h <IP_ADDRESS> -P <port, if non default> -proot -e 'CREATE DATABASE test_db;'
`mysql -u root -h <IP_ADDRESS> -P   -proot < extensions/database/tests/conf/test-mysql.sql
Finally, remove this line from the test configuration file https://github.com/OpenRefine/OpenRefine/blob/cdbdf362952914f7284dc182084293f12bc9f3b9/extensions/database/tests/conf/tests.xml#L42

If you get stuck, or this is too much hassle, let me know and I can fix up the PR so that we can get it merged.

Actually, I've got an easier solution. We'll just switch that test to use SQLite instead of MySQL. It doesn't look like I can push to your branch, so I've put the fix up in a branch on my repo. If you git cherry-pick this commit unto your branch, you should get test failures locally. https://github.com/tfmorris/OpenRefine/commit/89ee6050d08123155b927605827317c4a442ae2a

Once again, thank you for the incredible support! I think all tests pass now, but please let me know if there is anything you want me to change.

Okay, sorry about that, but thanks for telling me so I know how to do it properly in the future!


Restrict Firestore access

Hey there,
first: thank you for this amazing and useful application!
When I went through the firestore configuration I noticed that you use the test mode to make database read & write possible. Is there a possibility that you can make the database more safe so it cant get hijacked?
Thanks!
Greets Timo

Hi bro, maybe you can read at this documentation about Firestore Security to make more secure configuration.
I hope it can help you bro :)

Hey, thanks for your answer. Could you maybe make an example? Because when I restrict the access then the client is not working anymore because it has no permissions anymore to check for the requested user account.


[comp:menu] menu组件在 关闭后 依然保持了之前被选中项的样式

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link
https://codesandbox.io/s/oriqbn
Steps to reproduce

选中菜单的某一项
将菜单收起
展开菜单，之前被选中的项依然被标蓝

What is expected?
之前被选中的项不被标蓝
What is actually happening?
之前被选中的项依然被标蓝
Environment Info
v1.0.0-beta.0

可以通过 :selectable="false" 解决


SwaggerException.Response is empty

We have a webapp that is using our service's NSwag generated client.  This has all been working for months with NSwag 12.0.13 on Dotnet Core 2.2.  This week I updated our service and webapp to NSwag 13.2.0 on Dotnet Core 3.1 and now the client is no longer working as it previously did.
In my test case, I am calling a service API with a payload that returns:
HTTP/1.1 400 Bad Request
Date: Sat, 11 Jan 2020 17:20:25 GMT
Content-Type: application/json; charset=utf-8
Server: Kestrel
Content-Length: 157
Cache-Control: no-store,no-cache
Pragma: no-cache

{"errors":[{"id":"CAP-0001","message":"Cases search does not support searching for all cases"},{"id":"CAP-0007","message":"Tenant name must not be empty."}]}

In the webapp, we are doing the following to get the payload from the SwaggerException and deserialize it as our type (instead of using the client's generated types):
    private RestResponse<T> GetErrorsFromException<T>(SwaggerException ex)
    {
      RestResponse<T> restResponse = JsonConvert.DeserializeObject<RestResponse<T>>(ex.Response);
      restResponse.StatusCode = ex.StatusCode;
      return restResponse;
    }

This was working perfectly in 12.0.13.  We have upgraded to 13.2.0 and now, the exception's Response object is null/empty.  From debugging the generated client, I see that the ReadObjectResponseAsync() method is getting the data, deserializing it, and storing it as an Object but the Text field is now empty.  It seems like that is intentional since the second argument in the following line is string.Empty:
return new ObjectResponseResult<T>(typedBody, string.Empty);

So my question is what should I do to fix this so that I avoid breaking changes going forward?

You can set LoadExceptionResponse (or similar) to true on the client to have the old behavior (disabled for performance reasons

Thanks!


Improve use of Oauth scopes

Prior to 0.0.7 the token provider assumed the UID scope. Since 0.0.7, the API's been changed to allow using the defined scopes for event posting and streaming. The rest of the API needs to be covered to use scopes as well. Also, it's a bit implicit, eg you can't state which scopes to use where. Maybe this needs to be changed to support using custom scopes on the subscription API.

All the resources that have documented scopes in the yaml file are covered via #53, #54, #55, #56, and allow custom scopes to be set (this should allow custom access to streams as well).
There are a few issues with scopes in the current API:

a number of paths are not covered by scopes (eg metrics, registry)
some resources are using read named scopes for write access

Will file issues on Nakadi for these.


Switch the Twig integration to use non-deprecated APIs

This makes the bundle compatible with Twig 2.0 and avoids the deprecation warning in 1.21+
This requires Twig 1.12+, but Symfony already has this min requirement on Twig since 2.3.0

Thanks


Package names are suggested for service identifier

When we use service live template, package names are suggested for service name.

Fixed with https://github.com/ballerinalang/plugin-intellij/commit/6512b9d93193858ac672082ca3c5786a41eabf91.


[Bug] If latest package version is prerelease, download url for latest version fails with error

Here is a package that has no prereleases - http://nuget.org/api/v2/package/roundhouse/
The api downloads the latest version of the feed.
Here is a package that has a prerelease as the latest version - http://nuget.org/api/v2/package/chocolatey/
The operation failed: The relationship could not be changed because one or more of the foreign-key properties is non-nullable. When a change is made to a relationship, the related foreign-key property is set to a null value. If the foreign-key does not support null values, a new relationship must be defined, the foreign-key property must be assigned another non-null value, or the unrelated object must be deleted.

This behavior is the same on v1 version of the feed.

I haven't had a chance to dive into the code yet.

If you put the version in the end, the download works fine...

@ferventcoder So, the download link intentionally does not include pre-releases unless you specify the version (we figure without the version, you want the latest stable version). There was a bug where were doing something that made EF angry, and it turns out we shouldn't have, and it only manifested after getting a pre-release package and then performing a write (in this case, adding the download record).
So, the bug will be fixed in preview tomorrow (and I'll probably deploy to production tomorrow, too), but the behavior will be you'll get the latest stable, not the latest if the latest is pre-release. That make sense?

That's exactly what I'd expect. The latest stable package.


add new school

we need to add new school : Ain Shams University - bus.asu.edu.eg
كلية تجارة عين شمس

@mmoniem I am sorry, the domain asu.edu.eg and all its subdomains are removed from our trusted domains list because they were abused in the past.
I have to close this request without merging.
If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Fix dutch translation

The text 'vandaag' means 'today' which is not accurate enough.
The text 'now' should translate to 'nu'

Sorry for the delay on this. It is now in the dev branch


@17 Created a contact us page frontend

Hello PA @subhadipbhowmik ,
I created a contact Us page frontend for the project . Could you kindly review and approve if its alright.

@sanjay-kv @subhadipbhowmik kindly review and merge this as this is clean and a clear frontend for contact us page. Also kindly label this under gssoc extended and hactoberfest. Thank you


flow Restore-from-private-key

The route after finishing this flow > back where you came from in the first place
Add the default avatar+username when account created out of this flow
Remove all extra spaces out of the input-field

The classname of the component display-restore-private-key.html is wrongly DisplayMyInfo - I'll change that.

the above is fixed in this commit : https://github.com/swarmcity/SwarmCitySite/commit/717269ae638d9fd4882a0616aff32d03ba89c55c


Move all require statements to use custom errors

Can I work on this issue?

Would recommend using custom errors with require statements given that all compile processes now support it.
error SomeError();
require(false, SomeError());

Would recommend using custom errors with require statements given that all compile processes now support it.
error SomeError();
require(false, SomeError());

Thanks arr00.


How to get Now formatting without opening spreadsheet?

Hoping this is a simple question. The auto-formatting performed when using the Now option works perfectly for what I need. How do I get the same formatting without opening the spreadsheet?

I'm not sure what you mean by "without opening". The -Now sets a few things, -AutoSize, -TableName Table and -AutoFilter.

It was the TableName that got me. Figured it out now. Thanks again.  Works great.


[R-package] lgb.importance(): Error: R character strings are limited to 2^31-1 bytes

Hello!
I built a lightgbm model, and then I used the lgb.importance(model).
finally, the r show the Error: R character strings are limited to 2^31-1 bytes.
how do I solve this error? Thank you!

Thanks for using LightGBM.
An error message alone is not enough information for us to help you. Please provide the following:

version of R
version of {lightgbm}
how you installed LightGBM
operating system
output of running sessionInfo() in your R session (if possible)
a minimal, reproducible example that generates this error (docs with some guidance on that)

Here's an example of how to create a reproducible example for the R package: https://github.com/microsoft/LightGBM/issues/4721#issue-1036595701

version of R: 4.3.2
version of {lightgbm}: 3.3.5
how you installed LightGBM: Tool>Install package
operating system: windows
output of running sessionInfo() in your R session (if possible)
R version 4.3.2 (2023-10-31 ucrt)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 19045)
Matrix products: default

Thank you for that!
Tool>Install package

What does this mean?
version of {lightgbm}: 3.3.5

Can you please update to the latest version (v4.3.0) from CRAN and try again?
install.packages("lightgbm", repos = "https://cran.r-project.org")

And after that...we still won't be able to help much without a minimal, reproducible example.

my lightgbm package version is 3.3.5
it is still have an error. and I will try reproducible example

train model
model = lgb.train(params = list(objective = "regression",
num_iterations = 100,
metric = "l2",
min_data = 1L,
min_data_in_bin=100,
min_gain_to_split = 10),
data = train,
nrounds =  100))
lgb_imp <- lgb.importance(model)
otherwise, when I decrease the  num_iterations, the will not show the error.

my lightgbm package version is 3.3.5

Sorry if my placement was confusing. I'm asking what "Tool>Install package" means. Are those buttons you're clicking in an application? If so, what application?

train model

Thanks for this! But it is not a reproducible example.
Crucially... what does train contain? Much of LightGBM's behavior (like any machine learning framework) is dependent on the size, shape, and distribution of the input data.
For example, based only on the error message you've provided, I can think of a few possibilities:

your data has features with huge feature names
your data has a very large number of rows
you have a very large number of features

If you can't provide a reproducible example, can you please at least show the code you used to construct train? Including any code for reading in data from files, databases, etc.
And report the size of the dataset (number of rows, number of columns, exact feature names if there are any).

yes, I click the buttons in R studio to install the lightgbm package.
data <- readRDS("D:/data.rds")
dtrain <- lgb.Dataset(data=as.matrix(data[,-1]), label = data[,1])
model = lgb.train(params = list(objective = "regression",
num_iterations = 100,
metric = "l2",
min_data = 1L,
min_data_in_bin=100,
min_gain_to_split = 10),
data = train,
nrounds = 100))
lgb_imp <- lgb.importance(model)
my dataset:
number of rows: 4610000
number of columns: 21

Hi @jameslamb, I'm having the same issue when running lightgbm::lgb.model.dt.tree(lgb_model) and I believe the culprit is here lgb.dump(booster = model, num_iteration = num_iteration). My dataset is also very large with a large number of rows and columns, and is fit with a complex model (i.e., not easy to share).
It looks like lgb.dump is trying to return a single long character string, when num_iteration = NULL, I wonder if instead it could be iterated into a large list or something? i.e., lapply(1:booster$current_iter(), booster$dump_model)

I'm closing this in favor of #6380, which describes the same problem thoroughly with a reproducible example. Let's please focus there.


Add refactoring action to export selected JSX code as stateless component

Please add refactoring action that will improve productivity while working with JSX/TSX. Suppose that I have the following code:
class Page extends React.Component {
    render() {
        return (

{this.props.header}

        );
    }
}

I would like to select line with
 and chose from context menu "Extract to stateless component".  And that would create the following stateless component:
const NewComponent = ({ header }) => (
{header}
);

The selected text would be replaced with that:
<NewComponent header={this.props.header} />

It would also be fine if it can convert component class into stateless component if possible and reverse convertion
class Page extends React.Component<PageProps, {}> {
    render() {
        return (

{this.props.header}

        );
    }
}

// refactor converts to

function Page({header}: PageProps) {
    return (

{header}

    );
}

I think it possible to do with language service's plugin
https://github.com/RyanCavanaugh/sample-ts-plugin

I have a partially working version of this, but it was hard to get this working without #15868. The problem was accomodating propTypes.

@DanielRosenwasser - is it because after extracting selected text to stateless function you also want to generate propTypes? If so then maybe it would be nice to generate that without propTypes - as far as I know it's not necessary to have it

This should no longer be blocked, so if you already have a quickfix, let's see it. 😄

This is very similar to https://github.com/microsoft/TypeScript/issues/25891.
I made a proposal here: https://github.com/microsoft/TypeScript/issues/25891#issuecomment-839682315


heatmap for users, programs, data

Nice feature that would build on our activity log
https://github.com/DKirwan/calendar-heatmap

This is actually most useful for Dr Elephant:
https://content.linkedin.com/content/dam/engineering/site-assets/images/blog/posts/2016/04/dr--elephant/DrElephant-6-jobhistory.jpg


Update Caddy to v2

Currently the Caddy image that is used is on v1. Since then v2 has been released which is much more stable. It would be helpful if this could be ported to use the official v2 image.
Also there have been issue with older version of Caddy and newer version of Docker(See https://github.com/stefanprodan/dockprom/issues/203).
I would like to take up the issue and create a PR if approved.

@bhumijgupta looks like someone has already attempted this at #207.
Please follow progress at that PR.

@bhumijgupta looks like someone has already attempted this at #207.
Please follow progress at that PR.

@nightah The PR seems to be stale. I'll open a new PR with the suggested changes and some more minor changes


Can't handle lock with complex commands

    command => '(/usr/bin/docker run --rm busybox ping -c 4 <IP_ADDRESS>) || /nail/sys/bin/graceful_docker_restart',
    lock    => true,

Produces:
kwa@srv5-r6-sfo2:/etc/cron.d$ cat graceful_docker_restart_if_broken
# WARNING - this file is managed by puppet - do not edit!

MAILTO=""
SHELL=/bin/bash
PATH=/usr/local/bin:/usr/local/sbin:/nail/sys/bin:/usr/sbin:/usr/bin:/sbin:/bin
34 * * * * root (flock -n "/var/lock/cron_graceful_docker_restart_if_broken.lock" (tar cv --files-from /dev/null | docker import - end2endtest:latest) || /nail/sys/bin/graceful_docker_restart) 2>&1 | logger -t cron_graceful_docker_restart_if_broken

kwa@srv5-r6-sfo2:/etc/cron.d$ (flock -n "/var/lock/cron_graceful_docker_restart_if_broken.lock" (tar cv --files-from /dev/null | docker import - end2endtest:latest) || /nail/sys/bin/graceful_docker_restart)
-bash: syntax error near unexpected token `('

I don't know what to do here. More quotes?

I've worked around this with bash -c ""


Multi-cursor support

Are there any plans to support multiple cursors in the future?

Absolutely, as I'm a big proponent of multi-cursors. Should be the next improvement after support for more languages (already working on that, should be ready really soon).


Add support for snowflake.yml in procedures

Pre-review checklist

[ ] I've confirmed that instructions included in README.md are still correct after my changes in the codebase.
[ ] I've added new automated tests to verify correctness of my new code.
[ ] I've confirmed that my changes are up-to-date with the target branch.
[ ] I've described my changes in the release notes.
[ ] I've described my changes in the section below.

Changes description
...

looks good to me

Please add release notes.


[Codeceptjs] instead of error message get an 500 error

if in main VRT project not increase value of BODY_PARSER_JSON_LIMIT get an 500 error after tests running:
[Wrapped Error] "Request failed with status code 500"
I vrt track "Default"
  › Screenshot is saving to<EMAIL_ADDRESS> [1] Error | Request failed with status code 500
 Emitted | step.failed (I vrt track "Default")
 Emitted | step.finish (I vrt track "Default")
[1] Error | Error: [Wrapped Error] "Request failed with status code 500"
Emitted | step.failed (I vrt track "Additional options",
{"os":"windows","device":"device","browser":"chrome","diffTollerancePercent":13})
  Emitted | step.finish (I vrt track "Additional options",
{"os":"windows","device":"device","browser":"chrome","diffTollerancePercent":13})
 [1] Error | Error: [Wrapped Error] "Request failed with status code 500"
  [1] Starting <teardown> session
 Emitted | test.failed ([object Object])
 Emitted | test.finish ([object Object])
[1] <teardown> Stopping recording promises
docker-compose logs
Nest] 17   - 09/03/2020, 1:29:10 PM   [ExceptionsHandler] request entity too large +9633002ms
PayloadTooLargeError: request entity too large

I'm also receiving this, from a pretty straight-forward test case (Load site, take screenshot.)

So this relates to NestJS's config, which can be modified from the .env file.
The value that needs to change is BODY_PARSER_JSON_LIMIT
By default, it's set to 1mb. That doesn't seem sufficient, or the body that's being parsed is unusually large for some reason. I set mine to 500mb to be ridiculous.

Next version will contain error message with stacktrace instead of just 500
this will be done in scope of https://github.com/Visual-Regression-Tracker/Visual-Regression-Tracker/issues/128
so this is more the issue of not descriptive error


warn if the output path might be too long

If the output path is too long, which often happens in multi-variable sweeps, Windows 10 rejects the nextnano simulation without nextnanopy knowing the issue.
This pull request is an attempt to raise a warning in such cases. The warning message is readable if show_log = False, but otherwise immediately flows away in the console.

LGFM! In case the problem with too long path occurs because of the naming of swepp, I recommend to use
Sweep.save_sweep(integer_only_in_name = True)


[Remove]: Removed discontinued course link

Removal of the project link which is now discontinued by the author.
Description
The project is discontinued and outdated(as said by the author himself).
Motivation and Context
As stated by the author in the project link that the tutorial is discontinued and outdated.
Types of changes

[x] Content Update (change which fixes an issue or updates an already existing submission)
[ ] New Article (change which adds functionality)
[ ] Documentation change

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have made checks to ensure URLs and other resources are valid

Fixed in a previous PR.


IEM always input in advance

VSCode Version: 1.40.1
OS Version: Window7 X64

When I use QQ IME to input Chinese, Notepad ++ and SublimeTetx3.0 will not output in advance. Only after I confirm the input, what I want to enter will appear in the editor, but in VSCode, it will always be in When I have n’t decided what to enter, I wrote the content into the editor. The picture below shows the performance of the input method in Notepad ++ and VSCode. It can be found that Notepad ++ will only enter the content into the editor after the selection is determined：

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

This is simply how web browsers work, since we run on Electron which is built with Chromium, we work like web browsers.

@alexandrudima Thanks for your reply, I tested it and found that the input of Chrome is indeed like this, but when I use IE and Firefox, and some other Webkit Core browsers, their effects is similar to Notepad++ instead of VSCode, maybe These browsers have made special adjustments for IME, so can I also set them in VSCode to achieve effects like Notepad++ 😲 ?


Remove Intercom

Should be quick with segment.

:tada: :balloon:

Let's be sure to update the corresponding welcome drip email. We can still send it through Intercom, or move it to Customer.io. @wcpines

aye-aye

Keep it in intercom but convert to email instead of In-app. SG @chasballew  ?


Disable IMA in Kernel config

We don't use it, we don't rely on it (we bring trust over from locking down the filesystem), but it might make our life more difficult by measuring things we don't expect.
Also, less code more better.

Related replacements for integrity:
https://github.com/monogon-dev/monogon/issues/108
https://github.com/monogon-dev/monogon/issues/109


Merge pull request #1 from edx/master

Pull from original to fork Jan 2017
Description
MA-XXXX
Add a description of your changes here.
Notes

Example: Use example.sandbox.edx.org to test against
Example: This PR will not address x,y, and z, because of a, b, and c.
Example: Feature is behind a flag
Example: Why I didn't add any tests

Testing

[ ] i18n
[ ] RTL
[ ] safecommit shows 0 violations
[ ] Unit, integration, acceptance tests as appropriate
[ ] Analytics
[ ] Performance
[ ] Database migrations are backwards-compatible

Reviewers
If you've been tagged for review, please "Start a review" with the GitHub GUI.

Code review: @Reviewer1, @Reviewer2
UI strings/error msgs review: @...
UX review: @...
Accessibility review: @...
Product review: @...
FYI @someone

Thanks for the pull request, @neevknowledge! I've created OSPR-1636 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.
Feel free to add as much of the following information to the ticket:

supporting documentation
edx-code email threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.
We can't start reviewing your pull request until you've submitted a     signed contributor agreement     or indicated your institutional affiliation.     If you like, you can add yourself to the AUTHORS      file for this repo, though that isn't required.     Please see the CONTRIBUTING     file for more information.

Current coverage is 17.33%

Merging #899 into master will not change coverage

@@             master       #899   diff @@
==========================================
  Files           467        467
  Lines         19619      19619
  Methods           0          0
  Messages          0          0
  Branches       2881       2881
==========================================
  Hits           3401       3401
  Misses        15832      15832
  Partials        386        386

Powered by Codecov. Last update a7b66e0...321fd36

@neevknowledge Looks like this PR was unintentional. Closing.


use more exact return type in Notification#do

Description:
This prevents a return type of any when the type can be inferred from callback types.

// Before: return type is always `any`

// ============

// After:

// when all 3 callbacks are specified: return type becomes (union of their return types)
const a = notification.do(
  _ => 'str',
  _ => 123,
  _ => ({a: 1})
);  // => a: string | number | {a: number}

// when not all of callbacks are specified:
// return type becomes (undefined | union of specified return types)
const b = notification.do(
  _ => 'str'
); // => b: string | undefined

Related issue (if exists):

Closing this in favor of #5478
Thank you for your effort, @jokester ... it's helpful and it drew our attention to the issue.


Cache - add a new test that has B2B write hit, and a test with B2B Read Hit. (delay(0))

To get the hit, first, we must get the data from FM with a RD/WR miss - wait ~20 cycles, then we can start the B2B sequence
(Note - the PP checker wont work for B2B - first you must fix the #178 issue.)

Done
Tests names:
wr_b2b_hit
rd_b2b_hit


commented a print and converted to func

print resp.text => # print (resp.text) on ln 78

no merge?

Sorry about that - thought I did


Provide ways to customize spans in reactor

I have the spring reactor app, and would like to customize spans, generated automatically.
I enabled sleuth:
spring.sleuth.enabled: true
spring.sleuth.reactor.enabled: true

and my code like the following:
@EventListener(classes = ApplicationReadyEvent.class)
    public void executeSendingNotificationToServer() {

   return Mono.fromRunnable(() -> {
     tracer.currentSpan().tag("sample-key", "sample-value");
     tracer.currentSpanCustomizer().tag("sample-key", "sample-value2");
   })
   .then(<other business logic>)...

I can observer traces in jaeger , but they don't contain 'sample-key' tags at all.
How to properly customize span in spring reactor?

Are you sure that the span is sampled?

Are you sure that the span is sampled?

Yes. I have jaeger server running, and in jaeger UI i can see only 2 default tags, please take a look:

Can you log out the span from the runnable code?

I added some debug logging to my method:
`@EventListener(classes = ApplicationReadyEvent.class)
public void executeSendingNotificationToServer() {
return Mono.fromRunnable(() -> {
Span span = tracer.currentSpan();
if (span != null)
log.debug((span.toString() + " span with id: " + span.hashCode()));
tracer.currentSpan().tag("sample-key", "sample-value");
tracer.currentSpanCustomizer().tag("sample-key", "sample-value2");
})
.then()...Portion of logs:
2020-08-17 10:50:33.138 DEBUG [notifications-gw,0a2544a5dbc01d8d,0a2544a5dbc01d8d,true] 11344 --- [     parallel-2] r.s.s.n.i.s.NotificationServiceExecutor  : LazySpan(0a2544a5dbc01d8d/0a2544a5dbc01d8d) span with id: 604568545
2020-08-17 10:50:33.196 DEBUG [notifications-gw,,,] 11344 --- [ender@6c9074e4}] .s.ZipkinRestTemplateSenderConfiguration : The port in Zipkin's URL [http://localhost:9411] is provided - that means that load balancing will not take place
2020-08-17 10:50:33.196 DEBUG [notifications-gw,,,] 11344 --- [ender@6c9074e4}] o.s.c.s.z.s.ZipkinRestTemplateWrapper    : HTTP POST http://localhost:9411/api/v2/spans
2020-08-17 10:50:33.196 DEBUG [notifications-gw,,,] 11344 --- [ender@6c9074e4}] o.s.c.s.z.s.ZipkinRestTemplateWrapper    : Accept=[text/plain, application/json, application/*+json, /]
2020-08-17 10:50:33.196 DEBUG [notifications-gw,,,] 11344 --- [ender@6c9074e4}] o.s.c.s.z.s.ZipkinRestTemplateWrapper    : Writing [[B@1cd9dedc] as "application/json"
2020-08-17 10:50:33.198 DEBUG [notifications-gw,,,] 11344 --- [ender@6c9074e4}] o.s.c.s.z.s.ZipkinRestTemplateWrapper    : Response 202 ACCEPTED
`

Most likely it's the reactor instrumentation issue.
In Sleuth 3.0.0 we'll give the user much more freedom to operate on reactor via https://docs.spring.io/spring-cloud-sleuth/docs/3.0.0-SNAPSHOT/reference/html/#webflux-support or https://docs.spring.io/spring-cloud-sleuth/docs/3.0.0-SNAPSHOT/reference/html/#spring-cloud-function-and-spring-cloud-stream
You can try turning off the decorate-on-each feature until you migrate https://docs.spring.io/spring-cloud-sleuth/docs/current/reference/html/#project-reactor

I'm going to turn off sleuth on reactor:
spring.sleuth.reactor.enabled: false
And manually create spans and finish them in flux.
Do you find it correct way of tracing reactor?

The default instrumentation should work fine most of the time. Without a sample it's hard for me to reason about what's going on exactly. For sure if you take care of the tracing manually it should work better cause you pass on the tracing stuff in an explicit manner.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.

@marcingrzejszczak  Hello
can you please look at https://github.com/spring-cloud/spring-cloud-sleuth/issues/1741

Please do not cross post. I'm looking into the issues as soon as possible.


Push the image after building it CI

is this still open?

Does this imply a bacalhau docker container for each commit in main?


[feat] Add new decorator assign_units for unit processing on input and output for a function that does not support unit operations

This pull request introduces a new decorator function assign_units in the brainunit module to handle unit transformations for function arguments and return values. It also includes tests for this new functionality. The most important changes include adding the assign_units function, updating the _base.py file to include the new decorator, and adding corresponding tests.
New functionality for unit transformation:

Added assign_units decorator function to transform units of arguments passed to a function. (brainunit/_base.py)

Codebase updates:

Included assign_units in the list of functions for checking units. (brainunit/_base.py)
Added _assign_unit helper function to handle unit assignment within the assign_units decorator. (brainunit/_base.py)

Testing:

Added test_assign_units method to test various scenarios for the assign_units decorator, ensuring it correctly handles unit transformations and raises appropriate errors. (brainunit/_base_test.py)

https://github.com/chaobrain/brainunit/pull/72


Larvel Session file is not supported

When starting "php artisan serve", localhost:8000 says that /koel/config/session.php (I cloned into /koel) is not supported
InvalidArgumentException in compiled.php line 12040: Driver [/koel/config/session.php] not supported.

and
InvalidArgumentException in compiled.php line 12040: Driver [/koel/config/session.php] not supported.

The exact error is posted below. I am not familiar enough with the Larvel framework to know how to fix it, so I've been doing some google foo to try and solve the problem. I haven't been able to find a solution so far.

Ok, I just solved it, but I don't know if it is the way that it should be solved. In the ./config/session.php file, I changed 'driver' to
'driver' => 'native',

I also had to do the same with the cache.php file
'driver' => 'file',

That's weird, but glad you sorted it out. I'm closing this issue, but feel free to amend it or open a new one should you have any new findings.


Implement 'INativeValue' interface

I've implemented the proposed interface as disccused in issue #438. Native::ObjectToNative will now check if the object implements the INativeValue or IHandleable interface to retrieve the appropriate value.
I've also updated a couple of classes to inherit from some common interfaces (e.g. System::IEquatable<T>, IHandleable, etc). I figured I'd do some code cleanup while I was at it, so there's quite a few updates as well. Everything should still be 100% backwards compatible.

I'm still not convinced we need INativeValue when the one and only class using it is Model.

@crosire Funny you mention that, #441 actually added a new value type that is the perfect candidate for INativeValue :P

I'm planning to add AnimationDictionary and AnimationSet, so I'd like to implement IAsset(, IHashedAsset & INamedAsset), rather than INativeValue. http://docs.ragepluginhook.net/html/AB7AB5A4.htm

Ok, I'll remove INativeValue for now.


Auto complete for file/path name and pandas column name with bracket access

In Juypter notebook, when you are trying to read a file with command df = pd.read_csv('./myfile.csv')
for the file name, you can just type './m then press TAB to get a list of your file that you can pick from.

In Jupyter notebook, when you are typing the name of a column, e.g. you are accessing df['date_start'], while you are typing df['d then you press TAB, it's will show the option or autocomplete for you,

I think it's would be nice if we have this feature in VSCODE

Both of these rely on runtime data, things we don't have access to as a static analyzer. Jupyter itself has this data in memory and should already be suggesting it. Are you not seeing these completions when using the Jupyter extension in a notebook?

Both of these rely on runtime data, things we don't have access to as a static analyzer. Jupyter itself has this data in memory and should already be suggesting it. Are you not seeing these completions when using the Jupyter extension in a notebook?

Both of these rely on runtime data, things we don't have access to as a static analyzer. Jupyter itself has this data in memory and should already be suggesting it. Are you not seeing these completions when using the Jupyter extension in a notebook?

I can't see these completions when using Jupyter in VSCODE.
I'm not sure if I use VSCODE correctly, below are my steps for this issue.

Create a new folder
Create Python virtual environment in this folder
Open VSCODE
Select Python interpreter.
Install pandas to the environment
Create a CSV file
Create a .ipynb file, click on it
Get prompt to install ipykernel
Press install
Create some codes(as shown in the images below)
Try to get autocompletion while pd.read_csv
Try to get autocompletion while typing

Reference images.

I expected to see filename autocompletion here.

I expected to see column name autocompletion here.

Both of these rely on runtime data, things we don't have access to as a static analyzer. Jupyter itself has this data in memory and should already be suggesting it. Are you not seeing these completions when using the Jupyter extension in a notebook?

I can't see these completions when using Jupyter in VSCODE.
I'm not sure if I use VSCODE correctly, below are my steps for this issue.

Create a new folder
Create Python virtual environment in this folder
Open VSCODE
Select Python interpreter.
Install pandas to the environment
Create a CSV file
Create a .ipynb file, click on it
Get prompt to install ipykernel
Press install
Create some codes(as shown in the images below)
Try to get autocompletion while pd.read_csv
Try to get autocompletion while typing

Reference images.

I expected to see filename autocompletion here.

I expected to see column name autocompletion here.

Can you try manually triggering a completion by hitting Ctrl+Space inside of one of those strings? And is this the new or old notebook system in the Jupyter extension?

Can you try manually triggering a completion by hitting Ctrl+Space inside of one of those strings? And is this the new or old notebook system in the Jupyter extension?

The pandas part is a dup? https://github.com/microsoft/pylance-release/issues/102

Yes, it is.
As for the file-path one, I can't find it, but I believe that's a feature request we said we weren't going to do either (as it escapes the bounds of what's reasonable in a python analyzer). For notebooks, they already handle this because jupyter has this built-in and already accesses loads of runtime data.


[BUG]: quicktype version 23.0.161 command not found

When installing version 23.0.161 you can't execute the quicktype command. It still works with version 23.0.158.
npm WARN exec The following package was not found and will be installed<EMAIL_ADDRESS>sh: quicktype: command not found

@OracleDin Can you try again with 23.0.162 ?

Now it works! We were working with latest in my project, now its resolved. Thanks alot :)

Resolved as of 23.0.162


Add guards for names max length

names in postgres have a max length of 63 chars by default. We generate names in many places, like:

Version schema names
Temporary table/column names
indices
triggers
etc

We should make sure we don't generate names longer than the expected max, as that could cause issues down the line.
https://www.postgresql.org/docs/current/sql-syntax-lexical.html#SQL-SYNTAX-IDENTIFIERS

I would like to work on this one.


Feature request: add jib.skip configuration option to the gradle plugin

Environment:

Grade plugin: 3.1.4
Build tool: Gradle version 6.5.1
OS: MacOS

Description of the issue:
In Jib maven plugin, there is a configuration option for jib.skip to skip executing Jib in library module in a multi-module project. Original request for this feature in maven plugin: https://github.com/GoogleContainerTools/jib/issues/865
This option is not available in Gradle plugin
Expected behavior:
jib.skip configuration option is also available to gradle plugin. Yes, I know I can do :{service}:jib but it may not be an option in a CI/CD with standardize pipeline build process.
Steps to reproduce:

Add jib.skip into build.gradle in module directory (e.g. client-lib)
Run ./gradlew build jib
Jib task should ignore client-lib module

build.gradle Configuration:
jib {
   skip = true
}

Log output:
What went wrong:
A problem occurred evaluating project ':client-lib'.
Could not set unknown property 'skip' for extension 'jib' of type com.google.cloud.tools.jib.gradle.JibExtension.

Hi @wlchan, thank you for raising this feature request. We'll look into this 👍🏻

The jib.skip-like feature is already doable with Gradle. (In fact, anything is possible in Gradle.) You can just disable whatever tasks in question (whether jib, jibDockerBuild, or jibBuildTar) by setting enabled to false.
However, if you find yourself wanting to do that, it's likely that your project is not correct. Just don't apply the Jib plugin to the library module. jib.skip is only there for Maven, due to the particular way Maven works as discussed in #865. As explained in this FAQ, you should structure your project in this way:

define all of your plugins in the base project and apply them selectively in your subprojects as needed.

tasks.findByName("jib").setEnabled(false)


Migrate file paths to QDir

This is going to be needed if/when we eventually support Windows due to the path separator being different. We might as well do this now and have it in place so it's there when we need it. Doing do requires that we bump the C++ standard up to C++17 but I don't anticipate that being an issue. We should be able to build the paths with std::filesystem::path and convert it into a String/c_str which can be fed into the constructors that currently use the path.

It looks like Qt has it's own QDir class that handles correctly using directory separators depending on the operating system. Let's use that instead of std::filesystem:path instead to make this easier to compile.

Yeah, I almost responded with something similar, but figured std::filepath might be safer/more universal


[AES-CBC] Add some standard/recommended NIST tests for AES256

Description
While our core/crypto/primitives has a pair of PKCS7Padding and PKCS7UnPadding which is used extensively. We want to add more unit tests to lock/document and verify their correct/expected/standard/compliant behavior.
In particular, test using a few recommended test-vectors which NIST recommended and some of these variants.
This is part of #2127 Add a family of AES-CBC related tests
Describe How to Reproduce

N/A. Should be covered in the future by make unit-test

Hi @JonathanLevi , I can take this up (with some help)
Sudeepta

Hi @sudeepta-bhuyan as per our chat on Slack, this item has some internal dependencies, etc... as in order to get these in we are working on refactoring the underlying package.
So, thanks again, and we'll discuss some others items/issues next week!


FadeIn/FadeOut effects mess up the channel volume after playback is finished

If a channel is done playing before fade_length is reached, its volume is never restored to fade_volume_reset. This messes up the volume for the next chunk to be played on the same channel.
Here, the volume is restored upon reaching fade_length. But here, the channel is done playing, its fading is set to MIX_NO_FADING and the volume is never restored. The problem is, in the next call to Mix_PlayChannel() on the same channel, the volume is stuck at whatever value was at the end of previous playback.
I believe there would be the same behaviour for expired channels.
Is this a bug? And what's the best way to deal with this problem when Mix_MasterVolume isn't available and the global volume is controlled by Mix_HaltChannel(-1); Mix_Volume(-1, volume)?
Potentional workaround 1
auto ms_to_fade = min(fade_length, chunk_ticks_left);
Mix_FadeOutChannel(channel, ms_to_fade);

In this case, the condition for restoring the volume will be fulfilled, but I don't know how to get or calculate chunk_ticks_left, as even SDL_Mixer doesn't know or expose how many ticks are left before chunk is done playing.
Potentional workaround 2
if ( Mix_Playing(channel) == true )
{
  auto volume = Mix_Volume(channel, -1);
  Mix_FadeOutChannel(channel, ms_to_fade);
}

// elsewhere
if ( Mix_Playing(channel) == false )
{
  Mix_Volume(channel, volume);
  Mix_PlayChannel(channel, chunk, 0);
}

This is only possible with dedicated channels, and it's a bit more error-prone, as the channel volume must be properly initialized before first playback.

Here's how to reproduce:
Mix_Volume(0, 100);
auto volumeBeforeFadeOut = Mix_Volume(0, -1);

// sample.wav contains 1 second of white noise
Mix_Chunk* sound = Mix_LoadWAV( "sample.wav" );
Mix_PlayChannel(0, sound, 0);

using namespace std::chrono_literals;
std::this_thread::sleep_for(500ms);

Mix_FadeOutChannel(0, 1000);

// we can increase sleep time to 1100ms,
// it won't change anything
std::this_thread::sleep_for(600ms);

auto volumeAfterFadeOut = Mix_Volume(0, -1);

Mix_PlayChannel(0, sound, 0);

auto volumeForNextPlayback = Mix_Volume(0, -1);

std::cout << volumeBeforeFadeOut << "\n";
std::cout << volumeAfterFadeOut << "\n";
std::cout << volumeForNextPlayback << "\n";

// I suppose this should be true
assert(volumeBeforeFadeOut == volumeAfterFadeOut == volumeForNextPlayback);


AttributeError: 'Xception' object has no attribute 'fc'

It seems you have notice that in xception:
model.last_linear = model.fc
del model.fc

but in xception.logist:
output = self.fc(x)
It seems a bug?

Thanks for submitting this issue. It was definitely a bug.
I just fixed it: https://github.com/Cadene/pretrained-models.pytorch/commit/f2ede774b06d2006c02cc6fb0148f719be560038
pip install --upgrade pretrainedmodels


Add an unsharp mask node to core nodes

Unsharp mask is an image operation that, despite its name, sharpens an image. Like a Gaussian blur, it takes a radius and strength.
What type of PR is this? (check all applicable)

[ ] Refactor
[X] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[X] Yes
[ ] No, because:

Have you updated all relevant documentation?

[ ] Yes
[ ] No

Description
Adds an unsharp mask node to core nodes.
Related Tickets & Documents

Related Issue #
Closes #

QA Instructions, Screenshots, Recordings

Added/updated tests?

[ ] Yes
[X] No : please replace this line with details on why tests
have not been included

[optional] Are there any post deployment tasks we need to perform?

We've added a classification arg to the @invocation decorator. This indicates the feature classification of the invocation: Stable, Beta or Prototype.
Can you please add classification=Classification.Beta to this node? See the example in this PR where I'm doing the same: #5288

Done.


make kubectx / kubens kubectl krew plugins

I love kubectx and kubens ! -- It saves me a ton of time, and I've been using it for a long time already.
Now there is also a new kid on the block. "Krew" https://github.com/kubernetes-sigs/krew. Krew is a plugin manager for kubectl and kubectx and kubens are excellent candidates for it.
Since kubectx and kubens work well, and are well tested I think it's worth porting them to this plugin system.
to be commands could be: kubectl ctx and kubectl ns These commands are available still.

Haha, I wonder who develops krew. They should include it!

They are currently implemented as krew plugins: https://github.com/kubernetes-sigs/krew-index/blob/master/plugins.md

Awesome guys!! Thanks!


Starfield missing files issue

I have been checking out why I don't always get my Starfield saves when I use the script. After doing some deduction it seems that when you save in game it doesn't actually create the save file in the "...\SystemAppData\wgs..." but somewhere else (idk where). A folder for the save file will be created instantly in the "/wgs" directory without the actual save data. Then with some Xbox "magic" the save file/data, after some delay, is moved to the "/wgs" directory in the background. My best guess is that it has something to do with cloud backup. The save files transfers are done individually and has a couple minutes delay between each of them.
Basic fix: Wait 10 to 20 minutes (depending on how many saves are pending) then run the script again.
Should probably be a warning for the user when save data is missing that the data is still being (cloudified?) by Xbox and that the user should wait and try running the script again after a few minutes

I added some warnings about missing files when running the script. See the newest release and 1e07f59.


[FEATURE] Multi Platform Support

Preface
I was surprised to see this uses an external application. You don’t see many tools intended for VRChat development built that way.
Request
Builds for Linux and macOS.
Additional Context
I can help test macOS builds.

I did a little digging and it looks like python doesn't allow cross-compilation (or whatever the equivalent would be for python) link.
So adding in installation instructions might be the easiest way, especially without knowing which linux distro the app would be run on. I could try and create installers for specific distros on request in the meantime.

I added an install and run script for running on Ubuntu which should be easily modified to run on other distros (but I'm unsure of how it would work on MacOS). Also added platform support in the code for Linux/MacOS.
It's on the dev branch right now if you wanted to test it. Just have to cd into the UdonProfilerApp directory and you'll find the install and run scripts there.
I wasn't able to test the unitypackage on Linux because I was having problems getting unity 2019.4.31f1 LTS running on my VM. Not sure what the issue is as newer versions run fine but are incompatible with the VRC sdk.
Let me know if it works locally for you (as well as if the scripts need any changes for MacOS).


[Doc] 2.4 update navigation

What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[x] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.21 out of 22 committers have signed the CLA.:white_check_mark: stdpain:white_check_mark: sevev:white_check_mark: Smith-Cruise:white_check_mark: Astralidea:white_check_mark: LiShuMing:white_check_mark: Seaven:white_check_mark: amber-create:white_check_mark: decster:white_check_mark: EsoragotoSpirit:white_check_mark: andyziye:white_check_mark: packy92:white_check_mark: luohaha:white_check_mark: meegoo:white_check_mark: trueeyu:white_check_mark: gengjun-git:white_check_mark: hellolilyliuyi:white_check_mark: ZiheLiu:white_check_mark: xiaoyong-z:white_check_mark: letian-jiang:white_check_mark: zaorangyang:white_check_mark: evelynzhaojie:x: mergify[bot]You have signed the CLA already but the status is still pending? Let us recheck it.


bracket-push: remove error from function signature

The Bracket() function returns (bool, error), but there's no reasonable case where an error might be returned, nor do the tests expect one. This is puzzling some students, so I suggest changing the required signature of Bracket() to just bool.
(I have read #1072, but it doesn't seem to resolve this problem.)

I think based on the history we probably intended to remove the error case, but I think that got lost in the process somewhere.
I agree that we should change the signature.


Improve condition for more readability

I read code in package org.springframework.amqp.core and I think some codebase conditions are too complex and we can make it simple for readability purpose.
That's why I create draft PR and I wonder whether I can continue to do this, at least in package org.springframework.amqp.core. Please let me know your thoughts

I changed more files and I think it's ready to turn into a real PR.

@ngocnhan-tran1996 ,
thank you for contribution; looking forward for more!


Is possible add new action like UIAlertController?

Hello,
In iOS  with UIAlertController you can set many action with:
[alert addAction:action];
[alert addAction:action2];
[alert addAction:action3];
Etc...
Here apparently only two action (one is cancel). Is there a way to have only a list of buttons with this cool design?

Hi,
yes, this is definitely possible. You may take a look at the Tests to see an example (for example this one).
Just call addAction: multiple times and you can add as many actions as you want.
Best regards,
Roland


upgrade project akka version 2.6

Since akka 2.6 release https://akka.io/blog/news/2019/11/06/akka-2.6.0-released, we need to upgrade the project for the same.

@sclasen any updates on this?


Guide on how to update Team page

Actual Behaviour
Hi, after our committer gets selected, he/she needs to add himself/herself to our Apache APISIX Team page[1].
[1] https://apisix.apache.org/team/
Expected Behaviour
We need a Guide on How to add himself/herself to Team page.

How to fix this?

Refer to https://github.com/apache/apisix-website/pull/998
The Committer needs to submit a PR to add himself/herself to the team page, this issue mainly provides a doc for new committers.

Hi, @juzhiyuan I would like to work on this issue. Also, while documenting how a committer needs to add himself/herself to the Team's page, do we need to create a new page in the docs or add it to an existing one? 😊

Hi @BhavyaT-135, thanks for your help! Electing new committers is not a frequent case. I would prefer to invite new committers to do this. Could you please check other issues?

Sure @juzhiyuan, thanks for the early response! 😊

Hi, @juzhiyuan I would like to work on this issue.

Hey @juzhiyuan , is this still available?

Kindly @juzhiyuan, was this completed?

@rukundob451 There are currently no community members who have contributed documentation to us for this guide. 😿
(Sorry, there were a lot of emails and even though I subscribed, I still lost this message. )

Hey @juzhiyuan ,
I'd like to work on this issue. Could you let me know if it's still open and if there's anything specific I should know before getting started?
Thanks!


Methods have their own page, outside of the service page

Hello team,
Nice work, I like the simple, cleaner way of writing the doc.
However I get an issue, methods have their own page, outside of the service page.
With the demo provided in src, the structure is :

modules

app
mod1

services

service1

mod2

controllers

MyCtrl

directives

directive1

get.html

I'm expecting that get.html is located in service1, something to deal with angular-packages/angularjs ?

That makes sense to me too. The methods should be part of the service's page, not separate.
Hopefully that can be fixed.

I have the same problem with Controllers. Any known way around this? Thanks!


Allow catching relay errors

This is not good as it will crash the program instead of disconnecting a user on a users' session going bad.

failing

This is not good as it will crash the program instead of disconnecting a user on a users' session going bad.

It crashed the program before even if the error wasn’t handled. Maybe add relay.on(‘error’) and pass the error and player to it.

As the primary purpose of the Relay proxy here is debugging, it is intentional that errors from the player (which we do not expect) will crash the proxy, but errors from a server affecting a specific client will lead to just their disconnection.

What you're doing in this PR is removing error handling and leaving clients hanging (which will ultimately timeout without knowing why). If we could disconnect the player on player errors, we would, however we can't do that because it's the player side itself with the error. I think kicking the client and telling them why so they can reconnect and logging the output is better from a default functionality standpoint. Aside from your edge case, do you see much other benefit for general cases here?

maybe add an option to the relay to disable error handling and allow catching errors, disabled by default

Same comment as above

Aside from your edge case, do you see much other benefit for general cases here?

We already are adding a feature to handle errors in #435. Adding a feature to the API is a commitment that we keep it going without breaking functionality. If it’s only something that fixes a problem in one situation it may make more sense to implement a custom Relay to handle those behaviors as opposed to hacking around the error handling, which seems like messy API to me.


feat: enable thorchain longtail to L1

Description
Turns on longtail (on Ethereum) to THORChain L1 trading. Specifically, we should handle quotes from assets on Ethereum that have 3000 fee pool pair with WETH on UniswapV3 to any native THORChain pool.
A note to this is that the assets must also be whitelisted here: https://gitlab.com/thorchain/thornode/-/raw/develop/common/tokenlist/ethtokens/eth_mainnet_latest.json
This is currently undergoing testing by operations on the juice deployment. I'll leave it in draft until approved.
Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
N/A
Risk
Fairly high.
This impacts, and risks breaking THORChain quotes and trade execution.
There are also risks of user funds being lost if the contract implementation, or otherwise, is incorrect.
Testing
With this feature flag on:

We should now facilitate quotes and swaps from longtail Ethereum assets that are both in the whitelist URL above, and have UniswapV3 pools with liquidity at the 3000 fee range.
We should not give longtail quotes for any assets that are not in the whitelist, or do not have V3 pools in that range.

When performing longtail trades, two transactions will occur:

The first is on ethereum mainnet, hitting the swapIn method of the aggregator contract
The second is on THORChain, once the first call has been observed

The transaction hash in the trade completion modal is for the second transaction.
See srceenshots below for an example trade pair that is supported by longtail swaps.
Engineering
☝
Scrutinise how this feature could go wrong, and cause lost funds or bad UX.
Operations
☝
Screenshots (if applicable)

Ops approved, ready to release 🎉


ci: add android ci

closes #10

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.84%. Comparing base (949924a) to head (388fef3).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #58   +/-   ##
=======================================
  Coverage   94.84%   94.84%
=======================================
  Files          12       12
  Lines         233      233
  Branches       26       26
=======================================
  Hits          221      221
  Misses          7        7
  Partials        5        5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add Private Link Methods

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

HI @ankushbindlish2  this is some validation error detected. e.g. Semantic and Modal Validation and PretterCheck, can you fix it in this order?

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

According to discuss with ankushbindlish2 offline, will request ARM review later when merge into master


Fix ip address whiltelist value for hcl2 templates

Description
The value that is used for allowed_inbound_ip_addresses argument with HCL2 template is wrong so GenerateResourcesAndImage function fails when RestrictToAgentIpAddress switch is on and ImageType is Ubuntu2204. This request is to fix the value.
Related issue: https://github.com/actions/runner-images/issues/7143
Check list

[x] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

Received


User interaction

How do i set user interaction enable or disable while animation hud

The above will be a great feature to add. I was just thinking how can I use this with Error messages I need to display to the user? Great job on this HUD. Works Perfectly!

thanks will check after some time if you could provide more detail about feature would be great thanks
Sent from my iPhone

On 29-Dec-2017, at 6:08 AM, SaifSideDev<EMAIL_ADDRESS>wrote:
The above will be a great feature to add. I was just thinking how can I use this with Error messages I need to display to the user? Great job on this HUD. Works Perfectly!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@ErAbhishekChandani No problem. basically all it’ll be is the same idea as how you do the HUD but this will incorporate user interaction like an alert controller or a unique way such as a tap gesture. You can either tie it directly to the HUD where you have a HUD that appears on the UI and integrates with a tap gesture recognizer so that you have to tap to dismiss it or you just use alert controller and make the customization seamless for devs.  Nothing too complicated, just will make this pod an all in one easy to use solution. In my example, say I’m authenticating a user and there’s an error message I need to show...I will first use an activity indicator hud but for the error that should appear after without the indicator and stay on the screen with tap to dismiss functionality or however you want to engineer the solution. Let me know if you have any other questions.

Thank you for details and I will try to incorporate kind of features soon thanks.
Sent from my iPhone

On 29-Dec-2017, at 10:57 AM, SaifSideDev<EMAIL_ADDRESS>wrote:
@ErAbhishekChandani No problem. basically all it’ll be is the same idea as how you do the HUD but this will incorporate user interaction like an alert controller or a unique way such as a tap gesture. You can either tie it directly to the HUD where you have a HUD that appears on the UI and integrates with a tap gesture recognizer so that you have to tap to dismiss it or you just use alert controller and make the customization seamless for devs. Nothing too complicated, just will make this pod an all in one easy to use solution. In my example, say I’m authenticating a user and there’s an error message I need to show...I will first use an activity indicator hud but for the error that should appear after without the indicator and stay on the screen with tap to dismiss functionality or however you want to engineer the solution. Let me know if you have any other questions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


MacOS: VoiceOver Utility - "Verbosity -> Hints -> Speak instructions" enabled by default

"Verbosity -> Hints -> Speak instructions" enabled by default
Setup:

NPM version: all
OS Platform: macos
OS Release: latest (CI)

Details
@cmorten @jlp-craigmorten, first of all, thanks for your work :)
We are testing guidepup and there is some option for voiceOver which might be a little bit annoying.
You can see it here the output is:
["Label. You are currently on a text element, inside of web content. To exit this web area, press Control-Option-Shift-Up Arrow.","Label (Default) Basic edit text. You are currently on a text field, inside of web content. To enter text in this field, type. To exit this web area, press Control-Option-Shift-Up Arrow."]
The default voiceOver config enables speak instructions for items ("You are currently on a text element, inside of web content. To exit this web area, press Control-Option-Shift-Up Arrow"). It would be great if we could optin/out the checkbox under VoiceOver Utility - "Verbosity -> Hints -> Speak instructions" on our CI, maybe like this:
      - name: 🦮 Guidepup Setup
        uses<EMAIL_ADDRESS>        with:
          enableSpeakInstructions: false

Hey @nmerget 👋
I'm tempted to suggest this is more something for the https://github.com/guidepup/guidepup project - hope for setup is that it is a minimal environment setup to allow for automation, but otherwise all AT settings eventually should be configurable as part of the core guidepup package.
To potentially unblock you for now, have you tried using the capture: "initial" option for the voiceOver.start({ ... }) command?
REF: https://www.guidepup.dev/docs/api/class-command-options#command-options-capture
Setting this verbosity setting feels relevant to https://github.com/guidepup/guidepup/issues/75 where it would be good to expose the ability to fully configure your AT as required.

I will try it with capture: "initial". Thank you
For now, it's fine, I will add a workaround to filter the snapshot output in the CI.
It makes sense to add it to add a config for the screen readers.
A "config" will execute commands in the shell like this right?

A "config" will execute commands in the shell like this right?

Yeah that kind of thing, see also https://github.com/guidepup/guidepup/blob/main/src/macOS/VoiceOver/configureSettings.ts which is probably what would look to consolidate on, and then expose the ability to configure rather than have this hardcoded.


[NFC] Refactor docker scripting to generalize away from Kokoro

Factors out a generic docker_run script from our Kokoro-specific
docker setup. This will enable reusing the scripting in GitHub actions
(follow-up PR).

Forgot to tag originally, but part of https://github.com/iree-org/iree/issues/9855


Added C++ codes for standard binary tree problems

problems:

Pre Order
In Order
Post Order
Lowest Common Ancestor
Left view
Right view
Burn Tree
Diameter of tree
Child Sum Property
Height of Tree

@SouMy16
Great work, excellent 🚀
You can follow me and star this repo for further updates.
Have fun!


Splitjaw/Orby Tweaks

Is your feature request related to a problem?
I haven't really seen anyone else discussing this stuff so this is just largely my own personal opinion, I can't speak for the rest of the server but i don't think these changes would be controversial hopefully.

I didn't personally have this problem when I started out but imo splitjaws are just a bit too troublesome for newer players, and it's really common to see newer players getting massacred over and over again by splitjaws in layer 1/2. The wiki also says 'it is exceedingly rare to see them in the most shallow sections of the Abyss.'

Orbpiercers are way too rare, you've probably seen me or other people complaining about this but I think they're too uncommon rn even after their criteria is met. As of rn there hasn't been a new orbpiercer caused death in 24 hours which I can only assume means there've been virtually no spawns for the past day. While I get that they're canonically rare, for such an iconic mob i think it's weird how 99% of new players won't find one on their journey through L4. I also think they should be easier to find for players actively trying to seek them out. It's the survival server after all, PvE is some people's favourite endgame.

Describe the solution you'd like.

I think splitjaws should be just a bit less common in layer 1. Possibly decrease the spawn priority in layer 1 slightly? It's a bit strange for them to be more common in edge of abyss than 3 which is their canon living area. By layer 2/3 players should have gotten the hang of the gameplay so their rarity is fine there

I didnt see anything about orbies being nocturnal on the wiki but I don't really have an issue with that being in the game. For the other spawn conditions, if im not mistaken, priority was 0.5 and spawndelay was 5s before the big change to the spawn table a week ago. That's probably too high, so maybe it could be changed to priority 0.2 or 0.25 + spawndelay 5s? To my knowledge, the only use for orby spikes rn is 7 per orth coin, maybe that could be tweaked to 9 or 10 if necessary to balance it out. And if orby faces for backpacks is a concern, drop rate of orby faces could be halved.

Describe alternatives you've considered.

Agreements

[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] I have ensured the feature I'm requesting isn't already in the latest supported Paper build.

Other
I think the main concern about orbies being more common rn is them being farmed. Honestly though, I dont think farming orbies for coins should be something to be actively discouraged by the server considering how dangerous they are in comparison to other coin methods like just passive makihige and great fault flying mob hunting. And if the backpacks are a concern then the recipe could either be made harder with 2 faces instead of 1 or the face drop rates should go down, though I doubt anyone is mass hunting orbies for backpacks when they're available for 10 orth. Plus, I don't think anyone was living off orby-based income even when the rates were as high as they were a week ago, so I really don't think this will cause any unfair shift in the economy.

You can always make an actual SpawnConfig suggestion change by following this
Testing it is very annoying as it requires it to be on survival and a full restart for any little change though so

Ahh i didnt know there was an actual page for it thanks! i'll follow that! probably good that i included the reasoning here though so you can see x)


Not able to send file size > 3 mb on pub sub

PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
I am trying to send a csv file on pubsub by converting it to byte[]. When file size is more than 3mb I am getting  io.grpc.StatusRuntimeException: DEADLINE_EXCEEDED: deadline exceeded after 4.999830173s. I tried multiple forums but i didn't get any response.

Hi there! The current limits on message sizes can be found here: https://cloud.google.com/pubsub/quotas If you are still experiencing this issue, please reopen with additional repro steps.

Hi there! The current limits on message sizes can be found here: https://cloud.google.com/pubsub/quotas If you are still experiencing this issue, please reopen with additional repro steps.


Fix for removal of v-get-web-domain-value

The command v-get-web-domain-value was removed in Vesta 0.9.8-17. This fixes by replacing with v-list-web-domain-ssl to perform the same function.

@samcleathero  :  Why this modification still has not been merged to the master repository?
Did the author of this project had aborted it?


Cannot build framework with Alamofire

Hi,
I'm a Cocoapods Rome user and I cannot build my dependencies with the Alamofire subspec.
I've made some investigations.
With the following Podfile:
plugin 'cocoapods-rome'
platform :ios, '9.0'

target :Dependencies do
    pod 'PromiseKit/Alamofire', '~> 6.0.1'
end

post_install do |installer|
    installer.pods_project.targets.each do |target|
        target.build_configurations.each do |config|
            config.build_settings['SWIFT_VERSION'] = '4.0'
        end
    end
end

I ran the pod install command which has generated a PromiseKit.framework without Alamofire subspec headers.

In order to compare, I have tested same Podfile but with another subspec
plugin 'cocoapods-rome'
platform :ios, '9.0'

target :Dependencies do
    pod 'PromiseKit/OMGHTTPURLRQ', '~> 6.0.1'
end

post_install do |installer|
    installer.pods_project.targets.each do |target|
        target.build_configurations.each do |config|
            config.build_settings['SWIFT_VERSION'] = '4.0'
        end
    end
end

I this case, generated framework has OMGHTTPURLRQ headers.

Maybe the problem comes from the trailing -  in github Alamofire projet name (Alamofire-).
Regards,
Quentin

What’s the error message?

Sounds like a time issue report it there.

There is not error message, just missing files in generated framework. I'll check with CocoaPoads team.

Yeah, if CocoaPods doesn't generate the dependency then it seems like it must be their bug, we specific the dependency:
https://github.com/mxcl/PromiseKit/blob/master/PromiseKit.podspec#L45-L53


Fix #321 by recreating the tools.namespace records before using them

The :reload-all causes the tools.namespace protocols to be reloaded, which causes:
java.lang.IllegalArgumentException: No implementation of method: :nodes of protocol: #'clojure.tools.namespace.dependency/DependencyGraph found for class: clojure.tools.namespace.dependency.MapDependencyGraph
at clojure.core$cache_protocol_fn.invoke(core_deftype.clj:554)
at clojure.tools.namespace.dependency$eval24795$fn__24848$G__24786__24853.invoke(dependency.clj:14)
at clojure.tools.namespace.track$remove.invoke(track.clj:97)
at clojure.tools.namespace.file$remove_files.invoke(file.clj:63)
at clojure.tools.namespace.dir$update_files.invoke(dir.clj:38)
at clojure.tools.namespace.dir$scan_all.doInvoke(dir.clj:73)
at clojure.lang.RestFn.applyTo(RestFn.java:139)
at clojure.core$apply.invoke(core.clj:632)
at midje.data.project_state$mkfn_COLON_scan_and_react$fn__24132$fn__24133.invoke(project_state.clj:142)
at clojure.lang.Atom.swap(Atom.java:37)
at clojure.core$swap_BANG.invoke(core.clj:2238)
at midje.data.project_state$mkfn_COLON_scan_and_react$fn__24132.invoke(project_state.clj:141)
at midje.data.project_state$load_everything.invoke(project_state.clj:151)
To fix the issue, the tools.namespace MapDependencyGraph record held in the state tracker needs to be recreated before using it

I saw the same problem you did, so I merged the pull request. However, I now get an immediate:
crede.core=> (autotest)
Because failures in the initial load break autotest's dependency tracking,
autotest has been cancelled.
true

Investigating...

which project are you running? I tested it with the #321

I can reproduce by cloning https://github.com/marick/sample-midje-project and making this change in project.clj:
  :profiles {:dev {:dependencies [[midje "1.9.0-SNAPSHOT"]]

... then doing this:
667 $ lein repl
nREPL server started on port 64344 on host <IP_ADDRESS> - nrepl://<IP_ADDRESS>:64344
REPL-y 0.3.7, nREPL 0.2.10
Clojure 1.7.0
Java HotSpot(TM) 64-Bit Server VM 1.8.0_25-b17
    Docs: (doc function-name-here)
          (find-doc "part-of-name-here")
  Source: (source function-name-here)
 Javadoc: (javadoc java-object-or-class-here)
    Exit: Control+D or (exit) or (quit)
 Results: Stored in vars *1, *2, *3, an exception in *e

user=> (use 'midje.repl)
This is from the Midje configuration file in ${HOME}.
You can change the configuration independently for the repl.
Run `(doc midje)` for Midje usage.
Run `(doc midje-repl)` for descriptions of Midje repl functions.
nil
user=> (autotest)
Because failures in the initial load break autotest's dependency tracking,
autotest has been cancelled.
true

I see a NPE. To fix:
(defn with-fresh-copy-of-dependency-map
"Records must be recreated if the protocols that they implement are reloaded."
[state]
(if (get state deps-key)
(update-in state [deps-key] clojure.tools.namespace.dependency/map->MapDependencyGraph)
state))
Let me know if you prefer a pull request

That's helped so far - I'll let you know if I detect any other problems.


Port, redirect and modernize all .NET networking content.

Port, redirect and modernize all .NET networking content. There is a whole sub-doc-set located here:

Network Programming in the .NET Framework

Much of this content is severely out-of-date, and many code examples reference obsolete APIs. The strategy is to move (port) conceptual content from the Network Programming in the .NET Framework under the modern .NET Fundamentals / Runtime Libraries / Networking TOC:

We should converge the existing HTTP-specific content under a new TOC node, .NET Fundamentals / Runtime Libraries / Networking / HTTP. Then we should create several new nodes to capture content specific to additional protocols, such as TCP, UDP, FTP, WebSockets, and whatever else I might be missing. A lot of the legacy content specific to .NET Framework can be modernized and code samples updated accordingly.
Asynchronous patterns such as:

Begin{X} / End{X}
AsyncCallback
IAsyncResult

Should be removed in favor of, async/await, Task, and ValueTask (or other task-like alternatives). conceptual rewrites should strive to serve new learned audiences, making adoption easy.
/cc @richlander

This looks great and has very high value. Turns out that modern scenarios require a lot of networking, right?
Some things to consider (likely as advanced scenarios; unordered):

Sync over async
Transport-agnostic APIs (both producing and consuming)
How to wrap those old async patterns. I agree we don't want folks producing new code that uses those patterns, but interacting with them may still be required for some scenarios.
Using all the modern networking patterns like circuit breakers. If those are not available in-box, which libraries should folks use to take advantage of those.
How to handle flaky networkings, including cellular
Are there any scenarios where .NET devs need to reason about LTE vs 5G for cell links?
Networking in containers
Global vs local DNS (including sidecars)
Unix domain sockets vs DNS
HTTPS/TLS termination
When to use HTTP, HTTPS, HTTP/2, HTTP/3 (and which capabilities are (not) supported for each, if relevant). AFAIK, there are valid use cases for all of these. For example, I know some folks terminate TLS at the load balancer (if that's the correct term) and then use raw HTTP between nodes. There are pros and cons with that.
ValueTask should probably be considered an advanced scenario. Would be good to validate that.
Stream vs Pipelines

@davidfowl

We do have coverage of Pipelines, and it is discussed in the context of I/O and streams:

System.IO.Pipelines in .NET


CRYP-114: refactored folder structure for api to be by type instead of by domain

Looks good but can you double check in your ide to make sure all the paths are good, should give a red underline error if any are bad.

Looks good but can you double check in your ide to make sure all the paths are good, should give a red underline error if any are bad.

all good on my end, also wouldn't the CI fail?

Looks good but can you double check in your ide to make sure all the paths are good, should give a red underline error if any are bad.

all good on my end, also wouldn't the CI fail?

Only if the path is imported as part of the CI like if the file is used. JS is weird in that it doesn't know if paths are good or not until it runs that line. A linter wont catch it for example.