Add a section to Minder docs about creating custom rule types

Please describe the enhancement
We know from our conversations with potential users that one key benefit / differentiator of Minder is its extensibility, and the flexibility for users to create and enforce their own rule types using Rego or jq. We should create a new section in Minder's docs to explain how this works. This will help both promote discoverability of this capability (docs are among our most-visited webpages) and guide people in how to do this.
Solution Proposal
We have two blog posts that explain this functionality that you can reference:

Writing Minder rule types using OPA and Rego
Extending Minder to create custom rule types

These docs could probably live under the "Using Minder" section and possibly also be referenced in the "Reference Types - Rules" section.
Describe alternatives you've considered
No response
Additional context
No response
Acceptance Criteria
No response

cc @jhrozek (we discussed this in Slack)


Add pinot-yammer to pinot-tools pom

Description
This PR adds pinot-yammer to pinot-tools pom in order to fix the following exception:
2021/03/11 20:28:36.874 ERROR [StartServiceManagerCommand] [main] Failed to start a Pinot [CONTROLLER] at 10.357 since launch
java.lang.IllegalStateException: Failed to initialize PinotMetricsFactory. Please check if any pinot-metrics related jar is actually added to the classpath.
	at com.google.common.base.Preconditions.checkState(Preconditions.java:444) ~[guava-20.0.jar:?]
	at org.apache.pinot.common.metrics.PinotMetricUtils.initializePinotMetricsFactory(PinotMetricUtils.java:85) ~[classes/:?]
	at org.apache.pinot.common.metrics.PinotMetricUtils.init(PinotMetricUtils.java:57) ~[classes/:?]
	at org.apache.pinot.controller.ControllerStarter.initControllerMetrics(ControllerStarter.java:483) ~[classes/:?]
	at org.apache.pinot.controller.ControllerStarter.start(ControllerStarter.java:279) ~[classes/:?]
	at org.apache.pinot.tools.service.PinotServiceManager.startController(PinotServiceManager.java:116) ~[classes/:?]
	at org.apache.pinot.tools.service.PinotServiceManager.startRole(PinotServiceManager.java:91) ~[classes/:?]
	at org.apache.pinot.tools.admin.command.StartServiceManagerCommand.lambda$startBootstrapServices$0(StartServiceManagerCommand.java:234) ~[classes/:?]
	at org.apache.pinot.tools.admin.command.StartServiceManagerCommand.startPinotService(StartServiceManagerCommand.java:286) [classes/:?]
	at org.apache.pinot.tools.admin.command.StartServiceManagerCommand.startBootstrapServices(StartServiceManagerCommand.java:233) [classes/:?]
	at org.apache.pinot.tools.admin.command.StartServiceManagerCommand.execute(StartServiceManagerCommand.java:183) [classes/:?]
	at org.apache.pinot.tools.admin.command.StartControllerCommand.execute(StartControllerCommand.java:130) [classes/:?]
	at org.apache.pinot.tools.admin.command.QuickstartRunner.startControllers(QuickstartRunner.java:124) [classes/:?]
	at org.apache.pinot.tools.admin.command.QuickstartRunner.startAll(QuickstartRunner.java:170) [classes/:?]
	at org.apache.pinot.tools.Quickstart.execute(Quickstart.java:169) [classes/:?]
	at org.apache.pinot.tools.admin.command.QuickStartCommand.execute(QuickStartCommand.java:78) [classes/:?]
	at org.apache.pinot.tools.admin.PinotAdministrator.execute(PinotAdministrator.java:164) [classes/:?]
	at org.apache.pinot.tools.admin.PinotAdministrator.main(PinotAdministrator.java:184) [classes/:?]

Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes
If you have tagged this as either backward-incompat or release-notes,
you MUST add text here that you would like to see appear in release notes of the
next release.
If you have a series of commits adding or enabling a feature, then
add this section only in final commit that marks the feature completed.
Refer to earlier release notes to see examples of text
Documentation
If you have introduced a new feature or configuration, please add it to the documentation as well.
See https://docs.pinot.apache.org/developers/developers-and-contributors/update-document

Codecov Report

Merging #6675 (1ea5270) into master (1beaab5) will decrease coverage by 23.01%.
The diff coverage is 42.85%.

@@             Coverage Diff             @@
##           master    #6675       +/-   ##
===========================================
- Coverage   66.44%   43.42%   -23.02%
===========================================
  Files        1075     1387      +312
  Lines       54773    67479    +12706
  Branches     8168     9793     +1625
===========================================
- Hits        36396    29306     -7090
- Misses      15700    35728    +20028
+ Partials     2677     2445      -232

Flag
Coverage Δ

integration
43.42% <42.85%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ot/broker/broker/AllowAllAccessControlFactory.java
100.00% <ø> (ø)

...t/broker/broker/BasicAuthAccessControlFactory.java
0.00% <0.00%> (ø)

.../helix/BrokerUserDefinedMessageHandlerFactory.java
52.83% <0.00%> (-13.84%)
:arrow_down:

...org/apache/pinot/broker/queryquota/HitCounter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...che/pinot/broker/queryquota/MaxHitRateTracker.java
0.00% <0.00%> (ø)

...ache/pinot/broker/queryquota/QueryQuotaEntity.java
0.00% <0.00%> (-50.00%)
:arrow_down:

...ker/routing/instanceselector/InstanceSelector.java
100.00% <ø> (ø)

...ceselector/StrictReplicaGroupInstanceSelector.java
0.00% <0.00%> (ø)

...roker/routing/segmentpruner/TimeSegmentPruner.java
0.00% <0.00%> (ø)

...roker/routing/segmentpruner/interval/Interval.java
0.00% <0.00%> (ø)

... and 1416 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 31bdc2b...1ea5270. Read the comment docs.


Show Video Stats in an optional overlay window

With MediaPortal you can show the following stats:
https://dl.team-mediaportal.com/2015/12/261851_upload_2015-12-21_0-9-8.png
We should also add something like this, because it is very usefull to see if everything is working as expected :-)

@Sascha-L the link to the image is broken

Fixed :D

One problem is that, in WPF, we don't have access to anywhere near that kind of detailed information about measured frame rate and the rest. Despite the fancy scheduling the WPF-MediaKit EVRPresenter tries to do (remember, our presenter is essentially identical to a Microsoft sample which was designed as a maximal example of various features), it's all wasted when we then use D3DImage.SetBackBuffer D3DRenderer.cs#L370, which puts us at the mercy of WPF (re-)scheduling.
I've spent a great deal of time and research investigating how to overhaul this, with some tentatively positive results so far. For example:

Since the current approach to sample 'pooling' and 'scheduling' is overkill given WPF (as I noted above), these parts of our EVRPresenter can be drastically scaled back. In fact, in one experiment I've integrated an EVRPresenter written in C#, abandoning the C++ code altogether.
Synchronizing with the Dispatcher thread to set the back buffer is a huge issue also, so I think one of my most important improvements so far has been to use lock-free concurrency (Interlocked atomic operations) to transfer the IDirect3DSurface9 across the native/managed boundary, which not only solves the tricky problems around having code negotiate that transition, but also eliminates the need to synchronize with the UI thread once you're on the managed side.
And finally, I believe the only way to true get a grip on WPF VSync control is through some kind of runtime adaptive measurement/feedback loop using the Direct3D API GetScanLine,  as described here. This part is still under investigation; I'm still not even sure it will be possible--even assuming you know exactly when the blanking interval starts--to reliably wake up WPF and such that you can set the back buffer in time.

But these are dramatic departures, and when my code is ready it may not resemble the WPF-MediaKit code base very much at all.

We do not need to display all of the informations as the mediaportal. We may start with the info we have: the renderer, the video and audio filter, the video size, etc. It would be handy to display some info when media are opening (e.g. "Media opening ... " is enough) and some error when media opening fails (just something like "Media opening error: {message}").

@glenn-slayden I do not understand this low level DirectX stuff much, but may this project https://github.com/Microsoft/WPFDXInterop help? (I just worry it's not maintained.)

Yes, I am aware of that project and it raises additional possibilities that I didn't happen to mention above, namely,  moving from Direct3D 9  to Direct3D 10 or 11. The approach detailed by the code on https://github.com/Microsoft/WPFDXInterop looks pretty straightforward and doesn't affect things about the more basic interop issues I'm looking at, so I figured it could wait as a lower priority.

Just to follow up my third bullet point above, VSync control is a lost cause on WPF--and in fact for DWM in general. Even if you could detect and wrangle the blanking interval, the user can trivially foil such delicate tuning efforts by adjusting the various VSync options in their (e.g.) Nvidia control panel. Furthermore, the whole effort might be vacuous if the user enables double- or triple-buffering in those aforementioned settings, and programatically detecting such scenarios in order to condition on them just complicates matters much further. Check here for more details.


Merge new changes in RetroPie fork

Thank you.  I've been needing to do this.  I'm trying it out right now.


Hosted VS Code Web

I would like to run tests against a VS Code instances that is hosted in my cloud (similar to vscode.dev).
So it would be great, if I could stop the testrunner from starting vscode web locally, but rather point it towards my (already running) instance.
I did not find a way to do this, but there seemed to be some preparation in the code for this use case.
The variable _vscodeServerPort in VSCodeServiceLauncher is checked before starting the server:
      /**
         * no need to do any work if we already started the server
         */
        if (this._vscodeServerPort || !cap[VSCODE_CAPABILITY_KEY]) {
            return
        }

but _vscodeServerPort is never set anywhere.
So my proposal would be:
-- a/src/launcher.ts
+++ b/src/launcher.ts
@@ -56,6 +56,7 @@ export default class VSCodeServiceLauncher extends ChromedriverServiceLauncher {
     ) {
         super(_options, _capabilities, config)
         this._cachePath = this._options.cachePath || DEFAULT_CACHE_PATH
+        this._vscodeServerPort = this._capabilities[VSCODE_CAPABILITY_KEY]?.serverOptions?.port
         this._mapCapabilities = () => {}
     }

Is this viable?

@jan872 thanks for raising the issue. I think it makes total sense to also allow that.
How is the extension installed on the hosted VS Code instance?
Depending on this answer we need to decide what to do with the extensionPath option.

I our case the extension is already installed - outside the actual test run. So I have no good idea what do with the extensionPath

I think in this case it doesn't really make sense to use the service because all the VS Code extension setup is not needed. You can just run a normal WebdriverIO session and only use the page objects. Would that work?

Yes, that was my first thought. But I was not able to instantiate the page objects without properly starting wdio-vscode-service.
So a hint on how to do this would be greatly appreciated. Sorry, I am just not very well versed in javascript.

I am not sure when I get the time to look into this, therefor we greatly appreciate any contributions that help resolve the bug. While we understand that active contributors have their own priorities, we kindly request your assistance if you rely on this bug being fixed. We encourage you to take a look at our contribution guidelines or join our friendly Discord development server, where you can ask any questions you may have. Thank you for your support, and cheers!

