Call to function "element" failed: cannot use element function with an empty list

Hi,
I am trying to destroy everything that i created with terraform but i am getting a new error.
**`Error: Error in function call

  on ../modules/vpc/main.tf line 935, in resource "aws_route_table_association" "private":
 935:   route_table_id = element(
 936:
 937:
 938:
    |----------------
    | aws_route_table.private is empty tuple
    | count.index is 3
    | var.single_nat_gateway is false

Call to function "element" failed: cannot use element function with an empty
list.

Error: Error in function call

  on ../modules/vpc/main.tf line 946, in resource "aws_route_table_association" "database":
 946:     coalescelist(aws_route_table.database.*.id, aws_route_table.private.*.id),
    |----------------
    | aws_route_table.database is empty tuple
    | aws_route_table.private is empty tuple

Call to function "coalescelist" failed: no non-null arguments.

Error: Error in function call

  on ../modules/vpc/main.tf line 946, in resource "aws_route_table_association" "database":
 946:     coalescelist(aws_route_table.database.*.id, aws_route_table.private.*.id),
    |----------------
    | aws_route_table.database is empty tuple
    | aws_route_table.private is empty tuple

Call to function "coalescelist" failed: no non-null arguments.`**

Any idea why this is happening?

+1 I have this same error when trying to change an existing VPC to have multiple NAT Gateways for each AZ.
Before
single_nat_gateway = true
one_nat_gateway_per_az = false

after
single_nat_gateway = false
one_nat_gateway_per_az = true

When running terraform plan I get a similar error due to aws_subnet.public is empty tuple

Getting this as well with Terraform 0.12.24 and provider.aws 2.55 working on creating two AZs in one region, one public subnet per AZ and each with their own NAT gateway. I deleted my network stack to try and resolve it not properly updating the sg-XXXXX identifier from a terraform_remote_state data source, thus erroring out, and have ran into this:
Error: Error in function call

  on network-subnets.tf line 69, in resource "aws_nat_gateway" "gw":
  69:   subnet_id     = element(aws_subnet.public[*].id, count.index)
    |----------------
    | aws_subnet.public is empty tuple
    | count.index is 0

Call to function "element" failed: cannot use element function with an empty
list.

Error: Error in function call

  on network-subnets.tf line 69, in resource "aws_nat_gateway" "gw":
  69:   subnet_id     = element(aws_subnet.public[*].id, count.index)
    |----------------
    | aws_subnet.public is empty tuple
    | count.index is 1

Call to function "element" failed: cannot use element function with an empty
list.
The following was working just fine half an hour ago, creating two subnets and two NAT gateways:
network-subnets.tf
# Create var.az_count public subnets, each in a different AZ
resource "aws_subnet" "public" {
  count                   = var.az_count
  cidr_block              = cidrsubnet(aws_vpc.Rearc.cidr_block, 8, count.index)
  availability_zone       = data.aws_availability_zones.available.names[count.index]
  vpc_id                  = aws_vpc.Rearc.id
  map_public_ip_on_launch = true

  tags = {
    name = "Public Subnet"
    Region = var.aws_region
    Environment = var.environment
  }
}

snip
# Create a NAT gateway with an Elastic IP for each private subnet to get internet connectivity
resource "aws_eip" "gw" {
  count      = var.az_count
  vpc        = true
  depends_on = [aws_internet_gateway.gw]

  tags = {
    Name = "Elastic IP Address"
    Region =var.aws_region
    Environment = var.environment
  }
}

resource "aws_nat_gateway" "gw" {
  count         = var.az_count
  subnet_id     = element(aws_subnet.public[*].id, count.index)
  allocation_id = element(aws_eip.gw[*].id, count.index)

  tags = {
    Name = "NAT Gateway"
    Region = var.aws_region
    Environment = var.environment
  }
}

The only way I was able to resolve this was to delete the terraform.tfstate file in the pertinent subdirectory in its S3 bucket, the accompanying MD5 sum entry in its DynamoDB table, and rebuild from scratch.
I've made a backup of the current network directory if further information is requested.

I guess this will occur in cases where we are trying to create public subnets < number of azs

I guess this will occur in cases where we are trying to create public subnets < number of azs

Nice thought, but I hard-coded 3  AZ's name (I have 3 subnets) and it still gives the error.

I guess this will occur in cases where we are trying to create public subnets < number of azs

Nice thought, but I hard-coded 3 AZ's name (I have 3 subnets) and it still gives the error.

Actually my  hard-coded test probably isn't right.  My original  code is azs             = data.aws_availability_zones.available.names which has count 4 in us-west-2, but I defined only 3 subnets,  hence the error. I will stick with NAT per subnets configuration.

Also getting the same error when standing up a new VPC, with static assignment of 3 AZs and 3 public subnets (one per AZ).  Ideally would like to see 1 NGW per AZ, using the following flags:

single_nat_gateway = false
one_nat_gateway_per_az = true
enable_nat_gateway = true

Getting the following error (Terraform v0.12.26 and AWS ">= 2.28.1"):
`Error: Error in function call
on .terraform/modules/vpc/terraform-aws-vpc-2.6.0/main.tf line 795, in resource "aws_nat_gateway" "this":
795:   subnet_id = element(
796:
797:
798:
|----------------
| aws_subnet.public is empty tuple
| count.index is 2
| var.single_nat_gateway is false
Call to function "element" failed: cannot use element function with an empty
list.
`

