UnobservedTaskException TargetClosedException

Not sure if this a bug or a problem in my code, probably the latter, but I would appreciate any hints to help resolve it.
I have a background process that executes once a day and runs some ops on a website.
After a few hours of the script running, I sometimes get this exception caught by the unobserved task handler:
PuppeteerSharp.TargetClosedException
Target closed (Session closed)

Sadly the exception doesn't contain any other information (stack etc) - it's caught by Sentry, which we use for error tracking.
Both the browser and page are part of using statements so I wouldn't expect this to happen.
Also, I am NOT attaching any event handlers to the page.
Any ideas?
Thanks

Target closed errors are hard to track. They are caused by the tab being closed (or crashed) or the browser getting closed.
There is no generic answer for this.

Okay, I guess I'll ignore them, thanks


More uk brands

add more uk brands A to Z without the long line of code

Looking good, going to merge those!


Missing a libsass requirement.

Thanks... this pushed me into the right direction.
Adding django-libsass into requirements.txt?

This can be closed now I think @bruskiza?
The benefit of using hub for turning issues into pull-requests is that when the PR is closed the issue is gone too. Less house keeping :)


[ISSUE] connectors enabled by default despite README

Describe the issue:
The README states that the connectors are disabled by default. This seems to disagree with source: https://github.com/camunda/camunda-platform-helm/blame/main/charts/camunda-platform/values.yaml#L1760

Actual behavior:
The connectors container is getting pulled.

Expected behavior:
I don't expect any connectors container in the deployment.

How to reproduce:
helm deploy vanilla chart

Logs:

Environment:
Please note: Without the following info, it's hard to resolve the issue and probably it will be closed.

Platform:
Helm CLI version:
Chart version:
Values file:

Hi @nhomble :wave:
Thanks for reporting this :+1:
It's actually the other way around. So I've updated the readme file.


ISTIO External Authorization :  getting error '503 upstream connect error or disconnect/reset before headers. reset reason: connection terminationroot' when access over HTTPS.

ISTIO version:  1.9.4
EKS Cluster version: 1.14
We have deployed ISTIO APP mesh in our project. We have deployed External Authorization using istio's documentation i.e. https://istio.io/latest/docs/tasks/security/authorization/authz-custom/.
External authorizer used (as mentioned in above documentation) : https://raw.githubusercontent.com/istio/istio/release-1.9/samples/extauthz/ext-authz.yaml
When we access any API from going into pod of another API (i.e. over http), using curl command, all works fine. External auth service gets call and all the headers are passed into external authorizer's v3 check method. Below information is passed
source, principal, destination, headers: authority, method, path, accept, content-length, user-agent, x-b3-sampled, x-b3-spanid, x-b3-traceid, x-envoy-attempt-count, x-ext-authz, x-forwarded-client-certx-forwarded-proto, x-request-id.
But when we try to access the same service over https using postman, browser or from going into pod of another API and using curl with https endpoint, we get denied response from external authorizer's v3 check method. Also when we check the logs of external authorizer's v3 check method no headers are passed to it in this case.
Any help to fix this is really appreciated
Below is setup
Name spaces with ISTIO ejection enable : foo
1. ISTIO Config map changes
data:
  mesh: |-
    # Add the following content to define the external authorizers.
    extensionProviders:
    - name: "sample-ext-authz-grpc"
      envoyExtAuthzGrpc:
        service: "ext-authz.foo.svc.cluster.local"
        port: "9000"
    - name: "sample-ext-authz-http"
      envoyExtAuthzHttp:
        service: "ext-authz.foo.svc.cluster.local"
        port: "8000"
        includeHeadersInCheck: ["x-ext-authz"]

2. External Authorizer
apiVersion: v1
kind: Service
metadata:
  name: ext-authz
  namespace: foo
  labels:
    app: ext-authz
spec:
  ports:
  - name: http
    port: 8000
    targetPort: 8000
  - name: grpc
    port: 9000
    targetPort: 9000
  selector:
    app: ext-authz
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: ext-authz
  namespace: foo
spec:
  replicas: 1
  selector:
    matchLabels:
      app: ext-authz
  template:
    metadata:
      labels:
        app: ext-authz
    spec:
      containers:
      - image: docker.io/istio/ext-authz:0.6
        imagePullPolicy: IfNotPresent
        name: ext-authz
        ports:
        - containerPort: 8000
        - containerPort: 9000

3. Enable the external authorization Config
apiVersion: security.istio.io/v1beta1
kind: AuthorizationPolicy
metadata:
  name: ext-authz
  namespace: foo
spec:
  selector:
    matchLabels:
      app: user-api
  action: CUSTOM
  provider:
    name: sample-ext-authz-grpc
  rules:
  - to:
    - operation:
        paths: ["/user/api/*"]

4. PeerAuth Chagnes
apiVersion: security.istio.io/v1beta1
kind: PeerAuthentication
metadata:
  name: mtlsauth
  namespace: foo
spec:
  mtls:
    mode: STRICT

5. Destination Rule
apiVersion: networking.istio.io/v1alpha3
kind: DestinationRule
metadata:
  name: default
  namespace: foo
spec:
  host: "*.samplehost.svc.cluster.local"
  trafficPolicy:
    tls:
      mode: ISTIO_MUTUAL

6. Virtual Service File
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  name: sample-gateway
  namespace: foo
spec:
  selector:
    istio: ingressgateway
  servers:
  - port:
      number: 80
      name: http
      protocol: HTTP
    hosts:
    - "sample.com"
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: user-api
  namespace: foo
spec:
  hosts:
  - "sample.com"
  gateways:
  - sample-gateway
  http:
  - match:
    - uri:
        prefix: /user/api/
    route:
    - destination:
        host: user-api
        port:
          number: 9500

It will be really great if someone can take a look and help me understand if there is issue with above configuration.

 Need help on the same

We are no even using ext authz and I see this in our istio-proxy sidecar logs. @surveamol did you find out the cause ?


[UBS Admin Cabinet ] Admin doesn't see the borders of City field and Street field in 'Waste export address' section after he/she moves from User account to Admin account

Environment: Windows 10 Pro v.21H2, Google Chrome 109.0.5414.120 (64-bit).
Reproducible: always.
Build found: 30/01/23
Preconditions

User is registered as admin.
User is registered as user.
The registered user is logged in.
Orders tab is opened.
User has at least 1 order.

Steps to reproduce

Click on the 'New order' button.
Click on the 'Back' button.
Click on the name in the top right.
Click Sign out.
Click Sign in.
Log in as admin.
Click on the name in the top right
Click UBS-admin.
Click on particular order
Click on 'Waste export address'
Pay attention to the City field.
Pay attention to the Street field.

Actual result
Admin doesn't see the borders of the City field and Street field in the 'Waste export address' section.

https://user-images.githubusercontent.com/119864646/215581615-83d3413b-4d96-493d-8435-7d7b03d37fc6.mp4
Expected result
Admin sees the borders of the City field and Street field in the 'Waste export address' section.

Postconditions
User story and test case links
E.g.: "User story #5101
Labels to be added
"Bug", Priority ("pri: Low"), Severity ("severity:Trivial"), Type ("UI").

Cannot be reproduce


Community menu not visibile fully in the header of site

On the header of reactjs.org, Community option is half-visible. It is overlapped by adjacent Search Box input.

Hi,
Think this issue should be closed now


How to deploy the opensearch  EKS cluster with multi zone in same region in.

Hi Team,
We want to deploy the EKS cluster with https multi zone in same region.
In helm charts what changes need to  be change in values.yaml file.
We added the below lines in our config file.
node.attr.availabilityzone: ${zone}
cluster.routing.allocation.awareness.attributes: zone
cluster.routing.allocation.awareness.force.zone.values: ["us-east-2a","us-east-2b","us-east-2c"]
We are not able to see zone info inside the pod.
Please let us know how to add a zone to environment variables.
Please let us know what changes are to do from our side.
Thanks,
Anil

moved to opensearch-devops

Move to helm-charts.


Change YAML format to Avram specification

Avram specification is still being worked on to get to 1.0.0 but the core parts are settled. With this pull request there is also another scraper to convert MARC21 specification from HTML to Avram. This script can be used to check whether a JSON file is valid Avram. Differences to the current YAML format:

Wrap list of fields in root element fields:
Rename name to label
Remove empty subfields: [] for fixed fields
Use object instead of array for lists of fields and for lists of subfields

Expected format like this:
fields:
  001:
    label: Control Number
    repeatable: false
    tag: '001' # optional
  010:
    label: Library of Congress Control Number
    repeatable: false
    tag: '010' # optional
    subfields:
      a:
        code: a # optional
        label: LC control number
        repeatable: false
      b:
        code: b # optional
        label: NUCMC control number
        repeatable: true
      z:
        code: z # optional
        label: Canceled/invalid LC control number
        repeatable: true
      8:
        code: '8' # optional
        label: Field link and sequence number
        repeatable: true

This is a great idea, and one that must have been bubbling in my subconscious since I had looked at Avram recently!


Day54

Resolves #118 .

Pull Request Test Coverage Report for Build 257

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

Totals

Change from base Build 253:
0.0%

Covered Lines:
24

Relevant Lines:
24

💛  - Coveralls


Deployment via Travis CI

To master-Branch for Github Pages

.github.io structure. No Deployment-Process needed


PV reconciliation error causes PV to be outOfSync (requires pruning) on argocd

Hello SigNoZ team.
I am trying to deploy SigNoZ with ArgoCD and I have the following problem:

ArgoCD is trying to delete a PV that they should not manage and is in a constant loop.

CRD immediatly OutOfSync
ArgoCD chart version: 5.5.8
I found  the following related issues:
https://github.com/Altinity/clickhouse-operator/issues/958
https://github.com/argoproj/argo-cd/issues/3391

any update of this?

any update of this?

