Vipinhas/fix capacity4cluster

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

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[x] PowerShell
[x] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label “WaitForARMFeedback” will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit (including refactoring) and then push new changes, including version updates, in separate commits.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Hey @akning-ms,
This PR is a fix for approved breaking change, link to PR -> https://github.com/Azure/azure-rest-api-specs/pull/14953
Due to an issue in azure-resource-manager-schemas repo with processing enums of type int, the new api-version and property update was never published to customers. you can see docs here -
workspaces- https://docs.microsoft.com/en-us/rest/api/loganalytics/workspaces/create-or-update
clusters - https://docs.microsoft.com/en-us/rest/api/loganalytics/clusters/create-or-update
Thanks,
Viki


Increasing size limit for paddlepaddle_gpu

Hi. We will release a new version for paddlepaddle_gpu recently. Our current size limit is 250M, and the last version is 236M. Because a few new features are enabled in this version, the package size will go to 320MB. Can you please help to increase this to around 350MB.
Project:
https://github.com/PaddlePaddle/Paddle
https://pypi.org/project/paddlepaddle-gpu/

The limit has been set to 350MB on pypi.org.
Note that is getting to be a very large release, making updates painful for users and the PyPI infrastructure. Please be mindful about how often you make releases of this package given the size.


Remix v2 Route File Naming Seems to Mangle Pathnames with @shopify/remix-oxygen

What is the location of your example repository?
pnpm create @shopify/hydrogen@latest with hello world template
What version of Hydrogen are you using?
"@shopify/hydrogen": "^2023.1.3", "@shopify/cli": "3.29.0", "@shopify/cli-hydrogen": "^4.0.5",
What version of Remix are you using?
1.12.0
Steps to Reproduce

pnpm create @shopify/hydrogen@latest with hello world template
create ./app/routes/index.tsx and export a simple route
edit remix.config.js

future: {
    v2_routeConvention: true,
  },

pnpm shopify hydrogen build

Expected Behavior
Builds successfully.
Actual Behavior
Build fails with [plugin browser-route-module] ENOENT: no such file or directory, open '/home/mw10013/src/hello-world/app/outes/index.tsx' . Seems the r in routes gets dropped.

Able to get build and local dev to work with following steps.

pnpm up --latest @shopify/hydrogen @shopify/cli @shopify/cli-hydrogen @shopify/remix-oxygen @remix-run/react
pnpm add @remix-run/server-runtime @shopify/hydrogen-react
pnpm add -D @remix-run/dev
rename ./app/routes/index.tsx to ./app/routes/_index.tsx


multi-account example does not work when using multiple AWS accounts

Description
I am in the process of rolling out a transit gateway network with attached VPCs in multiple AWS accounts.  I followed the example provided in the multi-account directory, but the TGW account side will fail with the following error:
│ Error: creating EC2 Transit Gateway VPC Attachment: InvalidSubnetID.NotFound: The subnet ID <peer account subnet id> does not exist

NOTE: subnet id replaced by me for clarity
The subnet in question definitely does exist, but the provider used by the resources in the TGW account can not see it because it lives in the peer account.

[x] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version [Required]: 2.10.0

Terraform version: Terraform v1.3.9

Provider version(s):

Terraform v1.3.9
on linux_amd64
+ provider registry.terraform.io/hashicorp/aws v5.29.0

Reproduction Code [Required]
Deploy the multi-account example with two different accounts by configuring the providers to use IAM roles in each account that have full administrative access to each account.
Our AWS infrastructure uses AWS Organizations so the administrative IAM role is named OrganizationAccountAccessRole, but any IAM role with sufficient access will do.
For example:
provider "aws" {
  region = local.region
  assume_role {
    role_arn = "arn:aws:iam::<account-1>:role/OrganizationAccountAccessRole"
  }
}

# This provider is required for attachment only installation in another AWS Account
provider "aws" {
  region = local.region
  alias  = "peer"
  assume_role {
    role_arn = "arn:aws:iam::<account-2>:role/OrganizationAccountAccessRole"
  }
}

Steps to reproduce the behavior:
Not using workspaces and cache was cleared

Setup provider as shown above
run terraform init and terraform apply

Expected behavior
The multi account example should work
Actual behavior
Fails with
│ Error: creating EC2 Transit Gateway VPC Attachment: InvalidSubnetID.NotFound: The subnet ID <peer account subnet id> does not exist

NOTE: I replaced the actual subnet id with <peer account subnet id> to make the error message more understandable

Same error

i have the same issue.

I had to add my peer vpc attachment separately and create the routes separately as well for this to work.
main.tf

############ Data Sources ###########

data "aws_vpcs" "prod_vpc" {
  tags = {
    Terraform = "true"
  }
}

data "aws_vpc" "prod_info" {
  id    = data.aws_vpcs.prod_vpc.ids[0]
}

data "aws_vpcs" "dev_vpc" {
  tags = {
    Terraform = "true"
  }
  provider = aws.peer
}

data "aws_vpc" "dev_info" {
  id    = data.aws_vpcs.dev_vpc.ids[0]
  provider = aws.peer
}

data "aws_subnets" "prod_private" {
  filter {
    name   = "vpc-id"
    values = [data.aws_vpc.prod_info.id]
  }
  tags = {
    "kubernetes.io/role/internal-elb" = 1
  }
}

data "aws_subnet" "prod_private_info" {
  for_each = toset(data.aws_subnets.prod_private.ids)
  id = each.value
}

data "aws_route_tables" "prod_route_tables" {
  vpc_id = data.aws_vpc.prod_info.id
}

data "aws_subnets" "dev_private" {
  filter {
    name   = "vpc-id"
    values = [data.aws_vpc.dev_info.id]
  }
  tags = {
    "kubernetes.io/role/internal-elb" = 1
  }
  provider = aws.peer
}

data "aws_subnet" "dev_private_info" {
  for_each = toset(data.aws_subnets.dev_private.ids)
  id = each.value
  provider = aws.peer
}

data "aws_route_tables" "dev_route_tables" {
  vpc_id = data.aws_vpc.dev_info.id
  provider = aws.peer
}

data "aws_ec2_transit_gateway_route_tables" "tgw_rtb" {
  depends_on = [module.tgw]
  filter {
    name   = "transit-gateway-id"
    values = [module.tgw.ec2_transit_gateway_id]
  }
  filter {
    name   = "default-association-route-table"
    values = [true]
  }
}

############ Transit Gateway ###########

module "tgw" {
  source  = "terraform-aws-modules/transit-gateway/aws"
  version = "~> 2.12.1"

  name            = var.tgw_name
  description     = var.tgw_description
  amazon_side_asn = 64532

  enable_dns_support = true

  transit_gateway_cidr_blocks = [cidrsubnet(data.aws_vpc.prod_info.cidr_block, 4, 6)]

  enable_auto_accept_shared_attachments = true

  enable_multicast_support = false

  create_tgw_routes = false

  vpc_attachments = {
    prod_vpc = {
      vpc_id     = data.aws_vpc.prod_info.id
      subnet_ids = local.prod_private_subnets
      dns_support  = true
      ipv6_support = false

      transit_gateway_default_route_table_association = true
      transit_gateway_default_route_table_propagation = true

      tags = var.tags
    },
  }

  ram_name = var.ram_name
  ram_allow_external_principals = true
  ram_principals                = [var.dev_account]

  tags = var.tags
}

resource "aws_ec2_transit_gateway_vpc_attachment" "dev" {
  depends_on = [module.tgw]
  provider = aws.peer
  transit_gateway_id = module.tgw.ec2_transit_gateway_id
  vpc_id             = data.aws_vpc.dev_info.id
  subnet_ids         = local.dev_private_subnets

  dns_support                                     = "enable"
  ipv6_support                                    = "disable"
  appliance_mode_support                          = "disable"
  transit_gateway_default_route_table_association = true
  transit_gateway_default_route_table_propagation = true

  tags = var.tags
}

resource "aws_ec2_transit_gateway_route" "prod" {
  depends_on = [module.tgw]
  for_each = local.tgw_routes
  destination_cidr_block = each.value.destination_cidr_block
  blackhole              = try(each.value.blackhole, false)

  transit_gateway_route_table_id = data.aws_ec2_transit_gateway_route_tables.tgw_rtb.ids[0]
  transit_gateway_attachment_id  = each.value.attach_id
}

resource "aws_route" "dest_prod" {
  for_each = toset(data.aws_route_tables.prod_route_tables.ids)

  route_table_id              = each.value
  destination_cidr_block      = var.dest_cloud_subnet
  destination_ipv6_cidr_block = null
  transit_gateway_id          = module.tgw.ec2_transit_gateway_id
}

resource "aws_route" "dest_dev" {
  depends_on = [aws_ec2_transit_gateway_vpc_attachment.dev]
  for_each = toset(data.aws_route_tables.dev_route_tables.ids)
  provider = aws.peer

  route_table_id              = each.value
  destination_cidr_block      = var.dest_cloud_subnet
  destination_ipv6_cidr_block = null
  transit_gateway_id          = module.tgw.ec2_transit_gateway_id
}

locals.tf
locals {
  prod_subnet_az_to_id   = zipmap(values(data.aws_subnet.prod_private_info).*.availability_zone, values(data.aws_subnet.prod_private_info).*.id)
  prod_sorted_subnet_azs = sort(values(data.aws_subnet.prod_private_info).*.availability_zone)

  prod_private_subnets = flatten([
    for az, id in local.prod_subnet_az_to_id : [
      for sorted_az in local.prod_sorted_subnet_azs :
            id if az == sorted_az
      ]
    ])

  dev_subnet_az_to_id   = zipmap(values(data.aws_subnet.dev_private_info).*.availability_zone, values(data.aws_subnet.dev_private_info).*.id)
  dev_sorted_subnet_azs = sort(values(data.aws_subnet.dev_private_info).*.availability_zone)

  dev_private_subnets = flatten([
    for az, id in local.dev_subnet_az_to_id : [
      for sorted_az in local.dev_sorted_subnet_azs :
            id if az == sorted_az
      ]
    ])

    tgw_routes_map = [
      {
          destination_cidr_block = data.aws_vpc.prod_info.cidr_block
          attach_id = module.tgw.ec2_transit_gateway_vpc_attachment_ids[0]
        },
        {
          destination_cidr_block = data.aws_vpc.dev_info.cidr_block
          attach_id = aws_ec2_transit_gateway_vpc_attachment.dev.id
        }
      ]
  tgw_routes = tomap({for r in local.tgw_routes_map:  r.destination_cidr_block => r})

}

variables.tf
variable env {
  description = "Environment"
  default = null
}

variable aws_account {
  description = "Account to provision in"
  default = null
}

variable dev_account {
  description = "Dev/Peer Account to provision in"
  default = null
}