I was able to run wdio tests for hosted VS Code instance (our target environment is coder/code-server) following @christian-bromann advice to use just page object, without vscode service
Since I need to run suites for both desktop and browser (hosted instance), I have two conf files - in my case wdio.conf.electron.ts and wdio.conf.browser.ts
wdio.conf.electron.ts is almost fully generated by cli (as described in docs), with capabilities and services for VS Code
capabilities: [{
    browserName: 'vscode',
    browserVersion: '1.87.2',
    'wdio:vscodeOptions': {
      extensionPath: extensionDir,
    }
  }],
  services: ['vscode'],

wdio.conf.browser.ts doesn't contain config for VS Code:
capabilities: [{
    browserName: 'chrome'
  }],
services: [],

Then, we need to initialize Workbench for hosted VS Code. I've placed that code in util function, which I call in before block (I use mocha)
Few notes:

wdio-vscode-service has function getLocators, but unfortunately it is not exported as public API, so I have to copy-paste it
In actual code I pass version in runnerEnv arguments from configuration, but for simplicity it is hardcoded here
Path of locators should be changed according to actual project structure

import fs from 'node:fs/promises';
import * as path from 'path';
import { VSCodeLocatorMap, Workbench } from 'wdio-vscode-service';
import { fileURLToPath } from 'url';
import { expect } from '@wdio/globals';

export function prepareBrowserForVSCode(): void {
  browser.getWorkbench = async () => {
    // In actual code I pass version in runnerEnv arguments from configuration, but for simplicity it is hardcoded here
    const vsCodeVersion = '1.87.2';

    // __dirname only works in CommonJS modules
    const dirname = path.dirname(fileURLToPath(import.meta.url));

    // Copied with small changes from https://github.com/webdriverio-community/wdio-vscode-service/blob/99fd9a5c32ef540c7778dbca92ad1e7335207975/src/utils.ts#L12
    const locatorFiles = (await fs.readdir(path.join(dirname, '..', 'node_modules', 'wdio-vscode-service', 'dist', 'locators'), { encoding: 'utf-8' }))
      .filter((filename) => filename.endsWith('.js') && !filename.endsWith('.d.ts'))
      .map((filename) => filename.slice(0, -3));

    const [major, minor] = vsCodeVersion.split('.');
    const sanitizedVersion = `${major}.${minor}.0`;

    const locatorFile = locatorFiles.find((f, i) => (
      f === sanitizedVersion
        || (locatorFiles[i + 1] && locatorFiles[i + 1] > sanitizedVersion)
    )) || locatorFiles[locatorFiles.length - 1];

    const locators = await (import(`file://${path.join(dirname, '..', 'node_modules', 'wdio-vscode-service', 'dist', 'locators')}/${locatorFile}.js`) as Promise<VSCodeLocatorMap>);
    return new Workbench(locators);
  };

  browser.getVSCodeChannel = () => Promise.resolve('vscode');
  browser.getVSCodeVersion = () => Promise.resolve(vsCodeVersion);
}}

Then, in suite
describe('Hosted Instance Suite', () => {
  before(() => {
    prepareBrowserForVSCode();
  });

  it('should use workbench with locators', async () => {
    // navigate to VS Code in browser, in my case it is located on localhost:8081
    await browser.url('localhost:8081');

    // wait for workbench to load
    await browser.waitUntil(async () => {
        const w = await browser.getWorkbench();
        return w;
    });

    // We can use it now
    const workbench = await browser.getWorkbench();
    const editor = workbench.getEditorView();
  });
});

I assume that it is not the cleanest solution, but it worked for me. Please, let me know if there is a way to do it better, I would be happy to get rid of 'hacks' :)
Hope it helps someone!


Processing HTTP requests

I am using 3.0.1 version of vertx-when with Vertx 2.1.5. What is the proper way to use WhenHttpClient to

Set HTTP headers
Send POST body

Neither the documentation, nor the source code make it clear. I could not find a place in the code where headers or requestBody can be passed in or where they would be added to the HttpClientRequest object.

You can provide a setup handler which gives you full access to the HttpClientRequest to add headers and call request.end() providing a post body.
Here's one of the overloads:
https://github.com/englishtown/vertx-when/blob/3.0.1/src/main/java/com/englishtown/vertx/promises/WhenHttpClient.java#L37

Adrian,
Does vertx-when 3.0.1 support SSL connection with its DefaultWhenHttpClient? Is there a way to set SSL to true and pass in key store and trust store parameters?
Thank you,
Michael

Unfortunately it does not support SSL out of the box.  You can override createClient() on DefaultWhenHttpClient and fix it in your code.
    protected <T> HttpClient createClient(URI url, final Deferred<T> d) {

        if (url == null) throw new IllegalArgumentException("url is null");
        if (!url.isAbsolute()) throw new IllegalArgumentException("url must be absolute");

        int port = (url.getPort() > 0) ? url.getPort() : 80;

        return vertx.createHttpClient()
                .setHost(url.getHost())
                .setPort(port)
                .setConnectTimeout(CONNECT_TIMEOUT)
                .exceptionHandler(t -> d.reject(t));
    }

The when-vertx 4.0+ version which works with vert.x 3 should support SSL without any issues.  If you want to send a pull request, I would be happy to create a 3.0.2 build with SSL support.


Master

mysql

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

This was an error, while making changes to my git account.

Closing PR. Marked as error.


feat: introduces builder images

Short description of what this resolves:
Leverages docker builder images/ multistage build to build controller and test service.
Changes proposed in this pull request:

rewritten Dockerfiles
.dockerignore
adjustments in Makefile

Fixes #588

/test openshift

@bartoszmajsak: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e-tests-openshift
ebab5b287a142842f26fa8c82d61c4b721e28906
link
/test openshift

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

/retest

@bartoszmajsak: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e-tests-openshift
ebab5b287a142842f26fa8c82d61c4b721e28906
link
/test openshift

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

/retest

@bartoszmajsak: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e-tests-openshift
ebab5b287a142842f26fa8c82d61c4b721e28906
link
/test openshift

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

/retest

/retest

@bartoszmajsak: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e-tests-openshift
ebab5b287a142842f26fa8c82d61c4b721e28906
link
/test openshift

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


Better error messages on illegal dynamic-shape args to public functions

At the moment, this a.mlir
func @foo(%a : tensor<?x?xi32>) -> tensor<?x?xi32> {
  return %a : tensor<?x?xi32>
}

func @bar(%a : tensor<?x?xi32>) -> tensor<?x?xi32> {
    %b = call @foo(%a) : (tensor<?x?xi32>) -> tensor<?x?xi32>
    return %b : tensor<?x?xi32>
}

causes iree-translate to generate this error:
error: 'std.call' op operand type mismatch: expected operand type '!hal.buffer_view', but provided 'tensor<?x?xi32>' for operand number 0
    %b = call @foo(%a) : (tensor<?x?xi32>) -> tensor<?x?xi32>

@benvanik pointed out that the issue was that foo should be private, and that indeed does remove the error, so it would be nice to generate a better error message here, and perhaps also document, if not already documented, what is legal/illegal inputs to iree-translate. In particular, what rule was violated by having foo public here, and how are the dynamic shapes here relevant.

Duplicate of Issue #8391.


websocket connction question