I was able to solve/workaround it as per the issue you mentioned - https://github.com/Altinity/clickhouse-operator/issues/958
by creating a ClickHouseOperatorConfiguration resource:
apiVersion: clickhouse.altinity.com/v1
kind: ClickHouseOperatorConfiguration
metadata:
  name: signoz-clickhouse-argocd-pv-nomanage
  namespace: signoz
spec:
  excludeFromPropagationLabels: ["app.kubernetes.io/instance", "argocd.argoproj.io/instance"]
  label:
    exclude: ["app.kubernetes.io/instance", "argocd.argoproj.io/instance"]

Could it possibly be added to signoz chart as configuration parameter in values.yaml?


Bug on form reset

Hi,
autoNumeric 2.0beta have a bug on form reset. After you reset the form and focusin the input the last value returns.
I have created a demo. Just type something, clear form and then click on the input.
https://github.com/rui-ribeiro/autoNumeric-bug
Thanks :)

Thanks you.
Did not catch in my testing - will fix and post

I am hoping you could test this fix - place it above the submit event approx line 1518
				$this.closest('form').on('reset.autoNumeric', function () {
					holder = getHolder($this);
					if (holder) {
						$this.autoNumeric('wipe');
						var reSet = function() {
							$this.autoNumeric('set', $this.val());
						};
						setTimeout(reSet, 100);
					}
				});

It uses setTimeout which I am not thrilled about

That works.
But why not only this:
$this.closest('form').on('reset.autoNumeric', function () {
   $this.autoNumeric('wipe');
});

That works to.

Can you test on multiple browsers?

Well, I only tested on chrome.
Version: 51.0.2704.84 m
But Yes, I will test on different browsers
2016-06-09 12:19 GMT+01:00 Bob Knothe<EMAIL_ADDRESS>
Can you test on multiple browsers?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BobKnothe/autoNumeric/issues/219#issuecomment-224866850,
or mute the thread
https://github.com/notifications/unsubscribe/AJ_dfno-8mYfKAslcNtaPgmHiU5j2RqUks5qJ_bQgaJpZM4IxEPd
.

I have done some testing and the only way to have the default values formatted on reset is the use setTimeout.
Have you found any other solution?

Is this bug still present in the latest next branch?

Due to inactivity, I'm closing this issue considering it resolved.
If that's not the case, feel free to re-open it!

I wish this was a part of autoNumeric instead of us having to do this.

@ducman if you feel that there is a bug in AutoNumeric, please open an issue!

@AlexandreBonneau, can we re-open this issue instead of creating a new one (lazy)?
Here is the codepen outlining the issues.

Form reset does reset the input back to its original value but there is no autonumeric formatting.
After doing above (1), hovering over the input added the formatting, but it format the wrong value. Step to reproduce below.

Step to reproduce for (2)

Go to the above code pen.
Notice the default value (1,234.00).
Modify it to a different value (i.e. 1,111.00).
Click the reset button.
Notice the default value (1234) re-appear on the input but un-formatted.
On hover over the input, the input is formatted with $1,111.00 instead of $1,234.00.

Thanks for the reproducible test case @ducman.
This is indeed a bug :)

Just an FYI, the above sample fix from @BobKnothe, doesn't seem to work. I'm sorry I don't have any example to show but the UI looks fine but on post after a reset, the input value was empty. I think the use of wipe messed it up.
I ended up using something below and I still had to use a setTimeout.
setTimeout(() => anElement.set($('input').prop('defaultValue')), 100);

For info, from the MDN documentation:

You should usually avoid including reset buttons in your forms. They're rarely useful, and are instead more likely to frustrate users who click them by mistake (often while trying to click the submit button).

So I added a fix for this bug, although ideally this should never be used :smirk:


error

链式响应做的太好了，就是那个智能提示不怎么好

Google Translation: Chain Response to do good, not so good is that smart tips.

I'm not sure if this is a suggestion on how to improve the documentation or not, but without more information, I can't be certain. Since there's no actionable item here, I'm going to close this issue out.


Could not find bazel executable-Iris python configuration implementation in windows

@sgowroji when i ran python setup.py install --link-opencv, it is throwing a error as represented below
$ py setup.py install --link-opencv
running install
running build_binary_graphs
could not find bazel executable. Please install bazel tobuild the MediaPipe Python package.
I have followed the installation steps mentioned https://google.github.io/mediapipe/getting_started/install.html#installing-on-windows here to install bazel and also set the environment variables as mentioned below

Hi @sandesh1994199, Could you please share the complete error logs and steps to reproduce it. Thanks!

Hi @sgowroji when i run python3 setup.py gen_protos works fine. Above is the only error I am facing .

Hi @sgowroji $ when i run bazel version, I encounter below error in git bash
bash: bazel: command not found


enha: make finish_pending_block hold write locks for less time

PR Type
Enhancement

Description

Improved performance of finish_pending_block method by reducing the time write locks are held
Reordered operations to minimize lock contention:

Write operations are now performed first, then locks are dropped
Read operations and block cloning are done after releasing write locks

Added error handling for accessing the latest block after finishing the pending block
Simplified code by removing redundant assignments and leveraging existing variables
Maintained conditional compilation for dev features

Changes walkthrough 📝
Relevant filesEnhancement

inmemory.rsOptimize lock usage in finish_pending_block method

src/eth/storage/temporary/inmemory.rs
Reordered operations in finish_pending_block to reduce lock holding time  Moved block cloning outside of write lock  Simplified code by removing redundant assignments  Added error handling for latest block access

+18/-8

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

/benchmark

Benchmark:
Run ID: bench-602209036
Git Info:

Branch: speed_up_finish_pending_block
Commit: 32f912e
PR: #1910

Configuration:

Target Account Strategy: Default

RPS Stats: Max: 1240.00, Min: 821.00, Avg: 1047.87, StdDev: 66.85
TPS Stats: Max: 1234.00, Min: 893.00, Avg: 1015.84, StdDev: 78.42
Plot: View Plot

/benchmark

Benchmark:
Run ID: bench-3109498759
Git Info:

Branch: speed_up_finish_pending_block
Commit: 903e936
PR: #1910

Configuration:

Target Account Strategy: Default

RPS Stats: Max: 1214.00, Min: 867.00, Avg: 1026.50, StdDev: 68.89
TPS Stats: Max: 1186.00, Min: 842.00, Avg: 994.82, StdDev: 75.18
Plot: View Plot

/benchmark

Benchmark:
Run ID: bench-2223728927
Git Info:

Branch: speed_up_finish_pending_block
Commit: 16c897b
PR: #1910

Configuration:

Target Account Strategy: Default

RPS Stats: Max: 1257.00, Min: 866.00, Avg: 1057.30, StdDev: 79.21
TPS Stats: Max: 1266.00, Min: 756.00, Avg: 1024.48, StdDev: 85.28
Plot: View Plot

/benchmark

Benchmark:
Run ID: bench-489202905
Git Info:

Branch: speed_up_finish_pending_block
Commit: 65a1752
PR: #1910

Configuration:

Target Account Strategy: Default

RPS Stats: Max: 1251.00, Min: 706.00, Avg: 1075.05, StdDev: 86.18
TPS Stats: Max: 1266.00, Min: 866.00, Avg: 1042.81, StdDev: 89.92
Plot: View Plot


Implement content submission workflow

Hi @nilandev, I would like to work on the issue. Could you please assign me this issue.


Incorrect description in SCIM bulk group import

Corrected this in doc versions 5.3.0 through 5.7.0.


Schedule Viewer

Need to make sure to have:

ratio perfect time block display
time block labels in the first full cell occupied by a block (don't label them if they don't take up a full block because that would take some time to figure out.), and
clamping + hide/show functionality.

before this can be considered to be done.


Adding URL parameter to Reddit.info

Fixes #798
Feature Summary and Justification
This feature allows for a url to be passed instead of a list of fullnames to Reddit.info in order to collect a list of all submissions to the given url string.

Can you squash these commits?

Updates look good, would just like to reduce the number of commits down to 1 or 2 here.
If you haven't done that before https://stackoverflow.com/a/5189600/176978

Sorry about my floundering about with the squash. For some reason it looks like it's not actually squashing the commits?? I must be doing it wrong somehow, though I'm following how they're saying you're supposed to do it.

No worries, this is not super intuitive the base command you probably want to run is:
git rebase -i d418f9b2d872cdc5837e31e0f1c56391bf7b18e5

d418f9b2d872cdc5837e31e0f1c56391bf7b18e5 is the upstream version of master. If you have a remote upstream that points to praw-dev/praw on github, then the command need only be:
git rebase -i upstream/master

Conversely, if you completed the work on a branch, and not your master branch, then as long as your master branch did not deviate from this project's master branch you could run:
git rebase -i master

Anyhow, give the first one a try, and then for all commits (lines), save for the first, in the edit prompt, replace "pick" with "s".

@bboe thanks! I knew how to do the squash and all, I was just having trouble getting the proper point to rebase from, I guess. Looks like it should be all set now,

Awesome work. Thanks!

Always happy to help!


Tasks defined in GitReleaseManager.cake do not support 2FA

I recently enabled 2FA on my GitHub account and I can no longer execute .\build -t releasenotes. I get the following exception:  Octokit.TwoFactorRequiredException: Must specify two-factor authentication OTP code.

@Jericho yes, this makes sense.  When using GitReleaseManager (and therefore Octokit) when also using 2FA, the recommendation is to use a Personal Access Token, rather than your actual password.  When doing this, the 2FA will not be triggered, and it will work correctly.
Although the wording of the variables in Cake.Recipe suggest that it is a Password that is required, you can actually also pass a Token in as the parameter.  This is what I do, and it works fine.

GitReleaseManager itself was recently updated to indicate that both a username/password, or a token can be used.  Perhaps Cake.Recipe should be updated to reflect this as well.

