pipdeptree --freeze fails with AttributeError exception (no attribute 'editable')

Hello! I came across this error in my typical usage pattern, where --freeze is now failing. Maybe the internal pip API changed?
Console output:
$ python3 -m pipdeptree --freeze
Warning!!! Possibly conflicting dependencies found:
* launchpadlib==1.10.6
 - testresources [required: Any, installed: ?]
------------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/usr/local/lib/python3.6/dist-packages/pipdeptree.py", line 878, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.6/dist-packages/pipdeptree.py", line 872, in main
    render_text(tree, args.all, args.freeze)
  File "/usr/local/lib/python3.6/dist-packages/pipdeptree.py", line 510, in render_text
    lines = flatten([aux(p) for p in nodes])
  File "/usr/local/lib/python3.6/dist-packages/pipdeptree.py", line 510, in <listcomp>
    lines = flatten([aux(p) for p in nodes])
  File "/usr/local/lib/python3.6/dist-packages/pipdeptree.py", line 498, in aux
    node_str = node.render(parent, frozen)
  File "/usr/local/lib/python3.6/dist-packages/pipdeptree.py", line 107, in render
    return self.render_as_root(frozen)
  File "/usr/local/lib/python3.6/dist-packages/pipdeptree.py", line 141, in render_as_root
    return self.__class__.frozen_repr(self._obj)
  File "/usr/local/lib/python3.6/dist-packages/pipdeptree.py", line 113, in frozen_repr
    fr = frozen_req_from_dist(obj)
  File "/usr/local/lib/python3.6/dist-packages/pipdeptree.py", line 81, in frozen_req_from_dist
    return FrozenRequirement.from_dist(dist)
  File "/usr/local/lib/python3.6/dist-packages/pip/_internal/operations/freeze.py", line 257, in from_dist
    req, editable, comments = _get_editable_info(dist)
  File "/usr/local/lib/python3.6/dist-packages/pip/_internal/operations/freeze.py", line 165, in _get_editable_info
    if not dist.editable:
  File "/usr/local/lib/python3.6/dist-packages/pip/_vendor/pkg_resources/__init__.py", line 2816, in __getattr__
    return getattr(self._provider, attr)
AttributeError: 'PathMetadata' object has no attribute 'editable'

Versions:
Lubuntu 18.04.5 (and separate system on Ubuntu 18.04.5)
Python 3.6.9
pip 21.2.1
pipdeptree 2.0.0
In case it is needed, here is how to get there from a fresh system:
sudo apt-get install python3-pip
sudo pip3 install --upgrade pip
sudo pip3 install --upgrade virtualenv pipdeptree
pip3 install --user numpy # or more or less any package
python3 -m pipdeptree --freeze # error output above

Having this problem too
  File "/home/liezl/.local/lib/python3.7/site-packages/pipdeptree.py", line
510, in <listcomp>
    lines = flatten([aux(p) for p in nodes])
  File "/home/liezl/.local/lib/python3.7/site-packages/pipdeptree.py", line
498, in aux
    node_str = node.render(parent, frozen)
  File "/home/liezl/.local/lib/python3.7/site-packages/pipdeptree.py", line
107, in render
    return self.render_as_root(frozen)
  File "/home/liezl/.local/lib/python3.7/site-packages/pipdeptree.py", line
141, in render_as_root
    return self.__class__.frozen_repr(self._obj)
  File "/home/liezl/.local/lib/python3.7/site-packages/pipdeptree.py", line
113, in frozen_repr
    fr = frozen_req_from_dist(obj)
  File "/home/liezl/.local/lib/python3.7/site-packages/pipdeptree.py", line
81, in frozen_req_from_dist
    return FrozenRequirement.from_dist(dist)
  File "/home/liezl/.local/lib/python3.7/site-packages/pip/_internal/operati
ons/freeze.py", line 257, in from_dist
    req, editable, comments = _get_editable_info(dist)
  File "/home/liezl/.local/lib/python3.7/site-packages/pip/_internal/operati
ons/freeze.py", line 165, in _get_editable_info
    if not dist.editable:
  File "/home/liezl/.local/lib/python3.7/site-packages/pip/_vendor/pkg_resou
rces/__init__.py", line 2816, in __getattr__
    return getattr(self._provider, attr)
AttributeError: 'FileMetadata' object has no attribute 'editable'

Running just pipdeptree works fine though

Thanks for reporting. Yes, it's because pip's internal API change. I've identified the fix, will push it shortly.

Released version 2.1.0 to PyPI - https://pypi.org/project/pipdeptree/2.1.0/


test results in idea test runner should show tests in the same order they have in the file

I'm pretty sure this works correctly and there is a test for that


Error when running Terraspace on modules using explicitly passed-on providers

Checklist

[X] Upgrade Terraspace: Are you using the latest version of Terraspace? This allows Terraspace to fix issues fast. There's an Upgrading Guide: https://terraspace.cloud/docs/misc/upgrading/
[X] Reproducibility: Are you reporting a bug others will be able to reproduce and not asking a question. If you're unsure or want to ask a question, do so on https://community.boltops.com
[X] Code sample: Have you put together a code sample to reproduce the issue and make it available?  Code samples help speed up fixes dramatically. If it's an easily reproducible issue, then code samples are not needed. If you're unsure, please include a code sample.

My Environment

Software
Version

Operating System
Ubuntu 21.10

Terraform
1.0.11

Terraspace
0.6.18

Ruby
2.7.4p191

Problem Description
For certain tasks we need to do things in a different AWS region (e.g. ACM, KMS etc) than the one the main infrastructure is in.
For instance: If one wants to use an ACM certificate with CloudFront, it needs to be present in the us-east-1 region rather then the region the other account resources are in.
In vanilla Terraform (which also works with Terragrunt) we solved this by adding a secondary AWS Provider within the "global" region:
# Primary provider for provisioning main infra
provider "aws" {
  region = "eu-west-1"

  assume_role {
      role_arn = "arn:aws:iam:XXXXXXXXXXXXX:role/ProvisionerRole"
   }
}

# Secondary provider for ACM / Route53
provider "aws" {
  region = "us-east-1"
  alias = "aws_global"

  assume_role {
      role_arn = "arn:aws:iam:XXXXXXXXXXXXX:role/ProvisionerRole"
   }
}

The module (called by the stack) passes along the appropriate provider like so:
module "certificate" {
  source = "git::https://github.com/cloudposse/terraform-aws-acm-request-certificate.git?ref=0.15.1"
  providers = {
    aws = aws.aws_global
  }

  zone_name = "example.com"
  domain_name = "app.example.com"
}

This approach works fine with vanilla Terraform and with Terragrunt. However, doing so with TerraSpace results in receiving the following error:
│ Error: missing provider provider["registry.terraform.io/hashicorp/aws"].aws_global

Step-by-step reproduction instructions

See above

Code Sample

See above

Unsure. Tried to reproduce it with this terraspace example project that

creates security groups
with different provider aliases
on different AWS accounts
using different roles to assume

Here's the repo: https://github.com/tongueroo/terraspace-multiple-providers
Here's also the debugging output:

Click to Expand and See Debugging Session
$ terraspace up demo -y
Building .terraspace-cache/us-west-2/dev/stacks/demo
Built in .terraspace-cache/us-west-2/dev/stacks/demo
Current directory: .terraspace-cache/us-west-2/dev/stacks/demo
=> terraform plan -input=false -out /tmp/terraspace/plans/demo-5620d45a4deda68126b91aff6dc4e562.plan

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:
  # random_pet.pet1 will be created
  + resource "random_pet" "pet1" {
      + id        = (known after apply)
      + length    = 2
      + separator = "-"
    }
  # random_pet.pet2 will be created
  + resource "random_pet" "pet2" {
      + id        = (known after apply)
      + length    = 2
      + separator = "-"
    }
  # module.sg1.aws_security_group.this will be created
  + resource "aws_security_group" "this" {
      + arn                    = (known after apply)
      + description            = (known after apply)
      + egress                 = (known after apply)
      + id                     = (known after apply)
      + ingress                = (known after apply)
      + name                   = (known after apply)
      + name_prefix            = (known after apply)
      + owner_id               = (known after apply)
      + revoke_rules_on_delete = false
      + tags_all               = (known after apply)
      + vpc_id                 = (known after apply)
    }
  # module.sg2.aws_security_group.this will be created
  + resource "aws_security_group" "this" {
      + arn                    = (known after apply)
      + description            = (known after apply)
      + egress                 = (known after apply)
      + id                     = (known after apply)
      + ingress                = (known after apply)
      + name                   = (known after apply)
      + name_prefix            = (known after apply)
      + owner_id               = (known after apply)
      + revoke_rules_on_delete = false
      + tags_all               = (known after apply)
      + vpc_id                 = (known after apply)
    }

Plan: 4 to add, 0 to change, 0 to destroy.

Changes to Outputs:
  + sg1_arn = (known after apply)
  + sg2_arn = (known after apply)

─────────────────────────────────────────────────────────────────────────────

Saved the plan to: /tmp/terraspace/plans/demo-5620d45a4deda68126b91aff6dc4e562.plan

To perform exactly these actions, run the following command to apply:
    terraform apply "/tmp/terraspace/plans/demo-5620d45a4deda68126b91aff6dc4e562.plan"
=> terraform apply -auto-approve -input=false /tmp/terraspace/plans/demo-5620d45a4deda68126b91aff6dc4e562.plan
random_pet.pet1: Creating...
random_pet.pet2: Creating...
random_pet.pet1: Creation complete after 0s [id=notable-lobster]
random_pet.pet2: Creation complete after 0s [id=sensible-seal]
module.sg1.aws_security_group.this: Creating...
module.sg2.aws_security_group.this: Creating...
module.sg1.aws_security_group.this: Creation complete after 1s [id=sg-063c8fb1ab95ec2fe]
module.sg2.aws_security_group.this: Creation complete after 3s [id=sg-04e5c4ed437502e2a]

Apply complete! Resources: 4 added, 0 changed, 0 destroyed.

Outputs:

sg1_arn = "arn:aws:ec2:us-west-2:111111111111:security-group/sg-063c8fb1ab95ec2fe"
sg2_arn = "arn:aws:ec2:us-east-1:222222222222:security-group/sg-04e5c4ed437502e2a"
Time took: 7s
$

It just deploys 2 security groups. Each one using a different provider and different region. It outputs the security ARN so can easily debug and confirm that they are being created in different regions/accounts.
    Outputs:

    sg1_arn = "arn:aws:ec2:us-west-2:111111111111:security-group/sg-063c8fb1ab95ec2fe"
    sg2_arn = "arn:aws:ec2:us-east-1:222222222222:security-group/sg-04e5c4ed437502e2a"