when i use gateway to forward websocket connection . a problem was happend.
eg:
2020-12-16 14:15:46.232 ERROR 76168 --- [nio-9442-exec-3] a.w.r.e.AbstractErrorWebExceptionHandler : [5898d92f]  500 Server Error for HTTP GET "/odv/ai_live/1_dbf7d70117f340a6a41b23dc346d8ef1"
java.lang.ClassCastException: org.apache.catalina.connector.ResponseFacade cannot be cast to reactor.netty.http.server.HttpServerResponse
at org.springframework.web.reactive.socket.server.upgrade.ReactorNettyRequestUpgradeStrategy.getNativeResponse(ReactorNettyRequestUpgradeStrategy.java:182) ~[spring-webflux-5.2.8.RELEASE.jar:5.2.8.RELEASE]
Suppressed: reactor.core.publisher.FluxOnAssembly$OnAssemblyException:
Error has been observed at the following site(s):
|_ checkpoint ⇢ org.springframework.web.cors.reactive.CorsWebFilter [DefaultWebFilterChain]
|_ checkpoint ⇢ org.springframework.cloud.gateway.filter.WeightCalculatorWebFilter [DefaultWebFilterChain]
|_ checkpoint ⇢ HTTP GET "/odv/ai_live/1_dbf7d70117f340a6a41b23dc346d8ef1" [ExceptionHandlingWebHandler]
Stack trace:
at org.springframework.web.reactive.socket.server.upgrade.ReactorNettyRequestUpgradeStrategy.getNativeResponse(ReactorNettyRequestUpgradeStrategy.java:182) ~[spring-webflux-5.2.8.RELEASE.jar:5.2.8.RELEASE]
at org.springframework.web.reactive.socket.server.upgrade.ReactorNettyRequestUpgradeStrategy.upgrade(ReactorNettyRequestUpgradeStrategy.java:162) ~[spring-webflux-5.2.8.RELEASE.jar:5.2.8.RELEASE]
at org.springframework.web.reactive.socket.server.support.HandshakeWebSocketService.lambda$handleRequest$1(HandshakeWebSocketService.java:235) ~[spring-webflux-5.2.8.RELEASE.jar:5.2.8.RELEASE]
at reactor.core.publisher.FluxFlatMap.trySubscribeScalarMap(FluxFlatMap.java:151) ~[reactor-core-3.3.9.RELEASE.jar:3.3.9.RELEASE]
at reactor.core.publisher.MonoFlatMap.subscribeOrReturn(MonoFlatMap.java:53) ~[reactor-core-3.3.9.RELEASE.jar:3.3.9.RELEASE]
at reactor.core.publisher.InternalMonoOperator.subscribe(InternalMonoOperator.java:57) ~[reactor-core-3.3.9.RELEASE.jar:3.3.9.RELEASE]
at reactor.core.publisher.MonoDefer.subscribe(MonoDefer.java:52) ~[reactor-core-3.3.9.RELEASE.jar:3.3.9.RELEASE]
at reactor.core.publisher.MonoDefer.subscribe(MonoDefer.java:52) ~[reactor-core-3.3.9.RELEASE.jar:3.3.9.RELEASE]
at reactor.core.publisher.MonoDefer.subscribe(MonoDefer.java:52) ~[reactor-core-3.3.9.RELEASE.jar:3.3.9.RELEASE]
at reactor.core.publisher.MonoDefer.subscribe(MonoDefer.java:52) ~[reactor-core-3.3.9.RELEASE.jar:3.3.9.RELEASE]
at reactor.core.publisher.MonoDefer.subscribe(MonoDefer.java:52) ~[reactor-core-3.3.9.RELEASE.jar:3.3.9.RELEASE]
these my routeconfig
 routes:
    - id: odv-ws
      uri: ws://localhost:8501
      predicates:
        - Path=/odv/ai_live/**
      filters:
        - StripPrefix=1

localhost:8501 is my websocketserver.。the websocketserver endpoint uses @ServerEndpoint.
connect eg: javascrtipt
var ws = new WebSocket("ws://localhost:9442/odv/ai_live/1_dbf7d70117f340a6a41b23dc346d8ef1");
localhost:9442 is my gateway.
i saw another issue said that "Gateway doesn't support tomcat. There is already an open issue for that".
is it right?

I encountered it and did not solve it. I also configured it
`2020-12-17 20:21:41.049  WARN 288718 --- [or-http-epoll-6] r.netty.http.client.HttpClientConnect    : [id: 0x27daedae, L:/<IP_ADDRESS>:52784 - R:<IP_ADDRESS>/<IP_ADDRESS>:9444] The connection observed an error
io.netty.handler.codec.http.websocketx.WebSocketHandshakeException: Invalid handshake response getStatus: 404 `

I encountered it and did not solve it. I also configured it
`2020-12-17 20:21:41.049 WARN 288718 --- [or-http-epoll-6] r.netty.http.client.HttpClientConnect : [id: 0x27daedae, L:/<IP_ADDRESS>:52784 - R:<IP_ADDRESS>/<IP_ADDRESS>:9444] The connection observed an error
io.netty.handler.codec.http.websocketx.WebSocketHandshakeException: Invalid handshake response getStatus: 404 `

我和你不一样 我是握手500，后来发现是因为不支持tomcat启动

I encountered it and did not solve it. I also configured it
2020-12-17 20:21:41.049 WARN 288718 --- [or-http-epoll-6] r.netty.http.client.HttpClientConnect : [id: 0x27daedae, L:/<IP_ADDRESS>:52784 - R:<IP_ADDRESS>/<IP_ADDRESS>:9444] The connection observed an error io.netty.handler.codec.http.websocketx.WebSocketHandshakeException: Invalid handshake response getStatus: 404

我和你不一样 我是握手500，后来发现是因为不支持tomcat启动

Tomcat要8以上的，那你的是怎么写
我看看吗

i saw another issue said that "Gateway doesn't support tomcat. There is already an open issue for that".
is it right?

Yes, here #1544  , Maybe soon for Tomcat and Undertow, Thanks @spencergibb  for all of the work.

@Sh1nno 同样问题，问题解决了吗？

同样问题，解决了吗？ @Sh1nno


chore: Pass custom formats to strategy factories

Codecov Report

Merging #1421 (f203874) into master (16171c0) will decrease coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1421      +/-   ##
==========================================
- Coverage   99.16%   99.05%   -0.11%
==========================================
  Files          84       84
  Lines        6487     6487
  Branches     1138     1138
==========================================
- Hits         6433     6426       -7
- Misses         27       31       +4
- Partials       27       30       +3

Flag
Coverage Δ

unittests
99.05% <100.00%> (-0.08%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/schemathesis/specs/openapi/_hypothesis.py
100.00% <100.00%> (ø)

src/schemathesis/runner/impl/threadpool.py
98.23% <0.00%> (-1.77%)
:arrow_down:

src/schemathesis/extra/pytest_plugin.py
98.24% <0.00%> (-1.76%)
:arrow_down:

src/schemathesis/cli/callbacks.py
98.33% <0.00%> (-1.67%)
:arrow_down:

src/schemathesis/specs/openapi/loaders.py
98.31% <0.00%> (-0.85%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 16171c0...f203874. Read the comment docs.


C++ API Deprecation

Hi,
In Release Note for version 1.45.0 we had this statement:

Aeron currently has two C++ APIs, a pure C++ version and a wrapper over the C API. This release we are no longer adding new features to the C++ API.

Some questions about it:

Why?
Only pure C++ version API will be deprecated or both of them?
What about later releases?

Reduce the maintenance cost.  We currently have three implementations of the Aeron client.  When we need to make client changes they need to happen in all three places independently.  This will reduce it to two.  Also adding support for other languages like Python and Rust is easiest as a layer over C rather than an independent implementation or wrapping C++.

Only the pure C++ will be deprecated in favour of the C++ wrapper over the C API.

I'm not sure what you mean about later releases.  Eventually we will remove the pure C++ API.

Thank you for the reply!


Removed excessive message print

Short description
Bug summary: when creating a user whose name already exists, the system displays the message "Pending account activation", which is incorrect.
Proposed changes
Removed excessive message print
Side effects
Didn't find any.
Resolved issues
Fixes: #1703

Pull Request Review Guidelines

Congratulation for the first PR and thank you for your contribution 😄
The error message is gone and it's nice that the change is applied also in the global user form.
Can you add an entry in CHANGELOG.md under UNRELEASED?

@seluianova
Thank you! It's ready to be merged 😃


[UX Map] Setting Point not by coordinates but by text query

For UX Map it would be helpful if the points for map center and marker would not only be available by Long-Lat-Combinations but by text query. Google maps e.g. has this in their url request:
https://www.google.com/maps/embed/v1/place?key={{ GoogleMapsApiKey }}&q={{ GoogleMapsLocation }}&zoom=15
Where GoogleMapsLocation could be "Times Square, Manhattan, New York" for example. Then Google would center the Map around that point instead of giving coordinates.

Hi, I understand it can be useful, but that means introducing Geocoding to UX Map, and I dont think we really want that. UX Map is designed to display a map on your website.
But I kind of agree that having something like Point::fromAddress('Times Square, Manhattan, New York') (or equivalent) would be sexy, but:

imagine you use Leaflet to render your map, should we geocode through Leaflet/OpenStreetMap (if such a service exists) or through Google Maps API?
Point only take two (required) parameters $latitude and $longitude, it means that Point::fromAddress() must rely on some hacks to properly initialize the object, maybe it should also rely on some static object (to call a geocoding service)
we also want to store geocoding results in cache, calling the geocoding service each time you render the map is a no go

In conclusion, I believe such logic must stay external to UX Map. You can easily implement this feature by using by yourself a geocoding client (with cache), and create your Point instance with new Point(...$geocodedAddress['_latlng']).

I'm closing, feel free to re-open if needed!


Add Labels to Map Markers

Map markers should have a job title and distance.

Resources

https://tomchentw.github.io/react-google-maps/#markerwithlabel
https://tomchentw.github.io/react-google-maps/#marker
https://codesandbox.io/s/2xyw6n4o9y

Feature implemented and live


Native scrollbars are visible when using multiple nanoScrollers

When I'm using multiple nanoScrollers on a page, only for the first one the OS native scrollbars are hidden. Other element have a native scrollbar and a nanoScroller scrollbar.
Specs: OSX Lion + FF 8.0.1, Safari 5.1.2, Chrome 16.0.9

Hi Peter, can you setup a jsfiddle example of this instance? Thanks!

Hi James,
I couldn't get it working on jsfiddle, so I made a small demopage: http://petervangrieken.nl/demo/

@petervangrieken , try this code instead.
$("#first .nano").nanoScroller();
$("#second .nano").nanoScroller();
$("#third .nano").nanoScroller();

The reason is that the plugin only applies the custom scrollbar mechanism to specific DOM elements.

That works like a charm. Thanks!

Cheers, Peter!

Another solution would be:
$('.nano').each( function() {
  $(this).nanoscroller();
});

when i try the first solution everything stops working and i get this error:
TypeError: 'undefined' is not an object (evaluating 'pane.addEventListener')


Update go-shlex package path

Since Googole Code closed, go-shlex package path move from code.google.com/p/go-shlex to github.com/google/shlex.

Sorry, I don't noticed your PR. This is already fixed in 818c758.
Thanks.


How to handle uploads with ufront?

Seems the server just crashes when it receives some file upload requests

Out of curiosity - which target and server are you using?
PHP / Neko.
Nekotools / mod_neko / mod_tora.
Apache / Nginx etc.
nekotools server does not handle multipart uploads correctly, so if that is what you are using I guess that might be the suspect.

I am using ufront s for development purpose which should be nekotools server.
So, if I am using a regular production server like apache or nginx, what is the code to handle uploads with ufront? I opened the thread because I just dunno where to start with.
But just now I searched a bit the source code and found that there is a ufront.web.upload package, I will look into that, thanks!

I'd recommend using this middleware:
http://api.ufront.net/ufront/web/upload/TmpFileUploadMiddleware.html
If it is a multipart request, it will save any files to a temporary file, and you can move it to where you want from there.  You can access the uploaded files from context.request.files.
I've only used file uploads in a couple of places, so let me know if you run into any road blocks.
I'll also try find out if there's a way we can give a better error message on nekotools server... because it's pretty frustrating that it just crashes.

In fact, that is included by default in UfrontConfiguration, so there's a good chance you have this already.
(p.s. I have no idea what approach will be best on the client side... If you have any ideas let me know)

Ok I will try to work with the uploadmiddleware and report any issue I found.
For client side, I am not very sure. Because I'm still quite new to client-side web apps and my mindset is still in the "client issue simple http request + server do all the stuff" era. Anyway I will post some issues if I got any new ideas.
But talking about client side, my main issue is that I wanted to make use of the controller but not to trigger a page/url change (by pushstate). Is that possible? For example, I have a "save" button which saves whatever data to server. Currently I add rel='pushstate' to the button so that pushstate will intercept the click and direct to the corresponding controller which in turns calls some asyncApi. But in the same process the url is changed, even I return an EmptyResult so that the page doesn't change. So, for client-side, what I am now mainly looking for is a proper way to direct html element interactions to the UFcontrollers and provide feedback to the html elements once the controller completes execution.

Well I think that should be discussed in another thread


Add torchmetrics' own implementation of Rouge score metrics

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure to update the docs?
[x] Did you write any new necessary tests?

What does this PR do?
This PR is a part of the endeavour of #433 to implement own metrics and minimize third-party dependencies. It adds the own implementation of Rouge score metrics.
In this PR, the rouge-score package (https://pypi.org/project/rouge-score/) is replaced with torchmetrics' own implementation. The behaviour of this new implementation should be equivalent tothe aforementioned package.
Furthermore, this PR strips the use of the decimal_places argument as this seems not to be used by any other metric. TODO: Add some depreciation warning (I will need a help here as I haven't found any own Warning class as it is the case in the pytorch-lightning.
Future TODOs for another PR: Replace nltk package (I will need to dig a bit deeper to find out if there's a way)
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Reviewers: @Borda @SkafteNicki
Did you have fun?
Make sure you had fun coding 🙃

Hello @stancld! Thanks for opening this PR.

In the file torchmetrics/functional/text/rouge.py:

Line 82:1: W293 blank line contains whitespace

Do see the Hitchhiker's guide to code style


Add deprecation note in README.md

People are opening PRs here to this day, I was also confused at the beginning.
I think it probably make sense putting a prominent note about the new maintainer announced in #566
Thanks

Cc @psibi


[WIP] fix: update min gas price of 0.38uusd

multiplied 2.54x to all fixedGas (0.25 -> 0.38)

Codecov Report

Merging #163 (ff17ae2) into master (9fb135d) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head ff17ae2 differs from pull request most recent head 518b626. Consider uploading reports for the commit 518b626 to get more accurate results

@@          Coverage Diff           @@
##           master    #163   +/-   ##
======================================
  Coverage    3.81%   3.81%
======================================
  Files         526     526
  Lines        9592    9592
  Branches     2746    2746
======================================
  Hits          366     366
  Misses       9226    9226

Impacted Files
Coverage Δ

<EMAIL_ADDRESS>100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9fb135d...518b626. Read the comment docs.


enable128-bit-traces has no effect

Setting opentracing.jaeger.enable128-bit-traces to true has no effects while I find following codes of JaegerAutoConfiguration
https://github.com/opentracing-contrib/java-spring-jaeger/blob/a44aedec17cd80947dd931edd4418407e7a88e6e/opentracing-spring-jaeger-starter/src/main/java/io/opentracing/contrib/java/spring/jaeger/starter/JaegerAutoConfiguration.java#L63-L79
It seems not set the flags of enable128-bit-traces and enable-b3-propagation.
Affected version: 3.1.1

It seems to be done here
https://github.com/opentracing-contrib/java-spring-jaeger/blob/master/opentracing-spring-jaeger-starter/src/main/java/io/opentracing/contrib/java/spring/jaeger/starter/JaegerAutoConfiguration.java#L207-L234

Thats is correct

hmmm it seems like the property name might missmatch
In the config code it's @ConditionalOnProperty(value = "opentracing.jaeger.enable-128-bit-traces")
But in the config file, VS Code doesn't suggest me the same name

So maybe the mapping with the config file and the property name doesn't work ? I dont know how to validate that mapping in Spring.

Ah, good catch!
This probably changed with Spring Boot 2.x.
Would you like to submit a PR to fix @ConditionalOnProperty?

Should we use enable128-bit-traces and change @ConditionalOnProperty and other references, or use @Value("${opentracing.jaeger.enable-128-bit-traces}") on the property ?

Let's just make the change to @ConditionalOnProperty(value = "opentracing.jaeger.enable128-bit-traces").
IntelliJ is also suggesting that it's the right match

Actually this doesn't seems to be the problem. The test are passing with "opentracing.jaeger.enable128-bit-traces=true" or "opentracing.jaeger.enable-128-bit-traces=true". And according to SpringBoot 2.0, both should be supported https://spring.io/blog/2018/03/28/property-binding-in-spring-boot-2-0. I will test it in our current deployment see if it works or not.

Well, it works fine, so I think we can close this.
Configuration: enable128-bit-traces: true
128-bit Trace: https://localhost:8080/trace/a4f06630032838b3a941b5d7350f2978
Regular Trace: https://localhost:8080/trace/c4a5944822e16513

I tried this as well and it worked for me too.
I'll close this, but feel free to reopen if you have more information to provide @JasonMing


improve adaptivity to im2markup

Serveral changes to make the onmt code generalizable to the im2markup project (and beyond). All changes shall not affect the existing application. (yet to be tested)

Let BiEncoder return gradInput during backward(). (onmt/modules/BiEncoder.lua)
Let Batch support double features. Currently the source features are word ids (2D integer tensor of size (sequence_length, batch_size)), in this patch we add interface setSourceInput(sourceInput) to support general double features, e.g. 3D double tensor of size (sequence_length, batch_size, feature_dim). Besides, we add interface setTargetInput, setTargetOutput to directly set targets to provide more flexibility. (onmt/data/batch.lua).
Add default zero state gradients for BiEncoder backward(), default zero states for Decoder forward(). This makes it easier for models that do not initialize the decoder states with encoder final states. (onmt/modules/BiEncoder.lua, onmt/modules/Decoder.lua).

Seems like recent changes on master introduced a conflict here. As it was just renaming, it should be easy to fix.

@guillaumekln When you mentioned "when you list arguments on multiple lines, they should ideally be aligned.", where did you refer?

For future reference, this change should not have been merged. Does not match comment style etc. Let's be strict about this.

Sure. Should I use another pr to fix the style?

No it's okay, I just want to be more vigilant about not letting in PRs that
violate the standard.
On Sun, Dec 18, 2016 at 12:41 PM, Yuntian<EMAIL_ADDRESS>wrote:

Sure. Should I use another pr to fix the style?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_OpenNMT_OpenNMT_pull_7-23issuecomment-2D267834944&d=CwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=wnHFZ7D4m-9MRwk-CWlvCGbWEiQX_AvUO2LuMy4Vj7c&m=daeCk5HIXYc3nBygDuCzYvpTZGkpxcyZ2I8VOHUXl_A&s=8S4cXIrqrH7lC9Jlh8SdQ-vhaxJ6aN2OLQaecjaXCCc&e=,
or mute the thread
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AACMKrbSllKLDb1Y8YZjvTxPVUIPkaVtks5rJXA0gaJpZM4LNuzh&d=CwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=wnHFZ7D4m-9MRwk-CWlvCGbWEiQX_AvUO2LuMy4Vj7c&m=daeCk5HIXYc3nBygDuCzYvpTZGkpxcyZ2I8VOHUXl_A&s=MoHxDeqedvWHAWHwPaNnFgeqNesrDb0p7a3pe22HhTE&e=
.


Remove access to private variable from Context

VM\Context was attempting to set private property VM\Variable->bool
when dealing with a boolean literal. This updates the value to be
injected via the constructor.

I wouldn't take it as a constructor parameter, but instead use the ->bool($value) method, since it needs to properly set the type flag as well.

The type was already set in these two cases, because it's immediately after instantiating a new Variable of type bool, but I updated the code to use the bool method instead so that the constructor doesn't give booleans special treatment.
(I don't mind if you either close or merge this, because the constructor commit was already merged in #19 as a pre-req for fixing that bug, so at this point it's just a stylistic choice..)


[GSOC] Create a Model Card Using New Template

Though somewhat of a mirror to the Data Cards starter task, this starter task focuses instead on Model Cards.
We're currently in the process of publishing a new template for Model Cards. The new template should be ready within a day or so. But in the interim, you could use existing examples as your guiding template.
It is of note that:

You're welcome to create a Model Card from either a machine learning model you're currently working on or from a pre-trained model that isn't already accompanied by a Model Card.
If you're producing a Model Card based off a pre-trained model that is publicly accessible, then be sure to gather any existing documentation associated with that model, which will assist in completing a Model Card.
Check out Google Cloud's Model Cards page to learn more about Model Cards. This page includes two designed examples of Cloud Vision APIs (Face Detection and Object Detection) that help explore the possibilities with Model Cards.
Read our original paper to develop a conceptual understanding of Model Cards.
There is also a repo filled with actual Model Cards that you could draw upon for inspiration.

This starter task is open for multiple submissions by multiple participants.
Timeline and submission logistics:

While filling out a Model Card template could take up to a few hours, keep in mind that needing to conduct additional analysis or investigations could substantially increase the time taken to complete a template.
If you are using Colab to inspect the trained model or conduct analysis, then please include them in your submission.
When you are done, please commit your completed template to the Benchmark Detectives directory as a Markdown or PDF file.
Include a short note on your experience in your proposal for Benchmark Detectives.

As promised, we now have the new template ready for you all to use. I'll include these details to the main comment.
You can find in the /templates directory the following:

The new Model Cards template  in .docx format
A filled out example using the aforementioned Model Card template in .pdf format


2 - Optional extra credit

Optional extra credit/challenge exercise (1 point): Impute possible judge demographic correlates of sentencing
The previous exercises showed large differences in sentences between judges/differences in disparities. You become interested in how the judge's own demographic attributes are correlated with sentencing. Going back to the judge's name (SENTENCE JUDGE), parse their first name and try to probabilistically infer his or her gender. Then, investigate whether disparities differ between "likely female" and "likely male" judges.
Note on ethics of probabilistic inference of attributes based on name: Using names to infer demographic characteristics has become increasingly popular with the rise of "digital trace data" that often lacks explicit demographic fields (e.g., tweets just have usernames and profiles; academic citation networks just have author names; voter registration records sometimes just have names and not race/ethnicity). But there are many valid ethical critiques of this practice. In the case of gender, a person's assigned name at birth clearly does not always map onto their self-identified gender, both due to gender fluidity and names like "Morgan." A couple critiques I link to are:
This blog post
Urban Institute ethical risks of race/ethnicity imputation - applies to gender

yep fine to use an external library; you may want to look into some of the results to see how well it's working based on your manual reading of a few cases classified as each type; depending on the library, you can also augment the dictionary they use

Should we focus on narcs offenses for this problem too? And the same characteristics (age, gender and race) to match on?

Should we focus on narcs offenses for this problem too? And the same characteristics (age, gender and race) to match on?

totally up to you on that! main focus of this is different ways to impute judge demographics based on name and cleaning/parsing of names to do so


Not possible to upload as flat files, ignoring source directory structure

Which version of the AzCopy was used?
AzCopy 10.17.0
Which platform are you using? (ex: Windows, Mac, Linux)
Linux
What command did you run?
azcopy copy \
    'LOCALDIR' \
    'BLOBSTORAGE' \
    --list-of-files '/foo/bar/allthefolders.txt' \
    --recursive=true

What problem was encountered?
azcopy always re-creates the source folder structure on the target blob account. All I want is to have the listed files uploaded as plain files, without folder structure.
In my special case, I'd like to give azcopy a file list, containing only folders. Then azcopy should upload files contained in these folders.
How can we reproduce the problem in the simplest way?
Give azcopy a file list, containing files to upload within nested folders.
Have you found a mitigation/solution?
Not for now.

hey @devployment,
Have you managed to find an workaround for this?


Update docs-client dependencies, including migrating across breaking changes.

In preparation for #1949 I've started by upgrading all dependencies, including some fairly major upgrades.
Major updates are material-ui - https://material-ui.com/guides/migration-v3/
I also made demo mode controllable with an environment variable, which is convenient since I can just set it as an IntelliJ run configuration without modifying code.
And finally fixed missing key warnings and fixed hot reload.

I see some aesthetic issues though:

Too much space between HTTP method name and RPC method name
HTTP method name label has inconsistent widths.

Search bar font could be smaller?

Thanks - guess deprecations aren't only code...
I wasn't able to reproduce inconsistent width issue on Chrome Windows. Which browser did you see it? Also, maybe it won't reproduce anymore anyways since I completely recoded it using Grid since the behavior of inset seemed to change drastically.
I also restored some of the visuals of the goto-select, but am having trouble getting it to look good / same as before. The styling is pretty custom, and maybe a bad interaction with new default behavior of material-ui. @ikhoon do you think you could help take a look at what could be causing it?

Which browser did you see it?
Firefox - Just can't live without Tree Style Tab.

do you think you could help take a look at what could be causing it?

@anuraaga Sure! I will investigate the issue soon. :->

@trustin Thanks - started firefox for the first time in a while, don't see the issue with the latest code so hopefully it was fixed.
@ikhoon Thanks a lot!

It looks much better now, but search bar still looks weird. I think @ikhoon will help us on it. :stuck_out_tongue_winking_eye:

As you see, the text is not vertically aligned correctly and we could use a monospace font.

@anuraaga I made another pull request based on your working branch. Sorry for the hard cording. :-(
https://github.com/anuraaga/armeria/pull/6

Thanks for the PR :) For the future, if you wanted to, feel free to push directly into the branch - I always tick the checkbox that maintainers can commit into my forks.

Thanks, all. :-)


Update portuguese translation

Update portuguese translation

Thanks! Do you have another Portuguese speaker that could double check these? Just to sanity check with a second set of eyes (spelling, grammar, etc)

You are welcome. Sorry, but I don't have another portuguese speaker for check these. Thanks!

@fortes could you double check these?

@feliperuhland could you check these?

A note about the delete_* translations, otherwise :+1:

Good observation @fortes . I did the suggested changes. You can see here: http://goo.gl/tMJy8L

Hey @gustavobite @fortes.
Personally I don't like the word apagar, I find most interesting remover or excluir.
But, @fortes has a good argument, independent of the word, it must be the same in all posts.

@fortes  @feliperuhland
The word varies by context, in my opinion.
For example, the right for me in this sentence 'Deleting failed' is 'Exclusão falhou'.
Thank your opinions and help.

Like I said, it's a personal opinion.
Anyway, your translation is fine and should be merged.

Remover works for me as well, but if others feel Excluir is better, then so be it.
@gustavobite I would just make sure the word is consistent, your last checkin still has excluir in delete_successful

Let's go with consistency, both in the translations and relative to other languages. If remover makes more sense, let's go with that. And also update this line to match

Also thanks to everyone for the discussion, glad to have good translations :)

I believe this is ready to merge, right?

@fadils Despite any disagreement, the translation is good and is ready to merge.

Thanks everyone!


qdrant add should condition filter

Description:
Add support for conditions such as should in the _qdrant_filter_from_dict query method of the qdrant.py file class.
Issue:
When I was using _qdrant_filter_from_dict, I found that I couldn't perform multi condition queries through should because it is currently restricted to the must mode and I hope it can be customized as should or must-not.
Dependencies:
NA

This is a private method so it's not exposed as part of the public API.
To expose filter control we either need to make this public or else create another public API entry point.
With a private method adding an argument is no different from overriding the entire method

Thank you for your answer. If I want to query through 'should', what public method would be more appropriate to call?

Looks like you need to provide qdrant filters directly:
https://github.com/langchain-ai/langchain/blob/main/libs/community/langchain_community/vectorstores/qdrant.py#L596-L596


screen got stuck after some random arrows click

URL: 'https://swiffyslider.com/.'
Screenshots
https://github.com/dynamicweb/swiffy-slider/assets/60537610/5363ead8-3f1f-42f9-8e60-151f7bbadc8b
Smartphone (please complete the following information):

Device: [Oppo reno 10]
OS: [ColorOS 14.0, Android 14]
Browser [chrome]
Version [Chrome 126.0.6478.122]

@nicped I have opened a PR. Could you please review it?
PR Link: https://github.com/dynamicweb/swiffy-slider/pull/97

@nicped any update on this?


LocalDB persist in docker and change ui port

What is Happening / What is expected

All dns config (set in ui) reset after docker restart
Add persistent volume to save dns config

Specs

OS: Ubuntu 22.04 server
Docker Version: latest
DPS Version: 1.16.0

Read all docs, didn't find how to save dns config in docker or docker compose (latest version) before/after restart/stop container.
Is it possible to add environment variable to change web ui port?
Thanks

Hey, sorry for the late answer:

You can create a persistent JSON config by voluming container path /app/conf
You can change the webserver port on the JSON config or at the command args by using --web-server-port 80 for example.

services:
  dps:
    image: defreitas/dns-proxy-server:3.19.1-snapshot
    command: --web-server-port 80
    volumes:
      - dps_vol:/app/conf
    ports:
      - 8081:80

volumes:
  dps_vol:

Please reopen if you have any doubts


Set shardId on blob requests

Resolve the shardId upfront of creating any blob request instead
of using the deprecated resolveRequest() method which resolves
the shardId during transport execution.
This is needed as recent ES changes removes usages of
resolveRequest() inside the replication logic.
See https://github.com/elastic/elasticsearch/commit/3ad8aa6d465.

Will it still re-resolve the shardId if the shard is moving around?

I guess you mean resolving the ShardRouting.
It is still resolved inside the  Reroute phase, see https://github.com/crate/crate/blob/master/server/src/main/java/org/elasticsearch/action/support/replication/TransportReplicationAction.java#L724.


Bank explorer redesign

A redesign of the Fmod banks explorer, that loosely follows godot's native UI design.

There's a lot of changes that are tab size. Can you check your editor settings ? It is pretty hard to review.

Ok, this should fix the indentation issues, if there are any other issues let me know.

Improved in another PR


[REQ] Add the "FoundationColors" theme switcher.

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

What idea do you have?
As the title says. Please add the feature to enable/disable "FoundationColors" theme (the one with the blue play button and more dark colors)

we're not adding flags that are likely to release soon


[ArPow] Produce tarball from the installer repo

As a preliminary step to source-build for 5.0 and eliminating the source-build repo, this moves most source-build functionality to core-sdk.

Build source-build tarball from core-sdk
Doesn't include transport package creation or consumption

For future reference: https://github.com/dotnet/source-build/issues/1466 has some notes on cleanup that will need to happen to get this done, and the prototype branch.

Closing this as it is a duplicate of https://github.com/dotnet/source-build/issues/2063


The orchestrator function 'SendQueueOrchestrator' failed. Failed to sync all users. Insufficient privileges to complete the operation

Same issue.
Hello,
Getting the following errors/warning when I Hit Send in chat to everyone. It won't send to anyone.
Send in chat to everyone
Errors:
Failed to prepare the message for sending:The orchestrator function 'SendQueueOrchestrator' failed: "The activity function 'DataAggregationTriggerActivity' failed: "Specified argument was out of the range of valid values. (Parameter 'Recipient count should be > 0. Value: 0') Actual value was .". See the function execution logs for additional details.". See the function execution logs for additional details.
Warnings:
Failed to sync all users. Status Code: Forbidden Exception: Code: Authorization_RequestDenied Message: Insufficient privileges to complete the operation. Inner error: AdditionalData: date: 2021-02-08T14:55:27 request-id: 652e22c8-14b6-4d49-8ee6-11e2655699e0 client-request-id: 652e22c8-14b6-4d49-8ee6-11e2655699e0 ClientRequestId: 652e22c8-14b6-4d49-8ee6-11e2655699e0

Did you add the account you are using to send notification to the "sender UPN List?"
. also make sure you grant admin permission https://github.com/OfficeDev/microsoft-teams-apps-company-communicator/wiki/Deployment-guide#4-add-permissions-to-your-app

Yes

and yes

Hello @TGiersberg,
It happens due to permission but I can see you have added all the permission. Please try once with admin account and just confirm that you have added "Semicolon-delimited list of the user principal names (UPNs)" in  AuthorCreatorUpn.
Let us know if issue still persists.
Thanks

Hello @TGiersberg,
You should also have a look at this issue. #316
A user has mention

I found a solution: in the "Microsoft Teams admin center" go to "Teams apps"->"Manage Apps" and select the Company Communicator author app.; after clicking the author app you can go to the Permissions section and click "Review permissions and consent". This action will grant consent to the author app to read the groups info.
I successfully tested this process in two different tenants!

Let us know if this works for you.
Thanks.


Add one-way response configuration options

Some clients, for instance SAP SOAP Client, expect different behavior when calling one-way operations. For example SAP expects a 202 status code with no body or an empty body with a status code 200.
This little change makes it possible to configure both options.
This is my first contribution to this project, so please give me a constructive feedback ;-)

Coverage decreased (-0.03%) to 93.558% when pulling 9491d5961fc59a7f85a1847cec5bcdf23e08b725 on fid-dev:master into bcc41e6c41e79d2cbbca8887c2fdb36502b10d36 on vpulim:master.

@jsdevel I've adjusted the code to your review remarks and added two unit tests. I hope this is fine now.

@leachiM2k can you address this? https://github.com/vpulim/node-soap/pull/1004/files#r177319579

@jsdevel after your review, I've implemented the initialization in the method _initializeOptions. I left line 42 as it was to match the style of other lines in the constructor.

@leachiM2k see my comment https://github.com/vpulim/node-soap/pull/1004#discussion_r178121914

@jsdevel I've updated the code line. Hope it's ok right now.

Thanks @leachiM2k !


Final Group Capstone: set up your Kanban board

Hi !
We are in a team of four people @chudisoft @MayPyone  @mryeminthu  and @ishimwepati ,
Below is the link to our KanBan Project :
Below are the screenshots :

https://github.com/users/ishimwepati/projects/10/views/2
Your suggestions are most welcome !

Hi,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that does not contain a screenshot of the kanban board.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hi,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
To highlight:

Alll tasks have descriptions✔

Required Changes ♻

[ ] According to the project requirements:

At the time of this review, these are the pointings:
MayPyone 8
chudisoft 8
mryeminthu 14
ishimwepati 10
As you can see, @mryeminthu has significant more points than the others.
The general rule is:  "Nobody should have more than 5 points than anyone else in the team".
Kindly, make your kanban board comply with this requirement. 👍
tip: You'll need to count to achieve this.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi,
Thanks a lot we have work on it,

Wow, you did it 🎉

Thank you for implementing the changes requested by the previous reviewer 💪 🥇 ㊗
Well done!
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
To highlight

Pointings are now equal✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Not working on hydro dam roads

As shown in screenshot, the Medium Dam Road is still at 30cm (other roads are all at custom 10cm):

Confirmed - DamAI is not currently included in eligibility filter.  Segment shader is still valid.

Fixed in f060b597857847fc11902dc55ca3562be3249b99

Tested, works great now - thanks!


Unable to log into Codewars due to 5XX: Server Error

Accessing https://www.codewars.com/ gives me the "5xx: Server Error" :(
I believe my username is: BananaTraider.
I tried private Firefox mode but can access the login screen. Thats why I can't try to reset my password.

How can I solve this problem? :)

Duplicate of https://github.com/Codewars/codewars.com/issues/1867

Ok, thanks 👍


Update Sony - PlayStation 2 (Redump).json

New clone groups/titles:

"Choro Q"
"Dot Hack G.U. Vol. 1 - Rebirth - Terminal Disc"
"Flying Circus - RC Copter Adventure Championship (Propo-gata Controller Doukonban)"
"Kanojo no Densetsu, Boku no Sekiban - Amirion no Ken to Tomo ni"
"Lupin Sansei - Lupin ni wa Shi o, Zenigata ni wa Koi o"
"MS Saga - A New Dawn"
"R-C Sports Copter Challenge"
"Thread Colors - Sayonara no Mukougawa"
"Walt Disney Pictures Bolt"

New compilations:

"Geesen USA - Midway Arcade Treasures"
"Street Fighter Anniversary Collection"
"Simple 2000 Series Vol. 62 - The Super Puzzle Bobble DX"

Untangling and corrections:

"Centre Court - Hard Hitter" and "Hard Hitter Tennis" are two different games, corrected based on personal testing and Wikipedia article
Downgraded priority of "Dot Hack Fraegment (Senkou Release-ban)" as it's a cut down pre-release version of the game (source)
Reshuffle for "Matching Maker" and "Matching Maker 2" clone groups, using supersets to tie the improved Simple 2000 releases with the English localizations
Corrections for "Oneechanbara" series using supersets for updated versions, based off Wikipedia and Fandom articles

Thanks, impressive work :)


ClaimsAuthorizationManager

Perhaps the question is misplaced (as it applies more to .NET 4.5 framework than Thinktecture.IdentityModel), but it would be nice if the CheckAccess method would return a data structure rather than boolean.
Here's why:
Suppose the application needs to display a meaningful message to the end user stating the specific reason instead of a generic "Not authorized", how can this be done with the current framework?

Good idea - feel free to fork the repo and make whatever changes you like.


Automatically switch to dark theme on Windows

The iOS build automatically switches themes according to the current system color scheme, and if IIRC the macOS build does it too. I wonder, could this feature land on Windows too? I think it's pretty useful for folks like me that use AutoDarkMode to switch color schemes on a schedule.

This should be doable.
However, there is a small wrinkle in that this feature requires Windows 10 while the app should run on Windows 7, too. Something to take into consideration...

There's the AppsUseLightTheme registry key (located at Computer\HKEY_CURRENT_USER\SOFTWARE\Microsoft\Windows\CurrentVersion\Themes\Personalize). By being accesible through the registry i think it should work on previous versions too.
Wish I could help with the implementation but I don't really know any C. 😕 Thank you for this great piece of software though! Hope my suggestion was helpful.

I've read through the code and it seems like the app already does some form of dark mode detection, which is for the title bar only. Could it be possible to also enable the option in the UI and execute a os.theme.changed command like in MacOS & iOS?


json-rpc code snippets should use stellar sdks

Currently the code snippets use general purpose dependencies (like json and requests in python).
However, if we expect developers to be using the respective SDKs (like stellar-sdk in python), which I think is a reasonable assumption, we should demonstrate how that's done with them instead.

Link?

Ah these pages?

Ref: https://soroban.stellar.org/api/methods/getHealth
I think it's helpful that we have some raw examples for how to interact with what is a simple http server. People don't need to use an SDK to talk to a http server.
If we do add examples with the SDKs, it'd be helpful to keep the cURL and JSON tabs at least, if not the others too.

Yes, the cURL and JSON examples should remain as is. The JS, Python and any other programming language we add should use the canonical sdk

Or possibly add sub tabs with / without sdk.

Or possibly add sub tabs with / without sdk.

sounds like unnecessary complexity to me

@leighmcculloch @tomerweller I think the sub-tabs would indeed be "unnecessary complexity" in our circumstance. The way we make these tabs currently is a hack of a customization using a patch on a metamask package. I think trying to shove in some sub-tabs would make it even more complex. To be honest, it's possible we're already deep into "unnecessary complexity" territory lol
I really like the idea of using the SDKs though for the JS/Python/etc. examples, though. Really smart!

For the record, here's the patch file I was referring to: https://github.com/stellar/soroban-docs/blob/aa79eff15d82367e3097b5f53525ccda3d50434a/patches/%40metamask%2Bopen-rpc-docs-react%2B0.1.2.patch


In WindowBase.PositionChanged event handler Point value is wrong

Describe the bug
The value of PixelPointEvenArgs.Point inside event handler is wrong.
To Reproduce
Steps to reproduce the behavior:

Create new Avalonia C# project.
In MainWindow.axaml.cs add line in constructor to output argument value and actual value:

    public MainWindow()
    {
        InitializeComponent();
        PositionChanged += (s, e) => Debug.WriteLine($"Position={e.Point}, actual={Position}");
        //Position = new(666, 666); // optional
    }

Run and observe debugger "Output" window.
My outputs are

Position=190, 213, actual=188, 182
or
Position=668, 697, actual=666, 666

Expected behavior
I expect e.Point value to be equal to actual window Position.
Environment

OS: Windows 11
Avalonia-Version: 11.0.9

Additional context
I tried to use PositionChanged event to store position of window in normal state, see #14517 ,but unfortunately discovered a new frustrating bug in Avalonia.

Looks like Position is considered the position of the client area in some places, and the position of the window frame in others.

Maximising the window reveals that e.Point is the location of the client area, but Position is the location of the whole window including its frame.
Position=0, 23, actual=-1, -8

The raw value that comes from GetWindowRect is -8, -8. An attempt is being made to counteract the window's drop shadow, but it doesn't seem to be working very well.
What should the Position property really represent? The client area? The window frame? (I think we can discard the option of the window frame including drop shadow.)

cc @emmauss


Performance Optimization: OpenACC acceleration

Summary of the new feature/enhancement
Operations in PowerShell which consist of a single task being performed on many things at once could be accelerated significantly using OpenACC Libraries as PowerShell would be able to utilize the parallel computing power of modern GPUs such as those provided by Nvidia, AMD, and even the integrated graphics on various Intel CPUs. For owners of APUs and consumer-class Intel CPUs especially, it would basically be a free performance upgrade!
OpenACC is best suited for Single-Instruction-Multiple-Data (SIMD) operations, so not all code would benefit from parallelization in this manner.

@Amaroq-Clearwater PowerShell Core is based on C#, .Net Core and Roslyn and used all benefits from its. I think your suggestion is more for Roslyn. You could start discussion in Roslyn repo.
On the other hand, you can find several discussions in the repo on how we would benefit from using parallel (calculations).

OpenACC could probably be adapted for C#, and .NET Core could probably be adapted with OpenACC. Not sure how difficult it would be, but PowerShell would be far from the only beneficiary if this were to happen.

Precisely, and that's what @iSazonov was saying. This kind of change would have to happen at a much lower level than PowerShell, either in Roslyn or in .NET Core. 😄


[SPARK-29402][PYTHON][TESTS] Added tests for grouped map pandas_udf with window

What changes were proposed in this pull request?

Added tests for grouped map pandas_udf using a window.
Why are the changes needed?

Current tests for grouped map do not use a window and this had previously caused an error due the window range being a struct column, which was not yet supported.
Does this PR introduce any user-facing change?

No
How was this patch tested?

New tests added.

cc @HyukjinKwon @viirya @icexelloss

Test build #111926 has started for PR 26063 at commit 7ff90e3.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16905/
Test PASSed.

Test build #111926 has finished for PR 26063 at commit 7ff90e3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111926/
Test PASSed.

This only adds some basic tests, so merging to master

lgtm too.


Trying to connect to cluster with kubectl, does not complete

I have installed the latest tack and have on issue:
➜  ~ kubectl get po
Unable to connect to the server: dial tcp: lookup kz8s-apiserver-test-xxx.eu-west-2.elb.amazonaws.com on <IP_ADDRESS>:53: no such host
➜  ~
so the last step, does not complete:
❤ Trying to connect to cluster with kubectl...............................................

i am able to login to the pki-machine, bastion and etcd machines, on the pki machine, i have this error:
core@ip-10-0-10-9 ~ $ journalctl -fl
-- Logs begin at Wed 2017-06-07 08:45:51 UTC. --
Jun 07 09:01:40 ip-10-0-10-9.eu-west-2.compute.internal systemd[1]: Started Session 2 of user core.
Jun 07 09:01:40 ip-10-0-10-9.eu-west-2.compute.internal systemd-logind[846]: New session 2 of user core.
Jun 07 09:01:40 ip-10-0-10-9.eu-west-2.compute.internal systemd[1412]: Reached target Paths.
Jun 07 09:01:40 ip-10-0-10-9.eu-west-2.compute.internal systemd[1412]: Reached target Sockets.
Jun 07 09:01:40 ip-10-0-10-9.eu-west-2.compute.internal systemd[1412]: Reached target Timers.
Jun 07 09:01:40 ip-10-0-10-9.eu-west-2.compute.internal systemd[1412]: Reached target Basic System.
Jun 07 09:01:40 ip-10-0-10-9.eu-west-2.compute.internal systemd[1412]: Reached target Default.
Jun 07 09:01:40 ip-10-0-10-9.eu-west-2.compute.internal systemd[1412]: Startup finished in 15ms.
Jun 07 09:01:40 ip-10-0-10-9.eu-west-2.compute.internal systemd[1]: Started User Manager for UID 500.
Jun 07 09:02:19 ip-10-0-10-9.eu-west-2.compute.internal locksmithd[1295]: Unlocking old locks failed: error setting up lock: Error initializing etcd client: client: etcd cluster is unavailable or misconfigured. Retrying in 5m0s.

core@ip-10-0-10-9 ~ $ systemctl status
● ip-10-0-10-9.eu-west-2.compute.internal
    State: running
     Jobs: 0 queued
   Failed: 0 units
    Since: Wed 2017-06-07 08:45:57 UTC; 25min ago
   CGroup: /
           ├─user.slice
           │ └─user-500.slice
           │   ├─session-2.scope
           │   │ ├─1410 sshd: core [priv]
           │   │ ├─1418 sshd: core@pts/0
           │   │ ├─1419 -bash
           │   │ ├─1430 systemctl status
           │   │ └─1431 /usr/bin/less
           │<EMAIL_ADDRESS>           │     └─init.scope
           │       ├─1412 /usr/lib/systemd/systemd --user
           │       └─1413 (sd-pam)
           ├─init.scope
           │ └─1 /usr/lib/systemd/systemd --switched-root --system --deserialize 15
           └─system.slice
             ├─systemd-machined.service
             │ └─1123 /usr/lib/systemd/systemd-machined
             ├─systemd-timesyncd.service
             │ └─734 /usr/lib/systemd/systemd-timesyncd
             ├─cfssl.service
             │ └─1279 /opt/bin/cfssl serve -address <IP_ADDRESS> -ca /etc/cfssl/ca.pem -ca-key /etc/cfssl/ca-key.pem -config /etc/cfssl/ca-config.json
             ├─dbus.service
             │ └─797 /usr/bin/dbus-daemon --system --address=systemd: --nofork --nopidfile --systemd-activation
             ├─update-engine.service
             │ └─845 /usr/sbin/update_engine -foreground -logtostderr
             ├─system-serial\x2dgetty.slice
             │<EMAIL_ADDRESS>             │   └─921 /sbin/agetty --keep-baud<PHONE_NUMBER>0 9600 ttyS0 vt220
             ├─system-getty.slice
             │<EMAIL_ADDRESS>             │   └─922 /sbin/agetty --noclear tty1 linux
             ├─systemd-logind.service
             │ └─846 /usr/lib/systemd/systemd-logind
             ├─locksmithd.service
             │ └─1295 /usr/lib/locksmith/locksmithd
             ├─systemd-resolved.service
             │ └─852 /usr/lib/systemd/systemd-resolved
             ├─polkit.service
             │ └─868 /usr/lib/polkit-1/polkitd --no-debug
             ├─systemd-udevd.service
             │ └─566 /usr/lib/systemd/systemd-udevd
             ├─systemd-journald.service
             │ └─532 /usr/lib/systemd/systemd-journald
             └─systemd-networkd.service
               └─779 /usr/lib/systemd/systemd-networkd

on the etcd, i have this:
Container Linux by CoreOS stable (1353.8.0)
core@ip-10-0-10-10 ~ $  journalctl -fl
-- Logs begin at Wed 2017-06-07 08:46:10 UTC. --
Jun 07 09:12:29 ip-10-0-10-10.eu-west-2.compute.internal systemd[3696]: Startup finished in 35ms.
Jun 07 09:12:29 ip-10-0-10-10.eu-west-2.compute.internal systemd[1]: Started User Manager for UID 500.
Jun 07 09:14:59 ip-10-0-10-10.eu-west-2.compute.internal kubelet-wrapper[1777]: E0607 09:14:59.073010    1777 container_manager_linux.go:638] error opening pid file /run/docker/libcontainerd/docker-containerd.pid: open /run/docker/libcontainerd/docker-containerd.pid: no such file or directory
Jun 07 09:15:07 ip-10-0-10-10.eu-west-2.compute.internal etcd-wrapper[1300]: 2017-06-07 09:15:07.478614 I | mvcc: store.index: compact 2323
Jun 07 09:15:07 ip-10-0-10-10.eu-west-2.compute.internal etcd-wrapper[1300]: 2017-06-07 09:15:07.480529 I | mvcc: finished scheduled compaction at 2323 (took 1.271578ms)
Jun 07 09:16:19 ip-10-0-10-10.eu-west-2.compute.internal systemd-timesyncd[726]: Network configuration changed, trying to establish connection.
Jun 07 09:16:20 ip-10-0-10-10.eu-west-2.compute.internal systemd-timesyncd[726]: Synchronized to time server <IP_ADDRESS>:123 (2.coreos.pool.ntp.org).
Jun 07 09:19:59 ip-10-0-10-10.eu-west-2.compute.internal kubelet-wrapper[1777]: E0607 09:19:59.089258    1777 container_manager_linux.go:638] error opening pid file /run/docker/libcontainerd/docker-containerd.pid: open /run/docker/libcontainerd/docker-containerd.pid: no such file or directory
Jun 07 09:20:07 ip-10-0-10-10.eu-west-2.compute.internal etcd-wrapper[1300]: 2017-06-07 09:20:07.495803 I | mvcc: store.index: compact 2804
Jun 07 09:20:07 ip-10-0-10-10.eu-west-2.compute.internal etcd-wrapper[1300]: 2017-06-07 09:20:07.496958 I | mvcc: finished scheduled compaction at 2804 (took 874.22µs)
Jun 07 09:24:59 ip-10-0-10-10.eu-west-2.compute.internal kubelet-wrapper[1777]: E0607 09:24:59.104149    1777 container_manager_linux.go:638] error opening pid file /run/docker/libcontainerd/docker-containerd.pid: open /run/docker/libcontainerd/docker-containerd.pid: no such file or directory
Jun 07 09:25:07 ip-10-0-10-10.eu-west-2.compute.internal etcd-wrapper[1300]: 2017-06-07 09:25:07.503462 I | mvcc: store.index: compact 3283
Jun 07 09:25:07 ip-10-0-10-10.eu-west-2.compute.internal etcd-wrapper[1300]: 2017-06-07 09:25:07.504568 I | mvcc: finished scheduled compaction at 3283 (took 869.887µs)

any advise is much appreciated

I'm a beginner. I had similar issues a few times when I started. Sometimes just waiting helped, e.g. wait 15 minutes for the cluster to "settle". Other times I had to make clean and then make all again. I would say 1 in 3 attempts succeeded. Once it does succeed it is stable (at least for the 2 weeks I've been trialing Kubernetes).

When I tried eu-west-2 it failed because that region didn't have the m3.large instance types that tack uses.
Maybe try with the default tack region just to see if it works. then you will have some confidence that it can work in London region with some "fixes".


"Default Domain:" should be indicated as a not required field in OpenLDAP access control page.

Server version - Build form master - Dec 10
When trying to configure access control using "OpenLDAP" , "Default Domain:" is indicated with a "*" to mean it is a required field.
This field is optional for OpenLDAP configuration and UI should reflect this.

AFAIK this field is meaningless with OpenLDAP, so it's hidden now.

Tested with server version - v0.50.0-rc2 . This field is not presented in not presented in the "OpenLDAP access control page" anymore.


Staticcall to fallback function burns all gas

On line 90 of Permit2Lib there is an attempt to do a staticcall to get the DOMAIN_SEPARATOR value. If the token does not have a DOMAIN_SEPARATOR, the call fails and the function falls back to using permit2. However, if the token has a fallback function (such as WETH), all gas sent in the static call gets burned.
This can have negative effects if a user sets a high gasLimit in their transaction. For example, while testing this function my test environment had a default gas limit equal to the block limit, so the transaction used over 29 million gas.
You can see a similar issue come up in this StackExchange post: https://ethereum.stackexchange.com/questions/96547/high-gas-consumption-when-using-staticcall-on-a-non-view-pure-function

Interesting, thanks for raising! @transmissions11 maybe we should cap the gas for DS check? Beyond just wasting user gas this also borks p2lib for these tokens as fallback to permit2 permit() wouldn't happen

ah yikes— yes sounds like a gas cap is in order


📃: Addition in Automation Tools

:red_circle: Title : Addition of an Image tool in Automation tools section
:red_circle: Aim : To add a project of image resizer and converter in tools section
:red_circle: Brief Explanation : A tool that Automatically resizes or converts a batch of images applying uniform dimensions or format conversion (e.g., PNG to JPEG).
Screenshots 📷
N/A

:white_check_mark:

Full name : Himanshi Maheshwari
What is your participant role? gssoc-extd contributor and hacktoberfest

Thank you I'm working on it. Also, i think the label of Hacktoberfest should be hacktoberfest-accepted as per guidelines... Please take a look at the same.

Thank you I'm working on it. Also, i think the label of Hacktoberfest should be hacktoberfest-accepted as per guidelines... Please take a look at the same.

hacktoberfest-accepted should be added to the merged PR and only hacktoberfest label to issues. Thats what me and mentors have decided to maintain proper structure.

Alright.


python callback for update analysis and wait call that blocks

Looking through the bugs I just realized we don't have an issue for this already though we've mentioned it a few times and plan on resolving it.
There is currently no reliable way to script up the python API on large binaries for calls that require analysis without long sleeps after calling update_analysis. This is due to the
This is due to the mult-threaded nature of the analysis.
The proper fix is a blocking call or a callback upon completion of analysis so the rest of the plugin can continue.

blocking on #272

Fixed in dev with the update_analysis_and_wait API.


feat: prepare to release APISIX-BASE <IP_ADDRESS>.7

hi, i cant find APISIX-BASE <IP_ADDRESS>.7 docker image in hub, https://hub.docker.com/r/api7/apisix-base/tags?page=1.
when could you guys provide this image in docker hub pls?


Add an effect for Aid bonus

We don't really have a place to put "Basic Action Effects" so I stuck it in Feature Effects


Add “—output build_tree” option (i.e., cmake —B/path/to/build/folder)

report error if specified directory does not exist
report error if specified directory is not writeable
(else) set operating “current” directory to specified build_tree path and proceed

Note: Use “output” name instead of “build” to avoid confusion with cmake —build

Github thinks travis is failing.  I've checked the travis logs and don't see any errors.  Thoughts?


Anchor links only fire once when manipulating page store

Describe the bug
Clicking a link that goes to an anchor on the page works but after you remove the anchor via the page store the link no longer works.
The use case i had was using anchors for modals, modal opens on anchor link and anchor link is removed on modal close.
Reproduction
Here's my original repro
https://stackblitz.com/edit/sveltejs-kit-template-default-qwsbag?file=src%2Froutes%2F%2Bpage.svelte
and here is a different approach that highlights the same issue made by a friend
https://stackblitz.com/edit/sveltejs-kit-template-default-fnttrd?file=src%2Froutes%2F%2Bpage.svelte
Logs
No response
System Info
System:
    OS: macOS 13.4.1
    CPU: (10) arm64 Apple M1 Max
    Memory: 900.20 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.16.0 - /opt/homebrew/opt/node@18/bin/node
    npm: 9.5.1 - /opt/homebrew/opt/node@18/bin/npm
  Browsers:
    Firefox: 114.0.1
    Safari: 16.5.1

Severity
serious, but I can work around it
Additional Information
The only work around atm is to force a page reload on modal close, which is not ideal

This is the exact thing I have been struggling with. I ended up using querystring to solve it temporarily, but I want to track this because using anchors is preferable to me.

I don't think the issue is tied to the page store. It stems from the data-sveltekit-preload-data attribute preload causing the router to think we've already navigated to the anchor (thus doing nothing).
The preload method gets called when hovering over the link, updating our current.url to the anchor before we even click the link.
https://github.com/sveltejs/kit/blob/715dfdd2236111d475ba451046ac8c4e272a937f/packages/kit/src/runtime/client/client.js#L1574


Command to list the profiles

Add a command which lists all stored profiles, This helps using the tool after while again.

This feature requires a huge rework of the architecture. To the point that it can only be released in a v3 of the tool.
It lies mainly on the fact the profile actions are currently dependent on reaching the JIRA server, however listing the servers does not specifiy a server and therefore the application closes with an error before event looking at the profiles.


ConnectBot crashes when opening new connection

Bug description
Seemingly out of nowhere on an LG V30+ phone the ConnectBot application will force close on a n attempt to establish a connection. The application was working previously fine for months and now no matter what release version (stable or latest GIT) it crashes when trying to initiate a connection
I've installed Android Studio and retrieve the stack trace and attached
Expected behavior
Expect the terminal window to pop up, a connection to be established, and then to be prompted for credentials on this portion
Screenshots
Video Added
Android device

Device: LG LS998
OS: Android 8.0
ConnectBot Version: Latest stable but doesnt matter if latest GIT is used

Server information
Server is irrelevant
Additional Info
Trying to attach video and stacktrace in next post

Stack
stacktrace.txt

Video in Action
https://send.firefox.com/download/361efee6dd3d69bf/#ZPdGJleh03erveEmGA_FHw

While trying to step thru the program and find the issue I did notice that I was getting messages that state "Sourcecode does not match the bytecode". Not sure if that's relevant, but here is an example

Same issue here
Device: LG G8
OS: Android 10.0

Perhaps could be LG phone related these are similar generation models. 😥


[JS] Overflow actions (flyout menu) #4715

Related Issue
ActionSet overflow flyout (... actions menu) #4715
Description
To introduce a new feature of overflow actions, the implementation follows the spec:

This new feature will bring as part of Schema version 1.5

Introduce a new property field mode to all Action type. Its value can be primary (default) or secondary. For all secondary action instances in the same actions array (either the root level actions section or ActionSet inside card elements) they'll group as overflow actions, wrap up within a "..." button and display flyout context menu when user clicks "..." button.

Introduce a new global setting flag (schema version-free) allowMoreThanMaxActionsInOverflowMenu. By default it's false, but when it's true then the actions exceeding the number defined in hostConfig.actions.maxActions will be auto wrapped up as overflow actions and "..." button for context menu will be displayed.

hostConfig.actions.maxActions doesn't count "..." button itself.

Two client callbacks are added to control overflow button "..." and flyout context menu rendering:

onRenderOverflowActions?: (actions: readonly Action[], isRootLevelActions: boolean) => boolean

It'll be called when "..." overflow button will be rendered. First argument actions are those overflow actions wrapped as flyout menu actions, and isRootLevelActions indicates if the current "..." overflow action belongs to root level actions array. If it's false then it indicates it's ActionSet as body elements.
Client callback requires to return a boolean value. Returning false means let SDK continue to render "..." button, while true informs SDK don't render the current "..." button (which implies client code might grab those flyout actions and render / handle overflow actions by itself).

onDisplayOverflowActionMenu?: (actions: readonly Action[], target?: HTMLElement) => boolean

It'll be called when overflow context menu will be shown. The first argument actionsare those overflow actions wrapped as flyout menu actions, and  target is the HTML element triggered the context menu, which usually is the "..." button. It can be used by client to position the context menu if client wants to render flyout by itself.
Client callback requires to return a boolean value. Returning false means let SDK continue to render flyout context menu, while true informs SDK don't render flyout (which implies client code might render context menu by itself).

Sample Card
{
  "type": "AdaptiveCard",
  "version": "1.5",
  "body": [
    {
      "type": "Image",
      "url": "https://adaptivecards.io/content/adaptive-card-50.png"
    },
    {
      "type": "TextBlock",
      "text": "Hello **Adaptive Cards!**"
    },
    {
      "type": "ActionSet",
      "actions": [
        {
          "type": "Action.Submit",
          "title": "Action Submit",
          "data": {
            "key": "Submit from overflow menu"
          },
          "mode": "secondary"
        },
        {
          "type": "Action.OpenUrl",
          "title": "OpenUrl",
          "url": "https://github.com/Microsoft/AdaptiveCards",
          "iconUrl": "https://scontent.ftpe5-1.fna.fbcdn.net/v/t1.0-9/122050521_10222320423416866_2987313765978046472_n.jpg?_nc_cat=103&ccb=2&_nc_sid=09cbfe&_nc_ohc=gLcNo1PMYHAAX_KI9HV&_nc_ht=scontent.ftpe5-1.fna&oh=1adaf81bd075ef9ba9d42c7eec78218a&oe=603E2FCE",
          "mode": "secondary"
        }
      ]
    },
    {
      "type": "TextBlock",
      "text": "some text..........."
    },
    {
      "type": "ActionSet",
      "actions": [
        {
          "type": "Action.Submit",
          "title": "View",
          "data": {
            "key": "View"
          },
          "mode": "primary"
        },
        {
          "type": "Action.Submit",
          "title": "Edit",
          "data": {
            "key": "Edit"
          },
          "mode": "secondary"
        },
        {
          "type": "Action.Submit",
          "title": "Delete",
          "data": {
            "key": "Delete"
          },
          "mode": "secondary"
        }
      ]
    },
    {
      "type": "TextBlock",
      "text": "some other text..........."
    }
  ],
  "actions": [
    {
      "type": "Action.OpenUrl",
      "title": "OpenUrl 1",
      "url": "https://adaptivecards.io"
    },
    {
      "type": "Action.Submit",
      "title": "Action Submit",
      "data": {
        "key": "Submit from overflow menu"
      },
      "mode": "secondary"
    },
    {
      "type": "Action.OpenUrl",
      "title": "OpenUrl",
      "url": "https://github.com/Microsoft/AdaptiveCards",
      "iconUrl": "https://scontent.ftpe5-1.fna.fbcdn.net/v/t1.0-9/122050521_10222320423416866_2987313765978046472_n.jpg?_nc_cat=103&ccb=2&_nc_sid=09cbfe&_nc_ohc=gLcNo1PMYHAAX_KI9HV&_nc_ht=scontent.ftpe5-1.fna&oh=1adaf81bd075ef9ba9d42c7eec78218a&oe=603E2FCE",
      "mode": "secondary"
    }
  ]
}

How Verified
How you verified the fix, including one or all of the following:

New unit tests that were added if any. If none were added please add a quick line explaining why not.
Existing relevant unit/regression tests that you ran
Manual scenario verification if any; Do include .gif's or screenshots of the testing you performed here if you think that it
will aid in code reviews or corresponding fixes on other platforms for eg.

Microsoft Reviewers: Open in CodeFlow

Closed this as it's not going to release in main branch thins month. Move this to another feature pre-lease branch as PR #5446


Roll Flutter from c5e8757fcb79 to 61f6a0bcd476 (18 revisions)

https://github.com/flutter/flutter/compare/c5e8757fcb79...61f6a0bcd476
2023-02-04<EMAIL_ADDRESS>Roll Flutter Engine from 1b132e44194d to c7a4bbab0e75 (6 revisions) (flutter/flutter#119990)
2023-02-04<EMAIL_ADDRESS>Roll Flutter Engine from e1b265bb52aa to 1b132e44194d (8 revisions) (flutter/flutter#119980)
2023-02-04<EMAIL_ADDRESS>Do not run customer testing on release candidate branches. (flutter/flutter#119979)
2023-02-04<EMAIL_ADDRESS>Revert "Fix unable to find bundled Java version (#119244)" (flutter/flutter#119981)
2023-02-04<EMAIL_ADDRESS>Fix unable to find bundled Java version (flutter/flutter#119244)
2023-02-04<EMAIL_ADDRESS>Verify Mac artifact codesigning on x64 and arm64 (flutter/flutter#119971)
2023-02-03<EMAIL_ADDRESS>e1b265bb5 Roll Skia from 07a95bb37760 to 83a3d8b16c94 (5 revisions) (flutter/engine#39373) (flutter/flutter#119967)
2023-02-03<EMAIL_ADDRESS>Run macOS benchmarks in prod pool to upload metrics (flutter/flutter#119963)
2023-02-03<EMAIL_ADDRESS>Run Mac hostonly tests on any available arch (flutter/flutter#119884)
2023-02-03<EMAIL_ADDRESS>Tweak to floating-cursor-end behaviour (flutter/flutter#119893)
2023-02-03<EMAIL_ADDRESS>Add mac_benchmark ci.yaml property (flutter/flutter#119871)
2023-02-03<EMAIL_ADDRESS>Roll Flutter Engine from 33d932efc68e to 110c643d6ac2 (3 revisions) (flutter/flutter#119957)
2023-02-03<EMAIL_ADDRESS>[flutter_tools] fix Cannot delete file ENOENT from fuchsia_asset_builder (flutter/flutter#119867)
2023-02-03<EMAIL_ADDRESS>33d932efc Add gen_snapshot to windows flutter artifact. (flutter/engine#39353) (flutter/flutter#119951)
2023-02-03<EMAIL_ADDRESS>Roll Flutter Engine from c39047ffb2a6 to 745d7efb5736 (3 revisions) (flutter/flutter#119943)
2023-02-03<EMAIL_ADDRESS>Fix iOS context menu position when flipped below (flutter/flutter#119565)
2023-02-03<EMAIL_ADDRESS>Roll Flutter Engine from 0fb48ce5b118 to c39047ffb2a6 (2 revisions) (flutter/flutter#119939)
2023-02-03<EMAIL_ADDRESS>Add Material 3 RadioListTile example and update existing examples (flutter/flutter#119716)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins
Please CC rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Plugins: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


There is no example for updating/patching in the document

I'm wondering if we can find standard way in postgrest to update a record. I can only see the 'patch' with bulk insert.

Hey there, let's say you've selected a record like GET /people?id=eq.1. To update that record change the verb to patch and include the fields in the body of your request that you'd like to change like
PATCH /people?id=eq.1

{ "name": "New name", "age": 102 }

Let me know if this answers your question and I can add it to the docs.

Yes, that answers the question :)

Moving this issue to the new postgrest-docs repo.


Add pre-commit action to run interrogate

I looked into it a bit and recommend adding the following config. in particular,
pass_filenames: false
is needed for some reason, as per https://github.com/econchick/interrogate/issues/60
  - repo: https://github.com/econchick/interrogate
    # cf http://www.pydocstyle.org/en/stable/index.html
    rev: 1.4.0
    hooks:
      - id: interrogate
        args: [ -vv, --config=pyproject.toml ]
        # see https://github.com/econchick/interrogate/issues/60
        pass_filenames: false

other documentation checkers:

https://github.com/terrencepreilly/darglint
http://www.pydocstyle.org/en/stable/index.html
I think we're fine with the basics though


Websocket support

Just curious to know, if you're planning to support websockets instead of (or in addition to) SSE?

Not currently planned, since at least as of two weeks ago, there's no good tokio-based websockets  library for Rust that I could find. That said, if one appears, I'll definitely be adding it — it shouldn't even require an API change for users of Backtalk. Also, once HTTP/2 support lands for Backtalk, or even if you stick it behind a HTTP/2 reverse proxy, my understanding is that the overhead from placing new requests to the server and the per-domain browser limits on number of requests should also stop becoming a problem, mitigating the major reasons to prefer websockets over SSE.
I'll keep this open as a reminder to check for tokio websocket libraries ever now and then. There's currently https://github.com/snapview/tokio-tungstenite, but it doesn't quite look ready maybe? We'd also want to make sure there was a way to pass an upgrade request from the hyper server to whatever websocket library we end up using, so that we don't have to host multiple servers on different ports.

Ok cool - I've written very simple layer in nodejs and react/redux for client side where all the comms are done over websocket. There's no REST API, all the "event" requests go to central registry and registry service invokes all registered services with the payload. Then at a later point in time individual services send data back to central registry which pushes data with event identifier to web socket client(or clients). And on client side these events are mapped to different functionalities. This model worked very well and I was looking around for doing something similar on server side with rust. This requires 2 way comms though. I'll watch out for websocket support and how this library shapes going forward.


Can it be used for inference on ESP32?

The main frequency of ESP32-S3 can reach up to 240MHz. What problems may I encounter?

What do you mean? Why would you have any problems with that?

What do you mean? Why would you have any problems with that?

Hi,I just think it would be an interesting thing. I saw someone using esp32 to reason about AI, and I happened to see your article on AI training and reasoning that does not rely on any third-party libraries.

Genann should work great on the ESP32. I know others are using it there, and even on smaller chips too.