I didn't realize that I could use my token in lieu of my password. I know that there is an overload for the GitReleaseManagerCreate method where you can specify your token so I was going to edit the task to look like this (this is pseudo-code of the top of my head):
if (!string.IsNullOrEmpty(BuildParameters.GitHub.Token))    // Token would be a new parameter
{
    GitReleaseManagerCreate(BuildParameters.GitHub..Token, BuildParameters.GitHub.UserName, blablabla);
}
else
{
    GitReleaseManagerCreate(BuildParameters.GitHub.UserName, BuildParameters.GitHub.Password, blablabla);
}

Yip, I think that makes a lot of sense, and certainly makes things a lot clearer.


Add external rules file option

Hi,
I've added an option to allow you to specify an external file for rules as suggested here:
https://github.com/jtrupiano/rack-rewrite/issues/6
Let me know if anything needs changing.
Cheers,
Jordan

Hi,
Is this gem still being maintained here or is there another canonical repo?
Cheers,
Jordan


Home assistant doesn't detect Chromecast because pychromecast install fails

I started hass --open-ui from Command Prompt running as administrator and it failed to detect my Chromecast. Over at the console, I could see this error:

INFO:homeassistant.core:Bus:Handling <Event component_loaded[L]: component=media_player>
INFO:homeassistant.core:Bus:Handling <Event platform_discovered[L]: service=google_cast, discovered=('<IP_ADDRESS>', 8009)>
INFO:homeassistant.loader:Loaded media_player.cast from homeassistant.components.media_player.cast
INFO:homeassistant.util.package:Attempting install of pychromecast==0.6.12
Exception:
Traceback (most recent call last):
  File "F:\Python34\lib\site-packages\pip\basecommand.py", line 211, in main
    status = self.run(options, args)
  File "F:\Python34\lib\site-packages\pip\commands\install.py", line 373, in run
    os.remove(target_item_dir)
PermissionError: [WinError 5] Access is denied: 'C:\\Users\\Sathyajith\\AppData\\Roaming\\.homeassistant\\lib\\netifaces.pyd'
ERROR:homeassistant.bootstrap:Not initializing media_player.cast because could not install dependency pychromecast==0.6.12

Maybe something to do with:
PermissionError: [WinError 5] Access is denied: 'C:\\Users\\Sathyajith\\AppData\\Roaming\\.homeassistant\\lib\\netifaces.pyd'

I'm not familiar with python on windows. The netifaces python module shows up more as issue on 'special' platforms.

Can you try removing your lib directory and try agian?
On Tue, Oct 20, 2015 at 9:53 AM, Tom Duijf<EMAIL_ADDRESS>wrote:

Maybe something to do with:
PermissionError: [WinError 5] Access is denied: 'C:\Users\Sathyajith\AppData\Roaming\.homeassistant\lib\netifaces.pyd'
I'm not familiar with python on windows. The netifaces python module shows
up more as issue on 'special' platforms.
—
Reply to this email directly or view it on GitHub
https://github.com/balloob/home-assistant/issues/538#issuecomment-149631179
.

--
PaulusSchoutsen.nl
It's nice to be important but it's more important to be nice.

I removed the lib directory & ran hass --open-ui, unfortunately the same error.
 File "F:\Python34\lib\site-packages\pip\commands\install.py", line 373, in run
   os.remove(target_item_dir)

I guess netifaces.pyd is in use and hence the delete fails. In the meanwhile, I installed pychromecast and copied them to .homeassistant\lib & now it finds the Chromecast

I wish pip would check the modules already installed globally before it attempts a dependency install.
Same problem on RPI and other platforms, where installation fails modules requiring compilation.

Package management might be very well my least favorite part of Home
Assistant.
I've added it as a story in my backlog but can't guarantee when I feel like
working on it.
On Tue, Oct 27, 2015 at 4:24 PM, Tom Duijf<EMAIL_ADDRESS>wrote:

I wish pip would check the modules already installed globally before it
attempts a dependency install.
Same problem on RPI and other platforms, where installation fails modules
requiring compilation.
—
Reply to this email directly or view it on GitHub
https://github.com/balloob/home-assistant/issues/538#issuecomment-151673742
.

--
PaulusSchoutsen.nl
It's nice to be important but it's more important to be nice.

Installing packages globally should work in the dev branch after #601 has been merged.

I just installed Home-assistant tonight (on a Windows 10 machine) and it seems that I'm getting the same Access is denied error.
Any suggestions on how to fix this?

Install pychromecast manually to avoid this problem:
pip3 install pychromecast


Help and organization of work

@tombrothers How can we assist in the development or testing of the EF Core provider for Visual FoxPro? Do you have any guidance for organizing the work or past experience that we can build on? Do you want this help? Can we communicate through ISSUES?

This project is in its very early stages so I’m still trying to figure out the work at the moment.  Mainly, I’m just looking at the VfpEntityFrameworkProvider2 unit tests and trying to pull over any relevant tests, one at a time, to get started.  These tests help provide focus for what code needs to be implemented at a minimum. For example, I started with a couple really simple query tests to see that I could wire everything up to perform those queries.  Implementing more complex query tests could be a good place for someone to help with the code.  I’ve just been looking at the EF Core source to understand what needs to be done to get this provider to work.  Additionally, the VfpEntityFrameworkProvider2 code is used to understand the Vfp specific needs for data access and data manipulation.
I’m open to any ideas for this project and communicating though Issues works for me.

I'm open to colaborate. I will be waiting for issue.

I wanted to use your EntityFrameworkCore.Vfp project in my solution. But the Build of VS gave a dozen errors related to the Dependencies. I am not very good in C#, but maybe you can give me a little explanation how to use EntityFrameworkCore.Vfp in other solutions?

@Valdas13 I created a nuget  package that you can use in your project.
Here is an rough example of how you can use to provider.
namespace ConsoleApp1 {
    public class Program {
        static void Main(string[] args) {
            using(var context = new NorthwindContext(@"C:\Northwind\Northwind.dbc")) {
                var customers = context.Customers.ToList();
            }
        }
    }

    public class NorthwindContext : DbContext {
        public DbSet<Customer> Customers { get; set; }

        private readonly string _connectionString;

        public NorthwindContext(string connectionString) {
            _connectionString = connectionString;
        }

        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder) =>
            optionsBuilder.UseVfp(_connectionString);
    }

    public class Customer {
        public string CustomerId { get; set; }
        public string CompanyName { get; set; }
    }
}

@cvivieca At the moment, I have two coding items planned.  The first item to is to implement an XmlToCursor rewriter to get the XmlToCursorTests to pass.  The second item is to implement migrations.  You could give one of these a try if you want.  Additionally, I can think of two non-coding items that would be helpful.  The first non-coding issue would be helping me build of a list of issues by testing the provider in a production like scenario (only querying and data manipulation at this point) for the purpose of building up a list of issues.  The second non-coding item would be to start creating a wiki for how to use the provider.

@Valdas13 I created a nuget  package that you can use in your project.
Here is an rough example of how you can use to provider.
namespace ConsoleApp1 {
    public class Program {
        static void Main(string[] args) {
            using(var context = new NorthwindContext(@"C:\Northwind\Northwind.dbc")) {
                var customers = context.Customers.ToList();
            }
        }
    }

    public class NorthwindContext : DbContext {
        public DbSet<Customer> Customers { get; set; }

        private readonly string _connectionString;

        public NorthwindContext(string connectionString) {
            _connectionString = connectionString;
        }

        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder) =>
            optionsBuilder.UseVfp(_connectionString);
    }

    public class Customer {
        public string CustomerId { get; set; }
        public string CompanyName { get; set; }
    }
}

Thank you, it works.

As I mentioned in a previous comment, I am trying to use EntityFrameworkCore.Vfp in my simple project that works with Pomelo.EntityFrameworkCore.MySql. Using EntityFrameworkCore.Vfp I tried to use the Add-Migration command and got an exception:
PM> Add-Migration InitialMigration Build started... Build succeeded. System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.  ---> System.TypeLoadException: Could not load type 'Microsoft.EntityFrameworkCore.Internal.SemanticVersionComparer' from assembly 'Microsoft.EntityFrameworkCore, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'.    at Microsoft.EntityFrameworkCore.Design.OperationExecutor..ctor(IOperationReportHandler reportHandler, IDictionary args)    --- End of inner exception stack trace ---    at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor, Boolean wrapExceptions)    at System.Reflection.RuntimeConstructorInfo.Invoke(BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)    at System.RuntimeType.CreateInstanceImpl(BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture)    at System.Activator.CreateInstance(Type type, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes)    at System.Activator.CreateInstance(Type type, Object[] args)    at Microsoft.EntityFrameworkCore.Tools.ReflectionOperationExecutor..ctor(String assembly, String startupAssembly, String projectDir, String dataDirectory, String rootNamespace, String language)    at Microsoft.EntityFrameworkCore.Tools.Commands.ProjectCommandBase.CreateExecutor()    at Microsoft.EntityFrameworkCore.Tools.Commands.MigrationsAddCommand.Execute()    at Microsoft.EntityFrameworkCore.Tools.Commands.CommandBase.<>c__DisplayClass0_0.<Configure>b__0()    at Microsoft.DotNet.Cli.CommandLine.CommandLineApplication.Execute(String[] args)    at Microsoft.EntityFrameworkCore.Tools.Program.Main(String[] args) Exception has been thrown by the target of an invocation. PM>
Maybe it's too early to use this feature ?

@Valdas13 right, it is too early.  I've haven't done any work on migrations yet.


Cannot edit path / extension of extracted images if Emmet is installed

https://forums.adobe.com/thread/1643318

You can do this already.  @monirulalom can you provide the exact steps you're following?
For example, this works for me:

Select an image in the Extract panel
In an HTML file, type <img and then press Space
Use arrows to highlight the "src=Extract Asset..." code hint item, then press Enter
Now I see code like src="images/Layer-2.png", and the images/Layer-2.png part has a blue highlight drawn around it.
I edit the highlighted text so it says, for example, images/test.jpg -- then press Enter.
No newline is inserted (the cursor is positioned after the closing "), and the file is generated in my project under the 'images' folder.

Does the behavior differ if you follow those steps?  If not, what steps cause a problem for you?
If you're seeing odd behavior when you press Enter, check if you have the Emmet extension installed -- that has been known to cause problems with code hints before...

When I press ENTER a newline is interted. But It works fine when I disable EMMET extension.@peterflynn

That's still broke in 1.3 and emmet 1.2.2:( Disabling emmet helps as monirulalom mentioned.

@prafulVaishnav Could you take a look at this and see if emmet extension is overriding anything that is causing this behavior?

Deprecated feature See details . Closing.


Bluetooth: controller: allow to cancel per adv after ull_scan_setup

ll_sync_create_cancel called after ull_scan_setup and before
periodic advertising synchronization estalished will end
with BT_HCI_ERR_CMD_DISALLOWED error.
It should be possible to call the cancel function after
until periodic advertising synchronization is established
(first AUX_SYNC_IND pdu is received after call to
ll_sync_create).
The commit moves clearing scan->per_scan.sync to
ull_sync_established_report from ull_scan_setup.
That allows to call the function until sync is
successfully established.
Signed-off-by: Piotr Pryga<EMAIL_ADDRESS>

This is a WIP PR that provides part changes required to solve issue: https://github.com/zephyrproject-rtos/zephyr/issues/40204.
@cvinayak you can use it for further changes required to solve the issue.


Update image_store.rb

Solve issue where figures exist but are not referred to -- in this case @tmp_path is not created and code may fail.

Hello,
Thank you for your correction, it has been merged.
Regards.


argument --version NameError: global name 'versions' is not defined

When I used argument --version the outpud looked like (tried on Fedora 28):
version: 6.1.0-dev
LLVM version: 4.0.1
Traceback (most recent call last):
  File "./symbiotic", line 219, in <module>
    tool, sources, opts, environment = setup_symbiotic()
  File "./symbiotic", line 170, in setup_symbiotic
    opts, args = parse_command_line(environment)
  File "/.../symbiotic/install/lib/symbioticpy/symbiotic/options.py", line 201, in parse_command_line
    print_versions()
  File "/.../symbiotic/install/lib/symbioticpy/symbiotic/options.py", line 119, in print_versions
    for (k, v) in versions.items():
NameError: global name 'versions' is not defined

A fix should be in master now. Can you check? Thanks.

Ok, now the error is no longer present, thanks.


Sounds are not very clear

I tried to run the lib with
var text = "Good Morning, friends, how are you.";
console.log(" text: ", text);
let out1 = await text2wav('测试', {voice: 'zh'});
console.log(" out1: ", out1);
fs.writeFileSync("testch.wav", Buffer.from(out1.buffer));
let txtWav = await text2wav(text);
console.log(" txtWav: ", txtWav);
fs.writeFileSync("testfile.wav", Buffer.from(txtWav.buffer));

then I play the wav files, both played ok, but both the sounds are not clear, anyway to configure the lib to make the sounds more accurate?
Thanks

I tried to run the lib with
var text = "Good Morning, friends, how are you."; console.log(" text: ", text); let out1 = await text2wav('测试', {voice: 'zh'}); console.log(" out1: ", out1); fs.writeFileSync("testch.wav", Buffer.from(out1.buffer)); let txtWav = await text2wav(text); console.log(" txtWav: ", txtWav);
fs.writeFileSync("testfile.wav", Buffer.from(txtWav.buffer));

then I play the wav files, both played ok, but both the sounds are not clear, anyway to configure the lib to make the sounds more accurate?
Thanks

How did you get this managed ?? Urgent help please


Show Pdf in Recyclerview

 Issue by AhmadullahSaikat
Wednesday May 16, 2018 at 07:10 GMT
Originally opened as https://github.com/googlesamples/android-PdfRendererBasic/issues/24

Did anyone implement this code for recyclerview.

 Comment by aayushthakur
Wednesday Jul 18, 2018 at 07:20 GMT

I have implemented this.

 Comment by hunter-t
Friday Aug 10, 2018 at 07:31 GMT

@aayushthakur That would be very appreciated if you could share your solution with people.

 Comment by AhmadullahSaikat
Sunday Jul 22, 2018 at 10:45 GMT

Can you please send me the source code.
my email address is<EMAIL_ADDRESS>

 Comment by LaatonWalaBhoot
Monday Oct 29, 2018 at 11:42 GMT

This is my implementation of loading PDF in Recycler View with pinch zooming. Only API 21+. Will be looking for anyone to contribute to make a legacy version and improve it further...
https://github.com/LaatonWalaBhoot/PdfRedererRecyclerView

 Comment by srgiitd
Saturday Mar 09, 2019 at 11:38 GMT

The sample given by @LaatonWalaBhoot is in Kotlin. Has anybody done it with java? Please share. Thanks.

 Comment by LaatonWalaBhoot
Saturday Mar 09, 2019 at 20:02 GMT

I can do this in Java as well... @srgiitd


fix(cleaner); fix cleaning partitions before wiping main disk

Signed-off-by: Akhil Mohan<EMAIL_ADDRESS>Pull Request template
Why is this PR required? What issue does it fix?:
When an application creates partitions on a claimed disk, currently only the partition table is wiped as part of
cleanup. There are chances that filesystem can exist on the partitions.  In such a scenario, creating another partition
starting at the same sector as the earlier one, causes system picking up the filesystem signature from the
previous partition and using it.
What this PR does?:
To fix the above issue, first the filesystem signature in the partitions need to be wiped and then the partition table.
This PR modifies the wipefs command to list all the partitions of the claimed disk and wipe each one of them, starting with the FS signature in the partitions. Also partprobe command is invoked so as to re-read the partition table on the disk after wiping.
Does this PR require any upgrade changes?:
No
If the changes in this PR are manually verified, list down the scenarios covered::
Yes. Created a cstor pool on a raw disk. Then deleted the pool. After that partition was manually created. System is not reporting
any known filesystems, earlier zfs_member filesystem was reported.
Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[ ] Fixes #
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[x] Has the change log section been updated?
[ ] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

Codecov Report

Merging #445 into master will decrease coverage by 0.23%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #445      +/-   ##
==========================================
- Coverage   43.49%   43.26%   -0.24%
==========================================
  Files          61       61
  Lines        2968     2968
==========================================
- Hits         1291     1284       -7
- Misses       1575     1580       +5
- Partials      102      104       +2

Impacted Files
Coverage Δ

cmd/ndm_daemonset/probe/udevprobe.go
60.56% <0.00%> (-2.82%)
:arrow_down:

cmd/ndm_daemonset/probe/eventhandler.go
10.82% <0.00%> (-1.55%)
:arrow_down:

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 77ee99c...afbf3b1. Read the comment docs.

looks good. test the command for other type of disks also (like nvme and any other we have).

Already tested on nvme drive. It works as expected.

Already tested on nvme drive. It works as expected.

Awesome!!


test-bot does not recognize renamed tap

Hi,
Going by @dawidd6's homebrew-tap workflows, I noticed that the command does not pick up my tap, and defaults to Homebrew/core.
Here's the failing yaml:
https://github.com/ddelange/brewformulae/blob/594254e1cbd2c4a906f3308d2633e838fd7f6bcb/.github/workflows/brew-test-bot.yml
Here's an example run of the command correctly picking up the tap it's in:
https://github.com/dawidd6/homebrew-tap/runs/975810574#step:4:7
My hunch is that the command looks for the homebrew- prefix in the repo it's in, but doesn't find it on my repo, because it's no longer called https://github.com/ddelange/homebrew-brewformulae (and instead redirects, so no problem for brew when I'd do a brew install ddelange/brewformulae/yt.
Any chance I can trick the test-bot command into picking up my tap, and doing the checks?

Couldn't you just rename your repo to something homebrew- prefixed?

Yes of course that would be fallback, but I actually quite liked the looks of ddelange/brewformulae, and having a repo called ddelange/homebrew-brewformulae somehow sounds unnecessary.
If it's nothing I can fix by e.g. passing certain command line args (or anything you can think of), feel free to close, as probably not many ppl have taps that don't start with homebrew- prefix.

Did you try passing --tap option with proper argument to brew test-bot?

Yes, brew test-bot --tap ${{ github.repository }} which expands to ddelange/brewformulae seems to be ignored

Note brew tap ddelange/brewformulae only works because it's using the repository redirection from ddelange/homebrew-brewformulae.

I'd strongly recommend renaming your tap to something with a homebrew- prefix.


drupal-vm:4.8.1 Docker image fails to build when php7.1-bz2 is added in

Issue Type

Bug Report / Support Request

Your Environment
Vagrant 2.0.2
VirtualBox 5.2.6r120293
Docker version 18.03.0-ce, build 0520e24

Your OS

macOS (version)

Full console output
Setting up php7.1-bz2 (7.1.16-1+0~20180405085419.16+stretch~1.gbp4937aa) ...

Creating config file /etc/php/7.1/mods-available/bz2.ini with new version
Setting up php7.1-cli (7.1.16-1+0~20180405085419.16+stretch~1.gbp4937aa) ...
Setting up php7.1-dev (7.1.16-1+0~20180405085419.16+stretch~1.gbp4937aa) ...
Setting up php7.1-fpm (7.1.16-1+0~20180405085419.16+stretch~1.gbp4937aa) ...

Configuration file '/etc/php/7.1/fpm/pool.d/www.conf'
 ==> Modified (by you or by a script) since installation.
 ==> Package distributor has shipped an updated version.
   What would you like to do about it ?  Your options are:
    Y or I  : install the package maintainer's version
    N or O  : keep your currently-installed version
      D     : show the differences between the versions
      Z     : start a shell to examine the situation
 The default action is to keep your current version.
*** www.conf (Y/I/N/O/D/Z) [default=N] ? dpkg: error processing package php7.1-fpm (--configure):
 end of file on stdin at conffile prompt
Errors were encountered while processing:
 php7.1-fpm
E: Sub-process /usr/bin/dpkg returned an error code (1)
ERROR: Service 'myproject' failed to build: The command '/bin/sh -c apt-get update && apt-get install -y php7.1-bz2' returned a non-zero code: 100

Summary
Starting 2018-04-05 my Drupal-VM Docker builds are failing because of a configuration conflict with '/etc/php/7.1/fpm/pool.d/www.conf'.  In my Dockerfile my dependency installation block is as follows:
RUN apt-get update && apt-get install -y \
    apt-utils \
    imagemagick \
    jq \
    nano \
    php7.1-bz2 \
    wget \
    && apt-get autoremove

When I remove php7.1-bz2 from the dependencies, the build completes without errors, but I am unable to complete a composer install on the project since phantomjs requires the bz extension:
jakoch/phantomjs-installer 2.1.1-p07 requires ext-bz2 * -> the requested PHP extension bz2 is missing from your system
I have tried calling the bz2 package as both php7.1-bz2 and php-bz2, but without luck in either case.
I have done some research and it looks like I can override the config issue in the build process by explicitly setting the configuration options to use (apt-get --yes --force-yes -o Dpkg::Options::="--force-confdef" -o Dpkg::Options::="--force-confold"), but this feels like a brute force approach to the issue.
Is there a cleaner way to make sure that the bz2 extension is available in the drupal-vm docker container?

Also, can you post a complete Dockerfile I can use to reproduce this issue (removing any proprietary bits of course)?

Using this Dockerfile worked:
FROM geerlingguy/drupal-vm:latest

RUN apt-get update && apt-get install -o Dpkg::Options::="--force-confold" -y \
    php7.1-bz2 \
    && apt-get autoremove

So it looks like just adding -o Dpkg::Options::="--force-confold" to the install command should get you there. Please re-open if this is not the case!

@geerlingguy confirmed: adding -o Dpkg::Options::="--force-confold" allowed my build to complete without errors. Thanks for taking a look!


fixed deleting repositories doenst always work

This commit fixes a bug when deleting repository (repositories). Because you navigate away immediately after the DELETE request is issued, it gets interrupted and its just a matter of luck, whether the repository gets deleted or not.
http://grab.by/IuDm

Perfect! Merged! Thank you very much @bydga. I'm curious if the other service methods are also affected and need to be fixed.

@bydga Do you agree with merging PR #54 ? It should fix a potentially similar problem to this one except  when deleting a tag instead of a repo.

Hi, thanks for merging, however it doesnt seem to solve issue i was trying to fix :)
Im having difficulties when running the frontend locally and testing my changes.. npm install inside the container (i followed your steps on https://github.com/kwk/docker-registry-frontend/blob/master/develop/README.md) takes like 50mins. :D Might be some disk-driver issue when mounting the volume in the compose.yml and the fact that i run it on mac with docker-machine.
Anyways, the problem im trying to (primarily) solve is when ticking multiple images on the overview page and then deleting them - i guess there will be the same problem as the one i "solved" in the PR. Im digging into it now

aaa, i think i see it now. i didnt notice this loop: https://github.com/kwk/docker-registry-frontend/blob/master/app/scripts/controllers/delete-repository-controller.js#L18
so now, the code is exitting (setting win.loc.herf) in the first iteration :/

@bydga I've reverted your change to not change the overall intended behavior of deleting multiple repos at once in a loop. Please find another way to fix this. Thank you in advance.

@bydga I've updated the development instructions. You should cd into the directory and call docker-compose from there. Otherwise everything in your current directory will be send to the docker build context.

Thanks, i've ended up running it locally (on OSX) just via
node_modules/grunt-cli/bin/grunt serve --allow-root
pushstate and urls are sometimes not working (during restarts) but i can live with that :)
I have the fix done, expect PR in a while - i need to rebase it on the top of your revert..
btw - as other ppl reported you the missing styles: When i run the above mentioned grunt serve, the index.html changes and bootstrap.css is gone:
bydga@bydga-mac ~/Documents/projects/docker-registry-frontend (deletefix) $ git s
On branch deletefix
nothing to commit, working directory clean
bydga@bydga-mac ~/Documents/projects/docker-registry-frontend (deletefix) $ node_modules/grunt-cli/bin/grunt serve --allow-root
....blabla....
Running "watch" task
Waiting...
^C