variable region {
  description = "Region of the cluster"
  default = "us-east-2"
}

variable "dest_cloud_subnet" {
  description = "Dest Subnet in CIDR form"
  type = string
  default = null
}

variable "tags" {
  description = "Resource tags"
  type = any
  default = null
}

variable "tgw_name" {
  description = "TGW Name"
  type = string
  default = null
}

variable "tgw_description" {
  description = "TGW Description"
  type = string
  default = null
}

variable "ram_name" {
  description = "RAM Name"
  type = string
  default = null
}

Hope it helps!

ping

ping


added --update to install.sh

Install.sh --update
This PR adds in --update to the tools/install.sh. This options allows a user to just update and install required packages and install the pipenv for King Phisher with out effecting King Phisher configuration and service files.
Testing

[x] sudo tools/install.sh still works correctly
[x] sudo tools/install.sh --update only install system packages and pipenv environment
[x] `sudo tools/install.sh --update' respects --skip-* options

@wolfthefallen sorry but this needs to be rebased now.

Merging now


Fix sampling issue with formic acid resulting in uneven distribution of points

Completely overhauled the sampling regime.  This is no longer occurring.


Add .bazelversion with 5.4.0

Adds .bazelversion which points  Bazelisk to install bazel 5.4.0.
bazel build ... succeeded for me with 5.4.0  `

[ X ] Appropriate changes to README are included in PR

also -- thank y'all for taking on this project. I read over the discussion over at pmd, and I too, have many of the same pain points you expressed.
I started a side project utility that would lint/warn of Apex Triggers that don't follow a certain pattern (ex. a trigger that doesn't eager load related objects or some other undesired pattern). Shortly after it became apparent that this would be pretty difficult with Jorje API being undocumented/closed.
So, I'm rooting for y'all, and I'll try to help when time/ability allows!

Thank you kindly @zerkz for the contribution. Glad to hear that this was useful.


Creating an ARM k8s cluster using RKE

sub task of https://github.com/rancher/rancher/issues/16461
Detail : https://github.com/jianghang8421/rke/wiki/ARM64-Support

https://github.com/rancher/rke-tools/pull/29
https://github.com/rancher/hyperkube/pull/9
https://github.com/rancher/hyperkube/pull/10
https://github.com/rancher/ingress-nginx/pull/13

This is super exciting, can wait for arm support in rke/rancher.
I see one issue however, it seems that arm64 is being targeted, however Raspbian (probably the most prolific ARM based OS) is only ARMv7 32bit. Is there a plan to offer ARMv6 or ARMv7 builds of RKE/rancher?

@kingsd041 Please test it with RKE v0.2.0-rc8

Please test with RKE v0.2.0-rc9

Version: master 2.2 (3/15/19)
Confirmed fix.
According to https://github.com/rancherlabs/the-ranch/issues/215 has completed the Rancher ARM64 test, all test cases passed, I will close this issue


[Feature][Branch] Select -> "Root" section

Description
🙋🏻♂ please do not merge this until making TODO done

add theme prop: none, box, underline
add rootViewStyle prop: custom ViewStyle stylesheet to root part of selectbox's view
add rootTextStyle prop: custom TextStyle stylesheet to root part of selectbox's text

Related Issues
#60
Tests

check theme and rootTextStyle and placeholder with case "notheme"
check theme and rootTextStyle and placeholder with case "inputTheme"
check theme and rootViewStyle and placeholder with case "themeAndRootView"
check theme & rootViewStyle & rootTextTheme and placeholder with case "disabled"

TODO

[x] make styling box-shadow looking good
[x] typechecking using tsc
[x] making test code related this

Checklist
Before you create this PR confirms that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] Run yarn test or yarn test -u if you need to update snapshot.
[x] Run yarn lint
[x] I am willing to follow-up on review comments in a timely manner.

It seems we need a style for disable.

I add style for disable!

I can be off-topic but looks like @geoseong has merged the branch instead of rebasing which causes bringing all the prev commits to history.

@hyochan I got it. I modified commit histories

@geoseong Thank you for your efforts in #62 and #75!!.
While testing with your code, I found something wrong in "Props" interface that I didn't see in the review. So I ask you to fix it.
export interface Props {
...
placeholder?: boolean;   <-- edit here boolean to string
...
}

@dlgmltjr0925 sorry I missed it. I will modify this type and issue PR shortly.


fix(packageSigning): Remove identifier in the json key name

What does this PR do?
Fix the json key name to fit with what is designed on infra/backend side

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

/merge

/merge


Add dataset: biodiversity_heritage_library

A URL for this dataset
https://www.flickr.com/photos/biodivlibrary/
Dataset description
Dataset of images uploaded from the Biodiversity Heritage Library hosted at the Smithsonian Institution. Useful for GANs, understanding the history of biodiversity, understanding changing aesthetic values over time, etc.
Dataset modality
Image
Dataset licence
Creative Commons Attribution Non Commercial Share Alike 4.0 International
Other licence
No response
How can you access this data
As a download from a repository/website
Confirm the dataset has an open licence

[X] To the best of my knowledge, this dataset is accessible via an open licence

Contact details for data custodian
No response

Sorry for coming to this discussion bit late. My preference would be to include the tags/descriptions generated by cataloguers/Flickr users rather than add labels output from a generic ML model.
For example, this image https://www.flickr.com/photos/biodivlibrary/7268619716/in/photolist-2kM8xSd-9vVsjU-2kM8xXD-XGMoPR-2kF7kHZ-wnaCYy-2jAiBTp-bu7ij8-c5izpA-bvSk4u-atGxWy-atDT9z-aUT4DB-atGxQS-akH6mL-d9s1hq-2m8ksrw-vGLxu7-wEgNjr-eV4dYa-wBsXJj-2j1mFUu-wEgJuH-wnhPxM-wEgKMH-ag15Gu-ag15CN-bK4bht contains the following EXIF data on Flickr
JFIFVersion - 1.02
X-Resolution - 96 dpi
Y-Resolution - 96 dpi
XMPToolkit - Image::ExifTool 12.30
About - uuid:faf5bdd5-ba3d-11da-ad31-d33d75182f1b
Creator - Craig, Hugh, ed.
Description - Johnson's household book of nature,. New York,H.J.Johnson,[1880]. http://biodiversitylibrary.org/page/39741740
Rights - Public Domain
Subject - Mammals
Credit - Image courtesy of BHL
Source - http://biodiversitylibrary.org

and has the following tags:

Although that information is noisy, I think it is more useful and could be used to evaluate noisy supervision and/or contrastive learning methods.
I'm not familiar with memespector but from looking at the GitHub for the tool, it seems like this offers a GUI to run images through one of the commercial vision platforms?
I just put an example through the Google API:

Although this prediction isn't 'wrong', it's also arguably not that useful. I'm also quite wary of using commercial platforms like this because it's often difficult/impossible to get a complete list of possible labels they can assign to an image. Potentially something that could be evaluated using the existing metadata for the images is how closely the predictions of one of the commercial API vision services match the labels assigned on Flickr?
Maybe it makes sense to see what @MikeTrizna already has prepared and take it from there?

@davanstrien That makes sense. It turns out my ISP has a limit on downloading so I can't download all the images. @MikeTrizna will have to upload them.
I think what @MikeTrizna has is the metadata of the files, which include the EXIF data along with some Flickr API stuff. I will go ahead and clean that data up and make it a CSV file. I'll put the results here.
In terms of memespector. There are the commercial APIs it provides along with free versions like VGNet, etc. It can also do a lot of more interesting network stuff.  See http://tallerdeletras.letras.uc.cl/index.php/Disena/article/view/27271/33509.
The open source options provide labels of the items, but for 300k images, it is probably too much. I will use that for a different dataset perhaps in the future. For this, I think the metadata should be sufficient. I will start working on that.

I don't think I ever mentioned that I would be uploading EXIF data, and as @davanstrien  showed above it is superfluous to the tags and information provided directly through Flickr. And Daniel thanks for the clarification on the model-generated labels. I agree that it makes sense to leave those out, since they can always be re-generated separately as new and better models are released.
A quick question on the metadata: Is it possible to upload multiple CSVs to an image dataset? I'm specifically thinking of the tag data, which I would have to come up with some sort of concatenation scheme to squeeze into a single table.

Oh, and @nabsiddiqui, where did you get the "Creative Commons Attribution Non Commercial Share Alike 4.0 International" license information that you show above? I haven't checked through all of the images, but it was my understanding that they are all marked as https://creativecommons.org/publicdomain/mark/1.0/. But if you saw somewhere in the documentation that maybe the dataset as a whole is CC BY-NC-SA 4.0, we can defer to that.

For multiple metadata tags, you could put it in a list/other iterable in
the schema and it should be fine
On Thu, Jul 28, 2022 at 10:11 AM Mike Trizna @.***>
wrote:

I don't think I ever mentioned that I would be uploading EXIF data, and as
@davanstrien https://github.com/davanstrien showed above it is
superfluous to the tags and information provided directly through Flickr.
And Daniel thanks for the clarification on the model-generated labels. I
agree that it makes sense to leave those out, since they can always be
re-generated separately as new and better models are released.
A quick question on the metadata: Is it possible to upload multiple CSVs
to an image dataset? I'm specifically thinking of the tag data, which I
would have to come up with some sort of concatenation scheme to squeeze
into a single table.
—
Reply to this email directly, view it on GitHub
https://github.com/bigscience-workshop/lam/issues/72#issuecomment-1198196854,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMD3OJJCAORVCQUWBCJ7SIDVWKIJTANCNFSM543J6GHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
-Regards,  Shamik Bose

@MikeTrizna I looked at the script that you had. The keywords, tags, etc. that I believe you have are copies of the EXIF data but they have additional metadata such as "ispublic" etc from the FlickrAPI. I am using the same thing and am cleaning the data up a bit so that it is easier to work with. For instance, the page information in the "keywords" Iv made into a separate column called "page". Hope that makes sense. I'll upload both the JSON and CSV following the suggestion of @shamikbose.
And yes, if you send me a CSV or JSON, I can merge it.
In terms of the copyright, I got it from https://about.biodiversitylibrary.org/help/copyright-and-reuse/#reuse. This may be incorrect.

A quick question on the metadata: Is it possible to upload multiple CSVs to an image dataset? I'm specifically thinking of the tag data, which I would have to come up with some sort of concatenation scheme to squeeze into a single table.

In the actual dataset you can have nested data/sequences so it could look something like:

image
width
height
tags

image1.jpg
200
100
["bird of paradise", "botany"]