Confirmed that it does.
You've provided a lot of info, but still unable to reproduce. Maybe an example repo would help. Closing this out. Feel free to provide more info though if you find more clues.
Also, consider trying:
terraspace build demo
cd .terraspace-cache/us-west-2/dev/stacks/demo
terraform init
terraform apply

That's pretty much what terraspace does. https://terraspace.cloud/docs/intro/how-terraspace-works/
So if you can get it working with plain terraform, it should in theory work with terraspace.
Note: The .terraspace-cache folder will use whatever your AWS_REGION is set to. Or if that is not set, it'll try to get it from your ~/.aws/config, and then default to us-east-1. Generally speaking, that's what happens. Here's the lookup logic: https://github.com/boltops-tools/aws_data/blob/master/lib/aws_data.rb


Move away from ad-hoc module injection

Consider compiling using require.js. This would require that we alter testing format, as well. We want to maintain modular tests of private and public components.

Moved to requirejs


remove not working settings

Can you please provide a description why this fix is needed?

And please acknowlege, that this repository is under the Unlicense  License, therefore please state the CLA mentioned here.
This is no garantee, that your contribution will be allowed, but it is a prerequisite for approval

bc. ist not needed
I dedicate any and all copyright interest in this software to the
public domain. I make this dedication for the benefit of the public at
large and to the detriment of my heirs and successors. I intend this
dedication to be an overt act of relinquishment in perpetuity of all
present and future rights to this software under copyright law.

Thanks for the acknowledgement of the CLA.

bc. it's not needed

Why should I accept this PR, if the changes are not needed?

LaTeX is a document preparation system that is widely used in the academic and scientific community to create documents that require advanced formatting and mathematical equations. VS Code is a popular code editor that is used by developers for software development. The LaTeX VS Code setting includes the "LaTeX Workshop" extension, which provides a wide range of features for working with LaTeX documents, such as syntax highlighting, code completion, and built-in support for PDF viewers. However, the "latex.recipes" setting provided by the extension is not needed, as it cannot be used by the backend.
The "latex.recipes" setting is a configuration file that contains pre-defined build configurations for LaTeX documents. It allows users to easily customize the build process for their documents by specifying various options, such as the compiler, output format, and command-line arguments. However, this setting cannot be used by the backend, which is responsible for actually building the LaTeX document. The backend uses a different configuration file, known as "latexmkrc", to configure the build process.
The "latexmkrc" file is a configuration file that is used by the LaTeX backend to configure the build process. It allows users to specify various options, such as the compiler, output format, and command-line arguments, in a way that is compatible with the backend. The backend automatically reads the "latexmkrc" file and uses it to build the LaTeX document. The "latex.recipes" setting, on the other hand, cannot be used by the backend, as it is not compatible with the backend's configuration file format.
Furthermore, the "latex.recipes" setting may not always work for every document. LaTeX documents can have complex requirements, and the pre-defined configurations provided by the "latex.recipes" setting may not always meet those requirements. In such cases, it may be necessary to manually customize the build process using the "latexmkrc" file. This requires a deeper understanding of the LaTeX build process, which may not be available to all users.
In conclusion, the "latex.recipes" setting provided by the "LaTeX Workshop" extension is not needed, as it cannot be used by the backend to build LaTeX documents. The backend uses a different configuration file, known as "latexmkrc", to configure the build process. Moreover, the pre-defined configurations provided by the "latex.recipes" setting may not always meet the requirements of every LaTeX document, which may require a deeper understanding of the LaTeX build process to manually customize the build process. Therefore, the "latex.recipes" setting can be safely ignored by most users, as it does not provide any additional benefits over the "latexmkrc" file.

"ltex.recepies" : [
   ...
]

is needed to define the recepies used by the LaTeX Workshop extension.

The change is in a configuration file for the "LaTeX Workshop" extension in VS Code. Specifically, it is an example of a build recipe that can be used to build a LaTeX document using a series of tools, including pdflatex, bibtex, and pdflatex again.
However, the reason why this configuration is not needed is that it does not work. There are several issues with this configuration that prevent it from being usable.
Firstly, the configuration is missing important information about the input and output files. Without this information, it is impossible for the build process to know which files to operate on and what the expected output should be. This is a fundamental requirement for any build recipe, and the absence of this information makes the configuration invalid.
Secondly, the order of the tools in the configuration is incorrect. The standard build process for a LaTeX document involves running pdflatex first, followed by bibtex, and then pdflatex again (twice). This ensures that all necessary references and citations are properly resolved before the final PDF is generated. However, the configuration provided in the prompt runs bibtex before pdflatex, which is incorrect and will not work.
Finally, even if the configuration were fixed to include the necessary input and output information and to use the correct order of tools, it would still not work as expected. The reason for this is that the configuration assumes a very specific structure for the LaTeX document, including the location and naming of the bibliography file. If the document structure differs from what the configuration expects, the build process will fail.
In conclusion, the configuration provided in the prompt is not needed because it does not work. It is missing important information, uses the incorrect order of tools, and assumes a specific document structure that may not be valid for all LaTeX documents. As a result, it cannot be used to build a LaTeX document reliably and should be avoided. Instead, users should rely on the standard build process for LaTeX documents or customize the build process using a valid build recipe that includes all necessary information and follows the correct order of tools.

@minerop5000
I took a look at the code, since xour arguments seem inconsistent and generated. In the future of your open-source contributions please make sure to not waste anyones  time, and be sure to provide a short and simple explenation.
In the case of your contribution it could be somthing like:

I removed the bibtex tool and all recipies containing it, because the citation backend is biber, as stated in the readme.


Issues unpacking lo.tar.br

Hi @vladgolubev,
I have been trying to use this package instead, but continue to have trouble. I saw that abdulbarik was able to get it working so I assume the issue is on my end. The function seems to hang on the getExecutablePath() function, and specifically the last line where it's piping the decompressStream. I've pasted my lambda function below, do you mind seeing if it's something in my implementation?
`const {writeFileSync, readFileSync} = require('fs');
const {execSync} = require('child_process');
const {parse} = require('path');
const {S3} = require('aws-sdk');
const fs = require('fs');
const s3 = new S3({params: {Bucket: 'convert-doc-to-pdf'}});
exports.handler = async ({filename}) => {
console.log(filename);
const {Body: inputFileBuffer} = await s3.getObject({Key: filename}).promise();
writeFileSync(/tmp/${filename}, inputFileBuffer);
fs.readdirSync('/tmp').forEach(file => {
console.log(file);
});
const {getExecutablePath, defaultArgs} = require('aws-lambda-libreoffice');
const loBinary = await getExecutablePath(); // /tmp/instdir/program/soffice
execSync(${loBinary} ${defaultArgs.join(' ')} --convert-to pdf /tmp/${filename} --outdir /tmp)
const outputFilename = ${parse(filename).name}.pdf;
const outputFileBuffer = readFileSync(/tmp/${outputFilename});
await s3
.upload({
Key: outputFilename, Body: outputFileBuffer,
ACL: 'public-read', ContentType: 'application/pdf'
})
.promise();
return https://s3.amazonaws.com/convert-doc-to-pdf/${outputFilename};
};`

Hi @jonogan!
Can you please paste the output of Lambda? Especially what this code prints?
fs.readdirSync('/tmp').forEach(file => {
	console.log(file);
});

@vladgolubev I think I figured it out. I had created the lambda in the console instead of the CLI, and hadn't allocated enough memory to the function. When I upped it to > 300mb the function runs great! I did have a follow up. Should I move the call to getExecutablePath() outside of the main lambda async? Just wondering about calling it everytime the lambda runs vs just on the initial coldstart. Any advice there?
Thanks for the terrific work! This module is great!

@jonogan glad to hear that! You better keep getExecutablePath() inside the handler. It checks if an archive is unpacked and doesn't impact performance on subsequent invocations. In case /tmp folder was wiped out due to a cold start or by accident - it will unpack archive again and cache for future use.

Awesome! Thanks for the help! Closing this issue


MODE-2423 Moved to ISPN 7.2.0.CR1

Looks good. Mostly configuration changes (which is to be expected since that changed dramatically), and very limited code changes.


Remove reference to "Scottish Control (SCO_CTR)" on VT Guide

Remove reference to "Scottish Control (SCO_CTR) on the vt-guide (/resources/views/site/community/vt-guide.blade.php)

#1823


useBlockLayout.md - 404

The "See Column Options" link redirects to a 404 on the website. Provide two links, one for the website and one for GitHub.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix(mongoose,json-mapper): Add methods toClass to deserialize mongoose object to a proper instance.

Closes: #1141

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

v6.14.4
GitHub release

Your semantic-release bot :package::rocket:


⚠ AGEG website (prod) has degraded performance

In dd47705, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 1794 ms

Resolved: AGEG website (prod) performance has improved in 32f4a21.


QWhiteSurGtkDecorations init at 0.1.0

Things done
i need help testing this i think it works, im new to this

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

#363783

By the ways, you can use closes #363783 to close the issue when finish.

Base on CONTRIBUTING.md requirement,
you should use git rebase to squash the different commit on a same topic into one, and forced push it back to your repos.

When adding yourself as maintainer in the same pull request,
make a separate commit with the message maintainers: add <user>
Add the commit before those making changes to the package or module.
See Nixpkgs Maintainers for details.


Fix: fix highlightNode xxx in xxx syntax to revert back to simple dot notation for checking of existence

Connects https://github.com/graphql/graphiql/issues/1565
fixes highlightNode xxx in xxx syntax to revert back to simple dot notation for checking of existence

If someone could please advise on the best way to fix the ts linting errors in this PR that would be appreciated:
packages/graphql-language-service-interface/src/getDiagnostics.ts(123,40): error TS2339: Property 'name' does not exist on type 'FieldDefinitionNode | InterfaceTypeDefinitionNode | UnionTypeDefinitionNode | ObjectTypeDefinitionNode | ... 37 more ... | InputObjectTypeExtensionNode'.
  Property 'name' does not exist on type 'NameNode'.
packages/graphql-language-service-interface/src/getDiagnostics.ts(124,16): error TS2339: Property 'name' does not exist on type 'FieldDefinitionNode | InterfaceTypeDefinitionNode | UnionTypeDefinitionNode | ObjectTypeDefinitionNode | ... 37 more ... | InputObjectTypeExtensionNode'.
  Property 'name' does not exist on type 'NameNode'.