bydga@bydga-mac ~/Documents/projects/docker-registry-frontend (deletefix) $ git s
On branch deletefix
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

	modified:   app/index.html

no changes added to commit (use "git add" and/or "git commit -a")
bydga@bydga-mac ~/Documents/projects/docker-registry-frontend (deletefix) $ git diff
diff --git a/app/index.html b/app/index.html
index a135a84..d88f108 100644
--- a/app/index.html
+++ b/app/index.html
@@ -9,7 +9,6 @@
     <!-- Place favicon.ico and apple-touch-icon.png in the root directory -->
     <!-- build:css(.) styles/vendor.css -->
     <!-- bower:css -->
-    <link rel="stylesheet" href="bower_components/bootstrap/dist/css/bootstrap.css" />
     <link rel="stylesheet" href="bower_components/toastr/toastr.css" />
     <link rel="stylesheet" href="bower_components/angular-loading-bar/build/loading-bar.css" />
     <!-- endbower -->
bydga@bydga-mac ~/Documents/projects/docker-registry-frontend (deletefix) $

Might help you with the other bug :)


Module resolution behaves differently when typeRoots is specified

🔎 Search Terms
typeRoots, module resolution, moduleResolution, bundler
🕗 Version & Regression Information

This changed between versions 5.0.4 and 5.1.3 (variants with Preserve since 5.4.2)

⏯ Playground Link
No response
💻 Code
No response
🙁 Actual behavior
Setting typeRoots to ./node_modules/@types changes how modules are resolved when using the following compiler options:

moduleResolution: Node16, module: Node16
moduleResolution: Node10, module: Preserve
moduleResolution: Bundler, module: ES2015 | ES2020 | ES2022 | ESNext | Preserve

🙂 Expected behavior
Setting typeRoots to ./node_modules/@types should not change how modules are resolved.
Additional information about the issue
Clone this gist and execute setup.sh to see a minimal reproduction. It sets up a package with various export types, and tries to compile a file with corresponding imports using various module/moduleResolution combinations. Not all imports are expected to work for all combinations, but the errors are expected to be the same regardless of whether typeRoots is specified. If the errors are different, the script will log the errors along with the compiler options used. (No output from the script means there were no mismatched errors.)
Sample output:
{ module: 'Preserve', moduleResolution: 'Bundler' }
=== no typeRoots ===
index.ts(1,22): error TS2307: Cannot find module 'pkg' or its corresponding type declarations.
index.ts(2,30): error TS2307: Cannot find module 'pkg/index' or its corresponding type declarations.
index.ts(3,32): error TS2307: Cannot find module 'pkg/index.js' or its corresponding type declarations.
index.ts(4,32): error TS2307: Cannot find module 'pkg/internal' or its corresponding type declarations.
index.ts(5,34): error TS2307: Cannot find module 'pkg/internal.js' or its corresponding type declarations.
index.ts(6,32): error TS2307: Cannot find module 'pkg/external' or its corresponding type declarations.
index.ts(7,34): error TS2307: Cannot find module 'pkg/external.js' or its corresponding type declarations.
index.ts(8,31): error TS2307: Cannot find module 'pkg/exports' or its corresponding type declarations.

== with typeRoots ==
index.ts(8,31): error TS2307: Cannot find module 'pkg/exports' or its corresponding type declarations.

{ module: 'Preserve', moduleResolution: 'Node10' }
=== no typeRoots ===
index.ts(1,22): error TS2307: Cannot find module 'pkg' or its corresponding type declarations.
index.ts(2,30): error TS2307: Cannot find module 'pkg/index' or its corresponding type declarations.
index.ts(3,32): error TS2307: Cannot find module 'pkg/index.js' or its corresponding type declarations.
index.ts(4,32): error TS2307: Cannot find module 'pkg/internal' or its corresponding type declarations.
index.ts(5,34): error TS2307: Cannot find module 'pkg/internal.js' or its corresponding type declarations.
index.ts(6,32): error TS2307: Cannot find module 'pkg/external' or its corresponding type declarations.
index.ts(7,34): error TS2307: Cannot find module 'pkg/external.js' or its corresponding type declarations.
index.ts(8,31): error TS2307: Cannot find module 'pkg/exports' or its corresponding type declarations.

== with typeRoots ==
index.ts(8,31): error TS2307: Cannot find module 'pkg/exports' or its corresponding type declarations.

Can you specify exactly one config which you think should behave differently? We're also not super keen on running .sh files; a concrete set of flat files which we can copy locally would be preferred.

Here's a proper repo you can clone. It's not quite flat because typeRoots needs a directory structure.
This is the tsconfig.json from the repo:
{
  "files": ["index.ts"],
  "compilerOptions": {
    "noEmit": true,

    // When "typeRoots" is not specified, this project has 8 compiler errors.
    // When "typeRoots" is specified, this project has 1 compiler error.
    // "typeRoots": ["./node_modules/@types"],

    "module": "Node16",
    "moduleResolution": "Node16",
    // Preserve/Node10, Preserve/Bundler, and ES2015+/Bundler all have
    // differences in behavior depending on the presence of "typeRoots".
    // "moduleResolution": "Node10",
    // "moduleResolution": "Bundler"
    // "module": "Preserve",
    // "module": "ES2015",
    // "module": "ES2020",
    // "module": "ES2022",
    // "module": "ESNext",
  }
}

I also noticed that different combinations of "type": "module" or "type": "commonjs" in the root package.json and in typeRoots/pkg/package.json result in different errors being reported when you have typeRoots set in the tsconfig. That's expected behavior. However, if you don't have typeRoots set, then the errors are always the same. Which seems weird.

Root package
typeRoots package
Errors

cjs
cjs
TS2307

cjs
esm
TS1479, TS1479, TS1479, TS1479, TS1479, TS1479, TS1479, TS2307