This also makes it a bit easier to deal with tags of different lengths etc.
This is for example how the annotations are stored in this dataset: https://huggingface.co/datasets/biglam/nls_chapbook_illustrations.
Practically it's possible to upload CSV/JSON files etc that contain this metadata. The dataset script could then do a lookup to get relevant tags etc. @MikeTrizna I'm happy to have a look at how the data is structured at the moment and see how it might fit in a dataset loading script if useful.

@nabsiddiqui, thanks for pointing out that copyright page. That's the exact info I was looking for. I don't think I pulled down the image-level copyright status with my download, but I can go back and get that. If a large majority are public domain, we might be better off filtering to those so that we can label the whole dataset as such.

@davanstrien yes. What you are describing is what I am working on doing. It requires extracting the tags from the JSON and cleaning it up. There are non-tags in the tags section so there are some errors that I have to go through and it is about 300k files.
@MikeTrizna I'm fine with doing that but don't know how we would filter it.

In checking out the example that @davanstrien linked, I noticed that the files are being pulled directly from NLS servers. I'm sure that the data custodians at BHL would be more amenable to directly serving these files from BHL servers (or the Smithsonian FigShare repo) to show a little bit more "ownership". I am in discussions to meet with the BHL data manager next week, and I can run it by them then.
So in the meantime, @nabsiddiqui, I would hold off on the JSON file you are building.

Sounds good. I just have the script for the JSON file running in the background. I'll still upload it here just in case but we can delete it if it's not needed.
I wanted to practice using the Flickr API anyway.


Add import line in sample code

Having the import is useful. It is in the pub page, but having it in the sample code can save some digging when you are getting started.

Pull Request Test Coverage Report for Build 62

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.571%

Totals

Change from base Build 58:
0.0%

Covered Lines:
62

Relevant Lines:
70

💛  - Coveralls

Thanks!


Support Ticket doesn't show up in extension, even after payment is confirmed.

My LNbits transaction history shows a 25 sats payment for a Support Ticket (it has the #lnticket tag), but this ticket doesn't show up in the extension. The webhook was also never dispatched.

Restarting (temporarily?) fixes this. I feel like there might still be issues with the nursery. I will keep an eye on any payments I get.

The issue might be on the sender side. During normal operation, the site refreshes after the payment is confirmed, and the form is empty again.
For that 25 word ticket, the site never refreshed (it was kept open for several minutes after the payment was confirmed), and the form was still filled out.

It is. On this extension the client keeps polling the server tô check for the payment confirmation.
It must be changed to listen for incoming payment notifications on the server side instead.

Fixed


[Bug]: SyntaxError: source code string cannot contain null bytes

To Dos

[X] I tested with the latest released version.

[X] I have checked the issues and think that this is not a duplicate.

[X] I added a very descriptive title to this issue.

Example Code (rst)
git ls-files '*.rst' | xargs rstcheck --config=pyproject.toml

...against psutil code base.
Relevant log output
2024-04-06T19:52:07.4410323Z make lint-rst
2024-04-06T19:52:07.4425054Z make[1]: Entering directory '/home/runner/work/psutil/psutil'
2024-04-06T19:52:07.8247347Z Traceback (most recent call last):
2024-04-06T19:52:07.8257753Z   File "/opt/hostedtoolcache/Python/3.12.2/x64/bin/rstcheck", line 5, in <module>
2024-04-06T19:52:07.8258871Z     from rstcheck._cli import main
2024-04-06T19:52:07.8260727Z   File "/opt/hostedtoolcache/Python/3.12.2/x64/lib/python3.12/site-packages/rstcheck/_cli.py", line 11, in <module>
2024-04-06T19:52:07.8263628Z     from rstcheck_core import _extras, config as config_mod, runner
2024-04-06T19:52:07.8265684Z   File "/opt/hostedtoolcache/Python/3.12.2/x64/lib/python3.12/site-packages/rstcheck_core/runner.py", line 13, in <module>
2024-04-06T19:52:07.8267086Z     from . import _sphinx, checker, config, types
2024-04-06T19:52:07.8268090Z   File "/opt/hostedtoolcache/Python/3.12.2/x64/lib/python3.12/site-packages/rstcheck_core/checker.py", line 7, in <module>
2024-04-06T19:52:07.8268985Z     import doctest
2024-04-06T19:52:07.8269417Z SyntaxError: source code string cannot contain null bytes

Description
Hi. I have rstcheck integrated as part of psutil CI pipeline. It suddenly started failing due to SyntaxError: source code string cannot contain null bytes
https://github.com/giampaolo/psutil/actions/runs/8583440387/job/23522647404
Operating System
Linux
Operating System Details
No response
Python Version
3.12
rstcheck Version
6.2.1 (CLI), 1.2.1 (core)
Additional Context
No response

On a first glace this issue seems to come from cPython and not rstcheck_core/rstcheck. The error happens when doctest is imported, which is part of the standard lib: https://github.com/python/cpython/blob/v3.12.2/Lib/doctest.py. But I cannot imagine that a module is shipped which errors on import.
Have to check if I can reproduce and if 3.12.3 or 3.12.4 fixes this.

Seems like the issue solved itself. I took a look at the latest CI runs in your repo.


Provide sample policies for CiliumNetworkPolicy

Note: provide the samples in http://docs.cilium.io/en/latest/policy/

https://github.com/cilium/cilium/blob/master/pkg/policy/api/rule.go


Make tuning table headers stick to the top

It would be nice to see the headers of the tuning table at all times, but if that requires too many hacks to look good it might not be worth the added complexity.

This turned out to be an easy change. It's in the scale-builder branch already so I'll close this issue.


Breaks iOS auto colored status bar

Before submitting a new issue

[X] I tested using the latest version of the library, as the bug might be already fixed.
[X] I tested using a supported version of react native.
[X] I checked for possible duplicate issues, with possible answers.

Bug summary
Prior to using this library, iOS build would automatically change the status bar color based on the background, just like the navigation bar on both Android and iOS does. However, after setting up this library, the status bar is always colored statically (based on dark mode). It would be very useful to get the auto coloring back when using this library.
Library version
1.1.3
Environment info
System:
  OS: macOS 15.1.1
  CPU: (12) arm64 Apple M2 Pro
  Memory: 219.98 MB / 32.00 GB
  Shell:
    version: 3.7.1
    path: /opt/homebrew/bin/fish
Binaries:
  Node:
    version: 22.11.0
    path: ~/.nvm/versions/node/v22.11.0/bin/node
  Yarn:
    version: 1.22.22
    path: /opt/homebrew/bin/yarn
  npm:
    version: 10.9.0
    path: ~/.nvm/versions/node/v22.11.0/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.16.2
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 24.1
      - iOS 18.1
      - macOS 15.1
      - tvOS 18.1
      - visionOS 2.1
      - watchOS 11.1
  Android SDK: Not Found
IDEs:
  Android Studio: 2024.1 AI-241.15989.150.2411.11948838
  Xcode:
    version: 16.1/16B40
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 23.0.1
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli":
    installed: 15.1.2
    wanted: 15.1.2
  react:
    installed: 18.3.1
    wanted: 18.3.1
  react-native:
    installed: 0.76.3
    wanted: 0.76.3
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

### Steps to reproduce

1. …
2. …

### Reproducible sample code

```js
Should be in any app, but can be seen when changing contrast behind the status bar in iOS

@flipflop97 SystemBars aligns with expo-status-bar, for a consistant behavior between Android and iOS.
It you want a platform specific behavior, you can wrap it in a custom component with:
<SystemBars
  style={Platform.OS === "ios" ? undefined : style}
  // …

An option to have truly automatic bar style (based on contrast) might be added if at some point the feature is also available on Android.


[BUG] Docs page missing nav

Description
This issue is to add the docs nav menu the Open Source Roadmap page to increase consistency in all of the docs pages.
Note: Check to make sure this update is reflected on desktop and mobile views
Screenshot of current docs page:

Screenshot of docs page with side nav:

Screenshots
No response
Do you want to work on this issue?
None
Additional information
No response

I would like to work on it

I want to work on this issue!

I want to work on it, amandamartin-dev can you assign me this issue ?

@Akib-Vijapura as you were the first to request this I have assigned this to you. In future please avoid tagging a specific maintainer - all maintainers are notified and can assign issues.

I will read up on the progress on this issue for learning purposes. I understand that the issue is already assigned to another member of the community.


SHACL Implementation

I've been speaking to the RDF Shapes WG & DCMI off and on about the possibility of an implementation of SHACL.
I think that the most recent draft is much closer to stability than previous releases. It might be time to start looking at serious work on this. I believe the WG needs at least one more implementation this calendar year.
Obviously, this would be an external gem, outside the core; but I wanted to raise it here since there's no space for it yet. Depending on my bandwidth in the summer and fall, I might pick this up toward mid-year. If someone else is motivated to tackle it before then, that would be very welcome.

I've been thinking about this too; it would be useful. I may have time earlier than this summer to get started.

Hey @gkellogg! Any news on that implementation?

Shape Expression (ShEx) seems very powerful too: https://shexspec.github.io/spec/

They're both interesting, but my time's limited right now.  It likely to do something in time for CR.

Okay, so FYI: https://github.com/ruby-rdf/shex. It's a start.


Sneakers 2.5.0 vs. 2.6.0 Error Handling

So, I have hardly had any issues with Sneakers so far and I love it!
Today, I was spinning up a new Rails 5.1 application that is API only and started to try to get it set up with my Bigwig RabbitMQ service. I already have three separate rails applications that are consuming messages from this instance just fine.
I ran bundle install and instead of Sneakers 2.5.0 it installed 2.6.0, fine, I like to keep my Gems up to date.
The issue came when I started up the app. I use foreman to manage my workers and low and behold, not even an attempt to connect to my end point. Just
15:18:17 task_workers.1            | 2017-11-16T23:18:17Z p-1677 t-oww72seug INFO: Worker 1 finished unexpectedly with status 1
15:18:17 task_workers.1            | 2017-11-16T23:18:17Z p-17860 t-oww72seug DEBUG: [worker-assigned_plan.task:1:q508fb][#<Thread:0x007fd8b687f050>][assigned_plan.task][#<Sneakers::Configuration:0x007fd8ba6e6050>] New worker: subscribing.
15:18:17 task_workers.1            | 2017-11-16T23:18:17Z p-17860 t-oww72seug DEBUG: Sent protocol preamble
15:18:17 task_workers.1            | Unexpected error Empty response received from the server.
15:18:17 task_workers.1            |   /Users/andrew/.rvm/gems/ruby-2.4.1@assigned-action-plans/gems/amq-protocol-2.2.0/lib/amq/protocol/frame.rb:60:in `decode_header'
15:18:17 task_workers.1            |   /Users/andrew/.rvm/gems/ruby-2.4.1@assigned-action-plans/gems/bunny-2.7.2/lib/bunny/transport.rb:237:in `read_next_frame'
15:18:17 task_workers.1            |   /Users/andrew/.rvm/gems/ruby-2.4.1@assigned-action-plans/gems/bunny-2.7.2/lib/bunny/session.rb:1127:in `init_connection'
15:18:17 task_workers.1            |   /Users/andrew/.rvm/gems/ruby-2.4.1@assigned-action-plans/gems/bunny-2.7.2/lib/bunny/session.rb:310:in `start'

over and over and over again.
I did not expect it to connect immediately, but it didn't even try and my Rabbit MQ  instance doesn't even register an attempted connection.
I have no global Sneakers configuration, and I use Bunny for connections.
My workers are of the format:
class ExampleWorker
    include Sneakers::Worker
    Sneakers.configure connection: Bunny.new(ENV['AMQP_PATH']),
      :exchange => 'worker_fanout',
      :exchange_type => :fanout

    from_queue 'some.appropriate.queue',
      exchange: 'worker_fanout',
      exchange_type: :fanout,
      durable: true,
      auto_delete: true

    def work(payload)
      do_something_important_here
    end
end

I was able to pull a separate app, upgrade to Sneakers to 2.6.0 and I saw the same effect when trying to run my app.
If I go back down to 2.5.0 I see
15:41:13 milestone_workers.1 | Unexpected error PRECONDITION_FAILED - cannot redeclare exchange 'worker_fanout' in vhost '0LA7Cc3WfJ-l' with different type, durable, internal or autodelete value
15:41:13 milestone_workers.1 |   /Users/andrew/.rvm/gems/ruby-2.4.1@tasks/gems/bunny-2.7.2/lib/bunny/channel.rb:1938:in `raise_if_continuation_resulted_in_a_channel_error!'

This is the expected result and in the RabbitMQ management UI I am able to see my connection, even though Sneakers is unsuccessful.

Hmm. Its a little concerning that its been 2 weeks since I have submitted this issue with no comment. Is sneakers on hold for dev work?

Unexpected error PRECONDITION_FAILED - cannot redeclare exchange 'worker_fanout' in vhost '0LA7Cc3WfJ-l' with different type, durable, internal or autodelete value

means different versions try to declare an exchange with different properties, which is a protocol-level channel exception. I don't immediately see anything in 2.6.0 release notes that would potentially result in this. Taking a traffic capture would provide a lot of very valuable information in investigating this.
I'm not sure I understand the

I have no global Sneakers configuration, and I use Bunny for connections.

part.

That error is completely expected and seen when I bring the sneakers version back down to 2.5.0.

I have no global Sneakers configuration, and I use Bunny for connections.

Just trying to say I have no sneakers.conf, and I configure Sneakers instances utilizing a Bunny instance like:
     Sneakers.configure connection: Bunny.new(ENV['SOME_AMQP_PATH']),
       durable: true,
       exchange: 'some_fanout',
       exchange_type: :fanout,
       timeout_job_after: 30

    from_queue 'some.queue',
      exchange: 'some_fanout',
      exchange_type: :fanout,
      durable: true

I will work on the traffic capture soon.


Switch to popup window

add keywords:
POPUP (switch to popup window)
MAIN (switch back to main window)

MAIN
DEFAULT

new Keyword:
SWITCH window[index]

close


Add to official Casbin adapter list?

Hi @pckhoi - I'm looking for a pgx adapter for Casbin and this one looks great! Is there any reason you haven't added this to the official adapter list? I see you're part of the Casbin organization, so I was wondering if this is feature complete enough to be used? Thanks so much!

Well I told Yang Luo about this repo but he hasn't responded to me so I'm not sure how he felt about it.
I did use this library for a short period of time a few months ago and at that point it had feature parity with the casbin-pg-adapter library so it is probably good enough for you.
I would actually recommend you check out my tulip library though as it can achieve much higher performance than using Casbin. It uses a different interface though and might be a tad harder to use but if you measure performance at all you'll love it. If you care to find out more, I'll add documentation and whatever else that make it easy to use.

Hi @pckhoi I didn't remember that you told me about this adapter.. This adapter is quite new as only created 4 months ago.

Actually it looks great, why not add it? Can you add it to official list via PR to: https://github.com/casbin/casbin-website ? If you don't have time for this, I can do it too :)

@hsluoyz I sent you an email, maybe it landed in spam folder? I can open a PR. ;)

