Evaluating surface integrals of a grid function

Hello all,
I've been using MFEM for the last couple of weeks, and I've been very impressed by it. Thanks for making it available!
I'm working on an electromagnetic problem, but my question is much more general. Basically, I need to evaluate surface integrals of things like intensity of the electric field, i.e. |E|^2 = E dot E, where E is a 3D vector field that is solved for. This is a post-processing step to get certain quantities of interest. My understanding is that one simple way to do this is define a bilinear form such as
  Array<int> bdr_marker(1);
  bdr_marker[0] = 1;

  BilinearForm *bi = new BilinearForm(m_ptr->fes);
  bi->AddBoundaryIntegrator(new VectorFEMassIntegrator(), bdr_marker);
  bi->Assemble();

and then use that via bi->InnerProduct(x, x) on an appropriately-defined grid function x. I tried this, but I couldn't get it working for boundary integrals.
As such, I have created a minimal example that replicates my issue. In this example, I'm simply trying to use a BilinearForm to compute the volume and surface area of a meshed unit sphere (the mesh file is attached). I'm using scalar functions here, so I simply set the grid function to be 1 and integrate over the entire volume or what I think should be the spherical boundary. The volume comes out fine, the surface area comes out very wrong. Am I missing something here? How can I use a bilinear form to compute a surface integral with a grid function?
int main(int argc, char *argv[]) {
  Mesh *mesh = new Mesh("sphere.vtk");

  // Set the space for the high-order mesh nodes.
  H1_FECollection fec(2, mesh->Dimension());
  FiniteElementSpace nodal_fes(mesh, &fec, mesh->SpaceDimension());
  mesh->SetNodalFESpace(&nodal_fes);

  FiniteElementSpace *fespace = new FiniteElementSpace(mesh, &fec);

  GridFunction x(fespace);
  x = 1.0;

  BilinearForm *vol = new BilinearForm(fespace);
  vol->AddDomainIntegrator(new MassIntegrator());
  vol->Assemble();

  Array<int> bdr_markers(1);
  bdr_markers[0] = 1;

  BilinearForm *area = new BilinearForm(fespace);
  area->AddBoundaryIntegrator(new MassIntegrator(), bdr_markers);
  area->Assemble();

  cout << "Volume of the meshed unit sphere is " << vol->InnerProduct(x, x) << endl;
  cout << "Should be " << (4 * M_PI / 3) << endl;
  cout << "----" << endl;
  cout << "Area of the meshed unit sphere is " << area->InnerProduct(x, x) << endl;
  cout << "Should be " << (4 * M_PI) << endl;

//  ofstream mesh_ofs("sphere_refined.vtk");
//  mesh->PrintVTK(mesh_ofs);

  return 0;
}

sphere.vtk.zip

Hi,
Just reading this reminded me my question. I was wondering how does MFEM handle
the surface normal vector direction.
—
Syun'ichi

On Sep 4, 2018, at 4:38 AM, Irwin Zaid<EMAIL_ADDRESS>wrote:
Hello all,
I've been using MFEM for the last couple of weeks, and I've been very impressed by it. Thanks for making it available!
I'm working on an electromagnetic problem, but my question is much more general. Basically, I need to evaluate surface integrals of things like intensity of the electric field, i.e. |E|^2 = E dot E, where E is a 3D vector field that is solved for. This is a post-processing step to get certain quantities of interest. My understanding is that one simple way to do this is define a bilinear form such as
Array bdr_marker(1);
bdr_marker[0] = 1;
BilinearForm *bi = new BilinearForm(m_ptr->fes);
bi->AddBoundaryIntegrator(new VectorFEMassIntegrator(), bdr_marker);
bi->Assemble();
and then use that via bi->InnerProduct(x, x) on an appropriately-defined grid function x. I tried this, but I couldn't get it working for boundary integrals.
As such, I have created a minimal example that replicates my issue. In this example, I'm simply trying to use a BilinearForm to compute the volume and surface area of a meshed unit sphere (the mesh file is attached). I'm using scalar functions here, so I simply set the grid function to be 1 and integrate over the entire volume or what I think should be the spherical boundary. The volume comes out fine, the surface area comes out very wrong. Am I missing something here? How can I use a bilinear form to compute a surface integral with a grid function?
int main(int argc, char *argv[]) {
Mesh *mesh = new Mesh("sphere.vtk");
// Set the space for the high-order mesh nodes.
H1_FECollection fec(2, mesh->Dimension());
FiniteElementSpace nodal_fes(mesh, &fec, mesh->SpaceDimension());
mesh->SetNodalFESpace(&nodal_fes);
FiniteElementSpace *fespace = new FiniteElementSpace(mesh, &fec);
GridFunction x(fespace);
x = 1.0;
BilinearForm *vol = new BilinearForm(fespace);
vol->AddDomainIntegrator(new MassIntegrator());
vol->Assemble();
Array bdr_markers(1);
bdr_markers[0] = 1;
BilinearForm *area = new BilinearForm(fespace);
area->AddBoundaryIntegrator(new MassIntegrator(), bdr_markers);
area->Assemble();
cout << "Volume of the meshed unit sphere is " << vol->InnerProduct(x, x) << endl;
cout << "Should be " << (4 * M_PI / 3) << endl;
cout << "----" << endl;
cout << "Area of the meshed unit sphere is " << area->InnerProduct(x, x) << endl;
cout << "Should be " << (4 * M_PI) << endl;
//  ofstream mesh_ofs("sphere_refined.vtk");
//  mesh->PrintVTK(mesh_ofs);
return 0;
}
sphere.vtk.zip
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@mlstowell Thanks for the quick and detailed reply!
First, thanks for seeing the errors in that mesh. I'm generating my mesh externally with some other programs, it is just a tetrahedralization of the unit sphere. However, the file I gave you was written with MFEM's own VTK writer. I found that using the signature with the ref argument saved the mesh correctly, leading to the correct surface area and volume. (I also found saving it as a .mesh file was also fine.)
This brings back to my original question. @mlstowell, did you happen to try my example with different finite element collections, for instance an ND_FECollection? I find this does not work. This was actually the original problem I was struggling with today.
I've updated my example program to demonstrate this, and attached a new .mesh file. Using this program with this mesh will produce the correct surface area and volume for the H1_FECollection, but not for the ND_FECollection. Am I missing something here? The code is almost identical.
#include "mfem.hpp"
#include <fstream>
#include <iostream>

using namespace std;
using namespace mfem;

void do_h1() {
  Mesh *mesh = new Mesh("sphere.mesh");
  mesh->ReorientTetMesh();

  // Set the space for the high-order mesh nodes.
  FiniteElementCollection *fec = new H1_FECollection(2, mesh->Dimension());
  FiniteElementSpace nodal_fes(mesh, fec, mesh->SpaceDimension());

  FiniteElementSpace *fespace = new FiniteElementSpace(mesh, fec);

  FunctionCoefficient func([](const Vector &x) { return 1.0; });

  GridFunction x(fespace);
  x.ProjectCoefficient(func);

  BilinearForm *vol = new BilinearForm(fespace);
  vol->AddDomainIntegrator(new MassIntegrator());
  vol->Assemble();

  Array<int> bdr_markers(1);
  bdr_markers[0] = 1;

  BilinearForm *area = new BilinearForm(fespace);
  area->AddBoundaryIntegrator(new MassIntegrator(), bdr_markers);
  area->Assemble();

  cout << "Volume of the meshed unit sphere is " << vol->InnerProduct(x, x) << endl;
  cout << "Should be " << (4 * M_PI / 3) << endl;
  cout << "----" << endl;
  cout << "Area of the meshed unit sphere is " << area->InnerProduct(x, x) << endl;
  cout << "Should be " << (4 * M_PI) << endl;

  //  ofstream mesh_ofs("sphere_refined.vtk");
  //  mesh->PrintVTK(mesh_ofs);
}

void do_nd() {
  Mesh *mesh = new Mesh("sphere.mesh");
  mesh->ReorientTetMesh();

  // Set the space for the high-order mesh nodes.
  FiniteElementCollection *fec = new ND_FECollection(2, mesh->Dimension());
  FiniteElementSpace nodal_fes(mesh, fec, mesh->SpaceDimension());

  FiniteElementSpace *fespace = new FiniteElementSpace(mesh, fec);

  VectorFunctionCoefficient func(3, [](const Vector &x, Vector &v) {
    v(0) = 1;
    v(1) = 0;
    v(2) = 0;
  });

  GridFunction x(fespace);
  x.ProjectCoefficient(func);

  BilinearForm *vol = new BilinearForm(fespace);
  vol->AddDomainIntegrator(new VectorFEMassIntegrator());
  vol->Assemble();

  Array<int> bdr_markers(1);
  bdr_markers[0] = 1;

  BilinearForm *area = new BilinearForm(fespace);
  area->AddBoundaryIntegrator(new VectorFEMassIntegrator(), bdr_markers);
  area->Assemble();

  cout << "Volume of the meshed unit sphere is " << vol->InnerProduct(x, x) << endl;
  cout << "Should be " << (4 * M_PI / 3) << endl;
  cout << "----" << endl;
  cout << "Area of the meshed unit sphere is " << area->InnerProduct(x, x) << endl;
  cout << "Should be " << (4 * M_PI) << endl;
}

int main(int argc, char *argv[]) {
  do_h1();
  cout << "-----" << endl;
  do_nd();

  return 0;
}

sphere.mesh.zip

HI, @izaid ,
I'm glad you were able to sort out the issue with the mesh.  The problem with the ND space is perhaps more subtle.  You may think that if your vector field is v=(1,0,0) then v.v would equal 1 so your surface integral would integrate 1 over the surface of the mesh.  This is not the case.  What happens is that the surface integral first restricts the ND basis to the surface then takes the inner product and computes the surface integral.  Restriction to the surface means that the vector field becomes s=nx(vxn).  In this case n=(Cos[[Phi]] Sin[[Theta]], Sin[[Phi]] Sin[[Theta]], Cos[[Theta]]) which leads to a more complicated function to be integrated.  The answer turns out to be 8pi/3=8.37758...  Your new code computes 8.20508 which is close.
The ND basis is suitable for computing the tangential part of a vector field at a surface (this is what nx(vxn) gives you).  So, you could accurately compute surface currents, for example.  You could also use ND spaces to accurately compute the normal component of the curl of a vector field at the surface.  Other quantities will be difficult to compute at the surface using this basis.
If you need to compute the normal component of a vector field on a surface it is better to use RT spaces.  For example, you could convert an electric field E (using the ND basis) into D=epsilon E (using the RT basis).  The normal component of D in the RT basis would give you an accurate representation of the surface charge density.  Similar tricks apply to B and H where B is usually expressed using RT bases and H with ND.
I am curious why you need |E|^2 integrated on a surface.  Typically terms quadratic in E are related to energy density which is a volumetric quantity so a surface integral would not be appropriate.  Perhaps this is related to energy deposition in a very thin layer which cannot be resolved by the mesh?  If so, it might make sense to treat the components of E which are tangent to or parallel to this layer differently.
Best wishes,
Mark

@mlstowell Thanks for this, I now understand what is going on. Your explanations have been very helpful.
So, I'm working on a scattering problem and I want to compute the cross sections, which are basically power flux. These are defined as surface integrals either in terms of the Poynting vector S = E cross H, where H = curl(E), or in terms of E dot E. I'd be happy computing either of them -- I assumed the E dot E form was easier here, but maybe that's wrong.
I guess my question now is, can I compute either of these integrals in the ND basis easily? Or do I need to use an interpolator? The integrands are:

