waitForAngular does not wait for $httpBackend pass through requests

browser.waitForAngular() does not wait for $httpBackend pass through requests. The root cause is that there are two $browser instances, one for mock $httpBackend requests and one for real $httpBackend requests (i.e. pass through requests) and protractor calls notifyWhenNoOutstandingRequests() on only the instance used for mock requests.
By "pass through requests" I mean those configured like this:
    $httpBackend.whenGET(/.*/).passThrough();

I am able to reproduce this with a slightly modified angular-seed project.
$ git diff --ignore-space-change
diff --git a/app/app.js b/app/app.js
index 6ea2401..d1ac31e 100644
--- a/app/app.js
+++ b/app/app.js
@@ -3,12 +3,15 @@

 // Declare app level module which depends on views, and components
 angular.module('myApp', [
        'ngRoute',
/* + */       'ngMockE2E',
        'myApp.view1',
        'myApp.view2',
        'myApp.version'
 ]).
//-config(['$locationProvider', '$routeProvider', function($locationProvider, $routeProvider) {
/* + */       config(['$locationProvider', '$routeProvider', '$provide', function($locationProvider, $routeProvider, $provide) {
        $locationProvider.hashPrefix('!');

        $routeProvider.otherwise({redirectTo: '/view1'});
//-}]);
/* + */}]).run(function ($httpBackend) {
/* + */       $httpBackend.whenGET(/.*/).passThrough();
/* + */});

diff --git a/app/index.html b/app/index.html
index ec9da03..c6e3046 100644
--- a/app/index.html
+++ b/app/index.html
@@ -33,6 +33,7 @@

   -->
   <script src="bower_components/angular/angular.js"></script>
   <script src="bower_components/angular-route/angular-route.js"></script>
<!-- + -->  <script src="bower_components/angular-mocks/angular-mocks.js"></script>
   <script src="app.js"></script>
   <script src="view1/view1.js"></script>
   <script src="view2/view2.js"></script>

Then in angular.js, comment out this line in createHttpBackend().
		  /*
        completeRequest(callback,
            status,
            response,
            xhr.getAllResponseHeaders(),
            statusText);
		  */

This will cause real HTTP requests to never complete, but Protractor won't wait.
Now run the e2e tests and watch them fail immediately without waiting for the HTTP request to complete.
Node v6.9.2
Protractor 4.0.14
Angular 1.5.8
Chrome 55.0.2883.87
Windows 7

Note that the second $browser instance is created in angular-mocks.js here:
angular.module('ngMockE2E', ['ng']).config(['$provide', function($provide) {
  $provide.value('$httpBackend', angular.injector(['ng']).get('$httpBackend'));
  $provide.decorator('$httpBackend', angular.mock.e2e.$httpBackendDecorator);
}]);

This bug was introduced in this commit.
commit 6685043ad40acc50d7088f87e2a71f76d2265306
Author: Andrew Schmadel<EMAIL_ADDRESS>Date:   Mon Oct 19 10:31:43 2015 -0400

    fix(ngMockE2E): allow $httpBackend.passThrough() to work when ngMock is loaded

    Allow $httpBackend.passThrough() to work normally when ngMock is loaded
    concurrently with ngMockE2E, as is typically the case when writing tests with
    angular.mock.module()

    Fixes #1434
    Closes #13124

