Stack overflow after upgrading to dependency-check 8.0.0

Describe the bug
Seeing a stack overflow after upgrading to dependency-check 8.0.0
Version of dependency-check used
The problem occurs using version 8.0.0 of the gradle plugin
* What went wrong:
Execution failed for task ':download:dependencyCheckAnalyze'.
> java.lang.StackOverflowError (no error message)

* Try:
> Run with --info or --debug option to get more log output.

* Exception is:
org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':download:dependencyCheckAnalyze'.
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.lambda$executeIfValid$1(ExecuteActionsTaskExecuter.java:142)
        at org.gradle.internal.Try$Failure.ifSuccessfulOrElse(Try.java:282)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeIfValid(ExecuteActionsTaskExecuter.java:140)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.execute(ExecuteActionsTaskExecuter.java:128)
        at org.gradle.api.internal.tasks.execution.CleanupStaleOutputsExecuter.execute(CleanupStaleOutputsExecuter.java:77)
        at org.gradle.api.internal.tasks.execution.FinalizePropertiesTaskExecuter.execute(FinalizePropertiesTaskExecuter.java:46)
        at org.gradle.api.internal.tasks.execution.ResolveTaskExecutionModeExecuter.execute(ResolveTaskExecutionModeExecuter.java:51)
        at org.gradle.api.internal.tasks.execution.SkipTaskWithNoActionsExecuter.execute(SkipTaskWithNoActionsExecuter.java:57)
        at org.gradle.api.internal.tasks.execution.SkipOnlyIfTaskExecuter.execute(SkipOnlyIfTaskExecuter.java:57)
        at org.gradle.api.internal.tasks.execution.CatchExceptionTaskExecuter.execute(CatchExceptionTaskExecuter.java:36)
        at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.executeTask(EventFiringTaskExecuter.java:77)
        at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.call(EventFiringTaskExecuter.java:55)
        at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.call(EventFiringTaskExecuter.java:52)
        at org.gradle.internal.operations.DefaultBuildOperationRunner$CallableBuildOperationWorker.execute(DefaultBuildOperationRunner.java:204)
        at org.gradle.internal.operations.DefaultBuildOperationRunner$CallableBuildOperationWorker.execute(DefaultBuildOperationRunner.java:199)
        at org.gradle.internal.operations.DefaultBuildOperationRunner$2.execute(DefaultBuildOperationRunner.java:66)
        at org.gradle.internal.operations.DefaultBuildOperationRunner$2.execute(DefaultBuildOperationRunner.java:59)
        at org.gradle.internal.operations.DefaultBuildOperationRunner.execute(DefaultBuildOperationRunner.java:157)
        at org.gradle.internal.operations.DefaultBuildOperationRunner.execute(DefaultBuildOperationRunner.java:59)
        at org.gradle.internal.operations.DefaultBuildOperationRunner.call(DefaultBuildOperationRunner.java:53)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor.call(DefaultBuildOperationExecutor.java:73)
        at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter.execute(EventFiringTaskExecuter.java:52)
        at org.gradle.execution.plan.LocalTaskNodeExecutor.execute(LocalTaskNodeExecutor.java:69)
        at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$InvokeNodeExecutorsAction.execute(DefaultTaskExecutionGraph.java:322)
        at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$InvokeNodeExecutorsAction.execute(DefaultTaskExecutionGraph.java:309)
        at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$BuildOperationAwareExecutionAction.execute(DefaultTaskExecutionGraph.java:302)
        at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$BuildOperationAwareExecutionAction.execute(DefaultTaskExecutionGraph.java:288)
        at org.gradle.execution.plan.DefaultPlanExecutor$ExecutorWorker.execute(DefaultPlanExecutor.java:462)
        at org.gradle.execution.plan.DefaultPlanExecutor$ExecutorWorker.run(DefaultPlanExecutor.java:379)
        at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:64)
        at org.gradle.internal.concurrent.ManagedExecutorImpl$1.run(ManagedExecutorImpl.java:49)
Caused by: java.lang.StackOverflowError
        at jdk.internal.reflect.GeneratedMethodAccessor296.invoke(Unknown Source)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze$_collectDependencyMap_closure17.doCall(AbstractAnalyze.groovy:475)
        at jdk.internal.reflect.GeneratedMethodAccessor301.invoke(Unknown Source)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze.collectDependencyMap(AbstractAnalyze.groovy:471)
        at jdk.internal.reflect.GeneratedMethodAccessor300.invoke(Unknown Source)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze$_collectDependencyMap_closure17.doCall(AbstractAnalyze.groovy:484)
        at jdk.internal.reflect.GeneratedMethodAccessor301.invoke(Unknown Source)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze.collectDependencyMap(AbstractAnalyze.groovy:471)
        at jdk.internal.reflect.GeneratedMethodAccessor300.invoke(Unknown Source)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze$_collectDependencyMap_closure17.doCall(AbstractAnalyze.groovy:484)
        at jdk.internal.reflect.GeneratedMethodAccessor301.invoke(Unknown Source)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze.collectDependencyMap(AbstractAnalyze.groovy:471)
        at jdk.internal.reflect.GeneratedMethodAccessor300.invoke(Unknown Source)
        ...

steps to reproduce? (i.e., example build.gradle?)

steps to reproduce? (i.e., example build.gradle?)

In my case, nothing was upgraded.
Suddenly, just running ./gradlew dependencyCheckAnalyze gives me the same stack overflow error.

steps to reproduce? (i.e., example build.gradle?)

Taking my project and running it outside of our corporate build infrastructure works fine, so I'm wondering if this is related to existing cache state somehow?