packages/graphql-language-service-interface/src/getDiagnostics.ts(125,16): error TS2339: Property 'variable' does not exist on type 'ASTNode'.
  Property 'variable' does not exist on type 'FieldDefinitionNode'.
packages/graphql-language-service-interface/src/getDiagnostics.ts(126,16): error TS2339: Property 'variable' does not exist on type 'ASTNode'.
  Property 'variable' does not exist on type 'FieldDefinitionNode'.
packages/graphql-language-service-interface/src/getDiagnostics.ts(123,40): error TS2339: Property 'name' does not exist on type 'FieldDefinitionNode | InterfaceTypeDefinitionNode | UnionTypeDefinitionNode | ObjectTypeDefinitionNode | ... 37 more ... | InputObjectTypeExtensionNode'.

in is required for this for typescript, or a better inference than just dot notation
is there a reason you want to get rid of 'in' operator?

also, @tnrich it seems these are not eslint errors, they are typescript compiler errors.

@acao hmmmm it looks like someone added a check 4 months ago that causes this issue to no longer show up on master, however it still is an issue on 0.17.5:

if (highlightNode) {

that check makes this no longer get hit.
With that if-statement commented out, I do see the error in the test I wrote:

I think that the current code is better but still has issues because the node that should be highlighted is now just ignored.
I've updated this PR changing the code slightly and adding a test to make sure that errors where node.name = undefined will still show up.

@acao let me know what else I can do to get this in. Is there anything else in this PR that needs clarification? Thanks!

there's a big comment block so I assumed it was unfinished. can you clean that up if this PR is ready for review?

thanks for all your help on this @tnrich! LGTM!


add persistDefaultValue and npm audit fix

This change enables users to update/persist the default data as it changes over time. Currently the only way to achieve this is manually deleting the item from the S3 bucket.

@nicekiwi || @pharindoko - Could you let me know if this PR is sufficient or requires any changes?

@iandvt  hmm I checked your code but don`t get the point.
All I see here is that you refactored the code piece into a new function...
https://github.com/nicekiwi/lowdb-adapter-aws-s3/blob/666b4129fdd349e5f90063ce43cc8a2d68f79aa4/src/main.js#L64

@nicekiwi do you still maintain this repository ?

@iandvt hmm I checked your code but don`t get the point.
All I see here is that you refactored the code piece into a new function...
https://github.com/nicekiwi/lowdb-adapter-aws-s3/blob/666b4129fdd349e5f90063ce43cc8a2d68f79aa4/src/main.js#L64

@pharindoko - This PR enables users to override the data that resides in S3.

@iandvt hmm I checked your code but don`t get the point.
All I see here is that you refactored the code piece into a new function...
https://github.com/nicekiwi/lowdb-adapter-aws-s3/blob/666b4129fdd349e5f90063ce43cc8a2d68f79aa4/src/main.js#L64

@pharindoko - This PR enables users to override the data that resides in S3.

How does it do that though? The plugin itself updates data in S3, this just seems to break that into a new function with no added feature or benefit.

@nicekiwi - It will enable me to call await persistDefaultValue before my server starts which will override the data inside of S3.

@nicekiwi  && @pharindoko - Would y'all like me to remove node@4 from Travis with this PR? I believe it would allow you to close out #4.

@nicekiwi && @pharindoko - Would you like me to remove node@4 from Travis with this PR? I believe it would allow you to close out #4.

@pharindoko - PR updated

I fully understand now what you want to achieve but the main issue here is that lowdb itself does not support such a method.
there are only these methods:
read, write, setstate, getstate
The question that comes up in my mind. Why you don't write yourself a module that supports your specific method ?
And another thing - would it not be the same (persistdefaultvalue) as the setstate method lowdb already provides ?


Improve CI diagnosability

I was looking at enabling CI for full-framework builds in xplat and noticed again that we don't reliably fail the script on intermediate step failures.  This has caused some confusion in the past and it's clearly just a bad idea.

Looks like this will continue to fail until the update to netci.groovy is pushed.  Does it look ok, @cdmihai?

Yup, It should work :)
OSX and linux just produce "msbuild.log" for now, but that will get captured by "msbuild*log"


[Table] Multiple footers/headers with different data

Bug, feature request, or proposal:
Feature Request
What is the expected behavior?
To be able to make a second footer/header in which I can use the same columns but different data for the cells then the other footer/header
What is the current behavior?
Currently you can have multiple mat-footer-rows on a table but it has to have matColumnDefs for each one. IE let's say I have a second footer row that I want to go into the same columns that are setup but with different data. The only way I can see to do this is to make a new array of columns, send that into that footerDef and make the matColumnDefs for those.
What are the steps to reproduce?
https://stackblitz.com/edit/angular-tcdbaj
What is the use-case or motivation for changing an existing behavior?
The use case is to have multiple footers or headers that have different data, about that column
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 6.0.9, Newest Material, etc
Is there anything else we should know?

We also need that, why there is no reaction ?
And why everything is always half baked.
There comes footers but no ability to pass data.
Why this is not working:
<ng-container *ngFor="let footerRow of footerRows">

     </ng-container>
         {{element[i]}}

I need that

I need that too!

I have the same problem

Has anyone found a workaround?

Facing the same issue

Also having this issue

I need this to add multiple footer with having different number of columns

One ugly, but working solution I found, is to define multiple cells dynamically as shown in the Example below:
<ng-container *ngFor="let footerEntry of footerData; let footerEntryIndex = index">
    <ng-container matColumnDef="column1-{{footerEntryIndex}}">
        <mat-footer-cell *matFooterCellDef>
            {{ footerEntry.footerEntryProp}}
        </mat-footer-cell>
    </ng-container>

    <ng-container matColumnDef="column2-{{footerEntryIndex}}">
        <mat-footer-cell *matFooterCellDef>
             {{ footerEntry.footerEntryProp2}}
        </mat-footer-cell>
    </ng-container>
</ng-container>

You can then render all of your footer data as shown in this snippet:
<ng-container *ngFor="let footerEntry of footerData; let footerEntryIndex = index">
      <mat-footer-row *matFooterRowDef="['column1-'+footerEntryIndex, 'column2-'+footerEntryIndex]; sticky: true"  ></mat-footer-row>
</ng-container>

hope that someone can use this workaround.

I had this same need (multiple footer rows), and solved it similar to the above comment. I modified the Angular Material example table footer code to show this in use, in case it helps anyone: https://stackblitz.com/edit/angular-ztyat1?file=src/app/table-footer-row-example.html

Related to https://github.com/angular/components/pull/11245
https://github.com/angular/components/issues/5997


Installation errors: 'invalid syntax'

Hi!
I just found your project and tried installing nvim-gtk3. However I'm getting the following errors when I try to do the installation method provided in the Readme.md:
running install
running build
running build_py
running install_lib
byte-compiling /home/tuxflo/.local/lib/python3.5/site-packages/nvim_pygtk3/application.py to application.cpython-35.pyc
  File "home/tuxflo/.local/lib/python3.5/site-packages/nvim_pygtk3/application.py", line 12
    application_id=f'org.{name}',
                               ^
SyntaxError: invalid syntax

byte-compiling /home/tuxflo/.local/lib/python3.5/site-packages/nvim_pygtk3/window.py to window.cpython-35.pyc
  File "home/tuxflo/.local/lib/python3.5/site-packages/nvim_pygtk3/window.py", line 237
    ['nvim', f'+set rtp^={rtp}', *argv],
                              ^
SyntaxError: invalid syntax

running install_data
running install_egg_info
running egg_info
writing nvim_pygtk3.egg-info/PKG-INFO
writing dependency_links to nvim_pygtk3.egg-info/dependency_links.txt
writing top-level names to nvim_pygtk3.egg-info/top_level.txt
writing requirements to nvim_pygtk3.egg-info/requires.txt
writing entry points to nvim_pygtk3.egg-info/entry_points.txt
reading manifest file 'nvim_pygtk3.egg-info/SOURCES.txt'
writing manifest file 'nvim_pygtk3.egg-info/SOURCES.txt'
removing '/home/tuxflo/.local/lib/python3.5/site-packages/nvim_pygtk3-0.3.4.egg-info' (and everything under it)
Copying nvim_pygtk3.egg-info to /home/tuxflo/.local/lib/python3.5/site-packages/nvim_pygtk3-0.3.4.egg-info
running install_scripts
Installing nvim-pygtk3 script to /home/tuxflo/.local/bin

Any ideas what to do about that?
Thanks for helping,
Flo

Hi,
It seems that the f{} syntax is only supported in python 3.6+.
I'll be sure to mention that in the Readme.


natively load datasets with Arrow.jl

https://arrow.juliadata.org/stable/

some code showing how to read/write color arrays from/to arrow tables
https://gist.github.com/CarloLucibello/51d713ec4a1612b46e6c90e53c0f88e8

this doesn't seem feasible since datasets can contain python preprocessing scripts


SCSS language services appear to have issues

on SCSS files autocompletion doesnt works !
body {
b..
}
not propose border or background

Hi @mediaclap, can you give us more information about this problem you're facing? Were you the guest or the host in this session? Also, your logs may provide us with helpful information. You can access them via the Live Share: Export Logs command from the command palette (ctrl + shift + p) and attach them to this issue.

i'm the guest on the host no problem ! we works at 3 guest and 1 host ! no probleme with css files on guest but scss impossible to have autocompletion on { }. outside { } its ok !
vscode.zip

We’re not able to prioritize this issue over the other higher-impact issues we receive every week, based on the votes and comments from others in the community and our understanding of the issue. However, rest assured that we love your input. If you feel it deserves to stay open, then clarify your use case and contact us to let us know how severe it is for you.


docs: add badges

Signed-off-by: “seriouszyx”<EMAIL_ADDRESS>Fix: https://github.com/casdoor/casdoor-java-sdk/issues/8

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ef5ca2e). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master     #12   +/-   ##
========================================
  Coverage          ?   0.00%
========================================
  Files             ?      15
  Lines             ?     207
  Branches          ?      11
========================================
  Hits              ?       0
  Misses            ?     207
  Partials          ?       0

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 ef5ca2e...d70157e. Read the comment docs.

@hsluoyz Codebeat and Codecov access request for Casdoor Organization has been sent, plz approve.

@seriouszyx approved.


Publish tracker results

Currently the tracker does not publish results in messages. Add that.

Working on this now

I added this feature in commit b75a598fd
All it odes for now is publish the coordinates of the bounding box in the format (minX, minY, maxX, maxY). There are no values for confidence level or things like that. Not yet at least
The results are published in topic visptracker_data using message type TrackerData. These should probably be changed once we get into the integration phase, but it's a start.


Standardize ProviderConfig for Cluster and Machine

What this PR does / why we need it:
In order to maintain type parity for providerConfig across both Machine
and Cluster types, change the ClusterSpec.ProviderConfig to the
ProviderConfig type.
Additionally, as all changes to the API require regenerating code, add a
Makefile that will simplify that process.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

None

@kubernetes/kube-deploy-reviewers

/ok-to-test

/lgtm


Reduce Duplication in Like Kernels

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

The like kernels contain a significant amount of copy-pasted code with very minor modifications for the various like, ilike, nlike, nilike variants. It would be good to reduce this
Describe the solution you'd like

The existing like_scalar_op should allow for significant reduction in the code duplication
Describe alternatives you've considered

Additional context

I can pick up some of these.

That would be awesome, thank you :+1:

Good idea 👍. I remember those kernels were non-trivial to change before and are in need of some good cleanup.


[New game]: JavaScript-based Quiz Game

🎮 Game Request
The game is based on answering some questions, which have 4 options for each question.
It is made using HTML, CSS, and JS!
Point down the features
The user can start the game and at the end view their Score as well!
https://github.com/kunjgit/GameZone/assets/100849696/7c5e4983-0a0c-4ac6-94fb-e63384027c4b
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @sabhisharma-ise , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#1713

Hey @sabhisharma-ise ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @sabhisharma-ise!
We are already having a similar game request in #981 👀
Make sure you come up with a cool unique idea 😀
Waiting for your new game idea 💗.


Model not supported: H61BA

What issue do you have? Please be as thorough and explicit as possible.
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?
Homebridge UI-X

Which version of Homebridge/HOOBS do you have?
v1.6.1 (Homebridge A8E0)

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.
Synology DSM; v18.12.0

Which version of this plugin (homebridge-govee) do you have? Has the issue started since upgrading from a previous version?
homebridge-govee v9.4.2; It works normally with my other LED Light strips, but This model just won't add on to my HomeKit.

Which Govee devices do you have that are causing issues? Please include product models if applicable.
H61BA

Please paste any relevant logs below.

{"device":"7D:C8:C6:38:32:34:48:54","deviceName":"TV Strip","model":"H61BA","httpInfo":{"groupId":0,"device":"7D:C8:C6:38:32:34:48:54","sku":"H61BA","spec":"","versionHard":"3.05.00","versionSoft":"3.00.10","deviceName":"TV Strip","deviceExt":{"deviceSettings":"{\"wifiName\":\"The duck goes quack\",\"address\":\"C6:38:32:34:48:54\",\"bleName\":\"Govee_H61BA_4854\",\"topic\":\"GD/0c96388da4c5a54408ecc08ff3406e9b\",\"wifiMac\":\"60:74:F4:35:64:88\",\"pactType\":1,\"pactCode\":1,\"wifiSoftVersion\":\"\",\"wifiHardVersion\":\"\",\"ic\":20,\"secretCode\":\"TDfGODI0SFQ=\",\"boilWaterCompletedNotiOnOff\":1,\"completionNotiOnOff\":1,\"autoShutDownOnOff\":1,\"sku\":\"H61BA\",\"device\":\"7D:C8:C6:38:32:34:48:54\",\"deviceName\":\"TV Strip\",\"versionHard\":\"3.05.00\",\"versionSoft\":\"3.00.10\",\"playState\":false}","lastDeviceData":"{\"online\":false}","extResources":"{\"skuUrl\":\"https://d1f2504ijhdyjw.cloudfront.net/sku-img/5c8525ab5509e3346cf8daacbfbc58ee-add_list_type_device_61ba.png\",\"headOnImg\":\"https://d1f2504ijhdyjw.cloudfront.net/sku-img/07c10ede03d01d3b8dca963414a16d64-h61ba_light_title_on.png\",\"headOffImg\":\"https://d1f2504ijhdyjw.cloudfront.net/sku-img/bba0c5d45c2f1c73dfbc2929d2bc5bb3-h61ba_light_title_off.png\",\"ext\":\"\",\"ic\":20}"},"pactType":1,"pactCode":1,"share":0,"goodsType":178}}

Hi @samthelamb
Please update to the beta version of the plugin:
https://github.com/homebridge/homebridge/wiki/How-to-Install-Alternate-Plugin-Versions

@bwp91 That worked! Thank you :)


Deprecate network version match - develop

Change Description

nodeos option network-version-match when set to non-default true, needlessly restricted connections between two nodeos. The option requires both nodeos to have the exact same peer network version even if the versions are compatible. Since the next major release will bump the network version but still support and work seamlessly with the older network version, this option is being deprecated as enabling it provides no benefit.

Consensus Changes

[ ] Consensus Changes

API Changes

[ ] API Changes

Documentation Additions

[x] Documentation Additions
Note deprecation and planned removal of network-version-match.

Should we log a warning when someone uses the depreciated option? Like we do for private-key.

I started to add that this morning, but didn't know if we wanted it or not.

It would at least make it consistent with the other depreciated option. And it would probably hasten its deactivation by users, or at least hasten the visibility of the depreciation. Seems like a good thing to me.


as i am using this library to extract code property graph for my code files  in windows 10 but this tutorial is very confusing . would any of you elaborate it insimple steps so that i can use it in my research project as its very very urgent i request if any of you would be in touch with me to let me understand how to simply use this library

Stack trace
Stack trace in case of an Exception
Unexpected behavior
When the analysis terminates exception free but leads to unexpected graphs: Describe the difference between the generated graph and what you would have expected
Code of interest
codeOfInterest

Extensions to the library, e.g. added or changed LanguageFrontends or Passes
changes to or added components
Changes to the configuration
configuration print

I guess you are referent to the tutorial.md right? We actually have a small video (https://www.youtube.com/watch?v=ngKtH6kYxx0) that goes through the steps of the tutorial. Problematic might be that we do not really support windows at this moment. Most of our developers use either Linux or Mac. It might work on windows, but we cannot really guarantee it.
Unfortunately, the overall documentation "how to use" this library is definitely something we need to improve.

thanks for providing this tutorial  to use cpg-console library
https://www.youtube.com/watch?v=ngKtH6kYxx0
i cloned the whole cpg directory
but when i am trying to do ../gradlew installDist its showing this error .

The Go frontend is now optional and should not be included by default if you clone the recent main branch.


update: Reset lastScanResult on resumeScanning

I want to scan several barcodes in a row including two same barcodes in a row. But the plugin prevents to scan twice the same barcode.
Removing this condition creates more problems than anything else, returns the value many times and there is no notion of scan frequency to remedy this.
So for my need I decide to pause the scan and by user interaction to resume it. This way the scan is always open and I can scan several barcodes including several times the same one and the user controls what he scans

This is a function that i would really appreaciate . Why this is not merged? Its mandatory for many i guess.

It's not merged right now because I need to check the behaviour on iOS once I merge this PR so the two implementation behave in the same way and I simply have not found time to do so.
I will have a look at it this weekend!

@PierreB40 how are you preventing the scanner from closing after the first scan? I can't seem to find anything in the APIs. And it is auto destroyed.

@silviogutierrez You can use startScanning() to scan multiple barcode in a row.
But if you want the capabilities to scan the same barcode twice you have to use pauseScanning() after scan and resumeScanning() to scan another (with an user interaction). Like this the lastScanned value will be reset

Oh interesting, I'll have to check it out. I see startScanning listed in the API, but no mention of the differences between the regular startScan.
There is one issue does mention it though: https://github.com/capacitor-community/barcode-scanner/pull/93
I'll try it out at some point and try to document it.


[Host.Outbox.Sql] Exception causes outbox polling to fail in an in infinite loop (unable to resolve MessageType)

When the MessageType fails to resolve, the OutboxSendingTask will fall into an infinite loop attempting to LockAndSelect the same messages, perpetually failing to publish/abort any of them.
This can occur when a message is added to the outbox but not published prior to the type registration being removed from the host application.
public async Task<IReadOnlyCollection<OutboxMessage>> LockAndSelect(string instanceId, int batchSize, bool tableLock, TimeSpan lockDuration, CancellationToken token)
{
        ...
        while (await reader.ReadAsync(token).ConfigureAwait(false))
        {
                ...
                BusName = reader.GetString(busNameOrdinal),
                MessageType = Settings.MessageTypeResolver.ToType(messageType) ?? throw new MessageBusException($"Outbox message with Id {id} - the MessageType {messageType} is not recognized. The type might have been renamed or moved namespaces."),
                MessagePayload = reader.GetSqlBinary(payloadOrdinal).Value,
                ...
        }
        ...
}

I believe this was addressed in #285. Closing.


Fixed #34618 -- Added icon descriptions in "Recent Actions" on the admin index.

Fixes https://code.djangoproject.com/ticket/34618

Hi, Just to avoid linting and formatting errors you can set up pre commit checks. See here for instructions.

@DevilsAutumn as far as I can see there are no pre-commit checks for CSS. Be great to add some?

But i can't see any linting errors related to CSS. I hope pre commit checks work in admin_views/tests.py. 😄

@nmenezes0 Do you have time to keep working on this? I only left reformatting suggestions.

@nmenezes0 Do you have time to keep working on this? I only left reformatting suggestions.

Yes, I have time to keep working on this today and will make the suggested changes.

@nmenezes0 Thanks :+1: Welcome aboard :boat: You're a Django contributor no. 2400 :partying_face:

buildbot, test on selenium.


Include idf components

Is there a way to import an existing C idf component into a project based on the esp-template? I want to use epdiy but I want to write Rust code. The docs in epdiy state that I could just include it in the components folder of another idf project (I assume this is a common way to include dependencies). Is there such a pattern for a cargo first rust esp project?

Seems like you are using Rust on top of esp-idf - in that case you are looking for https://github.com/esp-rs/esp-idf-template
This is the repository of the bare-metal template


Stacking multiple modals in parent element

When I want to open multiple modals, new ones are put before old ones in DOM. This way, if I have a class that gives for example z-index 1000 to my modals, as all modals share same z-index, the ones that appears before in the DOM are covered by its modal siblings.
Looking at the code, I'm seeing that it is using $animate.enter to append the modal to the parent. This method has the following signature:
function enter(element, parent, after, options)
Currently, modal is using always "parent" attribute of "enter" function. But in this way, angular is internally prepending element in parten, NOT appending.
Probably the solution would be to check if there are some other modals opened in current parent. If they exists, then give reference to last modal in parent to append new modal after.
Angular: 1.4
UIBS: 1.3.2
Bootstrap: 3

Possible solution, in modal.js line 2660:
var after = appendToElement.find('[uib-modal-window]:last');
appendToElement.addClass(modalBodyClass);
$animate.enter($compile(angularDomEl)(modal.scope), appendToElement, after.length > 0 ? after : null);

I have the same problem, when using "angular-bootstrap": "~2.0.0"
I was able to make it work by using the previous "angular-bootstrap": "~1.3.3" ..dunno if I'm missing a lot, but it works correctly that way.
My dependencies are...
{
    "angular": "~1.5",
    "angular-resource": "~1.5.8",
    "angular-sanitize": "~1.5.8",
    "angular-ui-router": "~0.2.0",
    "angular-bootstrap": "~1.3.3",
    "restangular": "~1.5.2",
    "satellizer": "~0.15.4",
    "ngstorage": "~0.3.10",
    "components-font-awesome": "~4.4.0",
    "bootswatch-dist": "united",
    "ngToast": "ngtoast#~2.0.0",
    "jquery": "~2.1.0"
}

I have the same error now after upgrading from 1.3.3

Also having this problem. Smells like a bug that appendTo is prepending.

I think its the z-index not properly set somewhere, will clone this to see

I solved the problem replacing lines 2661 and 2661 (vesion 1.3.2).
"after" parameter was available but never used.
Old lines:
appendToElement.addClass(modalBodyClass);
$animate.enter($compile(angularDomEl)(modal.scope), appendToElement);
New lines:
var after = appendToElement.find('[uib-modal-window]:last');
appendToElement.addClass(modalBodyClass);
$animate.enter($compile(angularDomEl)(modal.scope), appendToElement, after.length > 0 && after);
I don't have cloned this repo, so if someone want to pull this modification is welcomed to do it.
This solution worked OK for me.

if (modal.animation) {
angularDomEl.attr('modal-animation', 'true');
}
    modal.scope.index = topModalIndex; // ADDED To Patch index bug

    appendToElement.addClass(modalBodyClass);

    $animate.enter($compile(angularDomEl)(modal.scope), appendToElement);

Oh, somebody just fixed in a new build.

Closing as addressed by #6104

Issue has reappeared in latest.


feat:completed NF7 assets and README

Pushed my edits in 5b92a08be44442be2f3cce5135233a60b5feb8cd, and I’m just going to go ahead and merge this as we prep for the accelerator page release.
If anything looks off shoot me a message on Slack.


Error with runpy.py when executing rasa train

What problem are you trying to solve?
Hi, I encounter this error when executing rasa train
my envs :
Sys              : Windows10
Python        : 3.6
Rasa            : 2.0
Tensorflow  : 2.3.4
Error:
PS C:\Users\corma\PycharmProjects\pharmacy_chatbot> rasa train
Traceback (most recent call last):
File "C:\Users\corma\AppData\Local\Programs\Python\Python36\lib\runpy.py", line 193, in run_module_as_main
"main", mod_spec)
File "C:\Users\corma\AppData\Local\Programs\Python\Python36\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Users\corma\PycharmProjects\pharmacy_chatbot\venv\Scripts\rasa.exe_main.py", line 4, in
File "c:\users\corma\pycharmprojects\pharmacy_chatbot\venv\lib\site-packages\rasa_init.py", line 8, in
from rasa.run import run
File "c:\users\corma\pycharmprojects\pharmacy_chatbot\venv\lib\site-packages\rasa\run.py", line 6, in
from rasa.shared.utils.cli import print_warning
File "c:\users\corma\pycharmprojects\pharmacy_chatbot\venv\lib\site-packages\rasa\shared\utils\cli.py", line 2, in
from typing import Any, Text, NoReturn
ImportError: cannot import name 'NoReturn'
What's your suggested solution?
No response
Examples (if relevant)
No response
Is anything blocking this from being implemented? (if relevant)
No response
Definition of Done
No response

Thanks for submitting this feature request 🚀 @kalkbrennerei will get back to you about it soon! ✨

Hi @coffeebean27! Which python version are you using exactly? Seems like NoReturn was added in python 3.6.2 (link to python docs), so maybe you can update your python version?
Also with the rasaos 3.0 release, support for python 3.6 has been dropped. So if you want to upgrade to rasa 3.0 (this is why you should!) you also have to consider to update your python version
Hope this helps!! ✨


New DeprecationWarnings in cuda 12.4

A large number of DeprecationWarnings are generated when using "make" to build bifrost. Fortunately most of the warnings seem to limited to transpose.cu. See attached log of the make command.
makelog.txt

Looks like the main culprit is:
warning: ‘cudaError_t
   cudaDeviceSetSharedMemConfig(cudaSharedMemConfig)’ is deprecated [
   -Wdeprecated-declarations]

I found this on SO which indicates that there isn't a replacement and that it really doesn't matter.  If we are concerned about it we can wrap it with some kind of THIS_IS_OLD_CUDA macro.


Interactive main getting stuck

When loading this into repl https://github.com/puffnfresh/fp-course/blob/twitch/projects/TicTacToe/Main.hs and running main the repl seems to break and does not output anything:

Plugin was configured to use stack repl. Using stack repl it works fine:

If I click Interrupt current computation I get:

Process crashed while running request

stderr:
stdout:
...
...
...
request:
:{
main
:}
Hide Stack Trace
TypeError: Cannot read property 'stderr' of undefined
    at Object.promiseGenerator (C:\Users\freeman42x\.atom\packages\ide-haskell-repl\lib\interactive-process.js:13[0]():46)

I also want to use this oportunity to thank you for creating this extension. The reload on file save and re-run last command is a super useful feature which I have not seen in any other extension/repl's. Having syntax highlighting and autocomplete is also something rare or maybe even unique.

There's a bit of a trick to it, you need to explicitly include a newline in your input if your program expects a newline, e.g.
P1<enter><shift+enter> (<shift-enter> is "execute", i.e. send the input to the repl)
The reason it's like this is because of getChar etc. Not very intuitive, I admit, probably there's a better option.


chore: remove/ignore tests which download from a remote server

What was wrong?
We had tests which would fail due to missing data on a website, some tests still fail too due to this, but we want those tests so I set them to ignore
How was it fixed?
removing the tests or ignore them. Trin Execution will still exit on start with the errors if any era files are missing so we are good

Yeah, sounds good. I'll close #1546
I do think we still want to have the error include which index is missing, like this commit: 531b0e1

Sounds good I made the error look like Epoch indices are not starting from zero or not consecutive: missing epochs [50, 500] as it looks nice


Act like cron?

Hello, excellent piece of software, thanks.
I would like to know if mailreport can act like cron went we setup the time to run the task?
With cron we can schedule to run our task every minute, 3 minutes, 15 minutes etc etc.
This can let us track some behavior with my system.
U think would be a good feature?
Regards!!!

Thank you for the kind words, I'm glad it's helping the community.
pfMailReport can be incorporated into cron jobs. Simply run the commands via Services > Cron and output to a location like /tmp and subsequently cat the results into the Status > Email Reports. I have an example of this in commands/speedtest_full.sh
Of course the next logical step in this progression is tracking of data over time, which would lead us to solutions like databases or log rotation tools. I have instead utilized the built-in functions of the utilities which the example commands come from in order to track data over time. One example of this is commands/traffic_totals.sh where you can change the option flag of vnstat -d to -h to display traffic counts hourly rather than daily.
And the next step of tracking data over time is to visually display the data in a meaningful chart or graph. Which I have been working toward completing in the development branch of code in commands/traffic_totals_graph.sh along with other features such as rich email templates.
Please have a look into the development branch for more insights, and feel free to contribute back if you have success with scheduled jobs or graphing. We're always looking for new techniques.

Excellent work, I have test your reports and looks good.
The idea is to use cron+mailreport and we have a Swiss knife.
I will watch your progress and see how can I contribute.
Thanks again!!!


Support principal-match REPORT

Defined here:
http://tools.ietf.org/html/rfc3744#section-9.3
Low priority as nobody seems to need this.

Would be nice to have for 3.1


Can't access Key Vault with service principal having Client-Id and Certificate

Requirement:

Store secrets in key vault such as database credentials etc through java application and ARM scripts
Read secrets from Key vault into JAVA based spring boot application using microsoft provided library "com.microsoft.azure:azure-keyvault-secrets-spring-boot-starter:0.2.3"

Problem:
We are unable to find documentation to access Key Vault with service principal having Client-Id and Certificate.

Thanks for raising this feature request. I would like to highlight here that this is primarily for ADFS deployment of Azure Stack appliance.

Access keyvault with client-id and certificate is supported in pull request #474.


Orquesta - Make auth token available to tasks in workflow

SUMMARY
Let me use st2.actions.list or core.inject_trigger (or anything that needs access to the API token) from within an orquesta workflow
ISSUE TYPE

Bug Report

STACKSTORM VERSION
st2 2.9.1, on Python 2.7.5
OS / ENVIRONMENT / INSTALL METHOD
CentOS 7
Installed via ansible-st2
STEPS TO REPRODUCE
version: '1.0'
description: get actions list
tasks:
  actions_list:
    action: st2.actions.list

EXPECTED RESULTS
Return actions list.
ACTUAL RESULTS
traceback due to missing auth token
{
  "result": "None",
  "exit_code": 1,
  "stderr": "st2.actions.python.St2ActionsListAction: DEBUG    Calling client method \"get_all\" with kwargs \"{'limit': 10}\"
Traceback (most recent call last):
  File \"/opt/stackstorm/runners/python_runner/python_runner/python_action_wrapper.py\", line 320, in <module>
    obj.run()
  File \"/opt/stackstorm/runners/python_runner/python_runner/python_action_wrapper.py\", line 179, in run
    output = action.run(**self._parameters)
  File \"/opt/stackstorm/packs/st2/actions/actions_list.py\", line 27, in run
    'exclude_attributes': exclude
  File \"/opt/stackstorm/packs/st2/actions/lib/action.py\", line 102, in _run_client_method
    result = method(**method_kwargs)
  File \"/opt/stackstorm/st2/lib/python2.7/site-packages/st2client/models/core.py\", line 40, in decorate
    return func(*args, **kwargs)
  File \"/opt/stackstorm/st2/lib/python2.7/site-packages/st2client/models/core.py\", line 193, in get_all
    self.handle_error(response)
  File \"/opt/stackstorm/st2/lib/python2.7/site-packages/st2client/models/core.py\", line 166, in handle_error
    response.raise_for_status()
  File \"/opt/stackstorm/st2/lib/python2.7/site-packages/requests/models.py\", line 928, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 401 Client Error: Unauthorized
MESSAGE: Unauthorized - One of Token or API key required. for url: http://<IP_ADDRESS>:9101/v1/actions?limit=10
",
  "stdout": ""
}

Spelunking through the code, it looks like this is where the runner gets the auth token:
https://github.com/StackStorm/st2/blob/caadb0b1d166f64684c976a006dcfc39a7f4f99f/st2actions/st2actions/container/base.py#L91-L96
This is the beginning of _create_auth_token showing when it does not return a token:
https://github.com/StackStorm/st2/blob/caadb0b1d166f64684c976a006dcfc39a7f4f99f/st2actions/st2actions/container/base.py#L369-L375
I inserted a LOG line to see what context was getting passed in.
For the orquesta workflow itself, it does get the auth token, but tasks in that workflow do not. Ultimately the reason the orquesta tasks are not getting an auth_token appears to be because there is no user in the context.

user is missing in the parent context of the action executions in an orquesta workflow. The action service automatically applies the parent's user to the child task, but only if user is in the parent context.
Relevant snippets of code are embedded below.
The parent context (st2_ctx) is created (see code snippets below) in two places in 2.9.1 but only one place in master (following the with-items PR). To hack in a fix, I added the user parameter in both places in my 2.9.1 install:
st2_ctx = {
    'execution_id': wf_ex_db.action_execution,
    'user': wf_ex_db.context['st2']['user'],
}

This allows me to run st2.actions.list and similar actions that need a token.
I don't know if getting the user here is the right approach. I wonder if st2_ctx shouldn't be a copy of wf_ex_db.context['parent'] instead of grabbing just the user from wf_ex_db.context['st2']. I didn't submit a PR because I'm not sure what the logically correct way to fix this is. And, I'm not sure what the implications of this change are in the grand scheme of orquesta and ST2, so I'd rather someone else write the actual fix and the tests.
===========================
in handle_workflow_execution (2.9.1 only)
https://github.com/StackStorm/st2/blob/a2bf25e085dfc9d2d407e8160a2febd48e5a4920/st2actions/st2actions/workflows/workflows.py#L144-L145
in request_next_tasks (2.9.1 and master - this st2_ctx is parent in the ae_ex_ctx context below)
https://github.com/StackStorm/st2/blob/7c27c9e1ec224c7a7103423453225dc999a6f747/st2common/st2common/services/workflows.py#L816-L817
in request_task_execution
https://github.com/StackStorm/st2/blob/7c27c9e1ec224c7a7103423453225dc999a6f747/st2common/st2common/services/workflows.py#L462
in request_action_execution
https://github.com/StackStorm/st2/blob/7c27c9e1ec224c7a7103423453225dc999a6f747/st2common/st2common/services/workflows.py#L497-L508
liveaction actually gets the user from parent context if available
https://github.com/StackStorm/st2/blob/7c27c9e1ec224c7a7103423453225dc999a6f747/st2common/st2common/services/action.py#L76-L83

Fix at https://github.com/StackStorm/st2/pull/4421


Eliminated the need to define the validation database environment var…

Summary Change Description
Fixed the ability to run without validation DB environment variables.
Affected Issue Numbers

Resolves #385

Code Review Notes
It's straightforward/self explanatory.  I tested by commenting out the variables in my .env.
Checklist

[x] All issue requirements satisfied (or no linked issues)
[x] Linting passes.
[x] Migrations created & committed (or no model changes)
[ ] Tests implemented (or no code changes)
[x] All tests pass

I don't think 3 reviews are necessary, so I may merge after 1 or 2 approvals - whoever takes a look first.

The thinking behind the True default was that I would think we pretty much always want to give the validation interface to users.  The print is sort of a catch in case we forget to configure it.
But I get your gist, so I propose an alternate best-of-both worlds solution.  Let's not set a default either way for VALIDATION_ENABLED, so if it's not there, there's an exception.  If it's there and false, the other env vars are not required.  If it's there and true, you get the current error.

Actually, I think if VALIDATION_ENABLED is set to True it should fail to start unless the database can be configured. Using defaults for the database name, etc. is OK, but I think TraceBase should raise and exception and not startup if validation is enabled and cannot be started.

OK, sounds good.

I feel like a default of false is convenient for sandboxes, but the default of true has utility in production and on the dev site. It's helpful in that it allows you to notice a feature we pretty much always expect to be present is not set up. But having it required (not default) accomplishes the same thing.


ValueError: No fast pitch-shift ratios could be computed for the given sample rate and transpose range.

We are trying to build open wake word for 8khz, We have changed the sample frequency to 8000 in data.py, train.py and genrate_samples.py ( in piper-sample-generator folder)  but when we are running  following step
_# Step 2: Augment the generated clips
!{sys.executable} openwakeword/openwakeword/train.py --training_config my_model.yaml --augment_clips_
we are getting "ValueError: No fast pitch-shift ratios could be computed for the given sample rate and transpose range." error
For easy reference iam attching the script in which we have done modifications and also error screenshot

@manushaanabham unfortunately, the underlying models and training code does not support sample rates different from 16000 Hz.
However, this does not mean that you cannot use the models with 8000 Hz input audio. You simply need to upsample the audio to 16000 HZ prior to passing it to the openWakeWord models. If you are training custom models, training on upsampled data will also help ensure that the model has the best possible accuracy for native 8000 Hz audio.


Networking - HNS crashing repeatedly on newer builds

Fix is ready. Needs to go through servicing pipeline.

This has been serviced 2004 in the 9B update


Change name from LocalCommandlineCodeExecutor to LocalCommandLineCodeExecutor

Why are these changes needed?
Fixes a small naming issue

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
Attention: Patch coverage is 0% with 35 lines in your changes are missing coverage. Please review.

Project coverage is 13.14%. Comparing base (0a79512) to head (21adbd4).

Files
Patch %
Lines

autogen/coding/local_commandline_code_executor.py
0.00%
33 Missing :warning:

autogen/coding/factory.py
0.00%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1859       +/-   ##
===========================================
- Coverage   36.03%   13.14%   -22.89%
===========================================
  Files          63       63
  Lines        6618     6647       +29
  Branches     1459     1591      +132
===========================================
- Hits         2385      874     -1511
- Misses       4039     5752     +1713
+ Partials      194       21      -173

Flag
Coverage Δ

unittests
13.14% <0.00%> (-22.89%)
:arrow_down:

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

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

Just some context: we haven't documented this class anywhere yet. So it is a good time to make this type of changes.
@sonichi @qingyun-wu for awareness and final approval.

This change requires an openai test. Could you make the PR from the upstream repo?

Moved to ##1859


No auto-complete for single quote

In PHP files single quote has no auto-complete.

the same to double quote.

+1

please add this feature quickly, it is a basic feature.

+1

This works for me now.


adds PythonOpsResolver

This PR does the following:

Introduces PythonOpsResolver Target
removes a reference to AllOpsResolver in remaing places in TFLM repo outside of integration_tests

doc for changes
google3 cl

Please add the bug to the PR description.


Make MSAN work with C++ tests

Needs protobuf library to be compiled in each configuration

Fixed?

dupe of #1924


Define NoFloat version when -nofloat is passed; HardFloat otherwise

The idea with these changes is to make it easy to write code that depends on whether we're compiling with or without floating point support.
The idea is to standardize the three version identifiers, NoFloat, HardFloat, and SoftFloat. This is not just to support the -nofloat switch, but to allow compilers to just not support floating point at all on some architecture/platform configurations.
So, in the general case (not just for DMD):

NoFloat shall be defined if -nofloat is passed or the configuration being compiled for simply does not support floating point.
If NoFloat is defined, neither HardFloat nor SoftFloat shall be defined.
If the configuration being compiled for supports hardware-assisted floating point computations, HardFloat shall be defined. Otherwise, SoftFloat shall be defined (which indicates that software-assisted floating point is being used).
HardFloat and SoftFloat shall not be defined at the same time.

Some architectures may support some float precisions while not supporting others. I think versions like D_Half, D_Float, D_Double and D_Quadruple would be more appropriate. They would mean that particular precision is hardware supported. Otherwise float types should be emulated by a library.

Point taken. However, is this the case for any of the archs listed on http://www.d-p-l.org/version.html?

They do not support half and quadruple precision natively but some newer CPU's include half <-> single conversion instructions. If this means hardware support - I don't know. I think quadruples will be supported in the future, because they are included in the new IEEE standard (already supported on some System/370 platforms).

I don't know whether there are plans to have such high-precision floating point types in the language. Walter?

Are there really any x86 systems that don't support floating point instructions? The original impetus for -nofloat was to avoid linking in the fp formatting code. These days, the D generated executables are large enough that this seems irrelevant.

Honestly, I don't think so. Also, GDC ignores -nofloat. Maybe the option should be deprecated altogether?
I still think SoftFloat/HardFloat have uses, however.

Any input on this? Should I redo the patch to just add HardFloat/SoftFloat? (And add those to the docs)


Resolvable async .get() not complete for onExit hook

This is a (check one box):

[x] Bug Report
[ ] Feature Request
[ ] General Query

My version of UI-Router is:
"@uirouter/angularjs": "^1.0.20",
"@uirouter/core": "^5.0.21",
Bug Report
I'm getting a Resolvable async .get() not complete error when trying to obtain a resolved value in an onExit hook. This strikes me as odd, as the route has already been resolved, and the resolvables are all there. So why are they not available in an onExit hook?
E.g:
$transitions.onExit({exiting: 'foo'}, transition => {
    const user = transition.injector().get('user'); //errors
});

Expected Behavior:
To have all the resolvables of the state available in the onExit hook.
Link to Plunker or stackblitz that reproduces the issue:
http://plnkr.co/edit/KOClmTCu8ajm5BYP93uM
From playing around with it in Plunkr, it appears that the onExit hook is actually trying to get resolvables for the new state. How can I get the resolvables for the state I'm exiting?
I need access to them to do some cleanup.

I'll leave this open for feedback, but in the end I solved the issue by accessing the resolvables from the state object (2nd parameter in the hook).


Add workflow and output files

This PR adds a (hidden) workflow (.Rmd file not published on website) to add information about the regions each grid cell belongs to. This is done by adding three columns, isFlanders, isWallonia, isBrussels to the official EEA utm1_bel, utm10_bel and utm100_bel grids. The output is composed out of three Geopackage files in data/output.
This PR should solve https://github.com/inbo/alien-species-portal/issues/37

@SanderDevisscher this works perfectly at our side.

@damianooldoni is there a reason to maintain cells which are completly outside of belgium ? for example "100kmE37N30" ?

or

I didn't remove it in order to not apply any destructive change to the original EEA data. This data include a buffer as mentioned on their website:

The grids cover at least country borders - plus 15km buffer - and, where applicable, marine Exclusive Economic Zones v7.0 - plus 15km buffer - (www.vliz.be/vmdcdata/marbound). Note that the extent of the grid into the marine area does not reflect the extent of the territorial waters.

I added a column called isBelgium, similar to the other columns with regional information. Please, give a look and if ok, approve the PR


Add sodium_crypto_stream_xchacha20_xor_ic()

See https://github.com/php/php-src/pull/8276 for upstream

This will target v1.18.0 and is contingent on php-src adding the same function.


Don't update generated .h file if its not changed.

This changes logic of slang-generate to detect whether the newly generated .h file is different from the existing file, and update the existing file only when the actual content has changed. This helps prevent visual studio from repetitively rebuilding the slang project due to the header file being updated on every build.

It includes "secure_crt.h" which defines fopen_s for non MSVC compilers.


slack: 4.40.128 -> 4.40.133

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for slack is: Desktop client for Slack
meta.homepage for slack is: https://slack.com
meta.changelog for slack is: https://slack.com/release-notes
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.40.133 with grep in /nix/store/2dlb6i8j3i6zhppzpalm68hllmjam8fj-slack-4.40.133
found 4.40.133 in filename of file in /nix/store/2dlb6i8j3i6zhppzpalm68hllmjam8fj-slack-4.40.133

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

slack

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2dlb6i8j3i6zhppzpalm68hllmjam8fj-slack-4.40.133 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A slack https://github.com/r-ryantm/nixpkgs/archive/16d6fb9377bb5fd2d1bb1b42c7112e8219af91b7.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/16d6fb9377bb5fd2d1bb1b42c7112e8219af91b7#slack

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2dlb6i8j3i6zhppzpalm68hllmjam8fj-slack-4.40.133
ls -la /nix/store/2dlb6i8j3i6zhppzpalm68hllmjam8fj-slack-4.40.133/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

slack

Maintainer pings
cc @mmahut @teutat3s for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 351581 run on aarch64-linux 1


Update references to cred store

Part of https://github.com/alphagov/design-system-team-internal/issues/634

Thanks both!


cake5: convert CurrentConfigShell to command

Thx!
I tagged https://github.com/dereuromark/cakephp-setup/releases/tag/3.0.0-RC

There are many other shells which also need converting but this is just the first one

The remaining ones are not that important. I usually only use them for some DB migration topics.


Logging errors in imageselector dialog

Logging error
There's a repeated error from the script in InsertImage.cshtml whenever the image selection dialog is open.
It seems to originate from InsertImage.cshtml line 85.
var imageData = JSON.parse(event.data.split(";")[0]);
Throwing this error:
insertimage/?TINYMCE=true&TINYMCEEXTENDED=true:59 Uncaught TypeError: event.data.split is not a function     at listener (insertimage/?TINYMCE=true&TINYMCEEXTENDED=true:59:40)
Undefined log
InsertImage.cshtml line:122 - this.currentImage should be replaced with currentImage. This currently logs as undefined.
window.console && console.log(this.currentImage);

Thank you for your report! we will look at this and resolve it to next version.

Please verify the newest version https://nuget.optimizely.com/package/?id=Imageshop.Optimizely.Plugin&v=<IP_ADDRESS>

This issue still occurs.
The error:
insertimage/?TINYMCE=true&TINYMCEEXTENDED=true&CULTURE=en-us&IMAGESHOPLANGUAGE=no&image=undefined:128 Error data from Imageshop  TypeError: event.data.split is not a function     at listener (insertimage/?TINYMCE=true&TINYMCEEXTENDED=true&CULTURE=en-us&IMAGESHOPLANGUAGE=no&image=undefined:83:59)
It happens whenever the tinyMCE imageshop dialog is open and i havent selected an image.
Function listener seems to be triggered constantly before an image is selected.
Event.data is an object instead of a string and contains the following:

I believe a string-check should be enough to fix this issue

okey, that helps, angular devtools is some kind of devtool widget in your browser, okej, i will address this in next version.
you can temporary disable it to make it work. Not everyone has that installed.


Not respecting overridden ComputeThrottleKey method

I'm overriding the ComputeThrottleKey method to make a shorter key that follows our standard for keys, but the code seems to only use the core class's method.

Created pull request 105 to fix this issue.

Your changes have been published to nuget v1.5.3
Thanks


Allow mapArray to map to an empty array

This is a fix for https://github.com/Hearst-DD/ObjectMapper/issues/262.
In ObjectMapper 0.19 the array mapping behavior changed to map an
empty JSON array to nil. Previous versions mapped an empty JSON
array to an empty swift array.

Thanks @jdzak

:+1:

Great, thanks 💪


Enable Websockets broker by default

Codecov Report

Merging #854 (72c09e2) into master (e8f94f7) will increase coverage by 1.56%.
The diff coverage is 68.75%.

@@            Coverage Diff             @@
##           master     #854      +/-   ##
==========================================
+ Coverage   48.74%   50.30%   +1.56%
==========================================
  Files         142      145       +3
  Lines        7004     7498     +494
==========================================
+ Hits         3414     3772     +358
- Misses       2984     3063      +79
- Partials      606      663      +57

Impacted Files
Coverage Δ

pkg/api/server.go
74.41% <ø> (ø)

pkg/config/consts.go
0.00% <ø> (ø)

pkg/config/loader.go
58.22% <ø> (ø)

pkg/core/candidate/requestor.go
72.72% <ø> (ø)

pkg/core/candidate/validator.go
57.14% <ø> (ø)

pkg/core/chain/chain.go
41.76% <0.00%> (+0.72%)
:arrow_up:

pkg/core/chain/legacy.go
28.57% <ø> (+28.57%)
:arrow_up:

pkg/core/chain/loader.go
16.32% <ø> (ø)

pkg/core/chain/mock.go
8.33% <ø> (ø)

pkg/core/chain/sequencer.go
66.66% <ø> (ø)

... and 140 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #854 (72c09e2) into master (e8f94f7) will increase coverage by 1.56%.
The diff coverage is 68.75%.

@@            Coverage Diff             @@
##           master     #854      +/-   ##
==========================================
+ Coverage   48.74%   50.30%   +1.56%
==========================================
  Files         142      145       +3
  Lines        7004     7498     +494
==========================================
+ Hits         3414     3772     +358
- Misses       2984     3063      +79
- Partials      606      663      +57

Impacted Files
Coverage Δ

pkg/api/server.go
74.41% <ø> (ø)

pkg/config/consts.go
0.00% <ø> (ø)

pkg/config/loader.go
58.22% <ø> (ø)

pkg/core/candidate/requestor.go
72.72% <ø> (ø)

pkg/core/candidate/validator.go
57.14% <ø> (ø)

pkg/core/chain/chain.go
41.76% <0.00%> (+0.72%)
:arrow_up:

pkg/core/chain/legacy.go
28.57% <ø> (+28.57%)
:arrow_up:

pkg/core/chain/loader.go
16.32% <ø> (ø)

pkg/core/chain/mock.go
8.33% <ø> (ø)

pkg/core/chain/sequencer.go
66.66% <ø> (ø)

... and 140 more

Continue to review full report at Codecov.

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


Add Connection String Documentation

⚠ Is your feature request related to a problem? Please describe

I'm not an azure expert, so it's not clear how to go about getting a connection string.
💡 Describe the solution you'd like

Add some documentation or links to documentation to be able to properly configure the plugin.
🤚 Do you want to develop this feature yourself?

[ ] Yes
[X] No

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

@mandrachek thanks for trying to use the plugin. I will update the documentation in the morning.

Thanks for making it!  I eventually figured everything out and have it working.
I found the connectionString (Security + networking -> Access keys).  It suppose it would be nice if it was possible to use a shared access signature (SAS) instead of the storage account key.
Then I got stuck on illegal character errors for a bit.  Turns out packageName is a bit weird - it's not "com.mybiz.app" (what any java person thinks of as a package name) - It's actually a prefix for the container name.  So if you have "public" as the container, you have to set a packageName, say "x", and the container is actually "x-public" - there doesn't appear to be any way to use a bare container name.
But once I got past all that, it's working great!


Image missing in documentation for C++ Completion Queue

I found a bug in the documentation for Grpc Core at https://grpc.github.io/grpc/core/md_doc_core_grpc-cq.html. The image address is incorrect so the image is not displayed. It is currently https://grpc.github.io/grpc/images/grpc-cq.png.
Looking at the raw markdown article for that documentation at https://github.com/grpc/grpc/blob/master/doc/core/grpc-cq.md I see the image and the address used there is https://github.com/grpc/grpc/raw/master/doc/images/grpc-cq.png.

Ben, this looks like some sort of doxygen problem.  Can you take a look?

Yes, other projects are experiencing problems with relative image paths in markdown: doxygen/doxygen#6783.

I will play with the doxygen config on Monday. If I cannot get it to work, I can open a PR that converts all images to absolute paths while we wait for doxygen to address this issue.


”Zăpodeni” is listed in the drop-down menu

Description:
"Zăpodeni" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Zăpodeni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Zăpodeni, VASLUI".


Update UserGuide for v1.3

Codecov Report

Merging #65 (be1fb5e) into master (735b786) will increase coverage by 0.83%.
The diff coverage is n/a.

:exclamation: Current head be1fb5e differs from pull request most recent head fa72332. Consider uploading reports for the commit fa72332 to get more accurate results

@@             Coverage Diff              @@
##             master      #65      +/-   ##
============================================
+ Coverage     62.29%   63.12%   +0.83%
+ Complexity      427      426       -1
============================================
  Files            85       85
  Lines          1517     1489      -28
  Branches        175      169       -6
============================================
- Hits            945      940       -5
+ Misses          526      506      -20
+ Partials         46       43       -3

Impacted Files
Coverage Δ

...ava/eatwhere/foodguide/logic/parser/CliSyntax.java
85.71% <0.00%> (-1.79%)
:arrow_down:

...here/foodguide/logic/parser/FindCommandParser.java
100.00% <0.00%> (+47.61%)
:arrow_up:

...eatwhere/foodguide/logic/commands/FindCommand.java
100.00% <0.00%> (+52.00%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add benchmarks for high RTT environment

added jenkins job: https://grpc-testing.appspot.com/job/gRPC_performance_high_latency/. This uses two 2-core machines, one in us-central zone and other in asia-east zone - netperf latency appears about 150-160ms. Mostly for testing flow control perf.
see go results with different message sizes in https://gist.github.com/apolcyn/8519af7d3db3c5c6f5b08873cc5f11b6
Currently testing unary ping pong with equal request/response size, but probably could add some more.
also gets rid of async_server_threads=1 in go since it's ignored

btw a basic dashboard is in https://performanceedit-dot-grpc-testing.appspot.com/explore?dashboard=5659822271758336&container=1803582438&widget=1037642812
E.g., for go seeing:
-32K request response (within 64K window)  --- about 150ms, netperf
-2M request and response here -- about 10 seconds (about 60 round trips)

Looks good. Can I request you to put in place the equivalent tests for at least C++... and maybe Java if it would be useful to @ejona86?

latest updates add in c++/java equivalents to go scenarios added here.
Dashboard with results so far in https://performance-dot-grpc-testing.appspot.com/explore?dashboard=5659822271758336
Jenkins job in https://grpc-testing.appspot.com/job/gRPC_performance_high_latency/9/#showFailuresLink. cc @ejona86 java benchmarks seem to be crashing on 1GB scenarios.

cc @MakMukhi running this PR as the grpc root in https://grpc-testing.appspot.com/job/gRPC_performance_high_latency/, should now work for running go-unconstrained benchmarks with larger sizes

Assume this is stale


Add docs generation (e.g. sphinx)

Done in https://github.com/schlarpc/python-base-flake/commit/839032970e5cd3c38292b4cf98ed38253fd7c8a3


refactor all (squared) euclidean algorithms

Name based on the author of the algorithm, not the calculated result to be more specific

Codecov Report
Base: 82.88% // Head: 82.93% // Increases project coverage by +0.05% :tada:

Coverage data is based on head (f63788d) compared to base (6803b6f).
Patch coverage: 88.88% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
+ Coverage   82.88%   82.93%   +0.05%
==========================================
  Files           5        5
  Lines         333      334       +1
==========================================
+ Hits          276      277       +1
  Misses         57       57

Impacted Files
Coverage Δ

src/felzenszwalb.jl
77.19% <83.33%> (ø)

src/borgefors.jl
97.82% <100.00%> (ø)

src/maurer.jl
100.00% <100.00%> (ø)

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.


Enable no-floating-promises for mssql extension

No real issues found for this one either

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
26507

Relevant Lines:
57181

💛  - Coveralls


Doesn't work in Win10?

Doesn't seem to give any data in Win10, was this removed?

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

ID: 6566d5bd-df6f-e4dc-3075-ad47cc1fa1d5
Version Independent ID: 33407294-d33e-887a-9265-a91a753f2dee
Content: Get-StorageReliabilityCounter (storage)
Content Source: docset/windows/storage/get-storagereliabilitycounter.md
Product: w10
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

@officedocsbot unassign @e0i

@officedocsbot assign @yogkumgit

Hello @silversword411
Thanks for your feedback.
If you refer to this:
DeviceId Temperature ReadErrorsUncorrected Wear PowerOnHours
-------- ----------- --------------------- ---- ------------
0                                          0

Actually, if you run the example as in the documentation, it will only list by default those attributes, but if you add a | Format-List it will list all possible attributes, and some of them, with data. So I'll create a Pull Request to improve the example.
Please take note that, for some attributes to have data, it depends on the features provided by the storage hardware.
Thank you


add: qtile-git (Package Add)

Adding qtile to the list of supported packages.
qtile-git.pacscript has been added.
The packagelist is also updated with qtile-git.

Thanks, I will make the suggested changes.

I have made all the changes. Just one thing, pacstall is giving me a warning about the package maintenance, so do I need to add myself as the maintainer?

I have made all the changes. Just one thing, pacstall is giving me a warning about the package maintenance, so do I need to add myself as the maintainer? So, once I know this I will commit the changes.
Thanks

Yes add yourself as the maintainer.

I have made all the changes. Kindly review it.
Thanks

I have added the commands to the install() function as suggested.

Yeah, the qtile session shows up. I have made the changes. Please review them.
Thanks

Ok Thanks

Are there any other changes that I need to make?

Should I push these changes then?

CI is failing, but it works on my computer

I have tested this in multiple virtual machines

Finally after 83 conversations lets merge this! 🎉

Thank you guys, for all the support. This was my first open-source contribution, I learnt a lot.


chore: fix the sonarcloud configuration

This PR fixes an issue with the SonarCloud configuration, which couldn't be tested until #26 was merged.
It also fixes the version number of the deploy script, fix is preventing the website from working.

@Destiny-02 could you take another look please, this should work now


[feature] Multiple tokens of the same type

This issue is here for future reference
I had numerous issues while experimenting with this, but at the end it all boils down to that it wouldn't make so much sense aside some code-sharing (Using the same interceptor for everything). But it was a good chance to experiment with the dependency injection system of Angular and to see how far I can stretch it's abilities, even if it wouldn't make sense to have multiple instances of the same token, because you really should have only one identity on the application, and it's the back-end services job to authenticate against a common authentication service if you do use multiple of them.
First issue was that multiple tokens had to be configured. (Note that configuring multiple types of them is a different, much simpler issue as the types of them are finite, and different Modules can be created for them with their own interceptors) For this I experimented with multiple builder style approaches (https://github.com/AlexAegis/auth/tree/module-builder-experiment) but that's not supported under Ivy because of static analyzation of the forRoot method. Even after recent improvements of this in Angular 10 (https://github.com/angular/angular/pull/37126 and https://github.com/angular/angular/issues/36415). I would have to resort to helper methods to be able to let the user define unique generics per token configuration. And I'd have no way to enforce it.
Even multiple configuration could've been solved like this, if there would've been multiple configurations of the same type, I couldn't define separate defaults for them because where I would use those configuration they would all look the same. One way to solve it would be to have the user define a type which then can be paired with the defaults when it's used. But that's again, one extra step for the user to take.
Another issue with this would be that it wouldn't actually be true separation, because all of these types would've to be known to the core interceptor before defining them, and that would mean it's not truly extensible. The logic of the interceptor can be defined elsewhere to provide code sharing, but the interceptor itself couldn't be shared.
I tried having configurable services/interceptors using injectable configuration objects, but these injection tokens are all overriding the previous one in their scope. If modules had their own injectors this could've work but they don't. You could do this with lazy modules but that wouldn't make much sense here.


Precompute and store powers of omega inside basic_radix2_domain_aux.hpp

In basic_radix2_domain_aux.hpp, we are spending about 1/3 of the time to compute powers of omega. Most of the time, this function is called multiple times with the same omega, since we are resizing all the polynomials into the same size. We may consider refactoring all the code, to store the powers of omega somewhere, instead of computing them every time inside this function.

I verified with a very bad code that this speeds up prover by a factor of 1.5x.


Use SQLite instead of Postgres?

There is a lot of chatter about how great SQLite is.

https://thmsmlr.com/cheap-infra
https://fractaledmind.github.io/2024/04/15/sqlite-on-rails-the-how-and-why-of-optimal-performance/
https://github.com/benbjohnson/litestream
https://github.com/oldmoe/litestack

However:

https://www.sqlite.org/omitted.html
https://www.sqlite.org/quirks.html

I ran into https://github.com/jeremyevans/sequel/discussions/2176

Data can be migrated using sequel
$ sequel --copy-databases postgres://localhost/wikimum sqlite://storage/wiki.db
Databases connections successful
Migrations dumped successfully
Tables created
Begin copying data
Begin copying records for table: schema_info
Finished copying 1 records for table: schema_info
Begin copying records for table: users
Finished copying 1 records for table: users
Begin copying records for table: revisions
Finished copying 0 records for table: revisions
Begin copying records for table: pages
Finished copying 2 records for table: pages
Finished copying data
Begin creating indexes
Finished creating indexes
Begin adding foreign key constraints
Finished adding foreign key constraints
Database copy finished in 0.067332 seconds

No problem with production
$ sequel --copy-databases postgres://localhost/prod-wikimum sqlite://storage/prod-wiki.db
Databases connections successful
Migrations dumped successfully
Tables created
Begin copying data
Begin copying records for table: schema_info
Finished copying 1 records for table: schema_info
Begin copying records for table: users
Finished copying 145 records for table: users
Begin copying records for table: pages
Finished copying 452 records for table: pages
Begin copying records for table: revisions
Finished copying 4186 records for table: revisions
Finished copying data
Begin creating indexes
Finished creating indexes
Begin adding foreign key constraints
Finished adding foreign key constraints
Database copy finished in 0.422092 seconds

Some stats
$ ls -ahl starkast_wiki_backup_2024-06-15_215012.sql
-rw-r--r--@ 1 dentarg  staff    24M Jun 15 21:51 starkast_wiki_backup_2024-06-15_215012.sql

$ ls -ahl storage/prod-wiki.db
-rw-r--r--  1 dentarg  staff    49M Jun 15 21:53 storage/prod-wiki.db


fixes and enhancements

Description
This PR includes multiple fixes and features.
backends.py
restrict to one gpu per worker. I believe this is the normal case since multi-gpu running hurts the performance for the tasks supported in evaluator.
dask.py
allow multiple workers in DaskLocalCluster backend to share one gpu through cuda-mps to improve hardware utilization and computation throughput. GPUs are assigned using round-robin assignment.
reweighting.py
fixes computeExpectations for pymbar 4 API
forcefield.py
fixes TLeap/GAFF forcefield, include:

incorrect charges.txt format
incorrect indentation leading to incorrect molecule number in assign parameter step after build coordinates
incorrect quit command and error log parsing when running tleap
outdated compute_am1bcc_charges interface to openff-toolkit

have to revert the changes to dask backends due to random hang problem when running with cuda-mps enabled.


visual studio preview 3 MacOS error MSB4057: The target "Build" does not exist in the project

I have created a simple C# .net core console application project in VS MacOS. But I get error MSB4057.  There a similar question on stackoverflow but doesn't explain anything how to fix this kind of error.
The problem is same when I create asp.net core project.
Build FAILED.

/Users/abhimanyuaryan/Unity3D/LearnCSharp/LearnCSharp/LearnCSharp.csproj : error MSB4057: The target "Build" does not exist in the project.
    0 Warning(s)
    1 Error(s)

Time Elapsed 00:00:00.02

---------------------- Done ----------------------

Build: 1 error, 0 warnings

How did you solve?

You need to download newer version of SDK @PeterAronZentai
Try 1.1 preview 4


Added code coverage button to README

Codecov Report

Merging #12 (d95819f) into main (230c6d4) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #12   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            7         7
  Lines         1111      1111
=========================================
  Hits          1111      1111

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.