esm
cjs
TS2307, TS2307, TS2307, TS2307

esm
esm
TS2307, TS2307, TS2307, TS2307

When typeRoots is not specified, the set of errors is "TS2307, TS2307, TS2307, TS2307, TS2307, TS2307, TS2307, TS2307" in all four scenarios.

Perhaps unsurprisingly, I've found that it's the presence of "exports" in the package.json that is causing the issue. When that key is removed, then the behavior of tsc is the same regardless of whether typeRoots is specified.

I think this is a duplicate of https://github.com/microsoft/TypeScript/issues/47444. I tried to fix it at https://github.com/microsoft/TypeScript/pull/58638 and the breaks looked untenable. It may be worth experimenting with resolving "exports" first and falling back to resolving "types", but I held off since non-type-reference module resolution does not continue with fallbacks if "exports" resolution fails, and keeping that consistency with the spec is a plus.

Marking as a duplicate and scheduling another crack at #47444 for 5.8. (Any fix we can possibly do here is going to be a breaking change, so it’s too late for 5.7.)


[PE-D][Tester B] Improvement for undo commands

When undo a deletion on certain row in note,

The application won't show the specific details of the undo object, instead, it displays nothing. It will be better to show the full details of the modified object, reducing the hassle for users to navigate back to object again.

Labels: type.FeatureFlaw severity.VeryLow
original: WangCheng0116/ped#4

Agreed on this! It would help to give greater clarity to the user on what has happened.

Feature Flaw and not Bug


remove dead mailing list from resources recommended to newbies

it is not desirable to fragment community - nearly everybody is using forum in Poland
Facebook group has some limited value for people not willing to join forum who already have FB accounts
there is no compelling reason to recommend mailing list
followup to #52 (ping @RicoElectrico )

To compare activity:
https://lists.openstreetmap.org/pipermail/talk-pl/
https://forum.openstreetmap.org/viewforum.php?id=23

Sounds good @matkoniecz - I think you are doing the right thing! 👍


Adding a Contributor License Agreement

I just wanted to give a heads up that new pull requests will now have a status for the "GitHub CLA". Think of a CLA like an open source software license from the contributor to the project, letting us and other users know that we have permission to use the contributed code.
For first pull request that someone makes to this repo, contributors will have to click through the failure link to the CLA and accept the agreement. Any future pull pull requests will immediately go green.
This is a new process that we are just experimenting with. We want to find the right balance of protecting the projects we maintain without adding too much friction for contributors. I welcome any and all feedback.
/cc @arfon @pchaigno @larsbrinkhoff

:zap:

I fully understand that GitHub would need something like this.  As a GNU contributor, I find your suggestion rather light-weight.  The only complication would be when adding sample files.  Do we need to get an explicit permission for those?

Is the license agreement text available online for empoyers to review?  I mean, other than me copy-pasting it and emailing my boss.

:+1: I always found it weird that no CLA was required.
Will this be required for all GitHub projects from now on or only Linguist?
Shouldn't contributors who made their contribution before today be required to "sign" it?

I had this thought...
To encourage contributions, once a PR is accepted, the contributor could be invited to some kind of "GitHub contributor" organization.  This would add a bit of flair to the user profile.

As a GNU contributor, I find your suggestion rather light-weight.

What do you mean?

Is the license agreement text available online for empoyers to review? I mean, other than me copy-pasting it and emailing my boss.

https://cla.github.com/
It currently requires being signed in and authorizing the app, but I'll get rid of that.

Will this be required for all GitHub projects from now on or only Linguist?

We will be adding it to all GitHub projects soon.

Shouldn't contributors who made their contribution before today be required to "sign" it?

Ideally, yes, but it will be very tedious and to chase them all down. Right now we are focusing on getting CLAs for future work. We may revisit for previous contributions later.

To encourage contributions, once a PR is accepted, the contributor could be invited to some kind of "GitHub contributor" organization. This would add a bit of flair to the user profile.

That's a cool idea. I'll see if I can get some traction for that.

As a GNU contributor, I find your suggestion rather light-weight.
What do you mean?

A GNU contributor is required to sign an actual dead-tree paper and send it by snail mail.  In light of that, the GitHub CLA process is quite smooth.

:+1: That's our goal.


Fix [ROS] support for repos with multiple packages

Follow up to #8169 to fix an issue reported in https://discourse.ros.org/t/shields-io-badge-service/26379/3. Some repositories have multiple packages, in which case a packages array is present. We need to search these arrays for the requested package name, not just the keys under repositories.
cc @ijnek @cst0

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @jtbandes!

  Generated by :no_entry_sign: dangerJS against 2914088c7439fc5d48833ac9937ef5acda20aaf1

I'm not familiar enough with JS and this project to do a proper review, but the changes look like they address the exact isssue, thanks @jtbandes!

I realize there's an unusual level of complexity here given the subsequent schema validation, but I'm having trouble following what the expected behavior is relative to the schema shared in the tests.
From what I can tell if a repository has multiple packages, all the packages will have the same version as dictated by the release key on the repository?
Are package names guaranteed unique across the system or is there a namescoping function of sorts wherein the package names are only unique within a given repository?

What it mainly comes down to is that packages and repositories are different things in the ROS ecosystem. It's very common to see one repository hosting multiple interconnected packages: for example, the standard robot navigation stack is made up of several packages (global path planning, local path planning, costmaps, recovery mechanisms) which each share a single repository (ros-planning/navigation). It's also common to see an entire robot stack sharing a repository (e.g., the Fetch) to allow installation of single packages while keeping the build-from-source overhead low.

From what I can tell if a repository has multiple packages, all the packages will have the same version as dictated by the release key on the repository?

Releases of a package are determined by the package.xml found within each package. I've only ever seen that correspond to a git tag across the repository, but the tag isn't what makes a version and you could probably produce a repository with multiple package versions. I've never tried that myself though.
However, what is very common is different versions targeting different ROS distributions. For example, I have a repository with one package, released for ROS kinetic and ROS melodic. Kinetic is EOL so it's no longer allowing updates, but Melodic is actively accepting contributions. So these will have different version numbers (despite being the same repository). So that would be a case where there are different versions despite one relevant tag. This case is already covered in the badge though, since it takes a ROS distribution as a parameter.

Are package names guaranteed unique across the system

Yes. Packages are made available via the OSRF's apt repository, so duplicate names would be rejected. However, #8169 takes a string pointing to a repository: if I create my own "navigation" repository with zero package name overlap, it wouldn't break anything ROS-side. What would the shield behavior be?
My question would be: is a badge intended to represent a package, or a repository? Badges in #8169 take in a package name, but they really look for a repository which happens to have the same name as the package. Provide an actual package and the badge breaks. This PR appears to fix that problem by now looking explicitly at a package. The alternative solution would be to point at a repository by looking for a full URL (note that not all ROS packages are on GitHub). But, this is nonideal partly for the previously-mentioned cases.

Thanks @cst0

Releases of a package are determined by the package.xml found within each package

If there's a desire to provide package-level version badges, then this would be the data source we would want to go after, and I still don't see that happening.
As previously submitted in #8169, we're driving everything based off the contents of a yaml file retrieved using a tag, and afaict that yaml (or at least the parts of it that have been shared here with us non-ROS users) only includes versions for a repository object. The adjustment proposed in this PR seems to simply be allowing the parameter to be a package name instead of the repository name, but the result is still the version of the repository.
Assuming that's correct, I'd suggest we repurpose the existing badge (even if it's just name changes) to be explicit that it's a repository version badge, if such a thing is valuable for your community. If there's then a desire for package version badges, then we'd need to have a separate discussion around how Shields would be able to go about getting the version data from the correct, authoritative location (wherever the package.xml files are)

The yaml file you're referring to is already informed by the package.xml files within a package. When a package is released in the ROS ecosystem, the final step is to list it as available-for-installation by making a pull request in the relevant yaml file. This is all an automated process (via the bloom tool).

Perhaps I'm missing something.
If there's any documentation that covers all of this that would be most helpful, but I'd never heard of your service until #8169, so that PR plus this PR is the totality of information I have to work with. We integrate with hundreds of services and don't really need to understand much about them beyond their API surface that provide the data point(s) we need to retrieve the values we plug into the badge text.

My question would be: is a badge intended to represent a package, or a repository?

I'm not sure who this question was directed to, but I hope it isn't me/the Shields team. We can't answer this question,and it's the type of thing we ask when people want us to provide badges for some service/platform they use.
From what I can tell, the only version data point that's available in the data source that's been shared with us is the version key for a repository release in the yaml file. For example, the test added here as a demonstration of getting a version for a package:
https://github.com/badges/shields/pull/8271/files#diff-6b1a75d06bae90b956da0e6dafbd6d99f53158fb9bbfd4be8f2d657fb94eb195R33-R52
  navigation_2d:
    doc:
      type: git
      url: https://github.com/skasperski/navigation_2d.git
      version: noetic
    release:
      packages:
      - nav2d
      - nav2d_exploration
      - nav2d_karto
      - nav2d_localizer
      - nav2d_msgs
      - nav2d_navigator
      - nav2d_operator
      - nav2d_remote
      - nav2d_tutorials
      tags:
        release: release/noetic/{package}/{version}
      url: https://github.com/skasperski/navigation_2d-release.git
      version: 0.4.3-1
    source:
      type: git
      url: https://github.com/skasperski/navigation_2d.git
      version: noetic
    status: maintained

where, as defined in that test yaml snippet, navigation_2d is the repository. It's release key contains multiple packages, but there's just one version
The point I'm driving at is that we only see that singular version key so that's the only data point we can use for a badge. It's still unclear to me whether individual packages are versioned independently, but regardless, we can only work with the data that's available and the yaml file is the only one that's been shared with us so far.
As such, it seems to me that it's only possible for us to display a repository version badge. Was the goal/expectation of these changes simply to be able to use a package name instead of the repository name in the badge url, but still end up with the exact same badge (the 0.4.3 version from the example above)?
If so then I'd most likely be opposed to these changes. We've already got a fairly complex and time sensitive process (sequential network calls with processing) just to get to the correct yaml file, and these changes would only add more work we have to perform as part of each and every request. It also adds even more complexity to the codebase, and the only benefit I could see is to provide a minor convenience to users for a one time action (i.e. create badge url, copy/paste into readme). Those tradeoffs aren't worth it in IMO, and I'd much rather just rebrand things on our badge to make it clear it's working with repositories, not packages.
However, if I've gotten something wrong and packages can/do have versions that are divergent from the aforementioned key, and you/ROS users would like such a badge, then we will need more information about where that data can be found.

I'd much rather just rebrand things on our badge to make it clear it's working with repositories, not packages.

This would make sense to me. I originally used the package terminology because packages are what I’m used to working with, but after following this discussion and considering how the badges are meant to be used on repos, and packages in a repo share the same version, I think it’s probably fine to keep the current behavior. @cst0 and @ijnek, do you think it makes sense for the badges to represent a whole repository as they do today, if we just change the wording to correct my misuse of “package”?

do you think it makes sense for the badges to represent a whole repository as they do today, if we just change the wording to correct my misuse of “package”?

Yes, that makes sense.

The alternative solution would be to point at a repository by looking for a full URL (note that not all ROS packages are on GitHub). But, this is nonideal partly for the previously-mentioned cases.

@cst0 could you clarify why a full URL would be needed, rather than just a repository name as used by the current implementation?

could you clarify why a full URL would be needed, rather than just a repository name

Sorry if I'm introducing confusion, I'm not familiar enough with this project, language, or implementation to provide clear insight into that side of things. What I was referring to was how ROS doesn't restrict to github vs. any other version control host, so specifying (for example) "ros-perception/slam_gmapping" wouldn't be suitable.
So whatever final solution is chosen would need to account for that, but it seems like you already have a solution to that:

do you think it makes sense for the badges to represent a whole repository as they do today, if we just change the wording to correct my misuse of “package”?

As far as I can tell, yeah.

But since the data source we are using is the rosdistro yaml file, isn’t the name slam_gmapping alone enough?

Closing in favor of https://github.com/badges/shields/pull/8292


suggestion: upload to nuget.org?

makes things so much easier... :)
a more "speaking" package name would be fine