diff --git a/src/ngMock/angular-mocks.js b/src/ngMock/angular-mocks.js
index 4ec4638..5bb2dc5 100644
--- a/src/ngMock/angular-mocks.js
+++ b/src/ngMock/angular-mocks.js
@@ -2328,6 +2328,7 @@ angular.module('ngMock', ['ng']).provider({
  * the {@link ngMockE2E.$httpBackend e2e $httpBackend} mock.
  */
 angular.module('ngMockE2E', ['ng']).config(['$provide', function($provide) {
+  $provide.value('$httpBackend', angular.injector(['ng']).get('$httpBackend'));
   $provide.decorator('$httpBackend', angular.mock.e2e.$httpBackendDecorator);
 }]);

diff --git a/test/ngMock/angular-mocksSpec.js b/test/ngMock/angular-mocksSpec.js
index 1cfe423..d3a2892 100644
--- a/test/ngMock/angular-mocksSpec.js
+++ b/test/ngMock/angular-mocksSpec.js
@@ -2150,12 +2150,12 @@ describe('ngMockE2E', function() {
     var hb, realHttpBackend, callback;

     beforeEach(function() {
-      module(function($provide) {
-        callback = jasmine.createSpy('callback');
+      callback = jasmine.createSpy('callback');
+      angular.module('ng').config(function($provide) {
         realHttpBackend = jasmine.createSpy('real $httpBackend');
         $provide.value('$httpBackend', realHttpBackend);
-        $provide.decorator('$httpBackend', angular.mock.e2e.$httpBackendDecorator);
       });
+      module('ngMockE2E');
       inject(function($injector) {
         hb = $injector.get('$httpBackend');
       });

Sorry, why are you using ngMock and Protractor?  ngMock is for unit testing, Protractor is for integration testing.  There are a number of ways these two don't play well together

This issue effects normal e2e tests, e.g. the ones in angular-seed. ngMock is not involved. If you clone angular-seed, modify it the way I describe, and then run the e2e tests, you will see the issue. It's also possible to see the problem via code inspection. This change https://github.com/angular/angular.js/pull/13124 caused there to be 2 $browser instances and thus 2 outstandingRequestCount variables. If you then look at waitForAngular in protractor, you will see that it will only wait for one of the outstandingRequestCount variables to be set to 0.

IMHO, the problem is with Angular and not with Protractor and this change https://github.com/angular/angular.js/pull/13124 should be reverted. I have created an angular issue: https://github.com/angular/angular.js/issues/15593.

BTW, downgrading to Angular 1.5.6 resolves the issue.

Yeah but you're adding angular mocks to the seed app.  Why would you ever do that?

I'm confused by your question. I added angular-mocks.js so that I could use ngMockE2E per the documentation. Is there some other way to use ngMockE2E.$httpBackend in e2e tests?
https://docs.angularjs.org/api/ngMockE2E

Oh wait... are you are confused why I am using the angular-seed app in the first place? I am using it as a simple test case to demonstrate the issue.

Oh I see, never mind.  I didn't realize that ngMockE2E was from the same file as ngMock

But actually I'm still confused.  Unless your module depends on ngMock, that line from the PR you referenced should do nothing?  I guess I'll have to load up the seed app myself and look around

I think the problem is indeed with Angular - so I think you can close this one.
@mike-bresnahan can you open a new issue in the angular.js repo for this?

Sjelin, if you are referring to this line, it is part of ngMockE2E.
angular.module('ngMockE2E', ['ng']).config(['$provide', function($provide) {
  $provide.value('$httpBackend', angular.injector(['ng']).get('$httpBackend'));
  $provide.decorator('$httpBackend', angular.mock.e2e.$httpBackendDecorator);
}]);

I believe the problem is that angular.injector(['ng']) creates a brand new injector, which also creates new singleton objects for $httpBackend and $browser.

Closed in favor of https://github.com/angular/angular.js/issues/15593


Build multiple binaries in single command

This speeds up builds 5-10x
Timing:
Cached: 6s -> 1s
Clean: 40s -> 7s
[x] Test and Release
Marking as WIP because this probably breaks tests -- needs a lot more thought

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lintmodern_istio
a1a9cee1b0a3b5c91e1416fec254e117d130765b
link
/test lintmodern

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lintmodern_istio
a1a9cee1b0a3b5c91e1416fec254e117d130765b
link
/test lintmodern

integ-security-k8s-tests_istio
a1a9cee1b0a3b5c91e1416fec254e117d130765b
link
/test integ-security-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
a1a9cee1b0a3b5c91e1416fec254e117d130765b
link
/test integ-security-k8s-tests

unit-tests_istio
757f06e53a45ebbdb2e477faf379c70bb916cccb
link
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
a1a9cee1b0a3b5c91e1416fec254e117d130765b
link
/test integ-security-k8s-tests

integ-new-install-k8s-tests_istio
98d1e7b6cdb87f19a0c51a53f9a5847508692a9b
link
/test integ-new-install-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

I will update common files once this merges

@istio/wg-policies-and-telemetry-maintainers mind taking a look?


[NOIPM] Duplicate Accused Officers

Describe the bug
When adding accused officers, if they existed in the old roster, they're duplicated in the current release. We need to check if they are also duplicated in the complainants section, not just accused.
To Reproduce
Steps to reproduce the behavior:

Login to prod db for NO
Check accused officers table for duplicates

We are updating the officers table today. If officers were added before and were used on cases we can't just remove them from the system.

Notes from kickoff

This is a one time change
Need a future story to allow client to update the officer inmates themselves
When get get the new spreadsheet (potentially some time in May?), should get into call w/ New Orleans to go through our DB
Shak/Reshmi to do some deep dive into the workflow to understand how to improve process moving fwd

The story #458 was created to make the import of the data in the database

This architecture decision record was created with the solution implemented: https://sites.google.com/thoughtworks.com/tw-invisible-institute-wiki/adrs

blocked on the foreign key issue


Update ocropus-rtrain

(The latest C++ version is not a drop-in replacement anymore, however, so this won't currently work.)


Project alias gives 404

I have a main project, chemfiles. I also have another project called chemfiles-python.
When I add chemfiles-python as a subproject for chemfiles, with the python alias, the proposed URL in the admin view is http://chemfiles.readthedocs.org/projects/python/, which leads to a 404 error.
The http://chemfiles.readthedocs.org/projects/chemfiles-python/ URL leads to the documentation.
What was expected: the http://chemfiles.readthedocs.org/projects/python/ URL should work.

The above URL works for me as expected. Perhaps the project was being rebuilt when you attempted the alias?

OK, so this one works now, but http://chemfiles.readthedocs.org/projects/fortran/en/latest/ and http://chemfiles.readthedocs.org/projects/julia/en/latest/ do not ... (The corresponding http://chemfiles.readthedocs.org/projects/chemfiles-fortran/en/latest/ and http://chemfiles.readthedocs.org/projects/chemfiles-julia/en/latest/ do).
I just clicked on "Compile a new version" to rebuild the latest version of the doc, and I still get this error. Is there any post-processing delay here ?

Some more investigation: the http://chemfiles.readthedocs.org/projects/fortran/ link leads to a 404, but adding /en/latest/ leads to the correct place (i.e. using http://chemfiles.readthedocs.org/projects/fortran/en/latest/)
So it might be missing a redirection to /<lang>/<version>

Closing this, these projects seem to lead to 404s now.


MediaDevice.GetDevices() returns incomplete MediaDevice-Objects

We are calling MediaDevice.GetDevices() in our program from different threads (but never at the same time) and have noticed that only the first call (from the main thread) provides a complete list of all connected (Android) devices. Although all subsequent calls return a list with the correct number of MediaDevice objects, only the first object of this list is completely filled/initialized correctly. All subsequent objects are uninitialized, the DeviceId and Manufacturer fields are null.
Does something have to be considered when calling MediaDevice.GetDevices() from different threads?

I just noticed that the problem also occurs when MediaDevice.GetDevices() is called from the EventArrived-EventHandler of the ManagementEventWatcher class.
If two devices are connected and a device is disconnected, we call the MediaDevice.GetDevices() method directly from the EventArrived-EventHandler. The MediaDevice object returned by this method is not completely initialized, e.g. the Manufacturer field contains this string instead of the manufacturer: "M\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0"

Which apartment model is required?


Add a --count option to nsc sub

This is a suggestion only, but it would be nice if nats could just exit or autounsubscribe + exit after --count n many messages.

https://github.com/nats-io/natscli/pull/250


Automatically roll out changes to Models

Automate the recreation of model Pods when their expected spec's change.
Currently implemented as "delete-all-pods", followed by "create-all-pods".
Fixes #219

NOTE: TestModelPodRecovery integration test fails on main, but it fixed here:
KUBEBUILDER_ASSETS="$(./bin/setup-envtest use 1.30.0 --bin-dir $(pwd)/bin -p path)" go test -v ./test/integration -run Recovery

    model_pod_recovery_test.go:33:
                Error Trace:    /Users/nick/work/substratusai/kubeai/test/integration/model_pod_recovery_test.go:33
                Error:          Condition never satisfied
                Test:           TestModelPodRecovery
                Messages:       Pod-1 should be recreated
--- FAIL: TestModelPodRecovery (1.12s)
FAIL

...

2024-09-22T18:50:05-04:00       INFO    manager run finished
2024/09/22 18:50:05 Waiting for Run() to finish
2024/09/22 18:50:05 Run() finished
FAIL    github.com/substratusai/kubeai/test/integration 11.881s
FAIL


more prometheus metrics

average response time from cloudflare
queries count to cloudflare (with 95p)

Fixed via https://github.com/crowdsecurity/cs-cloudflare-bouncer/pull/58


Thread-safe synchronization of dup-related rewrite rules

The following program:
(Sum Nil        ) = 0
(Sum (Cons x xs)) = (+ x (Sum xs))

(Mul Nil        ) = 1
(Mul (Cons x xs)) = (* x (Mul xs))

(Main x) =
  let xs = (Cons 1 (Cons 2 (Cons 3 Nil)))
  (Pair (Sum xs) (Mul xs))

Sometimes gives the wrong result when evaluated in parallel, due to synchronization of dup nodes. As I mention on HOW.md, the fact values exist only at one place greatly simplifies parallelism, and most rewrite rules are thread-safe by design, since the thread has full ownership of every memory location it reads/writes to. There is an exception to that rule, though: dup nodes. For example, consider the program below:
dup a b = (Cons 1 (Cons 2 (Cons 3 Nil))))
...

It can happen that two different threads arrive at the same dup node (which is impossible for all other nodes): one coming from the a binder, and other coming from the b binder. This will case both threads to attempt to apply the DUP-CTR rule at the same time, which may result in errors. To avoid this issue, we must make sure that, whenever a thread accesses a dup node from one of its binders (either a or b here), it locks the area, preventing outside threads from accessing it. This was (naively) prevented by using atomic_inc as a lock here, but this is a temporary solution, which was expected to break in some cases, as depicted above. A more robust solution must be carefully designed to ensure that two threads won't access the same dup node at the same time; i.e., once a thread accesses a dup node, it must lock any subsequent access to it, and a thread that attempts to do so must spinlock until the other is done.

What about a lookup table; instead of spinlocking it references the future value, which is substituted later.

Apologies if I am not understanding this correctly; is this an opportunity to introduce lazy computation via std::future?

To be valid C++, static_cast must be used for malloc/realloc, and strdup can be used:

I don't understand what you're trying to propose, sorry. std::future has nothing to do with this problem, it is a matter of using atomics/locks in the right places.

From what I understand, this issue sounds related to what GHC's RTS calls "blackholing". This SO answer seems like a good introduction: https://stackoverflow.com/a/31974133/388010.
Well, probably more in that "once you properly synchronise access to dup nodes (however that works), you might see that it's not so harmful to do some dupped RHSs more than once".

Yes, that looks like the exact same situation, translated to HVM realm. Nice catch. And yes, the solution could be to "just dup twice", but that would still require a properly thought use of atomics to let it happen without two threads tripping on each other.

Closing, since the new parallel runtime performs atomic duplications and substitutions.


Short blog that maps various workloads w.r.t OpenEBS storage classes

This is a low priority. Hence closing.


Windows Clang: Make it work and add to CI

Topic says it all
The trick ys
cmake (blah) -DCMAKE_CXX_COMPILER=clang++ -DCMAKE_C_COMPILER=clang

So I tried this (see https://github.com/free-audio/clap-wrapper/pull/79)
short version

with llvm clang++ gnu like extensions, vst3 spits masses of warnings and errors
with clang-cl and ninja a bunch of earlier errors occur with MT and the like

So my conclusion is: windows clang isn't working now.

OK so as of now
Windows g++ builds with Werror in CI with a couple of -Wno-blah in place
Clang builds in CI but we turn off Werror altogether. We need to review the CI errors and see if we can fix or -Wno- them away.
But I'm going to mark this one solved in next since it does build.

Merged to 0.6.0


Make the code clearer about what is being overwritten

I think it would be good to split out all the functions that are overwriting methods in Base/REPL into their own file and all the auxiliary stuff into another file. Then you could enable precompilation on this package but include the file that overwrites method in the __init__ function.

I recall putting code in __init__ prevents Revise from working, but I'm happy to separate the parts as best I can.

Reorg done, but I'll leave open if there are any better solutions to disabling precompilation or avoiding warnings.

Fixed in v0.1.10

Fixed in v0.1.10


Function with this type can be assigned to a type without

TypeScript Version:  Version 2.8.0-dev.20180302
Search Terms: strict this assigned property
Code
The following code, compiled with --strict, gives a type error:
class C {
  f(this: this) : void {}
}
const c : C = new C();
const f = c.f;
f();

This correctly prints "error TS2684: The 'this' context of type 'void' is not assignable to method's 'this' of type 'C'."
However, I can assign a type to the variable f without the this context, and things will typecheck (but fail if the function actually uses this internally):
class C {
  f(this: this) : void {}
}
const c : C = new C();
const f : () => void = c.f;
f();

Expected behavior:
The code gives a type error like "Type '(this: C) => void' is not assignable to type '() => void'.
Actual behavior:
The code compiles without errors.
Playground Link: here
Related Issues:

Also fails on 2.7.2

Note that this issue has slightly broader consequences than the situation sketched above.
Because by default class methods don't auto-infer a this type on their signature you can pass them directly (unbound to this) as a callback to another function take the signature without this. This clearly causes runtime errors:
function twice(f: () => number) {
    alert(2 * f())
}

class C {
    constructor(private v: number) { }
    mul() { return this.v }
}

const c: C = new C(2);

// Both typecheck:
twice(() => c.mul()); // Will alert 4
twice(c.mul); // Will alert NaN

The error is not reported even if the option "strict" is enabled.
declare const method: (this: Date) => void
declare let f: () => void
f = method // no error in TS 2.8-dev

Looks like a duplicate of #7968.

@mattmccutchen That seems to be about a flag to change the default this type for functions, to provide better call-site and assignability checking. This is about the assignability checking itself not catching a type error, so I think this is a different issue.

@hesselink As #7968 suggests, the current behavior is that when the this type of a function signature is not specified, it defaults to any for the purpose of assignability checking.  That's why the assignability check is not reporting an error in your example.  So it's the same issue AFAICT.


WCAG - Legg til lenketekst logo

What It Means
A link contains no text.
Why It Matters
If a link contains no text, the function or purpose of the link will not be presented to the user. This can introduce confusion for keyboard and screen reader users.
How to Fix It
Remove the empty link or provide text within the link that describes the functionality and/or target of that link.

Already fixed


Ability to add multiple docker endpoint (failover)

Hi,
I have a swarm cluster with 2 managers
I have a traefik instance (not in that swarm)
I want to tell traefik to fetch docker info from manager1:2375
But if this node is going down (for maintenance or crash), I want traefik to failover on manager2:2375
Is that possible ?
Thanx for reading

You can use an external solution like DNS RR or keepalived with a vip.

@WTFKr0 , can you explain the context of you need.
Indeed, the normal behavior is to use Traefik as a Docker container on the swarm manager node.
So, if the manager goes down, Docker will move your Traefik container on the other manager.

Hi, thanx for reading.
I don't remember exactly the purpose of this issue :rofl:
But I think it was to provide a failover IP
If traefik in swarm, and one node goes down, you have to reach to other node (other IP)
Today we have a haproxy (on a third host) doing that failover between our 2 nodes, so I think we can close that !
Thanx


Re-enable terminal workspace settings

I disabled the settings in https://github.com/microsoft/vscode/pull/121176/commits/c7d26be9a5cc298bf51d7d671417313d147e461c to allow moving forward with the PR. This commit needs to be reverted when workspace trust ships to re-enable the feature. It will be broken in the meantime.
Apologies if you are impacted by this but it can be worked around by bringing the relevant settings to your user settings file.

A temporary fix was added to re-enable workspace shell settings when "terminal.integrated.allowWorkspaceConfiguration": true is set https://github.com/microsoft/vscode/issues/122104

The work-around is not working for me, using VS Code Insiders on Windows 10 with WSL integrated terminal:

In settings.json I add the following lines:

{
	"terminal.integrated.allowWorkspaceConfiguration": true,
	// added both the following as I'm not sure which is correct
	"terminal.integrated.env.linux": {
		"TEST_KEY":"foo"
	},
	"terminal.integrated.env.windows": {
		"TEST_KEY":"bah"
	}
}

Then I open a new terminal and do:

echo printenv | grep 'TEST_KEY'

And there is no result!

The "terminal.integrated.allowWorkspaceConfiguration": true workaround isn't working for me either in the stable 1.56 using Remote SSH. I tried changing the setting both in the User and Remote contexts.

"terminal.integrated.defaultProfile.windows": "My Profile" setting in workspace is ineffective even after configuring "terminal.integrated.allowWorkspaceConfiguration": true in user settings.
Version: 1.57.0-insider (user setup)
Commit: fe23c2041f848d6c2f6bd27875112a37bcc82f84
Date: 2021-05-06T05:16:25.564Z
Electron: 12.0.4
Chrome: 89.0.4389.114
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19043

@Tyriar can you point to specific part of #123133 that I should look at? All I see it link to similar issues, your comments and link back to this issue.

@zendu that issue was fixed 6 days ago, but only in Insiders

I came here to report issues with terminal.integrated.profiles and terminal.integrated.defaultProfile, and it's not clear to me from this what the supposed status of this is. Here's what I have observed in 1.56.2:

Setting "terminal.integrated.defaultProfile.windows": "Command Prompt" will successfully get the default terminal to be Command Prompt instead of PowerShell.
Setting "terminal.integrated.allowWorkspaceConfiguration": true Breaks this and causes the default to revert to PowerShell.
The only way I can find to force the workspace to drive the terminal back to command prompt is with the old deprecated setting: "terminal.integrated.shell.windows": "${env:windir}\\System32\\cmd.exe"

I would expect at least one of the following to work instead of number 3 above:

Specify "terminal.integrated.defaultProfile.windows": "Command Prompt" in the workspace file,
Create a valid "terminal.integrated.profiles.windows" block in the workspace file that includes a Command Prompt section, OR
Do nothing - if the "terminal.integrated.defaultProfile.windows" setting is not valid in the workspace file, VS Code should honor the setting from the settings.json file.

@bluemonkmn a lot of problems around default profiles was recently fixed in Insiders if you want to give that a spin. Here's the big change that fixed most of the problems https://github.com/microsoft/vscode/pull/123729

@Tyriar The injection of environment variables into the shell is still not working (Insiders). These are my settings:
  "terminal.integrated.allowWorkspaceConfiguration": true,
  "terminal.integrated.env.linux": {
    "TEST_KEY": "foo"
  },
  "terminal.integrated.env.windows": {
    "TEST_KEY": "bar"
  },

@robcca "terminal.integrated.allowWorkspaceConfiguration": true, only works in user settings, is that the problem?

@Tyriar - I continue to see the issue in Insiders. I created a custom terminal profile in workspace settings, but following line shows error and that profile is not used to create terminal.
"terminal.integrated.defaultProfile.windows": "MyCMD"

Error message in "problems" window of VS Code:
Value is not accepted. Valid values: "PowerShell", "Command Prompt", "Git Bash".
Version: 1.57.0-insider (user setup)
Commit: 45aafeb326d0d3d56cbc9e2932f87e368dbf652d
Date: 2021-05-18T05:14:15.253Z
Electron: 12.0.7
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19043

@robcca "terminal.integrated.allowWorkspaceConfiguration": true, only works in user settings, is that the problem?

No, if I understand your question correctly. Those settings are in the settings.json file in "C:\Users<user>\AppData\Roaming\Code - Insiders\User" and not the one in in the .vscode folder in the root of my project.

I encounter similar issue.  All i hope is that we can set the environment variables for terminal at Workspace level.  I  was hopping nothing need to be set in the user settings.

Complication reported here https://github.com/microsoft/vscode/issues/125440

Marking this as verified bc complication and the fix are tracked here #125440.

Why is the issue closed? The problem still exists and I can't see the relevance of #125440.

@robcca it's closed because workspace terminal settings now works in Insiders, if it does not for you please provide more information

@robcca it's closed because workspace terminal settings now works in Insiders, if it does not for you please provide more information

The situation is the same as I described in a previous comment , although I see that "terminal.integrated.allowWorkspaceConfiguration": true is greyed-out since you removed it and made it the default behaqviour. The environment variable that I added in settings.json does not appear in the WSL Ubuntu 20.04 inline terminal (my default) but does appear in the environment of, say, Git Bash and every other non-WSL inline terminal.


Always give an error when unable to find named type

Until now, we are giving warning sometimes when unable to find named type. With this PR, we will be consistently giving an error from now on.

Is it breaking drafter tests? I suspect that it actually may.

Yes, I think it will.

Make sure that drafter works when somebody decides that is time to integrate it :)


CX Reflected_XSS_All_Clients @ src/main/java/org/t246osslab/easybuggy/vulnerabilities/OGNLExpressionInjectionServlet.java [master]

Reflected_XSS_All_Clients issue exists @ src/main/java/org/t246osslab/easybuggy/vulnerabilities/OGNLExpressionInjectionServlet.java in branch master
Method service at line 24 of src\main\java\org\t246osslab\easybuggy\vulnerabilities\OGNLExpressionInjectionServlet.java gets user input for the ""expression"" element. This element’s value then flows through the code without being properly sanitized or validated and is eventually displayed to the user in method responseToClient at line 31 of src\main\java\org\t246osslab\easybuggy\core\servlets\AbstractServlet.java. This may enable a Cross-Site-Scripting attack.
Severity: High
CWE:79
Checkmarx
Lines: 31

Code (Line #31):
        String expression = req.getParameter("expression");

Issue still exists.
SUMMARY
Issue has total 1 vulnerabilities left to be fix (Please scroll to the top for more information)

Issue still exists.
SUMMARY
Issue has total 1 vulnerabilities left to be fix (Please scroll to the top for more information)


AppState: meteorflux:dispatcher dependency?

I don't use meteorflux:dispatcher. Why is it now installed with meteorflux:appstate?

Because it's needed by the new AppState.modify() function.

Maybe you could reopen this issue and add the "feedback-needed" label, so that people could suggest ways to remove this annoying dependency.

Let me think about how can I keep that package out if not needed.

We will solve this releasing the ReactiveState package.


encryption bug setup E2EE

### Expected Behavior
encryption bug, Enable E2EE modal has bug

Actual Behavior
always getting error

Reproduction Steps
No response
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
latest
What operating system are you using?
macOS
Operating System Version
mac
Installation method
browser
Last Known Working Insomnia version
No response

relate to #7401

@tomermaz We believe we resolved this issue, can you try again?


:onnxruntime:, session_state.cc:1169 onnxruntime::VerifyEachNodeIsAssignedToAnEp] Some nodes were not assigned to the preferred execution

while running inference coming across this error .
2024-09-11 11:17:28 | INFO | infer.modules.vc.modules | Get sid: guanguanV1.pth
2024-09-11 11:17:28 | INFO | infer.modules.vc.modules | Loading: assets/weights/guanguanV1.pth
2024-09-11 11:17:28 | INFO | infer.modules.vc.modules | Select index: logs\guanguanV1.index
2024-09-11 11:17:55 | INFO | infer.modules.vc.pipeline | Loading rmvpe model,assets/rmvpe/rmvpe.pt
2024-09-11 11:17:56.0223850 [W:onnxruntime:, session_state.cc:1169 onnxruntime::VerifyEachNodeIsAssignedToAnEp] Some nodes were not assigned to the preferred execution providers which may or may not have an negative impact on performance. e.g. ORT explicitly assigns shape related ops to CPU to improve perf.
2024-09-11 11:17:56.0261045 [W:onnxruntime:, session_state.cc:1171 onnxruntime::VerifyEachNodeIsAssignedToAnEp] Rerunning with verbose output on a non-minimal build will show node assignments.

Same error
`2024-12-17 11:30:36 | INFO | infer.modules.vc.modules | Get sid: WSB.pth
2024-12-17 11:30:36 | INFO | infer.modules.vc.modules | Loading: assets/weights/WSB.pth
2024-12-17 11:30:36 | INFO | infer.modules.vc.modules | Select index: logs\WSB\added_IVF256_Flat_nprobe_1_WSB_v2.index
2024-12-17 11:31:02 | INFO | infer.modules.vc.pipeline | Loading rmvpe model,assets/rmvpe/rmvpe.pt
2024-12-17 11:31:02.6151983 [W:onnxruntime:, session_state.cc:1169 onnxruntime::VerifyEachNodeIsAssignedToAnEp] Some nodes were not assigned to the preferred execution providers which may or may not have an negative impact on performance. e.g. ORT explicitly assigns shape related ops to CPU to improve perf.
2024-12-17 11:31:02.6214749 [W:onnxruntime:, session_state.cc:1171 onnxruntime::VerifyEachNodeIsAssignedToAnEp] Rerunning with verbose output on a non-minimal build will show node assignments.
D:\RVC\RVC1006Nvidia>pause
请按任意键继续. . .`

I've found a solution
https://blog.csdn.net/qq_45660232/article/details/143486016


Webpack unable to minify dist/swiper.esm.js

This is a (multiple allowed):

[x] bug

[x] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 4.2.2

Platform/Target and Browser Versions: Node 8/webpack 4.

https://github.com/facebook/create-react-app/blob/master/packages/react-scripts/template/README.md#npm-run-build-fails-to-minify

What you did
Added Swiper to a React Create App bootstrapped/generated app
Expected Behavior
Webpack to compile the JS bundle
Actual Behavior
Build failed due to not being able to minify the swiper.esm.js file
Creating an optimized production build...
Failed to compile.

Failed to minify the code from this file:

 	./node_modules/swiper/dist/js/swiper.esm.bundle.js:56

P.S. Remember, an issue is not the place to ask questions. You can use Stack Overflow
for that.
Before you open an issue, please check if a similar issue already exists or has been closed before.

http://idangero.us/swiper/get-started/ "As an ES module" section

hey yea sorry was just about too close this myself
Ended up doing this:
Just import the swiper with the absolute path.
import Swiper from 'swiper/dist/js/swiper.js'


[stable/docker-registry] Fix haSharedSecret value documentation

haSharedSecret shoud be secrets.haSharedSecret:
https://github.com/helm/charts/blob/b7fe92e77da4254b98eb89c470b3201433d86978/stable/docker-registry/templates/secret.yaml#L16
Is this a new chart
no
What this PR does / why we need it:
Documentation fix
Which issue this PR fixes
Fixes value name (no issue created)
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

@jpds @rendhalver Checks are passed, PR is ready to go (not sure if tide and pull-charts-e2e will be passed before PR is approved)

/ok-to-test


Hue stopped working in 0.66.0.beta0

Home Assistant release (hass --version):
0.66.0.beta0
Component/platform:
Hue
Description of problem:
After upgrading to 0.66.0.beta0, home assistant refuses to properly setup my hue lights.
Expected:
Hue lights work properly.
Problem-relevant configuration.yaml entries and steps to reproduce:
hue:
  bridges:
    - host: !secret hue_host
      allow_unreachable: true
      allow_hue_groups: true

Traceback (if applicable):
2018-03-25 13:33:34 ERROR (MainThread) [homeassistant.components.light] Error while setting up platform hue
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/homeassistant/helpers/entity_platform.py", line 82, in async_setup
    SLOW_SETUP_MAX_WAIT, loop=hass.loop)
  File "/usr/lib/python3.6/asyncio/tasks.py", line 358, in wait_for
    return fut.result()
  File "/usr/lib/python3.6/site-packages/homeassistant/components/light/hue.py", line 55, in async_setup_platform
    bridge = hass.data[hue.DOMAIN][discovery_info['host']]
KeyError: 'host'

Additional info:

I did notice a comment about breaking changes regarding phue -> aiohue, however it wasn't clear to me if anything needs to be done to the config.

Same here - appears to work fine on start, but as soon as I switch a light on (or anything) I get error after error similar to the above and then anything hue related stops working.  Have had to roll back.

FWIW, no such errors on my setup. I have the following in my config:
hue:
  bridges:
    - host: <IP_ADDRESS>
      allow_hue_groups: false

I don't have allow_hue_groups declared (IIRC it's true by default?), I do have allow_unreachable: true

Worth trying without allow_unreachable: true.

That's exactly what I was thinking but I'm afk now, will try later and report back. 👍

According to the docs allow_unreachable is still valid.

I just tried without allow_unreachable and same result. The error KeyError: 'host' seems to indicate more than just a sub-entry like unreachable.

@arsaboo
Re->FWIW, no such errors on my setup
Try to turn on one of your Hue lights from the web interface and then check your logs.

@mp01111 Yes, I did that and even changed the colors, effects, etc. No errors in the logs.

Another piece of information that may be relevant...I am on Gen 2.

I'm gen 2 too, so it's not that I don't think.

Got the same issues.  I pared back my hue config to only have the host entry (ip address) and it still wouldn't setup the platform on startup.  Rolled back to .65.6 and back to normal (with the full set of config options I used originally).

Ok Hue stopped working for me as well starting beta1. Here are the logs https://hastebin.com/urenoqasip.sql

To see what other similarities or differences we all might have, how is everyone installed?
I am Ubuntu server 17.10 with a simple install, no virtual environment.

I am in hass.io

Debian, in a venv, no discovery.
I haven't done anything with this since my posts above as I haven't been at home for long enough in the meantime.

Interesting. I wasn't expecting this to be so varied.

@arsaboo since you are working, what is your setup?

I updated to beta 1 and everything is still working well here. I am on Ubuntu in venv.

0.66b1 still has the same errors for me. @arsaboo has obviously made his monthly payments to the devil. I should not have ignored the invoice. ;-)

Fix has been included in beta2

I tried beta2 and still have the same error
Log Details (ERROR)
Mon Mar 26 2018 19:32:45 GMT-0500 (Central Daylight Time)
Error while setting up platform hue
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/homeassistant/helpers/entity_platform.py", line 82, in async_setup
SLOW_SETUP_MAX_WAIT, loop=hass.loop)
File "/usr/lib/python3.6/asyncio/tasks.py", line 358, in wait_for
return fut.result()
File "/usr/local/lib/python3.6/dist-packages/homeassistant/components/light/hue.py", line 56, in async_setup_platform
bridge = hass.data[hue.DOMAIN][discovery_info['host']]
KeyError: 'host'

I'm also not getting the platform setup in b2.
Error
Invalid config for [hue]: required key not provided @ data['hue']['bridges'][1]['host']. Got None. (See /home/homeassistant/.homeassistant/configuration.yaml, line 114). Please check the docs at https://home-assistant.io/components/hue/
My Config
hue:
  bridges:
    - host: <IP_ADDRESS>
    # - allow_in_emulated_hue: false
    - allow_unreachable: true
    - allow_hue_groups: false

I commented out the allow_in_emulated_hue since that was what threw the error on my first try at b2

@GreenTentacle you have a misconfiguration problem so not the same as this issue. host is a required key in the list of bridgeskey-value pairs. Two of the pairs in the bridges list is missing host in your config.

@MartinHjelmare I pulled out the 3 options and it indeed looks like it's working now, however if the config was left as is, it worked without issue in .65.6, which is odd.  I'm not sure if I mangled the tabs in copying it into the issue, or if .65.6 was just more forgiving of bad configs-- I'll check out my past config commits later on.
Anyway, it looks like I may be good for now, thanks !

I'm confused about what the config should look like now, as it still doesn't work for me on b2. How should my current config be changed?
hue:
  bridges:
    - host: !secret hue_ip
      filename: phue.conf
      allow_unreachable: true
      allow_hue_groups: true

Do you have the same error as in the first post of this thread?

Yes sorry should have mentioned that, exact same error, lights don't show up at all. Will try to reinstall a totally fresh docker container as maybe there's a glitch in mine

Confirmed working with a fresh install!
Cheers

Just picked up 0.66.0.beta2 and ran into the same issue I reported. However, I realized that I had installed the Hue Motion Sensor custom component, and removing that solved the issue for me.
Does anybody else run that custom component?

@docbobo - that custom_component isn't going to work anymore, but @balloob assures me that native support for other hue devices is coming soon now that the base of the platform has changed.
Hope this helps.

I do have the custom component installed so it is my issue as well. I don't think I can do without it so I will just keep watching.


Counter for inquiry requests being marked as purchase

Would like to see a count of how many total inquiries have been marked as a purchase.
See https://github.com/artsy/collector-experience/issues/86#issuecomment-278797200 (Counter section)

Dropping this as per https://github.com/artsy/collector-experience/issues/86#issuecomment-280413761


Print Info when a strategy step starts

Fixes #1581
Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock

I tried to create two new modules Kore.Strategies.TransitionRule and Kore.Log.InfoReachability but then I wouldn't be able to solve circular dependencies:
data InfoReachability goal
    = InfoSimplify !goal
    | InfoRemoveDestination !goal
    | InfoDeriveSeq ![Rule goal] !goal
    | InfoDerivePar ![Rule goal] !goal

depends on the instances of
class Goal goal where
    data Rule goal
    ...
    transitionRule :: ...
    ...

that define Rule and transitionRule (at the same time)
But when defining transitionRule for instance Goal (AllPathRule Variable), transitionRuleTemplate is used, which should use whileSimplify, defined as whileSimplify goal = logWhile (**InfoSimplify** goal)
The problem is that definitions for data Rule and transitionRule can't be given in different modules, so we get the following circular dependency:
data ***InfoReachability*** goal -> data Rule goal linked to transitionRule -> transitionRuleTemplate -> whileSimplify -> InfoSimplify which is a constructor of data ***InfoReachability*** goal

I extracted InfoReachability from Kore.Strategies.Goal. There were different ways to do this, but I chose to extract Rule from the Goal class, making it a stand-alone datatype family. We also need to add InfoReachability to Kore.Log.Registry, but we have to remove the type parameter first. There are a few things left for you to do to make that happen:

[ ] Remove the logging from transitionRuleTemplate and make a transition rule transformer logTransitionRule :: (Prim goal -> ProofState goal goal -> Strategy.TransitionT (Rule goal) m (ProofState goal goal)) -> (Prim goal -> ProofState goal goal -> Strategy.TransitionT (Rule goal) m (ProofState goal goal)) which wraps its argument with the logging functions.
[ ] Remove the parameter from InfoReachability and specialize its type to goal ~ ReachabilityRule Variable.
[ ] Finally add InfoReachability to Kore.Log.Registry.


proxy: must support urls with basic auth

proxy capabilities should set basic auth based on URL content.
browsebmob_proxy=true
selenium_host=http://user:password@demo.qaprosoft.com:4444/wd/hub
so current approach doesn't work as GGR can't authenticate incoming request through proxy.
it is recommended to specify basic auth using below code explicitly:
// start the proxy
BrowserMobProxy proxy = new BrowserMobProxyServer();
proxy.start();
proxy.autoAuthorization("", "username", "password", AuthType.BASIC);

// get the Selenium proxy object
Proxy seleniumProxy = ClientUtil.createSeleniumProxy(proxy);

// configure it as a desired capability
DesiredCapabilities capabilities = new DesiredCapabilities();
capabilities.setCapability(CapabilityType.PROXY, seleniumProxy);

not a bug, just a configuration issue.
For use-cases when you launch java inside container you have to

share port to master host be able to use embedded proxy
override address of the browsermob proxy host using new argument
browsermob_host=my_host
disable system proxy and keep it only for WebDriver/API client:
proxy_set_to_system=false

we don't support system proxy calls using embedded browsermob proxy if java process is running inside docker container.
Improved integration using new browsermob_host param in 6.0.3-SNAPSHOT


Memory problems when batch classifying large directories

Search before asking

[X] I have searched the Pytorch-Wildlife issues and found no similar bug report.

Description
I have a directory of about 17,000 camera trap images, probably an average of a handful of detections per image.  When I try to run the batch megadetector on that directory from within a notebook, at around halfway through the batch, the machine runs out of memory (32GB).
If the high memory usage is unavoidable, a nice option would be to be able to run the detector on lists of images rather than directories, that way large directories could be broken up more easily.
Thanks for everything!
Use case
Large directories of images to be detected

Hello @davidwhealey, thank you so much for reporting this. We also realized this issue on our end and already have a solution to it. We are working on integrating it to the codebase and will give you an update as soon as the new inference function is released!

Hello @davidwhealey , we just pushed a new version with the fix to the batch detection memory issue. Could you try update the package and see if it fixes your issue?

Hi @zhmiao,
Not sure if this is the same issue, but still wanted to chime in. I'm currently running <IP_ADDRESS> which seems to be the latest version. But I'm also running into a memory issue. I'm running batch detect on a folder of 2000 images of between 300 and 1500 KB each.
Here's the log:
13%|██████████▎                                                                      | 8/63 [38:33<4:25:05, 289.19s/it]
Traceback (most recent call last):
File "batch_detect.py", line 25, in
results = detection_model.batch_image_detection(loader)
File "/home/jaimy/anaconda3/envs/pytorch-wildlife/lib/python3.8/site-packages/PytorchWildlife/models/detection/yolov5/base_detector.py", line 136, in batch_image_detection
for batch_index, (imgs, paths, sizes) in enumerate(dataloader):
File "/home/jaimy/anaconda3/envs/pytorch-wildlife/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 521, in next
data = self._next_data()
File "/home/jaimy/anaconda3/envs/pytorch-wildlife/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1203, in _next_data
return self._process_data(data)
File "/home/jaimy/anaconda3/envs/pytorch-wildlife/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1229, in _process_data
data.reraise()
File "/home/jaimy/anaconda3/envs/pytorch-wildlife/lib/python3.8/site-packages/torch/_utils.py", line 434, in reraise
raise exception
RuntimeError: Caught RuntimeError in pin memory thread for device 0.
Original Traceback (most recent call last):
File "/home/jaimy/anaconda3/envs/pytorch-wildlife/lib/python3.8/site-packages/torch/utils/data/_utils/pin_memory.py", line 34, in _pin_memory_loop
data = pin_memory(data)
File "/home/jaimy/anaconda3/envs/pytorch-wildlife/lib/python3.8/site-packages/torch/utils/data/_utils/pin_memory.py", line 58, in pin_memory
return [pin_memory(sample) for sample in data]
File "/home/jaimy/anaconda3/envs/pytorch-wildlife/lib/python3.8/site-packages/torch/utils/data/_utils/pin_memory.py", line 58, in
return [pin_memory(sample) for sample in data]
File "/home/jaimy/anaconda3/envs/pytorch-wildlife/lib/python3.8/site-packages/torch/utils/data/_utils/pin_memory.py", line 50, in pin_memory
return data.pin_memory()
RuntimeError: cuda runtime error (2) : out of memory at ../aten/src/THC/THCCachingHostAllocator.cpp:280

Hello @JaimyvS , I am sorry for the late reply! We will take a look at this issue and see if we can reproduce the memory issue on our side.

Thanks, even with some small datasets I've been having issues. I've been getting the error:
THCudaCheck FAIL file=../aten/src/THC/THCCachingHostAllocator.cpp line=280 error=2 : out of memory
Which seems the same as above. But not exactly. Hope you find something! If you need more info, I'd be happy to help.

@JaimyvS, so this whole thing might be a numpy issue. Here is some reference: https://github.com/microsoft/CameraTraps/issues/390 and https://github.com/jacobgil/pytorch-pruning/issues/16
Before we had this issue with our batch loading functions, and now we realize it happens in this for loop: https://github.com/microsoft/CameraTraps/blob/4c44b1ac4247025b8345f744fbf8b1b5b17b71a3/PytorchWildlife/models/detection/yolov5/base_detector.py#L142
If you could also help us get rid of this numpy issue, it would also be greatly appreciated! Otherwise, we will try fixing this on our end as well. Thank you so much!

@zhmiao I'm not 100% sure what you'd like me to do. I've looked at the references but the first seems to have been fixed by an update on your part. For the second one, I've tried running with pin_memory=False but this didn't work.
However when running with a batch size of 16 instead of 32. It seems to run. Which is weird, because I've already run a ton of detections with a batch size of 32 in the past. I sometimes have a feeling that it might be due to my Microsoft Surface Book 3 that I'm running Windows Subsystem for Linux on. Because the screen of the laptop is detachable it sometimes doesn't recognize the GPU in the base. And also the system throttles the GPU when it's not connected to net power. But I'm not sure how to test or fix this.

@JaimyvS , oh this is interesting! Dose your Surface book 3 have a nvidia gpu? From the spec page the only nvidia gpu on surface book 3 only has a 6g of gpu memory, which I think probably is relatively small for a 32 batch size. You mentioned that you have successfully run 32 batchsize in the past, was you using pytorchwildlife at that time or MegaDetectorv5? There might also be a differences in terms of model sizes. But I think the wsl issue you mentioned might also be possible.

@zhmiao Yeah it has a Nvidia Geforce GTX 1660 Ti with 6GB memory. I have definitely used the new pytorchwildlife library as well as the old Megadetector library with batchsize 32. But only recently ran into memory issues. Maybe after around version <IP_ADDRESS>.
But if nothing really changed in the last few minor versions. It might just be due to my system and in that case I'll just keep on using a batch size of 16 until I have better hardware.

Hello @JaimyvS, sorry for the late responses. We are in a two-week conference run and didn't have time to fully get back in the issue. I think we did make some changes in <IP_ADDRESS>, but not in <IP_ADDRESS>. If you had the issues before <IP_ADDRESS> and didn't have the issue before <IP_ADDRESS>, then I think it is not the code issue. But we will still try if we could reproduce the out of memory error from our end.

There is no further response since June. Therefore, we are closing this issue. If the issue still persist, please feel free to reopen it! Thank you so much for your participation!


Switch inside DropdownItem auto switch bug

NextUI Version
2.2.9
Describe the bug
I'm encountering a peculiar issue with a switch inside a dropdown.
Whenever I open the dropdown, the switch seems to toggle, but this only happens when the switch is not selected (selected: false). Any ideas or suggestions on what might be causing this behaviour?
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
create a component like:
<DropdownItem>
      <Switch />
</DropdownItem>

Expected behavior
I expected no action when I open the DropdownItem
Screenshots or Videos
No response
Operating System Version
Windows
Browser
Chrome

any updates?


[Enhancement] Improve Edit profile MOdal

Current UI:

This UI should me made better and more user friendly.

Hey @Kajol-Kumari , I've one doubt. Sorry for asking late But you mentioned in the issue to improve the edit profile model but in the screen shot the profile picture thing is showing.. Can you please elaborate a bit on the issue, It would be great ..

Hey @kritikaparmar-programmer in the image about you see the popup that has appeared for file upload. There you see the file upload option is not properly aligned. Also there is no message showing up the purpose of that dialogue box  ( which in this case should be please select a new image )etc.These things need to be improved and organised.

Ok 👍


no such file to load -- bcrypt

I am using activeadmin(0.3.4) along with rails 3.1.1
I am using windows to be my development. it's working fine in my development environment.
The issue came up when I was deploying with capristrano.
This error came up
no such file to load -- bcrypt
I am not sure how to fix the issue.
I am sure it is related to this (https://github.com/plataformatec/devise/issues/527)
but trying the suggestions from there the issue still remains.
thank you for you help.
Here is my gemfile -
source 'http://rubygems.org'
gem 'rails', '3.1.1'
Bundle edge Rails instead:
gem 'rails',     :git => 'git://github.com/rails/rails.git'
gem 'sqlite3'
Gems used only for assets and not required
in production environments by default.
group :assets do
gem 'coffee-rails', '~> 3.1.1'
gem 'uglifier', '>= 1.0.3'
end
group :development do
gem 'annotate', :git => 'git://github.com/ctran/annotate_models.git'
gem 'faker'
end
gem 'jquery-rails'
gem 'gravatar_image_tag'
gem 'will_paginate'
gem 'carrierwave'
gem 'sass-rails',   '~> 3.1.4'
gem 'activeadmin'
gem 'meta_search'
To use ActiveModel has_secure_password
gem 'bcrypt-ruby', :require => 'bcrypt'
Use unicorn as the web server
gem 'unicorn'
Deploy with Capistrano
gem 'capistrano'
To use debugger
gem 'ruby-debug19', :require => 'ruby-debug'
group :test do
Pretty printed test output
gem 'turn', :require => false
end
group :production do
gem 'therubyracer'
gem 'mysql2'
end

You might get more help on the mailing list.


[terraform-ibm-account-infrastructure-base] Give more flexibility on the bucket config used in DA

Should we expose more of the COS related variables in the DA?
Should we add support for cross regional buckets?

COS removed from module in https://github.com/terraform-ibm-modules/terraform-ibm-account-infrastructure-base/pull/83, closing


feat: add security.txt file

Closes #482
Description

Generated a security.txt file using https://securitytxt.org/
Digitally signed the file with an OpenPGP cleartext signature while making it sure to follow the RFC

@all-contributors please add @apraditya for security


BIGTOP-4155: Add support for openEuler OS in bigtop web

The bigtop 3.3.0 support openEuler OS : https://bigtop.apache.org/release-notes.html
Update the information in the bigtop web.
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (BIGTOP-4155)
[ ] Make sure that newly added files do not have any licensing issues. When in doubt refer to https://www.apache.org/licenses/

Looks like there is something wrong with our ci job, could you help take a look at it? @iwasakims @guyuqi

@kevinw66 I'm looking into the server log.

@iwasakims Thanks

@kevinw66 I fixed the issue on CI server and ran pipeline-site job to update the site contents.

@iwasakims Thanks, I see the website has been updated.


Only one of required scope is really required

Hello,
We found it a bit missleading that if you add multiple required scope middleware actually requires only one of required scope and authorizes if it exist. Maybe you should rename it to AllowedScopes or add some summary comment to explain it(I can create pr for ti)?

Please fix. It would be quite useful to be able to require multiple scopes.

There is nothing to fix as it is by design.
If you want to require multiple scopes - write a simple middleware (or an authorization attribute) that checks for the existence of your required scopes.
We renamed the property in IdentityServer4.AccessTokenValidation.

Thanks for clarifying.

I see that can supply multiple scopes in RequiredScopes however, I only need one scope & scopesecret (defined by clientId & clientSecret) to secure the webApi. So if the introspect end point is making sure that the caller has the scope (defined by clientId & clientSecret), what is the accesstokenvalidation middleware doing with the RequiredScopes attribute? How does it use RequiredScopes?


Keep pod-container relationship after disconnection

We need to keep this relationship for chargeback reports since we would like to know the project that an archived container belonged to
@simon3z @moolitayer @Fryguy Please review

@miq-bot add_label euwe/yes

Euwe Backport details:
$ git log
commit 42daac98535d6916f7d6ebd09d3a8567f62636b2
Author: Jason Frey<EMAIL_ADDRESS>Date:   Thu Sep 29 12:56:21 2016 -0400

    Merge pull request #11453 from zeari/keep_disconnected_pod_contianer

    Keep pod-container relationship after disconnection
    (cherry picked from commit 6af7a159c9eb1b9afe6b27783f9253b74ede8722)


action logging

Describe the feature
whenever any moderation/higher-privilege action is done (basically any leo supervisor/cad admin action) it should be logged somewhere incase of a feature being abused
Additional Context
No response

possibly discord webhook logging too

Currently very wack, but this will probably be the way it's displayed:

Sure. Soon.

of course no rush

eta? @Dev-CasperTheGhost

Not really, it's quite complex to work on this myself.

Closing since I currently can't figure out a way to do this properly. Hopefully will be able to get back to this in the future


Added filename function

This PR allows you to specify a custom function for setting the filename as part of the config options. This is in keeping with DiskStorage as requested by @LinusU.
I apologise that this kind of takes over #4 but I needed to get this working for myself now.

Removed semicolon and squashed

This looks good to me
I also didn't realize the distinction b/w the randomBytes vs pseudoRandomBytes in crypto.  It makes sense to use the pseudoRandom one.

Looks good :+1:


More Param 2.0 API cleanup

My priority with this PR was to further clean up the Parameterized namespace, following and even going further the dispositions suggested in https://github.com/holoviz/param/issues/543

I looked through all the holoviz repos + examples for usage of .pprint or ._pprint and found none, so I decided to remove them all together, using .param.pprint() was anyway the recommendation in the Version 1.12.0 release notes.
The same search showed no usage of .script_repr that has been deleted from Parameterized and also from ParameterizedFunction.
Instead of just deleting .state_pop in favor of ._state_pop, I decided to move it (and .state_push) to .param._state_pop. This API isn't documented and was planned to be made private, this is why I chose to make it private on the .param namespace.

EDIT:

Removed Time.next that was used for compatibility with Python 2

Just for context, Simon added some comments on code that wasn't moved around, thinking that was new code. I agree with the suggestions made, they'd be worth making in other PRs to keep this one focused on API cleanup.

The removals in this PR all make sense to me, other than one comment, looks good to me!


Demo app Authorisation UI changes and Public ID error handling

Demo app Authorisation UI changes and Public ID error handling integrated

r? @ustulation
(maidsafe_highfive has picked a reviewer for you, use r? to override)

r? @krishnaIndia

Reviewed 3 of 4 files at r1, 7 of 7 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Some discrepancies in request/response stats

Starting on scrapy-zyte-api >= 0.9.0, we're now able request for item extraction in Zyte API directly in the spider:
import scrapy
from scrapy_poet import DummyResponse
from zyte_common_items import Product

class SampleSpider(scrapy.Spider):
    name = "sample"

    def start_requests(self):
        yield scrapy.Request(
            "http://books.toscrape.com/catalogue/a-light-in-the-attic_1000/index.html",
            self.parse_product,
        )

    def parse_product(
        self,
        response: DummyResponse,
        product: Product,
    ):
        yield product

In the Scrapy crawl stats, we have these:
{
 ...
 'downloader/request_count': 1,
 'downloader/request_method_count/GET': 1,
 'downloader/response_count': 2,
 'downloader/response_status_count/200': 2,
 'response_received_count': 2,
 'scrapy-zyte-api/attempts': 1,
 'scrapy-zyte-api/status_codes/200': 1,
 'scrapy-zyte-api/success': 1,
 'scrapy_poet/dummy_response_count': 1,
 ...
}

These are kind of correct since DummyResponse is technically a response so our spider would have received 2 responses (i.e. DummyResponse and Product). Although I'd probably prefer to have a new downloader/dummy_response_count stat in lieu of the existing scrapy_poet/dummy_response_count just so that they are closer to one another (which helps when visually inspecting them):
{
 ...
 'downloader/dummy_response_count': 1,
 'downloader/response_count': 2,
 ...
}

Another discrepancy is in Scrapy Cloud since it would appear that we have 2 requests:

It's because of scrapinghub-entrypoint-scrapy.sh_scrapy.HubstorageDownloaderMiddleware (ref) records the requests in the Request-tab based on the response received.
def process_response(self, request, response, spider):
    self.pipe_writer.write_request(
        url=response.url,
        status=response.status,
        method=request.method,
        rs=len(response.body),
        duration=request.meta.get('download_latency', 0) * 1000,
        parent=request.meta.setdefault(HS_PARENT_ID_KEY),
        fp=request_fingerprint(request),
    )
    ...

An option would be to update https://github.com/scrapinghub/scrapinghub-entrypoint-scrapy to record the Requests based on processing the actual requests instead of responses. This seems quite straightforward though I'm not sure what's the reasoning of the original implementation.

based on processing the actual requests instead of responses

Information includes "response size", so it seems it requires the response.

We don’t need to make scrapy-poet a dependency, we only need to try to import from it, and don’t run the code if that did not work.
In the long run, the plan is not to have DummyResponse at all. Until that happens, I think modifying scrapinghub-entrypoint-scrapy may not be a bad idea.

Having the following in https://github.com/scrapinghub/scrapinghub-entrypoint-scrapy/blob/master/sh_scrapy/middlewares.py#L62-L71 seems to be the easiest way indeed:
    def process_response(self, request, response, spider):
        if type(response).__name__ == "DummyResponse":
            return response
        self.pipe_writer.write_request(...)

No need to import scrapy-poet.
I don't think the name DummyResponse would ever change. I also don't think it would be subclassed so having isinstance() checks won't be needed.

For the other approach, I'm not quite keen on the idea of passing a "skip_for_scrapy_cloud_stats" flag on the request-response pairs since it may get lost along the way (e.g. some other middleware messed with it).

Thanks @Gallaecio @BurnzZ! I'll test this approach then
def process_response(self, request, response, spider):
        if type(response).__name__ == "DummyResponse":
            return response
        self.pipe_writer.write_request(...)

PR https://github.com/scrapinghub/scrapinghub-entrypoint-scrapy/pull/73

Did https://github.com/scrapinghub/scrapinghub-entrypoint-scrapy/pull/73 fix this issue, or are there missing issues that need to be addressed here?

I'm still seeing the same issue in my SC jobs this week. Perhaps the new https://github.com/scrapinghub/scrapinghub-entrypoint-scrapy version with the fix was not deployed in all SC projects?

Are you using stack 2.11?


How to use decorator to render additional elements on the chart?

Hi I'm using line chart and I'm trying to render some solid lines with svg elements. I see there is option decorator for it but there is no instructions how to implement it. Any help? My chart looks like this
           <LineChart
              data={data}
              width={315}
              height={180}
              withDots={false}
              withInnerLines={false}
              withOuterLines={false}
              withShadow={false}
              style={swimHistoryStyles.chart}
              chartConfig={chartConfig}
            />

It looks something like this:
<LineChart
	decorator={({
        width,
        height,
        data,
        paddingTop,
        paddingRight
    }) => {
        // return an svg or an array of svg
        return <rect x="50" y="20" width="150" height="150"
  style="fill:blue;stroke:pink;stroke-width:5;opacity:0.5" />
    }}

/>

Thanks that worked.


Crashes on predicate expression

Input file
user.name == "foo"

Current output file
Crashes:
TypeError: Cannot read property '0' of undefined

Expected output file
user.name == "foo"

Also closing this because it passes in Ruby 2.6 and we need to handle more versions


RuntimeError: Load of misaligned address in deflate_quick.c:66

Sanitizer warning when using ClangCl:
zlib-ng\arch\x86\deflate_quick.c:66:9: runtime error: load of misaligned address 0x0106c781 for type 'unsigned short', which requires 2 byte alignment
0x0106c781: note: pointer points here
 48 07 08  6f cc 99 cc 3f cc 4f cc  c3 cc 68 cc a7 cc d5 cc  3e cc 17 cc 7f cc e0 cc  9d cc b9 cc 98

Line in question:
if (*(unsigned short *)(src0 + cnt) == *(unsigned short *)(src1 + cnt)) {

I was able to reproduce this on Windows using cmake . -DCMAKE_BUILD_TYPE=Debug -DWITH_SANITIZERS=ON -T ClangCl -A Win32 -DBUILD_SHARED_LIBS=OFF. Clang compiler for Visual Studio must be installed.


Extensions using webassembly (in child processes) are broken since the last release

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

VS Code Version: 1.82.2
OS Version: Macos 13.4.1

Since the latest update (yesterday?) our extensions which use webassembly modules have stopped working.
This functionality is isolated in forked processes and investigation has shown that wasm modules no longer load in these child processes. The error returned is:
Only absolute URLs are supported
Moving the functionality to the main extension process fixes the issue, but we'd rather keep the functionality isolated.
Steps to Reproduce:
This isn't trivial to reproduce without access to embedded hardware or for us creating a reproducer. Happy to investigate a repro extension, but was wondering if the loading strategy for webassembly (policies, node version, etc.) changed in the last release?

Some further digging shows this is a bug triggered by a change in behaviour.
In previous versions of vscode, the WebAssembply global object in the node extensionHost and any forked processes did not have an instantiateStreaming() function. In the latest release, the extensionHost still doesn't have this function, but any forked processes do!

I believe this is due to the new Node version and isn't an issue with VS Code


Fix build errors due to duplicated classes in dependencies

Increased version of jcabi-pom
Reenabled checks;
Excluded a dependency with duplicated classes: https://github.com/antlr/antlr4/issues/216

Fixes #103

@rulor merge

@rultor merge

@rultor merge

@suseika OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@suseika Done! FYI, the full log is here (took me 3min)


Outdated README

The README seems to still be for Babiri.net, which is down. Is there a live demo for Statsugiri and has the README just not been updated or is this still in the works?

A bit of both, unfortunately.
I'm in the process of bringing the platform back, so the README has not been updated. It'll be a "belts and suspenders" solution first until I can flesh out a backend service since the competitive season is beginning soon.
For updates, see Twitter (@StatsugiriGG) which is where I'll post production updates.


Zeal/Dash package

Zeal is a free port of Dash, an offline documentation browser. Dash docset is a standard of documentation for Zeal and DevDocs too.
Will be cool to generate Razor docset for these very useful tools like Zeal and DevDocs.

I'm not sure what we can do here at RazorEngine.
Make a Dash plugin? -> We should make a separate project
Use it four our documentation? -> We should add it to FSharp.Formatting instead, because then more people can use it!
Closing because I think this is not related to RazorEngine, but feel free to explain if I misunderstood your point.


Support readonly root filesystem

As discussed on slack:

create simple shims to /usr/local/bin for each supported tool, which calls real wrapper at /opt/containerbase/bin.
no write access for root group to /usr/local/bin

related issues

#101
#380
#1384
#1385

It cannot be stopped??

I'm not sure I fully follow, but I guess you mean that when the user mounts a volume to /opt/containerbase then anything we've put there prior is lost. So you're planning to put it in another location and symlink it over whenever the container runs?

Yes, that's the core idea.
When later running install-tool as user it will simply install to /opt/containerbase as now.
I can propably also check and run the containerbase-cli restore command implicit from install-tool  when something is missing inside /opt/containerbase.

Would it be a viable alternative to allow the user to configure the containerbase rw location via env variable? If configured then the restore happens from /opt/containerbase otherwise /opt/containerbase is used? Or is it too hard to set up PATH etc?

we can't ensure path then. a user would need to explicit set it 😞


Expression has changed after it was checked

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here

Current behavior

Expected behavior

Reproduction of the problem

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Table version: 6.3

Angular version: 2.4.x

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Language: [all | TypeScript X.X | ES6/7 | ES5]

`
 <x-datatable-column>
              <ngx-datatable-column name="操作">
             <template let-value="value" let-row="row" ngx-datatable-cell-template>

                     编辑删除

           </template>
       <x-datatable-column>

 editposition(row) {
  this.modalService.open(PositionModal, { backdrop: 'static', keyboard: false });
}

the click function will error, debug info:
./DataTableBodyRowComponent class DataTableBodyRowComponent - inline template:9:8 caused by: Expression has changed after it was checked. Previous value: 'true'. Current value: 'false'.
`

@Csirme I am getting the same error. Have you resolved it?

Same issue here. @Csirme, is there a fix for this since the issue has been closed?
Table version: 6.3
Angular version: 2.4.x

Also facing the same issue. @Csirme, @rogeru  @m0squito  any solution you guys have found ?