Handle missing riscv64-unknown-linux-gnu

$ head -5  /etc/os-release                                                                                03/01/24 16:06:05 PM
NAME="CentOS Stream"
VERSION="9"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="9"
$ rustc --version
rustc 1.75.0 (82e1608df 2023-12-21) (Red Hat 1.75.0-1.el9)
$ cargo-vendor-filterer
Gathering metadata
Skipping rpmostree-rust
Skipping rpmostree-client
Skipping libdnf-sys
Gathering metadata for platforms
error: Executing cargo metadata: Error during execution of `cargo metadata`: error: failed to run `rustc` to learn about target-specific information

Caused by:
  process didn't exit successfully: `rustc - --crate-name ___ --print=file-names --target riscv64gc-unknown-linux-gnu --crate-type bin --crate-type rlib --crate-type dylib --crate-type cdylib --crate-type staticlib --crate-type proc-macro --print=sysroot --print=split-debuginfo --print=crate-name --print=cfg` (exit status: 1)
  --- stderr
  error: could not create LLVM TargetMachine for triple: riscv64-unknown-linux-gn: No available targets are compatible with triple "riscv64-unknown-linux-gnu"
$

This however works with the Fedora rustc.
It seems to me RHEL rust toolchain apparently omits support for riscv64-unknown-linux-gnu...and hence using our default of tier = 2 breaks there.
Compare:

Fedora rust package
c9s rust package

This is messy.  If we do something like detect what targets the toolchain has, then the filtering is suddenly dependent on the host.
Maybe we can just allow opting-in to that via e.g.
[workspace.metadata.vendor-filter]
platforms = ["*-unknown-linux-gnu"]
tier = "2"
filter-host = true

And filter-host acknowledges some potential unreproducibility.

If we do want to filter, what we need to figure out is if there's a way to get the valid list of targets that the compiler has enabled.  (I don't understand actually why rustc --print=target-list isn't this)

Or hmm, we could add a tier = 2-rhel-9.2 or something that is defined to "intersection of tier-2 and rhel-9.2 targets".  In practice, the Fedora rustc enables windows...but we don't care about that, so that intersection may just end up omitting riscv.  (And also in practice, I doubt there are riscv specific crates anyways)

rustc --print=target-list reports everything that has an internal target spec, including many that we will never actually support in Fedora or RHEL builds.
The error you're getting is because RHEL's LLVM does not have the RISC-V target arch enabled, so it can't initialize the backend. The rest of the triple doesn't matter in this case, just the arch, so for example you could examine targets like x86_64-fuchsia without trouble.

This is messy. If we do something like detect what targets the toolchain has, then the filtering is suddenly dependent on the host.

Yes, but I think this is unavoidable. Cargo [target.'cfg(..)'.dependencies] can use any of rustc --print=cfg, including stuff like target_feature that will depend on the target CPU, so we have to ask LLVM about that. In theory, a crate built on Fedora can resolve a different dependency tree than on RHEL 9 just because the latter targets x86-64-v2! In practice, I don't know if anyone really does that though...

If we do want to filter, what we need to figure out is if there's a way to get the valid list of targets that the compiler has enabled.

I don't know any way that this is exposed by rustc, apart from just trying it, but you can use llvm-config:
(c9s)$ llvm-config --targets-built
X86 AMDGPU PowerPC NVPTX SystemZ AArch64 ARM Mips BPF WebAssembly

(f39)$ llvm-config --targets-built
AArch64 AMDGPU ARM AVR BPF Hexagon Lanai LoongArch Mips MSP430 NVPTX PowerPC RISCV Sparc SystemZ VE WebAssembly X86 XCore

we could add a tier = 2-rhel-9.2 or something that is defined to "intersection of tier-2 and rhel-9.2 targets".

That's not so bad either, because that list doesn't change very often. Maybe you don't even need to distinguish y-stream, because if we add a new arch, new vendorers are probably only working on the latest. I don't know that you need an intersection with tier-2 either -- you could consider tier = rhel-9 to be separate from upstream's support.

This issue started because someone wanted to build rpm-ostree on c9s (going through the vendor-filter path).  The more I think about this the more I think there are two separate paths:

As an upstream project, one may want to support a maximal set of targets (specifically, given Fedora is testing out riscv, I don't want to exclude it from our vendor tarballs)
As a local developer whose goal is really to build e.g. an RPM, I only care about filtering to the set that the build host (or I guess in theory the target OS) could support

So I think the fix is probably like:

rpm-ostree (and other projects) should use a container with Rust upstream toolchain or equivalent in order to generate the official vendor tarballs
We should add here tier = rhel-9.2
rpm-ostree (and other projects) can learn how to (when e.g. make rpm is invoked) pass --tier=rhel-9.2 into the cargo-vendor-filterer invocation in order to override the upstream default in Cargo.toml

We actually may want a --filter-to-host option for the last case, i.e. don't even bother with other architectures/os etc.

llvm-config --targets-built
X86 AMDGPU PowerPC NVPTX SystemZ AArch64 ARM Mips BPF WebAssembly

That set of strings looks totally different from the target triples as known to Rust though. Do you know offhand if there's a defined way to map between them? It looks like lowercasing would get a lot of them, but not all (SystemZ -> systemz != s390x) e.g.

Do you know offhand if there's a defined way to map between them?

Rust doesn't know the mapping at all, but LLVM parses the arch for its Triple class here:
https://github.com/llvm/llvm-project/blob/d36c9f828d795d77127ea0fada6be4b7b5e19dbb/llvm/lib/TargetParser/Triple.cpp#L529
... and each backend calls RegisterTarget for the Triple arches that it handles.


Dependencies.create_extension_list() cannot be called with ctx=None

I'd like to use this function outside of Cython. This line causes an AttributeError to be raised when ctx=None:
create_extension = ctx.options.create_extension or default_create_extension

I'll explain why I'm using this.
For pygame, we have most of the source code in src_c/, including .pyx files. Relative cimports from subpackages (eg src_c/subpackage) do not work, because Cython tries to import the module src_c.package.module at runtime, or it fails with an error message about relative imports beyond root.
So I moved all the .pyx files into src_c/cython/pygame. Now it works, but cythonize() didn't let me output .c files to src_c/ rather than src_c/pygame, so I duplicated some of the code from cythonize() in our setup script. But I'd also like to check dependencies.
So I have two other suggestions:

cythonize() might allow allow you to specify per-extension paths to .c files instead of using a hardcoded name.
Relative cimports might do relative imports with the C API at runtime rather than absolute imports, if possible.


FIX: geotable with "mask" column

Codecov Report

Merging #158 (5d4646e) into master (e8f197b) will increase coverage by 0.25%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #158      +/-   ##
==========================================
+ Coverage   94.24%   94.50%   +0.25%
==========================================
  Files          49       49
  Lines        1165     1165
==========================================
+ Hits         1098     1101       +3
+ Misses         67       64       -3

Files Changed
Coverage Δ

src/transforms/rasterize.jl
98.00% <100.00%> (+4.00%)
:arrow_up:

... and 1 file with indirect coverage changes


Add support for Prim.Char

Add support for Prim.Char and update the tests

Thanks! Looks good to me. @gbaz Any comments on this?

Looks good to me.
On Thu, Sep 10, 2015 at 7:08 PM, Felix Schlitter<EMAIL_ADDRESS>wrote:

Add support for Prim.Char and update the tests
You can view, comment on, or merge this pull request online at:
https://github.com/purescript/purescript-generics/pull/15
Commit Summary

Add support for Prim.Char

File Changes

M src/Data/Generic.purs
https://github.com/purescript/purescript-generics/pull/15/files#diff-0
(15)
M test/Main.purs
https://github.com/purescript/purescript-generics/pull/15/files#diff-1
(17)

Patch Links:

https://github.com/purescript/purescript-generics/pull/15.patch
https://github.com/purescript/purescript-generics/pull/15.diff

—
Reply to this email directly or view it on GitHub
https://github.com/purescript/purescript-generics/pull/15.

@garyb Do you know if we added generics dependencies anywhere in core?

I don't know off the top of my head, and I don't have an up to date version of the script I've been using - I'll check when I get back home (and actually update the repo so other people can use it too).

I think purescript-argonaut-codecs is the only dependency in core and contrib currently.


No locales on Github?

I update my addons with git. Here i have multiple localization errors. Here no localizations available?

Using the repository directly will not work, the localisation is done via curseforge and is only inserted when releases are made. If you use a release from GitHub, it should work fine.
If you do want a clone to do changes yourself (PRs welcome!), I suggest downloading a release and pasting Localization/* from the release into the repo (and ignoring the local changes). That's what I do.


Test full MIMO acquire_iq.jl

We were setting the wrong loopback when doing TRF loopback; the number
doesn't indicate which TX channel to read from, it indicates which TX
band to read from.  In our case, we're always transmitting from band
1, so select accordingly.

The graphs should now look something like this:

A few things to note:

We do not display tx[1], for simplicity's sake.
We don't have a physical loopback connected for tx[1] -> rx[1] on this device.


Access stat

Problem
Yet another approach for prefetching.
Right now we have implemented prefetch for seqscan bitmap heap scan, index scan and indexonly-scan,
In all this cases prefetch logic is determined by index search algorithm.
To avoid doing useless work with start with prefetch_distance=0 and then increase it.
Unfortunately this logic doesn't always work:
https://neondb.slack.com/archives/C04BLQ4LW7K/p1683874612586109
So if we perform seqscan through table with larg tuples (containing embeddings vector in this case), then we actually perform index join with TOAST table. Prefetch of index scan in TOAST table dpoesn't help much because we actually need just one TOAST record.
@Heikki Linnakangas suggested to implement read-ahead optimization.
Summary of changes
Collect table access statistic in neon extension and use it to prefetch next block independent on use access method.
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Before this PR:
Prefetch: hits=42855 misses=444273 expired=0 duplicates=0
Time: 66353.695 ms (01:06.354)

with this PR:
 Prefetch: hits=456048 misses=46745 expired=0 duplicates=27434
Time: 49869.761 ms (00:49.870)

So prefetch hit rate is much higher, number of duplicates is large and time is reduced but ... not so large. May be because I prefetch only next block.

More results with disabled concurrency and simulated latency=0.2msec:

Read-ahead
Query execution time

0
286847.923 ms

1
160328.085 ms

10
82776.463 ms

With 1msec latency the difference is much bigger.


Use snapshot.typeKey on adapter's _stripIDFromUrl

Using snapshot.constructor.typeKey is deprecated on latest ember-data.
snapshot.typeKey reference: http://emberjs.com/api/data/classes/DS.Snapshot.html#property_typeKey

Can you update the tests for this?

:+1: updated them, sorry for that.

Perfect, thanks!


Wrong next chapter with multiple languages

Device information

Tachiyomi version: 2.1.5. 6.1

Steps to reproduce

If a manga has multiple lenguages, when you finish a chapter it will send to the same chapter but different lenguage.
Example:

Expected behavior
It should load the next chapter in the same lenguage, possibly of the same translator.
Actual behavior
reading ch1 of the English translation. Finish it and go to the next chapter. It will load the ch1 in the other lenguage instead of ch2 in English
Other details
I don't know any nanga with multiple translator for the same lenguage, so I couldn't test it

If you have multiple languages enabled there is no filter for languages when reading.  You need to enable skip marked as read and mark those chapter read

2.10.4 you can filter out the other language now.


wrong value of jax.lax.lgamma(1.)

It appears that the lgamma function gives the wrong value for argument 1.
>>> jax.lax.lgamma(1.)
DeviceArray(4.7683716e-07, dtype=float32)
>>> scipy.special.gammaln(1)
0.0

The issue may lie deeper, but I can't trace it further than that.
(I'm using Python 3.8.5 with jax version 0.2.3)

Hi @phylyc – thanks for the report.
In this case, I think the issue is due to standard imprecision in floating point arithmetic. In 32-bit precision (JAX's default), 4E-7 is effectively zero. If you switch to 64-bit, you'll see that the result is closer to zero, becuase 64-bit floats have more precision:
import jax
jax.config.update('jax_enable_x64', True)

print(jax.lax.lgamma(1.))
# 8.881784197001252e-16

Hi @phylyc – thanks for the report.
In this case, I think the issue is due to standard imprecision in floating point arithmetic. In 32-bit precision (JAX's default), 4E-7 is effectively zero. If you switch to 64-bit, you'll see that the result is closer to zero, becuase 64-bit floats have more precision:
import jax
jax.config.update('jax_enable_x64', True)

print(jax.lax.lgamma(1.))
# 8.881784197001252e-16

Dear @jakevdp , thanks for the quick answer!
Thanks for pointing out the floating point precision issue. It does change my results slightly, so I'll use that when necessary going forward.

Dear @jakevdp , thanks for the quick answer!
Thanks for pointing out the floating point precision issue. It does change my results slightly, so I'll use that when necessary going forward.


[Java] Move StringSerializer#isLatin to StringUtils

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

Currently, the isLatin(char[]) judgment is coupled in org.apache.fury.serializer.StringSerializer, but it is not strongly related to org.apache.fury.serializer.StringSerializer. org.apache.fury.serializer.StringSerializer is only responsible for serializing related content.
Describe the solution you'd like

isLatin(char[]) should be moved to org.apache.fury.util.StringUtils
Additional context

Maybe we can add a good first issue label to the current task.


fix(ci/registry): push tags for artifacts in correct order

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area plugins

/area registry

/area build

/area documentation

What this PR does / why we need it:
The order of the tags affects how the registries' UI shows the info
about an artifact.
For example this is how the GHCR registry shows the tags for an artifact before the fix:

After the fix:

This fix is made possible by: falcosecurity/falcoctl#196
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: alacuku
Once this PR has been reviewed and has the lgtm label, please assign jasondellaluce for approval by writing /assign @jasondellaluce in a comment. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: alacuku, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 69f3e1a72cf02e94b8f10960cacbb14f4b7ac0f3


Why does registration always fail?

This error code keeps returning when I register the app. What does this error code mean?
TSLA_ACM_EXCP_4219

The scope of this issue tracker is limited to the Golang code in the SDK. Please see https://developer.tesla.com/docs/fleet-api#help-and-support.


[FEATURE] Elasticsearch Integration and Regression Analysis for User Preferences in Organization Feed

Is your feature request related to a problem? Please describe.
Currently, the organization feed is generated without taking full advantage of advanced user preference analysis and personalized content delivery. This limits the relevance and engagement of the content displayed to users, as the feed does not dynamically adapt to individual interests and behaviors.
Describe the solution you'd like
We propose integrating Elasticsearch to index and search user-generated content efficiently. By using Elasticsearch, we can leverage powerful search capabilities to enhance content discovery within the organization feed. Additionally, implementing a regression analysis on user preferences will allow the system to predict and prioritize content that aligns with the user's interests.
Specifically:

Elasticsearch Integration: Index all relevant content (posts, events, media, etc.) and enable full-text search capabilities across the organization feed.
User Preferences Regression Analysis: Implement a machine learning model to analyze user interaction data (likes, comments, shares, etc.) and perform regression analysis. This will help in predicting the most relevant content for each user, thereby improving the feed's relevance and user engagement.

Describe alternatives you've considered

Manual Content Curation: Curating content manually or through basic algorithms without Elasticsearch or regression analysis. However, this approach is not scalable and does not offer personalized content delivery.
Simple Keyword Matching: Using a simple keyword-based content filtering system. While this could partially improve content relevance, it lacks the depth and adaptability of a machine learning-driven approach combined with Elasticsearch's search capabilities.

Additional context

Technical Dependencies: This feature will require integration with existing user data analytics pipelines and Elasticsearch setup for content indexing.
Potential Impact: This enhancement is expected to significantly boost user engagement by delivering highly personalized content within the organization feed, leading to better user retention and satisfaction.

Please refer to the attached diagrams and documentation for a more detailed technical overview of the proposed system architecture. This includes the data flow for user preference analysis, Elasticsearch integration points, and the regression model's role in content ranking.

For the ML service the labeling will be gathered from the general user action history in the comments service, reaction service, events service. The feed will also be gathered from the user friends.

#380 recommendation services.


If there are warnings and errors, classNames produces only the warning class name

In the React example:
https://stackblitz.com/edit/vest-react-support-example
If an amount of 1$ is put into the the amount (which is an error - minimum of 2$ is required) - the class is still 'warning' and not 'error' (since it is also a warning - that the amount is less than 10)

Hey @liady, thanks for the feedback.
The classNames utility works as expected, as you can see in the screenshot, it adds both the invalid class and the warning class - as it should.

The problem was css precedence - I declared .warning {} after .invalid {} in style.scss, so it was the one that took effect.
Fixed the demo.


pull out playwright and optomize image. size ~2gb -> 777mb

Can we add this to the web researcher project when this goes in?


chartjs

Hi Team
I am curious why "chart.js": "2.6.0" is not included in Dependencies like Lodash and PropTypes. I believe if you add it then we dont have to install it separately. Please let me know.
Thanks

I think it's so that you have flexibility of choosing the exact version of chart.js to use -- for example, if you have chartjs plugins that require a certain version.

Wouldn't it be better to put a version range in that case? Maybe as a peer dependency also. What if a chart.js version is used that's not compatible with this wrapper?


[BUG] "count" cannot be determined until apply

Describe the bug
I just want to configure a VPC peering between 3 AWS account, but I have an error message "The "count" value depends on resource attributes that cannot be determined until apply, so Terraform cannot predict how many instances will be created."
The issue looks related to this : https://github.com/grem11n/terraform-aws-vpc-peering/issues/83 but as you can see in my configuration, I have set depends_on.
Terraform version:
1.4.6
Module version:
5.1.0
Error message:
╷
│ Error: Invalid count argument
│
│   on .terraform/modules/vpc-peering/main.tf line 55, in resource "aws_route" "this_routes":
│   55:   count                     = local.create_routes_this ? length(local.this_routes) : 0
│
│ The "count" value depends on resource attributes that cannot be determined until apply, so Terraform cannot predict how many instances will be created.
│ To work around this, use the -target argument to first apply only the resources that the count depends on.
╵
╷
│ Error: Invalid count argument
│
│   on .terraform/modules/vpc-peering/main.tf line 79, in resource "aws_route" "peer_routes":
│   79:   count                     = local.create_routes_peer ? length(local.peer_routes) : 0
│
│ The "count" value depends on resource attributes that cannot be determined until apply, so Terraform cannot predict how many instances will be created.
│ To work around this, use the -target argument to first apply only the resources that the count depends on.

Terraform code that produces the error:
module "vpc-peering" {
  source  = "grem11n/vpc-peering/aws"
  version = "5.1.0"

  depends_on = [
    module.vpc_source,
    module.vpc_destination
  ]

  providers = {
    aws.this = aws.source
    aws.peer = aws.destination
  }

  this_vpc_id = module.vpc_source.vpc_id
  peer_vpc_id = module.vpc_destination.vpc_id

  auto_accept_peering = true

  tags     = local.common_tags
}

Additional context
I use the "terraform-aws-modules/vpc/aws" module to create a VPC for each AWS account and I want to configure peering between theses VPC in the same terraform project.

I ran into a similar issue today. What helped was actually removing the complete depends_on block in my Terraform code:
  depends_on = [
    module.vpc_source,
    module.vpc_destination
  ]

I still have the same issue in both cases

@grem11n, have you got a solution ?

Same problem here.
I tried a workaround from here https://github.com/grem11n/terraform-aws-vpc-peering/tree/master/examples/associated-cidrs but it doesn't help

The full code example is down below.
It's not possible to create vpc-us, vpc-eu and vpc-peering in one go.
provider "aws" {
  region = "us-east-1"
  alias  = "us"
}

provider "aws" {
  region = "eu-west-1"
  alias  = "eu"
}

module "vpc-us" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "5.0.0"

  providers = { aws = aws.us }

  name = "north-virginia"
  cidr = "<IP_ADDRESS>/16"

  azs             = ["us-east-1a", "us-east-1b", "us-east-1c"]
  public_subnets  = ["<IP_ADDRESS>/21", "<IP_ADDRESS>/21", "<IP_ADDRESS>/21"]
  private_subnets = ["<IP_ADDRESS>/21", "<IP_ADDRESS>/21", "<IP_ADDRESS>/21"]

  enable_nat_gateway = false
}

module "vpc-eu" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "5.0.0"

  providers = { aws = aws.eu }

  name = "ireland"
  cidr = "<IP_ADDRESS>/16"

  azs             = ["eu-west-1a", "eu-west-1b", "eu-west-1c"]
  public_subnets  = ["<IP_ADDRESS>/21", "<IP_ADDRESS>/21", "<IP_ADDRESS>/21"]
  private_subnets = ["<IP_ADDRESS>/21", "<IP_ADDRESS>/21", "<IP_ADDRESS>/21"]

  enable_nat_gateway = false
}

module "vpc-peering" {
  source  = "grem11n/vpc-peering/aws"
  version = "6.0.0"

  providers = {
    aws.this = aws.us
    aws.peer = aws.eu
  }

  this_vpc_id = module.vpc-us.vpc_id
  peer_vpc_id = module.vpc-eu.vpc_id

  auto_accept_peering = true
}

Please, refer to the example of the module with depends_on which covers this case.
To avoid this error, one has to also provide route table ids.

For future searches, here is the updated vpc-peering module:
module "vpc-peering" {
  source  = "grem11n/vpc-peering/aws"
  version = "6.0.0"

  providers = {
    aws.this = aws.us
    aws.peer = aws.eu
  }

  this_vpc_id  = module.vpc-us.vpc_id
  peer_vpc_id  = module.vpc-eu.vpc_id
  this_rts_ids = concat(module.vpc-us.public_route_table_ids, module.vpc-us.private_route_table_ids)
  peer_rts_ids = concat(module.vpc-eu.public_route_table_ids, module.vpc-eu.private_route_table_ids)

  auto_accept_peering = true

  depends_on = [
    module.vpc-us,
    module.vpc-eu
  ]
}

Now apply works in one go.

Thank you for your feedback !


New chip-seq / atac-seq peak calling logic

Hello everyone!
In 1.2.8 we implemented

While consensus peak calling, keep the highest scoring peak instead of calling the summit for the highest scoring peak and expanding the peak to 250 bases.
Enable consensus peak calling for broad peaks. Thanks to @mistrm82 and @yoonsquared for pointing out this was missing.

There is some concerns from different sides, let us outline them here!
Thanks!

The concerns were clarified. Independently confirmed that it is the better way of calling peaks.


Incorrect use of  warning

Problem
It seems that the input returned by the <FileUploader /> does not have a matching id to it's parent label prompting a warning from issues on chrome. Since the input and label are not exposed, this change should be a global fix in the repo.

Error Message ( Chrome Issues Tab )
Incorrect use of
The label's for attribute doesn't match any element id. This might prevent the browser from correctly autofilling the form and accessibility tools from working correctly.
To fix this issue, make sure the label's for attribute references the correct id of a form field.
Browser
Chrome Version 113.0.5672.127 (Official Build) (64-bit)
Fix
The label returned from  should have a for="file" prop that matches the labels input id="file" property.
Resource
https://www.w3schools.com/tags/att_label_for.asp
https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/for

I confirm the bug - I tried to write tests using Testing Library and finished with the following error:

Found a label with the text of: Upload files, however no form control was found associated to that label. Make sure you're using the "for" attribute or "aria-labelledby" attribute correctly.

solved in https://github.com/KarimMokhtar/react-drag-drop-files/pull/131
you can use v. 2.3.11


pip install imatlab is out of date.

10751]: [D 2017-03-27 10:25:07.661 JupyterHub systemdspawner:253] user:s8weber unit jupyter-s8weber-singleuser is active
 [I 2017-03-27 10:25:09.014 s8weber restarter:103] KernelRestarter: restarting kernel (1/5)
 WARNING:root:kernel 25edcaf3-7e38-4767-bb2f-cc4b3109c1cf restarted
 Traceback (most recent call last):
   File "/usr/lib/python3.5/runpy.py", line 184, in _run_module_as_main
     "__main__", mod_spec)
   File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
     exec(code, run_globals)
   File "/usr/local/lib/python3.5/dist-packages/imatlab/__main__.py", line 5, in <module>
     IPKernelApp.launch_instance(kernel_class=MatlabKernel)
   File "/usr/local/lib/python3.5/dist-packages/traitlets/config/application.py", line 657, in launch_instance
     app.initialize(argv)
   File "<decorator-gen-120>", line 2, in initialize
   File "/usr/local/lib/python3.5/dist-packages/traitlets/config/application.py", line 87, in catch_config_error
     return method(app, *args, **kwargs)
   File "/usr/local/lib/python3.5/dist-packages/ipykernel/kernelapp.py", line 454, in initialize
     self.init_kernel()
   File "/usr/local/lib/python3.5/dist-packages/ipykernel/kernelapp.py", line 365, in init_kernel
     user_ns=self.user_ns,
   File "/usr/local/lib/python3.5/dist-packages/traitlets/config/configurable.py", line 412, in instance
     inst = cls(*args, **kwargs)
   File "/usr/local/lib/python3.5/dist-packages/imatlab/_kernel.py", line 119, in __init__
     Path(self._call("prefdir"), "History.xml"))
   File "/usr/local/lib/python3.5/dist-packages/imatlab/_kernel.py", line 37, in __init__
     self._et = ET.parse(str(path))
   File "/usr/lib/python3.5/xml/etree/ElementTree.py", line 1184, in parse
     tree.parse(source, parser)
   File "/usr/lib/python3.5/xml/etree/ElementTree.py", line 585, in parse
     source = open(source, "rb")
 FileNotFoundError: [Errno 2] No such file or directory: '/u/s8weber/.matlab/R2016b/History.xml'
 Assertion failed: nbytes == size

after updating my _kernel.py that was installed using pip with your current copy in the repo. got the kernel working.
so please push the latest code to pip so others like me don't fail :)

Thanks @steverweber. This error affects people running without the matlab GUI (since no History.xml file is created) and was fixed a few weeks ago in the repo. There are a few loose ends we are working on before pushing out the next release to PyPI. Namely: plotly/MATLAB-Online#118, https://github.com/anntzer/imatlab/issues/6#issuecomment-289318544, and https://github.com/anntzer/imatlab/issues/6#issuecomment-287859541.

Thanks @imatlab-helper for handling this.
In order to install the git version, either follow in instructions in the README, or https://github.com/anntzer/imatlab/issues/2#issuecomment-284005633 if you don't have git.


cannot import flask from flask

Serving Flask app "ap.py"
Environment: production
WARNING: This is a development server. Do not use it in a production deployment.
Use a production WSGI server instead.
Debug mode: off
Usage: flask run [OPTIONS]

Error: While importing "ap", an ImportError was raised:
Traceback (most recent call last):
File "/mnt/c/project0/project0/FLASK_PROJECT/venv/lib/python3.8/site-packages/flask/cli.py", line 240, in locate_app
import(module_name)
File "/mnt/c/project0/project0/FLASK_PROJECT/ap.py", line 1, in
from flask import flask
ImportError: cannot import name 'flask' from 'flask' (/mnt/c/project0/project0/FLASK_PROJECT/venv/lib/python3.8/site-packages/flask/init.py)

This issue tracker is for a command-line tool called "hub" which is written in Go. Your question seems unrelated to this project, and you should find a more appropriate forum to ask it in. Thank you.


Exception on importing pisa

I want to use xhtml2pdf in my Django project, but there is an exception on importing the pisa module:
(env) ❯ python manage.py shell
Python 3.5.2 (v3.5.2:4def2a2901a5, Jun 26 2016, 10:47:25)
[GCC 4.2.1 (Apple Inc. build 5666) (dot 3)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
(InteractiveConsole)
>>> from xhtml2pdf import pisa
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "/Users/anatskiy/parkour/env/lib/python3.5/site-packages/xhtml2pdf/__init__.py", line 41, in <module>
    from xhtml2pdf.util import REPORTLAB22
  File "/Users/anatskiy/parkour/env/lib/python3.5/site-packages/xhtml2pdf/util.py", line 304
    raise Exception, "box not defined right way"
                   ^
SyntaxError: invalid syntax

Python: 3.5.2
xhtml2pdf: 0.0.6
reportlab: 3.3.0
Django (if it matters): 1.10.2

You are running Python 3. xhtml2pdf <0.1.0 don't support py3, try installing the latest beta branch 0.1.1b to get python 3 support.

Clsoing this, as I don't have the resources to backport Py3 work.


Add kconfig/kbuild build system for micro-kernel

This patch adds kconfig/kbuild build system for micro-kernel.
However the old Makefile is still kept.
It can be used by:
$> make -f Makefile.old
To use new build system:
$> make versatilepb_defconfig
$> make
Signed-off-by: Lv Zheng<EMAIL_ADDRESS>

To change SOB.


Daily update

Automated changes by create-pull-request GitHub action

Build failed because of:
No cached entry, and Wall Street Blockchain Alliance has issues with logo: wall_street_blockchain_alliance.svg, This logo cannot be autocropped because of an unusual interaction with its viewbox. Please find a different logo or convert again from the original to SVG.
No cached entry, and Vilnius Gediminas Technical University has issues with twitter: https://twitter.com/manovgtu, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
No cached entry, and Sollers College has issues with twitter: https://twitter.com/SollersEdu, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for CSIRO/DATA 61: https://twitter.com/CSIROnews
No cached entry, and Volkswagen Aktiengesellschaft has issues with twitter: https://twitter.com/volkswagen_de, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for TYNYBAY Inc: https://twitter.com/tynybayinc
Empty twitter for Truepic, Inc.: https://twitter.com/truepicinc
Empty twitter for Tectonic Labs Ltd.: https://twitter.com/cloud_ridge
Empty twitter for eBaoTech International Pte Ltd: https://twitter.com/Ebaotech
Empty twitter for China Telecommunications Corporation: https://twitter.com/chinatelecom1
No cached entry, and Aisin AW Co., Ltd. has issues with twitter: https://twitter.com/hashtag/%E3%82%A2%E3%82%A4%E3%82%B7%E3%83%B3AW, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
Item Wall Street Blockchain Alliance has no image_data


Allow devs to override the inputs

It looks good to me. Before making a PR to upstream, can you explain a little about why we want to be able to override this? That way we can poke holes in your theory, if there are any, before you send this upstream.

If you take a look at DateRangePickerInput the html inside is very prescriptive and hard to manipulate. Our designs have very different inputs for mobile and desktop and what is currently available for input will not support them both.


Converting all top-level commands to async

Converting all top-level commands to async in preparation to also convert the implementations in a later PR

Just curious if there is a reason that this PR was closed? I would love to help moving xcodes over to async await.


Unknown option -C (git)

Trying to back up my organizations repos
Command: github-backup Leeps-Lab /home/leeps/backup/ -o Leeps-Lab -A
All this does is copies the repo names and I get this in the terminal:
INFO:github-backup:Repo doesn't exist, lets clone it
['git', '-C', '/home/leeps/backup/repositories/documentation', 'clone', 'https://github.com/Leeps-Lab/documentation.git', 'repository']
Unknown option: -C
usage: git [--version] [--help] [-c name=value]
           [--exec-path[=<path>]] [--html-path] [--man-path] [--info-path]
           [-p|--paginate|--no-pager] [--no-replace-objects] [--bare]
           [--git-dir=<path>] [--work-tree=<path>] [--namespace=<name>]
           <command> [<args>]

I got this error for all the repos

I think -C is just a short form of --git-dir. Maybe we should always use the long form here..

I think -C is just a short form of --git-dir. Maybe we should always use the long form here..

@stv0g Should I change my cli arguments?
Could the issue be because of my OS? I believe I am using CentOS.

Same problem here! :(


Bugfix tested with stable and without, random samples up to 10D

Mostly renaming of variables, some reformating and a small bugfix


Update to 0.57.3

Hi,
Is there any chance of updating to RN version 0.57.3?
Or documenting how to update versions ourselves?
Love this btw, just what I needed!

Hey @alexdunham, its in my pipeline, i will release a step by step guide to do it for upcoming versions, too. Stay tuned. :)
In theory this could even be automated but I did not have the time to configure it, yet.
best regards

Hi @voydz  I was creating the bindings myself when I saw your project, I'm almost done with the migration of  the version 6.1.5, maybe you could take a look and I can do a PR to your  project

Hi @egarim thank your very much for this effort.
Would be awesome to review and test your changes. A PR is very welcome.
best regards

@voydz  is there a way I can contact you, I sent you an email to this email<EMAIL_ADDRESS>last week.
You can find my contact info in my Github profile, maybe we can meet because I have a few doubts regarding the bindings and maybe you have already been through the same problems

@egarim  I'm interesting about migration to 0.61.5. and I completely done in iOS side but not android yet

@egarim if there is an update to the latest version would be happy to test and/or contribute. We are very new to xamarin, but not new to RN and any way to help?

Hey @alexdunham, its in my pipeline, i will release a step by step guide to do it for upcoming versions, too. Stay tuned. :)
In theory this could even be automated but I did not have the time to configure it, yet.
best regards

would appreciate any steps and can take it up...

@voydz by any chance you could give me some pointers?

Hi @elarasu ,
unfortunately there are some issue due to some major changes to react-native since the current xamarin-react-native version, so I was not able to easily upgrade in an easy effort.
We have transitioned to react-native a long time ago. As this project is thought to help to transition away from Xamarin towards react-native I merely see this project as a proof-of-concept and the fulfillment of a challenge I put myself onto. I would not advise anyone to use this in combination with Xamarin in production.
I am afraid there isn't really anything more I can do for you. But please feel free to fork and experiment with the code in any way (as this project is MIT licensed).


TODO

[x] Remove RequestHandler and SchemeHandlerFatory from CefSharp example as they will be in the 89 release
[x] Switch to a more generic TargetFramework, (net5.0)

It's not possible to switch to netstandard, switched to targeting .net 5.0, we could potentially drop that down to .Net Core 3.1

Now targets .Net Core 3.1


Requirement to pick an address doesn't function

Chrome autofill puts in a partial address but autocomplete doesn't make the user enter/ select an address as I thought was supposed to be the case.

Would disabling the autofill be an option?

I'll test it, I don't expect anyone would be using autofill in this at all/often but thought it was worth mentioning.

upon not picking an address I just get an error in the console
Uncaught ReferenceError: mono is not defined

I think that error is caused by your app, not the autocomplete directive. There are no variables called mono in there. Check the error stack if possible

Hang on, just checked your plunker. Is ng-messages required for this to function properly?

it requires a minimum of AngularJs 1.3 because it uses its validation methods. Having the ng-messages isn't required. You just won't see validation messages without them.


Feature Request: Add a simple xnet.redstone channel type.

I guess xnet.logic channel type can somehow be used to transmit redstone signals, but I could not figure out how to do it.
For example a simple setup:
lever---connector---cable---connector---redstone lamp
How do you configure a xnet.logic channel to have the redstone lamp light up when you flip the lever? No idea, it looks so overly complicated.
An xnet.redstone channel type where you just pick a color and input or output would be great. That way it can be used just like the EnderIO Redstone conduits or the Immersive Engineering Redstone cable and connectors.

Well the redstone support in the logic channel is exactly like that. It emits a redstone signal based on a color input. What exactly is not working with that? What are you trying to do exactly?

I am trying to setup a logic channel to turn off/on  EnderIO farming stations with levers, just like I do with EnderIO redstone conduits.
This is my current setup: redstone conduits and levers, each conduit connector has a different color configured. Each farming station connector has the corresponding color configured. This way I can turn on/off the farming stations from a central location

So lets say I want to lear how to set this up in XNet, so I do this as a test:

I am just guessing that I have to use a Redstone Proxy because the connector won´t connect to a plain stone block and I cant place the lever on the connector directly.
So, with this setup, how to turn the lamp on/off with the lever? Since there is no clear documentation and the UI looks kind of complicated since it seems to be very powerful, I will have to guess:

I guess "Rs" is "Redstone Signal"?  So when the lever is on, activate output color red?

Is this how you set it up? Just guessing...

Well. it works... so it must be right. Seems kind of complicated for just sending a simple redstone signal.
Compare it with the EnderIO UI:

You just pick a color and 2 other settings.
Or the IE credstone connectors and cables:

You pick a color and input/output by hitting the connector with a hammer.
The xnet.logic channel type seems wonderful and able to do a lot of things. but having to set it up to control a machine with an on/off lever feels like killing a fly with a shotgun.
Hope you take this as constructive criticism.

That's because it is really designed to do more complicated things then just that. I don't know how I could make this any easier while still staying within how the rest works?

I just edited the comment at the end answering just that. I understood perfectly that is meant to do complicated things.
Thank you very much for your responses.

Another alternative:
Instead of a separate new channel type, add "Simple signal" (or something like that) as an option to the "Sensor"/"Output" options, and the interface changes where you just pick a color and input or output

See https://github.com/McJty/XNet/issues/70 for proposed feature

thanks calloatti. i now know how to use it. :)

So I have a question. I am trying to use a similar setup to control redstone signal to stabilized spawners in a vertical column. I am running cable down and attaching connectors to redstone repeaters that are next to the column of spawners. I am using one logic channel for all.  a lever attached to a redstone proxy for each spawner is assigned its own color within the one channel. so far so good. Here is my problem. I would like to have one master lever to turn on all spawners at once. I want individual control and a master switch. I cant seem to figure it out. it results in the redstone signal blinking on and off.
I have tried a test area with just 3 levers attached to 3 redstone proxies, and 2 redstone lamps. to see if I could turn on each lamp separate and then a master. But like I said I cant seem to come up with anything. Suggestions?


Restore missing triggers between steps

fixes #2530
it:

reverts #2489
adds the missing triggering step
adds a check when creating the processing graph, to ensure all the config- and dataset-level steps have at least one triggering step of the same level, to handle the case where we couldn't get the configs or the splits.

After merging, the first full backfill should fix the issues.


Fix navbar link

Closes #55

Not sure why there's no preview.

Preview is there now, looks good to me.


IE browser issue

Tested Browser: Edge, IE, Chrome
In Edge and IE browser: On entering the first value in "particular TEXTBOX" the control (cursor) automatically goes to particular page Last "TEXTBOX"  - We can't able to calculate.
In chrome browser it's working properly.

I am facing the same issue.

I am facing the same issue.

Facing the same issue.
Are you guys was able to fix that with some workaround?

https://github.com/jsillitoe/react-currency-input/issues/90#issuecomment-507086811
In case anyone still facing this issue, the solution in the link works for me.


CI

📝 Description
looks like we need screenshot regen
❓ Context

Impacted projects: ``
Linked resource(s): ``

✅ Checklist

[ ] Test coverage
[ ] Atomic delivery
[ ] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

So CI is good then 🤔

/generate-screenshots


Skipt bandit check for getting first and last name in anonymisation.

Also get rid off unused function for creating random text

Bandit still complain about some lines :thinking:

Bandit still complain about some lines thinking
Is there some "skip this line" comment available for bandit?

I forgot the actual skip line comment :joy:


STORM-1362 Travis should run at least the multilang-test using Python2 and 3

Here's how I was able to get the multilang-test to be run separately with python3 in travis.

Putting this here for future reference only, because my testing shows that a larger python3 migration effort will be needed to really make this build cleanly.  This PR is just to record how I was able to get Travis to run the test using python3 when the container-based VM also has python2 as its default python installation.


Question: "Officialness"

Hey David! I see that you're an engineer at Stripe, but have published this under your own handle. Was wondering if there was anything stopping this from being embedded under Stripe's org. Any assumptions unsafe to make about this library besides what the license says?

Hey @kylemh, I made this library because I saw a testing gap and it was simple. I can check in with our open source team and see what the story is though!

Any assumptions unsafe to make about this library besides what the license says?

I consider the library stable and complete, barring any breaking API changes from Cypress!

Thanks a ton 🙏🏽

Would be amazing to get this supported by Stripe.
Struggling to test Stripe Elements with Cypress.


[Indigo] [Cypress] TreeView behavior testing.

Description
We are going to test if the expand all, collapse all behavior work as expected. And we will also test if the context menu (and its functionality) works as expected.
NOTE: The context menu functionality is not finished yet. But we can still make a test first (with expected behavior), and then implement them in the following sprint.
Checklist

[x] Add ids to the corresponding tree view components if necessary.
[x] Test the "expand all" and "collapse all" behavior by checking the children's length (if one node has children).

Do not check the real data yet.

[x] Check if the data matches.

We must get the original data by expanding the entire tree.
Make sure that we are not comparing them with sample data directly because it will be FAILED after we integrate it with the real data, where we cannot have a constant value to compare with.
After getting the original data, we collapse the entire tree, and use single click (and/or context menu functionality) to open some nodes and check if the data matches.

[ ] Test the context menu functionality (after implementing).

Backlog: Tree context menu functionality testing.


Consider migrating to the ember-codemods org?

I’ve slowly been moving the general community codemods that I’ve made into the ember-codemods GitHub org. Mostly as a way to:

increase visibility
reduce bus factor

We’d make sure to preserve the current permissions (e.g. no reduction in control) if we did the transition.
Would this be something y’all would be open to?

@Turbo87 - I believe that I've already shot you over an invite to the org, so if you are in favor you should be able to transfer.

ya, will move it next week. please remind me if I forget ;)


Made images responsive (fluid).

... and added a couple of examples to the demo.
The approach is similar to Bootstrap's one:
https://github.com/twbs/bootstrap/blob/e38f066d8c203c3e032da0ff23cd2d6098ee2dd6/less/mixins/image.less#L9
// Keep images from scaling beyond the width of their parents.
.img-responsive(@display: block) {
  display: @display;
  max-width: 100%; // Part 1: Set a maximum relative to the parent
  height: auto; // Part 2: Scale the height according to the width, otherwise you get stretching
}

ref #7

Could you make example images monochromic? In black and white colours.

Done. However, I don't really like the quality that Photoshop gave to me.

It should be better now. Please take a look.

Cool! Thank you!

Awesome. Thanks.


Some question about FloatingActionButton.

I have several things to ask about floating action button.

Why did you use raw svg when android studio already support vector drawable ?
And putting vector drawble in floating action button give an error saying "When using VectorDrawableCompat, you need to use app:srcCompat. " which is not working.

To change background color "android:background" seems works but in editor it shows a square box instead of a fab. And the icon also seems to be tinted by default.

How do I change my action menus color ?

I am using fab on my mapView and when action menus pops up my mapview stop listening to touch events( It stops moving ) until action menus are in hidden state again. Can I not interrupt the other touch event like map view when action menu pops up or can I make my screen background deem when action menus pops up ?

I was able to change fab menus color by setting up theme, it will be better if user don't have to put a theme  instead they can edit it from xml.

Ad. 1. When I started working on Carbon, VectorDrawableCompat wasn't there yet. And the reason I decided to keep SVGs is that Android's vector format doesn't support transformations, text, gradients and other things SVG does. Using SVGs is also faster as I don't have to convert files from Inkscape/Ilustrator.
Ad. 2.  This is because Carbon's FAB uses masking to get its shape, which is not supported in the editor. Carbon uses tons of things which are not supported by Android Studio. I could add an edit-mode-only background to make it round in the editor. Let me make an issue for that. Tinting is intended - FAB's icon color is a part of theme, so it should change with theme change.
Ad. 3. I guess there's no way to change colors right now. Let me think about it. FloatingActionMenu is a popup, so it has all its default properties. It's possible to make it pass events through. Dimming should be done by the user.

Ok I have another question, How do I add onClickListener on a Floating Action Menu ,I can't find view by id,  it keep returning null.

You can't do that. Menu is shown in a popup window, so accessing views by id would be hard. There's a method to set listeners for all menu items. It works like the regular menu listener:
setOnMenuItemClickListener(MenuItem.OnMenuItemClickListener listener)

Thanks, that saved my life, I was getting frustrated because I could not find something that suppose to be so simple.

I've added support for app:srcCompat and a method to get FloatingActionMenu from FloatingActionButton. The latter should be enough to dim background and change all properties of menu, just like you would with a simple popup.


Solved PDI-15290 and PDI-14225

This pull request contains the solution of the to JIRA issues. Fix for PDI-15290 on master branch will be provided soon thereafter

@mdamour1976 can you take a look? I believe R2D2 already addressed PDI-14225 in the context of BACKLOG-7185, can you confirm?

Thanks for this Sergio, but we cannot process PRs on a patch branch. Looking into your PR on Master though. Thanks for the contribution!


How to run on single GPU?

I'm not able to run the train.py script since it has to be run with distribuited GPUs and I only have 1 one. Is there a way to run it anyway on single GPU?

See #74


Specify how signal trigger updates

The doc need to specify how re-assigning signal.value trigger update for subscribers.
Is there any optimization done to avoid useless re-computation ?
Does it use 'identical' or '==' operator under the hood ?
Those details are quite important so developers can use the library in an optimal way.

There is an article that talks about the implementation:
https://preactjs.com/blog/signal-boosting/
Comparisons are done with == at set time and there is a version number for comparison.
But in addition to that computed only updates with dependencies change and will not even be computed if never read.

Thank you for the clarification. I think it's worth mentioning it somewhere in the documentation, it will be useful to some for sure.

In the next version of signals working on adding a SignalsEqualityCheck that lets you override the behavior while defaulting to ==:
https://github.com/rodydavis/signals.dart/pull/109
final s = signal(
  User(1, 'Bob'),
  equalityCheck: (a, b) => a.id != b.id,
);

This is a good idea, it provides a simple and flexible API. Maybe a way to specify the default policy would be useful as well.
About the ListEquality, a DeepEquality would be better IMO.
I think this is a pretty standard term and it would also cover cases like Map, Set, etc.
Having 3 built-in: Identical, Equal (==), and Deep would be enough for 90% of the case.
Ability to implement custom ones would be nice to.

Since signals had 0 dependencies it would not include the deep compare from collection, but the identical could be added!


Add base_url clarification for GitHub enterprise

If /api/v3 is left off of this URL, users will get confusing responses such as 406 Not Acceptable.

Hey @SpamapS !  Thanks for opening this PR! It looks good and should make it into the next release!


Reactify edit options [#174374781]

This is just failing the same test that started happening after merging in the auto search changes. See this PT comment: https://www.pivotaltracker.com/story/show/174381940/comments/217937014


[GraphServices] Add Microsoft.GraphServices cli module

Related command

az graphservices account
Description

We're generating the first version of the Azure CLI module for our RP called Microsoft.GraphServices. All of the available groups and commands are being added based on the generation from our swagger spec.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

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

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

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

GraphServices

Manual tests worked as expected but I still need to add integration tests. For our service to create a resource, we need an Azure AAD Application Id to be passed as a parameter. Is there a way to create an AAD app scoped to the test run or is it fine to use an existing app id in the test command params?

Manual tests worked as expected but I still need to add integration tests. For our service to create a resource, we need an Azure AAD Application Id to be passed as a parameter. Is there a way to create an AAD app scoped to the test run or is it fine to use an existing app id in the test command params?

You can try az ad app create to create a new AAD app.


Lodash leaked as a global

When I import the 'react-color' to my project it seems to assign lodash to the _ global. This is unfortunate because I already have _ defined and it gets overwritten and this breaks the app.

i had the same issue, had to go into all of my existing files that used lodash and do a specific import from 'lodash' module to keep old lodash for rest of app

This is a "feature" of lodash, see https://github.com/lodash/lodash/issues/2251.

This would be fixed by only importing the methods from lodash that are actually used, so import isString from 'lodash.isstring' or import isString from 'lodash/isString' rather than import _ from 'lodash'.


feat: dio client interceptor

GitHub Issue: #

Proposed Changes

[ ] Bug fix
[x] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build or CI related changes
[ ] Documentation content changes
[ ] Other, please describe:

What is the current behavior?
No dio client interceptor
What is the new behavior?
I'm completing what has been started on this PR #5, which it's to implement a recipe in dart to handle hashmac signature.
Checklist
Please check that your PR fulfills the following requirements:

[x] Documentation has been added/updated.
[x] Automated tests for the changes have been added/updated.
[x] Updated BREAKING_CHANGES.md (if you introduced a breaking change).

Other information

Make sure you don't keep all 40+ commits. Their high number is not justified.


Updated agent section in architecture - info about disabling capabilities

Configuration doc does not contain info about disabled_capabilities field yet. Is it updated automatically?

Semi-automatically - please generate new configuration with medschool (see mirrord/medschool)


04/12/2018 18:33 : Create Pull Request for branch master -> prTesting1_2018041218

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

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


Failed Build

When I try building the image using docker build -t ubuntu-test:latest ., it says:
Step 6/23 : COPY backend/go.mod backend/
COPY failed: file not found in build context or excluded by .dockerignore: stat backend/go.mod: file does not exist

See here and the run error here.
Please offer advise on the correct build command, or an easy fix.

Where are you seeing ubuntu-test? I don't see it anywhere in our docs. You need to build using the .. context not .

@maebeam Let me try the Ubuntu latest content and '..' instead of '.' and see if it makes a difference, but for the core it compiled and ran correctly using that command and will not build correctly the dockerfile using this command.
I assume perhaps for this at least:
'docker build -t Ubuntu:latest ..' might work? I will check, but it says for some reason some files are not found utilizing the other commands instead of a build error. It it isn't that command, then what is the proper build command?
Also, the runtime error which is the second link was an error on your CI workflow so am wondering how that is fixed or fixable?
Should I try a build option directly for the Dockerfile instead?

Hi, actually you are wrong. There is no build context that uses .. and I haven't seen that before. Anyways, here is the error log that happens when trying to build the Dockerfile for backend @maebeam
sudo docker build -t ubuntu-test:latest .

Executing shared-mime-info-2.1-r0.trigger
Executing gdk-pixbuf-2.42.6-r0.trigger
OK: 734 MiB in 171 packages
Removing intermediate container 874db0de9017
 ---> 0f9df6c7afd6
Step 5/23 : WORKDIR /bitclout/src
 ---> Running in 60966c5ef3a3
Removing intermediate container 60966c5ef3a3
 ---> be887900cd6c
Step 6/23 : COPY backend/go.mod backend/
COPY failed: stat /var/lib/docker/tmp/docker-builder931382092/backend/go.mod: no such file or directory

Would trying to build using buildkit make a difference or would it be a similar error log?

@andrewarrow Curious, how did you ever manage to compile the backend?

@maebeam https://github.com/bitclout/backend/commit/21a58bfbc397db410a4e8ccf77c6557bac97d951?_pjax=%23js-repo-pjax-container#commitcomment-51296627

docker build -t backend ..


Missing steps for OS X certificate setup

There are missing steps missing in the OS X certificate setup.  It doesn't tell you to open KeyChain Access first.

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

ID: 42b50f4b-5e10-6fef-ad73-612996d2001a
Version Independent ID: 8eda699d-40de-e20b-d030-719a6c4e2ff2
Content: Setting up an environment for developing a Power BI visual - Power BI
Content Source: powerbi-docs/developer/visuals/environment-setup.md
Service: powerbi
Sub-service: powerbi-custom-visuals
GitHub Login: @mberdugo
Microsoft Alias: monaberdugo

Thanks for your comment, @coreyroth. This issue has been assigned to the doc author for review.

Thanks for your comment, @coreyroth. I’ll add this request to our docs backlog.

Thanks again, @coreyroth. I added that step.


Some of README not ported from Python to C#

Some parts of the README have had code examples changed to use C#. However, everything after line 101 is still in Python.  Have the functions themselves not been ported to C#, or has the README just not been fully updated?

Looking at the code of Transcript.cs, we can see that the methods has been implemented. It seem to be an error on the readme


[KAG-2165] Add curl (CLI) to bazel build

Summary
Per the KAG ticket, this PR add curl to the build such that the CLI tool is packaged and shipped with Kong to allow customer-facing Kongers to have an easier time troubleshooting client issues.
More details in the ticket.
Opening this PR so that I can get PR-based GitHub Actions runs.
Checklist

[ ] The Pull Request has tests
[ ] A changelog file has been added to CHANGELOG/unreleased/kong or adding skip-changelog label on PR if unnecessary. README.md (Please ping @vm-001 if you need help)
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog

[Implement ...]

Issue reference

Fix #KAG-2165

A full matrix run is here: https://github.com/Kong/kong/actions/runs/6114498340/job/16597245208

Closing in favor of https://github.com/Kong/kong-ee/pull/6450
per https://github.com/Kong/kong-ee/pull/6391#issuecomment-1710913794

Closing in favor of https://github.com/Kong/kong-ee/pull/6450
per https://github.com/Kong/kong-ee/pull/6391#issuecomment-1710913794


Xref parser testing framework

Description
Adds a single unit test for a parser, and a database test harness to facilitate that. Small opening in BaseParser made to remove tight dependency on XrefParser::Database
Use case
Testing parsers is hard. They require a database handle as argument, and the testable output appears directly in the database.
Benefits
A test harness without needing to use Bio::EnsEMBL::Test. ORM functionality should make query-based testing easier, and allow schema upgrades to occur without undue suffering by way of DBIC.
Possible Drawbacks
DBIC dependencies, foreign key conflicts in constraint-free xref code, no Travis harness, but we should be adding that to the new repo instead of this one. Generated ORM is now diverged from original schema - there are changes that are required for DBIC to work with the schema.
Testing
I wrote the only tests that exist. No Travis yet.

I should add the ORM schema is incomplete. We could probably add more relationships, but I don't understand the original schema enough to follow through.

Coverage decreased (-0.004%) to 81.354% when pulling 0daeff51d3281764a2cfb8d5cd1a650cb049e209 on feature/parse_test into 67de977d0f348673366b658b319e449c30c2256a on feature/xref_sprint.


⚠ Vaultwarden has degraded performance

In 23f5461, Vaultwarden (https://vaultwarden.underthere.xyz) experienced degraded performance:

HTTP code: 502
Response time: 18493 ms

Resolved: Vaultwarden performance has improved in 501daf1 after .


Add checklist for things to do when we get a new steering council member

Quality steps, thanks @jasongrout.

@jasongrout Looks good. Would you mind fixing the typo and I will happily push the green button. Yay, process checklist!

Yay, process checklist!

One of the few meaningful three word sentences that has never been said before ;-)

:+1:

I'm going to go ahead and merge this. If you were waiting for the steering council to all weigh in, please feel free to revert. It looks good to me. And for @takluyver: Yay! Yay! Yay! I :heart: process checklists ;-)
Thanks @jasongrout


Keep getting tiktoken errors when running my code

Full disclosure: I am a complete noob with coding.
I am a fan of the youtuber DougDoug and I am trying to set up a program he wrote that allows me to speak to an AI and have it speak back to me verbally. He has the code publicly available so I got it and did the necessary steps of obtaining the API's that I needed, but when I run it I get this:

I am attempting to make sense of it, but haven't been able to make heads or tails of my problem. The link provided in the NotImplementedError doesn't take me anywhere. I've tried some googling but am not sure how to apply the advice others received for their cases to mine. Like I said, total noob.
Not sure if it helps but here is a link to the code: https://github.com/DougDougGithub/Babagaboosh

Use a newer version of tiktoken


Use of individual config filenames prevented

One line in the input.py prevents Reinventto use config files in different directories and with different file names, which among other things makes tutorials like this one (https://github.com/MolecularAI/ReinventCommunity/blob/d9fa00faaccb5bf40913d21250aac1dde52cdad7/notebooks/Complete_Use-Case-DRD2_Demo.ipynb) impossible without changing the code by hand.
https://github.com/MolecularAI/Reinvent/blob/b7324d222a49d18b08335a01649abdb0ac66a734/input.py#L11

Sorry for the very late reply but I have taken over the project just recently.
Yes, you are right.  We are currently reworking in how REINVENT will work in the future so we will also fix this.


Revive snowfall

Summary of Changes
Revives snowfall from last year. Only applies to web (desktop + mobile) as opposed to the native mobile app.
Demo or Before/After Pics
https://www.loom.com/share/4a2201aa60ac4e92b1d50f339741e882?sid=d4e8cb82-e76b-4727-899b-04acd9bbfc90
Edge Cases
List any common edge cases that you have considered and tested.
Testing Steps
Provide steps on how the reviewer can test the changes.
Checklist
Please make sure to review and check all of the following:

[x] I've tested the changes and all tests pass.
[x] (if web) I've tested the changes on various desktop screen sizes to ensure responsiveness.
[ ] (if mobile) I've tested the changes on both light and dark modes.

yes

soon :) thanks for the reminder @motzko


Proposal: replace ImmortalDB with direct localStorage calls

ImmortalDB cons

Makes everything Async (due to IndexedDB)
Weird self-healing mechanism (counts values in the stored object/array and picks the biggest one)
The need to duplicate data in two storages is questionable

Replacement
We could replace it with a simple localStorage library that would handle errors when localStorage is not available:

incognito mode in some browsers
Node.js (if we ever introduce server-side rendering)

Let's discuss this in comments.

Makes everything Async (due to IndexedDB)

Why is that a con?

Makes everything Async (due to IndexedDB)

Why is that a con?

Because it makes the code more complicated like we saw with the address book migration.
Now that I think about it, we could make the app wait for the migration promise to finish.

A lot of data we stored is stored offline and users kind of rely on it. I'd say it's very important for us to ensure the data stays. That's why we chose immortaldb in the first place

That's a good argument.

Closing.
Conclusion: there's a good reason for the extra complexity, so we keep it.

Because it makes the code more complicated like we saw with the address book migration.

Now that I think about it, we could make the app wait for the migration promise to finish.
This was a valid point though, we had many race conditions because of storage loads triggered in different places in the codebase. We could turn this ticket to address this problem or create a new one


Using JUnitParams causes NullPointerException

Using JUnitParams for running parameterized tests fail with NullPointerException.
To reproduce the issue you can use next branch
Just deploy RP locally and run mvn test to start com.epam.reportportal.example.junit.logging.JUnitParamsTest or use running via Junit plugin for more detailed stacktrace
15:30:30.079 DEBUG c.n.a.j.RunReflectiveCall - main - beforeInvocation: logPlain(17, false) 0
15:30:30.128 DEBUG c.n.a.j.RunReflectiveCall - main - afterInvocation: logPlain(17, false) 0
15:30:30.021 DEBUG c.n.a.j.RunAnnouncer - main - testStarted: logPlain(17, false) 0
15:30:30.165 DEBUG c.n.a.j.RunAnnouncer - main - testStarted: logPlain(22, true) 1
15:30:30.168 DEBUG c.n.a.j.RunReflectiveCall - main - beforeInvocation: logPlain(22, true) 1
15:30:30.169 DEBUG c.n.a.j.RunReflectiveCall - main - afterInvocation: logPlain(22, true) 1
15:30:30.158 DEBUG c.n.a.j.RunAnnouncer - main - testFinished: logPlain(17, false) 0
15:30:30.164 DEBUG c.n.a.j.CreateTest - main - testObjectCreated: com.epam.reportportal.example.junit.logging.JUnitParamsTest@7645b7d
15:30:30.171 DEBUG c.n.a.j.RunAnnouncer - main - testFinished: logPlain(22, true) 1
15:30:30.179 DEBUG c.n.a.j.Run - main - runFinished: junitparams.JUnitParamsRunner@45eab322
Process finished with exit code 255
java.lang.NullPointerException
at com.nordstrom.automation.junit.RunAnnouncer.isSupported(RunAnnouncer.java:144)
at com.nordstrom.automation.junit.RunAnnouncer.testStarted(RunAnnouncer.java:27)
at org.junit.runner.notification.SynchronizedRunListener.testStarted(SynchronizedRunListener.java:49)
at org.junit.runner.notification.RunNotifier$3.notifyListener(RunNotifier.java:121)
at org.junit.runner.notification.RunNotifier$SafeNotifier.run(RunNotifier.java:72)
at org.junit.runner.notification.RunNotifier.fireTestStarted(RunNotifier.java:118)
at org.junit.internal.runners.model.EachTestNotifier.fireTestStarted(EachTestNotifier.java:42)
at junitparams.internal.ParameterisedTestMethodRunner.runMethodInvoker(ParameterisedTestMethodRunner.java:45)

The version of agent-java-junit that's exhibiting this failure is using an old version of JUnit Foundation. Try upgrading to the latest version with Maven dependency management to see if this improves your results.
I've submitted a PR in agent-java-junit that upgrades JUnit Foundation, but this has not been merged yet.

By the way, where can I find instructions to deploy RP locally?

JUnitParams is a very weird animal. The project is not quite finished.The projected hasn't been updated in more than two years, and it has 25 open issues logged against it. Of special interest is this: https://github.com/Pragmatists/JUnitParams/issues/151
The implementation doesn't quite mesh with the JUnit extension points. However, while watching this example run, I discovered that I had the ability to improve the usability of the ArtifactParams feature dramatically by capturing the ReflectiveCallable object in the AtomIdentity run watcher.

JUnit Foundation version 12.0.0 has been released. I open a pull request against your JUnitParams example branch: https://github.com/reportportal/examples-java/pull/44

I've merged a flurry of revisions to resolve this issue and enhance the usability of the ArtifactParams feature. I published a new release with these revisions and upgraded your branch to use them.

This issue has been resolved.


可以不依赖 Node.js 吗？Can we not rely on node. js

🚀 Feature Proposal
对于后端来说，Node 环境不是必选项，因为这个插件安装 node 显得有点麻烦。能不能移除对 Node 的依赖？
For a backend developer, We are not  need  Node environment all  the time ,and It's a little trouble if  we install  Node  only use for this plugin.Can we not rely on node please?

For now, since the extension is based on leetcode-cli. There is no easy way to remove the dependency on Node.js. We are considering to write our own lib, but this will take times and will not happen in the near future.

Closing it for now.


Add a guide for contributing to code

We should come up with official workflow for contributing to the repository. A diagram would then be nice. Maybe built in mermaid. Example below:
https://github.com/RSE-Sheffield/RSEAdmin - Something like the diagram here.
@mhauru - Did we agree on a contributing policy? I.e. Dev branch?
Probabaly need to think about how this works with the forks for specific branches we're going to have moving forward.

I'd be happy to either copy that RSEAdmin diagram and paragraph verbatim, or just simplify it by removing the dev/main distinction. I think that distinction is very good if we ever have a production deployment of DTBase, but for now it's unnecessary, and I wonder if in the future all the production deployments will come from user-specific forks anyway.


IIS Deployment sub tasks no longer supported

Hi,
I followed the post instructions but I noticed that "IIS Deployment" sub tasks "IIS Web App Manage" and "IIS Web App Deploy" are no longer supported!
I got an error on both of them says "This task is no longer installed. Re-install the task or remove it from your pipeline." which prevent me from saving the CD pipeline.

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

ID: ca1c5394-611b-b119-dbf3-b88404e351d5
Version Independent ID: fcc23259-21ec-f9fa-4e25-f587e1e0f41b
Content: Deploy to an IIS web server on a Windows Virtual Machine
Content Source: docs/pipelines/apps/cd/deploy-webdeploy-iis-deploygroups.md
Product: devops
GitHub Login: @alexhomer1
Microsoft Alias: ahomer

Thank you for your feedback. I have passed this on to the product team for advice.

The post that you are referring to calls out IIS tasks in the Deployment Group phase. Can you confirm you are adding tasks in the Deployment Group phase ("Run on deployment group") as shown in step 5 of creating a release definition. The product team suggest you might be adding the deprecated IIS tasks to the Agent phase instead. You also need to use the Marketplace task
https://marketplace.visualstudio.com/items?itemName=ms-vscs-rm.iiswebapp

Hi @alexhomer1
I confirm it was my mistake. I should run it on  ("Run on deployment group"). Sorry for any inconvenience.
Again thank you and the team on your support.


Add WeightedDegreeRBFKernel parameters for model selection

Please expose the WD-RBF kernel parameters width and degree to model selection.

should be solved with your patch #3463


use recent jsdoc from github

Coverage remained the same at 90.421% when pulling 9f146d892a801f35da0f7ff0f6df033bf337113f on use-recent-jsdoc into 5e5f4ee960ae37deb6153538fdb910afc5ff2903 on master.


Bring back metadata for cells to be non-editable

For the CodeTour extension there used to be feature that looked like this: Tweet with GIF demo.
It's an option that allows you to view the whole tour in one Notebook file to get a top-level view of the tour.
In the example shown in the GIF, the cells are loaded with markdown and code is pulled from the workspace and put into code cells that are ideally not editable.
This used to be supported in previous versions of the API with the cell editable metadata and for this Notebook to work as intended, having this metadata back would be needed to make this Notebook work well.
You can view the PR that shows the usage of Notebooks here: https://github.com/microsoft/codetour/pull/167

+1, Freeze is on our feature roadmap for Jupyter

is on our feature roadmap for Jupyter

I have an extension for this, just edit cell metadata, stopped working after this was removed :(

Is there anyway to edit the metadata for a cell currently (any hack would be appreciated!!!)?

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


updated README.md

current document for minimumIdle property is a bit confusing, below is the statement
If the idle connections dip below this value, HikariCP will make a best effort to add additional connections quickly and efficiently
updated it to
If the idle connections dip below this value and total connections in the pool are less than maximumPoolSize, HikariCP will make a best effort to add additional connections quickly and efficiently.

Coverage increased (+0.05%) to 76.436% when pulling 0bd6630a622f1b9489b8cd6fe390af231f83b817 on udayshnk:dev into ae9d52a36261f057880faec46480cf0e524dd349 on brettwooldridge:dev.


Sample Documentation

It'd be really helpful to have a pointer in your main readme to find any sample (I assume) you have.

Thanks @nibanks, we are working on uploading samples and updating documentation. Will close this when they are updated

Closing this since develop just received a first round of documentation. We will be working to improve this documentation over time as well.

Closing this since develop just received a first round of documentation. We will be working to improve this documentation over time as well.


[question] change css parameters

hi,
Is it possible to change parameters in CSS, for example, of the tags l-info-control?
Inspect style:

I need help with this as well. Were you able to get around to this?

I need help with this as well. Were you able to get around to this?

No. Apparently the repository maintainer doesn't check anything here anymore.

Oh okai, idk why but my colour is coming as blue. Can't seem to change it.

For me too. Did you try to modify the plugin css internally?
They stay in the node_modules folder.

It seems that the background setting is "white", but there is something forcing him to "blue". Are you using another plugin, like vuelayers or vue2-leaflet?

My l-info-control background is also blue. I'm using Vuetify and it seems that the blue is coming from Vuetify's theme for the "info" color.
This .info class in this component also causes me issues with the Vuetify's own "info" CSS. After I have visited on the view with the Vue-Choropleth -component and then move to another view, some of the Vuetify's elements look wrong. This seems to be because Vue-Choropleth .info CSS is remembered by browser and overrides Vuetify's "info" CSS.
Could this .info class be renamed to something else? Or is it possible to set this components style to scoped?


SyntaxError line 135: f'link: {source_path} is not in the '

Getting a syntax error when trying to update a OpenShift bootstrap ignition file as described in: https://blog.openshift.com/openshift-4-bare-metal-install-quickstart/

$ ls -l
total 308
drwxr-x--- 2 psergiu psergiu   4096 Feb 18 10:08 auth
drwxr-xr-x 3 psergiu psergiu   4096 Feb 17 14:25 bootstrap
-rw-r----- 1 psergiu psergiu 292125 Feb 18 10:08 bootstrap.ign
-rw-r----- 1 psergiu psergiu   1833 Feb 18 10:08 master.ign
-rw-r----- 1 psergiu psergiu    104 Feb 18 10:08 metadata.json
-rw-r----- 1 psergiu psergiu   1833 Feb 18 10:08 worker.ign

$ tree bootstrap
bootstrap
└── etc
    ├── hostname
    ├── resolv.conf
    └── sysconfig
        └── network-scripts
            └── ifcfg-ens192

3 directories, 3 files

$ python3 ~/Downloads/filetranspiler-master/filetranspile -i bootstrap.ign -f bootstrap -o bootstrap-static.ign
  File "/home/psergiu/Downloads/filetranspiler-master/filetranspile", line 135
    f'link: {source_path} is not in the '
                                        ^
SyntaxError: invalid syntax

$ python2 ~/Downloads/filetranspiler-master/filetranspile -i bootstrap.ign -f bootstrap -o bootstrap-static.ign
  File "/home/psergiu/Downloads/filetranspiler-master/filetranspile", line 135
    f'link: {source_path} is not in the '
                                        ^
SyntaxError: invalid syntax

Thanks for the report! I'll look into it.

So far I am not able to reproduce this with python3. However, python2 does indeed throw a SyntaxError using the f-strings.
The python3 version I'm testing with is Python 3.7.6. From the above link on f-strings it looks like that feature was added in Python 3.6. Can you provide what Python 3 version you hit this error with?
With that said, it may be worth removing the f-string usage as some folks may not be able to upgrade their Python when their system has something before Python 3.6.0.

Opened https://github.com/ashcrow/filetranspiler/pull/17 which removes f-string usage and replaces it with '..{}..'.format(..). This should allow older Python 3 installs to pass.

I've tagged v1.1.1 which includes the removal of f-strings. Please re-open this if the issue persists or open a new issue if you run into another problem.

Thanks a lot !
This happened on a Debian 9 "oldstable" box with it's default python 3.5
$ python3 --version
Python 3.5.3

The error is no more, seems to be working now. Thank you !


Correct artifactId in dependencies for maven-userdaomodule

In JAVA-16264: review module names was the artifactId in dependencies not changes. This should fix the problem

With this correction is it possible to build and run multimodulemavenproject

PR is valid. This will be taken cared of in https://team.baeldung.com/browse/JAVA-27739. Closing.


fix: handle no config case

Codecov Report

Merging #35 into main will increase coverage by 0.22%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
+ Coverage   65.78%   66.01%   +0.22%
==========================================
  Files           5        5
  Lines         152      153       +1
  Branches       19       20       +1
==========================================
+ Hits          100      101       +1
  Misses         51       51
  Partials        1        1

Impacted Files
Coverage Δ

src/config.ts
100.00% <100.00%> (ø)

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 66d01b4...4ccfaff. Read the comment docs.


Project is alive?

Hi, your project is alive? i dont see update from years

HI @mredodos! Yes, the project is alive. Sorry for the lack of updates.


Issue with date inputs and default browser datepickers

Whenever there's a input with type of date, time or datetime-local the browser (especially Firefox) tries to show the default datepicker and this happens:

You should maybe change the type to text, as per the Materialize documentation

I still have this problem in firefox with version 1.1.17. The tag v1.1.17 does not contain the fix...
Furthermore this happens:


Having multiple AddNpgsqlDbContext calls results in second and subsequent contexts using the wrong connection string.

In the following playground we initialize multiple Postgres databases for use with EF:
https://github.com/dotnet/aspire/tree/main/playground/PostgresEndToEnd
While debugging I noticed that the database was only being created in the first Postgres server. Upon further inspection it looks like all the various DbContext types that are being injected into the / route have the wrong connection string applied (except the first one).
This is probably a bug in the registration of the underlying Postgres connection in DI.

Looks like this impacts other Aspire EF components as well.

This looks like a duplicate of https://github.com/npgsql/efcore.pg/issues/2891.
cc @roji

Yep, agree with @eerhardt that this look like a dup of https://github.com/npgsql/efcore.pg/issues/2891.
Can you please try creating the NpgsqlDataSource yourself, and then explicitly passing that to AddNpgsqlDbContext rather than having it implicitly taken from DI? That should work around the problem for now (and confirm that we're discussing the same bug).

Looks like this impacts other Aspire EF components as well.

I haven't seen this affect other EF components. Can you share a repro of it affecting a non-Npgsql EF component?

This looks like a duplicate of npgsql/efcore.pg#2891.

Looking at @mitchdenny's fix in #2097, I don't think this is same issue. The problem is that we are injecting an NpgsqlDataSource into DI in order to work around https://github.com/npgsql/efcore.pg/issues/2821. However, when you try adding multiple Npgsql DbContexts, we keep overwriting the NpgsqlDataSource in DI, since they aren't "keyed".
@roji - is there another way we can work around this bug? (Or even get the underlying bug fixed?) I don't really like that we are registering NpgsqlDataSource in DI when users add Npgsql EF. Especially since users can start depending on that DI service being registered. With the fix in #2097, it is actually making a breaking change because if users were getting an NpgsqlDataSource from DI, their code would no longer work because it is now keyed on the DbContext type.

Looks like this impacts other Aspire EF components as well.

I haven't seen this affect other EF components. Can you share a repro of it affecting a non-Npgsql EF component?

This was my mistake. Only an issue in npgsql.


Crash with error code 132 - JS out of memory

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

VS Code Version: 1.92.2, fee1edb8d6d72a0ddff41e5f71a671c23ed924b9, x64
OS Version: Fedora 40 (KDE) using 6.9.12-200.fc40.x86_64 kernel

Steps to Reproduce:

Open VSCode
Observe crash

$ code --disable-extensions --verbose
[96237:0902/134853.638897:WARNING:bluez_dbus_manager.cc(248)] Floss manager not present, cannot set Floss enable/disable.
<--- Last few GCs --->

[96237:0x36bc00100000]      334 ms: Mark-Compact (reduce) 6.4 (16.0) -> 6.4 (8.2) MB, pooled: 0 MB, 67.02 / 0.00 ms  (average mu = 0.003, current mu = 0.001) last resort; GC in old space requested
[96237:0x36bc00100000]      338 ms: Mark-Compact (reduce) 6.4 (8.2) -> 6.4 (8.2) MB, pooled: 0 MB, 3.86 / 0.00 ms  (average mu = 0.003, current mu = 0.003) last resort; GC in old space requested

<--- JS stacktrace --->

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
----- Native stack trace -----
[0902/134853.813563:ERROR:elf_dynamic_array_reader.h(64)] tag not found
[0902/134853.813641:ERROR:elf_dynamic_array_reader.h(64)] tag not found
[0902/134853.820566:ERROR:directory_reader_posix.cc(43)] opendir /home/br0kenpixel/.config/Code/Crashpad/attachments/1af0ef5c-fe03-4afe-beff-b033d6b4b175: No such file or directory (2)

Repeating the same command gives me something more verbose:
[96657:0902/135332.675408:WARNING:bluez_dbus_manager.cc(248)] Floss manager not present, cannot set Floss enable/disable.
[main 2024-09-02T11:53:33.006Z] [File Watcher (node.js)] Request to start watching: /home/br0kenpixel/.config/Code/User (excludes: <none>, includes: <all>, filter: <none>, correlationId: <none>),/home/br0kenpixel/.config/Code/User/settings.json (excludes: <none>, includes: <all>, filter: <none>, correlationId: <none>)
[main 2024-09-02T11:53:33.021Z] Starting VS Code
[main 2024-09-02T11:53:33.021Z] from: /usr/share/code/resources/app
[main 2024-09-02T11:53:33.022Z] args: {
  _: [],
  diff: false,
  merge: false,
  add: false,
  goto: false,
  'new-window': false,
  'reuse-window': false,
  wait: false,
  help: false,
  'list-extensions': false,
  'show-versions': false,
  'pre-release': false,
  'update-extensions': false,
  version: false,
  verbose: true,
  status: false,
  'prof-startup': false,
  'no-cached-data': false,
  'prof-v8-extensions': false,
  'disable-extensions': true,
  'disable-lcd-text': false,
  'disable-gpu': false,
  'disable-chromium-sandbox': false,
  sandbox: false,
  telemetry: false,
  debugRenderer: false,
  'enable-smoke-test-driver': false,
  logExtensionHostCommunication: false,
  'skip-release-notes': false,
  'skip-welcome': false,
  'disable-telemetry': false,
  'disable-updates': false,
  'use-inmemory-secretstorage': false,
  'disable-workspace-trust': false,
  'disable-crash-reporter': false,
  'crash-reporter-id': 'f23bc059-693b-420c-bcf6-09933fe13298',
  'skip-add-to-recently-opened': false,
  'open-url': false,
  'file-write': false,
  'file-chmod': false,
  force: false,
  'do-not-sync': false,
  trace: false,
  'preserve-env': false,
  'force-user-env': false,
  'force-disable-user-env': false,
  'open-devtools': false,
  'disable-gpu-sandbox': false,
  '__enable-file-policy': false,
  'enable-coi': false,
  'no-proxy-server': false,
  'no-sandbox': false,
  nolazy: false,
  'force-renderer-accessibility': false,
  'ignore-certificate-errors': false,
  'allow-insecure-localhost': false,
  'disable-dev-shm-usage': false,
  'profile-temp': false,
  logsPath: '/home/br0kenpixel/.config/Code/logs/20240902T135332'
}
[main 2024-09-02T11:53:33.023Z] Resolving machine identifier...
[main 2024-09-02T11:53:33.023Z] Resolved machine identifier: 5c2bcf2b107a455d2246ef01840978865d86b37f2ea0ee9c40581a4c15001262
[main 2024-09-02T11:53:33.023Z] Main->SharedProcess#connect
[main 2024-09-02T11:53:33.028Z] [File Watcher (node.js)] Started watching: '/home/br0kenpixel/.config/Code/User'
[main 2024-09-02T11:53:33.029Z] StorageMainService: creating application storage
[main 2024-09-02T11:53:33.039Z] ElectronURLListener: waiting for window to be ready to handle URLs...
[main 2024-09-02T11:53:33.039Z] lifecycle (main): phase changed (value: 2)
[main 2024-09-02T11:53:33.039Z] windowsManager#open
[main 2024-09-02T11:53:33.040Z] windowsManager#open pathsToOpen [
  {
    backupPath: '/home/br0kenpixel/.config/Code/Backups/1725277079613',
    remoteAuthority: undefined
  }
]
[main 2024-09-02T11:53:33.044Z] windowsManager#doOpenEmpty {
  restore: true,
  remoteAuthority: undefined,
  filesToOpen: undefined,
  forceNewWindow: true
}
[main 2024-09-02T11:53:33.045Z] IPC Object URL: Registered new channel vscode:db0eab28-146c-40f3-a8d4-0008f3df79d3.
[main 2024-09-02T11:53:33.045Z] window#validateWindowState: validating window state on 1 display(s) { mode: 0, x: 0, y: 28, width: 1708, height: 996 }
[main 2024-09-02T11:53:33.045Z] window#validateWindowState: single monitor working area { x: 0, y: 0, width: 1707, height: 1024 }
[main 2024-09-02T11:53:33.046Z] window#ctor: using window state { mode: 0, x: 0, y: 28, width: 1707, height: 996 }
[main 2024-09-02T11:53:33.119Z] window#load: attempt to load window (id: 1)
[96718:0902/135333.124833:WARNING:runtime_features.cc(730)] AttributionReportingCrossAppWeb cannot be enabled in this configuration. Use --enable-features=ConversionMeasurement,AttributionReportingCrossAppWeb in addition.
[main 2024-09-02T11:53:33.129Z] windowsManager#open used window count 1 (workspacesToOpen: 0, foldersToOpen: 0, emptyToRestore: 1, emptyToOpen: 0)
[main 2024-09-02T11:53:33.129Z] lifecycle (main): phase changed (value: 3)
[main 2024-09-02T11:53:33.129Z] resolveShellEnv(): skipped (VSCODE_CLI is set)
[main 2024-09-02T11:53:33.129Z] update#setState idle
[main 2024-09-02T11:53:33.146Z] [File Watcher (node.js)] Started watching: '/home/br0kenpixel/.config/Code/User/settings.json'
[96692:0902/135334.590971:WARNING:sandbox_linux.cc(420)] InitializeSandbox() called with multiple threads in process gpu-process.
[main 2024-09-02T11:53:34.624Z] resolveShellEnv(): skipped (VSCODE_CLI is set)
<--- Last few GCs --->

[1:0x312000578000]     1712 ms: Mark-Compact (reduce) 1.8 (6.4) -> 1.8 (3.4) MB, pooled: 0 MB, 1.04 / 0.00 ms  (average mu = 0.028, current mu = 0.023) last resort; GC in old space requested
[1:0x312000578000]     1713 ms: Mark-Compact (reduce) 1.8 (3.4) -> 1.8 (3.4) MB, pooled: 0 MB, 0.91 / 0.00 ms  (average mu = 0.021, current mu = 0.013) last resort; GC in old space requested

<--- JS stacktrace --->
[96718:0902/135334.840351:ERROR:v8_initializer.cc(811)] V8 javascript OOM (CALL_AND_RETRY_LAST).
[0902/135334.848224:ERROR:directory_reader_posix.cc(43)] opendir /home/br0kenpixel/.config/Code/Crashpad/attachments/9046ad83-0c8f-4b39-85b8-cb8b4544791d: No such file or directory (2)
Renderer process crashed - see https://www.electronjs.org/docs/tutorial/application-debugging for potential debugging information.
[main 2024-09-02T11:53:35.227Z] CodeWindow: renderer process gone (reason: crashed, code: 132)
[main 2024-09-02T11:53:35.632Z] lifecycle (main): phase changed (value: 4)
[96657:0902/135337.677380:ERROR:browser_main_loop.cc(280)] GLib-GObject: ../gobject/gsignal.c:2685: instance '0x13b400e4c3c0' has no handler with id '3746'
[main 2024-09-02T11:53:37.681Z] IPC Object URL: Removed channel vscode:db0eab28-146c-40f3-a8d4-0008f3df79d3.
[main 2024-09-02T11:53:37.682Z] Lifecycle#window.on('closed') - window ID 1
[main 2024-09-02T11:53:37.682Z] Lifecycle#onWillShutdown.fire()
[main 2024-09-02T11:53:37.738Z] Lifecycle#onWillShutdown - begin 'instanceLockfile'
[main 2024-09-02T11:53:37.739Z] storageMainService#onWillShutdown()
[main 2024-09-02T11:53:37.739Z] Lifecycle#onWillShutdown - begin 'applicationStorage'
[main 2024-09-02T11:53:37.739Z] Lifecycle#onWillShutdown - begin 'extHostStarter'
[main 2024-09-02T11:53:37.739Z] Lifecycle#app.on(window-all-closed)
[main 2024-09-02T11:53:37.739Z] Lifecycle#app.on(before-quit)
[main 2024-09-02T11:53:37.739Z] Lifecycle#onBeforeShutdown.fire()
[main 2024-09-02T11:53:37.740Z] [WindowsStateHandler] onBeforeShutdown {
  lastActiveWindow: undefined,
  lastPluginDevelopmentHostWindow: undefined,
  openedWindows: []
}
[main 2024-09-02T11:53:37.740Z] Lifecycle#app.on(will-quit) - begin
[main 2024-09-02T11:53:37.740Z] Lifecycle#onWillShutdown - end 'extHostStarter'
[main 2024-09-02T11:53:37.740Z] Lifecycle#onWillShutdown - end 'instanceLockfile'
[main 2024-09-02T11:53:37.742Z] [File Watcher (node.js)] Request to stop watching: /home/br0kenpixel/.config/Code/User (excludes: <none>, includes: <all>, filter: <none>, correlationId: <none>),/home/br0kenpixel/.config/Code/User/settings.json (excludes: <none>, includes: <all>, filter: <none>, correlationId: <none>)
[main 2024-09-02T11:53:37.742Z] [File Watcher (node.js)] stopping file watcher (/home/br0kenpixel/.config/Code/User (excludes: <none>, includes: <all>, filter: <none>, correlationId: <none>))
[main 2024-09-02T11:53:37.745Z] [File Watcher (node.js)] stopping file watcher (/home/br0kenpixel/.config/Code/User/settings.json (excludes: <none>, includes: <all>, filter: <none>, correlationId: <none>))
[main 2024-09-02T11:53:37.746Z] StorageMainService: closed application storage
[main 2024-09-02T11:53:37.746Z] Lifecycle#onWillShutdown - end 'applicationStorage'
[main 2024-09-02T11:53:37.765Z] Lifecycle#app.on(will-quit) - after fireOnWillShutdown
[main 2024-09-02T11:53:37.765Z] Lifecycle#app.on(will-quit) - calling app.quit()

Running the same command for the 3rd time, makes VSCode run just fine.
Related to
This issue may be related to or duplicate of:

#190560

Possible causes
AMD iGPU drivers could be the cause. I noticed that these crashes are more likely to happen after my laptop was woken up from sleep.

It might be duplicate of #221886 or #224673. See https://github.com/microsoft/vscode/issues/224673#issuecomment-2302332588

I don't think it's related to SELinux, as I couldn't find any denial logs in dmesg, nor journalctl. As for the 6.10.6-200.fc40 kernel, I already have it installed, but unfortunately I can't use it reliably. It causes my system to occasionally crash due to some issue with the AMD iGPU driver.

If the issue still persists with latest stable, can you follow the steps before to capture a heap profile,

From a fresh instance, run Developer: Toggle Developer Tools
Switch to the Memory tab and select the option as shown in the screenshot below

Start the recording, repeat the steps that repro the crash
Save the profile and attach it to this issue