@swernli FYI
@rs38 - The POR as of know is to distribute Docker-PowerShell through OneGet and the PowerShell gallery. The reason we have not yet published it to the gallery is that its not yet in its 1.0 release. I will work with my side to get a timeline here and get a public release out. Until then you can use it from the private location in exactly the same way as if it was published.
To update the module simply run Update-Module Docker from time to time and you will get any dev-feed updates.

thx for the fast update!
btw:
when doing the install-module on 1 from 3 machines I get this annoying error.
In C:\Program Files\WindowsPowerShell\Modules\PowerShellGet\<IP_ADDRESS>\PSModule.psm1:1661 Zeichen:21
+ ...          $null = PackageManagement\Install-Package @PSBoundParameters

tried already all the examples from preivious and closed issue.

No need to wait until 1.0! Would love to see Docker-PowerShell on the PowerShell Gallery. @jterry75, feel free to ping me offline to help make this happen.


Add support for nodepool's gpu_instance

Describe your changes
Issue number
#517
Checklist before requesting a review

[ ] The pr title can be used to describe what this pr did in CHANGELOG.md file
[ ] I have executed pre-commit on my machine
[ ] I have passed pr-check on my machine

Thanks for your cooperation!

Hi @zioproto , would you please give this pr a review? Thanks!


CupertinoTextField question

https://user-images.githubusercontent.com/42401363/140653151-50cf5087-438f-475c-8fe8-ae51e17b0430.MP4
When I type long text, two cursors appear
CupertinoTextField()

flutter doctor -v
[✓] Flutter (Channel unknown, 2.5.0, on macOS 11.6 20G165 darwin-x64, locale zh-Hans-CN)
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more
      details.
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.2.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2019.3.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2019.3.1)
[✓] VS Code (version 1.61.2)
[✓] Connected device (3 available)

Hi @zhanglijie5997, this issue is know and is being tracked https://github.com/flutter/flutter/issues/89228.
Kindly follow up there for more information.
Closing as duplicate of #89228


Clean up fri quotients

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#486  👈
#484
#483
#482
#481
#480
#479
#478
#463
#477
#476
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @spapinistarkware and the rest of your teammates on  Graphite

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.06%. Comparing base (5f96433) to head (eaa2a12).

Additional details and impacted files
@@                                 Coverage Diff                                  @@
##           spapini/03-15-save_inverse_on_quotient_numerator     #486      +/-   ##
====================================================================================
- Coverage                                             96.06%   96.06%   -0.01%
====================================================================================
  Files                                                    62       62
  Lines                                                  8800     8792       -8
  Branches                                               8800     8792       -8
====================================================================================
- Hits                                                   8454     8446       -8
  Misses                                                  298      298
  Partials                                                 48       48

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


Y033 bind members up top

my end goal is to address the functionality in #44. to become familiar with the code, i'd like to submit a series of PRs to john papa-fy the codebase.

@ealves are there any contribution guidelines i should follow or can we merge?

also since there will be multiple related PRs, we could merge them against an integration branch first before including it as part of one bigger release.


Fixing Virgo's arms directions

Correcting the arms' orientations of Virgo within src/jimgw/single_detector/detector.py.
The previous setting is
V1 = GroundBased2G(
    "V1",
    latitude=(43 + 37.0 / 60 + 53.0921 / 3600) * DEG_TO_RAD,
    longitude=(10 + 30.0 / 60 + 16.1887 / 3600) * DEG_TO_RAD,
    xarm_azimuth=243.0 * DEG_TO_RAD,
    yarm_azimuth=333.0 * DEG_TO_RAD,
    xarm_tilt=0,
    yarm_tilt=0,
    elevation=51.884,
    mode="pc",
)

The corrected setting is
V1 = GroundBased2G(
    "V1",
    latitude=(43 + 37.0 / 60 + 53.0921 / 3600) * DEG_TO_RAD,
    longitude=(10 + 30.0 / 60 + 16.1887 / 3600) * DEG_TO_RAD,
    xarm_azimuth=70.5674 * DEG_TO_RAD,
    yarm_azimuth=160.5674 * DEG_TO_RAD,
    xarm_tilt=0,
    yarm_tilt=0,
    elevation=51.884,
    mode="pc",
)

which aligns with bilby's setting and the orientation of Virgo (shown on google map)

The fix should only have a minimal impact on the published results. Since
i) For injection study, the injection and recovery are consistently using the "modified" Virgo
ii) For real events, only GW170817 can be affected. Yet, the SNR in Virgo is minimal as compared to the two LIGO detectors (see here), so the posterior of psi is still minimally impacted.


Unknown variables

Excuse me, how is the self.inst_path_label read and path_label_dict formed in line 501 of the semantic_kitti_polarmix code?

Thanks for your reply. Do I need to comment the first two lines of instance_xyz?
instance_p = self.twis(instance_p)
_, instance_xyz = self.pointwolf(instance_p[:,:3])
instance_p[:, :3] = instance_xyz


Class "Webklex\IMAP\Events\FlagNewEvent" not found

Hello
I've been trying to implement this package but for some reason I keep getting an error of which I can't seem to find all that much information.
Whenever I try to delete a message , or manipulate the folders/messages I get message like :
Class "Webklex\IMAP\Events\FlagNewEvent" not found
Class "Webklex\IMAP\Events\FlagDeletedEvent" not found
What am I missing here. Thanks for any feedback

Additional information with DEBUG=>true

Is there any support out here for this package? I can't seem to find any solution for the above.

@Webklex is there anything you can help me with ? It seems this package is abandoned ?


_set_stylesheet fails if passed a dictionary

    def _set_stylesheet(self, css):
        if not isinstance(css, CSS):
            css = CSS(**css) # <<-- Lacks a target argument
        self._style = css
        css.apply_recursive(self)

Not sure what an appropriate substitute key would be here, though maybe self.key is workable?
Or we could raise TypeError if not passed a CSS instance.

Does this fail for a case like this?
dummy = {'backgroundColor': 'red', 'width': 200}
Button(style=dummy)

That one fails, but this works:
dummy = {'backgroundColor': (1,0,0), 'width': 200}
Button(css=dummy)

its really just the Styled.stylesheet property that causes the issue, at it tries to create a CSS instance from the dict. Which fails because CSS expects a target param in addition to the usual style kwargs.
But after that everything just treats the _style attribute as a mapping.
The only circumstance I could see that would fail would be:
with VerticalForm(css={'backgroundColor': (1,0,0), 'width': 200}) as f:
    for i in xrange(5):
        Button()
f.stylesheet.apply_recursive(f)

Which will fail with:
# AttributeError: 'dict' object has no attribute 'apply_recursive' #

What is the use-case for specifying a dictionary outside of css= ?
I would opt for a TypeError if not CSS.

Maybe we just give it an empty, like this:
def _set_stylesheet(self, css):
css = css or {}
if not isinstance(css, CSS):
css = CSS(**css) # <<-- Lacks a target argument
self._style = css

It's just syntax sugar for users, since there are a lot of use cases where you just want to say 'use these options' and the more elaborate stuff is not on your radar

Doh, we could just target Control when we make the fake CSS....
def _set_stylesheet(self, css):
if not isinstance(css, CSS):
css = CSS(gui.Control, **css)
self._style = css
css.apply_recursive(self)
self.key would apply it only to this instance, control would do it to anything under this instance. I'm not sure which would be a commoner case.  Maybe self.key is better because it prevents outflows from this non-standard syntax.   Thoughts?


.NET Core support

Given that there is a .net core version of the Azure Storage SDK (albeit still pre-release), is it feasible to produce a .net core version of the TP?
@isaacabraham I'm happy to have a look into this if you think it would be feasible

