[Bug] [rest-api] /hazelcast/rest/maps/running-jobs Request failed java.lang.NullPointerException

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
When I deploy in cluster mode, roll up seatunel, and access runtime tasks using restApi, this error is reported
Due to some reasons, the image cannot be uploaded, and its null pointer occurs on line 344 of RestHttpGetCommandProcessor.java

When I completely stop the cluster and restart it, this problem will not occur. Once I roll restart, this problem will occur
SeaTunnel Version
2.3.4
SeaTunnel Config
pass

Running Command
pass

Error Exception
{"status":"fail","message":"java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.NullPointerException"}

Zeta or Flink or Spark Version
No response
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

And only the handleRunningJobsInfo method will have this issue, handleFinishedJobsInfo will not have this issue.
If this problem occurs again online in the future, the log will be filled in

I speculate that some task information was lost during the rolling restart, but jobId was still present, resulting in a null pointer when querying through jobId. Of course, this is just speculation. Should we make a fault tolerance for the situation where jobId exists but specific jobMetrics do not exist

Is there any detailed  log?

在滚动重启的时候，遇到了同样的问题，版本 2.3.4
以下为详细日志
2024-04-21 14:10:51,604 ERROR [s.e.s.o.GetJobMetricsOperation] [hz.main.generic-operation.thread-20] - [seatunnel-0.seatunnel.seatunnel.svc.cluster.local]:5801 [seatunnel] [5.1] java.util.concurrent.ExecutionException: java.lang.NullPointerException
java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.NullPointerException
	at org.apache.seatunnel.engine.server.operation.GetJobMetricsOperation.run(GetJobMetricsOperation.java:87) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationservice.Operation.call(Operation.java:189) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.call(OperationRunnerImpl.java:273) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.run(OperationRunnerImpl.java:248) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.run(OperationRunnerImpl.java:471) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.process(OperationThread.java:197) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.process(OperationThread.java:137) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.executeRun(OperationThread.java:123) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.internal.util.executor.HazelcastManagedThread.run(HazelcastManagedThread.java:102) ~[seatunnel-starter.jar:2.3.4]
Caused by: java.util.concurrent.ExecutionException: java.lang.NullPointerException
	at java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:357) ~[?:1.8.0_312]
	at java.util.concurrent.CompletableFuture.get(CompletableFuture.java:1908) ~[?:1.8.0_312]
	at org.apache.seatunnel.engine.server.operation.GetJobMetricsOperation.run(GetJobMetricsOperation.java:85) ~[seatunnel-starter.jar:2.3.4]
	... 8 more
Caused by: java.lang.NullPointerException
	at org.apache.seatunnel.engine.server.operation.GetJobMetricsOperation.lambda$run$0(GetJobMetricsOperation.java:78) ~[seatunnel-starter.jar:2.3.4]
	at java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1604) ~[?:1.8.0_312]
	at com.hazelcast.internal.util.executor.CachedExecutorServiceDelegate$Worker.run(CachedExecutorServiceDelegate.java:217) ~[seatunnel-starter.jar:2.3.4]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) ~[?:1.8.0_312]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) ~[?:1.8.0_312]
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_312]
	at com.hazelcast.internal.util.executor.HazelcastManagedThread.executeRun(HazelcastManagedThread.java:76) ~[seatunnel-starter.jar:2.3.4]
	... 1 more
2024-04-21 14:10:51,618 WARN  [.Log4j2HttpGetCommandProcessor] [hz.main.cached.thread-6] - [seatunnel-2.seatunnel.seatunnel.svc.cluster.local]:5801 [seatunnel] [5.1] An error occurred while handling request HttpCommand [HTTP_GET]{uri='/hazelcast/rest/maps/running-jobs'}AbstractTextCommand[HTTP_GET]{requestId=0}
java.util.concurrent.CompletionException: java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.NullPointerException
	at com.hazelcast.spi.impl.AbstractInvocationFuture.returnOrThrowWithJoinConventions(AbstractInvocationFuture.java:819) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.AbstractInvocationFuture.resolveAndThrowWithJoinConvention(AbstractInvocationFuture.java:835) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.AbstractInvocationFuture.join(AbstractInvocationFuture.java:553) ~[seatunnel-starter.jar:2.3.4]
	at org.apache.seatunnel.engine.server.rest.RestHttpGetCommandProcessor.convertToJson(RestHttpGetCommandProcessor.java:365) ~[seatunnel-starter.jar:2.3.4]
	at org.apache.seatunnel.engine.server.rest.RestHttpGetCommandProcessor.lambda$handleRunningJobsInfo$2(RestHttpGetCommandProcessor.java:175) ~[seatunnel-starter.jar:2.3.4]
	at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193) ~[?:1.8.0_312]
	at java.util.Iterator.forEachRemaining(Iterator.java:116) ~[?:1.8.0_312]
	at java.util.Spliterators$IteratorSpliterator.forEachRemaining(Spliterators.java:1801) ~[?:1.8.0_312]
	at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:482) ~[?:1.8.0_312]
	at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:472) ~[?:1.8.0_312]
	at java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708) ~[?:1.8.0_312]
	at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234) ~[?:1.8.0_312]
	at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:577) ~[?:1.8.0_312]
	at org.apache.seatunnel.engine.server.rest.RestHttpGetCommandProcessor.handleRunningJobsInfo(RestHttpGetCommandProcessor.java:177) ~[seatunnel-starter.jar:2.3.4]
	at org.apache.seatunnel.engine.server.rest.RestHttpGetCommandProcessor.handle(RestHttpGetCommandProcessor.java:101) ~[seatunnel-starter.jar:2.3.4]
	at org.apache.seatunnel.engine.server.rest.RestHttpGetCommandProcessor.handle(RestHttpGetCommandProcessor.java:73) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.internal.ascii.TextCommandServiceImpl$CommandExecutor.run(TextCommandServiceImpl.java:402) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.internal.util.executor.CachedExecutorServiceDelegate$Worker.run(CachedExecutorServiceDelegate.java:217) ~[seatunnel-starter.jar:2.3.4]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) ~[?:1.8.0_312]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) ~[?:1.8.0_312]
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_312]
	at com.hazelcast.internal.util.executor.HazelcastManagedThread.executeRun(HazelcastManagedThread.java:76) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.internal.util.executor.HazelcastManagedThread.run(HazelcastManagedThread.java:102) ~[seatunnel-starter.jar:2.3.4]
Caused by: java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.NullPointerException
	at org.apache.seatunnel.engine.server.operation.GetJobMetricsOperation.run(GetJobMetricsOperation.java:87) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationservice.Operation.call(Operation.java:189) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.call(OperationRunnerImpl.java:273) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.run(OperationRunnerImpl.java:248) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.run(OperationRunnerImpl.java:471) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.process(OperationThread.java:197) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.process(OperationThread.java:137) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.executeRun(OperationThread.java:123) ~[seatunnel-starter.jar:2.3.4]
	... 1 more
Caused by: java.util.concurrent.ExecutionException: java.lang.NullPointerException
	at java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:357) ~[?:1.8.0_312]
	at java.util.concurrent.CompletableFuture.get(CompletableFuture.java:1908) ~[?:1.8.0_312]
	at org.apache.seatunnel.engine.server.operation.GetJobMetricsOperation.run(GetJobMetricsOperation.java:85) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationservice.Operation.call(Operation.java:189) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.call(OperationRunnerImpl.java:273) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.run(OperationRunnerImpl.java:248) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.run(OperationRunnerImpl.java:471) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.process(OperationThread.java:197) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.process(OperationThread.java:137) ~[seatunnel-starter.jar:2.3.4]
	at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.executeRun(OperationThread.java:123) ~[seatunnel-starter.jar:2.3.4]
	... 1 more
Caused by: java.lang.NullPointerException
	at org.apache.seatunnel.engine.server.operation.GetJobMetricsOperation.lambda$run$0(GetJobMetricsOperation.java:78) ~[seatunnel-starter.jar:2.3.4]
	at java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1604) ~[?:1.8.0_312]
	... 6 more
2024-04-21 14:11:00,319 INFO  [o.a.s.e.s.CoordinatorService  ] [pool-5-thread-1] - [seatunnel-1.seatunnel.seatunnel.svc.cluster.local]:5801 [seatunnel] [5.1]
***********************************************

@liugddx cc

This problem occurred again today, and it has nothing to do with the rolling restart. We suspect that the metadata of a certain task cannot be obtained, or there is a problem with writing, causing an error when obtaining the running task list directly. Regardless of what caused the metadata of a certain task to not be obtained, should we consider the robustness of the code and not let the metadata error of a certain task affect the viewing of the entire running task list

@liugddx @Hisoka-X @hailin0 help cc

@liugddx @Hisoka-X @hailin0 help cc

What version are you using?

@liugddx @Hisoka-X @hailin0 help cc

What version are you using?

2.3.5


how to use client efficently

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v5 (for Elasticsearch 5.x)
Please describe the expected behavior
i use one client in my project,but sometimes it deae
Please describe the actual behavior
var es *elastic.Client
var once sync.Once
//single
func Instance() *elastic.Client {
once.Do(func() {
client, _ := elastic.NewClient(
elastic.SetURL("http://"+es_host+":9200"),
elastic.SetErrorLog(log.New(os.Stderr, "ELASTIC ", log.LstdFlags)),
elastic.SetInfoLog(log.New(os.Stdout, "", log.LstdFlags)),
elastic.SetTraceLog(log.New(os.Stderr, "[[ELASTIC]]", 0)))
es = client
})
return es
}
Any steps to reproduce the behavior?
is that right to use it

Using a single instance of *elastic.Client across your application is okay. It should be thread-safe.
However, you should always check for errors when creating the new client. You don't do that in your code snippet. If you're simply swallowing the error, your *elastic.Client will probably be nil, which is not what you want.


[azdo pipelines] Remove setting credential as secret

On this PR: https://github.com/Azure/azure-dev/pull/2667
The AZURE_CREDENTIALS was added to the pipeline config flow for AZDO.
This made it possible to remove the az dependency, but it became not idiomatic for azdo, as azdo uses a service-connection instead of secrets with auth credentials.
pipeline config is still creating a service-connection, so, this issue is about removing the AZURE CREDENTIALS secret, which depends on the work from https://github.com/Azure/azure-dev/issues/3093, where there's an azd-step to use a service connection directly.
The plan is, once we have the step, we would get rid of the secret and use service-connection only for Azdo (as before), but without depedning on az

implemented on https://github.com/Azure/azure-dev/pull/3100/files#diff-effa47a093ad0dd4f0c46d123af1e0ed94f79face891244b6fa60809d8a3759bL167