I'm getting same error using terragrunt.
inputs = {
  name = "vpc"
  cidr = "<IP_ADDRESS>/16"
  azs             = ["us-west-2a", "us-west-2b", "us-west-2c"]
  private_subnets = ["<IP_ADDRESS>/20", "<IP_ADDRESS>/20", "<IP_ADDRESS>/20"]
  public_subnets  = ["<IP_ADDRESS>/20", "<IP_ADDRESS>/20", "<IP_ADDRESS>/20"]
  #single_nat_gateway = true
  enable_nat_gateway   = true
  enable_vpn_gateway   = true #Virtual Private Gateway id=vgw-xxxxxxxxx
  enable_s3_endpoint   = true
  enable_dns_hostnames = true

terragrunt version v0.21.11
Terraform v0.12.20
provider.aws: version = "~> 3.0"
Error: Error in function call
00:51:50
00:51:50    on .terraform/modules/local_vpc/terraform-aws-vpc-2.21.0/main.tf line 905, in resource "aws_route" "private_nat_gateway":
00:51:50   905:   route_table_id         = element(aws_route_table.private.*.id, count.index)
00:51:50      |----------------
00:51:50      | aws_route_table.private is empty tuple
00:51:50      | count.index is 1
00:51:50
00:51:50  Call to function "element" failed: cannot use element function with an empty
00:51:50  list.
00:51:50
00:51:50
00:51:50  Error: Error in function call
00:51:50
00:51:50    on .terraform/modules/local_vpc/terraform-aws-vpc-2.21.0/main.tf line 905, in resource "aws_route" "private_nat_gateway":
00:51:50   905:   route_table_id         = element(aws_route_table.private.*.id, count.index)
00:51:50      |----------------
00:51:50      | aws_route_table.private is empty tuple
00:51:50      | count.index is 2
00:51:50
00:51:50  Call to function "element" failed: cannot use element function with an empty
00:51:50  list.
00:51:50
00:51:50
00:51:50  Error: Error in function call
00:51:50
00:51:50    on .terraform/modules/local_vpc/terraform-aws-vpc-2.21.0/main.tf line 905, in resource "aws_route" "private_nat_gateway":
00:51:50   905:   route_table_id         = element(aws_route_table.private.*.id, count.index)
00:51:50      |----------------
00:51:50      | aws_route_table.private is empty tuple
00:51:50      | count.index is 0
00:51:50
00:51:50  Call to function "element" failed: cannot use element function with an empty
00:51:50  list.
00:51:50
00:51:50
00:51:50  Error: Error in function call
00:51:50
00:51:50    on .terraform/modules/local_vpc/terraform-aws-vpc-2.21.0/main.tf line 929, in resource "aws_route_table_association" "private":
00:51:50   929:   route_table_id = element(
00:51:50   930:
00:51:50   931:
00:51:50   932:
00:51:50      |----------------
00:51:50      | aws_route_table.private is empty tuple
00:51:50      | count.index is 2
00:51:50      | var.single_nat_gateway is false
00:51:50
00:51:50  Call to function "element" failed: cannot use element function with an empty
00:51:50  list.
00:51:50
00:51:50
00:51:50  Error: Error in function call
00:51:50
00:51:50    on .terraform/modules/local_vpc/terraform-aws-vpc-2.21.0/main.tf line 929, in resource "aws_route_table_association" "private":
00:51:50   929:   route_table_id = element(
00:51:50   930:
00:51:50   931:
00:51:50   932:
00:51:50      |----------------
00:51:50      | aws_route_table.private is empty tuple
00:51:50      | count.index is 1
00:51:50      | var.single_nat_gateway is false
00:51:50
00:51:50  Call to function "element" failed: cannot use element function with an empty
00:51:50  list.
00:51:50
00:51:50
00:51:50  Error: Error in function call
00:51:50
00:51:50    on .terraform/modules/local_vpc/terraform-aws-vpc-2.21.0/main.tf line 929, in resource "aws_route_table_association" "private":
00:51:50   929:   route_table_id = element(
00:51:50   930:
00:51:50   931:
00:51:50   932:
00:51:50      |----------------
00:51:50      | aws_route_table.private is empty tuple
00:51:50      | count.index is 0
00:51:50      | var.single_nat_gateway is false
00:51:50
00:51:50  Call to function "element" failed: cannot use element function with an empty
00:51:50  list.
00:51:50
00:51:50
00:51:50  Error: Invalid index
00:51:50
00:51:50    on .terraform/modules/local_vpc/terraform-aws-vpc-2.21.0/main.tf line 989, in resource "aws_route_table_association" "public":
00:51:50   989:   route_table_id = aws_route_table.public[0].id
00:51:50      |----------------
00:51:50      | aws_route_table.public is empty tuple
00:51:50
00:51:50  The given key does not identify an element in this collection value.
00:51:50
00:51:50
00:51:50  Error: Invalid index
00:51:50
00:51:50    on .terraform/modules/local_vpc/terraform-aws-vpc-2.21.0/main.tf line 989, in resource "aws_route_table_association" "public":
00:51:50   989:   route_table_id = aws_route_table.public[0].id
00:51:50      |----------------
00:51:50      | aws_route_table.public is empty tuple
00:51:50
00:51:50  The given key does not identify an element in this collection value.
00:51:50
00:51:50
00:51:50  Error: Invalid index
00:51:50
00:51:50    on .terraform/modules/local_vpc/terraform-aws-vpc-2.21.0/main.tf line 989, in resource "aws_route_table_association" "public":
00:51:50   989:   route_table_id = aws_route_table.public[0].id
00:51:50      |----------------
00:51:50      | aws_route_table.public is empty tuple
00:51:50
00:51:50  The given key does not identify an element in this collection value.

Call to function "element" failed: cannot use element function with an empty list.

The given key does not identify an element in this collection value.

Also trying single_nat_gateway & one_nat_gateway_per_az but still getting same error

You can fix as someone previously mention just delete the  files terraform.tfstate, terraform.tfstate.backup, and run again the plan.
The problem appears because something was deleted outside terraform and the terraform state got desynchronized

I think all of these problems are related, and very hard to fix for all combinations of parameters and flows (create/update).
If problem exists, please provide exact set of arguments you specify to the module, so that we (@terraform-aws-modules/triage-supporters and others) can try to reproduce the exact issue on our end.
Also, try to run configurations in examples/simple-vpc with updated arguments to reduce the scope of an issue in your environment.

We're facing this issue with 2.54.0 and have following errors reported for our DEV environment (which are by definition cut down to absolute minimum):
Error: Error in function call
  on .terraform/modules/core_infrastructure.main_vpc.vpc/vpc-endpoints.tf line 360, in resource "aws_vpc_endpoint" "ecr_api":
 360:   subnet_ids          = coalescelist(var.ecr_api_endpoint_subnet_ids, aws_subnet.private.*.id)
    |----------------
    | aws_subnet.private is empty tuple
    | var.ecr_api_endpoint_subnet_ids is empty list of string
Call to function "coalescelist" failed: no non-null arguments.
Error: Error in function call
  on .terraform/modules/core_infrastructure.main_vpc.vpc/vpc-endpoints.tf line 540, in resource "aws_vpc_endpoint" "monitoring":
 540:   subnet_ids          = coalescelist(var.monitoring_endpoint_subnet_ids, aws_subnet.private.*.id)
    |----------------
    | aws_subnet.private is empty tuple
    | var.monitoring_endpoint_subnet_ids is empty list of string
Call to function "coalescelist" failed: no non-null arguments.

The configuration we set for the module is:
vpc_cidr                   = "<IP_ADDRESS>/18"
vpc_az_count_private       = 2 # Need at least 2 for ALBs
vpc_az_count_database      = 2 # Need at least 2 for ALBs
vpc_az_count_public        = 2 # Need at least 2 for RDSs
vpc_enable_nat_gateway     = true
vpc_one_nat_gateway_per_az = false
vpc_single_nat_gateway     = true

The weird thing is the notification that aws_subnet.private is empty tuple, when we explicitely request 2 private subnets.
I also wonder if/how that issue was related to https://github.com/hashicorp/terraform/issues/21370.
Oh, and one more. Our terraform jobs work correctly on 2.48.0. It was the attempted migration to 2.54.0 that causes that issue.
We're using Terraform 0.12.29 AWS Provider 3.8.0.

I am facing similar issue where I provide:
public_subnets       = []

I am facing similar issue where I provide:
public_subnets       = []

NAT gateway resides in the public subnet. Private subnets may have NAT gateway, which makes them also requiring public subnets to be set. public_subnets should not be empty for this reason.
Please provide a complete set of arguments as-is that you pass to this module. It is just impossible to help without seeing code.

NAT gateway resides in the public subnet. Private subnets may have NAT gateway, which makes them also requiring public subnets to be set. public_subnets should not be empty for this reason.
Please provide a complete set of arguments as-is that you pass to this module. It is just impossible to help without seeing code.

@antonbabenko Could you look into this thread? Some people get this problem too.

@jungwookim I wrote before that such issues are very hard to fix for all combinations of parameters and flows (create/update) and the only solution which comes to my mind now is to hint terraform by doing smaller changes at once:

change one argument and apply it, then change another one and apply again
terraform apply -target=module.vpc.aws_nat_gateway (or similar)

Please provide a complete set of arguments as-is that you pass to this module. It is just impossible to help without seeing code.

Debugging invisible configurations is very time-consuming.
I am going to close this issue.

@antonbabenko Thanks for a quick review. That makes sense.

In my case I needed to delete the terraform state file stored in S3


Upgrade operator version to 112

replaced by https://github.com/open-telemetry/opentelemetry-helm-charts/pull/1410 since @jaronoff97 is on PTO


How To Build a Progress Bar On Page Scroll

NOTE: We tend to stray away  or tend not to publish reviews/comparisons of commercial product offerings.
Brief Summary:
Building engaging user interfaces requires writing semantic CSS. One way to build such an engaging UI is via page scroll. In this article, we will cover how we can implement a scroll bar to show the scroll page position using CSS and JavaScript,
Key Takeaways:

Scaffolding HTML and CSS web page
Using CSS for styling
Implementing JavaScript to showcase progress bar in a webpage

References:
N/A

👋
@Larry-Mason
Thank you for your topic suggestion. Per the recent message posted in our EngEd Slack channel, we are pausing acceptance of new articles until further notice while we work through the backlog of submitted drafts. We will notify you when we are ready to accept new topic suggestions. Thank you for your patience.

Was closed before I could write a single article. When should I expect
this?
On Tue, Feb 9, 2021, 08:04 hectorkambow<EMAIL_ADDRESS>wrote:

👋
@Larry-Mason https://github.com/Larry-Mason
Thank you for your topic suggestion. Per the recent message posted in our
EngEd Slack channel, we are pausing acceptance of new articles until
further notice while we work through the backlog of submitted drafts. We
will notify you when we are ready to accept new topic suggestions. Thank
you for your patience.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/issues/1537#issuecomment-775665655,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASDE3TAITMYQERE57RSNU73S6C66LANCNFSM4XKC3HUA
.


Untie HF accelerate v2 and v4 dependency.

Description
Per the nightly runs https://screenshot.googleplex.com/BN7rpqy45tAvfSE, pt-nightly-accelerate-smoke-v2-8-1vm failed to run due to upstream_failed: 3. The upstream pt-nightly-accelerate-smoke-v4-8-1vm failed due to lack of tpu v4 resource. We really need a way to run the HF accelerate test in order to catch regression regardless the TPU VM version (v2/v4). IOW, if we have TPU v2 resource, we should try to run the test on it even if we don't have TPU v4. This PR breaks the dependency of  HF accelerate v2 and v4 dependency so we always try to run the test.
Tests
Please describe the tests that you ran on Cloud VM to verify changes.
Instruction and/or command lines to reproduce your tests: ...na
List links for your tests (use go/shortn-gen for any internal link): ...na
Checklist
Before submitting this PR, please make sure (put X in square brackets):

[x] I have performed a self-review of my code.
[x] I have necessary comments in my code, particularly in hard-to-understand areas.
[x] I have run one-shot tests and provided workload links above if applicable.
[x] I have made or will make corresponding changes to the doc if needed.

@will-cromar do we expect accelerate to run on v2 at all?

@will-cromar do we expect accelerate to run on v2 at all?

@will-cromar and I fixed it on v2/v3 a few weeks ago in https://github.com/huggingface/accelerate/pull/2542. So I think we should expect it to run on v2.

Thanks for the review Jack!


Support of local storage provider

Please add support of storage provider using local ftp for those who don't want to use amazon

we started to investigate possibility to use local s3-compatible local storage for screenshots/video etc artifacts.
Optimistically it should be available in May. Best candidate as of now is https://min.io/

already resolved in scope of zebrunner ce 1.0: https://github.com/zebrunner/zebrunner/releases/tag/5.2


Default registration marker positions x/y flipped

Looks like the x/y coordinates for default positions are flipped (x/y). Registration works fine so it's probably correct downstream (i.e. when markers are set correctly) but it's mildly annoying for the registration process.
Same goes for the target definition.

Fixed in f1ade6257c3d366b5c4eaa687fca619cd6055523


add callback for getting device changes

part of fix for https://github.com/element-hq/element-web/issues/26263

Can someone hit the "Merge" button for me?

test added


Enable requireSpread rule.

https://github.com/jscs-dev/node-jscs/blob/master/lib/rules/require-spread.js

:+1: - I like this requirement.
Can you please add these to the good examples list:
var Blah = Ember.Object.extend({
  init() {
    this._super(...arguments); // normal usage

    // odd, but still needed in some cases
    var _super = this._super;
    // some async here
    _super.apply(this, arguments);
  }
});

function bar() {
  // I expect `this` to be a specific thing
}

bar.apply(someSpecialSnowflake, argsArray);

Probably a few bugs or other cases it could catch I didn't get around to doing - https://github.com/jscs-dev/node-jscs/issues/1482

@rwjblue Alright I added those. Let me know if you'd like any other changes. Thanks!

@rwjblue: Fixed, sorry!

No worries! Looks like Travis is angry though.

Yeah it seems as though the second example doesn't actually fail even though it is listed as an invalid example in jscs.

Glad we caught it :stuck_out_tongue_winking_eye:, maybe you could report it to JSCS?

@xtian - Did you report this issue upstream?

@rwjblue I mentioned it on jscs-dev/node-jscs#1482 as the issue seems related

Going to merge this, despite https://github.com/jscs-dev/node-jscs/issues/1482 not being fixed. Having the rule in here means we'll have partial coverage at least (instead of none at all). And when that issue does get fixed, it should be pulled in automatically when users install ember-suave.


GTEx expression for GRCh38 transcripts?

GTEx expression data in the transcripts track is based on GRCh37.

On hold awaiting the release of public data for GTEx v8.

GTEx v8 is based on Gencode v26, while gnomAD v3 and the browser is based on Gencode v29. Because there are some genes/transcripts in v29 that are not present in v26, the API and transcript track needs to handle the case where data is missing for a gene or transcript.

This is blocked until GTEx and gnomAD are both on the same version of Gencode.


restructure dirs

@pld Does this mean we will move the current testing documentation to the new generic category?

No current test docs are about unit tests, generic/overall testing content goes in the new file

replaced by https://github.com/opensrp/fhircore/pull/2928/files


How keyboard macro programming works

I can't figure out how to program double clicks in keyboard macros.
For example I'm having trouble programming
LEFT(on)
DOWN(on)DOWN(off)
DOWN(on)DOWN(off)
LEFT{off)
so need to press down twice while holding left pressed.
Looked at the documentation, but there is very little explenation on how macro programming works.

Hey @rmamba ,
GP2040-CE will look at each line individually, so to have left held and have it press down twice while holding left you would want something like this:
LEFT(on)
LEFT(on) DOWN(on)
LEFT(on) DOWN(off)
LEFT(on) DOWN(on)
LEFT(on) DOWN(off)
LEFT{off)

Does it automatically figure out on/off?
I don't really get the logic in these form, but this is what you are saying @TheTrainGoes right?

FYI,
It works, replaced last entry with None only.


Link to windowsitpro.com not functional

The link with the label "Resource-based constrained delegation" and destination URL of http://windowsitpro.com/security/how-windows-server-2012-eases-pain-kerberos-constrained-delegation-part-1 is not functional as of this writing.
I am uncertain if submitting a PR to remove the dead link is the correct move, or if the doc authors would prefer a different third party explanation of the topic?

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

ID: d6273f49-fe06-5a18-80e3-863bd3a0cf4e
Version Independent ID: 4ed5bf3d-0c2a-a80e-5bd9-0e858dd8749b
Content: Configuring user access control and permissions
Content Source: WindowsServerDocs/manage/windows-admin-center/configure/user-access-control.md
Product: windows-server-threshold
Technology: manage
GitHub Login: @Haley-Rowland
Microsoft Alias: harowl

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


auto update interval

set either AUTO_UPDATE_INTERVAL or -i N when starting APIcast and see it update every N seconds!
No idea how to test this tough.

This is not a default value, but integer base.
Sent from my iPhone

On 10 Nov 2016, at 18:59, David Ortiz<EMAIL_ADDRESS>wrote:
@davidor commented on this pull request.
In apicast/src/apicast.lua:

@@ -30,6 +31,41 @@ function _M.init()
end
end

+function _M.init_worker()

local interval = tonumber(os.getenv('AUTO_UPDATE_INTERVAL'), 10)
I'd extract that 10 to a constant: DEFAULT_UPDATE_INTERVAL. Also, we are using 10 as the default interval here and 60 in the .env file. I'd rather see the same value in both places for consistency.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Enhancedment: use thread.Thread to execute util.file.determine_syntax_files

It seems that sublime.set_timeout_async still slow down the start up of
Sublime Text. For instance, the quick panel (e.g., git: log) is not ready until a few second
after ST opens. I suspect that they share the same second thread.
Changing sublime.set_timeout_async to threading.Thread so that the quick panel
(and other UI) does not have to wait for util.file.determine_syntax_files

This is good stuff.

Hmm... this one makes me nervous, since it introduces a race condition in accessing syntax_file_map.  I'm going to revert this merge for now, but feel free to re-submit with proper locking mechanisms in place.

@divmain, I don't understand. Can you explain a bit more in detail?

When talking about threading, we have to consider thread safety.  This block of code would run in one thread, while this one would run in a separate thread.  Python does have a GIL, which means that no two threads will be executing simultaneously - it's essentially a thread-level lock that must be acquired by a thread before it can access the shared memory and proceed with execution.
However, this only protects atomic operations.  By my understanding, the GIL will transition from one thread to the next in between bytecode instructions.  In Python, this roughly translates to: single statements on built-ins will operate atomically, without introducing race conditions (where one thread is reading/writing an object while another thread is simultaneously reading/writing).  However, these lines are not atomic.  And even if they were, we don't have any protections in our codebase to ensure that they remain that way.
What I'd prefer here is to Lock the syntax_file_map until writes are complete, and then acquire that lock whenever get_syntax_for_file is invoked.
Here are the docs.

I may be wrong.
when get_syntax_for_file is executed in the main thread, and we acquire a lock for get_syntax_for_file, it may block the main thread. So I use a different approach in https://github.com/divmain/GitSavvy/pull/479 to check the thread of running determine_syntax_files is still alive.


Remove virtual from Get/Set macro definitions

Declared Get/Set member functions non-virtual by default when ITK_FUTURE_LEGACY_REMOVE is set.
Added itkVirtual and itkNonVirtual Get/Set macro's, to explicitly choose between virtual and non-virtual.
Observed a significant reduction in the size of generated "ITK*.lib" files when declaring Get/Set member functions non-virtual by default. For example, "ITKTransformFactory-5.3.lib" went from 18.3 MB down to 17.5 MB, and "ITKCommon-5.3.lib" went from 8.1 MB down to 7.6 MB, using VS2019 (Release).

@N-Dekker   Now that we are at ITK 6.0, can we revisit this?  It seems like now is the time to put effort into evaluating this effort.

Now that we are at ITK 6.0, can we revisit this?

@hjmjohnson I have to think about it. Over the last few years (after submitting this PR) I did encounter some cases where Get and/or Set member functions were overridden. I don't really like that, I think it's error prone. (What if someone overrides the Get, but forgets to override the corresponding Set?) And when a class has a specific data member allocated for a Get and Set, it seems wasteful to override it to bypass the allocated data. But having said so, I know now that this feature of overridable Get and Set is actually being used in practice. So I hesitate.  🤔

im just revisiting old PRs
Get Outlook for iOShttps://aka.ms/o0ukef

From: Niels Dekker @.>
Sent: Monday, November 11, 2024 10:47:47 AM
To: InsightSoftwareConsortium/ITK @.>
Cc: Hans Johnson @.>; Mention @.>
Subject: Re: [InsightSoftwareConsortium/ITK] Remove virtual from Get/Set macro definitions (#2785)
Now that we are at ITK 6.0, can we revisit this?
@hjmjohnsonhttps://github.com/hjmjohnson I have to think about it. Over the last few years (after submitting this PR) I did encounter some cases where Get and/or Set member functions were overridden. I don't really like that, I think it's error prone. (What if someone overrides the Get, but forgets to override the corresponding Set?) And when a class has a specific data member allocated for a Get and Set, it seems wasteful to override it to bypass the allocated data. But having said so, I know now that this feature of overridable Get and Set is actually being used in practice. So I hesitate. 🤔
—
Reply to this email directly, view it on GitHubhttps://github.com/InsightSoftwareConsortium/ITK/pull/2785#issuecomment-2468614570, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AACMU4QGUJ5JSK2LB5SQSU32ADNTHAVCNFSM6AAAAABRSB4MTCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRYGYYTINJXGA.
You are receiving this because you were mentioned.Message ID: @.***>


Added new headers to fix functionality

Added the headers
'bereal-app-version-code': '14549', 'bereal-signature': 'berealsignature', 'bereal-device-id': 'berealdeviceid',
to all api endpoints that use authorization.

This is the solution used in rvaidun/befake@6ae70ca3ecd8b90c1563c5d3939caa8326ce1f10
I hope I didn't miss any but it should be an easy fix if I did.
In my brief testing, this fully restores functionality.

Thank you and rvaidun for clutching up!


Overlay setup.zsh: job table full or recursion limit exceeded

I have two workspaces gazebo9_ws and melodic_ws. When I try to overlay the gazebo9_ws with the melodic_ws I get a tons of errors with the output:
Steps I used to build:
cd ~/workspaces/gazebo9_ws
colcon build --merge-install
source install/setup.zsh

cd ~/workspaces/melodic_ws
colcon build --merge-install
source install/setup.zsh

This is the error output from the second source command:
_colcon_prefix_sh_prepend_unique_value:15: job table full or recursion limit exceeded
_colcon_prefix_sh_prepend_unique_value:15: job table full or recursion limit exceeded
not found: "/share/catkin/hook/unset_catkin_setup_util_args.sh"
_colcon_prefix_sh_prepend_unique_value:15: job table full or recursion limit exceeded
_colcon_prefix_sh_prepend_unique_value:15: job table full or recursion limit exceeded
not found: "/share/cmake_modules/hook/unset_catkin_setup_util_args.sh"
_colcon_prefix_sh_prepend_unique_value:15: job table full or recursion limit exceeded
_colcon_prefix_sh_prepend_unique_value:15: job table full or recursion limit exceeded
not found: "/share/cpp_common/hook/unset_catkin_setup_util_args.sh"

Things I have tried:

Building the underlay without --merge-install: Still breaks
Building the overlay without --merge-install: Works
Building both without --merge-install: Works
Building both with --merge-install, but sourcing ~/workspaces/melodic_ws/install/local_setup.zsh: Works, but doesn't have gazebo stuff.

Here is some output from sourcing with COLCON_TRACE=1
. "/home/michael/workspaces/gazebo9_ws/install/local_setup.zsh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-cmake0/share/ignition-cmake0/package.zsh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-cmake0/share/ignition-cmake0/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-cmake0/lib/cmake/ignition-cmake0/cmake0/share/ignition-cmake0/hook/cmake_module_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-cmake0/share/ignition-cmake0/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-tools/share/ignition-tools/package.zsh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-tools/share/ignition-tools/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-tools/share/ignition-tools/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-tools/share/ignition-tools/hook/path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-math4/share/ignition-math4/package.zsh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-math4/share/ignition-math4/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-math4/share/ignition-math4/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-math4/share/ignition-math4/hook/ld_library_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/SDFormat/share/SDFormat/package.zsh"
. "/home/michael/workspaces/gazebo9_ws/install/SDFormat/share/SDFormat/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/SDFormat/share/SDFormat/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/SDFormat/share/SDFormat/hook/ld_library_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-msgs1/share/ignition-msgs1/package.zsh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-msgs1/share/ignition-msgs1/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-msgs1/share/ignition-msgs1/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-msgs1/share/ignition-msgs1/hook/ld_library_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-transport4/share/ignition-transport4/package.zsh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-transport4/share/ignition-transport4/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-transport4/share/ignition-transport4/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-transport4/share/ignition-transport4/hook/ld_library_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/Gazebo/share/Gazebo/package.zsh"
. "/home/michael/workspaces/gazebo9_ws/install/Gazebo/share/Gazebo/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/Gazebo/share/Gazebo/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/Gazebo/share/Gazebo/hook/ld_library_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/Gazebo/share/Gazebo/hook/path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/Gazebo/share/gazebo/setup.sh"
. "/home/michael/workspaces/melodic_ws/install/local_setup.zsh"
. "/home/michael/workspaces/melodic_ws/install/share/catkin/package.zsh"
. "/home/michael/workspaces/melodic_ws/install/share/catkin/package.sh"
. "/home/michael/workspaces/melodic_ws/install/share/catkin/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/melodic_ws/install/share/catkin/hook/path.sh"
. "/home/michael/workspaces/melodic_ws/install/share/catkin/hook/set_catkin_setup_util_args.sh"
. "/home/michael/workspaces/melodic_ws/install/setup.sh"
. "/home/michael/workspaces/gazebo9_ws/install/local_setup.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-cmake0/share/ignition-cmake0/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-cmake0/lib/cmake/ignition-cmake0/cmake0/share/ignition-cmake0/hook/cmake_module_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-cmake0/share/ignition-cmake0/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-tools/share/ignition-tools/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-tools/share/ignition-tools/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-tools/share/ignition-tools/hook/path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-math4/share/ignition-math4/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-math4/share/ignition-math4/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-math4/share/ignition-math4/hook/ld_library_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/SDFormat/share/SDFormat/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/SDFormat/share/SDFormat/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/SDFormat/share/SDFormat/hook/ld_library_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-msgs1/share/ignition-msgs1/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-msgs1/share/ignition-msgs1/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-msgs1/share/ignition-msgs1/hook/ld_library_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-transport4/share/ignition-transport4/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-transport4/share/ignition-transport4/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/ignition-transport4/share/ignition-transport4/hook/ld_library_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/Gazebo/share/Gazebo/package.sh"
. "/home/michael/workspaces/gazebo9_ws/install/Gazebo/share/Gazebo/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/Gazebo/share/Gazebo/hook/ld_library_path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/Gazebo/share/Gazebo/hook/path.sh"
. "/home/michael/workspaces/gazebo9_ws/install/Gazebo/share/gazebo/setup.sh"
. "/home/michael/workspaces/melodic_ws/install/local_setup.sh"
. "/home/michael/workspaces/melodic_ws/install/share/catkin/package.sh"
. "/home/michael/workspaces/melodic_ws/install/share/catkin/hook/cmake_prefix_path.sh"
. "/home/michael/workspaces/melodic_ws/install/share/catkin/hook/path.sh"
. "/home/michael/workspaces/melodic_ws/install/share/catkin/hook/set_catkin_setup_util_args.sh"
. "/home/michael/workspaces/melodic_ws/install/setup.sh"

Minimal example:
mkdir -p test_ws/src
cd test_ws/src
git clone https://github.com/ros/catkin.git
cd ..
colcon build --merge-install
source install/setup.zsh

Fails with:
➜  test_ws source install/setup.zsh
_colcon_prefix_sh_prepend_unique_value:15: job table full or recursion limit exceeded
not found: "/share/catkin/hook/unset_catkin_setup_util_args.sh"
not found: "/share/catkin/hook/unset_catkin_setup_util_args.sh"
not found: "/share/catkin/hook/unset_catkin_setup_util_args.sh"
not found: "/share/catkin/hook/unset_catkin_setup_util_args.sh"
not found: "/share/catkin/hook/unset_catkin_setup_util_args.sh"
not found: "/share/catkin/hook/unset_catkin_setup_util_args.sh"
...

It seems that it would be useful to know if you are building a merged workspace and skip https://github.com/colcon/colcon-ros/blob/master/colcon_ros/task/ros.py#L105 accordingly, is that possible?


[Feature Request] ComfyUi support please?

Pretty pleaaaase?

ComfyUi is another story even we use pieces of code from this repo.

someone already implemented this feafure for ComfyUI https://github.com/hhhzzyang/Comfyui_Lama

awesome thanks!!


StateMemory and State structs changes

Some StateMemory and State changes, discussion started in https://github.com/spelunky-fyi/overlunky/pull/369#issuecomment-1985325199

Would you mind if i try and make my own version of this PR ?

Maybe it would be even better if you're the one that makes this PR tbh


[Feature Request] Option to Act as a Tag Based Slave/Server

I would like to provide my clients with the ability to control/talk (Read and Write) to my Codesys PLC Program from an Allen Bradley PLC. I have an existing library of code for Rockwell PLCs that do this for talking to our Rockwell PLCs, but I currently have no easy way to do this for our Codesys PLCs.
Reading and Writing from our PLCs to their PLCs, which is what we would have to do now using this library, just isn't feasible and doesn't work well with our program architecture.

If I understand correctly, you're interested in wanting the existing library to act as another EtherNet/IP device so that your Rockwell PLC can talk to a Codesys PLC... I guess similar to how one Rockwell PLC communicates with another via consume/produce tags?

Correct


Support cancellation

Please add CancellationToken to InitializeAsync. This would be a breaking change and thus require to bump the version to 2.0.
Or you can add an overload. To simplify the consumption, try to use new C# feature of default interface Implementation, i.e. the overload without CT pass CT.None to the overload with it. Or you can provide simple abstract class that would do it for me. So I'll have less boilerplate code to write.
Makes sense?

Hi @abatishchev,
Thanks for the suggestion.
To be honest, I thought about this, but couldn't think of a case where it would make sense. As far as I know, there's no way to interrupt the startup of an ASP.NET Core app. Where would the cancellation token come from?
Do you have a specific scenario in mind?

To simplify the consumption, try to use new C# feature of default interface Implementation

I don't think I'd do that. C# 8 and .NET Core 3.0 aren't RTM yet, and pre-.NET Core 3.0 runtimes won't support DIM. Also, .NET Framework will never support it. I'd rather not write a library that only works on .NET Core 3.0 :wink:

You're right, no one (i.e. ASP.NET Core infrastructure) would pass cancellation token. I misused your library so have a way to pass but this is wrong and make no sense. Don't work very late hours :)
If DIM requires .NET Core and won't work on .NET Framework than it sucks.


add unit tests

Unit tests to verify output of template. I added sprig, so there are consistent ways to add whitespace to the output.

Looks good overall. Just wanted to make sure that you tested the resulting CRs - they are pretty finicky about whitespace for multi-line values

I didn't can  you add a test for that in #24 ?


eslint issues?

some issues with eslint?
$ eslint "**/*.{js,json,jsx,ts,tsx}"
$ tsc --noEmit
src/instructor.ts:193:7 - error TS2353: Object literal may only specify known properties, and 'typeDefaults' does not exist in type '{ defaultData?: object | null | undefined; onKeyComplete?: ((data: any) => void | undefined) | undefined; }'.

193       typeDefaults: {

I think this can be closed @jxnl @roodboi


[Feature Request] Allow tree-map with mixed inputs of ordered and unordered dictionaries

Required prerequisites

[X] I have searched the Issue Tracker that this hasn't already been reported. (comment there if it has.)

Motivation

The tree-map function uses a treespec.flatten_up_to to deal with the multi-input scenario. The flatten_up_to method requires the incoming pytree to have the exact same type and metadata as the treespec. For convenience, users may want to map OrderedDict with dict, or vice versa. The tree_map function will raise a error:
In [1]: from optree import *

In [2]: from collections import OrderedDict

In [3]: d = {'a': 1, 'b': 2}

In [4]: od = OrderedDict(d)

In [5]: d
Out[5]: {'a': 1, 'b': 2}

In [6]: od
Out[6]: OrderedDict([('a', 1), ('b', 2)])

In [7]: tree_map(lambda x, y: x + y, d, od)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[7], line 1
----> 1 tree_map(lambda x, y: x + y, d, od)

File ~/Projects/optree/optree/ops.py:460, in tree_map(func, tree, is_leaf, none_is_leaf, namespace, *rests)
    417 """Map a multi-input function over pytree args to produce a new pytree.
    418
    419 See also :func:`tree_map_`, :func:`tree_map_with_path`, and :func:`tree_map_with_path_`.
   (...)
    457     is the tuple of values at corresponding nodes in ``rests``.
    458 """
    459 leaves, treespec = tree_flatten(tree, is_leaf, none_is_leaf=none_is_leaf, namespace=namespace)
--> 460 flat_args = [leaves] + [treespec.flatten_up_to(r) for r in rests]
    461 flat_results = map(func, *flat_args)
    462 return treespec.unflatten(flat_results)

File ~/Projects/optree/optree/ops.py:460, in <listcomp>(.0)
    417 """Map a multi-input function over pytree args to produce a new pytree.
    418
    419 See also :func:`tree_map_`, :func:`tree_map_with_path`, and :func:`tree_map_with_path_`.
   (...)
    457     is the tuple of values at corresponding nodes in ``rests``.
    458 """
    459 leaves, treespec = tree_flatten(tree, is_leaf, none_is_leaf=none_is_leaf, namespace=namespace)
--> 460 flat_args = [leaves] + [treespec.flatten_up_to(r) for r in rests]
    461 flat_results = map(func, *flat_args)
    462 return treespec.unflatten(flat_results)

ValueError: Expected dict, got OrderedDict([('a', 1), ('b', 2)]).

NOTE: this feature is not backward compatible.
Solution
Reimplement the PyTreeSpec::FlattenUpTo method. Use set equality and ignore the key orders.
Alternatives
No response
Additional context
No response

Other than sorting the keys, we can check the key equality by iterating over the keys and calling PyDict_Contains.


[nuxt] A composable that requires access to the Nuxt instance was called outside of a plugin, Nuxt hook, Nuxt middleware, or Vue setup function. This is probably not a Nuxt bug.

Hello people, im obtaining this error on access /login page
500
[nuxt] A composable that requires access to the Nuxt instance was called outside of a plugin, Nuxt hook, Nuxt middleware, or Vue setup function. This is probably not a Nuxt bug. Find out more at https://nuxt.com/docs/guide/concepts/auto-imports#vue-and-nuxt-composables.

Hey @GabrielFrohlich, can you provide a bit more details about the issue? Are you using a breeze-nuxt template or is it your own project?
From the error it looks like there is an incorrect place for the function call, but I need a code sample to help with solving it.

Hello @manchenkoff
Im using my own project, and it happens every page in the application.

@GabrielFrohlich I see, since it is a custom app, can you provide minimal repro where the issue also occurs? Unfortunately, in the fresh install and in the breeze-nuxt it works fine, so we need to reproduce your environment

@manchenkoff it was my error, I was calling login direct on script setup.
`const { login } = useSanctumAuth();
const userCredentials = {
email<EMAIL_ADDRESS>password: '123123',
};
await login(userCredentials)`


Compile in all BC localizations

Hey Waldemar,
I tried to install your app on BC17 Spanish localization and I got this error message:
Extension compilation failed
src/codeunit/AdminToolMgt.Codeunit.al(252,35): error AL0132: 'Database' does not contain a definition for 'Delivery Reminder Header'
src/codeunit/AdminToolMgt.Codeunit.al(252,89): error AL0132: 'Database' does not contain a definition for 'Phys. Invt. Diff. List Buffer'
So I have forked your repo to find the error and after downloading symbols I found this:

I thought it can't compile because the localization. In my database I don't have objects in the 5005270 - 5005363 range (Deutsch objects?). Instead I have in the 7000002 - 7000024 range (Spanish objects).
Changing tablenames to numbers make it work. I know this is not the cleanest solution in the world but this way this app could be growing to allow more localizations.

Here you have a pull request with these changes. Use it if you want to.
PD: Great work with this toolbox!!

Hey Ruben,
thank you a lot for creating this issue with the pull request. I am currently working on a few improvements for this app and will include your suggested solution. If you use the app, you should check the changes in the next 2 days as they might be useful for you. :-)
Feedback from people who use this app helps me a lot because sometimes I just cannot test it for all the different circumstances and environments. So thanks again and have a nice weekend.
Kind regards,
Waldemar


[ACR] az acr connected-registry install: deprecate ACR_SYNC_TOKEN_USERNAME from the output

Description
ACR_SYNC_TOKEN_USERNAME is being deprecated from the install command group. The new value will be ACR_SYNC_TOKEN_NAME
Testing Guide
az acr connected-registry install info -n  -r
az acr connected-registry install renew-credentials -n  -r
History Notes
[ACR connected-registry] az acr connected-registry install info: Add new key ACR_SYNC_TOKEN_NAME with the same value as ACR_SYNC_TOKEN_USERNAME. A warning that the latter will be deprecated is displayed.
[ACR connected-registry] az acr connected-registry install renew-credentials: Add new key ACR_SYNC_TOKEN_NAME with the same value as ACR_SYNC_TOKEN_USERNAME. A warning that the latter will be deprecated is displayed.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Can we update the notes as follows?
History Notes
[ACR connected-registry] az acr connected-registry install info: Added new key ACR_SYNC_TOKEN_NAME with the same value as ACR_SYNC_TOKEN_USERNAME. A warning that the latter will be deprecated is displayed.
[ACR connected-registry] az acr connected-registry install renew-credentials: Added new key ACR_SYNC_TOKEN_NAME with the same value as ACR_SYNC_TOKEN_USERNAME. A warning that the latter will be deprecated is displayed.

Can we update the notes as follows?
History Notes
[ACR connected-registry] az acr connected-registry install info: Added new key ACR_SYNC_TOKEN_NAME with the same value as ACR_SYNC_TOKEN_USERNAME. A warning that the latter will be deprecated is displayed.
[ACR connected-registry] az acr connected-registry install renew-credentials: Added new key ACR_SYNC_TOKEN_NAME with the same value as ACR_SYNC_TOKEN_USERNAME. A warning that the latter will be deprecated is displayed.

@fengzhou-msft  can you help review the pr please? It's ready for merge

@fengzhou-msft  can you help review the pr please? It's ready for merge

ACR

ACR

@yonzhan can you merge the pr?

@yonzhan can you merge the pr?

@fengzhou-msft to help with this PR.

@fengzhou-msft to help with this PR.


Linux musl support

Something wrong happen when run jest with @swc-node/jest in Jenkins pipeline.
But the same operation is working on local mac machine.
Here is the error message posted by Jenkins pipeline.
TypeError: Not compatible with your platform. Error message: Error loading shared library ld-linux-x86-64.so.2: No such file or directory

Seems like it hasn't glibc installed in your pipeline execution environment. Do you running pipeline in Alpine Linux or something like its?

I will add linux x64 musl support tonight.


Development

Add -profile command
Change rating command to use lichess.api instead of scraping the profile page


the code for calculating the FLOPS of sparse convolution

Could you please release the code for calculating the FLOPS of sparse convolution?

and could you please give me an example of how to use the code? Thank you so much!


I cleaned up imports with all the js files in root

I deleted all the files in root then added
"exports": {
	"./*": "./src/*.js",
	"./flat/*": "./flat/*.js"
}

to package json so we still have all the imports stay the same but now the code isn't all over root and is easier to understand.
I also tested it out by linking the project and it worked great.

This is superseded by #8 which I will finish... soon.. I promise.


Scaeva bonus crit chance isn't working

Coincidentally I've been messing with the item too, but it works fine for me. Are you sure you have it equipped in your main hand? Or perhaps using the wrong weapon set?

I can't find any problems with it either. I should note that the "increased Critical Strike Chance when in Main Hand" is global, so it doesn't alter the crit chance listed in the tooltip.

Sorry, now that I checked again - it is indeed working


IE(7 and 8) Exception identifier in JavaScriptCompiler

IE 7 and 8 are throwing an Exception identifier error in the JavaScriptCompiler createFunction() aprox. line 1135.
var fn = Function.apply(this, params);
Viewing the params[3] the JavaScript syntax in the string appears to using the ';' correctly. This 'Error on page' is perplexing.
I see that other Issues recommend updating to Jison 0.2.9+, but this does not seem to resolve the issue.
$ jison --version
0.2.9

I use jison 2.0.11, and it works fine on IE7/8.

@danielstocks really? I use Handlebars.js extensively in IE6-8 with no problems.

I have this issue too.

Can those of you experiencing this issue attempt to provide more information about it. Many of us have used it without incident so it's very difficult for us to debug without any additional information. Providing the params that are used in the failing Function.apply call could be helpful.

I had the issue, however whatever was causing it is gone. The issue doesn't exist in your master branch anymore as far I can tell.

Presuming resolved.


refactor: always use a cache directory

Description
I'm currently working #4120 and I found that it adds some complexity to handle a nullable cacheDir. The only case it can happened is when there is no package.json detected, which seems a niche case. I suggest to use .vite as a cache directory in that case. We could also keep the default of node_modules/.vite but it makes less sense in that case.
Additional context
While looking for usage of cacheDir, I found that the caching of the certificate was dropped in #5514. I adding it back except if there was a reason to remove it (cc @patak-dev)
Note: This could technically be a breaking change for people using a global installation of Vite in projects that don't use a package manager.

@ArnaudBarre after a discussion with @dominikg, looks like global usage of vite may also be affected by this change.
Would you split this PR in two so we can merge the cache dir fix before releasing 2.8? Thanks!

Let me know about the decision for this change.
Another option as suggested by @gluck is to allow to disable caching. This would be easy (adding | false) to the cacheDir option and updating some messages. But it would start to conflict with server.force (and would keep adding a bit of complexity each time a new feature need to handle file system caching)

I'm leaning towards allowing to disable caching. We're only going to be able to discuss this in the next team meeting (next Friday), I'll bring the topic there.

@ArnaudBarre @dominikg we talked about this PR in the last team meeting, and we concluded that it is a good idea to always have a cache. The DX in Vite is heavily damaged without it, and we don't have a real use case at hand for justifying the complexity of allowing Vite to run without write access during dev. Vite can also be safely run globally if the cache is held at the root of the Vite project. @ArnaudBarre would you resolve the conflicts and confirm that this looks good to you?

Just to be clear, this should land in a minor release due to this potential breaking change:
https://github.com/vitejs/vite/pull/6415/files#diff-db0f5f8dbaa7b2beda67cfc9f12bc0d22dedbbc7ed3d1935b3fdd5a3a7991350R125
Potentially this could be access from outside

What do you mean by "potentially this could be access from outside"?
The idea is to merge it now that we are in the 2.8 beta period, so it will be included as part of a minor 👍🏼

I just mean it is now a required parameter and typescript would report an error, nothing more
But as you say it is already planned to 2.8, everything is fine :slightly_smiling_face:

But resolveHttpsConfig is an internal function, no?

export async function resolveHttpsConfig
^^^^^^

That function is only imported as an internal utility by other files, or maybe I'm not seeing where it is exported to the user

@patak-dev https://www.typescriptlang.org/tsconfig#stripInternal :eyes:


ruby sdk release notes added

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: \ruby sdk release notes added
Jira/GitHub Issue numbers (if any): \ FFM-12203
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[x] Successful preview build.
[x] Code owner review.
[x] No merge conflicts.
[x] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://67349ad96c8bcfff980bfdc3--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6734d35472f7b438aab0f2d3--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://673623f65c59e817b2219fe1--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://673c85ed9e9aa50705b38bf3--harness-developer.netlify.app


Question: System configuration

I know there is Nix, NixOps, and NixOS. Let's say I wanted to control the permissions of all the created users by using ACLs, or I wanted $HOME/mydir to exist for all users (typically something done with tools like Puppet, Chef, etc.) or I wanted to do a checkout of some Mercurial repository as part of a nix-rebuild switch. How can that be controlled?
It appears the language simply isn't there to express these things or am I missing something?

To checkout a mercurial repository you can use fetchhg. Nix and nixos aren't meant to manage your home directory. What kind of ACL permissions do you want to set? Because everything is in /nix/store I don't know if there are too many permissions you have to manage. Home directories belong to the user and everything else is managed by nix. Nix expressions aren't meant to replace shell script or build tools, but are only meant to describe packages and configuration.
It kind of looks like you are looking for someone to convince you to use nix/nixos. If you don't know why you want to use nix, then you probably don't need it, and will save yourself a lot of frustration with something more mainstream ;)

@dvc94ch I don't mind writing shell scripts, but where can I put such homedir shell script management logic then?
Fetchhg is also again in a build output; I just want to put it in an arbitrary location on the filesystem.
This is also exactly what I asked. I didn't specifically ask for the Nix language; I asked whether there was any way to express this. Otherwise one still needs to resort to Puppet/Chef/ to do these kinds of things.
I think NixOS modules can do this, myself, because otherwise https://www.infoq.com/articles/configuration-management-with-nix would not make sense; it wouldn't be a complete configuration management system.

Maybe nixos-option environment.shellInit or nixos-option programs.bash suit your needs. To put it in an arbitrary location on the filesystem you'll have to add a mercurial client to the environment packages and a script. Why would you want it to be part of a nixos-rebuild switch?

@dvc94ch How can it be complete if it is not possible to set all properties of the system (this includes settings using interfaces like setfacl under the hood, etc.? I don't subscribe to the theory that one should draw an arbitrary line somewhere and say that managing "desktop wallpapers" is not part of configuration management; it is. If a solution is not able to manage such settings in some way, then it's simply an incomplete system. In the case of users, I am not sure what the interaction is between NixOS's declarative users and setting the permissions of home dirs via e.g. setfacl.
In theory one should be able to override the users module to make a myusers module, and then use that to do the actual setfacl commands.

@0xABAB

I don't subscribe to the theory that one should draw an arbitrary line somewhere and say that managing "desktop wallpapers" is not part of configuration management; it is.

I think, Nixos is incomplete in this sense. There is an authority, that defines what parts of a system should be available to all NixOS users.
Their is bunch of override mechanisms in Nixos to overcome this issue, but often you have to copypaste some solution (I'm talking about nixos modules now) and heavily patch code.

It appears the language simply isn't there to express these things

Hardly true. You can build your own nixos by patching nixpkgs sources when overrides are too limited.

I wanted to do a checkout of some Mercurial repository as part of a nix-rebuild switch

  system.activationScripts.fetch-some-hg-on-nixos-rebuild = {
    text = ''
          mkdir -p "$hgfolder"
          if [ ! -d "$hgfolder/.hg" ]; then
            echo "Creating repository $hgfolder for $hgurl..."
            cd $hgfolder && ${pkgs.mercurial}/bin/hg init ${exts}
            echo -e "[paths]\ndefault = $hgurl" > $hgfolder/.hg/hgrc
            echo "Finished creating repository!"
          fi
          ${pkgs.mercurial}/bin/hg pull -u ${exts} -R $hgfolder
    '';
    deps = [];
  };

(no timeout, but easy to add)

@danbst Interesting example.
I have no idea what you meant by "There is an authority, that defines what parts of a system should be available to all NixOS users.". Can you explain what you meant?


Fix documentation typos and make style more consistent

I've been reading gitbase documentation to get used to the project and the codebase, so I fixed the typos and errors I found.
Changes:

Fix a malformed query in examples.md. It used the non-existing commit_blob column where it should use blob_hash.
Make description of functions consistent throughout all the tables in functions.md. I decided to stick to the lower-case present-tense style, as each row is easily read as Function X returns Y. This is opinionated (consistency should be not), so I'm ok discussing the best style and changing it again once we agree on one.
Fix several typos.

(Not adding the PR template here because this only updates typos in documentation, so none of the items apply here)

Oops, forgot to sign-off the commit. Solved.

Removed the changes in standard functions in functions.md and all changes in supported_clients.md here. Opened https://github.com/src-d/go-mysql-server/pull/804.


Rename notice component

https://trello.com/b/mn2MqH3M/publishing-workflow-q4-2019-doing

We need some updates on the functional tests though

@alex-ju thanks - I wasn't expecting us to be using those classes in the tests :-). Anyway, could you approve again, following the changes?


New version: gopass.gopass-jsonapi version 1.15.13

Pull request has been created with komac v2.2.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\gopass\gopass.exe returned exit code: 6
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\gopass.gopass-jsonapi__DefaultSource\gopass-jsonapi.exe returned exit code: 1
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\gopass-jsonapi.exe returned exit code: 1

(Automated response - build 876.)

@wingetbot waivers Add Validation-Executable-Error


Fix color of image masks inside uncolored patterns

Fixes #8741, #3226.
This is a patch for uncolored (/PaintType 2) tiling patterns that contain image masks. When uncolored patterns are used as a fill or stroke, the operator scn or SCN specifies the color of the pattern. If the pattern contains an image mask, PDF.js incorrectly draws it using black instead of the color specified by scn or SCN.
In my patch, I assign the proper color to the fillColor property of the CanvasExtraState object. That property is used as the color of image masks in functions paintImageMaskXObject...() in canvas.js.

do we need to do that for this.current.strokeColor =?

I don't know if any PDF would need this but I added it, just to be sure.

do we need to do the same for PaintType.COLORED case?

I added lines similar to the other case. Colored tiling patterns are a bit different: now the color argument in setFillAndStrokeStyleToContext() is meaningless (or null) because the PDF operators scn and SCN have no color operands.

/botio-linux preview

/botio test

Fixes #8741, #3226.

Would it be possible to include the PDF file from either of those issues as a linked test-case, to prevent future regressions?

I added the test case.
By the way, my patch seems to fix also issue #8860.

I added the test case.

Thank you; one question though, wouldn't it be enough to just test the first page though?
If so, please add "lastPage": 1, to entry in test_manifest.json

/botio test

@janpe2 Looking at the test logs above, it appears that the server prevented the PDF file from being downloaded:
Downloading http://arxiv.org/pdf/0705.4298.pdf to pdfs/issue3226.pdf...
Error during downloading of http://arxiv.org/pdf/0705.4298.pdf: HTTP 403
WARNING: File was not downloaded. See "pdfs/issue3226.pdf.error" file.
Unable to verify the checksum for the files that are used for testing.
Please re-download the files, or adjust the MD5 checksum in the manifest for the files listed above.

Would it be possible to link to the PDF file through e.g. http://archive.org/web/ instead, or do we need to use another PDF file for the test-case?

/botio test

What are those failed regression tests? Should I be worried about them?

The problem is that the old file is not replaced. @yurydelendik Could you remove the old issue3226.pdf file from both bots so we can try again?

The problem is that the old file is not replaced. @yurydelendik Could you remove the old (cached) issue3226.pdf file from both bots so we can try again?

I don't think that's really the issue at play here, since the (Internet Archive) link does not actually point to a PDF file.
If you open https://web.archive.org/web/20170830175223/http://arxiv.org/pdf/0705.4298.pdf directly in a browser, you'll get the same thing that the test-suite gets when downloading the file: a HTML page which references a PDF file.
It seems to me that linking to this particular PDF file probably will not work, and that the easiest solution here would be to try and use a PDF file from one of the other referenced issues as a test-case instead.

/botio test

/botio makeref


Fix tcp_proxy: mark done reading on upstream trailers

When using HTTP/2 upstream with TCP tunneling feature, the only place the reading is done (i.e using "doneReading()" callback) is in cases the last data frame from the upstream also contains the NGHTTP2_FLAG_END_STREAM flag. This way the decodeData callback will have "end_stream" to true.
In certain streaming cases, the downstream sends delayed FIN packet and the server will not respond with a data frame, rather than a headers frame which will be the trailers data and an END_STREAM flag. This causes "doneReading()" callback to be skipped and the encoder would not be reset, which causes the filter chain not to drain.
Fixes #24704

@alyssawilk can you take another look? Had to close the previous PR and open a new one. I have added tests to the changes.

@alyssawilk can we merge this one? :)


Popover of use profile appears when clicked on mentions.

I have updated popover as required please review it , regarding issue #6380 in which they want to reveal the profile picture popover on clicking over @mentions in chat.
I hope it works :)

I think this isn't the feature described in that issue; you should make clicking on a mention (e.g. @**King Hamlet** ) open the popover; it's already the case that clicking a user's name opens the popover :).

I have changed the popover.js according to the requirements hope it works fine now. I would like to suggest that instead of click we can show popover on hover on the usernames and keep the click for mentions .

 Would be happy to merge afterwards since manual testing works great.

@synicalsyntax  i hope it is fine now ?

You changed the pull request name but your commit name is still "Added popover to mentions"

Closing, since @chocos10 doesn't seem to still be working on this.  Thanks for your time @chocos10!


OTAR042 -Tractability: pipeline implementation

Implementation of data injection into gene-data index as outlined in the Data Specification document: https://docs.google.com/document/d/1ezYkdU260IO-sqjswsidATvXgiykRbp71dAoy7ym2tU/edit

Implementation done and PR approved.


Error while initializing the Dart VM: Snapshot not compatible with the current VM configuration (on x64 hosts)

LD_LIBRARY_PATH=./flutter-embedded-linux/build/  ./flutter-embedded-linux/build/flutter-client -b ~/pj/sample/build/elinux/x64/release/bundle
when i run the flutter app
it shows the follow error
[FATAL:flutter/runtime/dart_vm_initializer.cc(22)] Error while initializing the Dart VM: Snapshot not compatible with the current VM configuration: the snapshot requires 'product no-code_comments no-dwarf_stack_traces_mode lazy_async_stacks no-lazy_dispatchers use_bare_instructions dedup_instructions no-asserts x64-sysv compressed-pointers null-safety' but the VM has 'product no-code_comments no-dwarf_stack_traces_mode lazy_async_stacks no-lazy_dispatchers use_bare_instructions dedup_instructions no-asserts x64-sysv no-compressed-pointers null-safety'
Aborted (core dumped)

Thank you for your report.
This is due to gen_snapshot. I guess you use the pre-build data in https://github.com/sony/flutter-embedded-linux/releases/tag/40a99c5951. If so, please use gen_snapshot that you built yourself.
See also: https://github.com/sony/flutter-elinux/issues/70

@HidenoriMatsubayashi can you explain the work around? Debug mode has terrible performance and testing with it for now is very difficult.

can you explain the work around?

Can you replace ${INSTALL_PATH}/flutter-elinux/flutter/bin/cache/artifacts/engine/android-arm64-*/linux-x64/gen_snapshot to your gen_snapshot  that you built?
See also: https://github.com/sony/flutter-elinux/issues/75

Is it strange that I can observe the issue on my x64 desktop but not when compiling my arm devices  :thinking:

Is it strange that I can observe the issue on my x64 desktop but not when compiling on my arm devices

Yes, that's right. This problem will happen when you build on x64 hosts. When building on an arm64 host, gen_snapshot for Linux is used. See: https://github.com/sony/flutter-elinux/blob/main/lib/commands/build.dart#L70

can you explain the work around?

Can you replace ${INSTALL_PATH}/flutter-elinux/flutter/bin/cache/artifacts/engine/android-arm64-*/linux-x64/gen_snapshot to your gen_snapshot  that you built? See also: sony/flutter-elinux#75

I don't fully understand how to do this because I'm not really configuring a lot of stuff. All I do is provide an arm64 sysroot and I run build, how can I do this?


Can't select text in wunderbar by dragging cursor over it in app

Bug
Normally you can select text by dragging the cursor over the text and highlighting it. However, in the wunderbar if you try to do this, it acts like you are grabbing the app and the entire app moves across the screen.
To Reproduce
Steps to reproduce the behavior:

Open the app
Enter some text in the wunderbar
Attempt to select it by dragging over the text with the cursor
Watch the entire app window move with your cursor!

Expected behavior
The desired text should be able to be selected.
Screenshots

System Configuration

LBRY Daemon version: 0.72.0
LBRY App version: 0.45.1
LBRY Installation ID: 6FW8Z6nraRjKxu7DxhayqYgMtHgovjAAKswHTSfhQYarw42MM289CWoAUzfc1CLkRi
Operating system: Mac OS 19.4.0

Additional context
Add any other context about the problem here.

Looking into this now...


Fixed bug in score calculation

The score calculation function was wrong. Fixing it.
Also made the config upload script more robust

Looks great! 🚀


Memmory leak Using tensorflow::NewSession , C++

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes.
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:N/A
TensorFlow installed from (source or binary):Source
TensorFlow version (use command below): 1.12.0-rc0
Python version: Python 2.7.12
Bazel version (if compiling from source): 0.18.0
GCC/Compiler version (if compiling from source): 5.4.0
CUDA/cuDNN version: N/A. Using Cpu Only.
GPU model and memory: N/A. Using Cpu Only.
Exact command to reproduce:

Code
class Temp
{
 public:
  Temp(){
    std::unique_ptr<tensorflow::Session> session(
      tensorflow::NewSession(tensorflow::SessionOptions()));
    };
};

int main(int argc, char **argv)
{
  Temp temp;
  return 0;
}

Describe the problem
When creating a unique_ptr to a session using tensorflow::NewSession, a memory leak occurs according to address sanitizer. There is a similar closed issue posted here: https://github.com/tensorflow/tensorflow/issues/19163 , however I am still having an issue.
Source code / logs
==259==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 24 byte(s) in 1 object(s) allocated from:
    #0 0x7f17c81b7532 in operator new(unsigned long) (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x99532)
    #1 0x7f17c1c1aca7 in tensorflow::RegisterXlaDeviceKernels(char const*, char const*) (/tensorflowincludes/bin/libtensorflow_cc.so+0x1cd2ca7)
    #2 0x7f17c1c0e19a in tensorflow::XlaCpuDeviceFactory::CreateDevices(tensorflow::SessionOptions const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::vector<tensorflow::Device*, std::allocator<tensorflow::Device*> >*) (/tensorflowincludes/bin/libtensorflow_cc.so+0x1cc619a)
    #3 0x7f17bf851fe8 in tensorflow::DeviceFactory::AddDevices(tensorflow::SessionOptions const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::vector<tensorflow::Device*, std::allocator<tensorflow::Device*> >*) (/tensorflowincludes/bin/libtensorflow_framework.so+0x62dfe8)
    #4 0x7f17c61ccea4 in tensorflow::DirectSessionFactory::NewSession(tensorflow::SessionOptions const&, tensorflow::Session**) (/tensorflowincludes/bin/libtensorflow_cc.so+0x6284ea4)
    #5 0x7f17bf8abfe8 in tensorflow::NewSession(tensorflow::SessionOptions const&, tensorflow::Session**) (/tensorflowincludes/bin/libtensorflow_framework.so+0x687fe8)
    #6 0x7f17bf8ac0ac in tensorflow::NewSession(tensorflow::SessionOptions const&) (/tensorflowincludes/bin/libtensorflow_framework.so+0x6880ac)
    #7 0x402bc2 in Temp::Temp() /var/jenkins_home/workspace/tensorflowleaktest_master-FVC6RQR2GX36IO3JBYVHBT3IJH3EQUQFR2B3OPUAPWVP7LOWN2WQ/tests/src/main.cpp:38
    #8 0x402245 in main /var/jenkins_home/workspace/tensorflowleaktest_master-FVC6RQR2GX36IO3JBYVHBT3IJH3EQUQFR2B3OPUAPWVP7LOWN2WQ/tests/src/main.cpp:44
    #9 0x7f17be6c582f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2082f)

Indirect leak of 4096 byte(s) in 1 object(s) allocated from:
    #0 0x7f17c81b7532 in operator new(unsigned long) (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x99532)
    #1 0x7f17c1c1ab50 in void std::vector<std::unique_ptr<tensorflow::kernel_factory::OpKernelRegistrar, std::default_delete<tensorflow::kernel_factory::OpKernelRegistrar> >, std::allocator<std::unique_ptr<tensorflow::kernel_factory::OpKernelRegistrar, std::default_delete<tensorflow::kernel_factory::OpKernelRegistrar> > > >::_M_emplace_back_aux<tensorflow::kernel_factory::OpKernelRegistrar*>(tensorflow::kernel_factory::OpKernelRegistrar*&&) (/tensorflowincludes/bin/libtensorflow_cc.so+0x1cd2b50)
    #2 0x7f17c1c1ae7e in tensorflow::RegisterXlaDeviceKernels(char const*, char const*) (/tensorflowincludes/bin/libtensorflow_cc.so+0x1cd2e7e)
    #3 0x7f17c1c0e19a in tensorflow::XlaCpuDeviceFactory::CreateDevices(tensorflow::SessionOptions const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::vector<tensorflow::Device*, std::allocator<tensorflow::Device*> >*) (/tensorflowincludes/bin/libtensorflow_cc.so+0x1cc619a)
    #4 0x7f17bf851fe8 in tensorflow::DeviceFactory::AddDevices(tensorflow::SessionOptions const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::vector<tensorflow::Device*, std::allocator<tensorflow::Device*> >*) (/tensorflowincludes/bin/libtensorflow_framework.so+0x62dfe8)
    #5 0x7f17c61ccea4 in tensorflow::DirectSessionFactory::NewSession(tensorflow::SessionOptions const&, tensorflow::Session**) (/tensorflowincludes/bin/libtensorflow_cc.so+0x6284ea4)
    #6 0x7f17bf8abfe8 in tensorflow::NewSession(tensorflow::SessionOptions const&, tensorflow::Session**) (/tensorflowincludes/bin/libtensorflow_framework.so+0x687fe8)
    #7 0x7f17bf8ac0ac in tensorflow::NewSession(tensorflow::SessionOptions const&) (/tensorflowincludes/bin/libtensorflow_framework.so+0x6880ac)
    #8 0x402bc2 in Temp::Temp() /var/jenkins_home/workspace/tensorflowleaktest_master-FVC6RQR2GX36IO3JBYVHBT3IJH3EQUQFR2B3OPUAPWVP7LOWN2WQ/tests/src/main.cpp:38
    #9 0x402245 in main /var/jenkins_home/workspace/tensorflowleaktest_master-FVC6RQR2GX36IO3JBYVHBT3IJH3EQUQFR2B3OPUAPWVP7LOWN2WQ/tests/src/main.cpp:44
    #10 0x7f17be6c582f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2082f)

Indirect leak of 323 byte(s) in 323 object(s) allocated from:
    #0 0x7f17c81b7532 in operator new(unsigned long) (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x99532)
    #1 0x7f17c1c1ad87 in tensorflow::RegisterXlaDeviceKernels(char const*, char const*) (/tensorflowincludes/bin/libtensorflow_cc.so+0x1cd2d87)
    #2 0x7f17c1c0e19a in tensorflow::XlaCpuDeviceFactory::CreateDevices(tensorflow::SessionOptions const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::vector<tensorflow::Device*, std::allocator<tensorflow::Device*> >*) (/tensorflowincludes/bin/libtensorflow_cc.so+0x1cc619a)
    #3 0x7f17bf851fe8 in tensorflow::DeviceFactory::AddDevices(tensorflow::SessionOptions const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::vector<tensorflow::Device*, std::allocator<tensorflow::Device*> >*) (/tensorflowincludes/bin/libtensorflow_framework.so+0x62dfe8)
    #4 0x7f17c61ccea4 in tensorflow::DirectSessionFactory::NewSession(tensorflow::SessionOptions const&, tensorflow::Session**) (/tensorflowincludes/bin/libtensorflow_cc.so+0x6284ea4)
    #5 0x7f17bf8abfe8 in tensorflow::NewSession(tensorflow::SessionOptions const&, tensorflow::Session**) (/tensorflowincludes/bin/libtensorflow_framework.so+0x687fe8)
    #6 0x7f17bf8ac0ac in tensorflow::NewSession(tensorflow::SessionOptions const&) (/tensorflowincludes/bin/libtensorflow_framework.so+0x6880ac)
    #7 0x402bc2 in Temp::Temp() /var/jenkins_home/workspace/tensorflowleaktest_master-FVC6RQR2GX36IO3JBYVHBT3IJH3EQUQFR2B3OPUAPWVP7LOWN2WQ/tests/src/main.cpp:38
    #8 0x402245 in main /var/jenkins_home/workspace/tensorflowleaktest_master-FVC6RQR2GX36IO3JBYVHBT3IJH3EQUQFR2B3OPUAPWVP7LOWN2WQ/tests/src/main.cpp:44
    #9 0x7f17be6c582f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2082f)

SUMMARY: AddressSanitizer: 4443 byte(s) leaked in 325 allocation(s).

Nagging Assignee @cy89: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Hello, I met the same problem, my TF version is 1.3. Anybody could help us solve the problem?

I do not think there is an actually leak. The relevant code:
static XlaDeviceOpRegistrations* registrations =
RegisterXlaDeviceKernels(DEVICE_TPU_NODE, DEVICE_TPU_XLA_JIT);
(void)registrations;
So we should have a global reference to the objects but it is possible the variable gets optimized away.
Do you have exact instructions (like build command) so i can reproduce the error.
Thanks

The options are :
/usr/bin/c++
-I/usr/local/include
-I/home/developer/workspace/tensorflowleaktest/build/../projects/include
-I/home/developer/workspace/tensorflowleaktest/build/../tests/include
-I/home/developer/tensorflow -I/home/developer/tensorflowincludes/bin
-I/home/developer/tensorflowincludes/genfiles
-I/usr/local/lib/python3.5/dist-packages/tensorflow/include
-I/usr/local/lib/python2.7/dist-packages/tensorflow/include
-I/home/developer/tensorflowincludes/google
-I/tensorflow
-I/tensorflowincludes/bin
-I/tensorflowincludes/genfiles
-I/home/developer/workspace/tensorflowleaktest/build/../libs/abseil-cpp
-Wall
-fprofile-arcs
-ftest-coverage
-g
-std=c++14
-O0
-Wall
-fprofile-arcs
-ftest-coverage
-g
-std=c++14
-O0
-fsanitize=address
-fno-omit-frame-pointer
-pthread
-std=gnu++14
-o CMakeFiles/tensorflowleaktest.dir/home/developer/workspace/tensorflowleaktest/tests/src/main.cpp.o
-c /home/developer/workspace/tensorflowleaktest/tests/src/main.cpp
The full output of the build command with the make can be found here attached.
build_text.txt
Additionally a pre-compiled .ii file is attached below in a zip file. (A command -save_temps was passed in to create this file. )
precompiled.zip

This problem seems to still exist on Tensorflow 1.15

Problem exists in TF 2.x

Any updates on this?

Tensorflow 1.x is not supported anymore.

@bhack I guess the best way of dealing with bugs is to wait for the EOL of the product.

The point is that TF has not a public EOL policy.
Also the new bug fixes are available generally in nightly or in the next release as we don't have patch releases other then for security bugs.

Is this still an issue? (In latest TF/patches)

@Cameron-Chancey,
Sorry for the late response. Can you please try with the latest stable version (TF2.9.1) and let us know if issue still persists? Thank you.


Added design doc for new rb algorithm

Summary
Created a design doc for the new algorithm for RB experiments under docs/api.
Details and comments
This is based on the implementation in PR #851. Since the implementation brought up many discussion items, I wrote this design docs. Please enter your feedback below. In particular, I raised several questions in bold in the document. @nkanazawa1989 , @yaelbh - you might want to copy your comments here.

As we've already discussed in various contexts, an experiment has several different functionalities: build (transpiled or non-transpiled) circuits, run them, analyze, save to the database. We've already noticed that the circuit building should become more independent (beyond override of the _transpiled_circuits method). By "more indpendent", I mean the ability of a user to specify the (usually transpiled) circuits as an input to the run. Then I'd imagine some procedure that looks roughly like this:

Outside of the experiment flow, outside of BaseExperiment.run and everything - a user saves whatever is required for her to generate transpiled RB circuits (can be e.g. the circuits themselves, or Clifford decompostions).  She can write her own code for this, and also there can be code in qiskit-experiments to help her do it, but probably as some utility and not as part of BaseExperiment or even of StandardRB. The qiskit-experiments code can refrain from writing to files; instead it returns Python objects, which the user can dump by herself.
The user generates the transpiled circuits. This again happens outside of the main flow, and can even be done by code that the user writes. The user can save them, if she wants (she's the one to invoke the commands that write to files).
The user retrieves the transpiled circuits. Perhaps she has just generated them, or she reads them from a file where she stored them.
The user runs the experiment, with the transpiled circuits specified in the input.

Other places where we've recently encountered the same topic:

The request for modularity came up, to my understanding, in conversations of @chriseclectic with users.
PR #815 that @chriseclectic started and @ItamarGoldman is about to continue - about circuit caching. Note by the way that Haggai and I would like a follow-up PR to save the cache to a file, which can be loaded later. So we see that not only circuit caching - also file handling is a recurring issue.
@ItamarGoldman has suggested to pre-transpile say 1000 RB circuits, then in every execution sample one of them.
In the whole-backend PR #859, I build something I've named BasicExperiment, which is an experiment whose only purpose is to store transpiled circuits.
@coruscating has asked, in the context of QV: "Would be nice to have standalone functions in experiments to help with playing around with the data (for example method to allow loading custom circuits while keeping analysis the same)".


Update TicketField custom field dropdown items

Missing endpoints to manage ticketfield custom dropdown menu options.
Even using getTicketField, is not possible to create Field.Option new option because it's private.

Fixed with https://github.com/cloudbees/zendesk-java-client/pull/174


Parse negative numbers from envvar Fix #799 and Fix #585

TOML reads signed integers as int on Python
num = -1

turns to
assert settings.num == -1
assert isinstance(settings.num, int)

On validators, when is_type_of=str is passed we need to bypass this conversion and force a str

Hello @rochacbruno, do you know when you're aiming to release the next version into pypi? We are hoping to utilize this fix.

@rwbeauchemin I am going on vacation for 2 weeks, I want to do more tests before releasing a minor version, maybe we add a 3.1.10 before the 3.2.0 with only the bug fixes.
However it will be in 2 weeks from now.


Update HttpLatencyMonitoringFilter.scala

I think this was closer to what was intended and believe the current implementation was a typo

Hi @hderms – thanks for this, you're right. One problem though: the code isn't compiling because ServiceLabeller takes type parameters: it needs to be ServiceLabeller[Request, Response].

@samstarling my bad I didn't actually fire up the IDE for this one. I'll fix and push

also @samstarling thanks so much for doing this. I think the design is pretty good and it's provided immense value for me trying to get started with Prometheus monitoring

@hderms No problem: I haven't done much work on this repository lately, so it's a little unloved – but I'm glad you're finding it useful. Let me know if you ever have ideas for features, or if you need a hand with the PR at all.

@samstarling the only thing I could use help with is understanding why 2 of the builds succeed and 2 fail.

@hderms That looks like a JDK7/Travis problem, rather than anything you've done: let me see if I can figure that out tonight: JDK7 is pretty old now.


Add bounds to limit

Closes https://github.com/holoviz/lumen/issues/483

Codecov Report

Merging #484 (79b126d) into main (db06d57) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #484      +/-   ##
==========================================
- Coverage   69.17%   69.16%   -0.02%
==========================================
  Files          79       79
  Lines        9084     9084
==========================================
- Hits         6284     6283       -1
- Misses       2800     2801       +1

Files Changed
Coverage Δ

lumen/views/base.py
64.69% <100.00%> (ø)

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

It's valid to select zero rows so the correct setting should be bounds=(0, None).


Fixed button click being triggered during ripple animation

As mentioned in issue #146 and pull request #147 multiple click events are being fired during click events.
The committed solution does not fix this issue since if the mClickScheduled flag is set to true, the button click will be triggered immediately instead of ignored.

Sorry should have created pull request to dev branch, not sure if I can delete this one and start a new one


HAWQ-1542. PXF Demo profile should support write use case.

Merged: https://github.com/apache/incubator-hawq/commit/fe33faaba9e4f0422240014c415d1b6a999db5f8


Master branch

Implement for master branch

I think in current situation, master = current dev branch, 3-0-stable for stable releases

@misteral yes, it should be like spree repo, now master have only blank extension

@artem-russkikh Please merge with 3-0-stable

Master now contain last code


aws: Decrease boot and stop timeout

Sometimes Amazon fails to create machines due to an "internal error on
launch". When this happens, 15 minutes is much too long to wait before
retrying.
The new timeout was somewhat arbitrarily picked based on the fact that
in the past 4 months on quilt-tester, the average boot time was 33
seconds, and the max was 90 seconds. The average and max times to stop
machines were 9 seconds and 40 seconds.

@ejj What do you think about the new timeout? I wasn't sure how low to make it. There's some stats in the commit message.


altitude_relative in ALTITUDE in not altitude above home position, but home altitude

In mavlink msg definition:
float altitude_relative; /*< This is the altitude above the home position. It resets on each change of the current home position.*/

but in src/modules/mavlink/mavlink_messages.cpp:
msg.altitude_relative = (_home_time > 0) ? home.alt : 0.0f / 0.0f;

it seems it should be:
msg.altitude_relative = (_home_time > 0) ? (pos.alt - home.alt) : 0.0f / 0.0f;

@LorenzMeier FYI

Fixed in 34baf01.


[Analytics][Newsroom] Incorrect module name and link type in the article-list component

Priority: 3 - Medium
Steps to Reproduce

Navigate to https://main--accenture-newsroom--hlxsites.hlx.live
Check data attributes of any inline link text in the content card component (i.e. "Anser Advisory")

"Expected Result"
data-analytics-link-type = "content module"
data-analytics-module-name = "nws-article-list"
Actual Result:
data-analytics-link-type = "engagement"
data-analytics-module-name = "nws-content-cards"

@ravkiran - The changes affected the other links in the card (i.e. card title and cta link). For this issue, we only need to apply the changes to the inline links. Can this be done?
Expected Results:
card title: data-analytics-link-type = "engagement"; data-analytics-module-name = "nws-content-cards"
cta link (read more): data-analytics-link-type = "engagement"; data-analytics-module-name = "nws-content-cards"
inline link: data-analytics-link-type = "content module"; data-analytics-module-name = "nws-article-list"

fixed in https://github.com/hlxsites/accenture-newsroom/pull/284


update udfs to produce columns and sit under names

We merged https://github.com/whylabs/langkit/pull/117 which I think covers this, recreate if we still need changes on top of main branch.


plugin(scaffolder-backend-module-sonarqube): clean up

Hey, I just made a Pull Request!

Fix all warnings reported in report.api.md
remove unnecessary files

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hi @schultzp2020, please let me know if I can merge this?


add /_ping endpoint

It should return "ok", the current max memory usage and maybe some other lightweight stats (no heavy things like gc.collect). Load balancers can use /_ping to check if a Splash instance is alive; memory usage info is helpful if load balancer itself wants to stop Splash based on its RAM usage.

:+1:

Fixed by #350.


The XamlDirect code sample is incorrect in #3

The following XamlDirect API calls required an instance of XamlDirect. They are not static members.
XamlDirect.CreateInstance
XamlDirect.SetColorProperty
XamlDirect.SetXamlDirectObjectProperty
XamlDirect.SetDoubleProperty
XamlDirect.GetObject
You can create an instance at the beginning of the snippet like this:
XamlDirect xamlDirect = XamlDirect.GetDefault();
and then use the xamlDirect instance in those places in the rest of the sample.

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

ID: a339aa2b-8aa4-a87a-17f8-e48cff9c0011
Version Independent ID: 8fa430de-7a18-f834-7335-99be7c789ad8
Content: Windows.UI.Xaml.Core.Direct Namespace - Windows UWP applications
Content Source: winrt-api-build/xml/ns-Windows.UI.Xaml.Core.Direct.xml
Product: uwp
Technology: winui
GitHub Login: @jwmsft
Microsoft Alias: jimwalk

@alvinashcraft, thanks for letting us know about this. The code has been fixed.


Handle checkbox by name

First of all, thanks for the great package.
I just want to know is that possible to handle checkbox by name?
Same the way xlsx-populate handles with defined name.
workbook.definedName("MyNewCheckboxName").value(true); // or false

Any advice is welcome!

Not currently supported. Sorry.


fix(ts): fetchAll & saveAll

Thans paraself to report this issue (22585).

Codecov Report

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

@@           Coverage Diff           @@
##           master     #593   +/-   ##
=======================================
  Coverage   65.26%   65.26%
=======================================
  Files          37       37
  Lines        3204     3204
=======================================
  Hits         2091     2091
  Misses       1113     1113

Continue to review full report at Codecov.

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

Already fixed by 192d5a44e2abd7c947ab38209b19c2e5b439acb4


Recognize  Arabic Words using SimpleHTR project

Hello!
@githubharald
Thanks a lot for your very helpful project!
I  added the arabic alphabets to charList.txt. But when I run the project it return this error
"""model = Model(open(FilePaths.fnCharList).read(), decoderType, mustRestore=True, dump=args.dump)
File "/usr/lib/python3.7/codecs.py", line 322, in decode
(result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xc7 in position 81: invalid continuation byte """
Can you please give some explanation how can I recognize Arabic Words using this project ?
Thank you so much!

@githubharald
I need to use the finetuning to train my Arabic database.
Please how can I procced ?
Thanks a lot for your response


Datadump of projects which support SPM from CocoaPods

@amorde mentioned this repo in the CP chat room, so :wave: - this data looks to be accurate till about Jan 2019 when I turned CocoaDocs off completely and we stopped getting rich metadata for pods.
You're welcome to use/not use it, given it's public domain
https://gist.github.com/orta/f650a9c924285c6b62242aaaebf38dbd
You can get the Pod's JSON for repos, attribution etc by doing
curl -L https://trunk.cocoapods.org/api/v1/pods/AnimatedField/specs/latest (it's a redirect)

For myself
SELECT pods.name, "spm_support"  FROM "public"."cocoadocs_pod_metrics" INNER JOIN pods ON  pods.id=cocoadocs_pod_metrics.pod_id WHERE ("spm_support" = true)  ORDER BY cocoadocs_pod_metrics.id ASC

Taken this one step further, CSV -> JSON and then done some filtering.
All the entries in this list below are URLs which don't exist in SPI with their swift version (according to CocoaPods) on the left. If we're ignoring all <4.0 then there's 223 in this list + 325 unknowns.

 If we cut off below 4.2 (which is probably another sensible cut-off), there's just ~140 packages that aren't in SPI. Looks like we've got a big overlap already 🎉

Yep, this is fantastic work. Thank you @orta and @Sherlouk!
I'd like to hold off on this moving this into the PackageList just temporarily. I know @leogdion is on the verge of being able to ship the new validator tool for the PackageList and that'll make merging these in a smoother process. I'll check in with him today and see where that's at.

Looks like we've got a big overlap already 🎉

My original seed of data for the SwiftPM Library was seeded from CocoaPods data a year ago. My guess is that these are primarily packages that have added Package.swift files in the last year. We should probably do this again every 6 months or so as this will keep happening.

We should probably do this again every 6 months or so as this will keep happening.

There's also more we could be doing with GitHub APIs to detect primarily Swift repositories with a Package.swift which were created within the last X months.
More than happy to wait on the validator tool 👍

If the new validator tool has a way to detect the current swift version then we should use that to get a more accurate data set!

It doesn't, but we're working on something that does! #466

The swift versions in a podspec should always be accurate, that comes from the version of Xcode used to lint a pod before deploys

One of the advantages of a centralised system where you get to extract and validate metadata as people upload versions!

That dump has since been imported - thanks again @orta for the data and @Sherlouk for adding that packages!