@pckhoi maybe. I have multiple email accounts and some of them have hundreds of mails every day. So maybe it is missing somewhere :( Well, Github should be the most reliable way to find me, Gitter is also OK :)

@hsluoyz got it, sorry for not finding you in other ways. Lesson learned ;)

Thanks! The PR has been done at: https://github.com/casbin/casbin-website/pull/293
@gilcrest plz close this ticket if everything is OK :)

Yup - looks great - thanks again!


Fail to import ActiveLightSensor in SAPIEN 3.0

System:

OS version: Ubuntu 20.04
Python version (if applicable): Python 3.9.18
SAPIEN version (pip freeze | grep sapien): sapien @ https://github.com/haosulab/SAPIEN/releases/download/nightly/sapien-3.0.0.dev20231122-cp39-cp39-manylinux2014_x86_64.whl#sha256=9a7d6fe666341ddae8c5a5d9f4449245e61eed5a643fdd24ee9ec462a449faf0
Environment: Server with xvfb

Describe the bug
Fail to import ActiveLightSensor in SAPIEN 3.0. The error is "ImportError: cannot import name 'Renderer' from 'sapien.core'".
To Reproduce
Steps to reproduce the behavior (use pastebin for code):

from sapien.sensor.activelight import ActiveLightSensor

Expected behavior
To import normally.
Screenshots

Additional context
Add any other context about the problem here.

I believe the ActiveLightSensor is deprecated and may be removed completely in SAPIEN 3 release. We will only support the GPU accelerated sensor. And example using this sensor is https://github.com/haosulab/SAPIEN/blob/dev/manualtest/stereodepth.py

Thanks for your reply! I tried this example. The result image and point cloud look pretty good!
However, I encountered a problem when I tried to load novel objects to the simulator, other than those simple build-in geometries like boxes and spheres.
For example, I want to replace the box in the example code with another box of side length 0.05m, which is stored in an .obj file. But in the stereo image, the box got missed accidentally. Therefore, I want to consult with you what is the correct way to import new objects. My code is shown as follows:
box_mesh = trimesh.load("data/assets/simple_geometry/box_0.05/model.obj")
render_component = sapien.render.RenderCudaMeshComponent(box_mesh.vertices.shape[0], box_mesh.faces.shape[0])
render_component.set_vertex_count(box_mesh.vertices.shape[0])
render_component.set_triangle_count(box_mesh.faces.shape[0])
render_component.set_triangles(box_mesh.faces)
render_component.set_material(sapien.render.RenderMaterial(
   base_color=[0.8, 0.2, 0.2, 1.0],
    # specular=0.8,
    roughness=0.01,
    metallic=1.0,
))
# render_component.set_physx_body_type("kinematic")
entity = sapien.Entity()
entity.add_component(render_component)
entity.set_name("box_0.05")
entity.set_pose(sapien.Pose(p=[0.05, 0.26797, 0.09], q=[1, 0, 0, 0]))
scene.add_entity(entity)

Thank you in advance for your assistance!

The CudaMesh should only be used for deformable objects. If you are building a body that does not need to be deformed, follow https://sapien-sim.github.io/docs/tutorial/basics/create_actors.html

Thanks for your reply! Unfortunately, our project has to contain deformable objects for tactile simulation. Does the current StereoDepthSensor support deformable objects? If so, could you please elaborate on how to correctly import deformable entities so that they can be rendered correctly? Thanks a lot!

abd_component seems to be from an internal extension library which is not released yet. You should contact the library developer directly. Anything that can be rendered with the ray tracer should work with StereoDepthSensor. The deformable body rendering is under development. If you have a specific use case you want me to look at, please provide a  minimal reproducible code sample with SAPIEN code and create an new issue.

Thanks for your reply!
The above error is because the actor_builder method must set scene.
However, it seems that the render_component method needn't to set scene. So I'm wondering if this render_component method in the tutorial can support loading objects from mesh files.
# create render body for visualization
render_component = sapien.render.RenderBodyComponent()
render_component.attach(
    # add a box visual shape with given size and rendering material
    sapien.render.RenderShapeBox(
        half_size, sapien.render.RenderMaterial(base_color=[*color[:3], 1])
    )
)
entity.add_component(render_component)


Pre push hook is broken when git not on PATH

➜  coil-extension git:(nd-update-safari-with-ios-support-2021-07-02) ✗ gpsup
Pre-push Hook: Checking branch name
parent_pid: 27901
parent_pid: ps -ocommand= -p 27901
ps: option requires an argument -- p
usage: ps [-AaCcEefhjlMmrSTvwXx] [-O fmt | -o fmt] [-G gid[,gid...]]
          [-g grp[,grp...]] [-u [uid,uid...]]
          [-p pid[,pid...]] [-t tty[,tty...]] [-U user[,user...]]
       ps [-L]
^C

Requires git to be on $PATH
("Fixed" in my case by a brew install)
Need to fix loop to find git in process tree:
until [[ "$push_command" == git* ]]; do

Must handle /Some/long/path/to/git

Is there a particular reason why we are protecting branches with a pre-push script rather than just using github's branch protection feature?

For others running into this issue, I had two git installations:

/usr/bin/git
/opt/homebrew/bin/git

The first installation was failing on this pre-push script. Upon switching to use the second installation, the pre-push script passed.

see also:
https://github.com/coilhq/web-monetization-projects/pull/2490/commits/40d1330f55e9d324bc84885d20f377e6e6fc9176#r782643437

@sublimator I can't recall why we settled on it being this way. I think it may have been since it worked with your local dev flow better @sublimator Could we run this in CI and then just setup branch protection?


⚠ Bofrost.fr has degraded performance

In 8e034f0, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1403 ms

Resolved: Bofrost.fr performance has improved in f7fc67a after 6 minutes.


Add Comparison with Other SSR Frameworks to Readme

I like it a lot when open source frameworks provide comparisons with other similar ones and make it easier for clients to decide on the best fitting framework for the task at hand.
There are several mature SSR solutions on the market, like next.js or gatsby, or nuxt. It would be bery nice if you could list similarities and differences to these and other frameworks in the Readme.

None of those provide incremental server side rendering as far as I know. The purpose is a bit different too. Those other tools can help with SEO as the entire page is rendered and sent at once to the client. VelociRender is only about performance.
Finally, VelociRender likely requires much less of the developer as it doesn’t require your code to be built in one of those frameworks. Instead it just needs your code to be able to run in JSDom.

Ok this is a good thing to know. So when heavily dependent on SEO, you would not recommend Velocirender?