I get
Cause: java.lang.StackOverflowError
	at jdk.internal.reflect.GeneratedMethodAccessor105.invoke(Unknown Source)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze$_collectDependencyMap_closure17.doCall(AbstractAnalyze.groovy:475)
	at jdk.internal.reflect.GeneratedMethodAccessor109.invoke(Unknown Source)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze.collectDependencyMap(AbstractAnalyze.groovy:471)
	at jdk.internal.reflect.GeneratedMethodAccessor110.invoke(Unknown Source)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze$_collectDependencyMap_closure17.doCall(AbstractAnalyze.groovy:484)
	at jdk.internal.reflect.GeneratedMethodAccessor109.invoke(Unknown Source)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze.collectDependencyMap(AbstractAnalyze.groovy:471)
	at jdk.internal.reflect.GeneratedMethodAccessor110.invoke(Unknown Source)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze$_collectDependencyMap_closure17.doCall(AbstractAnalyze.groovy:484)
	at jdk.internal.reflect.GeneratedMethodAccessor109.invoke(Unknown Source)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze.collectDependencyMap(AbstractAnalyze.groovy:471)
	at jdk.internal.reflect.GeneratedMethodAccessor110.invoke(Unknown Source)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze$_collectDependencyMap_closure17.doCall(AbstractAnalyze.groovy:484)
	at jdk.internal.reflect.GeneratedMethodAccessor109.invoke(Unknown Source)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze.collectDependencyMap(AbstractAnalyze.groovy:471)
	at jdk.internal.reflect.GeneratedMethodAccessor110.invoke(Unknown Source)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze$_collectDependencyMap_closure17.doCall(AbstractAnalyze.groovy:484)
	```
etc. Same case, almost empty project after upgrade to 8.0.0 (7.4.0 was perfectly fine)

I also have the same issue. Tried one of the guesses above - about cleaning cache - but after running dependencyCheckPurge  outputis - Unable to purge database; the database file does not exist: /.gradle/dependency-check-data/7.0/odc.mv.db
Nothing cleaned, but when I run - dependencyCheckAnalyze - it fails immediately - at the same second - that means it doesn't attempt to download anything. dependency-check-data folder - doesn't have any data - only empty forder 7.0. But on jenkins - clean environment - it fetches CVE files for several minutes before failing - cleaning cache doesn't trigger downloading dependencies - probably some another error

Getting the same issue. Here is the most minimalistic build.gradle.kts

Can confirm that I can reproduce it. If @jeremylong doesn't beat me to it I hope to take a look on Wednesday.

steps to reproduce? (i.e., example build.gradle?)

Taking my project and running it outside of our corporate build infrastructure works fine, so I'm wondering if this is related to existing cache state somehow?

Sorry for capturing this gradle bug, but maybe it's related.
I have a similar problem with the maven plugin since updating to 8.0.0
When running the plugin with "update-only" in our build environment we receive a 403 error when it tries to download https://www.cisa.gov/sites/default/files/feeds/known_exploited_vulnerabilities.json
I can reproduce this access denied error using curl on different machines with different public IP's. Don't know why the datacenter seems to be blocked by cisa.gov

Should be fixed with https://github.com/dependency-check/dependency-check-gradle/pull/308

@cmuchinsky if there is an issue within the corporate environment - my guess would be the proxy config as we added a connection to cisa.gov to download the known exploited vulnerability catalog.

@cmuchinsky if there is an issue within the corporate environment - my guess would be the proxy config as we added a connection to cisa.gov to download the known exploited vulnerability catalog.

Thanks for the tip. I think the fix for the stack overflow should take care of it for me, by "corporate" I was more referring to our custom gradle distribution, the networking side of it should be fine.

A bit of debug logging suggests the stackOverflow is rooted at not accommodating for bill-of-materials dependencies triggering a 2-phase cyclic recursive nesting.
collectDependencyMap start at depth 0
convertIdentifier for org.jetbrains.kotlinx:kotlinx-coroutines-core-jvm:1.6.4
collecting transitive dependencies for org.jetbrains.kotlinx:kotlinx-coroutines-core-jvm:1.6.4
collectDependencyMap start at depth 1
convertIdentifier for org.jetbrains.kotlinx:kotlinx-coroutines-bom:1.6.4
collecting transitive dependencies for org.jetbrains.kotlinx:kotlinx-coroutines-bom:1.6.4
collectDependencyMap start at depth 2
convertIdentifier for org.jetbrains.kotlinx:kotlinx-coroutines-core-jvm:1.6.4
collecting transitive dependencies for org.jetbrains.kotlinx:kotlinx-coroutines-core-jvm:1.6.4
collectDependencyMap start at depth 3
convertIdentifier for org.jetbrains.kotlinx:kotlinx-coroutines-bom:1.6.4
collecting transitive dependencies for org.jetbrains.kotlinx:kotlinx-coroutines-bom:1.6.4
collectDependencyMap start at depth 4
convertIdentifier for org.jetbrains.kotlinx:kotlinx-coroutines-core-jvm:1.6.4
collecting transitive dependencies for org.jetbrains.kotlinx:kotlinx-coroutines-core-jvm:1.6.4
collectDependencyMap start at depth 5
convertIdentifier for org.jetbrains.kotlinx:kotlinx-coroutines-bom:1.6.4
collecting transitive dependencies for org.jetbrains.kotlinx:kotlinx-coroutines-bom:1.6.4
collectDependencyMap start at depth 6
convertIdentifier for org.jetbrains.kotlinx:kotlinx-coroutines-core-jvm:1.6.4
collecting transitive dependencies for org.jetbrains.kotlinx:kotlinx-coroutines-core-jvm:1.6.4
collectDependencyMap start at depth 7
convertIdentifier for org.jetbrains.kotlinx:kotlinx-coroutines-bom:1.6.4
collecting transitive dependencies for org.jetbrains.kotlinx:kotlinx-coroutines-bom:1.6.4
collectDependencyMap start at depth 8
convertIdentifier for org.jetbrains.kotlinx:kotlinx-coroutines-core-jvm:1.6.4
...

continuing all the way until we reach the limits of the stacktrace

@aikebah I found the same thing with jackson-bom.

In the process of releasing 8.0.1 that will include the fix.

Ah... missed your messages... wonder whether your fix looks like mine... that need some final polishing

The core of my fix was just checking if we've already added the included by reference: https://github.com/dependency-check/dependency-check-gradle/pull/308/files#diff-30a351cd25e5d32781ee84335558775ee75d36c8d549b4e3947824ee50741f08R478-R481

My fix basically tried to do the same in a different, less elegant, way by creating a new HashSet containing only root to collect 'already seen' dependencies at the root of the transitive call, passing that set along downstream recursive calls.

Version 8.0.1 is failing with another error:
Some of the file system contents retained in the virtual file system are on file systems that Gradle doesn't support watching.
Caused by: org.gradle.api.GradleException: Analysis failed.
Caused by: org.owasp.dependencycheck.exception.ExceptionCollection: One or more exceptions occurred during analysis:
	DatabaseException: Unable to connect to the dependency-check database
		caused by DatabaseException: Database schema does not match this version of dependency-check
		caused by DatabaseException: Database schema is out of date

Stacktrace
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':dependencyCheckAnalyze'.
> Analysis failed.

* Try:
> Run with --debug option to get more log output.
> Run with --scan to get full insights.

* Exception is:
In dispose, destroying event queue.
In DISPOSE, [CENTRAL] disposing of memory cache.
Memory Cache dispose called.
In DISPOSE, [POM] fromRemote [false]
org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':dependencyCheckAnalyze'.
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.lambda$executeIfValid$1(ExecuteActionsTaskExecuter.java:142)
	at org.gradle.internal.Try$Failure.ifSuccessfulOrElse(Try.java:282)
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeIfValid(ExecuteActionsTaskExecuter.java:140)
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.execute(ExecuteActionsTaskExecuter.java:128)
	at org.gradle.api.internal.tasks.execution.CleanupStaleOutputsExecuter.execute(CleanupStaleOutputsExecuter.java:77)
	at org.gradle.api.internal.tasks.execution.FinalizePropertiesTaskExecuter.execute(FinalizePropertiesTaskExecuter.java:46)
	at org.gradle.api.internal.tasks.execution.ResolveTaskExecutionModeExecuter.execute(ResolveTaskExecutionModeExecuter.java:51)
	at org.gradle.api.internal.tasks.execution.SkipTaskWithNoActionsExecuter.execute(SkipTaskWithNoActionsExecuter.java:57)
	at org.gradle.api.internal.tasks.execution.SkipOnlyIfTaskExecuter.execute(SkipOnlyIfTaskExecuter.java:56)
	at org.gradle.api.internal.tasks.execution.CatchExceptionTaskExecuter.execute(CatchExceptionTaskExecuter.java:36)
	at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.executeTask(EventFiringTaskExecuter.java:77)
	at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.call(EventFiringTaskExecuter.java:55)
	at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.call(EventFiringTaskExecuter.java:52)
	at org.gradle.internal.operations.DefaultBuildOperationRunner$CallableBuildOperationWorker.execute(DefaultBuildOperationRunner.java:204)
	at org.gradle.internal.operations.DefaultBuildOperationRunner$CallableBuildOperationWorker.execute(DefaultBuildOperationRunner.java:199)
	at org.gradle.internal.operations.DefaultBuildOperationRunner$2.execute(DefaultBuildOperationRunner.java:66)
	at org.gradle.internal.operations.DefaultBuildOperationRunner$2.execute(DefaultBuildOperationRunner.java:59)
	at org.gradle.internal.execution.steps.RemoveUntrackedExecutionStateStep.execute(RemoveUntrackedExecutionStateStep.java:32)
	at org.gradle.internal.execution.steps.RemoveUntrackedExecutionStateStep.execute(RemoveUntrackedExecutionStateStep.java:21)
	at org.gradle.internal.execution.steps.legacy.MarkSnapshottingInputsStartedStep.execute(MarkSnapshottingInputsStartedStep.java:38)
	at org.gradle.internal.execution.steps.LoadPreviousExecutionStateStep.execute(LoadPreviousExecutionStateStep.java:43)
	at org.gradle.internal.execution.steps.LoadPreviousExecutionStateStep.execute(LoadPreviousExecutionStateStep.java:31)
	at org.gradle.internal.execution.steps.AssignWorkspaceStep.lambda$execute$0(AssignWorkspaceStep.java:40)
	at org.gradle.api.internal.tasks.execution.TaskExecution$4.withWorkspace(TaskExecution.java:281)
	at org.gradle.internal.execution.steps.AssignWorkspaceStep.execute(AssignWorkspaceStep.java:40)
	at org.gradle.internal.execution.steps.AssignWorkspaceStep.execute(AssignWorkspaceStep.java:30)
	at org.gradle.internal.execution.steps.IdentityCacheStep.execute(IdentityCacheStep.java:37)
	at org.gradle.internal.execution.steps.IdentityCacheStep.execute(IdentityCacheStep.java:27)
	at org.gradle.internal.execution.steps.IdentifyStep.execute(IdentifyStep.java:44)
	at org.gradle.internal.execution.steps.IdentifyStep.execute(IdentifyStep.java:33)
	at org.gradle.internal.execution.impl.DefaultExecutionEngine$1.execute(DefaultExecutionEngine.java:76)
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeIfValid(ExecuteActionsTaskExecuter.java:139)
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.execute(ExecuteActionsTaskExecuter.java:128)
	at org.gradle.api.internal.tasks.execution.CleanupStaleOutputsExecuter.execute(CleanupStaleOutputsExecuter.java:77)
	at org.gradle.api.internal.tasks.execution.FinalizePropertiesTaskExecuter.execute(FinalizePropertiesTaskExecuter.java:46)
	at org.gradle.api.internal.tasks.execution.ResolveTaskExecutionModeExecuter.execute(ResolveTaskExecutionModeExecuter.java:51)
	at org.gradle.api.internal.tasks.execution.SkipTaskWithNoActionsExecuter.execute(SkipTaskWithNoActionsExecuter.java:57)
	at org.gradle.api.internal.tasks.execution.SkipOnlyIfTaskExecuter.execute(SkipOnlyIfTaskExecuter.java:56)
	at org.gradle.api.internal.tasks.execution.CatchExceptionTaskExecuter.execute(CatchExceptionTaskExecuter.java:36)
	at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.executeTask(EventFiringTaskExecuter.java:77)
	at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.call(EventFiringTaskExecuter.java:55)
	at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.call(EventFiringTaskExecuter.java:52)
	at org.gradle.internal.operations.DefaultBuildOperationRunner$CallableBuildOperationWorker.execute(DefaultBuildOperationRunner.java:204)
	at org.gradle.internal.operations.DefaultBuildOperationRunner$CallableBuildOperationWorker.execute(DefaultBuildOperationRunner.java:199)
	at org.gradle.internal.operations.DefaultBuildOperationRunner$2.execute(DefaultBuildOperationRunner.java:66)
	at org.gradle.internal.operations.DefaultBuildOperationRunner$2.execute(DefaultBuildOperationRunner.java:59)
	at org.gradle.internal.operations.DefaultBuildOperationRunner.execute(DefaultBuildOperationRunner.java:157)
	at org.gradle.internal.operations.DefaultBuildOperationRunner.execute(DefaultBuildOperationRunner.java:59)
	at org.gradle.internal.operations.DefaultBuildOperationRunner.call(DefaultBuildOperationRunner.java:53)
	at org.gradle.internal.operations.DefaultBuildOperationExecutor.call(DefaultBuildOperationExecutor.java:73)
	at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter.execute(EventFiringTaskExecuter.java:52)
	at org.gradle.execution.plan.LocalTaskNodeExecutor.execute(LocalTaskNodeExecutor.java:69)
	at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$InvokeNodeExecutorsAction.execute(DefaultTaskExecutionGraph.java:327)
	at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$InvokeNodeExecutorsAction.execute(DefaultTaskExecutionGraph.java:314)
	at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$BuildOperationAwareExecutionAction.execute(DefaultTaskExecutionGraph.java:307)
	at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$BuildOperationAwareExecutionAction.execute(DefaultTaskExecutionGraph.java:293)
	at org.gradle.execution.plan.DefaultPlanExecutor$ExecutorWorker.execute(DefaultPlanExecutor.java:420)
	at org.gradle.execution.plan.DefaultPlanExecutor$ExecutorWorker.run(DefaultPlanExecutor.java:342)
	at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:64)
	at org.gradle.internal.concurrent.ManagedExecutorImpl$1.run(ManagedExecutorImpl.java:48)
Caused by: org.owasp.dependencycheck.exception.ExceptionCollection: One or more exceptions occurred during analysis:
	DatabaseException: Unable to connect to the dependency-check database
		caused by DatabaseException: Database schema does not match this version of dependency-check
		caused by DatabaseException: Database schema is out of date
	at org.owasp.dependencycheck.Engine.throwFatalDatabaseException(Engine.java:746)
	at org.owasp.dependencycheck.Engine.initializeAndUpdateDatabase(Engine.java:712)
	at org.owasp.dependencycheck.Engine.analyzeDependencies(Engine.java:631)
	at org.owasp.dependencycheck.Engine$analyzeDependencies$0.call(Unknown Source)
	at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze.analyze(AbstractAnalyze.groovy:97)
	... 116 more

Version 8.0.1 is failing with another error:
Some of the file system contents retained in the virtual file system are on file systems that Gradle doesn't support watching.
Caused by: org.gradle.api.GradleException: Analysis failed.
Caused by: org.owasp.dependencycheck.exception.ExceptionCollection: One or more exceptions occurred during analysis:
	DatabaseException: Unable to connect to the dependency-check database
		caused by DatabaseException: Database schema does not match this version of dependency-check
		caused by DatabaseException: Database schema is out of date

Stacktrace

As the error message says: you have to update your database schema. Have a look: https://github.com/jeremylong/DependencyCheck/tree/main/core/src/main/resources/data

@jeremylong confirmed working OK in 8.0.1


[SMB] Powershell history module rework

Hey @357384n !
As mentionned in your PR, I love this module. This is going to be pretty useful!
However I believe we can remove the powershell_execute part as it raises an unecessary process creation that can be flagged by an EDR.
To replace it, I manually crawl c:\Users and look for powershell history files.
On a second hand, I don't believe it is a good idea to look for specific key words as you can miss quite a lot of informations. Let's say a user types the "net user Administrateur Defte@WF" command, you won't see it.
As such I left the grep option but still print the entire powershell history, see:

I also removed redundant key strings such as password and passw when you can already grep for "pass".
Finally, and to be honest, I think we should enforce the export to a file option as I believe a lot of people will spray this module and will want to have an export somewhere.
Let me know what you guys think about that :P

Hey @Dfte !
As discussed in DM thanks for your update. You're right it's quite more efficient to get files without PS commands.
In my opinion it could be nice to print only the history which is matching with some keywords and download all the content to files which can be analysed later. Can be did after the export with grep too.
In intern pentest if you do it on a large range you can waste a lot of time to scroll up and down but not all the history is interesting.


UTF8 + B64 not decoded

Reference: https://github.com/soywod/himalaya/issues/380
Here the raw string:
=?UTF-8?B?UmU6IFlvdXIgQXBw4oCMbOKAjGUgSeKAjEQgaGFzIGJlZW4gbG9ja2XigIzigIxk4oCM4oCMIG9uIFRodXJzZGF5LCBNYXksIDE5IDIwMjIgW3JlZjpfMjU4NzQ2XQ====?=

For some reason, it cannot be decoded :thinking: (should be Re: Your Apple ID has been locked on Thursday, May, 19 2022 [ref:_258746]).

In fact it is a regression, because I updated the lib inside Himalaya and I have now subject that are not decoded anymore, like:
=?utf-8?B?QUdFTkNFIElNTU9CSUxJRVJFIERFUyBWQUxMRUVTIDogSW52aXRhdGlvbiDDoCBzaWduZXIgw6lsZWN0cm9uaXF1ZW1lbnQ=?=

or:
=?utf-8?B?QUdFTkNFIElNTU9CSUxJRVJFIERFUyBWQUxMRUVTIDogU2lnbmF0dXJlIMOpbGVjdHJvbmlxdWUgZmluYWxpc8OpZQ==?=

The problems comes from the max length, all those subjects that cannot be decoded contain more than 76 chars #1. So it looks like in theory it is not valid, but in practice email providers do not care about this rule :disappointed:

Should be fixed by #17.


Topic/v1 notes

Notes about what needs to be done and things and so on

also document what type becomes custom C types? these should be iarray_t, ibuf_t, imap_t afaict, these ADTs can go in data/sea/ too


A food restaurant website

Describe the project you want to add with tech stack

I want to add a project related to food restaurant website
Tech Stack
Html, Css, Js
Expected behavior

Screenshots (optional)

Additional context (optional )

This will  be my first contribution in this repo so please assign this issue to me @pranjay-poddar  under GSSOC'23.


Error

OS: MacOS Ventura
Hi, I am trying to export a demo monkey but get error on line 346 in init subprocess.check_output([luajit_cmd, dirpath, os.path.abspath(dir)]).

Hi, can you check a couple of things first.

did you make sure the luajit binary that was installed in the blender/addons/defender/defoldsync/luajit/darwin/ is able to be executed by your logged in user.
also check on the macOS that you can run that luajit from the command line: try something like luajit -v. You should see a response like "LuaJIT 2.0.4 -- Copyright (C) 2005-2015 Mike Pall. http://luajit.org/"
Hopefully it will be related to these (Check the documentation in the main page or the defold forum - we have covered many issues there).
https://forum.defold.com/t/building-a-sync-tool-for-blender-to-defold/69920/43

No response. marking as resolved.


Open Matching URLS in another nativefier app.

When using Nativefier to wrap several web apps, it would be ideal to be able to assign a regex that when a URL matches It in the current nativefier, opens that URL in another Nativefier app.  This could be implemented in a number of ways, including using the feature requests: #314 & #405

@briancordanyoung thanks for the feedback 🙂. Yup, supposing a fix for #405 and combining with --internal-urls you could do that. Closing, feel free to follow #405.


KAFKA-14133: Migrate remaining mocks in StoreChangelogReaderTest to Mockito

This pull request takes a similar approach to how TaskManagerTest is being migrated to Mockito mock by mock for easier reviews.

The modified test is successful https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-15125/1/testReport/org.apache.kafka.streams.processor.internals/StoreChangelogReaderTest/


Cache walking distance calculations

The walking distance calculation is very time consuming (4+ hours) due to the rate limit imposed by Mapbox. In reality we only need to run this once per pair of locations (building+station). Stations and buildings rarely move (mostly in cases of data errors not actually due to moving). However, if the lat/lon of a building or station changes (though the ID of that station did not change) we do want to rerun the calculation.
Here's a proposed quick solution that has minimal impact on the rest of our workflow as it doesn't require us to do tricky state management of changing lat/lons for specific ids.

Add the 4 numbers for start and end lat/lon to the wmata_dist table, to cache the specific lat/lon values that were pulled from the project and wmata_info tables when the distance was calculated.
Add an optional argument to WmataApiConn.init for 'use_cached_distance' and set it to True by default. if self.use_cached_distance is true, we don't need to get the walking distance if we already have it. if False, we'll behave as normal and get fresh walking calculations for all points.
Edit the self._get_walking_distance method to first query the database for a row corresponding to that exact set of 4 lat/lon values. If the wmata_dist table does not exist (i.e. we are building it the first time or want fresh data) or if there is no matching row (either because it's a new building or because one of the lat/lon values has been changed) proceed with getting the walking distance from the API.

@jgordo04 If you're not too sick of working on distance calculations - this and #312 are two enhancements so that our server won't have to run the wmata_dist code for 5 hours every week!

No prob. Happy to!

Cleaning up related data ingestion issues - closing because it looks like this has been completed.


Feature: Showing all bugs/fish that spawn that day, regardless of time

I think it'll be useful to have an option to list all the fish and bugs that can show up that day, ignoring the time. This will enable people to plan when they need to play in order to get all the fish.
Maybe while it's shown, have that row greyed out or something to let them know it's not available at that specific time.
I can probably work on this in a few days when I have the time, let me know if you think it's a desired feature

I love that suggestion and was thinking of it and I think your implementation is a great solution!


A hidden mouse cursor doesn't re-appear when hovering the title bar

Subject of the issue
A hidden mouse cursor doesn't re-appear when hovering title bar or resize area of a window.
Your environment
Windows 10
git master
Visual Studio 2015
Steps to reproduce
#include <SFML/Window.hpp>
int main()
{
    sf::Window window(sf::VideoMode(400, 300), "Cursor Bug");
    window.setMouseCursorVisible(false);
    while (window.isOpen())
    {
        sf::Event event;
        while (window.pollEvent(event))
        {
            if (event.type == sf::Event::Closed)
                window.close();
        }
        window.display();
    }
}

Expected behavior
Cursor should re-appear when over title bar.
Actual behavior
It doesn't re-appear.

From my experience this heavily depends on the individual program/app and isn't consistent at all. I'd at least say it would make sense, since the title bar still reacts to the cursor. How does it look on Linux and MacOS?

On ubuntu, with GNOME, cursor re-appeared on the title bar.

Won't be back at work computer until next week but I wonder if setting a blank sf::Cursor will result in the same behaviour, and if not, could be a suitable workaround.

Won't be back at work computer until next week but I wonder if setting a blank sf::Cursor will result in the same behaviour, and if not, could be a suitable workaround.

Okay, so it seems that if you set a sf::Cursor with a blank image, then the default windows cursor does re-appear when hovering the title bar. So that's a workaround for this issue.
I've also attached a patch which fixes this issue, if you deem it a bug.
Cheers

Is this closed by #1570 then?


removal of PKG_CONFIG_PATH in Cargo.toml

Hi, could you remove
https://github.com/rcastill/mat2image/blob/22356668a92aeeade30294fff1fd88647662f88c/Cargo.toml#L20
It causes my build to fail :(

Change available in 0.1.1

thanks :)


fix(AV-1112): Dataset category linking

On the dataset page, the categories now link to the matching category page.

Closing due to changes being implemented in another pull request


WIP: Add support for org.asynchttpclient

WORK IN PROGRESS
This PR adds tracing support for org.asynchtclient.
TODOs

Test cases for redirects are not pasing. Need to investigate how the client configuration is supposed to be or work around by adding some tricks into the APM instrumentation to keep the tracing headers

Superseded by https://github.com/elastic/apm-agent-java/pull/625. Let's continue the discussion there 🙂


imported save_in_gallery/save_in_gallery-Swift.h does not exist

https://github.com/FiveDotTwelve/save_in_gallery/blob/e222159f29be64ededaccf0c96a4e77e5a9bdbdb/ios/Classes/SaveInGalleryPlugin.m#L2

Sorry for the very concise issue report before ;)
Thanks for your package. It works as expected on Android.
However, when building on iOS we encounter 2 issues:

fatal error: 'save_in_gallery/save_in_gallery-Swift.h' file not found
no swift version is provided. We therefor setup a project-wide swift version, but this feels a bit hacky (non-blocking issue).

Thanks for looking into this swiftly (pun intended ;)
Kind regards,
Pieter

Hi,
This problem occurs with flutter apps that do not have swift support (set up when you create the project), there is a work around but it hasn't always worked for me.
https://github.com/ko2ic/image_downloader/issues/9#issuecomment-471608482 details the steps.
For my currently project I found the only thing was to recreate the project and ensure swift and Kotlin support was added.
hope this helps.


Add generic to gRPC

To make terraform-provider-matchbox works with kickstart templates (https://github.com/coreos/matchbox/issues/224), we need generic on gRPC.
This is also needed for https://github.com/coreos/terraform-provider-matchbox/issues/16

Yes, adding generic templates into the gRPC API alongside Container Linux configs is in-scope to provide this functionality to gRPC clients. I've added an issue at https://github.com/coreos/matchbox/issues/562.

Please squash and follow the commit message format. Also, I think if you run make codegen the diff is slightly different. Its nice you've updated the bootcmd example client (although we don't ship it currently), thanks.
How have you tested?
./bin/bootcmd --endpoints matchbox.example.com:8081 generic create -f arbitrary.thing
Error:  rpc error: code = Unimplemented desc = unknown service rpcpb.Generic

You'll need to register the server in grpc.go

Closes #562 once complete

I didn't test trough the bootcmd, I tested straight on the terraform-matchbox plugin with the changes I made. I will send the terraform PR soon as I have this merged. I'm sending the squash commit.

Squashing refers to (rebasing and) squashing changes into one commit.

sorry, I made a merge before pushing the squash. just fixed.

Thanks, LGTM.
Small note, generally when you do a feature branch and rebase/squash, you should not end up with a merge "bubble" so the proposed changes are on top. http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html This will merge as https://github.com/coreos/matchbox/tree/add-generic


Intellisense/autocomplete is slow

I love your extension and have used it a long time, thanks.
A few weeks ago, I installed a few updates, including a new version of the solidity extension.
Since then the autocomplete feature has become very slow (maybe 10 s. for a local variable or keyword). Compile time and other functionality is unchanged, but the lookup functionality is severely crippled.
I used other solidity extensions for some weeks now, but in my mind they are just not as good. Today I tried disabling all other extensions, disabled "as you type compilation..", disabled linting, disabled formatting etc. It still didn't help.
I finally reverted to "solidity  v0.0.141" extension version, and everything works well again.
Even with the previous mentioned functionality re-enabled, autocomplete/lookup is still fast and efficient.
I will be happy to help out providing info on my setup, if it is of any help to you.
Let me know what you want me to send/test.
Again, thanks for the nice work.

Same issue

dupe of #401 ?

hi everyone, hopefully it is fixed now with the latest releases, let me know if you have further issues.

hi everyone, hopefully it is fixed now with the latest releases, let me know if you have further issues.

Thanks, it's really appreciated.
I have approx 1 sec delay for the auto-complete suggestions to appear now.
It's a little slower than the 0.0.141 version, but it's very acceptable.

@MlSportsbook what libraries are you using? ie open zeppelin etc..

"@ethersproject/abi": "^5.4.7",
"@ethersproject/bytes": "^5.4.7",
"@ethersproject/providers": "^5.4.7",
"@nomicfoundation/hardhat-chai-matchers": "^1.0.6",
"@nomiclabs/hardhat-ethers": "^2.0.5",
"@nomiclabs/hardhat-etherscan": "^3.0.3",
"@openzeppelin/contracts": "^4.6.0",
"@openzeppelin/contracts-upgradeable": "^4.8.1",
"@openzeppelin/hardhat-upgrades": "^1.22.1",
"@typechain/ethers-v5": "^10.0.0",
"@typechain/hardhat": "^6.0.0",
"@types/chai": "^4.3.1",
"@types/mocha": "^10.0.1",
"@types/node": "17.0.31",
"chai": "^4.3.6",
"chai-ethers": "^0.0.1",
"create-react-app": "^5.0.1",
"dotenv": "^16.0.0",
"eth-crypto": "^2.3.0",
"ethers": "^5.7.2",
"fs": "^0.0.1-security",
"hardhat": "^2.9.3",
"hardhat-contract-sizer": "^2.8.0",
"hardhat-deploy": "^0.11.20",
"hardhat-ethers": "^1.0.1",
"hardhat-gas-reporter": "^1.0.8",
"hardhat-preprocessor": "^0.1.5",
"metamask-react": "^2.3.0",
"path": "^0.12.7",
"solidity-coverage": "^0.8.2",
"ts-node": "^10.7.0",
"typechain": "^8.0.0",
"typescript": "^5.0.3"

I am not sure exactly what can influence it, so this is my full packagelist.
I can test some other, simpler projects, if it helps?


Updates broke data file formats

At the least, cultureMap.txt and slaveCultureMap.txt were broken by https://github.com/ParadoxGameConverters/EU4toVic2/commit/c98ab4dde3bea3c3743ab51fa5fe82365e597324 and https://github.com/ParadoxGameConverters/EU4toVic2/commit/d059133556d0957ae97307360d3a50c97ea6068a respectively.

CountryMappings.txt was broken by https://github.com/ParadoxGameConverters/EU4toVic2/commit/05637dc8b8d4978e578e180facbffeebf226488b#diff-ce1db9c37fbb46f5a5afefd4adc0f281, but not as badly

Ck2TitleMap.txt was slightly broken.


Not able to Download playlist.

Checklist

[ ] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--extract-audio', '--audio-format', 'mp3', '--verbose', '--output', 'Music/YouTube/%(playlist_title)s/%(playlist_index)s - %(title)s.%(ext)s', 'PLhKO-ucc19G8hIYGNkHyeQouMBtmMQO_V']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.22621
[debug] exe versions: ffmpeg 5.1.2-full_build-www.gyan.dev, ffprobe 5.1.2-full_build-www.gyan.dev
[debug] Proxy map: {}
[youtube:tab] PLhKO-ucc19G8hIYGNkHyeQouMBtmMQO_V: Downloading webpage
[download] Downloading playlist: english
[youtube:tab] playlist english: Downloading 26 videos
[download] Downloading video 1 of 26
[youtube] DKbfBSrjVHA: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 815, in wrapper
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 836, in __extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 534, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\youtube.py", line 1794, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 1012, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

WRITE DESCRIPTION HERE
My Bat file
for %%a in (%*) do (
.\youtube-dl.exe ^
--extract-audio ^
--audio-format mp3 ^
--verbose ^
--output "Music/YouTube/%%(playlist_title)s/%%(playlist_index)s - %%(title)s.%%(ext)s" ^
%%a
)

Command
Try.bat https://youtube.com/playlist?list=PLhKO-ucc19G8hIYGNkHyeQouMBtmMQO_V
youtube-dl is up-to-date (2021.12.17)

This is a broken site support issue, isn't it? Either way, same problem here on WSL ubuntu 20.02: tried to download a youtube playlist with version 2021.12.17, couldn't extract uploader ID. Verbose output here, command manually entered for debugging.

This is a broken site support issue, isn't it?

Yes.

[x] I've searched the bugtracker for similar bug reports including closed ones

Learn how by reading #30839 (again?).

[x] I've read bugs section in FAQ

Apply this.


Feature: add Middleware based on an HTTP response

I needed those today and I though it could be useful to other people as well. Feel free to close if they do not belong in zio-http

Codecov Report

Merging #1377 (68d1b55) into main (66b571c) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1377      +/-   ##
==========================================
+ Coverage   60.79%   60.82%   +0.03%
==========================================
  Files          71       71
  Lines        2482     2484       +2
  Branches       84       83       -1
==========================================
+ Hits         1509     1511       +2
  Misses        973      973

Impacted Files
Coverage Δ

...ttp/src/main/scala/zhttp/http/middleware/Web.scala
82.92% <100.00%> (+0.87%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.


Which one should I choose

Odyssey or unc0ver ?
If @coolstar does not reply me in 6h ,I will use unc0ver

Odyssey or unc0ver ?
If @coolstar does not reply me in 6h ,I will use unc0ver

Definitely Odyssey.
Why should he reply to you?


[release-1.4] Wait key cert ready when auto mTLS is enabled and SDS is not enabled.

This is an automated cherry-pick of #18108

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio_release-1.4
8358599238f8c174d3ad20bae069e2e3a23609ed
link
/test integ-telemetry-k8s-tests_istio_release-1.4

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.

/test integ-telemetry-k8s-tests_istio_release-1.4

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio_release-1.4
8358599238f8c174d3ad20bae069e2e3a23609ed
link
/test integ-telemetry-k8s-tests_istio_release-1.4

integ-istioio-k8s-tests_istio_release-1.4
8358599238f8c174d3ad20bae069e2e3a23609ed
link
/test integ-istioio-k8s-tests_istio_release-1.4

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.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioio-k8s-tests_istio_release-1.4
8358599238f8c174d3ad20bae069e2e3a23609ed
link
/test integ-istioio-k8s-tests_istio_release-1.4

integ-mixer-k8s-tests_istio_release-1.4
8358599238f8c174d3ad20bae069e2e3a23609ed
link
/test integ-mixer-k8s-tests_istio_release-1.4

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

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-tests_istio_release-1.4
8358599238f8c174d3ad20bae069e2e3a23609ed
link
/test integ-mixer-k8s-tests_istio_release-1.4

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


Ubuntu PPA

I have created an official Debian package which can be ported to older Ubuntu versions:
https://tracker.debian.org/pkg/libopenhmd

Wow, that's awesome!
You mentioned something about needing XML for hardware detection. What is it you need us to do?

quoted from the e-mail, for the record:
As suggested by Paul Wise, Isenkram on Debian could recommend installing the libopenhmd package
when a supported device is attached to the computer.
Please help by shipping the needed xml files.

this link was also provided:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=845272#10

Using backportpackage to maintain this Ubuntu PPA https://launchpad.net/~theonlyjoey/+archive/ubuntu/openhmd. Can be closed.


Add host archive and host-only object file to test.

This adds host-only code in both an archive and object file given on the command line.

Lgtm


Add a space after a full stop

This commit ensures that there is a space after a full stop when appropriate. This makes the grammar in this document consistent.

Hi! Thank you very much for taking your time on contributing to our docs. These missing gaps are caused by our migration to a different documentation platform: previously, there were soft returns on their place. I will now merge your PR.


KeyError in msl_handler.py when trying to play movie

Hello,
As suggested here https://github.com/peak3d/inputstream.adaptive/issues/282 I'm posting this error after trying the daily build as of today.
edited as @Smeulf comment:
I hope this is the relevant error: TypeError: 'NoneType' object has no attribute ['__getitem__']
Relevant error: msl_handler.py", line 353, in _decrypt_chunks decrypted_payload = decrypted_payload[1]['payload']
Here are the logs: http://ix.io/1MDq

HI.
The error you're pointing is not the revelent one. That first error is because it's a movie, so no episode is following. It's catched as expected.
Here you have an error in msl_handler.py line 353. Unfortunately this is a part of the addon I'm not yet familliar with, so you'll have to wait until @CastagnaIT can have a look, sorry.
Thanks for the report.

YEp, the errror you are pointing is Next Up related, the real issue is while decrypting MSL resonses, My opinion is that you update he AddOn, but you kept the old settings files or you did not reboot after update it,
Please delete the COOKIE file, msl_data.json, manifest.json and manifest.mpd, disable Up Next by  now, then use default skin and finally reboot and try again.
Please share your findings

@jakermx, thanks for helping me out. I'm using the daily build version as suggested by @CastagnaIT, you mean upgrading to which version?

@aleprovencio
for this problem i need more data,
please install this test version, run addon (with kodi debug info enbled) repeat the operations that cause the problem and post a new log
plugin.video.netflix_TEST.zip

done, here you go: http://ix.io/1MKO

thank you but need other test with this new test2,
Unfortunately i was not able to decode the data
plugin.video.netflix_TEST2.zip

Done use this and post the log
plugin.video.netflix_TEST3.zip

thanks, here the logs: http://ix.io/1MMm

"INSUFFICICENT_MATURITY" says the log...
But that's good to know how it behaves in this case.
If you're the owner of the account, you should take a look on your profile settings and unlock the maturity level

@CastagnaIT does that also means results for categories and search are not filtered by maturity? It shouldn't even be visible if a maturity restriction has been configured.

@Smeulf, if you're talking about the parental control, I had not set a PIN until now. I tried setting one up but leaving all content as allowed, ran the tests again and it doesn't work either.

On NF website, select manage profile, then edit your profile with the pen.
You should see 'allowed tv shows and movies', and you can select the maturity level here. I bet it's not set to 'all maturity levels' but to a lower value.
It's not directly related to the pin code.

Ha OK I missed that, but yes, it is already set to all maturity levels. In addiction I'm able to watch this tested movie on the netflix web or the TV app for instance.

Looks like you have 4 profiles defined on NF. Are you sure you're using the right one with Kodi ? If I'm worrect you"re using 'Xand' profile.
Maybe that's a dumb question, but it worth to ensure ^^

Don't worry about asking any questions :)
You're correct, I have 3 profiles and the additional kids profile that netflix creates, and yes, I'm using the 'Xand' one.

Well, for now, I'm running out of idea...
First, let's get some time to @CastagnaIT to fix the error. Then maybe he can tell what's going on.
NF says it's related to parental control, there's a help link in the logs. But if you're telling me you can see this video with the same profile on some other devices, that's very strange to me... Thanks for all the details!

hi
yes is parental control it looks like netflix thinks your profile is set to a lower level age
try to switch to 'teen' enter, play a video, exit, swith to normal, retry...
nothing can be done at the addon level
I did some cleaning now if it happens again the error popup should appear correctly

It's so strange, I still can't manage it to work, Here's what I have done:

Changed the 'Xand' profile to a lower level age, played a movie (via web), put the maturity level back
Created a new profile with 'all maturity level' set

Both do play the movie via web but not with the addon.
Crazy! Please help!

Can you please post screen capture of:

your profile settings
your parental management control settings (you can blur the pin code/any personal data if you want, or change the pin before posting)

And can you give us the English name of the movie you're trying to watch?
I wish to try to reproduce the issue with the exact same settings you use.
Parental control can also limit access by name. Please ensure you didn't add it to that list.

Sure. I've changed the language to English, this is me watching the tested movie on netflix web:

Thanks. It's making me crazy too... Here it's rated 13+ so your settings looks good.
If you don't use the Kodi library export, you still can try to remove the add-on, ensure plugin.video.netflix  folder is deleted in userdata/addons_data or delete it manually, and reinstall the addon.
It will flush all settings for the add-on. I'm not sure it helps but it worth a try regarding the situation.

Hello @Smeulf, done that but issue stays.
I've noticed that I can watch the available movies when logged in with the kids profile.  :/

Well, I'm really sorry but I don't know what to say...

Thanks for trying anyway. :)
Taking a close look on the second picture I posted above, I noticed that on the left side of the profile "angry bird" icon there is a "kids" text.
Do you know what is this or if it is somehow related to the problem?

I don't think so. It's just a shortcut for the kids profile, I have the same.

Hello @Smeulf, done that but issue stays.
I've noticed that I can watch the available movies when logged in with the kids profile. :/

explains better
how do you play a movie available in normal profile from a kid profile?

Sorry about not being clear enough @CastagnaIT.
What I meant is the other way around: I can watch all (at least what I have tested) the movies available on kids profile. With some exceptions I've also noticed (i.e. Heal 2017), I am not able to watch all other movies.
Where can I find out the maturity level of a given movie?

Here are the logs for Heal (2017), which do work. http://ix.io/1Nwq

aleprovencio did you enabled PIN code in the addon?
If in Netflix you kept pin protection off, than you don't need the PIN to be enabled in the addon. You should then disable it and try again.

The PIN was as default disabled, @Smeulf. Anyways, just in case, I just did the test enabling/disabling this setting both on netflix web and addon but unfortunately leading to no good.
Thanks again.

try install this test version, open netflix using the normal profile not Kid, so put a new log
it is strange that you are the only one having this problem, the only thing that comes to mind is a particular configuration of your profiles that cause some type of unexpected behavior then with this test maybe... i can understand something more
plugin.video.netflix_TEST1.zip

@aleprovencio maybe you can also try to disable "Test participation" on your account...
I don't have the same levels of maturity, mine are only "All", "7+", "13+" and "16+", but maybe it's country related.

I don't know what is happening but I can't login with my credentials anymore, even after uninstalling answering yes to erase settings and rebooting. Is there other things I must remove for it to work?

I don't know it's very strange...
if you have also confirmed the removal of the settings there should be nothing else.
if web access still works, the problem is elsewhere
are you sure you were connected to the internet after you re-installed the addon?

yes, other things from kodi connect :(
I can't think of anything for now...Will keep trying

This is just crazy, I just reinstalled LibreELEC from scratch, put on language/timezone settings, then installed the zip you provided and...can't login.
I always receive an email from netflix telling me there's a new access to its account.
And netflix web is still fine.
Summing this up with the main issue, I'm starting to think there's problem with my account on netflix and i might delete and create a new one.

Changing your ESN and deleting the COOKI file, should resolv the issue.
Manifest Loadinng Proccess does not involves Profile ID or Token, it is based on ESN, once the ESN is logged and is associatted with a Profile, it will do anything with the ESN, and since the cooke is set to never expire, it will ask anything to Netflix as the original user, in order to fis this, COOkIE needs to be renewed and I would suggest, set ESN as NFCDH-01-PROFILE_TKN_ID.

The Issue you posted on inputstreamadaptive was locale_id related, then you update your version twice, and the error changed, may I ask... did you uninstall with setttings delete before install the new versions and the TEST versions?...if not, please uninstall, click on yes with asked for delete settings, then reboot your system, install the lasted master build or daily one, and then try again....deleting settings does what I am suggesting.

Thanks for trying, @jakermx .
I've changed ESN, but since this is a new installation on a fresh LibreELEC there were no cookie files to delete.
I still can't login, its maybe related to #126.
About your last comment, the error actually did not change, it got just more evident after some adjustments on the plugin's test source. I've always followed the procedure you described on doing new installs, OK.

Did you got a lot of login emails from Netflix during this troblueshooting?

Hello @jakermx,
first of all thanks for your time and sorry about taking so long for me to do new tests, I couldn't get to my kodi box until now.
One more time I reinstalled LibreELEC and this addon from this repository from scratch.  I followed your advice to disconnect all my devices, and it worked fine, I can login again now. And yes, I was receiving emails from Netflix.
The parental control issue stays, I only can watch movies for kids. Here are the logs for a movie I can't watch: http://ix.io/1P59
Take note this is latest version from repo, not one of the 'test' versions provided on this thread.

do this:
open netflix page with Google Chrome
login and stop it, in the page of profile selection
Press F12 key to open dev page,
in the html code press CTRL+F to open search box
copy paste:
EFC3PECLRVDDJJNAJVHEDY537A
go down to a big text that starts with netflix.reactContext, then
right click on the big text to show the context menu and select "Copy Element"
then paste such as "link" same you do with log

here you go @CastagnaIT http://ix.io/1P6v

Hello @jakermx and @CastagnaIT, still no clues about this? You think I should delete my netflix account start a new one?

sorry unfortunately i didn't have enough time to check maybe over the weekend

your main profile "Lucas" is marked "child" this can be the cause

OMG it works !!!
I never knew there was a main profile account, but if it ever existed I would presume it was my profile since it was the first created. Somehow netflix has put my nephew's (child) account as the main one, and this way I could not access movies with higher maturity levels from any profile from this addon (but could from netflix web and other). Maybe this deserve a Readme/FAQ entry?
Anyways, this was such a long standing issue for me and you just nailed it @CastagnaIT, thank you so much!  I must not forget @jakermx who also borrowed a lot of its time trying to help me with this, thank you!


add before_layer_change as profile property

adds another textbox to the advanced tab to show before_layer_change.
This helps with the layer display plugin, which needs ;LAYER:[layer_num]

I'm so sorry for the delay! @vvvrrooomm
I'm looking for developer to take over this project from me, as I'm currently really busy with TSD. https://github.com/TheSpaghettiDetective/TheSpaghettiDetective
It looks like you are very familiar with the code of this plugin. If you want, you can take over this project and become the maintainer.
Let me know!

sorry, I don't work enough with this plugin to becoming a maintainer. This was just a fix for the warning of the DisplayLayerPlugin

No problem. just merged it assuming you have fully tested it. :)


Styling of Assessment table

[x] Remove border around drop downs

[x] Fix font - change to BC Sans

[x] make each CCBC ID color: #1A5A96

[x] hover state for CCBC ID hyperlink is color: blue and underlined

[x] Fix permitting assessment filter alignment

[x] After clicking on the hyperlink & on the assessments page, when "Dashboard" is clicked in the sidebar, it brings the user back to the assessment table, the filtering & sorting that were applied before they navigated away remain

[x] Remove words 'Assessment'  from headers  and narrow columns to show first 9 characters

Please add your planning poker estimate with Zenhub @marcellmueller

Please add your planning poker estimate with Zenhub @AntBush

@marcellmueller  can you please see if this AC is working for you:  After clicking on the hyperlink & on the assessments page, when "Dashboard" is clicked in the sidebar, it brings the user back to the assessment table, the filtering & sorting that were applied before they navigated away remain

@marcellmueller can you please see if this AC is working for you: After clicking on the hyperlink & on the assessments page, when "Dashboard" is clicked in the sidebar, it brings the user back to the assessment table, the filtering & sorting that were applied before they navigated away remain

Just tried it in test and sorting and filtering is saved for me when I click the link and return, or refresh the page.

I clicked on the CCBC hyperlink in the assessment table, then I clicked 'Dashboard' and I was expecting to be brought right back to the assessment  tab but it brought me here
Private Zenhub Image

I clicked on the CCBC hyperlink in the assessment table, then I clicked 'Dashboard' and I was expecting to be brought right back to the assessment tab but it brought me here Private Zenhub Image

Oh, well the dashboard link brings you back to the root of the /dashboard which is the main dashboard. This is the issue with trying to squeeze in multiple complex acceptance criteria in one giant sentence.
In addition to that I am not really sure if that is a good idea having the same link do different things depending on where you came from.

@MarsRomer I made a separate ticket for that here:
https://github.com/bcgov/CONN-CCBC-portal/issues/2575
Re-reading the acceptance criteria in this ticket I am unsure if you wanted the table filters and sorting to be saved when hitting back or refresh or only when they click the dashboard tab.

thanks @marcellmueller  for the new ticket.  yes we want the filters and sorting to be saved when they hit back or refresh and when they click the dashboard tab


Doubt: Does this have AppStream support?

AppStream is a metadata protocol for Linux by Freedesktop.org. I noticed that it was for Linux and not universal, and I tried looking for GUI package managers on BSDs and couod only find the retired PC-BSD package manager. Then I saw in a OS derivation chart that GhostBSD is derived from TrueOS, and checked it to see that it has a package manager too!
Seems barebones like Synaptic, and I haven't tried this, but I wonder if you have AppStream working in any manner. I think it would be great to see AppStream support to systems outside the Linux space!

Ohh I see. But it's LGPLv2.1, not GPL -- so you can use it dynamically without having to GPLv2 Software Station itself.

As for GPL, let my try my hand at explaining it's reasoning even though you might know it. When something is permissively licensed, someone can modify it and sell it, but without giving it's source. But with copyleft, they have to give the source too. In both cases, the software can be marketed, like ONLYOFFICE. The difference is that with BSD, companies get the code for free and they build on top of it features that we can't have, which is like purposefully putting free software in a weaker spot.
I know if your vision is altruistic, because I too wanted to license by code under CC0 (yes it works for code unlike CC-BY). Then I realized I'm simply giving my work to companies just so that they can progress faster than free software ever can, and so I'd be encouraging bad behavior by doing that.
The problem is not about not contributing back, the problem is about what happens if they don't contribute back - and that's something I didn't notice myself for a long while. It's like working for free software with the belief that it will always be inferior to proprietary software, and fighting to ensure that unknowingly.


[WIP] Generate blacklight i18n labels into the application

I am not sure if this is the right group of fields, tbh, esp. re: rights, licenses, rights statements.
Still needs Spanish and Chinese translations, which I will do once I have clarity on the above.
Fixes #432

If you go this way, I think you should also remove the labels from the CatalogController.  I think the labels in CatalogController have precedence over i18n.

@jcoyne Gave that a shot, and added the missing translations.

@jcoyne Done. Ready for a final review.

@jcoyne all changes pushed. Open question: the license and rights_statement fields aren't covered in the catalog controller or i18n keys. Do we need entries for these?

@mjgiarlo probably not yet. @straleyb is working on it in a different pr.


Error com.ibm.watson.developer.cloudConversation

I have setup the ios project i am getting com.ibm.watson.developer-cloud.ConversationV1 Code=401 "Not Authorised"
UserInfo = (NSLocalizedFailureReason = Not Authorized)

Hi!  That potentially sounds like you are not providing the correct credentials for your Conversation Service instance on Bluemix.
Look in the CognitiveConcierge-iOS/CognitiveConcierge folder for the CognitiveConcierge.plist file.  Do you have the correct credentials supplied for the Conversation service?

Hi Belinda Vennam! I have given google place api key in plist file. I have install blue mix, I am not able to setup the server. Can you please help me for setup the server, I am new to blue mix.

Hi thanks. I am able to follow the instruction.
update Cartfile with.
github "watson-developer-cloud/swift-sdk"
3rd point in the Blue Mix setup I got confusion. I have to select the downloaded from your repo or select sample application from blue mix.
Vacations and shows. From where to pull for all the functionality.
Can you please help me.

Hi @sunilkumarj -- Vacation and shows does not have any functionality as this is actually just a starter app for you to get started and see how the various watson services work.


LOG OUT IF I WANT TO SEE A FLIP

if i tried to open a flip in same epoch i got this:

and if i tried to open a flip from another epoch its log me out and when i log in again i got this error:

issue solved after epoch 95


Fix broken link in README.md

@Retsam thanks for your PR. I can merge it if the CLA check above is green.

Fixed the commit message, all checks are green now.

Thanks


Reduce passes for a complete project build

For certain annotations we require two passes (e.g. bash: ./gradlew :Xtendroid:build ; ./gradlew :Xtendroid:build), for a complete project build. The IDE has no concept of double passes, so the IDE ends up showing premature errors.
I'm to blame for some of these, for instance @AndroidLoader, requires a double pass.
Also, afaik, @AndroidView.
In an ideal situation we should have everything built in one go, how do we get around this?

After commit 83f61a9, :Xtendroid (the main project) can build in one pass.
I haven't managed to reduce :XtendroidTest yet, to run :XtendroidTest:cAT.
Right now, one needs to do the following to build and test with one command:
./gradlew :Xtendroid:build && ./gradlew :XtendroidTest:build ; \
./gradlew :XtendroidTest:build ; ./gradlew :XtendroidTest:cAT

from a clean state.

I need to redesign my AsyncTaskLoader stuff.


Added triangularity attribute to matrix_cl

Summary
Added triangular_view_ attribute to matrix_cl. All OpenCL functions now respect and propagate this attribute. Especially add(), subtract() and scalar_mul() kernels were significantly modified.
Tests
New tests were added for add(), subtract() and .sub_block(). Some small changes to others.
Side Effects
enum class TriangularViewCL now has an additional possible value: Diagonal. TriangularViewCL now works as a bitfield: Lower | Upper = Entire, Entire & Lower = Lower, Diagonal | Lower = Lower ...
constants.hpp renamed to triangular.hpp
Checklist

[x] Math issue #1198

[x] Copyright holder: Tadej Ciglarič, University of Ljubljana
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[ ] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

So I just want to understand why this is important, I want to develop some literacy.
So openCL is the kernel that manages operations on the GPU. This pull request is to enable operations having to do with PSD or triangular matrices, on the GPU. Exploiting triangular matrices has large speed gains and memory gains. For a simple example, we've added GPU operations for addition for triangular matrices. It's obviously faster to sum the n(n-1)/2 operations than sum half of (or half +/1 the diagonal) a matrix of zeros. But these speed gains are even better on the GPU, could be useful for any matrix operations in the future. Just very generally applicable.
Is this correct? I can read the code to see how it's functioning, but hard to see how to improve it besides style, so I'm not really sure I can review this.

Is this correct?

No. First there are no memory gains. We still store whole matrix, even if we need only a triangular matrix. This is simpler and prrobably does not affect performance that much. Operations that are faster on triangular matrices were optimized in this way before this PR.
What this PR changes is that it lets the matrix track which part of it is actually in use. So that does not need to be specified when calling a function that operates on a triangular part. For example addition of two upper triangular matrices produces an upper triangular matrix. Some functions needed to be changed to only operate on triangular part if it is specified on input matrix. However I expect no or minimal benefits to performance.
OIn OpenCL terminology a kernel is any function that is called from host (CPU) and executes on OpenCL device (GPU). I don't know what PSD is.

We still store whole matrix, even if we need only a triangular matrix. This is simpler and prrobably does not affect performance that much. Operations that are faster on triangular matrices were optimized in this way before this PR.

This is also how Eigen deals with triangular matrices.

What this PR changes is that it lets the matrix track which part of it is actually in use. So that does not need to be specified when calling a function that operates on a triangular part. For example addition of two upper triangular matrices produces an upper triangular matrix. Some functions needed to be changed to only operate on triangular part if it is specified on input matrix. However I expect no or minimal benefits to performance.

Wouldn't it be a factor of 2?  Or does it defeat built-in CPU vectorization?
The bigger deal is in autodiff, where those extra operations go on the autodiff stack and then add unnecessary work to the reverse pass of autodiff.

Wouldn't it be a factor of 2? Or does it defeat built-in CPU vectorization?

Im not sure what you mean with defeat built-in CPU vectorization. I think you probably mixed something up, as we are discussing OpenCL code.
I did not measure speedup. It might be up to 2. These simple kernels are not very optimized anyway, as they are not expected to bottlenecks in many scenarios. In case they are it is usually better to combine addition with as much other operations as possible into a single kernel. This reduces the amount data that needs to be transfered to/from relatively slow global GPU memory.

I should've said GPU vectorization.  The reason I wonder is that the memory in a triangular matrix isn't contiguous in Eigen's representation. It uses just the upper or lower triangular part of a dense matrix.
I have it on good authority that minimizing back and forth to non-GPU memory is critical.  The Tensorflow Probability devs are trying to put everything on the GPU so you never have to leave the GPU.

Or does it defeat built-in GPU vectorization?

No, our implementation still launches threads for elements that are out of selected triangular parts. These threads just write zeros to appropriate elements of result matrix.

I have it on good authority that minimizing back and forth to non-GPU memory is critical.

That is true. But I don't understand how it is relevant for this PR.

For clarification on this PR, this is a style change (but one that sorely needs done).
Right now if we want to call the kernels for multiply that accounts for triangularity we have to call
matrix_cl C = multiply<triangular_view_cl::UPPER,
                       triangular_view_cl::LOWER>(A, B);

But with this PR we will be able to call matrix_cl C = A * B
Until the Stan language is very definite on how to generally handle sparse types (something I think I'll be working on with Dan Simpson and Aki starting July) eod we still treat this as a dense matrix with some cleverness taking into account the triangularity

I have it on good authority that minimizing back and forth to non-GPU memory is critical.  The Tensorflow Probability devs are trying to put everything on the GPU so you never have to leave the GPU.

Yes def, I've been racking over how to best keep things over there for a while. The best thing imo is that, when we move to Stan 3, Sean and I have talked about doing this within the compiler (i.e. building signatures that take in matrix_cl objects when doing rev. That and moving over to OpenCL 2.0 to get non-zero copy buffers for CPUs and APUs (Devices where the CPU and GPU are embedded in one another).
But that's a different PR, this one is to make the API much friendlier :)

I have it on good authority that minimizing back and forth to non-GPU memory is critical.
That is true. But I don't understand how it is relevant for this PR.

You're right---not relevant for this PR.

Nope, no strong opinion from me on this.

There seems to be a problem with windows threading tests on Jenkins.

@t4c1 hope you don't mind, I thought merging to the refactor would be wicked lame so I did it for you (since the conflicts were my fault anyway)

@t4c1 can you checkout this PR? I made some changes on a branch that I think would make this good to go
https://github.com/bstatcomp/math/pull/14

I don't like the name change TriangularViewCL to PartialViewCL. I associate the word "partial" with partial derivatives which have nothing to do with matrix views. That could be confusing for anyone using this.
Instead I propose following changes to names:
PartialViewCL -> MatrixCLView (Since we are changing this we could also use snake_case instead of CamelCase)
matrix_cl->partial_view_ -> matrix_cl->view_
matrix_cl->partial_view() -> matrix_cl->view()
What dou you think @SteveBronder @rok-cesnovar?

@SteveBronder Why did you change functions for manipulation of TriangularViewCL to operator+ and operator*? You did essentially the same as I did at first when I used operator| and operator& - and you did not like it. Mine operators were only needed to call the same operation on the underlying type of enum class TriangularViewCL. Yours, however, are implemented by calling a different operation on the underlying type. That is extremely confusing!
If you changed your mind about operators I think you should revert that to | and & - the same as underlying operations. Otherwise I think functions should be used instead of operators. Using functions also has advantage that we can do same operations in c++ and OpenCL by calling same functions.

I agree with view().

That could be confusing for anyone using this.

Yes! Agree with the name changes Did not think of those names but they make a lot more sense.

Why did you change functions for manipulation of TriangularViewCL to operator+ and operator*? You did essentially the same as I did at first when I used operator| and operator& - and you did not like it. Mine operators were only needed to call the same operation on the underlying type of enum class TriangularViewCL. Yours, however, are implemented by calling a different operation on the underlying type. That is extremely confusing!

tbh I was pretty confused by the bitshift operators & and | when I was looking at the code. Like Entire = Upper & Lower didn't make a lot of sense to me abstractly (and what was going on was not documented or tested). But then playing with it I realized | gave the type of the addition of two triangular matrices and & gave the output of multiplication of two matrices. I thought Entire = Upper + Lower conveyed what was going on a more abstract level that made sense so I changed them.

If you changed your mind about operators I think you should revert that to | and & - the same as underlying operations. Otherwise I think functions should be used instead of operators. Using functions also has advantage that we can do same operations in c++ and OpenCL by calling same functions.

I'm fine with making them functions. How about just union for | and intersection for &? I think it makes sense that a union of upper and lower is an entire matrix while an intersection of upper and lower is just the diagonal.

Union and intersection are fine.

Union is keyword in c++. Any other suggestions?

conjunction and disjunction are in the standard library but I think are fine to use

conjunction and disjunction are already in stan. I tried any and both, but any is a function in OpenCL. I think either and both might be fine.

conjunction and disjunction are already in stan.

Did it fail to compile? I think it's fine to have a method with the same name but for different types. The conjunction and disjuction funcs in stan rn are there just because the standard only defines them in C++17 and we use C++14

Yep, failed to compile. Existing ones in stan are structs (template metaprograms), which can't have same name as a function.

Gaahh my mistake. either and both are okay. How about conjoint and disjoint. I doesn't feel like either(Upper, Lower) and both(Upper, Lower) convey what the operation does

I agree either and both are not perfect. But I stil think conjoint and disjoint are worse. For example if a matrix is bot lower- and upper- triangular it is diagonal. Reads like english.

That's fair. Let's fix the test error and then this should be good to go!

@rok-cesnovar do you want to approve this as well? Think both reviewers need to approve before we can mege

I think you could also dismiss my review, idk. But anyways yes, this is good to go. Thanks both!


AsyncAPI support

Hi everyone.
I'm trying to use Specmatic for AsyncAPI Contract testing, but I'm not sure it is supported yet.
In Specmatic home page, AsyncAPI is listed as one of the Supported Specification Standards. But when trying to run the test command against an AsyncAPI file I get the following error message.
$ java -jar specmatic-0.59.0.jar test asyncapi.yaml

Error parsing file asyncapi.yaml
  attribute openapi is missing
Could not parse contract asyncapi.yaml, please validate the syntax using https://editor.swagger.io
Load Error FAILED
Reason: Could not parse contract asyncapi.yaml, please validate the syntax using https://editor.swagger.io

Am I doing something wrong, or AsyncAPI isn't really supported yet? If not, is this on the roadmap? Will it be supported in the near future?

@pablo-curiel Thanks for reaching out. We are currently in private beta for async-api support. If you are interested in signing up to participate in our private beta, can you please share your contact information here so that we can reach to you on this?

@pablo-curiel We have received your interest to participate in async-api private beta. Will continue the conversation in a separate thread. Closing this issue. Thanks.


Add support for Eve

Even though Eve is a wrapper around Flask, using sentry on Eve throws an exception, e.g.:
# app.py
from eve import Eve
from raven.contrib.flask import Sentry

app = Eve()
sentry = Sentry(app, dsn='...')
app.run()

Traceback (most recent call last):
  File "flask-sentry.py", line 5, in <module>
    sentry = Sentry(app<EMAIL_ADDRESS>  File "MYPATH/env/lib/python3.5/site-packages/raven/contrib/flask.py", line 114, in __init__
    self.init_app(app)
  File "MYPATH/env/lib/python3.5/site-packages/raven/contrib/flask.py", line 284, in init_app
    got_request_exception.connect(self.handle_exception, sender=app)
  File "MYPATH/env/lib/python3.5/site-packages/blinker/base.py", line 130, in connect
    sender_ref = reference(sender, self._cleanup_sender)
  File "MYPATH/env/lib/python3.5/site-packages/blinker/_utilities.py", line 134, in reference
    weak = callable_reference(object, callback)
  File "MYPATH/env/lib/python3.5/site-packages/blinker/_utilities.py", line 145, in callable_reference
    return BoundMethodWeakref(target=object, on_delete=callback)
  File "MYPATH/env/lib/python3.5/site-packages/blinker/_saferef.py", line 135, in __new__
    key = cls.calculate_key(target)
  File "MYPATH/env/lib/python3.5/site-packages/blinker/_saferef.py", line 196, in calculate_key
    return (id(get_self(target)), id(get_func(target)))
  File "MYPATH/env/lib/python3.5/site-packages/events/events.py", line 41, in __getattr__
    (self.__class__.__name__, name))
AttributeError: type object 'Eve' has no attribute '__self__'
Sentry is attempting to send 1 pending error messages
Waiting up to 10 seconds
Press Ctrl-C to quit

+1
We need it a lot
thnx

+1

+1

No updates about it yet?

A workaround for this problem is to subclass Eve as follows:
class BlinkerCompatibleEve(Eve):
    """
    Workaround for https://github.com/pyeve/eve/issues/1087
    """
    def __getattr__(self, name):
        if name in {"im_self", "im_func"}:
            raise AttributeError("type object '%s' has no attribute '%s'" %
                                 (self.__class__.__name__, name))
        return super(BlinkerCompatibleEve, self).__getattr__(name)


Fixed content length handling issues. (#2894)

Issue (#2894) was fixed. Now the content length is handled as a long value (whenever possible - Servlet 3.1+) instead of integer. Test scenarios have been added to ensure the endpoints are receiving the accurate value.

@spencergibb @ryanjbaxter Hi, could you please review my changes?

@marioburatto so can we close #2897?  This PR seems to include those changes.

Yes we can just merge the code forward.

Hello @ryanjbaxter, do you have any estimates for this pull request to be integrated to master, so we could be planing the 2.0.0.RC validation in our project?
Thanks,

I merged the changes into master, it will be included in 2.0.0.RC2 once we release it


Add proxy_path parameter to flet_fastapi.app()

See issue 1824
https://github.com/flet-dev/flet/issues/1824#issuecomment-1737705064

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Return to git slow

Your Windows build number:  (Type ver at a Windows Command Prompt) - Microsoft Windows [Version 10.0.15063]

What you're doing and what's happening:  When I run a git command that starts a new process (log, commit --amend, etc...) it takes 3-5 seconds to get back to the shell after killing it (:q, etc...)

What's wrong / what should be happening instead: Running the same command in cmd gets me back to the shell instantly

What version of WSL are you running? On the bash shell use lsb_release -a. Since you are using the new creators update, if you are on 14.04 of ubuntu you are running an outdated version. The creators update supports the new version 16.04. Use the link to update your terminal and try again https://www.howtogeek.com/278152/how-to-update-the-windows-bash-shell/

@nlogozzo
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.1 LTS
Release:        16.04
Codename:       xenial

I have the same build. Git is very slow for me as well in the Windows bash shell. When I run git status, at best it will take about 3 seconds to complete. But it can much longer. When I run the same command on the same exact repo from cmd.exe it executes immediately.
I was on Git version 2.7.1 and I have upgraded to 2.11. Nothing has changed with the speed though. Any suggestions?

#1305 #981 and friends. User voice is here.

Can confirm this. git is slower than tortoise.

I got same issue when running  git status,too.
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.2 LTS
Release:        16.04
Codename:       xenial

Same for me
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.3 LTS
Release:        16.04
Codename:       xenial

Dupe #981.


Instance: Show non-editable fields in lxc config edit screen without expanded config

Following on from discussion here https://github.com/lxc/lxd/pull/10390#issuecomment-1123343345
This PR reverts the changes in https://github.com/lxc/lxd/pull/10390 and restores showing the non-editable fields in the lxc config edit screen (for consistency with edit screens for other types of entity, such as network, storage pools etc).
However avoids needing to show the expanded config by specifying those fields as omitempty and then manually clearing them in the lxc config edit command.

@stgraber in the end I went with your suggestion of using omitempty to reduce the overhead of maintenance on those structs in the future as would have meant needing to duplicate all of the fields in Instance and InstanceSnapshot.

@tomponline this branch keeps jamming all the test runners, it gets stuck calling the editor.

@tomponline this branch keeps jamming all the test runners, it gets stuck calling the editor.

How strange, works fine locally. Could it be trying to read from stdin perhaps, ill take a look.

root     1860640  0.0  0.0   5124  3292 pts/0    S+   08:23   0:00                  |                   \_ /bin/sh -eu ./main.sh standalone
root     1860649  0.0  0.0 1167916 14028 pts/0   Sl+  08:23   0:00                  |                   |   \_ /lxc-ci/build/tmp.145Jtqxp9n/go/bin/lxc config edit foo --verbose
root     1860677  0.0  0.0  16556  8420 pts/0    S+   08:23   0:00                  |                   |       \_ editor /tmp/lxd_editor_2577848191.yaml
root     1860641  0.0  0.0   2848   612 pts/0    S+   08:23   0:00                  |                   \_ grep name: foo

The fact its running editor rather than cat suggests the env var isn't working.
I'm wondering if I've been caught out by that lxc() function, trying a sub shell to see if that avoids it.

Hrm not hanging anymore but not working either.

I'd missed there was two test functions modified earlier.


The image and link feature depend on tooltip plugin

The image and link features depend on the tooltip plugin. If the tooltip plugin is not installed, the input box of the image and link  will not pop up. It feels that this design is unreasonable and should be decoupled.
image 和 link 特性依赖于 tooltip 插件，如果不安装tooltip 插件，image 和 link 更新链接的输入框就不会弹出，感觉这样设计不合理，应该解耦。

It's already been done in latest code.


Update contributing.md

The one time @stucamp pops in

Lol couldn't resist!  I'll have to jump in slack sometime, just to remind everyone who I am!  Good to see the boy is still getting some love!


add yahoo-stock-market to latest

RE-CHECK!

Hi,
release comments incoming :-)

please elete .commitinfo ( seems from an unsuscessfull release)
add the release script plugins and config for 3.x - seems that you upgraded the release script dep and did not checked major changes
you can not clear the timeout from https://github.com/Newan/ioBroker.yahoo-stock-market/blob/main/src/main.ts#L74 how you do it ... just make a normal setTimeout and not that promise stuff in this place because it also makes no sense
There are jsonconfig types that offer a "toast" like entering of multiple values and you get it as array or comma separated ... but easier for user then a normal "text". maybe check that out.

Please check and adjust the comments
Ingo

@Apollon77  Thanks for the feedback.

"add the release script plugins and config for 3.x - seems that you upgraded the release script dep and did not checked major changes" -> ok i use the adapter creator and then the release script. this failed wih the currentversion, so i musst update. so please update this too.
https://github.com/ioBroker/create-adapter/issues/824

"There are jsonconfig types that offer a "toast" like entering of multiple values and you get it as array or comma separated ... but easier for user then a normal "text". maybe check that out." -> Please give me a hint. I only know this .. but no "toast":  https://github.com/ioBroker/ioBroker.admin/blob/master/src-rx/src/components/JsonConfigComponent/SCHEMA.md

to 2.) sorry I used wrong word ... meaned "chips" (can be found in jsonconfig schema

fixed

ok the timeout option in the yahoo lib is on TODO:   timeout?: number; // TODO
new release online


Update: Use enum instead of constant strings for transaction fields

Use enum instead of constant strings for transaction fields

@sczembor We still allow string in the constructor. It won't be a breaking change.
I have moved the opcode fields to enum. Please check


Debug hover widget moves out from under cursor when size changes

Version: 1.50.0-insider
Commit: 93c2f0fbf16c5a4b10e4d5f89737d9c2c25488a3
Date: 2020-10-07T00:58:35.624Z (1 day ago)
Electron: 9.2.1
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.1.0
I think this is a recent regression in Insiders.
Steps to Reproduce:

Debug something with complex enough objects to take up a lot of vertical space when fully expanded
Scroll such that there’s more space below the identifier you’re going to hover than above
Pause the debugger and hover the identifier—the debug hover widget appears above the identifier
Expand properties until the size of the widget is too large to fit above the line. It will move below the line, such that you’re no longer hovering the widget, which means as soon as you move your cursor, it will disappear.

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

/duplicate https://github.com/microsoft/vscode/issues/108333


Waypoint: Manipulate Arrays With unshift  |   error message d[0].toLowerCase is not a function

Challenge http://www.freecodecamp.com/challenges/waypoint-manipulate-arrays-with-unshift has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
I entered myArray.unshift(["Paul"]);
the error message d[0].toLowerCase is not a function

Thanks

You're adding an array that has the string element "Paul."  The exercise is not asking you to add an array; it's asking you to add a string element.

thanks for getting back to me. I thought I'd tried to add the string and it
didn't accept it. Oh well. it's all good.
On Mon, Aug 24, 2015 at 8:07 PM, Aryan Jabbari<EMAIL_ADDRESS>wrote:

You're adding an array that has the string element "Paul." The exercise is
not asking you to add an array; it's asking you to add a string element.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/2644#issuecomment-134461032
.

@basiccreation thanks for posting this issue!
Closing this issue as the problem if solved.

Please review the Guidelines for Contributing, thank you!.


Search for location by lat, lon

Could you please allow for the searching of lattitude, longitude pairs and perhaps MGRS?

This is easy to implement using mapbox-gl-geocoder but would require some new UI to switch between normal address lookups and coordinate lookups.
MGRS/UTM would also be an interesting add-on, curious how much demand there would be for it.

Could you please allow for the searching of lattitude, longitude pairs and perhaps MGRS?

I just came here to report the same thing.