[FEATURE REQUEST] Import/Export Feature

I like to keep my different games in different worlds.  PFS2 in one, Local Age of Ashes (that my wife runs), Age of Ashes that I run.   And would be nice, if I like a configuration of the tooltip, that I can export it and import it into another world (without having to manually copy/paste the settings into the settings file)..

Ahh, I see that it was added.. I had to uninstall/reinstall to see the new version with the tabbed version.. This can be closed.


multiviewer-for-f1: 1.26.6 -> 1.28.2

Description of changes
Superseeds #255160
Program Changelog
v1.26.7

Add more playback rate options
Live timing seeking now updates while you drag
Fix various issues related to animating position changes and new radios/race control messages
Update FIAWEC timing for Fuji

v1.26.8
No changelog
v1.26.9

Add audio notification for WEC race control messages
Add "Laps since pitstop" column for WEC

v1.26.10

Fix IndyCar page not showing
Updates to sync algorithm

v1.26.11

Update fit/fill button tooltip
Add analytics to edit live timing layouts (to check if it's found by anyone)
Add query parameters to to external manifest deeplink for clipping, latency settings
Various improvements

v1.27.0

New: My List allows you to add videos to watch later, shared across your devices
Fix weekend warmup casing in schedule
Add donate banner to the homepage

v1.27.1

Launch of new Chrome/Firefox extension to help sign in and open content in MultiViewer
Fix login issue related to entitlement token

v1.28.0

New feature: Investigations tracker let's you keep track of all ongoing investigations, reviews and penalties

v1.28.1

Fix: also match new format for multi-car investigations/inchidents

v1.28.2

Hotfix: remove debug code from investigations

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.


Validate partitioned_by table property

Ensure that columns listed in partitioned_by table property for Hive connector are valid table columns.
E.g., this should fail:
CREATE TABLE t (
    col VARCHAR
)
WITH (partitioned_by = ARRAY['foo']);

Related to https://github.com/facebook/presto/issues/3793

This is already implemented.


Feat/bid request validation

closes #42

One minor thing before merging this. Please bump the version to 3.5.0

Also for future reference I think we should refactor the validation process of builders to a base builder which all other builder will inherit. Keep it DRY:)


error with PopupService

https://github.com/ronzeidman/ng2-ui-auth/blob/master/src/oauth2.service.ts#L67 in this line you return oauthData returned from the server as it is in my case, it's throw exception because date type not observable i work around it by creating class that extend PopupService
import { PopupService } from 'ng2-ui-auth';
import { Observable } from 'rxjs/Observable';

/**
 * wrapper of Ng2UiAuthModule to handle our way in auth
 */
export class CustomPopupService extends PopupService {
    pollPopup() {
        return super.pollPopup().map(data => {
            if (data instanceof Observable) return data;
            return Observable.of(data);
        });
    };
}