Did TPs already work on .net core? I thought that they did not.

Did TPs already work on .net core? I thought that they did not.

I saw a similar open 'up for grabs' issue on FSharp.Data . When I saw this issue I guess I just assumed TPs were generally do-able on the core clr so thought I'd open this issue up, check if Isaac (or anyone else) had any thoughts then start doing some digging as to how to make it happen.
Between your comment and my own digging it's starting to look like I may have been more than a bit naive with this assumption!

I'll be honest with both of you. I've tried my best to keep as far away from .NET Core until it's stable. If you want to have a crack at this, knock yourself out :-) But I'm not planning on doing anything on it for a few weeks / months just yet.

Currently working on it.
https://github.com/fsprojects/AzureStorageTypeProvider/pull/113

This is now fixed Whooo!


updates to tables for TeamsJS 2.21

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/teamsjs-support-m365.md
94
12
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/teamsjs-support-m365.md
94
12
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/teamsjs-support-m365.md
94
12
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/teamsjs-support-m365.md
92
14
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/teamsjs-support-m365.md
93
13
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


⚠ TorrentLeech has degraded performance

In 159ffaf, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 6277 ms

Resolved: TorrentLeech performance has improved in b4795f4 after 6 minutes.


ql-editor loses focus after one keypress

I'm having an issue where when I press a key the element with .ql-editor loses focus and I can no longer type. This occurs whenever i have an onChange={this.handleEditorChange} prop added. the following example is:

  <ReactQuill
    theme="snow"
    items={ReactQuill.Toolbar.defaultItems}
    className="small-12 columns"
    onChange={this.handleEditorChange}
    defaultValue={this.state.text}
    style={styles.getStyles(['no_side_padding'])}
      styles={{
        ".quill-toolbar":{
          'border-bottom':'1px solid black'
        },
        ".ql-editor":{
          'min-height': '15em'
        }
      }}
    />

the handleEditorChange is:
handleEditorChange: function(value, delta, source){
        this.setState({
            text: value
        });
}

i suspect that its because setState triggers a re-render, but I don't have any issues with my native form components.
Suggestions?

Update
i have narrowed it down the the styles property. as long as styles={{}} prop exists (regardless of the content inside it) the contenteditable div automatically loses focus.

Quill unfortunately has no facilities to update the configuration once instantiated, so it has to be reloaded for changes to take effect; because styles is a new object each render this causes constant re-instantiation.
If you're not changing your styles constantly you can define that styles object somewhere else where it stays the same, and that way you should be covered in the mean time while this is being fixed. You could also override your css elsewhere.
Long term, though, I'm not sure what to make of the style API. For one it shouldn't cause re-renders so easily, and even then the previous focus state should be brought back seamlessly after render. So this is going to be investigated and fixed soon.
But Quill doesn't offer a way to modify styles after instantiation, so the choices are either replacing it entirely with a completely independent implementation, or "suck it up" and tell people to just override CSS the usual way if they need to.
How often do you use styles? What for? I'm trying to get a feel for how people use that.
Thanks a lot for the report by the way.

Currently, I'm using the style API to override default styles in the stylesheets. My intention is to remove the stylesheets altogether and instead use inline styles entirely, but this bug prevents me from doing so. Currently it's not breaking issue to include a style sheet, but it isn't ideal.
what about this as a temporary solution to Quill's issue?
in componentWillRecieveProps:
If (nextProps.style !== this.props.style){
  this.setState({
    forceFocus: true
  });
}

then in componentDidMount:
if(this.state.forceFocus){
  editor.focus();
}

I only took a cursory look at your code, but I think that might provide a simple solution. I would fork it and test it myself but I'm currently in the middle of a big push, and just editing the style-sheets is a good enough (temporary) work around for me

The selection restoration when un-mounting and mounting the editor should work already, as there are provisions to save it and restore it after. Not sure why it isn't working here.
You can do something like this, in the mean time:
var styles = { … };

var React.createClass({
    ...
    render() {
        return (
            <ReactQuill ... styles={styles}/>
        );
    }
});

This should keep the styles object identical and avoid the re-rendering.
By the way, there is a focusing branch that adds a public focus and blur api to the editor component. It is imperative, so I don't really like it that much, but see if you find it helpful.

@zenoamaro thanks for the workaround. I was having this problem, but I'm fine to move my styles object out of the render method, so your fix worked for me.
It is unexpected behavior, though. I'm not sure what to do about that.

Yes, it should happen, and should be fixed.

I have tracked down what causes the issue, and unfortunately it has been caused by another workaround in use to apply custom Quill styles. I have a plan to fix this, but unfortunately it couldn't go in this release.
In the mean time, please declare non-primitives (such as objects or arrays) outside the render, so that they won't be created each time and so won't trigger a re-render.

I seem to be suffering a similar problem. Here's my code:
const Segment = props =>

    <ReactQuill toolbar={false} formats={["bold", "italic"]} value={props.segment.value}
      onChange={value => props.update(props.segment.id, value)} />

When props.update is called and the component is re-rendered, focus is lost completely.

This also happens when toolbar options are passed along. I suspect for the same or a similar reason.

Yes, I'm also getting this

If I'm not getting wrong this issue is related to all this props:
https://github.com/zenoamaro/react-quill/blob/master/src/component.js#L51-L60
But can confirm @zenoamaro's workaround works

This is fixed in the latest master. Deep prop validation is now in place, and objects can safely be passed to a ReactQuill component without triggering re-renders.

For anyone who comes back to this issue, there are many other reasons for the component containing your , such as module handlers, which can be wrapped in a useCallback to avoid remounting on an edit event. Make sure to provide the correct dependencies for the useCallback as well.

I had the same issue with react-quill.
in my case the parent component key was changing on each character I entered in the editor, therefore react-quill considered as a new render each time,
so, make sure that parent component key is not changing on while entering character into the quill editor.
you can check in the react-devtools extensiosn to check if the parent components key is changing or not while adding text into quill editor.

I have the same issue when I type certain character( zero-width non-joiner like &zwnj ) the "ql-editor" container lose focus and turned out it went to "ql-clipboard" instead.
I don't know it is the proper fix but when I manually remove "ql-clipboard" container and it worked for me.

It still happens.

It still happens.

I tried solving it this way and it works for me
`const caret = document.getSelection();
let savedCaret;
document.querySelector('.ql-editor').addEventListener('keydown', function () {
savedCaret = [caret.focusNode, caret.focusOffset];
})
$('.ql-clipboard').on('DOMSubtreeModified', function () {
document.querySelector('.ql-editor').focus();
caret.collapse(savedCaret[0], savedCaret[1]);
});`

@nyitarlatt Thanks bro.  Now, I just use react-quilljs which also has bugs but I only use the basic features. And it's ok with that.


How can I get which stocks have been sold/bought in TopkDropoutStrategy?

❓ Questions and Help
As the documentation says

Topk-Drop algorithm：
Topk: The number of stocks held
Drop: The number of stocks sold on each trading day
Currently, the number of held stocks is Topk. On each trading day, the Drop number of held stocks with the worst prediction > score will be sold, and the same number of unheld stocks with the best prediction score will be bought.

After I execute the backtest code below, how can I get to know which stocks have been sold/bought in TopkDropoutStrategy?
# backtest and analysis
with R.start(experiment_name="backtest_analysis"):
    # backtest & analysis
    par = PortAnaRecord(recorder, port_analysis_config, "day")
    par.generate()

You can get detailed information about portfolio positions from portfolio_analysis/positions_normal_1day.pkl
Example:
with R.start(experiment_name="backtest_analysis"):
    recorder = R.get_recorder(recorder_id=rid, experiment_name="train_model")
    model = recorder.load_object("trained_model")

    # prediction
    recorder = R.get_recorder()
    ba_rid = recorder.id
    sr = SignalRecord(model, dataset, recorder)
    sr.generate()

    # backtest & analysis
    par = PortAnaRecord(recorder, port_analysis_config, "day")
    par.generate()

recorder = R.get_recorder(recorder_id=ba_rid, experiment_name="backtest_analysis")
positions = recorder.load_object("portfolio_analysis/positions_normal_1day.pkl")

@al33m501 thank very much!
It worked successfully


VSCode Remote Dev container, using our Alpine base & Home Assistant Wheels

Proposed change

This PR changes our VSCode dev container to use Alpine, based on our own base images.
It also leverages our pre-build Python Wheels, to make the general installation and setup fly 🚀
All VSCode tasks are adjusted to have those same benefits.
⚠ Warning!
I've originally created this branch in October 2019, however I never published it because I found it unstable.
During my testing, I've noticed issues with the Python Language server crashing.
I've now rebased this branch and updated it to match our current codebase. However, on my end, it did still crash using the latest language servers and vscode. At this point, I'm doubting my set up.
Output from the Python Language Server at some point:
[Error - 9:10:40 AM] Starting client failed
Launching server using command /root/.vscode-server/extensions/ms-python.python-2020.1.58038/languageServer.0.5.30/Microsoft.Python.LanguageServer failed.

VSCode showing:

So some testing from others is required for this to be mergeable!
Supersedes: #31715
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[x] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[x] Code quality improvements to existing code or addition of tests

Example entry for configuration.yaml:

# Example configuration.yaml

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[ ] The code change is tested and works locally.
[ ] Local tests pass. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the development checklist
[ ] The code has been formatted using Black (black --fast homeassistant tests)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

[ ] No score or internal
[ ] 🥈 Silver
[ ] 🥇 Gold
[ ] 🏆 Platinum

Pascal just confirmed the same issue.

Closing for now, will raise an issue at Microsoft.


Checkpoint File shared link not publicly accessable

Hi,
The share-point link for trained checkpoints keeps asking for user-access. Are they private? If yes, can you share them with<EMAIL_ADDRESS>? I am trying to replicate your work for University research.It'll be very kind of you. Thanks in advance.

They are publically shared. You can try the onedrive link again or the newly added baiduyun link.