Google can crawl SPA/Ajax sites now. I’m not sure how or if Velocirender changes that.
It is possible to make Velocirender wait and serve up the entire HTML (but that undos its performance benefits). It might be possible to serve google complete HTML and users the incremental HTML.
I’m not an SEO expert and this hasn’t been designed with that in mind.
I would recommend Velocirender for folks wanting SEO for a SPA app that wasn’t built in one of these other frameworks. It should be much easier to get your app working in velocirender instead of rewriting your app.


add a way to co-relate migmigraion CR to a migplan

Couple of exploratory options to achieve the same goal:

The controller adds a label
Find a way to query via a field selector

Just updating this with @alaypatel07  own research, it's been determined only name/namespaces are usable as field selectors for custom resources.


how to use

I need a example ,thank you

I made a quick append to the README.
I hope it will be helpful


Support Flutter 3.24.0

../../../.pub-cache/hosted/pub.dev/nested_scroll_views-0.0.11/lib/src/flutter/widgets/scrollable.dart:1397:19: Error: The type 'SelectionEventType' is not exhaustively matched by the switch cases since it doesn't match 'SelectionEventType.selectParagraph'.
 - 'SelectionEventType' is from 'package:flutter/src/rendering/selection.dart' ('/opt/homebrew/Caskroom/flutter/3.0.0/flutter/packages/flutter/lib/src/rendering/selection.dart').
Try adding a default case or cases that match 'SelectionEventType.selectParagraph'.
    switch (event.type) {

nested_scroll_views 0.0.12


Fix checkbox and radio buttons not clipping properly

@dclaux,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


rolled back #184 due to fatal errors in testing

@ngosselin-mn please re-submit a pull request when this has been resolved--multiple existing multisite instances crashed after applying these changes.

What fatal errors were you seeing?  That way I can look into them.

It looks like for existing plugins that implement wp-mvc the autoloader can no longer find the right objects.
Note that I replaced the wp-mvc plugin inline (i.e. git pull) without deactivating/reactivating (which I believe is something the WP plugins repository would do).
Does this give you enough to go on to reproduce it?
Thanks again for your support of the project.

@cyberscribe
Ah, that is making sense, since the old plugins, would of used update_option(), vs update_site_option() for its options, meaning that they are in the main option tables vs a site's option table.
What probably needs to happen here, is to verify in both spots for the options, or to fallback if they aren't found in the new spot, ie: have some type of legacy support in place.
I will have to look at the code to see how that can be accomplished, only issue is I am not sure when I will get to it :).

Thanks for getting to it when you can; will be great to have this in place in a backwards-compatible manner whenever it's ready to go.


No changelog?

I don't see the changelog file being maintained which makes it really difficult to when upgrading the library to see what has changed between versions.
Could I ask the maintainers to keep the CHANGELOG up-to-date, preferably when merging PR, if it makes it easy to maintain the file better?
Thanks

Hey, sorry about this. We can definitely keep the changelog up to date going forward.

We'll make sure that new PRs keep the changelog up to date.


lftp: 4.8.2 -> 4.8.3

Motivation for this change
Version bump

2017-10-09: lftp-4.8.3 released. Coredumps on exit were fixed.

@bjornfor @sigma for review
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

$ ./result/bin/lftp --version
LFTP | Version 4.8.3 | Copyright (c) 1996-2017 Alexander V. Lukyanov

@GrahamcOfBorg build lftp

looks good! merging!

@jackpot1136 Do you mean revert? Is there a problem with it?


newIORef type wrong?

🐛 Bug report
Current Behavior
export declare function newIORef<A>(a: A): IO<IORef<A>>

Expected behavior
export declare function newIORef<A>(a: A): IORef<A>

Suggested solution(s)
Maybe I'm confused, but it seems that this type is wrong at the moment.
Your environment

Software
Version(s)

fp-ts
^2.12.2

TypeScript
4.7.3

The current behaviour matches the Haskell signature: https://hackage.haskell.org/package/base-<IP_ADDRESS>/docs/Data-IORef.html#v:newIORef
newIORef :: a -> IO (IORef a)

My understanding is that IORef holds the mutable reference but the side effects are felt when interacting with it, hence creating, reading, or updating one is done within IO.

I see. That makes sense. I wasn't thinking about creation as an interaction, but it, of course, is. Thanks for the clarification!


light-node: RPC server

under construction
Closes #219
Depends on #394

Supervisor changes have been split out into #394.

Depends on #401

Codecov Report

Merging #363 into master will increase coverage by 0.2%.
The diff coverage is 24.3%.

@@           Coverage Diff            @@
##           master    #363     +/-   ##
========================================
+ Coverage    30.4%   30.7%   +0.2%
========================================
  Files         130     131      +1
  Lines        5448    5464     +16
  Branches     1686    1695      +9
========================================
+ Hits         1659    1678     +19
+ Misses       2640    2603     -37
- Partials     1149    1183     +34

Impacted Files
Coverage Δ

light-client/src/components/scheduler.rs
47.6% <ø> (ø)

light-node/src/error.rs
0.0% <0.0%> (ø)

light-node/src/rpc.rs
36.0% <36.0%> (ø)

light-client/src/types.rs
47.8% <0.0%> (-7.2%)
:arrow_down:

light-client/src/supervisor.rs
31.0% <0.0%> (-0.9%)
:arrow_down:

tendermint/src/time.rs
47.2% <0.0%> (ø)

tendermint/src/block/commit.rs
54.5% <0.0%> (ø)

tendermint/src/validator.rs
50.0% <0.0%> (+0.9%)
:arrow_up:

tendermint/src/signature.rs
37.5% <0.0%> (+4.1%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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

Great work @xla! 🥇


[FEATURE REQUEST] Add simple pagination dot components.

The pagination dot styles are very various, So I'll implement a simple one, Just like

Features

Change styles.

active color
inactive color

Control the progress via reanimated values.

Hmmm, Any other features you want?

https://github.com/dohooo/react-native-reanimated-carousel/issues/264#issuecomment-1247689044

Click the dot to move to a new slide would be great. If you can drag on the dots to move (Google like) then you're a legend!

A looped pagination element with the ability to set the amount of visible dots would be a killer feature.
Description: let's say we have a large deck of slides, a 1000 for instance. We don't want to display 1000 dots for obvious reasons. For such cases pagination with 5 dots would be enough, but they should be animated to reflect swiping back and forth, not to reflect actual slide index. So we won't get a jump back when we go from slide index 1000 to slide index 0 and vice versa.
I tried to managed any solution on my own, but no luck. It would be great to have such feature.

A looped pagination element with the ability to set the amount of visible dots would be a killer feature.
Description: let's say we have a large deck of slides, a 1000 for instance. We don't want to display 1000 dots for obvious reasons. For such cases pagination with 5 dots would be enough, but they should be animated to reflect swiping back and forth, not to reflect actual slide index. So we won't get a jump back when we go from slide index 1000 to slide index 0 and vice versa.
I tried to managed any solution on my own, but no luck. It would be great to have such feature.

Ahh, You can use another carousel component with looped to play the pagination dot component, It will animated to next item when sweeping.

Wow, I haven't thought about such possibility :-) But I still miss how can I connect progress of my main carousel to the progress of the pagination. I can do it through the refs, but in this case pagination will react with the delay. Do you have any solution in your mind?

Ahh, You can use another carousel component with looped to play the pagination dot component, It will animated to next item when sweeping.

Wow, I haven't thought about such possibility :-) But I still miss how can I connect progress of my main carousel to the progress of the pagination. I can do it through the refs, but in this case pagination will react with the delay. Do you have any solution in your mind?

Maybe I'll try it on this weekend.💡

Ahh, You can use another carousel component with looped to play the pagination dot component, It will animated to next item when sweeping.

Wow, I haven't thought about such possibility :-) But I still miss how can I connect progress of my main carousel to the progress of the pagination. I can do it through the refs, but in this case pagination will react with the delay. Do you have any solution in your mind?

Maybe I'll try it on this weekend.💡

Hey, Dohooo! Any chance you've took a look at the infinite pagination component? Not to steal your weekends, just curious to see what may be the solution 😇

https://github.com/yasaricli/react-native-dots-pagination
Does decent job on X slides in carousel. Using onProgressChange can detect then slide is already 50%+ past scroll to change the index of dot

@dohooo are you releasing this feature?

Hey !
I can work on a simple pagination component with a few style customization props, something like that:
<Pagination
     dotsLength
     activeDotIndex
     containerStyle
     activeDotStyle
     inactiveDotStyle
 />

I think it would be nice to get the Pagination from this lib since it's common to have it in carousels, would this fit @dohooo ?

Hmmm, Any other features you want?
I think instead of styling, it would be nice to have the ability to create your own Pagination component.
For instance, we are in a scenario where the dots should represents a little preview image of the elements in the carousel. Just with styling, I wouldn't be able to pass an image in the component.

i try use press-swipe add agination , but current index It doesn't seem right

+1

Use this https://github.com/felire/react-native-animated-dots-carousel

Use this https://github.com/felire/react-native-animated-dots-carousel

I'll implement a new one and integrate it into RNRC tomorrow maybe.

@dohooo any updates on this one? Many thanks

I'm looking forward to a simple dot pagination component. Many thanks!

The pagination dot styles are very various, So I'll implement a simple one, Just like
Features

Change styles.

active color
inactive color

Control the progress via reanimated values.

Hmmm, Any other features you want?

active color
inactive color
size
gap between the dots

@dohooo May I please have a response? Is it possible for you to expose the current index, so that one could build a Dot Navigation component oneself?

https://github.com/dohooo/react-native-reanimated-carousel/blob/29425ee93a629e1b2dd2ab54470e885577c0cd8f/src/types.ts#L195C3-L195C19

@dohooo May I please have a response? Is it possible for you to expose the current index, so that one could build a Dot Navigation component oneself?

https://github.com/dohooo/react-native-reanimated-carousel/blob/29425ee93a629e1b2dd2ab54470e885577c0cd8f/src/types.ts#L195C3-L195C19

LOL now I feel stupid. Thank you so much. I am going to create a dot navigation component and share the code here.

https://github.com/yasaricli/react-native-dots-pagination Does decent job on X slides in carousel. Using onProgressChange can detect then slide is already 50%+ past scroll to change the index of dot

Could you show how you connected these two components? Unfortunately, my solution is very laggy, and I suspect it's because onProgressChange is called too often, and I'm not using an animated value. It would be really helpful for me, thanks!
const [activeCarouselIndex, setActiveCarouselIndex] = useState(0)

 <Carousel
  ...
  onProgressChange={(_, progress) => {
          setActiveCarouselIndex(progress)
  }}

<Dots
  ...
  active={activeCarouselIndex}
/>

@tomasmozeris Could you show how you connected these two components? ....

I'm doing the same thing, are you testing in production mode?

So, this issue has been open for a year and a half. Are there pagination dots out of the box, or not?

So, this issue has been open for a year and a half. Are there pagination dots out of the box, or not?
Edit:
Was able to create something simple quite easily:
export const ImageCarousel: React.FC<Props> = memo(({ images }) => {
  const [activeSlide, setActiveSlide] = useState(0)
  const totalSlides = images.length

  return (
    <>
      <Carousel
        onProgressChange={(_, slideProgress) => {
          if (slideProgress % 1 === 0) {
            setActiveSlide(slideProgress)
          }
        }}
        data={images}
        renderItem={({ item: image }) => (
          // Component
        )}
      />
      <Flex>
        {Array.from({ length: totalSlides }).map((_, index) => (
          <Flex
            key={`pagination-${index}`}
            backgroundColor={index === activeSlide ? 'text' : 'textHint'}
            borderRadius='circle'
          />
        ))}
      </Flex>
    </>
  )
})

How to import Flex here in this code?

See this example for more details: https://reanimated-carousel.dev/usage

import Carousel, { ICarouselInstance, Pagination } from 'react-native-reanimated-carousel';
gives this
Module '"react-native-reanimated-carousel"' has no exported member 'Pagination'.
using version 3.5.1

import Carousel, { ICarouselInstance, Pagination } from 'react-native-reanimated-carousel';
gives this Module '"react-native-reanimated-carousel"' has no exported member 'Pagination'.
using version 3.5.1

Same here

same here

same here

same here

import Carousel, { ICarouselInstance, Pagination } from 'react-native-reanimated-carousel';
gives this Module '"react-native-reanimated-carousel"' has no exported member 'Pagination'.
using version 3.5.1

Looks like it hasn't been pushed to npm yet, you can install it from git as a workaround. for example in package.json:
"react-native-reanimated-carousel": "dohooo/react-native-reanimated-carousel"
Or to pin to a specific commit like dede5eb:
"react-native-reanimated-carousel": "dohooo/react-native-reanimated-carousel#dede5eb"

@carter-0 use<EMAIL_ADDRESS>version


Categories on test fixtures not considered in test list

I use the CategoryAttribute on individual test cases. When all test cases of a test fixture belong to a specific category, I place the CategoryAttribute right on the test fixture.
I expect this to be functionally equivalent as if I had marked each test case, when dealing with test cases.
However, in the test list view, when grouping by categories, categories defined by means of the test fixtures are not considered.
The following code:
[Category("a")]
public class A
{
    [Test]
    public void A1() {
    }

    [Test]
    public void A2()
    {
    }
}

public class B
{
    [Test]
    [Category("a")]
    public void B1()
    {
    }

    [Test]
    [Category("a")]
    public void B2()
    {
    }
}

produces this grouping:

I'd expect A1 and A2 to appear under the category a.
I should be able to make a PR for this.

Taking the above example, the Visual Studio test adapter meets my expectations:

When grouping by category:

When filtering by trait:

The way we display in the adapter is an attempt to adjust to how VS wants to see things. In their view of the world a class is a group of tests but is not itself a test. NUnit, otoh, treats methods, classes, namespaces all as tests.
In the gui I have tried to stick with the native nunit view of things. While it would be more or less easy for you to show tests as if they actually had the category, what would we want to happen when you select that category? How would explicit tests be treated?
I'm not completely opposed to this idea, but I think we need to flesh it out a bit before deciding.

@chm-tm I agree with Charlie on this one. For those who use Resharper test runner only they've come to get used to being able to group by a few other options that Visual Studio doesn't provide. Also this is a tool by NUnit for NUnit.. I would expect it to possibly have features that are not available to Visual Studio or Resharper (for instance Warns)

I've looked into how the NUnit 2 GUI handles categories, also mixing in the question about explicit tests.
In order to correctly show the feature, I've added a test B3 without category, and marked the tests A2 and B2 as explicit.
In the GUI, I then selected the category a.

You can see that, basically, the test A1 is selected, since A has the category a. So, this would meet my expectations.
Regarding the explicit tests, A2 was not executed, but B2 was! The GUI won't execute A2 even when you double-click it in order to run it.
From the definition of the Explicit attribute, an explicit test is run when a filter enables it. So it's OK that B2 was executed. For A2, however, the parent category wasn't taken into account in conjunction with the Explicit attribute.
This seems to be a little inconsistent. However, it perfectly meets my requirements when writing tests: I have a convenient way to include all tests of a fixture in a category, but can exclude some of them. But if I really want to run explicit tests with a category, then I need to mark them with the category.
Let's take a look about how the console runner treats the attributes. I tried to mimic the test list GUI by selecting the individual tests explicitly.
> nunit3-console ClassLibrary2.dll --where:cat=a --labels:All --test=A.A1,A.A2,B.B1,B.B2,B.B3
NUnit Console Runner 3.6.1
Copyright (C) 2017 Charlie Poole

Runtime Environment
   OS Version: Microsoft Windows NT 10.0.14393.0
  CLR Version: 4.0.30319.42000

Test Files
    ClassLibrary2.dll

Test Filters
    Test: A.A1
    Test: A.A2
    Test: B.B1
    Test: B.B2
    Test: B.B3
    Where: cat=a

=> A.A1
=> A.A2
=> B.B1
=> B.B2

Tests Not Run

1) Explicit : A.A2

Run Settings
    DisposeRunners: True
    WorkDirectory: <snip>
    ImageRuntimeVersion: 4.0.30319
    ImageTargetFrameworkName: .NETFramework,Version=v4.6.1
    ImageRequiresX86: False
    ImageRequiresDefaultAppDomainAssemblyResolver: False
    NumberOfTestWorkers: 8

Test Run Summary
  Overall result: Passed
  Test Count: 4, Passed: 3, Failed: 0, Warnings: 0, Inconclusive: 0, Skipped: 1
    Skipped Tests - Ignored: 0, Explicit: 1, Other: 0
  Start time: 2017-05-04 06:53:25Z
    End time: 2017-05-04 06:53:26Z
    Duration: 0.680 seconds

Results (nunit3) saved as TestResult.xml

It seems to me that the behavior matches the NUnit2 GUI, in that the parent test's category is taken into account for A1 and A2.
I think the NUnit3 GUI should just implement the same semantics as the console runner.

@chm-tm
First, let's remember we are talking about the gui here. So any changes in how the NUnit framework itself works are off the table. Of course we have to take how it works into account and display things in an appropriate way in the gui.
The console runner is, of course, yet another (separate) project. But I agree that it should behave in the same way. I'll give you a small quibble about the word "implement", however. The console runner doesn't implement any behavior at all with respect to category selection. It passes the filter to the framework, which does all the implementing.
Actually, that's more than a quibble. The console runner does not know what filters are implemented by the framework or what their semantic may be. It doesn't even know what framework you are using to run tests. That's because it goes through the engine, which simply locates a driver for the particular framework. We currently implement two frameworks (NUnit v2 and NUnit 3) with different behavior. We may add others in the future. The GUI, like the console runner, has to be able to deal with the fact that it does not know what the framework will do in certain cases.
So... here is what the gui assumes in displaying categories:

That the particular framework has categories. That's not an issue because if any framework did not support categories, the representation of tests passed back by the driver would not include any, so the gui would not display any.

That selecting the name of the category in the category list will cause the listed categories to be executed. That's the tricky part, because of all the indirection involved through the engine and framework.

The NUnit 2 GUI has a way of selecting categories and displaying them in the tree that is very handy. As you point out it grays the tests that are not included and only leaves those that would be executed when running that category. The problem here is that the NUnit 2 GUI is only intended for use with the NUnit 2 framework and can incorporate duplicate logic based on the semantics of that framework. A GUI that is intended to work with multiple frameworks, some of which have not yet been invented, is more limited in what it can do. To implement a display like that in the NUnit 3 GUI, we would need to ask, through the API, "What tests would be run using this category selection" I think that's a possibility for our future development of the tree display.
I added the test list display to the new GUI, because so many people liked it in VS. Simple lists of tests seem to be a "thing" these days, as opposed to our traditional tree display. However, in adding it, I inherited all the implied issues that the VA adapter already has. I'm not sure what the answer is. It could be simply showing so-called "inherited" categories as we do for VS. It could be something else.
In summary... I'm not opposing your suggestion, just pointing out how many variables are involved in what may initially look like a simple solution. If you want to do a PR, I'll be glad to take a look. Downloading it and playing with it might give us a better idea of what we want to do eventually.

Thank you very much for your insights. Forking an NUnit repository is just a great experience - just fork and run.
I've created a fork/branch with a small change enabling categories to be inherited from parent and ancestor tests. This makes the GUI work like I'd expect.
I'm not making a PR at this time, since I didn't made any tests. In addition, I'd be glad to get some feedback whether to browse parent nodes of the test node is feasible in this context.

I created the PR since it's easier for you to play with it.


Fix codes for AOT.

Coverage remained the same at 55.047% when pulling e1b2ad946f2359553bb259f459f0594326921a15 on wknet123:master-new into 3543a4742f739aad2639e83c24b5bae981666bda on vmware:master.

Coverage remained the same at 55.047% when pulling a147b743078780b7ac1cf4725946634e2a9a7a5d on wknet123:master-new into 3543a4742f739aad2639e83c24b5bae981666bda on vmware:master.

Coverage remained the same at 55.228% when pulling 54154b59e916521a0574ba853ff5580a895245c7 on wknet123:master-new into ad02cab82a1b96151c276c82f5f9a77a591635d2 on vmware:master.

Coverage remained the same at 55.228% when pulling 54154b59e916521a0574ba853ff5580a895245c7 on wknet123:master-new into ad02cab82a1b96151c276c82f5f9a77a591635d2 on vmware:master.


Using EVAL as auth.redis.auth_cmd

When attempting to use the following command as the auth.redis.auth_cmd parameter it is not sent to redis correctly.
EVAL "return redis.call('HGET','App:Auth:Users:test','pass')" 0
The command is actually sent to redis as:
"EVAL" "\"return" "redis.call('HGET','App:Auth:Users:test','pass')\"" "0"
I'm guessing line 43 here is the issue: https://github.com/emqtt/emq-auth-redis/blob/master/src/emq_auth_redis_cli.erl
Could an option be added where the command is sent verbatim except for the %u and %c substitutions?

Hi, @Think7  Sorry for the late reply,Do you resolve this problem?


ShaftElement 6 gdl code

I am opening this issue to organize the implementation for a shaft element with 6 dof.

[ ] Implement mass matrix without rotary inertia;
[ ] Implement mass matrix with rotary inertia;
[ ] Implement stiffness matrix.

Hi there!
I have marked this issue as stale because it has not had activity for 45 days.
Consider the following options:

If the issue refers to a large task, break it in smaller issues that can be solved in
less than 45 days;
Label the issue as wontfix or wontfix for now and close it.

#502 implemented this.


Add common dimensions and tag dimensions to AWS Compliance Mod

Checklist

[ ] Issue(s) linked

@khushboo9024 could you please update the docs/index.md file to include information on common and tag dimensions? Please use https://github.com/turbot/steampipe-mod-aws-thrifty/blob/main/docs/index.md#common-and-tag-dimensions for reference.


Can this display hi res images without pixelizing ?

Can a hi res image be displayed without pixelation ?

I think not

Let's hope it's somehow possible. A presentation containing only walls of text and no useful images is dull.

I'm on Linux Mint 20.1, and when I $echo $TERM, I get xterm-256color - all of my images are pixelated as well :-(


catfs always runs in foreground

./catfs /mnt/gsdata /tmp/catfsdata /mnt/data

Hangs in foreground, but works (mounted folder exists)
Version: downloaded as 0.8.0, reported 0.7.0  (see #12 )
Caching folder from mounted goofys S3 folder.
How to run it in background?

I get this as well.
Debian 10, clean Rust environment, catfs-0.8.0.crate
Here's the output of the debug log:
2020-05-04 04:24:03 DEBUG - catfs "/home/user/tmp/source" "/home/user/tmp/cache"
2020-05-04 04:24:03 DEBUG - options are ["-o", "atomic_o_trunc", "-o", "default_permissions"]
2020-05-04 04:24:03 INFO  - Mounting cached
2020-05-04 04:24:03 DEBUG - INIT(1)   kernel: ABI 7.27, flags 0x3ffffb, max readahead 131072
2020-05-04 04:24:03 DEBUG - INIT(1) response: ABI 7.8, flags 0x1, max readahead 131072, max write 16777216

Same issue for me building 0.8.0 on Amazon Linux 2, trying to cache an s3 bucket mounted via goofys.
It works (if I start catfs in its own terminal and use another terminal to browse the cached path) but catfs seems to always run in the foreground...


Documentation link to "Plot" is a dead link

This is a template helping you to create an issue which can be processes as quickly as possible. Feel free to add additional information or remove not relevant points if you do not need them.
If you have a question rather than reporting a bug please go to http://answers.opencv.org where you get much faster responses.
Please state the information for your system

OpenCV version: 2.4 / 3.x
Host OS: Linux (Ubuntu 14.04)  / Mac OS X 10.11.3 / Windows 10
(if needed, only cross-platform builds) Target OS: host / Android 6.0 / ARM board / Raspberry Pi 2
(if needed) Compiler & CMake: GCC 5.3 & CMake 3.5

In which part of the OpenCV library you got the issue?
Examples:

objdetect, highgui, imgproc, cuda, tests
face recognition, resizing an image, reading an jpg image

Expected behaviour
Actual behaviour
Additional description
Code example to reproduce the issue / Steps to reproduce the issue
Please try to give a full example which will compile as is.
#include "opencv2/core.hpp"
#include <iostream>
using namespace std;
using namespace cv;

int main()
{
    double d[] = { 546,2435,7,4534,23423,3 };
    cout << "d = 0x" << reinterpret_cast<void*>(d) << endl;

    return 0;
}

OpenCv 3.1.0
Platform: N/A
docs.opencv.org
Main Page : OpenCV Modules
Main modules: Extra modules: plot:
http://docs.opencv.org/3.1.0/db/dfe/group__plot.html#gsc.tab=0
This is an empty page.

plot module is not documented.

This issue should go to opencv_contrib

Moved to opencv/opencv_contrib#728


getTileData accessor cannot access state variables from outside the TileLayer

I'm using the Tile Layer to fetch geojson tiles and then color them according to another api call.
My getTileData function fetches the tiles, and then loops through the polygons of each tile to get a list of polygon-id's that it will use to fetch data about those shapes (i.e. what color to draw them), which it then puts into a state variable (dictionary of id's -> colors).
The issue is: I want to minimize the number of requests to this api that gives me the colors of the polygons, I know that tiles from different zoom levels will often contain the same shapes, so i don't want to re-fetch all the id's that i have gotten from another zoom level.
Iif I add all the ids of the shown shapes on one zoom level and put them into a dictionary that i access with getFillColor, and then i zoom out one level, my getTileData function does not see that the updated dictionary variable of id's that i created from the first level. How do i make getTileData see the most recent  dictionary ?
I don't want to use update triggers like:
updateTriggers={{     getTileData: dictionary }}
because i update dictionary in getTileData which creates this weird loop.
Is there any way to make getTileData aware of the updated dictionary variable?

Have you tried using updateTriggers={{getFillColor}} instead? Technically speaking your tile data has not changed, so re-fetching them is excessive. You only need the sublayers to reevaluate the fill colors.