E dot E
S dot n = (E cross H) dot n, where H ~ curl(E)

Thanks again for your help!

Hi, @izaid ,
Yes, this should be easy as long as your surface is sufficiently far from your scatterer.  The trick is to use the Silver-Muller (a.k.a Sommerfeld) radiation condition which states that the outgoing waves must satisfy lim_{r->infty} [r(Hxn-E)]=0 where n is the unit vector in the radial direction.  This means that E will be tangential to the surface of the sphere and that the tangential part of H is a rotated copy of E.  If you follow through the nasty vector identities you end up with a surface integral of E_perp.E_perp which is, luckily, the most natural surface integral for ND bases.  The code from your original question should work perfectly:
  Array<int> bdr_marker(1);
  bdr_marker[0] = 1;

  BilinearForm *bi = new BilinearForm(m_ptr->fes);
  bi->AddBoundaryIntegrator(new VectorFEMassIntegrator(), bdr_marker);
  bi->Assemble();

If a sufficiently large spherical surface is impractical then you may need to integrate over a surface inside the sphere.  This is because you cannot accurately estimate the tangential portion of H on the outer boundary unless you know something more about it (like the Silver-Muller condition).  Using an interior surface allows you to approximate H as a ND field from CurlE, which is a RT field, in the least-squares-best-fit sense.  I doubt you want to do this but it should be possible.
Best wishes,
Mark


Add missing static 404 image

Questions
Answers

Description?
Add missing SVG static image following https://github.com/PrestaShop/ps-docs-theme/pull/2

Type?
bugfix

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes https://github.com/PrestaShop/docs/issues/870

How to test?
Builds the site, then visit a 404 page.

Possible impacts?

This time I hope it's going to work 😄 locally I had to do some tricks to get it working so it's hard to make sure it's going to be OK for production.

Aaaaaah I get it 😄 I thought "static" was a folder automatically generated.
But no.
The images/ folder is for images used in the README, the static/images/ folder is for images used in the theme.

Thank you

Fixed https://devdocs.prestashop.com/ajdh


make sure dependencies between patches on Phabricator works

When applying patches with parent diffs set something seems to go wrong repeatedly. Somehow arc patchis not able to apply these. I'm not really sure what the problem is and how we can work around this.
Maybe this is related to arc applying patches that were already merged. So maybe we need to manually iterate over the parents and check in the git log, if they were applied already...

maybe this problem solves itself when the github migration is over

seems to be solved


How To Do A Static Build

I am trying to compile ffmpeg for a static build for distrubiton. When I get this error when I do:
 ERROR: libvpl >= 2.6 not found

I am currently doing the scripts for installation on linux:
if build "oneVPL" "2023.3.1"; then
  # libaom bcfe6fb == v3.5.0
  download "https://github.com/oneapi-src/oneVPL/archive/refs/tags/v2023.3.1.tar.gz" "oneVPL-2023.3.1" "oneVPL"
  cd oneVPL-2023.3.1 || exit
  execute script/bootstrap
  execute script/build
  execute script/install
  build_done "libvpl" "2.3.1"
fi
CONFIGURE_OPTIONS+=("--enable-libvpl")

Works fine if I don't do a static build. So can libvpl be compiled for a static compiling of ffmpeg?

@ProdigyView please check the CMAKE file when compiling VPL, you can set the BUILD_SHARED_LIBS to OFF and this should provide the static lib you are looking for.
If you still have issues can you provide more information on your setup?

See also https://github.com/intel/cartwheel-ffmpeg/issues/286

Closed due to age and no further asks from the submitter.


Nuget restore fails on net461 project referencing netstandard2.0 project

Similar to https://github.com/Microsoft/vsts-tasks/issues/5047 and workaround https://github.com/Microsoft/vsts-tasks/issues/5065#issuecomment-324486955 solves it.

Create solution
Create netstandard2.0 library (A)
Create net461 library (B)
Reference (B) in (A)
Try NuGet restore task on solution

2017-08-30T13:53:29.4964698Z Errors in d:\a\1\s\test\MyLib.UnitTests\MyLib.UnitTests.csproj
2017-08-30T13:53:29.4964698Z     Project MyLib is not compatible with net461 (.NETFramework,Version=v4.6.1). Project MyLib supports: netstandard2.0 (.NETStandard,Version=v2.0)
2017-08-30T13:53:29.4964698Z     One or more projects are incompatible with .NETFramework,Version=v4.6.1.

Seems this task is using Nuget 4.0.0. This works with Nuget 4.3.0 (as in workaround above)
Potentially related: https://github.com/NuGet/Home/issues/5325, https://github.com/Microsoft/vsts-tasks/issues/5065

@Mardoxx Thanks for posting the workaround to this, much obliged.
VSTS team, when can we expect a bump to the nuget task to use >=4.3 of nuget?

@grantwar,
We recommend you use the Nuget tool installer to control the version of nuget used by all of your build steps.  Placing, the tool installer at the top of your build task will affect subsequent invocations of nuget because the tool installer will fetch the version you indicated an place it at the head of $PATH.
Keith

Similar to #5047 and workaround #5065 (comment) solves it.

Create solution
Create netstandard2.0 library (A)
Create net461 library (B)
Reference (B) in (A)
Try NuGet restore task on solution

2017-08-30T13:53:29.4964698Z Errors in d:\a\1\s\test\MyLib.UnitTests\MyLib.UnitTests.csproj
2017-08-30T13:53:29.4964698Z     Project MyLib is not compatible with net461 (.NETFramework,Version=v4.6.1). Project MyLib supports: netstandard2.0 (.NETStandard,Version=v2.0)
2017-08-30T13:53:29.4964698Z     One or more projects are incompatible with .NETFramework,Version=v4.6.1.

Seems this task is using Nuget 4.0.0. This works with Nuget 4.3.0 (as in workaround above)
Potentially related: NuGet/Home#5325, #5065

Update your NuGet installer to version 2.*. After that NuGet.exe will auto-update to version 4.3


Train function takes a --test option

Optionally allow you to predict and evaluate a test file at the end of training.
Saves time running a separate predict job.

Overdesign. It's a bit of a pain to run the extra command line when you're doing everything up on Floydhub, but not worth the UI complexity.


Update enable-public-ip-nsx-edge.md

Docs Build status updates of commit 6690ee4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/azure-vmware/enable-public-ip-nsx-edge.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@jjaygbay1

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


docs on adding recognizers

Documentation for:

Adding new local recognizers
Adding new remote recognizers + reference implementation
Updated README on the presidio-analyzer folder as well as in docs/analyzer/index.md

/azp run

/azp run


SIMD Harley-Seal Population Count

arXiv:1709.07821 Roaring Bitmaps: Implementation of an Optimized Software Library (Section 4.1.1)
arXiv:1611.07612 Faster Population Counts Using AVX2 Instructions

Our approach has been adopted by LLVM: it is used by its popular C compiler (clang).

Is this already done?

I am pretty sure it hasn't been implemented.

Our approach has been adopted by LLVM: it is used by its popular C compiler (clang).

llvm might be doing it for us :)

It does. Both for counting cardinality, and for counting while performing a set op.
https://godbolt.org/z/EKb3qvj5T


Sort arches in web view

Up to now, the list was sorted based on the order in which arches were
created. With manually added ones, the order was not correct.
JIRA: PDC-1107

Looks good to me.


New AWS-auth module doesn't respect the nodeGroup perrms