here you can find the error
TypeError: unknown type returned
    at Object.subscribeToResult (http://localhost:3000/vendor.bundle.js:10490:28)
    at MergeMapSubscriber._innerSub (http://localhost:3000/vendor.bundle.js:30879:39)
    at MergeMapSubscriber._tryNext (http://localhost:3000/vendor.bundle.js:30876:15)
    at MergeMapSubscriber._next (http://localhost:3000/vendor.bundle.js:30859:19)
    at MergeMapSubscriber.Subscriber.next (http://localhost:3000/vendor.bundle.js:10221:19)
    at DoSubscriber._next (http://localhost:3000/vendor.bundle.js:58225:31)
    at DoSubscriber.Subscriber.next (http://localhost:3000/vendor.bundle.js:10221:19)
    at TakeWhileSubscriber.nextOrComplete (http://localhost:3000/vendor.bundle.js:61259:26)
    at TakeWhileSubscriber._next (http://localhost:3000/vendor.bundle.js:61254:15)
    at TakeWhileSubscriber.Subscriber.next (http://localhost:3000/vendor.bundle.js:10221:19)
    at TakeSubscriber._next (http://localhost:3000/vendor.bundle.js:61049:31)
    at TakeSubscriber.Subscriber.next (http://localhost:3000/vendor.bundle.js:10221:19)
    at MergeMapSubscriber.notifyNext (http://localhost:3000/vendor.bundle.js:30892:31)
    at InnerSubscriber._next (http://localhost:3000/vendor.bundle.js:52466:22)
    at InnerSubscriber.Subscriber.next (http://localhost:3000/vendor.bundle.js:10221:19)
    at Object.subscribeToResult (http://localhost:3000/vendor.bundle.js:10447:26)
    at MergeMapSubscriber._innerSub (http://localhost:3000/vendor.bundle.js:30879:39)
    at MergeMapSubscriber._tryNext (http://localhost:3000/vendor.bundle.js:30876:15)
    at MergeMapSubscriber._next (http://localhost:3000/vendor.bundle.js:30859:19)
    at MergeMapSubscriber.Subscriber.next (http://localhost:3000/vendor.bundle.js:10221:19)
    at AsyncAction.IntervalObservable.dispatch (http://localhost:3000/vendor.bundle.js:54953:21)
    at AsyncAction._execute (http://localhost:3000/vendor.bundle.js:29070:19)
    at AsyncAction.execute (http://localhost:3000/vendor.bundle.js:29045:27)
    at AsyncScheduler.flush (http://localhost:3000/vendor.bundle.js:29142:33)
    at ZoneDelegate.invokeTask (http://localhost:3000/polyfills.bundle.js:7477:38)
    at Object.onInvokeTask (http://localhost:3000/vendor.bundle.js:6425:42)
    at ZoneDelegate.invokeTask (http://localhost:3000/polyfills.bundle.js:7476:43)
    at Zone.runTask (http://localhost:3000/polyfills.bundle.js:7377:48)
    at ZoneTask.invoke (http://localhost:3000/polyfills.bundle.js:7545:34)

Hi, sorry for the late response, I've been pretty busy the last month.
Can you tell me what version of rxjs you have, I think beta version 9 and above automatically wrap the data and that's why I haven't had issues like this.

I've tried to reproduce it, I cant see how. I've never expected the parameters returned from the popup to be an observable, they never are.

This my config and I will try to make plunder to simulate my use case
auth config service
@Injectable() export class CustomAuthConfig extends CustomConfig {     constructor({authservices, apiUrl}: ApiHelperService) {         super();         console.log(authservices);         this.defaultHeaders = { 'Content-Type': 'application/json' };         this.providers = {             OWINGoogle: {                 url: null,                 authorizationEndpoint: auth url,                 requiredUrlParams: ['provider'],                 clientId: 'id',                 provider: 'Google',                 redirectUri: window.location.origin             } as any         };     } }
popup service
`import { PopupService } from 'ng2-ui-auth';
import { Observable } from 'rxjs/Observable';
/**

wrapper of Ng2UiAuthModule to handle our way in auth
*/
export class CustomPopupService extends PopupService {
pollPopup() {
return super.pollPopup().map(data => {
if (data instanceof Observable) return data;
return Observable.of(data);
});
};
}
`

call service with this.uiAuth.authenticate('google')
sample of popup response
export interface OwinAuthResponse {     external_access_token: string;     provider: string;     haslocalaccount: string | boolean;     external_user_name: string; }

I don't need to return observable from popup service I return observable from it as hack,
As you see in this line oauth2.service are returning the data that coming from popup service with concatmap I think concatmap is the one need data to be observable

When you create a provider it's name is the provider key name and it's case-sensitive, so your provider should look like this:
{
            OWINGoogle: {
                /*not nessesary: url: null,*/
                authorizationEndpoint: `auth url`,
                requiredUrlParams: ['provider'],
                clientId: 'id',
                /*this does nothing: provider: 'Google',*/
                redirectUri: window.location.origin
            } as any
        }

and you call it like this:
this.uiAuth.authenticate('OWINGoogle')
I think I found the issue: https://github.com/ronzeidman/ng2-ui-auth/blob/b73dfc28a18b30cabb5d3399b24a932476d4ca7d/src/oauth2.service.ts#L67
this line should be Observable.of(oauthData), can you try it and tell me if this fix works?
Thanks

You can try install the master version: npm i -S ronzeidman/ng2-ui-auth and tell me if it works for you without workarounds.

yes it's now working as expected, thanks you
but about this comment
{             OWINGoogle: {                 /*not nessesary: url: null,*/                 authorizationEndpoint: auth url,                 requiredUrlParams: ['provider'],                 clientId: 'id',                 /*this does nothing: provider: 'Google',*/                 redirectUri: window.location.origin             } as any         }
/this does nothing: provider: 'Google'/
my auth end point expecting provider in query string i work around it with
requiredUrlParams: ['provider'],
and yes i calling with this.uiAuth.authenticate('OWINGoogle') it was a typo

If we can provide better solution to append it's to url query string like key pair object
requiredQueryStringParams:[{'provider':'Google'}]

it will be great if I can pass url in config and inject exchangeForToken
function with my implementation if possible

I thing this is an easy addition, I'll look into it next week.

Took me a bit longer, try the master now (look at the https://github.com/ronzeidman/ng2-ui-auth/blob/master/CHANGELOG.md for more details)

fixed in 4.1.0


GGUF cleanup

In reference to the last comments on #7046.
ggml_data_to_tensor does not return None if a type is not supported, it raises a KeyError. Mypy does not recognize this, which means that explicitly setting the return type to Tensor creates a mypy error.
Do we want to spend one line on an explicit error?

Ill make a separate PR for the error.


Language switcher in storefront does not use language fallback chain in displayed label

Description
See: https://github.com/shopware/platform/blob/trunk/src/Storefront/Resources/views/storefront/layout/header/actions/language-widget.html.twig#L54
.translated. is missing. However, just a few lines above in the title it is used correctly.
Possible Solution
Add .translated..
Btw. it would also be nice if just the name part was adjustable using twig blocks.

@mitelg I just checked and I actually made an error here. The error is actually reversed. Language is not translatable and therefore name is correct but the title attribute tries to use translated which is wrong.


Archive DW-Taxonomy fork on release

The repo https://github.com/graphsense/DW-VA-Taxonomy is not needed anymore, since all taxonomies are now maintained direclty in the tagpack tool repository.

Related to #50


Pan stops working after mousewheel zoom

In your demo, I can pan around just fine, but once I use the mousewheel to zoom in and out, the pan stops working. I will be able to pan again if I click one of the specified zoom buttons, but it will again stop working if I mousewheel zoom again.

Hi,
You're absolutely right. The mousewheel zoom is also broken; the zoom doesn't happen at the point that the mouse points at.
These are serious issues and I'll be looking into them ASAP.
Cheers,
/Martin

Turns out that it was caused by an API change in Hamster.JS. I've fixed the way we deal with HamsterJS and released the fix as v1.0.7 on bower.
@xingped, thanks for reporting the issue!
Cheers,
/Martin
Ref: https://github.com/monospaced/hamster.js/commit/e6da3e67519a8ad2f30899f1986dd8402860524a


Tensor views should compute slice on CPU then forward straight to GPU

Slice the arrow buffer that's coming from the store as needed, upload everything to a texture just-in-time, render to a textured rectangle.
Related: #1375

Fixed as part of the recent colormapping work.


FileChooserNative memory management issue

Causes crashes on Flatpak and nothing happening on the host.
I believe it's dropped prematurely, see e.g. the demo code, the object is not unreffed after show().
use gio::prelude::*;
use gtk::prelude::*;

use std::env::args;

fn build_ui(application: &gtk::Application) {
    let window = gtk::ApplicationWindow::new(application);

    window.set_title("First GTK+ Program");
    window.set_border_width(10);
    window.set_position(gtk::WindowPosition::Center);
    window.set_default_size(350, 70);

    let button = gtk::Button::with_label("Click me!");

    button.connect_clicked(|_| {
        let file_chooser = gtk::FileChooserNativeBuilder::new()
            .action(gtk::FileChooserAction::Open)
            .title("Open something")
            .build();

        file_chooser.connect_response(move |file_chooser, response| {
            eprintln!("{:?}", response);

            file_chooser.destroy();
        });

        file_chooser.show();

        // Uncomment to make it behave properly.
        // use glib::translate::*;
        // let _ = file_chooser.as_object_ref().to_glib_full();
    });

    window.add(&button);

    window.show_all();
}

fn main() {
    let application =
        gtk::Application::new(Some("com.github.gtk-rs.examples.basic"), Default::default())
            .expect("Initialization failed...");

    application.connect_activate(|app| {
        build_ui(app);
    });

    application.run(&args().collect::<Vec<_>>());
}

Causes crashes on Flatpak

Can you provide a backtrace of the crash (with debug symbols)? This really shouldn't crash and doesn't here, also doesn't give any valgrind warnings.

I believe it's dropped prematurely

Yes in your code above the file_chooser is dropped right at the end of the clicked signal handler. You probably want to keep it alive a bit longer until the user closes it in one way or another.
Why this behaves differently than other dialogs is not clear to me though. Other dialogs have their window owned by GTK and stay alive until they're explicitly destroyed.

Can you provide a backtrace of the crash (with debug symbols)?

I could if it didn't also crash gdb itself when trying to do a backtrace... 😄 (I reported that here)

Here's as far as I can go before the gdb crash:
#0  0x00007f8482f86615 in gtk_widget_get_display (widget=0x3500353500353537) at gtkwidget.c:11005
11005	  g_return_val_if_fail (GTK_IS_WIDGET (widget), NULL);

#1  0x00007f8482fa8843 in gtk_window_unexport_handle (window=0x3500353500353537) at gtkwindow.c:13000
13000	  if (GDK_IS_WAYLAND_DISPLAY (gtk_widget_get_display (GTK_WIDGET (window))))

#2  0x00007f8482dcc24f in filechooser_portal_data_free (data=data@entry=0x55da0bcd5050) at gtkfilechoosernativeportal.c:83
83	    gtk_window_unexport_handle (data->exported_window);

#3  0x00007f8482dcc9d4 in response_cb (connection=<optimized out>, sender_name=<optimized out>, object_path=<optimized out>, interface_name=<optimized out>, signal_name=<optimized out>, parameters=<optimized out>, user_data=0x55da0b966820) at gtkfilechoosernativeportal.c:175
175	  filechooser_portal_data_free (data);

#4  0x00007f848285a88f in emit_signal_instance_in_idle_cb (data=0x7f8474019950) at ../gio/gdbusconnection.c:3777
3777	    signal_instance->subscriber->callback (signal_instance->connection,

#5  0x00007f8482614c0e in g_main_dispatch (context=0x55da0b9108c0) at ../glib/gmain.c:3309
3309	          need_destroy = !(* dispatch) (source, callback, user_data);

#6  g_main_context_dispatch (context=context@entry=0x55da0b9108c0) at ../glib/gmain.c:3974
3974	      g_main_dispatch (context);

#7  0x00007f8482614fc0 in g_main_context_iterate (context=context@entry=0x55da0b9108c0, block=block@entry=1, dispatch=dispatch@entry=1, self=<optimized out>) at ../glib/gmain.c:4047
4047	    g_main_context_dispatch (context);

#8  0x00007f8482615063 in g_main_context_iteration (context=context@entry=0x55da0b9108c0, may_block=may_block@entry=1) at ../glib/gmain.c:4108
4108	  retval = g_main_context_iterate (context, may_block, TRUE, G_THREAD_SELF);

As discussed on IRC, the crash looks like a bug in the native file chooser implementation. It accesses an already freed pointer if the dialog is freed too fast, like here.
The issue that the dialog disappears immediately looks like an API ugliness in GTK and should be reported there to find out what the best way would be to keep the dialog alive long enough.


Fixed #23916 -- Made ForeignObject.deconstruct() use lowercase model name

Ticket
I believe this is the correct fix.
The reason continuous migrations were being generated was the comparison of the before and after deconstructed ForeignKeys in MigrationAutodetector.generate_altered_fields:
if old_field_dec != new_field_dec:

The model name in the migration history is based on the migration that added the model, but nothing ever changes this casing since the autodetector only uses the "model name" (lowercased class name) throughout. ForeignObject.deconstruct() seems to have been a place where this was missed.

Oh yes ofc
I added the below to the autodetector tests but it already passed before the patch. I think it's because the error wasn't in the autodetector but in the difference between model states / deconstructed fields loaded from migration history and actual models.
    def test_model_case_change(self):
        AuthoR = ModelState('testapp', 'AuthoR', [("id", models.AutoField(primary_key=True))])
        changes = self.get_changes([self.author_empty, self.book], [AuthoR, self.book])
        self.assertNumberMigrations(changes, 'app', 0)

That said, this patch does fix the problem on $clientproject that had the same bug as the ticket.
I will come back to this tomorrow to try add a unit test that does reproduce the scenario.

Is the only thing blocking the merge of this change a valid regression test? If it's the case I could likely reproduce if there was a simplified test case.

@charettes Yes, a regression test is missing.

Took another pass. Found a way to fix up my above test to fail on master and pass on this PR. Mostly it was my call to assertNumberMigrations using the wrong app label(s) 🤦♂

Great, LGTM


Move to tagged version of frontend repo

Codecov Report

Merging #1157 (526f925) into main (244da3d) will not change coverage.
Report is 2 commits behind head on main.
The diff coverage is n/a.

:exclamation: Current head 526f925 differs from pull request most recent head 3ab0b9b. Consider uploading reports for the commit 3ab0b9b to get more accurate results

@@           Coverage Diff           @@
##             main    #1157   +/-   ##
=======================================
  Coverage   91.29%   91.29%
=======================================
  Files          49       49
  Lines        1551     1551
  Branches      200      200
=======================================
  Hits         1416     1416
  Misses        122      122
  Partials       13       13

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Harvester: Fix unit-test error

Summary
Fixes #7160
Fix unit-test error

Occurred changes and/or fixed issues

https://github.com/rancher/dashboard/issues/7160

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

Closed in favor has conflicts with fix build-pkg PR.


Remove Underscore [1 of 2]

Mission Control to Velocity 📢

i get this on opening side panel

what is the 2nd part of this PR going to do?

Same thing but removes more underscores. It was pretty overwhelming to look at all at once

👍

There is still an underscore reference here: https://github.com/dcos/dcos-ui/pull/7/files#diff-56274b172383b14152eb3c89445c36b3R1

got a linting error (which is failing the test)

linting error addressed!


Update react-native sb init for 5.1

sb init for React Native modifies the user's package.json to call storybook start.
However in 5.1 this has changed to start-storybook, see:
https://github.com/storybooks/storybook/blob/next/MIGRATION.md#react-native-server
We need to update the CLI to fix this (and if there are any other 5.1 fixes needed?)

Jiminy cricket!! I just released https://github.com/storybooks/storybook/releases/tag/v5.1.0-alpha.28 containing PR #6515 that references this issue. Upgrade today to try it out!
Because it's a pre-release you can find it on the @next NPM tag.


Passing brokerId while deleting will

Coverage remained the same at 97.416% when pulling c9d3442e39ecf81a40b510a3789ddd4152c534c2 on passWill into efa65533cd8f97e90ed534b80759dbfc710bd621 on master.

Coverage remained the same at 97.416% when pulling cb16c29556ad6262165c27a942e5e7835ed6b794 on passWill into efa65533cd8f97e90ed534b80759dbfc710bd621 on master.

Coverage remained the same at 97.416% when pulling d113b157befd6c8a384af9d9b74dbc3776b2e71c on passWill into efa65533cd8f97e90ed534b80759dbfc710bd621 on master.

Should we publish old will message ?
https://github.com/mcollina/aedes/blob/master/aedes.js#L98
In case, will topic will publish on first time server _clearWillInterval before old will deleted ?
https://github.com/mcollina/aedes/pull/98#issuecomment-291090278

@nguyenthenguyen Nice question. I feel it should be published because the clients did disconnect ungracefully and then reconnect when the broker restarted . @mcollina  What do you think ?

@nguyenthenguyen no, the replacement is just internal, not logical.


Undesired behavior using same libp2p Node-Key on many nodes

Running identical nodes could have an impact or destabilize the network.
It should not be allowed to run several nodes using the exact same identity parameters. Connected nodes are using unnecessary resources and opening/closing connections constantly. Even if the full multi-address is different, the connections keep switching.
These nodes don't need to be validators, so it could happen on a considerable scale without any way of being slashed or punished.
Is this an expected behavior?, or there should be a way to penalize a libp2p ID for being connected twice.
substrate --name Test-1 \
--node-key bf9f922a3faa37cabba98ba7a96d74e64435097f328eaf808acf355bc522832c \
--node-key-type Ed25519

substrate --name Test-2 \
--node-key bf9f922a3faa37cabba98ba7a96d74e64435097f328eaf808acf355bc522832c \
--node-key-type Ed25519

We don't support using the same network key from multiple different nodes.
The network key is the node. If we receive a connection from a node whose identity is A while we're already connected to a node whose identity is A, we assume it's the same node and close one of the two.

It should not be allowed to run several nodes using the exact same identity parameters.

I don't see how we could detect that.

These nodes don't need to be validators, so it could happen on a considerable scale without any way of being slashed or punished.

We locally decrease the reputation value of a node every time it disconnects. Over time it will get less and less priority.

We don't support using the same network key from multiple different nodes.

This makes total sense. My concern is related to some tests that I did on Polkadot where I forced two nodes to have the same network key, and they stayed connected to many peers. I can also say that my test was for just one day so I don't know how fast decreasing the reputation can help.

The network key is the node. If we receive a connection from a node whose identity is A while we're already connected to a node whose identity is A, we assume it's the same node and close one of the two.
We locally decrease the reputation value of a node every time it disconnects. Over time it will get less and less priority.

This is something I was thinking about, but this reputation goes down with time. Changing the key will create a "new" node identity. It remains as a possibility to use the same key as a way to attack and then rotate keys every few hours. There will be no way to decrease reputation at the needed speed to prevent the attack.

I don't see how we could detect that. We know that we're talking to one node and not two based on the nodes' identity.

Neither do I. It looks to be a complex issue. Does the IP or the full multi-address could help?

This makes total sense. My concern is related to some tests that I did on Polkadot where I forced two nodes to have the same network key, and they stayed connected to many peers. I can also say that my test was for just one day, so I don't know how fast reputation should decrease to mitigate this.

This reputation system is pretty recent and not really fully implemented yet.

This is something I was thinking about, but this reputation goes down with time. Changing the key will create a "new" node identity. It remains as a possibility to use the same key as a way to attack and then rotate keys every few hours. There will be no way to decrease reputation at the needed speed to prevent the attack.

On the other hands, the reputation is regular nodes will increase over time. If you create a new identity, you start below existing nodes.
Of course you could act benevolent for a while, until you build up reputation, then start being dishonest, but that means attacks don't last very long.

Neither do I. It looks to be a complex issue. Does the IP or the full multi-address could help?

Some nodes could legitimately have multiple different ways to be reached.
Each node has generally at least both an IPv4 and an IPv6, and in the future maybe some connectivity through a proxy.
This would also raise the question of how long do you cache the IP address of a node.
Identifying a node based on its IP address or multiaddress is in my opinion a step in the wrong direction.

This reputation system is pretty recent and not really fully implemented yet. From a technical point of view it's very easy to make it behave the way we want, but we're not sure exactly what "the way we want" is.

Great to hear that there is a work in progress solution :)

On the other hands, the reputation of regular nodes will increase over time. If you create a new identity, you start below existing nodes.

Does starting from below creates a time restriction for connection retires or restricts number of nodes?
When you launch a new node it will connect to many nodes, it could be part of this reputation to limit the initial connections and over time gain access to more nodes. By doing this the attack vector could be smaller.

Of course you could act benevolent for a while, until you build up reputation, then start being dishonest, but that means attacks don't last very long.

Sound like the way to go. Working to increase your reputation is the best approach.

Some nodes could legitimately have multiple different ways to be reached.
Each node has generally at least both an IPv4 and an IPv6, and in the future maybe some connectivity through a proxy.
This would also raise the question of how long do you cache the IP address of a node.
Identifying a node based on its IP address or multiaddress is in my opinion a step in the wrong direction.

Agree, this is not the best approach, many ISPs change the IP very regularly. It can also expose the node to more attacks.
By the way, it seems that most of these concerns have already been analyzed. Thanks for replying :)
I just wanted to expose possible issues. If all this has been taken into account, do not hesitate to close the issue.

Does starting from below creates a time restriction for connection retires or restricts number of nodes?

Nodes would randomly periodically disconnect well-behaving nodes in order to leave room for newcomers.
We don't want to have, say, 25 nodes all connected together and the 26th cannot join because everybody is full.

I'm going to close this unless you have further questions.


Storybook: Implement A11y addon

Why?
We want to emphasize our focus on accessibility.
By adding the addon-a11y add-on to our Storybook, we are able to show our commitment to accessibility and the quality of our components.
Acceptance criteria

[x] The Accessibility tab of the add-on is available in our Storybook Stories
[ ] If possible, share the axe configuration (not the ignore rules)

Definition of Done

[x] follow-up tickets were created if necessary
[x] updated version + documentation is deployed
[x] Storybook can show the feature
[x] Set task to In Approval

Approval

Storybook, the ticket is about the accessibility tab right next to 'Controls' and 'Actions'.
[x] approved by designer @jannick-ux
[ ] approved by dev add dev tag

Implementation details

Follow the instructions on https://storybook.js.org/addons/@storybook/addon-a11y

Reference implementations

Trinity

@MajaZarkova Do you know which rules the addon applies/checks by default? Since we decided to aim for AA, we should ensure that all rules for AA are also included in the Storybook addon :)

I don't, but I can investigate

@larsrickert , I found the list with the rules -> https://github.com/dequelabs/axe-core/blob/develop/doc/rule-descriptions.md

@larsrickert , I found the list with the rules -> https://github.com/dequelabs/axe-core/blob/develop/doc/rule-descriptions.md

These are just all available rules, right? We should check which of them are actually checked by the Storybook addon. E.g. it might only check a few of them by default and we might need to configure it to check more. But I am not sure 🤔


Update the oracles to match LUIS.

Add roles to testing.
Revert node-only change that caused passing through raw resolutions.
In particular for phone instead of phone:["425-765-6532"] the bug
would cause phonenumber: [{score:0.9, value:"425-555-1212"].

Closing as we're not needing this in 4.4, same changes were merged into master via #911


Fixed display of working time in Grafana management dashboard

Fix SQL query after https://github.com/cvat-ai/cvat/pull/7958
Motivation and context

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have created a changelog fragment
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[x] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Summary by CodeRabbit

Bug Fixes

Modified analytics dashboard queries to include data from all sources, not just 'client'.

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

Project coverage is 83.63%. Comparing base (cdf50d3) to head (b04a7fc).
Report is 10 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #8112   +/-   ##
========================================
  Coverage    83.62%   83.63%
========================================
  Files          383      383
  Lines        40450    40502   +52
  Branches      3820     3830   +10
========================================
+ Hits         33828    33875   +47
- Misses        6622     6627    +5

Components
Coverage Δ

cvat-ui
79.68% <100.00%> (+0.05%)
:arrow_up:

cvat-server
87.25% <ø> (-0.03%)
:arrow_down:


Groups not working properly on SAH Commons

A user contacted us because she was unable to create a new topic in the SAH Historic Interiors Affiliate Group. Upon checking into this further, @bjr70 was unable to create a topic or join any SAH groups. This could be related to the pop-ups issue we experienced after the WP6 update.

I can replicate this on SAH on every group, but all other commonses are working. This appears to be an issue where the popup isn't firing when you join.

This same issue might be happening on UP Commons as well. We received an email about a user who wrote "I could have sworn that I was a member of a few groups – including Editorial, Design, and Production. But now I am not. So I was trying to “Request Membership” but can’t find the button on the home page."

Pietr Test works just fine. Can we ask Jennifer for more information and maybe set up a Zoom meeting to troubleshoot?

Huh, weird. I'll email her back to ask.

Was able to access and join groups as hctester40. Was not able to create a new topic on the discussion board. As described by user, when I clicked on "Create New Topic," nothing happens.

I believe this was a javascript issue caused by the Autoptimize plugin. I have deactivated the plugin for now---it wasn't active on all networks as seems to be more trouble that it's worth for very minor performance improvements.

@babaklar - Can you test this out and ask users to retry?

I used Ferdinand Test and it worked! I emailed the users to have them double check. I'll close once I hear back.


Initial support of heatmaps

Closes #27

Waiting for https://github.com/grafana-tools/sdk/pull/122

Waiting for https://github.com/grafana-tools/sdk/pull/122


Update aws-java-sdk-core to 1.12.717

About this PR
📦 Updates com.amazonaws:aws-java-sdk-core from 1.12.459 to 1.12.717
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #755.


⚠ Samsung has degraded performance

In 7a85b82, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2023 ms

Resolved: Samsung performance has improved in 4d9b3b1.


Add Application.GetController extension methods

Create branch: issue-35-add-application.getcontroller-extension-methods from main.


implement ocis storage-authmachine subcommand

while a command exists in https://github.com/owncloud/ocis/blob/eb3e0151f46cb0bc7740a1cfcc69934d502c6a1c/storage/pkg/command/authmachine.go#L23
and is started when starting ocis in https://github.com/owncloud/ocis/blob/eb3e0151f46cb0bc7740a1cfcc69934d502c6a1c/ocis/pkg/runtime/service/service.go#L108
there is no ocis subcommand for it in https://github.com/owncloud/ocis/tree/6d5a3733ffaf45690c66dca97c790aa21af94eae/ocis/pkg/command

fixed by https://github.com/owncloud/ocis/pull/2910


output file limit hint and align with resource files

Azure Batch Output Files have a similar limit as Resource Files (see Creating and using resource files).
This pull request:

Adjusts the layout of the docs page to group relevant subtopics under a single section. It splits the prerequisites (creating a container) and the actual definition of output files. The section Specify output files for task output now groups:
(1) Using a Shared Access Signature
(2) Using Managed Identity
(3) Specify a file pattern for matching
(4) Specify an upload condition
(5) Disambiguate files with the same name
(1) and (2) now align with resource-files.md. (3) to (5) are sub topics to configure Output files further.

This enables also adding a new sub section Many Output Files to inform users about the limit that the Azure Batch API imposes (similar to the Resource Files) and suggest options to work with these limits.

Let me know what you think (fyi: @alfpark , @Padmalathas)

@Padmalathas

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Docker compose fix

Thank you! Honestly I'd like to make more changes to docker-compose, like removing separate "volumes" section entirely.
Please note that the main development is done on my private GitLab repo, so I'd copy your changes and close this PR. Sorry for the inconvenience!

No problem, thanks :)

I'll keep this open for now until I fix this.

Fixed in ea866de47ed9b42448239b40282ed985f5212beb (along with other issues)


[Unit Test] Added CUnit test framework for target

* Added sample test framework for DSS target using CUnit
* Added Readme on how to create new test cases
* Update build script to run tests
* Added shellcheck disable=SC2086 for cmake command
* Tested building all combinations with coverage and run tests flags

@benixon Do we need to worry about the smells?

@benixon Can you please provide the path of the coverage report in the README? Unless I'm not seeing it 👨🦯


Adds Atom atom map and rlabel apis

Adds atom map and Rlabel apis to the Atom.  Atom maps probably should be on the Atom, rlabels could be free functions and not on the Atom.  We could do the same for molfile values and aliases.
C++
atom->setAtomMapNum(mapno);
atom->setRlabel(rlabel);
atom->getAtomMapNum();
atom->getRlabel();

python
atom.SetAtomMapNum(mapno)
atom.SetRlabel(rlabel)
atom.GetAtomMapNum()
atom.GetRlabel()

Values of 0 unset atom maps and rlabels.

These don't feel like an essential piece of the functionality of Atoms, so in the spirit of keeping the interface of classes minimal I'd be happier with these being functions defined in Atom.h instead of member functions.
So something like void setAtomMapNum(Atom &atom,int mapno); and void setAtomRLabel(Atom &atom, int rLabel). (Note that I also changed the capitalization on that last one)
The functions would be in the RDKit namespace.

I agree with RLagel being a free function, I would argue that since atom maps are used in smiles/smarts and sd files I feel that they are essential and, perhaps, should be promoted more.  Tagging an atom for bookkeeping seems a very common cheminformatics task.

The appveyor builds failures are looking real, although they make no sense:
C:\projects\RDKit\Code\RDGeneral/RDProps.h(67): error C2039: 'computedPropName' : is not a member of 'RDKit::detail' [C:\projects\RDKit\build\Code\GraphMol\Substruct\SubstructMatch.vcxproj]

These  namespaces and variables absolutely do exist, I'll have to do some digging on my windows box.

C:\projects\RDKit\External\INCHI-API\inchi.cpp(64): fatal error C1083: Cannot open include file: 'inchi_api.h': No such file or directory [C:\projects\RDKit\build\External\INCHI-API\RDInchiLib.vcxproj]
I've been seeing this lately.  It looks like some form of inchi is trying to be built even if inchi is turned off.  I'll try to hunt it down.


make it easy !

can create a method to control a widget children from functions & can remove it and change and more like JAVA-SCRIPT control a elements in HTML

Thanks for your efforts here, but this isn't quite the way we roll to beta.  Perhaps you could look at opening an issue to describe more clearly what you're trying to achieve, or you could open a pull request with your changes against master for consideration.
Feel free to take a look at our contributor's guide if you haven't already!


Highlight doxygen C++ tparam argument

C++ templates can be documented using the 'tparam' command. I would like the template argument name to be highlighted just like the argument name for regular arguments.
In the following exemple, "TSerialSettings" should be highlighted
/// @brief Represent a serial port.
/// @tparam TSerialSettings The settings to use with the serial port.
template <typename TSerialSettings>
class serial {};

It's almost identical to the 'param' command, except there is no direction (in, out). This means that a quick fix can be to change "param" into "t?param" everywhere it's supposed to be supported, like here: https://github.com/jeff-hykin/cpp-textmate-grammar/blob/master/syntaxes/cpp.tmLanguage.json#L1276
Obviously, this will also highlight the following syntax:
/// @brief Represent a serial port.
/// @tparam[in] TSerialSettings The settings to use with the serial port.
template <typename TSerialSettings>
class serial {};

But I don't think it's a big deal as it's not a valid syntax for doxygen.

Are there any updates or plans to address this open issue?


[Tests] re-enable roks and vsi pattern tests in SLZ using Key Protect instead of HPCS

Description
re-enabled roks and vsi pattern tests in SLZ using Key Protect instead of HPCS
Types of changes in this PR
Changes that affect the core Terraform module or submodules

[ ] Bug fix
[ ] New feature
[ ] Dependency update

Changes that don't affect the core Terraform module or submodules

[x] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other

Release required?
Identify the type of release. For information about the changes in a semantic versioning release, see Release versioning.

[x] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
If a release is required, replace this text with information that users need to know about the release. Write the release notes to help users understand the changes, and include information about how to update from the previous version.
Your notes help the merger write the commit message for the PR that is published in the release notes for the module.
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author. The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).
Merge by using "Squash and merge".

/run pipeline

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

GitHub release
v4.4.5

Your semantic-release bot :package::rocket:


Enable replacing video file

There is no way to replace an existing vimeo video.
This has been asked here:
https://stackoverflow.com/questions/53497622/vimeo-unity-sdk-how-do-i-replace-an-existing-video
Maybe make it implicit when uploading video with same name and same project ?

Thanks for filling here!
Replacing by video name & project is interesting but wouldn't say that is always the expected behavior. We'd probably want to give users some sort of drop-down selection of which video they want to replace (and just specify video ID via script)
Just curious, what are you working on / what is your use case?

Amazing. We just discovered blueturn earlier this morning - it's so cool! I'm going to reach out to you over email if that's OK :)

Really ? What a coincidence! Feel free to contact me.
On Wed, 28 Nov 2018 at 22:34, Casey Pugh<EMAIL_ADDRESS>wrote:

Amazing. We just discovered blueturn earlier this morning - it's so cool!
I'm going to reach out to you over email if that's OK :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vimeo/vimeo-unity-sdk/issues/55#issuecomment-442594894,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE39pBiNW70-yPv5lxv955vfEPYjZVFqks5uzvNHgaJpZM4Y3XBL
.

Hi Casey,
Can you please check this fix for this issue.
I did it all according to the documentation in here, but I'm getting an error from the server.
https://developer.vimeo.com/api/upload/videos#replacing-a-source-file
Thanks!
Michael


Add missing python standard libs

This libs were missing:

asyncio
concurrent
http
html
typing
xmlrpc
ftplib
numbers

Looks good, thanks!

@timothycrosley typing isn't standart module in python2. This PR breaks pep8.

@tkukushkin this PR doesn't break pep8:
isort can't know what version of Python your code will run on, and even if you are making a Python2 app it is good practice to avoid using Python3 module names in case you ever decide to port that app to Python3 or have it run on both - a very common decision.Finally,  If you really do want to include typing in a Python2 project you can always tell isort that it's a project level import and it will sort it correctly.
Thanks!
~Timothy


Update ghidraRun to support --debug flag

We provide support/ghidraDebug to support this type of thing.


Patched DataFusion, Oct 15, 2024

Part of https://github.com/influxdata/influxdb_iox/issues/12626
Follow on to https://github.com/influxdata/arrow-datafusion/pull/42

This PR is not intended to merge.
This PR documents the contents of the https://github.com/influxdata/arrow-datafusion/commits/df-upgrade-oct-15/ branch
Based on DataFusion at the end of day Oct 15 2024: 399e8403d2fb5a058fb263bc2449dbc9d7529841
Here are all the changes that went in upstream since https://github.com/influxdata/arrow-datafusion/pull/42

https://github.com/influxdata/arrow-datafusion/compare/02427674eef658b9b0acb7142f18e8c1520bdb17...influxdata:arrow-datafusion:399e8403d2fb5a058fb263bc2449dbc9d7529841

Patch list:

You can see the changes on this PR as commits

No longer used in iox. Closing.


feat: Updated for using TaskBuilder instead of PipelineTaskBuilder

Signed-off-by: Nathan Good<EMAIL_ADDRESS>Fixes #

Mistakenly opened PR against main. Closing.


A basic template with responsiveness

I hope this code is good enough for your collection . Please accept this PR asap!! and also mark it as hacktoberfest-accepted.

Do give a Star to this repo.. happy hacking.


Structurify compat crash

Hello there, i recently released mod named Structurify https://modrinth.com/mod/structurify, basically (simply said) i do load the registries before the actual world load here, so when i start the world this happens (see the logs):
Log file: https://mclo.gs/a844yaR

Also this doesnt happens, if i first go into my Structurify config (registries loads beforehand).
My suspicion is that this is probably some kind of bug on my side, when events in neoforge are triggered differently than in forge(probably later on)

I tried what i thought would be fix for the problem, but same problem: https://mclo.gs/KkL9QrS

I found the solution to this on my side.


Regression textbook_distribution

Textbook_distribution lo report defects.
1.If no data in the table report, Then the data display section becomes small.
2.year and month
Choose Year(2022-23)
Choose Month(September)
Choose Week(5)
Choose Date(2022-09-30)
Change the grade from 3 to all grades
Issue : reset is happening and re-directing to overall
3.Choose default week selection is not happening
4.year and month
Choose Year(2022-23)
Choose Month(September)
Choose Week(5)
Issue : tooltip showing district name as undefined
5.year and month
Choose Year(2022-23)
Choose Month(September)
Choose Week(5)
Choose Date(2022-09-30)
Choose Grade(4)
Choose Subject(english)
Choose District(Agra)
Choose Block(Acchnera)
Choose Cluster(Arrua Khash)
Issue : grade is showing as 0, subject is showing as grade_4 and download filename is not proper(textbook_undefined_distribution_Grade_4_schools_of_cluster_915020008_September_2022-23_24-10-2022)
6.Blocks and cluster names are bold
7.Download filename is not proper
ex: textbook_undefined_distribution_Grade_4_schools_of_cluster_915020008_September_2022-23_24-10-2022
8.Choose TimeRange(last 30 days)
Choose Grade(Grade_5)
Choose Subject(maths)
Choose District(Agra)
Choose Block(Acchnera)
Choose Cluster(Arrua Khash)
Change the time range from last_30_days to last_day
issue : data is displaying but last_day we don't have data
Textbook_distribution map report defects.
1.Click on textbook-distribution map report
Click on legend
issue : gray color markers are showing
2.Choose TimeRange(last 30 days)
Choose Grade(Grade_3)
Choose a subject(english)
issue : Footer value is mis matching at subject level, district,block and cluster level
2.Choose TimeRange(last 30 days)
Choose Grade(Grade_3)
Choose a subject(english)
Choose a district(agra)
Choose a block( Acchnera)
Choose a cluster
issue : Legend values are showing as 0 for district, block and cluster selection
3.Choose TimeRange(last 30 days)
Choose Grade(Grade_3)
Choose a subject(english)
Choose a district(agra)
Click on cluster, schools button
issue : disabling of buttons is not happening and footer value is not proper total schools showing 0
4.Downlod report names are not proper
ex: textbook_distribution_allDistrict_undefineds_25-10-2022.csv
5.Choose TimeRange(overall)
Choose Grade(Grade_3)
Choose a subject(english)
Choose a district(agra)
Choose  TimeRange(last_day)
Issue : No data found is not displaying
6.Click on textbook_distribution map tab
Click on textbook_distribution lotable
Click on textbook_distribution map tab
Issue : legend, about discription and footer values is not displaying

Textbook map report :
Size of the block ,cluster ,school button is large when compared to other reports

@RNarmatha
Re-tested the above defects are all - Not Fixed except
6.Blocks and cluster names are bold - Resolved

Defect 7: when we select grade and subject - legend card showing 0 for all the buttons

This issue is not covered in v5.0 and to be analysed

@htvenkatesh @pandutibil - request you to address this bug.

Text book distribution program not consider in the current version


notifications and changes on data

Hi, I know EF already has entity tracking, but its freakishly outdated and slow, for ReadOnly-Observable is there a way to hook up the table as a virtual list to the Database table and watch/subscribe to those changes. This would help dispatch toaster events to the UI.
Here is a gem https://tabledependency.codeplex.com/

As long as you update your tables through services, you don't need to worry much about caching. But this looks like a useful library. I'll have a look. Last time i tried SqlDependency, it had a bad habit of failing suddenly along with all code that depends on it, so i couldn't trust on its stability and gave up. Hope they fixed issues.


[Button] Contained with inherit prop not adapting on dark mode

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
As mentioned by @JeremyGrieshop in another thread https://github.com/mui-org/material-ui/issues/28363#issuecomment-921748227 the contained variant with the inherit color prop does not adapt its background color.
Code sandbox by Jeremy demonstrating the issue: https://codesandbox.io/s/togglecolormode-material-demo-forked-1j544?file=/demo.js
Expected behavior 🤔
Since it is using grey[300] for the light mode, maybe switching to something like grey[700] or dark mode?

Steps to reproduce 🕹
Steps:

Context 🔦
No response
Your environment 🌎
@mui/material": "5.1.0"

I can take this issue!


⚠ ClamAV has degraded performance

In b04cb21, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 63 ms

Resolved: ClamAV performance has improved in 0a75783.


⚠ Minecraft has degraded performance

In 5d97a0c, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1454 ms

Resolved: Minecraft performance has improved in bea2fe6.


Add functionality to convert JSON datasets to Gematria protos.

JSON datasets, while not very useful otherwise, being easy to read are handy for debugging while also being rich enough to store extra, possibly nested, data such as instruction annotations.
Note: the current JSON format includes instruction annotations and so depends on AnnotationProto from #27, and will not build without gematria/proto/annotation.proto.

It makes more sense to use the Gematria basic_block data structures and the proto format over something like JSON, so this is being closed.


Input: Document that accumulated input is disabled by default

This was actually disabled by mistake in 3.4 causing a regression, but since
this issue survived the whole 3.4.x series and it's now very close to 3.5,
it's too late to change it again.
We might consider it for 3.6 after some beta testing.

Fixes https://github.com/godotengine/godot/issues/55037.

Some more details on the defaults here:

Accumulated input was implemented in a1e73dcc944627ab7185aec7cd4141fe4ebb97d7, off by default at runtime but enabled in the editor.
2 days later it was enabled by default always in ecd8795755095e5d8c002f030e51bb26892d7cbb. Those two changes were made for Godot 3.1.
Then #42220 refactored things and added agile input flushing, and turned accumulated input off by mistake (still enabled in the editor by a call in EditorNode). This was changed in Godot 3.4.
In master (which started diverging with 3.x at Godot 3.2), #42220 was forward-ported by #51597 but accumulated input was not turned off then. On the other hand... I turned it off by error myself in #38697 while refactoring member initialization. #62665 should fix it.

Cherry-picked for 3.4.5.


storage: support setting public/private at time of upload

Fixes #1178 (whoa, exactly 100 issues/PRs ago!)
Set the predefinedAcl at the time of upload:
bucket.upload(filepath, { predefinedAcl: 'authenticatedRead' }, function(err, file) {})

var writeStream = file.createWriteStream({ predefinedAcl: 'authenticatedRead' })

Also with built-in public (predefinedAcl: publicRead) and private (predefinedAcl: private) options:
bucket.upload(filepath, { public: true }, function(err, file) {})

var writeStream = file.createWriteStream({ private: true })

Coverage decreased (-0.02%) to 99.978% when pulling aaf8982c9c0eef6ec7b11dec59c892be9d947957 on stephenplusplus:spp--1178 into 7c71ec5128059df6d79935464a8345c684ad0ff0 on GoogleCloudPlatform:master.

Thanks for this--I like the quick public private ones as well :)

Coverage remained the same at 100.0% when pulling e852ffdda94e3d4a55cc7a7a9ac8d8ee3fd20ca4 on stephenplusplus:spp--1178 into 7c71ec5128059df6d79935464a8345c684ad0ff0 on GoogleCloudPlatform:master.


Add to log data

The log screen currently shows lat, long, and time. Is there any way for it to show the activity and other major settings if you press each entry? Still trying to be able to debug.

if you switch the audio notifications in the panel settings, you'll hear the activity changes. I can add color of string to notify the activity change... it's an idea. Let me think about it.


Feature request: "Show only the title" alternative to "Number of lines"

I often scroll through feeds and glance at the titles. With current options, they are either cut off or there is content showing beneath them.

I feel like "Show only the title" option would make my feed views look much more clean and nicer to look at. :)

The way things work now is nice because it allows us to make each row the same height as the other rows, which we think makes for a nice and clean look.
If we showed entire titles, then we’d have variable-height rows, which is kind of a messy look. Some might even be quite long. (And then there’s the issue of what to do when an article doesn’t have a title.)

@brentsimmons Yeah that makes sense. Thank you for replying! :D


Abundant broken spawners

Hello,
So I have been playing a SSP map with this mod as well as RotartyCraft. I have been experiencing a 100% drop rate of broken spawners while using both the HLSA steel and bedrock pics. Both un-enchanted. Just wanted to share since I know they are chance based.

Sorry. Wrong forum. My mistake.


Add ability for nginx images to inject Matomo snippets for website tracking.

Matomo, formerly Piwik, is a free and open source web analytics application which tracks online visits to one or more websites and displays reports on these visits for analysis.
Rather than having to manage code snippets per project it would be nice to be able to inject code snippets into a page via environment variables. This would especially be useful for lagoon users with a large number of projects.
Using the ngx_http_sub_module capabilities of NGINX we could provide a the following environment variables :-
MATOMO_URL - URL to the Matomo isntallation
MATOMO_SITE_ID - Matomo site ID
MATOMO_TAG_MANAGER_URL - Optionally, providing support for Matomo tag manager.
Setting these variables on the NGINX container could automatically enable the snippets to be injected into the HEAD section of all HTML responses.

Moved to lagoon images repo -- https://github.com/uselagoon/lagoon-images/issues/95


fix: cleaning warning messages

Description
Cleaning of warning messages
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Gogs image not available

Hi,
The image for the gogs container isn't available. I'll be grateful if you can push it.
Thanks!

Thanks, I have pushed it.
In addition, the image is here: https://github.com/vulhub/vulhub/blob/master/base/gogs/0.11.66/Dockerfile, just inherited from gogs/gogs.


[Bug] Comments getting deleted

Describe the bug
https://github.com/curvefi/scrvusd/blob/main/contracts/TWA.vy#L167-L170
These lines keep getting deleted during reformatting

Fixed in 0.0.4a3


Building ESP32 demo not enough space

Hi
While compiling I get this error.

/amazon-freertos/demos/espressif/esp32_devkitc_esp_wrover_kit/make/partitions_example.csv' occupy 2.2MB of flash (2293760 bytes) which does not fit in configured flash size 2MB. Change the flash size in menuconfig under the 'Serial Flasher Config' menu.

I assume changing the flash size is here in this situation not the correct solution. Anybody that can reproduce this?
Thanks!

Apparently I misinterpreted the error, changing the flash size is the correct solution


Run in Section for notebook sticky scroll context menu

@Yoyokrazy do you get a chance to update the PR for further review?


Allow user to pass default-headers to gate client

This PR provides solution for #22
This allows user to set the new default-headers flag that was added as part of this PR https://github.com/spinnaker/spin/pull/182
This has been built and tested both hitting a locally running gate api with no headers, as well as gate exposed via an api gateway passing through authentication header.

Hey @ethanfrogers apologies for messaging directly, but would you mind having a look at this PR when you get a bit of free time to see if it looks okay please?

Hi @MarkFreebairn, this pull request looks reasonable.  If you resolve the conflicts since our recent Go 1.12 updates I think we can get this merged.


RuntimeError: CUDA error: an illegal memory access was encountered

Hello，everything is normal in the previous training, but the loss suddenly becomes very large in step 3250, and then the error occurs. What's the problem
2019-12-03 13:36:58,567 - INFO - Epoch [6][3150/3681]	lr: 0.02000, eta: 3:50:55, time: 0.924, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.0431, loss_rpn_bbox: 0.0594, loss_cls: 0.2494, acc: 90.6348, loss_bbox: 0.2668, loss: 0.6186
2019-12-03 13:37:26,314 - INFO - Epoch [6][3200/3681]	lr: 0.02000, eta: 3:50:21, time: 0.555, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.4505, loss_rpn_bbox: 0.1093, loss_cls: 0.2473, acc: 92.7031, loss_bbox: 0.2011, loss: 1.0083
2019-12-03 13:37:43,641 - INFO - Epoch [6][3250/3681]	lr: 0.02000, eta: 3:49:37, time: 0.347, data_time: 0.005, memory: 6147, loss_rpn_cls:<PHONE_NUMBER>, loss_rpn_bbox: 86211.9636, loss_cls: 18325.3354, acc: 83.0372, loss_bbox: 3535.8495, loss:<PHONE_NUMBER>
2019-12-03 13:38:23,517 - INFO - Epoch [6][3300/3681]	lr: 0.02000, eta: 3:49:16, time: 0.798, data_time: 0.005, memory: 6147, loss_rpn_cls: 91894506.7258, loss_rpn_bbox: 23305796.7583, loss_cls:<PHONE_NUMBER>.8865, acc: 83.2261, loss_bbox:<PHONE_NUMBER>4.5904, loss:<PHONE_NUMBER>7.6990
2019-12-03 13:39:02,348 - INFO - Epoch [6][3350/3681]	lr: 0.02000, eta: 3:48:54, time: 0.777, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.8049, loss_rpn_bbox: 0.1222, loss_cls:<PHONE_NUMBER>, acc: 93.7891, loss_bbox:<PHONE_NUMBER>, loss:<PHONE_NUMBER>
2019-12-03 13:39:27,972 - INFO - Epoch [6][3400/3681]	lr: 0.02000, eta: 3:48:18, time: 0.512, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5977, loss_rpn_bbox: 0.1056, loss_cls: 89.4625, acc: 94.3125, loss_bbox: 639.2675, loss: 729.4332
2019-12-03 13:39:50,872 - INFO - Epoch [6][3450/3681]	lr: 0.02000, eta: 3:47:39, time: 0.458, data_time: 0.005, memory: 6147, loss_rpn_cls: 5375.5316, loss_rpn_bbox: 5233.3437, loss_cls: 0.3641, acc: 92.4902, loss_bbox: 0.0204, loss: 10609.2588
2019-12-03 13:40:15,885 - INFO - Epoch [6][3500/3681]	lr: 0.02000, eta: 3:47:03, time: 0.500, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5581, loss_rpn_bbox: 0.1263, loss_cls: 0.2827, acc: 94.6035, loss_bbox: 0.0101, loss: 0.9772
2019-12-03 13:40:36,492 - INFO - Epoch [6][3550/3681]	lr: 0.02000, eta: 3:46:22, time: 0.412, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5754, loss_rpn_bbox: 0.1105, loss_cls: 0.3166, acc: 93.7832, loss_bbox: 0.0087, loss: 1.0113
2019-12-03 13:41:33,230 - INFO - Epoch [6][3600/3681]	lr: 0.02000, eta: 3:46:18, time: 1.135, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5789, loss_rpn_bbox: 0.1206, loss_cls: 0.3180, acc: 93.5781, loss_bbox: 0.0135, loss: 1.0310
2019-12-03 13:41:54,390 - INFO - Epoch [6][3650/3681]	lr: 0.02000, eta: 3:45:38, time: 0.423, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5287, loss_rpn_bbox: 0.0983, loss_cls: 0.2554, acc: 95.2637, loss_bbox: 0.0082, loss: 0.8906
2019-12-03 13:42:43,852 - INFO - Epoch [7][50/3681]	lr: 0.02000, eta: 3:44:23, time: 0.469, data_time: 0.019, memory: 6147, loss_rpn_cls: 0.5594, loss_rpn_bbox: 0.1067, loss_cls: 0.2959, acc: 94.1758, loss_bbox: 0.0112, loss: 0.9732
2019-12-03 13:43:31,021 - INFO - Epoch [7][100/3681]	lr: 0.02000, eta: 3:44:08, time: 0.943, data_time: 0.005, memory: 6147, loss_rpn_cls:<PHONE_NUMBER>7.9462, loss_rpn_bbox: 0.1410, loss_cls: 0.3417, acc: 92.7656, loss_bbox: 0.0083, loss:<PHONE_NUMBER>8.4338
2019-12-03 13:44:10,153 - INFO - Epoch [7][150/3681]	lr: 0.02000, eta: 3:43:46, time: 0.783, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5577, loss_rpn_bbox: 0.1291, loss_cls: 0.3321, acc: 93.1191, loss_bbox: 0.0073, loss: 1.0262
2019-12-03 13:44:34,169 - INFO - Epoch [7][200/3681]	lr: 0.02000, eta: 3:43:09, time: 0.480, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5878, loss_rpn_bbox: 0.1129, loss_cls: 0.3180, acc: 93.7383, loss_bbox: 0.0125, loss: 1.0312
2019-12-03 13:44:58,516 - INFO - Epoch [7][250/3681]	lr: 0.02000, eta: 3:42:33, time: 0.487, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5105, loss_rpn_bbox: 0.0916, loss_cls: 0.2554, acc: 95.2832, loss_bbox: 0.0158, loss: 0.8734
2019-12-03 13:45:27,276 - INFO - Epoch [7][300/3681]	lr: 0.02000, eta: 3:42:00, time: 0.575, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5878, loss_rpn_bbox: 0.1184, loss_cls: 0.3391, acc: 92.9375, loss_bbox: 0.0139, loss: 1.0593
2019-12-03 13:46:03,470 - INFO - Epoch [7][350/3681]	lr: 0.02000, eta: 3:41:35, time: 0.724, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5407, loss_rpn_bbox: 0.1281, loss_cls: 0.2719, acc: 94.6621, loss_bbox: 0.0111, loss: 0.9518
2019-12-03 13:46:45,389 - INFO - Epoch [7][400/3681]	lr: 0.02000, eta: 3:41:15, time: 0.838, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5501, loss_rpn_bbox: 0.1292, loss_cls: 0.2925, acc: 93.9961, loss_bbox: 0.0058, loss: 0.9776
2019-12-03 13:47:11,399 - INFO - Epoch [7][450/3681]	lr: 0.02000, eta: 3:40:40, time: 0.520, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5634, loss_rpn_bbox: 0.1169, loss_cls: 0.3164, acc: 93.5215, loss_bbox: 0.0100, loss: 1.0067
2019-12-03 13:47:44,780 - INFO - Epoch [7][500/3681]	lr: 0.02000, eta: 3:40:12, time: 0.668, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5887, loss_rpn_bbox: 0.1329, loss_cls: 0.3308, acc: 93.0215, loss_bbox: 0.0134, loss: 1.0658
2019-12-03 13:48:16,044 - INFO - Epoch [7][550/3681]	lr: 0.02000, eta: 3:39:43, time: 0.625, data_time: 0.005, memory: 6147, loss_rpn_cls: 0.5723, loss_rpn_bbox: 0.1283, loss_cls: 0.2985, acc: 94.0254, loss_bbox: 0.0053, loss: 1.0043
Traceback (most recent call last):
File "tools/train.py", line 112, in
main()
File "tools/train.py", line 108, in main
logger=logger)
File "/home/yantianwang/mmdetection/mmdet/apis/train.py", line 60, in train_detector
_non_dist_train(model, dataset, cfg, validate=validate)
File "/home/yantianwang/mmdetection/mmdet/apis/train.py", line 232, in non_dist_train
runner.run(data_loaders, cfg.workflow, cfg.total_epochs)
File "/home/yantianwang/anaconda2/envs/pytorch/lib/python3.6/site-packages/mmcv/runner/runner.py", line 358, in run
epoch_runner(data_loaders[i], **kwargs)
File "/home/yantianwang/anaconda2/envs/pytorch/lib/python3.6/site-packages/mmcv/runner/runner.py", line 271, in train
self.call_hook('after_train_iter')
File "/home/yantianwang/anaconda2/envs/pytorch/lib/python3.6/site-packages/mmcv/runner/runner.py", line 229, in call_hook
getattr(hook, fn_name)(self)
File "/home/yantianwang/anaconda2/envs/pytorch/lib/python3.6/site-packages/mmcv/runner/hooks/optimizer.py", line 19, in after_train_iter
self.clip_grads(runner.model.parameters())
File "/home/yantianwang/anaconda2/envs/pytorch/lib/python3.6/site-packages/mmcv/runner/hooks/optimizer.py", line 13, in clip_grads
filter(lambda p: p.requires_grad, params), **self.grad_clip)
File "/home/yantianwang/anaconda2/envs/pytorch/lib/python3.6/site-packages/torch/nn/utils/clip_grad.py", line 33, in clip_grad_norm
total_norm += param_norm.item() ** norm_type
RuntimeError: CUDA error: an illegal memory access was encountered
(pytorch) yantianwang@wyt2-computer:~/mmdetection$

Please use the Error Template to indicate your modification and configs etc., current information is too less and we are not able to help you.

@wytcsuch Hi did you find the reason why ?


[stable/traefik] Support mutual TLS authentication

Is this a request for help?:

Is this a BUG REPORT or FEATURE REQUEST? (choose one): FEATURE REQUEST

Version of Helm and Kubernetes:
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.0", GitCommit:"ddf47ac13c1a9483ea035a79cd7c10005ff21a6d", GitTreeState:"clean", BuildDate:"2018-12-04T07:51:55Z", GoVersion:"go1.11.2", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.11", GitCommit:"637c7e288581ee40ab4ca210618a89a555b6e7e9", GitTreeState:"clean", BuildDate:"2018-11-26T14:25:46Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

$ helm version
Client: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}

Which chart: stable/traefik
What happened: Unable to configure traefik to enable mutual TLS authentication
What you expected to happen: The chart should provide the ability to enable and configure mTLS
How to reproduce it (as minimally and precisely as possible):

Deploy traefik
Deploy a HTTPS web server that echoes received headers back to the client, such as echoheaders
Access the web server, providing a client certificate: curl --cert tls.crt --key tls.key https://myserver.example.com
Notice there is no "x-forwarded-tls-client-cert" header in the returned response body
Notice that it is possible to access the web server without using mTLS: curl https://myserver.example.com

Anything else we need to know:
No

Created PR #9829 for this

@tlmiller, @justurm PR #9829 didn't get any attention and, since we (in the project I am working on) have now decided to stay with nginx, I will not pursue this further.
If either of you would like to pick this up then be my guest and please don't hesitate to give me a shout if I can do anything to help.

@somcsel cheers,  I will see if I can get it over the line. Would love to have this to support some legacy apps.

New PR for this issue #13347


python version of your matlab code. process *.mrc

hi,
could you please release the python code that does the job of matalb code you provided here?
or release the final tif images produced by your matlab code? for full data.
the python code you provided with data in supp material does read/write only.
thanks

Hi, we didn't have python codes for data augmentation, we performed all the preprocessing, post-processing and evaluation on MATLAB. But in fact if you could read .mrc file via the provided python code in our Figshare page, the following part of data augmentation should be quite simple to write.

hi, thanks.
is it possible to provide directly the augmented data? it would be much more helpful. it avoids the need for matlab, or re-writing code. thanks
can you give a look into the matlab issue?
it runs fine on one folder f-actin. but it gives errors on other folders.
we appreciate your help and sharing the data.
thanks

The augmented training dataset contains over 40,000 files, which is extremely slow to upload from my workstation. We have tested that using the augmented data by the MATLAB codes could reproduce the results well in our paper.
I have answered the other problem in issue #10.

thanks.
we are trying to run the new matlab code. but it seems to have another issue over f-actin non-linear.
by the way, you can compress train and valid sets (40000 + validset files) into a single large file, and dump it somewhere. it may take some time to upload, but once done, the issues with data could be resolved. you can give it the necessary time to upload. we really appreciate your help.
thanks again

See issue #13


Added step to open custom options page

I found this step missing in newer versions of MSDN searches, so it's bothering me a bit, I'm hoping to add it here or somewhere else

Docs Build status updates of commit 1a3b1da:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/extensibility/creating-an-options-page.md
:warning:Warning

Details

docs/extensibility/creating-an-options-page.md

Line 289, Column 3: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/previous-versions/cc826083(v=vs.140)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 289, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/previous-versions/cc826083(v=vs.140)' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@SIGEER2222 - review of your changes is pending until you sign the CLA. Thanks!
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@leslierichardson95 @SIGEER2222
Thanks for the submission. I reviewed and made changes for correctness, validation issues, and typos, etc., but @lelierichardson would need to review to confirm that this is a good idea. I don't know why this section was previously removed. If there was a good reason, there might be a problem with adding it back.

Docs Build status updates of commit 21a101a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/extensibility/creating-an-options-page.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

It's always right to be careful
and thanks for the correction, I'm too careless hhh
------------------ 原始邮件 ------------------
发件人:                                                                                "MicrosoftDocs/visualstudio-docs"                                                                                @.>;
发送时间: 2022年8月18日(星期四) 凌晨0:47
@.>;
<EMAIL_ADDRESS>主题: Re: [MicrosoftDocs/visualstudio-docs] Added step to open custom options page (PR #8362)
@leslierichardson95 @SIGEER2222
Thanks for the submission. I reviewed and made changes for correctness, validation issues, and typos, etc., but @lelierichardson would need to review to confirm that this is a good idea. I don't know why this section was previously removed. If there was a good reason, there might be a problem with adding it back.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

#sign-off

@leslierichardson95 can you sign off please? thanks.

#sign-off

@SIGEER2222 - review of your changes is pending until you sign the CLA. It's all electronic and will take just a few minutes: https://cla.microsoft.com.
Thanks!

Learn Build status updates of commit 21a101a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/extensibility/creating-an-options-page.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@jborsecnik Sorry to reply so late, I have already submitted a signature request

#sign-off


dnm: debug chaos test only

What problem does this PR solve?

Issue Number: close #xxx
What is changed and how it works?
dnm
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/hold

Codecov Report

Merging #5860 (92c91e1) into master (6a451ea) will increase coverage by 0.0354%.
The diff coverage is 75.8620%.

Flag
Coverage Δ

cdc
62.0177% <66.6666%> (+0.0377%)
:arrow_up:

dm
51.9098% <ø> (-0.0204%)
:arrow_down:

engine
62.4431% <80.0000%> (+0.2585%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #5860        +/-   ##
================================================
+ Coverage   57.0764%   57.1118%   +0.0354%
================================================
  Files           682        682
  Lines         80224      80197        -27
================================================
+ Hits          45789      45802        +13
+ Misses        30146      30117        -29
+ Partials       4289       4278        -11

/run-verify


Request for support for xArm Series arm

Hello, many thanks for this solid work!
I would like to hear about your further plans that are related to the support for new robot arms, especially for xArm series constructed by uFactory.
Awaiting your reply and thanks in advance!

Hi @MARMOTatZJU
Thanks for your interest in RoboHive. Expanding the robot and task collections is in RoboHive's roadmap. However, we don't have any active thread for XArm as of now.
Adding a new arm should be quite straightforward. I'd be happy to help if you want to work together on a PR and get it merged. Let me know.

@vikashplus Sure, many thanks again for your reply!
I am considering conducting research on fine manipulation with xArm, and I am evaluating the feasibility of doing it with RoboHive (using it as our simulator)!
At this moment, I would like to know those quintessential steps that are required for the implementation of a new arm within RoboHive.

Add xarm sim to simhive. MuJoCo Menagerie has a xarm-model that we can use.
Create a config file to specify the hardware limit. Here is an example you can copy paste. You can also skip this step and RoboHive will create a default one for you
Register a new env (copy paste these lines) with paths to the new sim and the config(if created)

Add xarm sim to simhive. MuJoCo Menagerie has a xarm-model that we can use.
Create a config file to specify the hardware limit. Here is an example you can copy paste. You can also skip this step and RoboHive will create a default one for you
Register a new env (copy paste these lines) with paths to the new sim and the config(if created)

Thank you very much for this guidance! We will try to integrate xArm models into RoboHive and will consider create a PR after verification of the function.
I thereby close this issue. I will reopen it if we encounter any issue that worths further discussion.
Thank you again!


Correct the capitalization of Xcode in README

This pull request corrects the capitalization of Xcode :sweat_smile:
https://developer.apple.com/xcode/
Created with xcode-readme.

I thought I was going to escape this one - at least I spelt CocoaPods correctly!

:tada:


[PASS] Update StorageRewrite to Handle Allocate Depending on Loop Variable

Consider the following hypothetical example:
import tvm
N = tvm.var("N")
y = tvm.reduce_axis((0,N), 'y')
A = tvm.placeholder((2*N,2*N), "float32", "A")
C = tvm.compute((N, N), lambda i,j: A[2*i,j] + A[i+1,j], name='C')
s = tvm.create_schedule(C.op)
AA = s.cache_read(A, "local", [C])
s[AA].compute_at(s[C], s[C].op.axis[0])
print(tvm.lower(s, [A,C], simple_mode=True))

The resulting lowered code is as follows. See how the size of A.local depends on i which is a loop variable. I think the problem is that the two references to A, i.e., A[2*i,j] and A[i+1,j] have different functions of i (i.e., 2*i vs i) in their first dimension. That makes calculating buffer size more complicated, and TVM probably doesn't check for that. We should either handle it properly or emit an error message that such indexing is not supported.
// attr [A.local] storage_scope = "local"
allocate A.local[float32 * ((max((i*2), (i + 1)) - min((i*2), (i + 1))) + 1) * N]
produce C {
  for (i, 0, N) {
    produce A.local {
      for (ax0, 0, ((max((i*2), (i + 1)) - min((i*2), (i + 1))) + 1)) {
        for (ax1, 0, N) {
          if (likely((min((i*2), (i + 1)) < ((N*2) - ax0)))) {
            if (likely((ax1 < (N*2)))) {
              A.local[((ax0*N) + ax1)] = A[((((min((i*2), (i + 1)) + ax0)*N)*2) + ax1)]
            }
          }
        }
      }
    }
    for (j, 0, N) {
      C[((i*N) + j)] = (A.local[((max((i + -1), 0)*N) + j)] + A.local[(((1 - min(i, 1))*N) + j)])
    }
  }
}

In [8]: import tvm
   ...: N = tvm.var("N")
   ...: A = tvm.placeholder((N,N), "float32", "A")
   ...: C = tvm.compute((N, N), lambda i,j: A[i,j] + 1, name='C')
   ...: s = tvm.create_schedule(C.op)
   ...: AA = s.cache_read(A, "local", [C])
   ...: s[AA].compute_at(s[C], s[C].op.axis[0])
   ...: print(tvm.lower(s, [A,C], simple_mode=True))
   ...:
   ...:
// attr [A.local] storage_scope = "local"
allocate A.local[float32 * 1 * N]
produce C {
  for (i, 0, N) {
    produce A.local {
      for (ax1, 0, N) {
        A.local[ax1] = A[((i*N) + ax1)]
      }
    }
    for (j, 0, N) {
      C[((i*N) + j)] = (A.local[j] + 1.000000f)
    }
  }
}

Looks like even for allocate that doesn't depend on loop variable, the allocate stmt is in global scope. Is this expected? (@tqchen). Are there other IR passes that might possibly change the AttStmt's position after lower? I can take a look if it's not by design.

The "global scope" is a bit misleading, as it means global memory in GPU, or the memory that can actually be seen by the user. In CPU, everything is "global memory", although llvm may promote constant arrays into register.
The special memory "local, shared" is only enabled when we tag them to be so

Ah thanks for catching that! I meant the outer most scope in your first reply. Back to the example, do you still think the allocate statement is not in the right scope? Or it should be something similar to below?
produce C {
  for (i, 0, N) {
    // attr [A.local] storage_scope = "local"
    allocate A.local[float32 * 1 * N]

    produce A.local {
      for (ax1, 0, N) {
        A.local[ax1] = A[((i*N) + ax1)]
      }
    }
    for (j, 0, N) {
      C[((i*N) + j)] = (A.local[j] + 1.000000f)
    }
  }
}

Yes, in this case, the allocate should be in the for loop(as shown in your code). We probably want to send an warning as well, because usually non-fixed size temporal memory means dynamic memory allocation and we can only use DRAM(CPU memory) and global scope

OK. The reason the allocate was put into outer most scope is because the code only call PlanNewScope for ForType::Parallel:
https://github.com/dmlc/tvm/blob/7bafca4e550d451925abfa2f0bc897d080cbbf35/src/pass/storage_rewrite.cc#L754-L757
Is it still the case?
Also we enter/exit scope AFTER we call FindAlloc
https://github.com/dmlc/tvm/blob/7bafca4e550d451925abfa2f0bc897d080cbbf35/src/pass/storage_rewrite.cc#L736
It makes more sense to me that we move enter/exit scope code right after:
https://github.com/dmlc/tvm/blob/7bafca4e550d451925abfa2f0bc897d080cbbf35/src/pass/storage_rewrite.cc#L693
After making the fixes, it does generate the expected IR. While 6 python tests failed in test_pass_storage_rewrite.py.  I'll dig, but want to check that I'm heading to the right direction @tqchen

The original assumption was indeed that most memory allocation should appear in the outermost scope of the threading boundary. This is true and useful for most memory optimizations, as dynamic allocating and de-allocating is not good for performance and do not work for on-device memory.
Maybe in cases where there is dependent memory, we should either shoot an error, or relax the memory boundary to its maximum value then run the lifting, thoughts?

I see. I would vote to throw error to start, and if we see more occurrence of such pattern, we can consider handling it properly.

simple_mode skips the sanity check by IRUseDefAnalysis. So I added a test to show that we can detect such patterns.


..and custom javascript?

Hi, I'm trying to add to menu a custom button in order to open an simple alert box but it doesn't works, and in the console there is any  error.
I can see a test div (so the custom directive is working) but the javascript doesn't works... why?
I'm trying to add in the config/template/directive/menu.html

TEST

<button onclick="myFunction()">Try it</button>

but the button do nothing.
so I tried with ionic...

TEST

<button ng-click="showPopup()" class="button icon ion-edit">Show Popup</button>

still not workinkg, but I can see the TEST div, and the button
Any tips to solve this problem?

WPHC is just an Angular app, you need to learn how Angular works first. The code you shared cannot work. If you want a custom item within your menu you could create a new directive and overwrite the menu template to add the directive https://github.com/shprink/wordpress-hybrid-client/blob/develop/lib/templates/directive/menu.html
The docs explain how to overwrite templates https://github.com/shprink/wordpress-hybrid-client/blob/develop/CONFIGURATION.md#configtemplates
Good luck


Provide a more-detailed explanation of what EPSG codes are

Mention that an EPSG code is a way to reference an entry in a database of ways to translate numbers to points on the earth. Give examples like 'EPSG 27700 tells R that co-ordinates reference points on the earth in terms of how many metres they are north and east of a point in the Atlantic Ocean' vs [something else].

Tutorial 5 has this information.