Sorry I'm not sure i explained this well enough.
The coloring is working fine, I have updateTriggers={{getFillColor}} already. What i want to avoid is simply refetching the data as you said. To do that i have to see if the id that i would fetch is already in a dictionary from a previously loaded tile. The problem is that the getTileData accessor has no awareness of the updated dictionary.
ex)
const [idDictionary, setIdDictionary] = useState({})
....
<TileLayer
....
getTileData={(x,y,z) =>
for each geoJSON in tile {
if (id not in dictionary) {
dictionary[id] = fetch(colorForId(id))
}
}
}
`
but within the getTileData accessor, dictionary is always empty, even though i can see it's being populated outside of the layer
Thank you for the help!

I see. The issue is that setDictionary does not mutate the idDictionary object, but creates a new object. On the other hand, TileLayer ignores any subsequent updates to getTileData, and the first getTileData value only references the initial state (empty object).
Conceptually, you should not use useState for caching because updating idDictionary is not supposed to trigger a new render. To make the cache persist, you simply do:
const idDictionary = {};

<TileLayer
   ....
   getTileData={(x,y,z) =>
      for (...) {
         if (!(id in idDictionary)) {
            idDictionary[id] = true;
            fetch(id);
         }
      }
   }
/>

Ahhh, I understand completely. Thank you for the quick response and for the explanation. I truly appreciate it!


CAMEL-15170: initial commit (add message queue support to JT400 component)

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] If you're unsure, you can format the pull request title like [CAMEL-XXX] Fixes bug in camel-file component, where you replace CAMEL-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install -Psourcecheck in your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically.

Comments
This is a work in progress, but opening a Draft PR to welcome comments. This pass has very basic support, sufficient for my current needs. I propose we land this functionality (when no longer WIP) and queue up individual features for future goals (for which I will gladly itemize as issues in Jira)

Good start, you can continue work on this and also it would be good to add a bit into the docs, eg see in src/main/docs folder.


Error: Protobuf cannot be read: Error: no such Type or Enum 'google.type.Money' in Type

Google well known type not found error:
https://github.com/googleapis/api-common-protos/blob/master/google/type/money.proto
Error: Protobuf cannot be read: Error: no such Type or Enum 'google.type.Money' in Type

Hmm I guess I'll have to find a way to support all of those messages. Thanks for reporting! I'll work on this next week.

I found a solution
Just add them from your protobuf path


Spring Data module with tests for primitive types for custom query

Spring Data module with tests for primitive types for custom query
Additional coverage for https://github.com/quarkusio/quarkus/issues/9192
Covering also long case and boolean for both primitive and object case

:+1:


Dynamo.batch_write_item generates invalid DeleteItem syntax

Using Dynamo.batch_write_item to attempt to delete items generates a request item in the format %{"DeleteRequest" => %{"Key" => %{"S" => "api_key1"}}}. This errors with the following message:
{:error, {:http_error, 400, "{\"__type\":\"com.amazon.coral.service#SerializationException\",\"Message\":\"Expected null\"}"}}

Per the API docs  this should be formatted as
%{"DeleteRequest" => %{"Key" => %{ "name_of_key" => %{"S" => "api_key1"}}}}. We also need to be able to supply all key values in the event of a composite primary key.

Fixed in master.


Add mermaid support

why do we have three different contexts inside one PR?

custom resources injection
new plugin integration
esbuild POC


AB#4272 bulk-action notify-included

@elwinschmitz as for the 2nd (remove) commit: I chose to be really minimal, and leave the notification component in, because it might be re-used, but at the very least, code within it will be re-used (adding timestamp stuff to e.g. export-buttons is quite high on the backlog).


Add IndexMut traits.

Also adds a direct usize index (since it's not variable-length encoded, there's no reason not to do this here).
I'd add that it's a bit confusing to me why we are declaring the Index traits for AsciiString at all; can't we just let Deref delegate everything?  I didn't change it because I suppose at this point it's backwards-incompatible to fix, but that seems like a cleaner design.

Thanks!
Indeed Index and IndexMut could be delegated via deref. I just overlooked that just like the forgotten direct usize index. As long as it doesn't hurt I would keep the explicit implementations. If you like to clean up more just open another PR. I like the new clean code for Index and IndexMut!

FWIW, String and Vec<T> implement Index{,Mut}. If they didn’t the impls for str and [T] would be usable through Deref{,Mut}, but they wouldn’t be usable with generic things that have a bound on Index{,Mut}.


snowflake_account_parameter missing option

Provider Version
0.56.4
The provider version you are using.
Terraform Version
1.3.9
The version of Terraform you were using when the bug was encountered.
Describe the bug
Missing account parameter option: REQUIRE_STORAGE_INTEGRATION_FOR_STAGE_OPERATION as per official Snowflake documentation https://docs.snowflake.com/en/sql-reference/parameters#require-storage-integration-for-stage-operation
Error: expected key to be one of [PREVENT_UNLOAD_TO_INLINE_URL CLIENT_ENCRYPTION_KEY_SIZE INITIAL_REPLICATION_SIZE_LIMIT_IN_TB NETWORK_POLICY SSO_LOGIN_PAGE MIN_DATA_RETENTION_TIME_IN_DAYS ENFORCE_SESSION_POLICY REQUIRE_STORAGE_INTEGRATION_FOR_STAGE_CREATION EXTERNAL_OAUTH_ADD_PRIVILEGED_ROLES_TO_BLOCKED_LIST ALLOW_ID_TOKEN PERIODIC_DATA_REKEYING ALLOW_CLIENT_MFA_CACHING ENABLE_INTERNAL_STAGES_PRIVATELINK], got REQUIRE_STORAGE_INTEGRATION_FOR_STAGE_OPERATION
A clear and concise description of what the bug is.
Expected behavior
Terraform should apply the parameter with no issues
A clear and concise description of what you expected to happen.
Code samples and commands
resource "snowflake_account_parameter" "p" {
key   = "REQUIRE_STORAGE_INTEGRATION_FOR_STAGE_OPERATION "
value = "true"
}
Please add code examples and commands that were run to cause the problem.
resource "snowflake_account_parameter" "p" {
key   = "REQUIRE_STORAGE_INTEGRATION_FOR_STAGE_OPERATION "
value = "true"
}
Additional context
It is missing from parameters.go
https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/665c94480be82831ec33650175d905c048174f7c/pkg/snowflake/parameters.go#L45
Add any other context about the problem here.

Solution to resolve this would be to add it as the following:

		"REQUIRE_STORAGE_INTEGRATION_FOR_STAGE_OPERATION": {
			TypeSet:      []ParameterType{ParameterTypeAccount},
			DefaultValue: false,
			Validate:     validateBoolFunc,
		}

I can make a PR if needed.

Yeah if you could make a PR I would be happy to get this approved.

Closing because #1552 was merged.


Allow negative line spacing for the script editor in Editor Settings

This can be used to make text even more compact with certain fonts.
In the meantime, you can edit editor_settings-4.tres while the editor is closed and manually input a negative value for text_editor/appearance/whitespace/line_spacing (add that line if it's not present in your configuration file).

This closes https://github.com/godotengine/godot/issues/76385.

I don't think this is a good workaround. Using negative line_spacing on CodeEdit may cause a crash in some cases (condition font_size + line_spacing = -3 is met).


Mentioned git --config settings in the README

Also replaced grunt with some new npm run commands.
Cloning without endline config settings on Windows results in failed lint checks during tests because of different endline behavior. These are the same settings as https://github.com/microsoft/tslint-microsoft-contrib/#development.

fixes #1631


[website] Update Flink runner page to refer to classic instead of legacy runner

Oups I accidentally merged without fixing the name of the commit, sorry.

Should we give all requested reviewers a chance to review? Why the rush to merge this so quickly?

In any case, the updated terminology is much better.


[ParamManager] Support dynamic-shape parameters

Prior to this PR, ParamManager always assumes (and asserts) all the parameters have static shape. As MLC LLM goes to the support for LoRA, dynamic vocabulary size and more cases, the assumption of static-shape parameters no longer holds.
This PR updates ParamManager so that it supports dynamic-shape parameters.

cc @CharlieFRuan @cyx-6

Need validation before merge.

Verified that with tvm's PR #15713, this PR will allow dynamic implementation of vocab_size when compiling llama variants.
Tested: {build-model-only=True, build-model-only=False} X {sep-embed=True, sep-embed=False}
Tested: using Llama2-7B's model library (with dynamic vocab_size) to run WizardMath-7B's compiled params (with 32001 vocab_size)

Thanks @cyx-6 and @CharlieFRuan for confirming! We are gonna merge it once apache/tvm#15713 is ready.


Some systems failed to build for Quicklisp dist

Building with SBCL 2.2.1 / ASDF 3.3.5 for quicklisp dist creation.
Trying to build commit id 037fa2d0eab85ac3c91229098cc2674a7bf3780c
journal fails to build with the following error:
; caught ERROR:
;   READ error during COMPILE-FILE: Package SWANK does not exist. Line: 2682, Column: 33, File-Position: 102713 Stream: #<SB-INT:FORM-TRACKING-STREAM for "file /home/quicklisp/quicklisp-controller/dist/build-cache/journal/7a2fb61afddc71abfcd595e28e2058a01be39c16/journal-20220215-git/src/journal.lisp" {10070C4AE3}>
...
Unhandled UIOP/LISP-BUILD:COMPILE-FILE-ERROR in thread #<SB-THREAD:THREAD "main thread" RUNNING {1001B98003}>: COMPILE-FILE-ERROR while compiling #<CL-SOURCE-FILE "journal" "src" "journal">

journal/test fails to build because of a failure in  journal.
Full log here

That's what I get for only testing under Slime. Fixed in b0081953db93040a424bac2a8bf6e22e9b59adb7.


Product of elliptic curve with curve of genus \geq 2

Add this as an example of a Kodaira dimension 1 surface

This is now done actually.


Server crashes on world unload

Information

Server version:

Full output of /mv version -p:

Server log:

Details
I was <able / not able> to reproduce my issue on a freshly setup and up-to-date server with the latest version of Multiverse plugins with no other plugins and with no kinds of other server or client mods.
Description

Steps to reproduce

Expected behavior

Screenshots

I am using the newest version with 1.17 spigotmc
When I join a world it prob. works and loads the world but if I unload the world the server crashes (shut down)

Follow the issue template with logs of errors and other requested information.

WHY DO YOU CLOSE THIS
WICH LOG

Information
Server version:
Full output of /mv version -p:
Server log:

Provide proper information with valid prove of the issue. Then yes, it shall be reopen.

Ok server log...
I currently can't reproduce it but I assume it is:

Server issues crash because ticks too high or
Server issues crash because no free ram (But that's special because unloading a world shouldn't use much RAM)

But the bug persists. It depends on how many players are online and how much RAM is free on the server (there is also a nextcloud server running)
I copy the Server log if it happens again

Please open that again
Here your log:
http://cloud.bubicloud.ch/nextcloud/index.php/s/8e3qtwn69ZtWbTZ (no ssl)
All that can happen because of RAM wich is confusing but the Server tries to issue a command and it can't be executed because the zram is uncompressing the cached files wich struggles the server and ticks slow down so - well - the server issues a crash

Now futher Information:
Unloading a world wich isn't loaded will cause a world load and directly a world unload
That's the problem!
 WORLD IF I PRESS UNLOAD

I tested creating and unloading of worlds on a papermc 1.17 server. No issue faced, you can see the logs at https://paste.lucko.me/uwoCxYOFJz

Can't reproduce? I'm not the only one with this bug
reproduce:
Server has 1700M RAM (-xms)
Use for example a Virtual Machine to give the Ubuntu System (20.10, ubuntu.com) 4G RAM
Open Firefox Browser to simulate consumption of nextcloud
Set up and install zram* (sudo apt install zram* -y)
Start zram service: sudo service ((press TAB)) start
start Minecraft Server
generate a world (1.17) named world2 (normal world, pvp on difficulty NORMAL, AUTOLOAD OFF)
stop the server
start the server (make sure you hav min. 80% of ram used when started)
log in to the server
make 3 command blocks: mvunload world2, mvload world2, mvtp @p world2
place a button for every command block
Load world2
tp into it
unload world2
load it again
unload twice
Now at least the server should crash!

Oh maybe you want to skip some steps but please use the OS ubuntu

please reopen it BECAUSE NO FIX
Why do you close things with no fix

Haven't seen the crash log and stacktrace btw.

??

http://cloud.bubicloud.ch/nextcloud/index.php/s/8e3qtwn69ZtWbTZ (no ssl)

You keep saying there is an error and crash by mv. But the logs you send has no errors by mv.

mv causes the errors - server never crashed before and was more smooth
It is using a function by spigot that crashes the server (maybe syntax wrong or - unloading a world that's not prob. loaded

I see you said command blocks, duplicate of #2560

ok I'll take a look


$bulkdata-status needs to indicate when no data was exported

Is your feature request related to a problem? Please describe.
Ran an $export using _type and _typeFilter that I did not match any data so nothing was exported.
Ran $bulkdata-status to for the associated jobId.
{     "transactionTime": "2021-08-10T21:14:53.311Z",     "request": "https://localhost:9443/fhir-server/api/v4/Patient/$export?_type=Encounter&_typeFilter=Encounter?status=active",     "requiresAccessToken": false }
The status above left me guessing.
Describe the solution you'd like
I would like to see some indication returned by $bulkdata-status that the job finished successfully and no data was exported.
Describe alternatives you've considered
Maybe a return code would be better but I don't see this as a failure.

Options:
A. empty array
B. a separate field with an OperationOutcome to explain that everything worked but there was nothing to export
The response body is defined at https://hl7.org/fhir/uv/bulkdata/export/index.html#response---complete-status
Proposal:  include an empty array and an extension that has an OperationOutcome with the detaiils

Confirmed empty array and extension now returned:
{     "transactionTime": "2021-08-24T15:13:02.866Z",     "request": "https://localhost:9443/fhir-server/api/v4/Patient/$export?_type=Encounter&_typeFilter=Encounter?status=active",     "requiresAccessToken": false,     "output": [],     "extension": {         "outcome": {             "resourceType": "OperationOutcome",             "issue": [                 {                     "severity": "information",                     "code": "informational",                     "details": {                         "text": "No Data Exported"                     }                 }             ]         }     } }


Website theme revisions

Revisions to https://github.com/carbon-design-system/carbon-website/issues/303 based on feedback.
This issue is meant to solve two issues:

The legacy components (v9) need to be shown on the light blue #F5F7FA background. This will clash with any light grays used in the language and digital website backgrounds.

Move to a lighter themed UI shell to match language, digital, and current Carbon sites.

The plan is to have a slightly different theme for experimental and legacy (v9) component pages.

Latest

Are these table styles specifc to component pages or site-wide?

@alisonjoseph site wide. Need to confirm with @jeanservaas on the table style's she's using

h2 - margin bottom 24px 1.5rem, margin top 80px 5rem
h3 - margin bottom 48 3rem, margin top 16px 1rem
h4 - margin bottom 48 3rem, margin top 16px 1rem
paragraph - margin bottom 24px 1.5rem
list - margin bottom 24px 1.5rem

Updated latest local build w/ specs above

Sprint 25: follow up on light side nav


Sample weights

Hello,
is there a designed way to train models with weighted samples? especially to set specific weights to each output in multi-output tasks.
Thanks

I think you can multiply weight to the loss.
what is your task, regression or classification? what kind of loss function currently you are using?

both regression and classification tasks, with use mean_squared_error and sigmoid_cross_entropy loss functions.
Do i need to modify my loss functions for this trick? or i can provide a modified metric in "metrics_fun" argument?

I suppose metrics do not participate in backward pass, so i have to modify loss functions

Yes. You need to write to to set specific weights to each output in multi-output tasks.
I'm planning to implement task_weight argument in mean_squared_error and mean_absolute_error. Maybe it is useful for you.
https://github.com/pfnet-research/chainer-chemistry/blob/master/chainer_chemistry/functions/mean_squared_error.py#L14-L15

Weighted loss function is an easy part. Seems like sending only two arguments to loss function is hard-coded inside models/prediction/regressor.py and classifier.py, and also in GraphConvPredictor definition (which is not part of chainer_chemistry though).
    if self.weighted:
        loss_args = self.y, t, args[-1]
    else:
        loss_args = self.y, t

    self.loss = self.lossfun(*loss_args)

This works fine, but i do not fully understand lines 91-109 (regressor.py), so i'm curious if it is right way instead of calling self.loss = self.lossfun(self.y, t)


Redesigned resume page

Fixes #258
Re PR - #260

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