In version 19, the config map contained something like this
  - resource "kubernetes_config_map_v1_data" "aws_auth" {
      - data          = {
          - "mapAccounts" = jsonencode([])
          - "mapRoles"    = <<-EOT
                - "groups":
                  - "system:bootstrappers"
                  - "system:nodes"
                  "rolearn": "arn:aws:iam::xxx:role/initial-arm-eks-node-group-20240213202243321100000003"
                  "username": "system:node:{{EC2PrivateDNSName}}"
                - "groups":
                  - "system:masters"
                  "rolearn": "arn:aws:iam::xxx:role/GitHubActionsOIDC"
                  "username": "gh-role"
                - "groups":
                  - "system:masters"
                  "rolearn": "arn:aws:iam::xxx:role/hub-control-plane-hub-role"
                  "username": "allow-remote-control"

in 20 with the new aws-auth module change, we get this
  + resource "kubernetes_config_map_v1_data" "aws_auth" {
      + data          = {
          + "mapAccounts" = jsonencode([])
          + "mapRoles"    = <<-EOT
                - "groups":
                  - "system:masters"
                  "rolearn": "arn:aws:iam::xxx:role/GitHubActionsOIDC"
                  "username": "gh-role"
                - "groups":
                  - "system:masters"
                  "rolearn": "arn:aws:iam::xxx:role/hub-control-plane-hub-role"
                  "username": "allow-remote-control"
            EOT

In 19, the eks took the aws_auth_roles parameter, and ammeded it to a whole set of other roles. in 20, the input is the whole set.
The upgrade docs for 20 doesn't mention a good workaround.
The code lost in the upgrade took this
  node_iam_role_arns_non_windows = distinct(
    compact(
      concat(
        [for group in module.eks_managed_node_group : group.iam_role_arn if group.platform != "windows"],
        [for group in module.self_managed_node_group : group.iam_role_arn if group.platform != "windows"],
        var.aws_auth_node_iam_role_arns_non_windows,
      )
    )
  )

and merged it with the data supplied in the 0.19 module parameter of aws_auth_roles   in 0.20, aws_auth_roles is the sole truth. So if one applied the upgrade as documented, the node-groups would be removed from the aws-auth CM.

So if one applied the upgrade as documented, the node-groups would be removed from the aws-auth CM.

Correct, that's the intent

OK. As long as it's known. A mention of that in the upgrade notes might help. The terraform plan had a bit of noise in it, so people might miss the loss of node-group perms.
The Upgrade Migrations section implies a lossless transition. Put a little PR in place Not a word-smith by any means but added a brief warning here. https://github.com/terraform-aws-modules/terraform-aws-eks/pull/2953

I might have missed any other warning. But I feel that this is a pretty significant delta that would only bite people well after the miss.
If people miss this, I think their cluster will continue to work perfectly. But any managed node groups would slowly and confusingly shrink as their ASG's cycle. The new replacement nodes wouldn't register, and it undoubtedly happen week sor months after the mistake, making it really hard to diagnose the issue.

and

and

I think its pretty clear - unfortunately, this looks like a case where we might not have read the docs first before performing the upgrade 😅 . closing these out for now since I believe the upgrade docs are quite clear about the behaviors and actions

@bryantbiggs Speaking as someone going through this now, I'm not sure it's as clear as claimed.
The module v20 upgrade guide also contains the following note:

Note - EKS automatically adds access entries for the roles used by EKS managed nodegroups and Fargate profiles; users do not need to do anything additional for these roles

However, within the AWS migration guide, there's this seemingly contradictory note, that I believe is what the OP is referencing here

Important
Don't remove existing aws-auth ConfigMap entries that were created by Amazon EKS when you added a managed node group or a Fargate profile to your cluster. If you remove entries that Amazon EKS created in the ConfigMap, your cluster won't function properly. You can however, remove any entries for self-managed node groups after you've created access entries for them.

The plan on upgrading to v20 shows the removal of the ConfigMap entries that the AWS documentation seems to explicitly say not to remove. I understand that on new managed nodegroups, the aws-auth configmap entry would be correctly created by EKS. However, during an upgrade these nodegroups already exist and so does the configmap entry, which would not be recreated by EKS when removed by the Terraform plan

The plan on upgrading to v20 shows the removal of the ConfigMap entries that the AWS documentation seems to explicitly say not to remove.

I can try to work on getting the wording updated a little better in the docs; its perfectly safe and valid to remove the aws-auth ConfigMap entries for managed nodegroups and Fargate profiles after migrating to cluster access entries

Maybe this is a confusion over what "anything additional" refers to. From my reading, I would consider using the clowdhaus module snapshot and manually removing the ConfigMap from state as "additional" (as in, in-addition to upgrading the module), but you might be referring to steps that are in addition to those two steps?

I don't quite follow this. The point of the one-time upgrade module is to provide a better (smoother) path to upgrade existing clusters to use cluster access entries. If we did not provide this intermediate module/step, after changing the module version to v20.0 we would have a race condition that could pose problematic for users when upgrading. In that scenario, there is no guarantee that the cluster acess entry is enabled and the respective cluster access entries for managed nodegroups and Fargate profiles are created BEFORE Terraform deletes the aws-auth ConfigMap. So this intermediate module/step is a mix of v20.0 but with the aws-auth functionality still provided, which allows users to upgrade to cluster access entries without any distrubance to the aws-auth configmap. After cluster access entry is enabled, then you can move on to v20.x and this is where the ConfigMap will be destroyed by Terraform. If you need to retain/update the aws-auth ConfigMap, please see the upgrade docs for the different options and their respective paths for completing the upgrade

its perfectly safe and valid to remove the aws-auth ConfigMap entries for managed nodegroups and Fargate profiles after migrating to cluster access entries

What you're claiming here about the additional entries seems like its in direct conflict with the contents of AWS's own documentation. Not saying that you're wrong, just that the AWS docs say in a big red callout "don't remove these".

I don't quite follow this.

Me either, it was me attempting to try to understand if I was reading the guide as intended. It seems that I was reading it as intended, and its the claim about whether those entries are needed at all that's relevant here.


Use new env var to use bazel path

It would be helpful to support a var “IBAZEL_BAZEL_PATH” that would point to the desired bazel executable. This would allow me to explicitly setup my environment to point to the proper bazel.
The baze_path flag is less ideal as it requires me to configure it at invocation time.

In principle I don't oppose this change. The code would go in here and shouldn't require more than checking if that environment variable is set (see this for more docs on how to do environment variables in golang). If you could also add an IBAZEL_BAZELISK_PATH that would be appreciated. Also, please note that os.Getenv returns an empty string in the case that the value isn't set, so you'll probably do something like
if os.Getenv("IBAZEL_BAZEL_PATH") != "" {
  return os.Getenv("IBAZEL_BAZEL_PATH")
}

I would imagine that one would set IBAZEL_BAZEL_PATH to bazelisk instead of having separate variables. How would iBazel use IBAZEL_BAZELISK_PATH?

@joeljeske as I understand it, bazelisk is designed to have the same command line flags as Bazel. If you were to add the IBAZEL_BAZEL_PATH code described above here, you could just set it to be the path to your bazelisk binary.


Integration fails after login

Hi, I tried this integration when it was like v0.0.2 and it failed. Then I see #10 is fixed so I gave it another try(v0.0.6) but still in vein. I guess we have different issues.
Here is my debug log
2021-05-31 21:46:48 INFO (MainThread) [custom_components.wellbeing] Fetched data: [{'applianceName': 'CATZPI92', 'created': '2020-04-22T09:00:49.3906695Z', 'id': 246348, 'modelName': 'PUREi9', 'timeZoneStandardName': 'Asia/Taipei', 'pncId': '900277489943004361100106', 'domainId': 220194}, {'applianceName': 'CATZPA91', 'created': '2021-03-10T10:18:42.4117415Z', 'id': 674381, 'modelName': 'PUREA9', 'timeZoneStandardName': 'Asia/Taipei', 'pncId': '900943148051000585087076', 'domainId': 220194}]
2021-05-31 21:46:49 DEBUG (Recorder) [homeassistant.components.recorder.pool.RecorderPool] Connection <sqlite3.Connection object at 0xb2e0c350> checked out from pool
2021-05-31 21:46:49 DEBUG (Recorder) [homeassistant.components.recorder.pool.RecorderPool] Connection <sqlite3.Connection object at 0xb2e0c350> being returned to pool
2021-05-31 21:46:49 DEBUG (Recorder) [homeassistant.components.recorder.pool.RecorderPool] Connection <sqlite3.Connection object at 0xb2e0c350> rollback-on-return
2021-05-31 21:46:49 INFO (MainThread) [custom_components.wellbeing] Fetched data: {'pnc': '900277489', 'manufacturingDateCode': '943', 'serialNumber': '94300436', 'brand': 'ELECTROLUX', 'device': 'ROBOTIC_VACUUM_CLEANER', 'model': 'i92', 'variant': 'M1', 'colour': 'SPACETEAL', 'market': 'TAIWAN'}
2021-05-31 21:46:49 INFO (MainThread) [custom_components.wellbeing] CATZPI92: {'pncId': '900277489943004361100106', 'applianceData': {'applianceName': 'CATZPI92', 'created': '0001-01-01T00:00:00', 'modelName': 'PUREi9', 'pncId': '900277489943004361100106'}, 'twin': {'deviceId': '900277489943004361100106', 'properties': {'desired': {'applianceName': 'CATZPI92', 'firmwareVersion': '42.18', 'mute': False, 'tasks': {'5': {'enabled': False, 'start': {'weekDays': ['saturday'], 'time': '11:00:00'}}, '6': {'enabled': False, 'start': {'weekDays': ['sunday'], 'time': '11:00:00'}}, '7': {'enabled': True, 'start': {'weekDays': ['monday', 'tuesday', 'wednesday', 'thursday', 'friday'], 'time': '14:00:00', 'properties': {'zones': [{'powerMode': 2}]}}}}}, 'reported': {'availableLanguages': ['deu', 'eng', 'fra', 'pol', 'spa', 'kor', 'zht', 'zhs', 'jpn', 'ces', 'ita', 'rus'], 'capabilities': {'CustomPlay': {}, 'FreezeMapOnDemand': {}, 'InteractiveMap': {}, 'MultipleScheduledCleaningsPerDay': {}, 'PowerLevels': {}, 'PowerZones': {}}, 'batteryStatus': 6, 'robotStatus': 10, 'messageList': {'messages': []}, 'dustbinStatus': 'full', 'platform': '1.01', 'applianceName': 'CATZPI92', 'firmwareVersion': '42.18', 'language': 'eng', 'mute': False, 'powerMode': 2, 'tasks': {'5': {'enabled': False, 'start': {'weekDays': ['saturday'], 'time': '11:00:00'}}, '6': {'enabled': False, 'start': {'weekDays': ['sunday'], 'time': '11:00:00'}}, '7': {'enabled': True, 'start': {'weekDays': ['monday', 'tuesday', 'wednesday', 'thursday', 'friday'], 'time': '14:00:00', 'properties': {'zones': [{'powerMode': 2}]}}}}}}, 'status': 'enabled', 'connectionState': 'Connected'}, 'telemetry': {}}
2021-05-31 21:46:49 DEBUG (MainThread) [custom_components.wellbeing] Finished fetching wellbeing data in 0.791 seconds
2021-05-31 21:46:49 WARNING (MainThread) [homeassistant.config_entries] Config entry<EMAIL_ADDRESS>for wellbeing integration not ready yet; Retrying in background

In my wellbeing account, I have another vacuum robot device beside my A9. Not sure if this is what causing issues...

Hi.
You have a model that is currently not supported and that's probably why it's not working for you.
With the logs you have provided I hope I can add support for it.

Here's a thread in the forum regarding the Pure A9i vacuum cleaner.
https://community.home-assistant.io/t/integrating-eectrolux-pure-i9-robotic-vacuum/78648/19

Thanks for the info!
Is it possible to filter out the unsupported devices? As you can see I have an A9 and an PI9.2 in my account but all I want is to get the A9 working.

@hSATAC Have a go with the latest version. It should ignore everything except Air Purifiers.

@JohNan Thanks! It's working now.


Tree.mrca has side effect of changing rooting state

Here is a minimum working example:
>>> import dendropy as dp
>>> tree = dp.Tree.get(data="((10,(((5,(11)),7,9))),((((1)))));", schema="newick")
>>> leaf_taxa = [leaf.taxon for leaf in tree.leaf_node_iter()]
>>> print("before tree.is_rooted", tree.is_rooted)
before tree.is_rooted None
>>> tree.mrca(taxa=leaf_taxa[:2])
<Node object at 0x7fd2df9bfee0: 'None' (None)>
>>> print("after tree.is_rooted", tree.is_rooted)
after tree.is_rooted False

Works on multiple versions of Python, including Python 3.9.16 (main, Dec  7 2022, 01:12:08)  [GCC 11.3.0] on linux with dendropy 4.5.2.
In a more copy/pasteable form:
import dendropy as dp

tree = dp.Tree.get(
    data="((10,(((5,(11)),7,9))),((((1)))));",
    schema="newick",
)
leaf_taxa = [
    leaf.taxon for leaf in tree.leaf_node_iter()
]

print("before tree.is_rooted", tree.is_rooted)
tree.mrca(taxa=leaf_taxa[:2])
print("after tree.is_rooted", tree.is_rooted)

This side effect is can have very unexpected and perplexing consequences.
Consider, for example
>>> import dendropy as dp
>>> tree = dp.Tree.get(data="((10,(((5,(11)),7,9))),((((1)))));", schema="newick")
>>> leaf_taxa = [leaf.taxon for leaf in tree.leaf_node_iter()]
>>> print("before", tree.as_string(schema='newick'))
before ((10,(((5,(11)),7,9))),((((1)))));

>>> tree.mrca(taxa=leaf_taxa[:2])
<Node object at 0x7f70837fbee0: 'None' (None)>
>>> print("after", tree.as_string(schema='newick'))
after [&U] (10,(((5,(11)),7,9)),((((1)))));

(Note the addition of the [&U] prefix after calling mrca.)
as far as I can tell what is happening is Tree.mrca

calls Tree.encode_bipartitions

with suppress_unifurcations=False
and collapse_unrooted_basal_bifurcation=True, suppress_storage=False, and is_bipartitions_mutable=False left as default
https://github.com/jeetsukumaran/DendroPy/blob/0b0d8df448d32a8d829b91204b817ce6c98fb6ec/src/dendropy/datamodel/treemodel/_tree.py#L1553

calls Tree.collapse_basal_bifurcation

with set_as_unrooted_tree=True left as default
https://github.com/jeetsukumaran/DendroPy/blob/0b0d8df448d32a8d829b91204b817ce6c98fb6ec/src/dendropy/datamodel/treemodel/_tree.py#L3570

sets is_rooted = False

https://github.com/jeetsukumaran/DendroPy/blob/0b0d8df448d32a8d829b91204b817ce6c98fb6ec/src/dendropy/datamodel/treemodel/_tree.py#L2401

Thanks for bringing this to our attention and this fix!
I appreciate the issue and the fix makes sense. I'll look into this in a bit, and please don't hesitate to send me a reminder if I dawdle.
One question I have is whether or not the '[&U]' is cosmetic or is the tree.is_rooted property set. If the latter, is it only changed if it is None, or is it otherwise unconditionally set?
Maybe not related: does it make sense to ask for an MRCA of an unrooted tree? Should we throw an error?

Maybe not related: does it make sense to ask for an MRCA of an unrooted tree? Should we throw an error?

Not sure :sweat_smile: I was kind of wondering that too
It looks like the existing behavior is to use seed_node implicitly as the root, even when is_rooted is False.
Calculating the MRCA between leaves 10 and 11 on this tree gives the root node R as the result.

>>> import dendropy as dp
>>> tree = dp.Tree.get(
...     data="(10,(((5,(11)),7,9)),((((1)))))R;",
...     schema="newick",
... )
>>> leaf_taxa = [
...     leaf.taxon for leaf in tree.leaf_node_iter()
...     if leaf.taxon.label in ("10", "11")
... ]
>>> print(leaf_taxa)
[<Taxon 0x7efda9708ca0 '10'>, <Taxon 0x7efda9708f10 '11'>]
>>> print("first tree.is_rooted", tree.is_rooted)
first tree.is_rooted None
>>> tree.is_rooted = False
>>> print("second tree.is_rooted", tree.is_rooted)
second tree.is_rooted False
>>> print(tree.mrca(taxa=leaf_taxa[:2]))
<Node object at 0x7efda9708d90: 'R' (None)>

There could plausibly be some kind of argument to calcualte the MRCA between two nodes in an unrooted tree by taking the shortest path between them (would be well-defined/unique) and selecting the node closest to the middle of this path. How the MRCA between more than two nodes might be interpreted in an unrooted tree isn't immediately obvious to me Plus, for either of these cases, I doubt there's a substantial intentional use case for MRCA in an unrooted tree anyways.
In the scenario I ran into, I was basically relying on the ambiguity of the None rootedness state to prevent introduction of the [&R] or [&U] markers into newick string output. This can certainly be accomplished in other ways.
Warning or throwing an error for the False and None could be a reasonable choice, especially from a strict/literal interpretation of MRCA.

I did a quick sanity check to see if BioPython had made any kind of decision here, and it looks like they're also just using the implicit root of an unrooted tree without a warning or error.
>>> from io import StringIO
>>> from Bio import Phylo
>>> tree = Phylo.read(StringIO("(10,(((5,(11)),7,9)),((((1)))))R;"), "newick")
>>> tree.rooted
False
>>> leaf1 = tree.get_terminals()[0]
>>> leaf1
Clade(name='10')
>>> leaf2 = tree.get_terminals()[1]
>>> leaf2
Clade(name='5')
>>> mrca = tree.common_ancestor(leaf1, leaf2)
>>> mrca
Clade(name='R')

Ok, let's go ahead and merge this. Maybe a warning would be good thing to add though


Common flow for elastic constants calculations

Summary of Changes
I have implemented a common recipe to generate and calculate properties of deformed structures to obtain elastic constants.
I have also implemented and tested this flow for the EMT calculator. The code is heavily inspired by the slabs flow.
Checklist

[x] I have read the "Guidelines" section of the contributing guide. Don't lie! 😉
[x] My PR is on a custom branch and is not named main.
[x] I have used black, isort, and ruff as described in the style guide.
[x] I have added relevant, comprehensive unit tests.

Can one of the admins verify this patch?

Thanks, @Nekkrad! This looks great. Just a few minor comments from me.
Not for this PR, but just for your own reference, you may find the code in this module in matcalc helpful for the post-processing.

This is something I wanted to ask, should we include some of those post-processing into the result schema or do we leave the post-processing to the user?

Thanks, @Nekkrad! This looks great. Just a few minor comments from me.
Not for this PR, but just for your own reference, you may find the code in this module in matcalc helpful for the post-processing.

This is something I wanted to ask, should we include some of those post-processing into the result schema or do we leave the post-processing to the user?

For initial implementations, it is helpful to do what you've done thus far --- leave it to the user. If you find yourself using this recipe a bunch though, by all means, we can/should open a PR to add a step in the Flow that does post-processing of the final results so that the user doesn't have to think about it at all.

@Nekkrad Don't worry too much about the coderabbitai. I wanted to try it out and see what it did. Seems useful but perhaps too chatty....... 😅


[DeviantArt] Don't try to login if no account was provided

Currently, the DeviantArt ripper tries to login even if no user info is provided.
It's not harmful, but it generates exceptions needlessly and wastes time.
Ideally, the ripper should login only if a login and a password were provided.
https://github.com/RipMeApp/ripme/blob/24f07523086414256400f6b41af7f58350ed996b/src/main/java/com/rarchives/ripme/ripper/rippers/DeviantartRipper.java#L415-L445

I cooked up a fix for this, but at the start of the method there's already something to make sure that username and password are provided before continuing onto the section of code you provided.
In that case, it'll just add a thing to bypass the age barrier but won't actually login.
` // Populate postData fields
Map<String,String> postData = new HashMap<>();
    String username = Utils.getConfigString("deviantart.username", new String(Base64.decode("Z3JhYnB5")));

    String password = Utils.getConfigString("deviantart.password", new String(Base64.decode("ZmFrZXJz")));

    if (username == null || password == null) {

        throw new IOException("could not find username or password in config");

    }`


Update Debugging.md

The new debugging guide is great but the filename is wrong. Also added a screenshot. cc @ryanlntn

@jamonholmgren Are you sure the screenshot really has value? Maybe it would be better to show how to access remote devices

@Titozzz I find it's helpful to give people a preview of what they can expect when they do this. I can make it smaller if it's too overpowering?

Maybe it's too big... also we could add smaller screenshots of every steps in addition too 😄 ?

We need to merge this to fix the docs, we will iterate on it later

:tada: This PR is included in version 2.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Feat/acn 2

This PR adds the ACN to the ElCollectooorr.

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

Coverage data is based on head (32e9ec7) compared to base (c91ca48).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #170   +/-   ##
=======================================
  Coverage   89.56%   89.56%
=======================================
  Files          69       69
  Lines        5206     5206
=======================================
  Hits         4663     4663
  Misses        543      543

Flag
Coverage Δ

unittests
89.56% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Cannot load unpacked extension

Trying to follow the installation and getting stuck at the loading unpacked extension step. Using Arc.
Getting this error:

Could not load icon 'icons/16.png' specified in 'icons'.
Could not load manifest.

Running convert 48.png -resize 16x16 16.png inside extension/dist/icons seemed to fix it.

Sry about that, just fixed it. My .gitignore config was ignoring the icons public folder on my device

Yes! Found your post on r/ArcBrowser. I also stumbled upon your Sunbeam project the other day while looking for Deno modules 😂
Can I ask how you open a shell in a tab instead of the pop up?


🐛 Kotlin version

What were you trying to do?
* What went wrong:
A problem occurred configuring project ':react-native-vision-camera'.
> Could not open proj generic class cache for build file '/Users/alejandrorios/Desktop/proyectos/geniox-lpa/node_modules/react-native-vision-camera/android/build.gradle' (/Users/alejandrorios/.gradle/caches/8.0.1/scripts/jq16hqcegupdnccgdyeif3f4).
   > BUG! exception in phase 'semantic analysis' in source unit '_BuildScript_' Unsupported class file major version 64

Reproduceable Code
No response
What happened instead?

If I modify this from node_modules the error stops appearing, the problem is that I have to modify it every time I install a dependency
Relevant log output
No response
Device
NA
VisionCamera Version
latest
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

This should be fixed with this PR https://github.com/mrousavy/react-native-vision-camera/pull/1652


Allow for header based matching on the same path

In order to support matching based on headers or query params for the same path I had to change line 148 of
https://github.com/monimesl/istio-virtualservice-merger/blob/main/api/v1alpha1/virtualservicemerge_types.go
from:
targetRoutes = append(targetRoutes, pRoute)
to:
targetRoutes = append([]*v1alpha3.HTTPRoute{pRoute}, targetRoutes...)
This is because istio selects the first matching entry and since we have default services for the path and are matching headers for feature environments I needed the header matches to be listed first.
Not sure if you want this change as a PR but for now I've built my own image with the change and it is working for us.

Hello @jonathan-whitaker. We have been in your shoe.
I agree that the default implementation will have issues if a more generalize route is at then top of the list since istio choose a route by doing sequential match testing.
How

Any news here?
Is header matching with default route working?


Image.asset will block UI thread when load large image

If Image.asset load a large image(> 3MB),  it will block ui thread ~200ms in Pixel XL, when PlatformMessageResponseDart::Complete

Hi @zljj0818
Can you please describe the issue more in details
and provide your flutter doctor -v,
your flutter run --verbose
and a minimal complete reproducible code sample.
Please refrain from posting pictures if not necessary.
Thank you

The root casue is std::vector has performance issue for large size.

@liyuqian

Thanks for reporting @zljj0818 ! Do you have a minimal Flutter app that I can use to quickly reproduce this issue, or do you have a trace that shows what happened on the UI thread?

I tried measuring the PlatformMessageResponseDart::Complete UI thread task.  On a Moto G4 in profile mode using a 3.6MB asset the task ran in 30-40ms.
PlatformMessageResponseDart needs to copy the asset data into a buffer whose lifetime can be owned by Dart.
PlatformMessageResponseDart is creating a std::vector to hold the buffer:
https://github.com/flutter/engine/blob/master/lib/ui/window/platform_message_response_dart.cc#L41
The std::vector data is being initialized during construction even though it is immediately overwritten.  We could save time by using a malloc buffer or having Dart_NewTypedData allocate the buffer.

@jason-simmons Yes, PlatformMessageResponseDart::Complete do not consume time, std::vector does, see my comment above.
You can just test std::vector(3600000).
I have fixed this, it will reduce to ~5ms  on Pixel XL

I will create a pull request

@jason-simmons  I have created pull request: https://github.com/flutter/engine/pull/18808


Optimize IO#read_string(0)

When fetching an empty string from a remote server, we can avoid a heap allocation by returning an empty string as an optimized special case.
I also checked and IO#read_fully(slice : Slice) does not need this optimization. It already short-circuits if there is nothing to read.

Looks like the tests failing in CI have been failing on previous commits on the master branch, as well. Are they just flaky?

yup, interpreter std tests are flaky indeed.


Deleting a file with "ctrl+d" is not working on Windows11

Describe the bug
I use "ctrl+n" to create a new file in SPF. Then I use "ctrl+d" to delete the newly created file. But I can still see the file in SPF. I can also see the file in Windows file explorer.
System information (please complete the following information):

OS: [Windows11]
Version [22631.4169]
Superfile Version [v1.1.4]

Yes. I met the same bug on win 11.

duplicate of #335

@wxutopia if you are using powershell can you check if there is any powershell hotkey conflict with ctrl+d?

@yorukot
I use powershell inside "Terminal Windows Version : 1.21.3231.0"
It is the same behavior with cmd.exe inside "Terminal Windows"
I don t have any ctrl-d hotkey configured. It print ^D
I also tested cmd.exe and  WindowsPowerShell\v1.0\powershell.exe in the default windows console :
I have the same behavior : ctrl-d display the confirm dialog, the process show the deleted file, but does not delete file.


How do map tiles work?

What's the concept behind putting the map tiles onto the globe based on zoom level?

Cesium's terrain/imagery rendering is described here:
http://cesiumjs.org/presentations/Rendering the Whole Wide World on the World Wide Web.pdf
Please post questions like this to the forum instead of creating GitHub issues.

@kring That's amazing. Thank you!


docs: clarify supported distros for manager

part of: https://github.com/osism/testbed/issues/1834


Fixing the User Forum link on the welcome page

This corrects the issue brought up in WFLY-19044 where the User Forum link on the http://localhost:8080 welcome page is out of date.
This changes the URL from https://community.jboss.org/en/wildfly to https://groups.google.com/g/wildfly

Hello, awesomebjt. I'm waiting for one of the admins to verify this patch with /ok-to-test in a comment.

Thanks @awesomebjt!

@awesomebjt I assigned a "Benjamin Thompson" JIRA account to WFLY-19044 in an attempt to give you credit for fixing it. I figure that account is you; if it's not please let me know. Thanks!

You found the right one. Thanks!


Create initial support section

Context
We'd like a support section to contain functionality like:

Showing TRN Requests
Showing Emails
A Performance dashboard
Sidekiq (just a link, no actual UI since we will use Sidekiq's)
Auditing
Showing stored Validation errors

Changes proposed in this pull request
Mostly the initial boilerplate and dotenv.
Needs:

[ ] Two environment variables in the dev/preprod/production environments
[ ] A basic system test

Guidance to review

Checklist
https://trello.com/c/WEBORP57

[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

I'm not wild about having this section surfacing PII unless we definitely know we need it, which I'm not sure we do yet.

@benilovj agree, PII removed :+1:

Will there be a signed in as/sign out link at the top?

@fofr eventually, currently just with basic auth there is no real "sign out" state...


Commercial use

Hi there,
Could you please clarify the situation regarding use of the output from the various systems for commercial use?
All system but FourCastNet V1 seem to be under an Apache 2.0 licence which allows commercial use - but there is a conflicting note for Pangu Weather which says "The commercial use of these models is forbidden." (https://github.com/ecmwf-lab/ai-models-panguweather/tree/main).
Thanks
Dan Harding

This is correct. Although the code of the plugin is under the Apache 2.0 license, the model weights maybe under a different licence. In the case of Pangu-Weather, the weights are available under the CC-BY-NC-SA 4.0 license (NC is for "non-commercial"). See https://github.com/198808xc/Pangu-Weather for more information. I suggest that you check the license associated to each model on their main home page, that we link from each of the corresponding plugins.

Thank you


build(package & releaserc): semantic release bump to 22.0.5 and node to latest & gulp release plugin

semantic release bump to 22.05 & node to latest and gulp plugin to publish releases on teams channel

:tada: This PR is included in version 9.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Draft: Rework linting setup for JS, TS and Vue

Summary of Changes

Add JavaScript, TypeScript and Vue project with eslint flat config
Update JavaScript eslint rules to latest and include stylistic rules
Update TypeScript eslint rules to latest and include stylistic rules
Update Vue eslint rules to latest and include stylistic rules

Open Todos

[ ] Check support in VSCode and add documentation if required (there is at least one config option to change)
[ ] Update changelogs
[ ] Create releases to npm
[ ] Update nx setup to longer use compat mode

JS

[ ] Fix Parse errors in imported module 'eslint-config-team23-standard': parserPath or languageOptions.parser is required! (undefined:undefined)
[X] Check rule functionality by adding them to larger project
[X] Check rule flat config merging (probably need to update parser options)

TS

[ ] Update rules related to JS rules
[ ] Check unused TS rules
[ ] Apply same structure as in JS

Vue

[ ] Fix downstream vue project dependency usages (after checking e.g. functionality of JS, create a release and use in Vue)
[ ] Check unused Vue rules
[ ] Apply same structure as in JS

Additional Notes

This is currently a WIP

@erikherberg nitpick when running bun build ./src/index.ts --target=node --outfile eslint.config.js you will get an error:
1 | import tseslint from "typescript-eslint";
                         ^
error: Could not resolve: "typescript-eslint". Maybe you need to "bun install"?

Also there is no info to run this command from inside a library.
So your README might need an update.


Show Pet category list with count in the home page

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

This is not needed now


mtproto problem on telegram x

hi
we've setup mtproto server on windows server with JSMTProxy (nodejs) and it's work on all clients like android, desktop, ios but it didn't work on telegram x.
it sais error 400 on proxy list in the app.
please fix that.

This is a bug in the proxy implementation, which doesn't support MTProto intermediate TCP transport 0xeeeeeeee. There proxy checks only for abridged TCP transport 0xefefefef. 0xeeeeeeee should be allowed there to allow TDLib to use that proxy.

how we can edit the code to allow 0xeeeeeeee ?

It can be fixed pretty straightforward: only check on the line that I've specified need to be fixed. It currently allows only 0xefefefef as bytes 56-59 and it should also allow 0xeeeeeeee.

sorry i tried many times to edit code with && and || for adding 0xee to this line. can you edit that and type here?
			if (dec_auth_packet[56 + i] != 0xef) {

thanks

For example this is how it is done in the official MTProto proxy.

i tried to change that to
for (var i = 0; i < 4; i++) {
if (dec_auth_packet[56 + i] != 0xeeeeeeee) || (dec_auth_packet[56 + i] != 0xefefefef) {
socket.destroy();
return;
}
}
but it hangs on loop and didn't start, need i change somewhere else?

i have problem too
plz someone edit that code and send here
tnx

@levlam can you give us a help to resolve the issue?

@titancomputer
function check (num) {
  for (var i = 0; i < 4; i++) {
    if (dec_auth_packet[56 + i] != num)
      return false;
  }
  return true;
}

if (!check(0xee) && !check(0xef)) {
  socket.destroy();
  return;
}

@Bannerets thank you for codes.
i've replaces this section but didn't work on telegram x android as same as before. all others do fine.

@titancomputer It may be also needed to replace there 0xef with 0xee, but then proxy will work only with Telegram clients, which uses intermediate protocol. More changes are needed to support both protocols.

@levlam ok, thank you for responding, i'll wait for them to edit.

There are no bugs in the TDLib support of MTProto-proxy or MTProro-protocol in general, so I'm closing this issue.
Please address the issue to developers of affected third-party proxies.


 in table triggers MD33

We use  inside table to spreads the description in a cell over several lines to add readibility.
This triggers MD33 but I dont think it should.

It sounds like you think the rule is behaving as described, but don’t like that behavior in your scenario? If so, you can disable MD033 entirely or use the allowed_elements parameter to exempt just the br tag.

Closing as it seems the original and related questions are both addressed.


added demo meta tags and scripts

Hi @groovemen , added seo meta tags and scripts to the demo pages.

Hello @mberneti, thank you for your interest in working with our products and also for your contribution.
All the best,
Stefan

your welcome @groovemen


ISO 8601 date format

Just wanted to let you know that ISO 8601 is a rather complicated format and it's not easy to catch all of its possibilities with NSDateFormatter: for example, both 2015-07-01 and 2015-07-01T00:00:00 are allowed, but aren't parsed by current implementation.
It was very easy to combine iso-8601-date-formatter with KZPropertyMapper, but perhaps it should be used by default.

@ilyannn Yeah I know, but I don't want to add extra dependency, it's very straighforward to add any kind of boxing to kzpm


[REQUEST] button to remove all the not-found wordlists

Describe the solution you'd like
Please add a buton to the wordlists manager to remove all the not-found wordlists. (like if it was moved or something so the location is not being updated so they would be useless in the db)

I will close this now cuz it's added.


Added the crossbow from the GCI task

I added the crossbow. I will create a pull request with the crafting recipe in the WoodAndStone module. Not all tests passed, but that was already the case I believe.

Can one of the admins please verify this patch?

@GooeyHub: ok to test
Thanks for the contribution - will review within an hour 🙂

Hooray Jenkins reported success with all tests good!


Making the code dataset agnostic

This PR adds a datasets module with a load_dataset function that loads datasets, preprocesses them, and returns various important parameters (e.g. num channels, output dimension) to perform training.
This PR also makes the model_search.Network and model.NetworkCifar objects agnostic to the number of input channels of the data (as far as I'm aware).

Wrong place


Option to suppress global switches in subcmd help

Changes to support https://github.com/alire-project/alire/pull/1462

I'm merging given that https://github.com/alire-project/alire/pull/1462 was already approved.


STAC validator does not run as expected

Hi,
I'd like to test the STAC validator on my backend, but cannot get it running on my system: miniconda Python 3.10.4 on Windows 10 64bit.
After installing the package using pip install stac-validator I tried to run the executable: stac-validator -v stac-collections.out, which led to the error:
Please upgrade from version  to version 1.0.0!

[
    {
        "version": "",
        "path": "stac-collections.out",
        "schema": [
            ""
        ],
        "valid_stac": false,
        "error_type": "KeyError",
        "error_message": "'stac_version'"
    }
]

I think there's something wrong internally, so I haven't attached stac-collections.out, but can do so if you want me to.
Running from within the python CLI leads to a similar error:
>>> from stac_validator import stac_validator
>>> url = 'https://geodb.openeo.dev.brockmann-consult.de/collections'
>>> stac = stac_validator.StacValidate(url)
>>> stac.run()
False
>>> print(stac.message)
[{'version': '', 'path': 'https://geodb.openeo.dev.brockmann-consult.de/collections', 'schema': [''], 'valid_stac': False, 'error_type': 'KeyError', 'error_message': "'stac_version'"}]

Hi @thomasstorm Can I see your file if it's not too long? The error may be saying that you have not specified a stac version. It's not recognizing your file as an item or a collection either I think?

Sure, it's attached. Note that I had to rename it to .txt so I could upload it.
stac-collections.txt

Hi you have to take this part out {"collections":
Your file should start like this: {"stac_version": "0.9.0",

Here's an example https://github.com/stac-utils/stac-examples/blob/main/landsat8/LC81530252014153LGN00.json

Hi you have to take this part out {"collections":
Your file should start like this: {"stac_version": "0.9.0",

That's interesting! I was following the directions of openEO, who state that their specification is compatible with STAC API 0.9.0. So that piece of information is wrong, I guess, and they mean they are compatible with v1.0.0 instead?
My result seems to be compatible with STAC 1.0.0:
https://api.stacspec.org/v1.0.0-rc.1/collections/#tag/Collections

What you are looking at is an api response which is different than normal STAC format.

Ok - sorry about that, I was not aware that there are differences.
In any case, the error message is not helpful -- it insinuates that I should upgrade the stac validator, and does not really tell me what's wrong.

It could be better yes.

#220 is pretty much asking for what is missing here.


After elastic search support, it would be cool if marvel is available

https://github.com/dominikzogg/vagrant-php/pull/34

https://github.com/dominikzogg/vagrant-php/commit/52d1e363c49fd014dbb3551e3a0c0958af41d3f6


npm run build -> 'rollup:fesm' error

I tried the "npm run build" command. But it does not work :(
NPM -v: 5.6.0
NODE -v: 8.9.4
Do you have any idea about this? Thanks.
Results:
`npm run build

<EMAIL_ADDRESS>build /Users/ahmet/code/ng-test-package
gulp build

[16:40:46] Using gulpfile ~/code/ng-test-package/gulpfile.js
[16:40:46] Starting 'clean:dist'...
[16:40:46] Starting 'clean:tmp'...
[16:40:46] Starting 'clean:build'...
[16:40:46] Starting 'compile'...
[16:40:46] Finished 'compile' after 2.43 ms
[16:40:46] Finished 'clean:dist' after 13 ms
[16:40:46] Starting 'copy:source'...
[16:40:46] Finished 'clean:tmp' after 20 ms
[16:40:46] Finished 'clean:build' after 21 ms
[16:40:46] Starting 'clean'...
[16:40:46] Finished 'clean' after 297 μs
[16:40:46] Starting 'build'...
[16:40:46] Finished 'build' after 6.08 μs
[16:40:46] Finished 'copy:source' after 64 ms
[16:40:46] Starting 'inline-resources'...
[16:40:46] Finished 'inline-resources' after 8.09 ms
[16:40:46] Starting 'ngc'...
[16:40:48] Finished 'ngc' after 1.8 s
[16:40:48] Starting 'rollup:fesm'...
Error at /Users/ahmet/code/ng-test-package/playground/index.ts:9:31: Cannot find module 'ng-test-package'.
Error at /Users/ahmet/code/ng-test-package/src/sample.component.spec.ts:7:1: Cannot find name 'describe'.
Error at /Users/ahmet/code/ng-test-package/src/sample.component.spec.ts:14:3: Cannot find name 'beforeEach'.
Error at /Users/ahmet/code/ng-test-package/src/sample.component.spec.ts:28:3: Cannot find name 'it'.
Error at /Users/ahmet/code/ng-test-package/src/sample.component.spec.ts:29:5: Cannot find name 'expect'.
events.js:183
throw er; // Unhandled 'error' event
^
Error: Could not resolve entry (/Users/ahmet/code/ng-test-package/build/index.js)
at error (/Users/ahmet/code/ng-test-package/node_modules/gulp-rollup/node_modules/rollup/dist/rollup.js:185:14)
at resolveId.then.id (/Users/ahmet/code/ng-test-package/node_modules/gulp-rollup/node_modules/rollup/dist/rollup.js:9551:6)
at
at process._tickCallback (internal/process/next_tick.js:188:7)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: gulp build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      /Users/ahmet/.npm/_logs/2018-01-12T13_40_50_210Z-debug.log`

Closing as #267 is merged

Thank you, @izifortune 👍

I've just done a fresh install of node, yo and this generator and am getting the error when running npm run build.
node -v v9.9.0
npm -v 5.6.0


Suggesting improvements regarding the Rotten Tomatoes private API

Hi!
I've done some research on my part, and discovered a few interesting things about the Rotten Tomatoes private API. I guess you'd want to implement them :)

When browsing lists (for example, 'dvd-streaming-all'), you can specify the 'page' parameter, starting from 0. Example:

https://www.rottentomatoes.com/api/private/v2.0/browse?type=dvd-streaming-all&page=0

By default, lists like 'dvd-streaming-all' display 32 results on one page. You can change this, using the 'limit' parameter. Example:

https://www.rottentomatoes.com/api/private/v2.0/browse?type=dvd-streaming-all&page=0&limit=100

You can get detailed info on a specified movie like this 'https://www.rottentomatoes.com/api/private/v1.0/movies/{id}'. Example:

https://www.rottentomatoes.com/api/private/v1.0/movies/12911

Also, if while browsing a list you specify a page, that is out of range, you'll get this result:
{"counts":{"count":0,"total":0},"results":[],"debugUrl":"http://rt-client-facade-v2-6-1.aws.prod.flixster.com/movie?expand=true&include=%5B%22movieSupplementaryInfo%22%2C%22audienceSummary%22%2C%22affiliates%22%2C%22criticSummary%22%2C%22moviePersonnel%22%2C%22moviePersonnel.actors%22%2C%22moviePersonnel.actors.person%22%5D&filter=%7B%22country%22%3A%22UA%22%2C%22search%22%3A%7B%22mpaa-rating-min%22%3A%22G%22%2C%22release-year-max%22%3A2017%2C%22filter-release-date%22%3A%22false%22%2C%22offset%22%3A319968%2C%22sort%22%3A%22DVD_RELEASE_DATE_DESC%22%2C%22distribution%22%3A%22ANY%22%2C%22tomatometer-min%22%3A0%2C%22mpaa-rating-max%22%3A%22UN%22%2C%22release-day-of-month-max%22%3A17%2C%22limit%22%3A32%2C%22release-month-max%22%3A8%2C%22tomatometer-max%22%3A100%2C%22use-dvd-date%22%3Atrue%2C%22status%22%3A%22LIVE%22%7D%7D"}

Using this 'debugUrl', you can get direct access to their servers on AWS :)
This is what this url looks like when decoded:
http://rt-client-facade-v2-6-1.aws.prod.flixster.com/movie?expand=true&include=["movieSupplementaryInfo","audienceSummary","affiliates","criticSummary","moviePersonnel","moviePersonnel.actors","moviePersonnel.actors.person"]&filter={"country":"UA","search":{"mpaa-rating-min":"G","release-year-max":2017,"filter-release-date":"false","offset":319968,"sort":"DVD_RELEASE_DATE_DESC","distribution":"ANY","tomatometer-min":0,"mpaa-rating-max":"UN","release-day-of-month-max":17,"limit":32,"release-month-max":8,"tomatometer-max":100,"use-dvd-date":true,"status":"LIVE"}}

P.S. Also, I discovered the private API for Flixster android app. I used mitmproxy and my Android phone with SSLUnpinning2.0 installed. This API is very similar to Rotten Tomatoes API, and probably uses the same database (they have the same owner), as the movie IDs match.
Example requests to Flixster API(get detailed info on movie with id of 12911):
https://api.flixster.com/android/api/v2/movies/12911.json

Just thought you may be interested.

Thanks for the great information @savandriy!

Closing given #18 and #17

old api link
http://api.rottentomatoes.com/api/public/v1.0/movie_alias.json?type=imdb&id=
Do you know the new api link

@xsabianus can you add more context to your question, please?

I had movie info with imdb id. but the link has changed. not even with rotten apikey.
I am using this code. my English is bad. sorry.
$rott = file_get_contents("http://www.omdbapi.com/?apikey=xxxxxx&i=tt".urlencode($movie)."&tomatoes=true");  $regex = "#\"tomatoURL\":\"(.*)\"\,\"DVD\"#U";   preg_match_all($regex, $rott, $rot1, PREG_SET_ORDER);   $rotzek = $rot1[0][1];  $rotdebe = file_get_contents($rotzek);  $rotidx = get_match('/<meta name=\"movieID\" content=\"(.*)\">/isU',$rotdebe);          echo 'rotid'.$rotidx.'<br>';

This repository is not in any way associated with the OMDb API. I suggest you reach out to the maintainer of that API for any problems you might have.


chore(sink): improve force append only warning

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
A user asked about the force append only semantics and I found the warning somehow misleading.
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

A user asked about the force append only semantics and I found the warning somehow misleading.

do you think we should add an example in the doc about what force_append_only means?

A user asked about the force append only semantics and I found the warning somehow misleading.

do you think we should add an example in the doc about what force_append_only means?

Good to have one.


Support multiple-image inference

Hi,
it seems like only single-image inference is supported in StableDiffusionControlNetPipeline.
For multi-image inference, one could go to the location of the diffusers package and modify the "# 2. pre-process" code block (around line 600).
original:
# 2. pre-process
sample = self.conv_in(sample)
if controlnet_hint is not None:
    sample += self.controlnet_input_hint_block(controlnet_hint)

modified:
# 2. pre-process
sample = self.conv_in(sample)
if controlnet_hint is not None:
    hint = self.controlnet_input_hint_block(controlnet_hint)
    if hint.shape[0] > 1:
        hint = hint.repeat(2, 1, 1, 1)
    sample += hint

Hope it helps.

Thanks for pointing out, but StableDiffusionControlNetPipeline has been officially supported in diffusers now, I will update this tutorial soon to keep pace with it.

@haofanwang Thanks. I'll close the issue.


UP-4655:  Add some TRACE and DEBUG log entries at key milestones for understanding performance of some key requests

https://issues.jasig.org/browse/UP-4655

Coverage increased (+0.0009%) to 22.52% when pulling ecb8990366bd267dab71a80f9277936e1572265b on drewwills:UP-4655 into c2205994b6b5b1d860d69565ba3de34f3d802665 on Jasig:master.

Coverage decreased (-0.0007%) to 22.518% when pulling f1f9b534d9c8185a29162c2b48b2614787265ebe on drewwills:UP-4655 into c2205994b6b5b1d860d69565ba3de34f3d802665 on Jasig:master.

Coverage increased (+0.003%) to 22.521% when pulling 5fd71715612fe404925b411a4b4ca2c216b9573f on drewwills:UP-4655 into c2205994b6b5b1d860d69565ba3de34f3d802665 on Jasig:master.

Coverage increased (+0.003%) to 22.522% when pulling de749c7a7005909d9911fd50dc0915e460f264ec on drewwills:UP-4655 into c2205994b6b5b1d860d69565ba3de34f3d802665 on Jasig:master.


Add Material Design animations

Have you thought about adding material design animations like this library?
https://github.com/chinchang/cta.js
Maybe you can actually include that library.

We will work on adding more animations, but it won't be in the form of that library.

Great, thanks :)
It's the only thing left in materialize to develop full material design websites.


Merge prTesting_201902120846 branch to master branch

OPS Build status updates of commit 7e0df62:
:clock10: Preparing: average preparing time is 44 sec(s)

OPS Build status updates of commit 7e0df62:
:clock10: Incremental building: average incremental building time is 14 sec(s)

OPS Build status updates of commit 7e0df62:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add string slice parsing logic to klenv loader.

This will close issue #42.

Codecov Report

Merging #43 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #43      +/-   ##
==========================================
+ Coverage   91.66%   91.69%   +0.03%
==========================================
  Files          29       29
  Lines        1331     1336       +5
==========================================
+ Hits         1220     1225       +5
  Misses         71       71
  Partials       40       40

Impacted Files
Coverage Δ

loader/klenv/envloader.go
92.85% <100%> (+0.96%)
:arrow_up:

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 d4eea79...ab610c3. Read the comment docs.

This looks good to me. Would you mind adding a line to the klenv loader's README explaining this feature?
Thanks a lot

@francoispqt re-pushed with description in readme

Thank you


RenderDoc crashes without a chance for recovery if the GPU driver is disabled in device manager when using Vulkan

Description
Upon disabling the GPU used by a capture in Device Manager, RenderDoc crashes (without the normal crash report screen popping up or the device lost screen popping up, and without a chance to save the capture).  This happens even if the device is enabled immediately afterwards.
Now, it might sound like the fix for this is "don't disable the device; why would you ever do that?", but there are a few situations where I actually have ran into this in practice.  Until recently, NVIDIA removed other drivers from the adapter list in their vulkan layer (RenderDoc disables that by setting the DISABLE_LAYER_NV_OPTIMUS_1 environment variable, but it affected testing things outside of RenderDoc).  That no longer seems to be an issue as of NVIDIA driver version 10.0.19042.1466, fortunately.  However, I still need to disable the NVIDIA driver if I want to test OpenGL with my integrated Intel GPU (since OpenGL doesn't provide a way of selecting an adapter, or if there is a way, Dolphin doesn't expose it).  Although this may be an infrequent use case, it's annoying to need to remember to save all open RenderDoc captures or else they'll be lost.
This happens with both my NVIDIA GPU and my integrated Intel GPU.  This issue does not happen with D3D11; instead, the error message below appears (and RenderDoc stops displaying things, but it is still possible to save the capture, which is much better behavior than crashing with no chance of recovery).  This issue also doesn't happen if the disabled GPU isn't the one the capture is using.  I haven't tested D3D12 or OpenGL.

This is displayed with D3D11, but not with Vulkan.

Steps to reproduce

Create a capture in RenderDoc.
Open device manager (Win+R, enter devmgmt.msc); you may need to be using an administrator account.
Find the current GPU "display adapters".
Right-click it.
Select "disable device".
Right-click the same GPU.
Select "enable device".
Switch back to RenderDoc.
RenderDoc will immediately exit upon being switched to, with no opportunity to save the capture, and no error message.

Environment

RenderDoc version: v1.17
Operating System: Windows 10.0.19042.1466
Graphics API: Vulkan

As mentioned in the linked issue I suspect this is just a case where a device lost error isn't reported and the ICD or perhaps vulkan loader is either crashing or forcibly killing the process, which I've seen before in the ICD at least. I don't expect this is something I can do anything about.
I'll try to reproduce to confirm, but unless this is RenderDoc mishandling a reported device loss I don't expect this is something I will intend to support.
FWIW:

Until recently, NVIDIA removed other drivers from the adapter list in their vulkan layer

Yes this user-hostile behaviour is why I set the environment variable - rather than removing the device I'd suggest setting that environment variable globally, or removing the registration of the optimus layer.

However, I still need to disable the NVIDIA driver if I want to test OpenGL with my integrated Intel GPU

I've never tested on a hybrid system, but as I understand it either the nvidia control panel or windows will let you explicitly override the GPU to be used with OpenGL where you can't select one in code.

I've never tested on a hybrid system, but as I understand it either the nvidia control panel or windows will let you explicitly override the GPU to be used with OpenGL where you can't select one in code.

There does seem to be an option in the nvidia control panel for switching the "preferred graphics processor" between "auto-select", "High-performance NVIDIA processor", and "Integrated graphics", but it doesn't seem to actually affect OpenGL.  It also says that "Windows OS now manages the selection of the graphics processor.  Open Windows graphics settings", and the page that opens does work (and it requires explicitly choosing the program to change settings for).

As mentioned in the linked issue I suspect this is just a case where a device lost error isn't reported and the ICD or perhaps vulkan loader is either crashing or forcibly killing the process, which I've seen before in the ICD at least. I don't expect this is something I can do anything about.

   If I select "ignore for this session", nothing crashes (but nothing new renders either).  When I try to close Dolphin, it crashes in the call to vkDestroySwapchainKHR with a null-pointer dereference somewhere in nvwgf2umx.dll.

I wasn't quite able to reproduce this exactly as described, but I believe I saw the same behaviour. For me on my nvidia system it hung indefinitely inside a vkResetCommandBuffer call, but that sounds like it might be the same bug since for a while I've been seeing it do that any time I'd expect a device lost error to be reported. I tested on AMD and I did notice that the error handling was incorrect for actually detecting a device lost error (fixed in the first commit above) and that some cases in the code weren't properly handling vkMapMemory returning a NULL pointer (second commit above). However on AMD I still ran into a crash on shutdown inside AMD's driver.
If you're seeing a crash on shutdown on nvidia then I expect that's the same thing which is killing RenderDoc. You could try attaching a debugger to RenderDoc before doing the repro steps to see what the callstack is when it crashes, but I'd assume it's inside vkDestroySwapchainKHR as you saw on dolphin.
You could report these issues to the IHVs but I'm not sure this is something they expect to work so I don't know if they will be interested in fixing those crashes. If the vulkan driver reports VK_ERROR_DEVICE_LOST and shuts down cleanly then RenderDoc should handle this correctly and I'll fix any issue I can reproduce where it doesn't, but the device lost handling is only as good as the driver's.


Adding Integritee solochain

Changes:

Deleting subscan for Integritee Shell (they have only for standalone network)
Add new network Integritee standalone
Add new types for Integritee standalone

⚠ Need change color ⚠

✅ Color was changed ✅


[bug]

Describe the bug
Unable to upload media. Receiving the error "Segments do not add up to provided total file size." I'm trying to post a GIF that's 9.9MB in size. The code fails on upload media.
To Reproduce
async function postTweet() {
  const client = new TwitterApi({
    appKey: process.env.API_KEY,
    appSecret: process.env.API_SECRET_KEY,
    accessToken: process.env.ACCESS_TOKEN,
    accessSecret: process.env.ACCESS_TOKEN_SECRET
  })

  const mediaId = await client.v1.uploadMedia('mygif.gif')

  const { data: createdTweet } = await client.v2.tweet('Another day in paradise!', { media: {media_ids: [mediaId]}})
}

Version

Node.js version

v16.13.2

Lib version

1.12.2

OS (especially if you use Windows)

Ubuntu

Can you upload somewhere the Gif file you're using to test it on my side?

I found the issue - for some reason, the GIF wasn't complete on the filesystem yet, although the package I was using to generate it was emitting the file descriptor being closed. Waiting 3 seconds made this issue disappear 🤷🏻 .


Fix Errors caused by Updates #261

What?
Pylint wants to punish us for too many positional arguments. I don't think we need this linter.
I also ran Black to add/delete some spaces.
I also monkey patched historic_crypto to use the new Coinbase interface.

Test results are strange: some platforms fully succeeded, while others still break.

Test results are strange: some platforms fully succeeded, while others still break.

Yeah, I'm not sure what is going on. I was going to switch python version locally and see.

@eprbell I guess we just had to jiggle the lock. I think these errors are being caused by the call to coinbase. I'll try to put a VCR tape together so they aren't flakey anymore.

@eprbell I guess we just had to jiggle the lock. I think these errors are being caused by the call to coinbase. I'll try to put a VCR tape together so they aren't flakey anymore.
EDIT: Actually, this is much more complex since the test runs the program from the command line. Is there way to run it directly in Python?

I think we have to understand why the tests are failing only occasionally (I'm not sure which lock you're referring to: can you clarify?). If the problem is indeed Coinbase one quick workaround would be to pass to the tests a config file that uses one of the CCXT-based converters instead of the CB-based one. But the question remains if the Coinbase pair converter is still reliable or not (if not we could try to fix it or remove it altogether).

@eprbell jiggle the lock -> keep trying it until it works basically. If you keep re-running the failed tests, they finally go through.
The problem we have is if the historic_crypto plugin fails (or for the matter any pair converter plugin fails) the resulting error message is ambiguous and hard to troubleshoot because dali-rp2 simply exits with a 1 exit code. Because of that, the test ini/ods files aren't generated and that results in the tests failing.
If you can trigger the generation of the files in pure Python, it would be much more easier to troubleshoot.
I personally feel though that none of our tests should rely on an external API being up and available. It just complicates things that don't need to be complicated. If we want to test if the APIs are still available, these tests should be isolated and done with curl or something similar. It just makes things more performant and reliable. I worry about new developers encountering this error and having no idea how to fix it or what is causing it.
For this particular test, we just want to know that the report creation is working. We aren't (and shouldn't be) testing the API call.
Thinking on this for a bit, I think what is happening here is that the new Coinbase API is rate limiting IP addresses. When the test runs on Github, 12 or so tests are run simultaneously. All of them hit the Coinbase API at approximately the same time and trigger a rate limit. We can increase the re-tries of course, but it just adds more time to the test. It may also end up triggering a ban in the future. I really don't think it is a good idea.
Because of that, even if we use the CCXT Coinbase plugin, we will still have the same issue.
If we can't fake the response. We can build a dummy pair converter that generates the same price or a simple CSV pair converter that uses CSV files to generate the prices. The CSV plugin may be more useful for users. Either way, I really don't think we should rely on a connection.

@eprbell It looks like that eliminated the flakey tests. I had to convert the two Trezor wallets to manual input. The ODS files were edited to eliminate the note about where the price was pulled, since it is no longer getting pulled. Also, there was a small accuracy error that was caused by the old method.
This now doesn't require the API call and makes the test a little faster. I combine the trezor wallets into one CSV. I didn't see a reason to have them separated for the manual plugin.

@eprbell It looks like that eliminated the flakey tests. I had to convert the two Trezor wallets to manual input. The ODS files were edited to eliminate the note about where the price was pulled, since it is no longer getting pulled. Also, there was a small accuracy error that was caused by the old method.
This now doesn't require the API call and makes the test a little faster. I combine the trezor wallets into one CSV. I didn't see a reason to have them separated for the manual plugin.

Sounds good: I am currently traveling, but I should be able to review this in the weekend.

@eprbell According to #263, It seems like pre-2023 data is not available via the new API endpoint. It seems like we will have to eventually implement a coingecko or coinmarketcap backup pair converter eventually. It is just incredibly difficult to keep up with these APIs. I'd like a backup of some kind.

@eprbell Is there something else you need for this?


[INS-1693] Add WS echo server for smoke tests

Closes INS-1693
This PR adds a simple WS echo server for smoke tests into our existing express server.
It's available behind ws://localhost:4010 and any other dummy path, e.g. you can start a connection to ws://localhost:4010/echo and it will still work.

Confirming the echo with text works as expected with the most recent commit.

I have an approval and I can see that we have a preexisitng break on the websockets feature branch (e.g. https://github.com/Kong/insomnia/runs/7734126701) that this is breaking on, so I'm going to merge since this has been reviewed and functionally tested and approved.

@dimitropoulos this PR will fix it. https://github.com/Kong/insomnia/pull/5064


Optimized adding large numbers of items to the world.

These are more optimizations based on the idea of avoiding recalculating the world bounds until necessary with massive boasts to the performance of adding large numbers of items. (My tests showed a 3/4 reduction of the time it takes to add 2000 items). First, it sets autoRefigureSizes to false to cancel the world bounds calculation in world.addItem. Second, it only calls world.arrange at the end of each loadQueue.
I see that this could cause a loss of information about the items being arranged, but the only existing option that would be affected is collectionImmediately, and only if different values are set on items within the same loadQueue. I don't know if it was meant to support mixing up animating and snapping images into position because it didn't appear to work even before my changes, but if this seems like a thing we should support, I can look into ways to track that information.

Actually, I think having the last item in the queue's collectionImmediately control the animation vs. snapping is great.

@iangilman I added the call to setAutoRefigureSizes back to true.

Thanks! Looking good :)


Add command/whitelist service

Processes the commands from PRs/issues. Runs the CI/whitelisting if necessary.

Can I start working on this?

Is it still relevant?

@lukasburda I wouldn't mind splitting the tyr-core functionality into two modules. tyr-core would be just PR checks and a new tyr-ci module would be for whitelisting. Do you want to try this?

@xstefank ok i am going for it!

@lukasburda it's a good idea but don't create a new module, you can expose the whitelisting in the tyr-webhook for now. If there will be a requirement for a CLI then we will create a new module.


having issues running macro control

whenever i run startup.bat it gives me this
and i dont know how to fix it

You must enable intents for the bot.

in the Discord developer portal select bot then turn on all of the intents


Get-Date and .AddSeconds() inconsistent with UTC-local time conversion

This issue is copied from https://github.com/MicrosoftDocs/PowerShell-Docs/issues/4127.
Originally submitted by @rwlangeveld.

Issue Details
Converting from UTC to local time with DST (being CEST in my case) seems to give unexpected results.
I ran into this issue while converting Unix (epoch) time to a PowerShell/.Net datetime object. I take
00:00:00 AM, January 1, 1970 (UTC) as a starting point, and add the given Unix time (in seconds) to it. Then I want to translate that to the correct local time.
In winter, this works fine in multiple ways. In summer, using DST, this seems not to give me the expected results in most ways. Only one way, I got this to work. Which at least seems to be very inconsistent...
Examples
NB. I translated the day- and month names in the examples from Dutch to English manually.
I start with specifying the begin date as '01-01-1970 00:00:00Z', with 'Z' denoting this as being UTC.
When adding only one second, and converting to local time, all seems find.
Since January 1 is in the winter, no DST is involved in local time (CET = UTC+1). UTC does not have DST.
PS H:> (Get-Date '01-01-1970 00:00:00Z').AddSeconds(1).TolocalTime()
Thursdag 1 January 1970 01:00:01
PS H:> (Get-Date '01-01-1970 00:00:00Z').TolocalTime().AddSeconds(1)
Thursdag 1 January 1970 01:00:01
When not specifying UTC. The time region is unspecified, therefore should be assumed to be UTC, when translating to local time, I expect. This seems to be true:
PS H:> (Get-Date '01-01-1970 00:00:00').AddSeconds(1).TolocalTime()
Thursdag 1 January 1970 01:00:01
PS H:> (Get-Date '01-01-1970 00:00:00').TolocalTime().AddSeconds(1)
Thursdag 1 January 1970 01:00:01
When adding enough seconds to enter DST in local time (CEST = UTC+2), however, things get a little unexpected...
I add exactly 181 days (181d * 24h/d * 60m/h * 60s/m = 15638400 seconds).
In UTC I expect to be at midnight again. But since this new date is in the summer, in local time, I expect to be 2 hours later.. (Since local time is CEST now, being CET+DST, being UTC+2.)
PS H:> (Get-Date '01-01-1970 00:00:00Z').AddSeconds(15638400).TolocalTime()
Wednesday 1 July 1970 01:00:00
Seems to be a bug to me...! Converting UTC to CEST here, should be +2 hours!
PS H:> (Get-Date '01-01-1970 00:00:00Z').TolocalTime().AddSeconds(15638400)
Wednesday 1 July 1970 01:00:00
Seems to be a bug to me...! Converting UTC to CET here, should be +1 hour. However, adding enough time to enter CEST (CET+DST) this should have been corrected by +1 hour...
PS H:> (Get-Date '01-01-1970 00:00:00').AddSeconds(15638400).TolocalTime()
Wednesday 1 July 1970 02:00:00
CORRECT!
PS H:> (Get-Date '01-01-1970 00:00:00').TolocalTime().AddSeconds(15638400)
Wednesday 1 July 1970 01:00:00
Wrong again...
This seems to be a bug in the (lack of) implementation of DST conversion in time calculations to me.
Any help, feedback or bugfix would be appreciated.
Thank you in advance.
Rogier Langeveld
Version(s) of document impacted

[?] Impacts 6.next document
[?] Impacts 6 document
[X] Impacts 5.1 document
[?] Impacts 5.0 document
[?] Impacts 4.0 document
[?] Impacts 3.0 document

Reason(s) for not selecting all version of documents

[ ] The documented feature was introduced in selected version of PowerShell
[ ] This issue only shows up in selected version of the document

Some more results...:
PS H:> (Get-Date '01-01-1970 00:00:00Z')
donderdag 1 januari 1970 01:00:00
as expected. Automatically converted to local time, being in CET = UTC+1
PS H:> (Get-Date '01-01-1970 00:00:00Z').ToLocalTime()
donderdag 1 januari 1970 01:00:00
as expected. Automatically converted to local time, being in CEST = UTC+1, manually converted to local time, staying the same
PS H:> (Get-Date '01-07-1970 00:00:00Z')
woensdag 1 juli 1970 02:00:00
exactly 181 days after '01-01-1970 00:00:00Z'
as expected. Automatically converted to local time, being in CEST = CET+DST = UTC+2
PS H:> (Get-Date '01-07-1970 00:00:00Z').ToLocalTime()
woensdag 1 juli 1970 02:00:00
exactly 181 days after '01-01-1970 00:00:00Z'
as expected. Automatically converted to local time, being in CEST = CET+DST = UTC+2, manually converted to local time, staying the same
PS H:> (Get-Date '01-01-1970 00:00:00Z').AddSeconds(15638400)
woensdag 1 juli 1970 01:00:00
NOT as expected. Being exactly 181 days after '01-01-1970 00:00:00Z'_, but in CEST now. Should be as above.
PS H:> (Get-Date '01-01-1970 00:00:00Z').AddSeconds(15638400).ToLocalTime()
woensdag 1 juli 1970 01:00:00
NOT as expected. Being exactly 181 days after '01-01-1970 00:00:00Z'_, but in CEST now. Should be as above.
(15638400 seconds = 181 days)
Just to be sure...:
PS H:> Get-TimeZone
Id                         : W. Europe Standard Time
DisplayName                : (UTC+01:00) Amsterdam, Berlin, Bern, Rome, Stockholm, Vienna
StandardName               : W. Europe Standard Time
DaylightName               : W. Europe Summer Time
BaseUtcOffset              : 01:00:00
SupportsDaylightSavingTime : True

Looking into this, a problem is that being in UTC+1 leads to deceptive examples (I'm running in UTC-8).
These actual behaviours are in .NET rather than in PowerShell.
> ([datetime]'01-01-2018 00:00:00')

Monday, January 1, 2018 12:00:00 AM

> ([datetime]'01-01-2018 00:00:00').AddSeconds(15638400)

Sunday, July 1, 2018 12:00:00 AM

> ([datetime]'01-01-2018 00:00:00').AddSeconds(15638400).ToLocalTime()

Saturday, June 30, 2018 5:00:00 PM

The part to watch here is the Kind:
> (Get-Date '01-01-2018 00:00:00').Kind
Unspecified

> ([datetime]'01-01-2018 00:00:00').Kind
Unspecified

> (Get-Date '01-01-2018 00:00:00Z').Kind
Local

> ([datetime]'01-01-2018 00:00:00Z').Kind
Local

It looks like a DateTime value will retain its kind and will behave and be displayed appropriately. Unspecified seems to behave the same as Utc, but I think the distinction exists to record that the timezone was not explicitly given.
This means that when a Local time is not DST, it will not be converted to DST when time is added. It keeps its original timezone (where the DST timezone is treated as distinct from the standard one). But when ToLocalTime() is used, it converts the time to the appropriate local timezone that applies at the time of the datetime value:
> ([datetime]'01-01-2018 00:00:00')

Monday, January 1, 2018 12:00:00 AM

> ([datetime]'01-01-2018 00:00:00').ToLocalTime()

Sunday, December 31, 2017 4:00:00 PM

> ([datetime]'01-01-2018 00:00:00').AddSeconds(15638400)
Sunday, July 1, 2018 12:00:00 AM

> ([datetime]'01-01-2018 00:00:00').AddSeconds(15638400).ToLocalTime()

Saturday, June 30, 2018 5:00:00 PM

> ([datetime]'01-01-2018 00:00:00Z')

Sunday, December 31, 2017 4:00:00 PM

> ([datetime]'01-01-2018 00:00:00Z').ToLocalTime()

Sunday, December 31, 2017 4:00:00 PM

> ([datetime]'01-01-2018 00:00:00Z').AddSeconds(15638400)

Saturday, June 30, 2018 4:00:00 PM

> ([datetime]'01-01-2018 00:00:00Z').AddSeconds(15638400).ToLocalTime()

Saturday, June 30, 2018 4:00:00 PM

In any event, the actual semantics of System.DateTime are determined by .NET -- we can't really fix them.

But there's not great documentation of this stuff in the .NET docs. May with worth opening an issue with them.


feat(merkle tree): Allow random-order tree recovery

What ❔
Allow recovery the Merkle tree from entries provided in an arbitrary order.
Why ❔
This is necessary to implement the snapshot recovery PoC and could be beneficial in the long run.
Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[x] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.
[x] Code has been formatted via zk fmt and zk lint.

@RomanBrodetski @tomg10 Judging by local benchmarking (recovering 10M logs in 50 chunks of 200k entries), random-order recovery is indeed approximately 3x slower than in-order recovery:

In-order recovery took 48.0s (approx. 1s / chunk)
Random-order recovery took 137.3s (2.7s / chunk), with additional time spent mainly on loading data from RocksDB (1.1s / chunk; this stage is practically absent in in-order recovery) and additional time saving it to RocksDB (0.8s / chunk vs 0.2s / chunk in in-order recovery).

nice. Also well done estimating the performance impact - very close to 3x indeed!


Content: [Support] Compilation of support content

We currently have a lot of Q&As in our Slack channel that, if compiled and categorized, could give us a lot of information about what our adopters are doing and what they need.
There are two potential wins for this compiled information:

highlight areas of the website that need work, or where information is buried and hard to find
provide Q&As for website FAQs or a potential future conversational bot

We don't have a way to download this data, so it's going to be a bit of a manual process. And we'd need to work out the best way to capture the information and the parameters of interest.
Here is a spreadsheet created in Carbon for pulling content out of Slack. I'm including for reference only of potential things to track because the data graphs on the second and third sheets were really helpful in a lot of different ways.
https://app.zenhub.com/files/202376156/36e01306-4bc7-4e1c-b31b-eae27a20c35e/download
Not sure if Airtable or Excel would be best. Let's try going back 6 months and seeing what the data looks like.

Link to spreadhseet on box :)
https://ibm.box.com/s/i1qleng3emswv9ic9zciu57jigomxfk9


Typo in quiz section

Hello Googlers, hope your day is going well :)
I am not sure where to report this error, so will try here.  I found an typo in quiz section for the unit:

Which of the following are parameters of the AndroidView Composable? Select all that apply.
Choose as many answers as you see fit.

[ ] factory
[ ] inputLabel
[ ] upcate <----
[ ] modifier

https://developer.android.com/courses/quizzes/android-basics-compose-unit-8-pathway-2/android-basics-compose-unit-8-pathway-2?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-8-pathway-2%23quiz-%2Fcourses%2Fquizzes%2Fandroid-basics-compose-unit-8-pathway-2%2Fandroid-basics-compose-unit-8-pathway-2
Have a good day!

Hi @RaczkaKamil
Thank you for letting us know of this issue.
It has been fixed and it is already live in production.
Hope you are enjoying the course!
Regards.


Remove webapp and other application references: django, cgi, and spec.

Description
Removes webapps and other helpfull files for applications using pdfminer.six. These files should be maintained as part of the application itself, not in pdfminer.six.
Fixes #314
Fixes #105
How Has This Been Tested?
The tests all pass, but I did not verify if anyone is actually using this. I sincerely hope that nobody is dependent on these files to run there application. But if so, these files can be copied easily from github and placed in a proper location.
Checklist

[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the README.md and other documentation, or I am sure that this is not necessary
[x] I have added a consice human-readable description of the change to CHANGELOG.md
[ ] I have added docstrings to newly created methods and classes
[ ] I have optimized the code at least one time after creating the initial version

@pietermarsman
May I ask why these files were removed? Having these as a template to work from would have saved many hours.
Thank you for all the work you've put into this project.

Hi @CounterPly,
Because in my opinion good software is focused on solving one thing well. E.g. like the unix philosophy. Or the single-responsibility principle. Pdfminer.six is for extracting text and other content from PDF files. And not for setting up a webserver. There are other tools use can use for that.
Let me know if you have any specific questions on how to integrate pdfminer.six in a webserver.