Change Volume Alarms

Based on EURS requirements, lets modify teh volume alarms:
In Volume mode, generate Low Volume (Medium Priority) at or below 80% VMax
Generate High Volume alarm at 120% of VMax.
As always, this in the On state after 5 breaths....
In Pressure, stay as is with the same Low Volume alarm except set the limit at 250 ml..

Just to clarify. We need to add a volumeHigh alarm which did not exist before. Is this an alarm or warning.
Volume low goes from an alarm to a warning? In both modes?

Yes. In Volume mode, low and high are medium priority.
In pressure mode, volume below 250 ml is high priority.

This needs expanded to do this presumably:
https://github.com/slaclab/pabv_control/blob/d83d117002328973569dc3e3ef2438cf2efa231c/arduino/libraries/ambu_common/CycleControl.cpp#L180


support partition spec update in pyiceberg

Feature Request / Improvement
as title.

Thanks for raising this! @puchengy which catalog are you using?
Related is https://github.com/apache/iceberg-python/issues/22

Duplicate of https://github.com/apache/iceberg-python/issues/193


General cleanup

Improve search styles for more stark contrast

Update to yarn

Clean up formatting

Consolidate config

Update contributing docs

fyi @iammerrick @alexlafroscia this has been released in 3.7.0.

Ohhh excited about the better highlighting for search!


Fixed a bug that caused white squares to appear in current tab

Fixes #259.

Thank you so much! Glad to finally find a fix for this.


Notice for GTK users on tutorial-1.rst

Link to the issue on github which contains a workaround

Hey @smartsammler, thanks for the PR, sorry it's taken us so long to reply.
Is this an issue you yourself have experienced? From the #38, it looks like we didn't get complete confirmation that this is happening on multiple linux versions, and so I'm hesitant to link to it in the docs.

This update should no longer be required, as described on #38.


⚠ Bofrost.at has degraded performance

In cf004a1, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1272 ms

Resolved: Bofrost.at performance has improved in 0d384d8 after 7 minutes.


Server crashed when webhook receives 400 status code response

Description
If my nestjs server returns a 400 status code response to the onChange webhook, the hocuspocus will crash.
I see there is no catch clause in hocuspocus-webhook.esm.js
Is it suitable to add a catch like below?
async sendRequest(event, payload) {
  const json = JSON.stringify({ event, payload });
  return axios.post(this.configuration.url, json, { headers: { 'X-Hocuspocus-Signature-256': this.createSignature(json), 'Content-Type': 'application/json' } }).catch(err => console.log(err));
}

And, sendRequest uses axios to send requests, it seems all 2xx status code will be treated as a success, while others will throw a exception which crashes the entire server. The line 109 shows
if (response.status !== 200 || !response.data)
  return;

Maybe, don't hard code response.status !== 200 will be nice, because POST responses of nestjs will contain a 201 status code by default and axios considers it as a success.
Summary

add catch to axios.post
remove response.status !== 200

That's my own adverise, welcome to discuss.
Environment?

Hocuspocus version: v2.11.2

@wxfred I've just quickly checked but as far as I could see, all sendRequest calls are wrapped in try/catch with console.error as the error handler. Can you share the stack trace of the crash?
https://github.com/ueberdosis/hocuspocus/blob/main/packages/extension-webhook/src/index.ts#L122

@janthurau Thanks for your prompt, I located the error, the original code didn't add async/await to axios, sendRequest returns a promise, and here are my change
async onChange(data) {
    if (!this.configuration.events.includes(Events.onChange)) {
        return;
    }
    const save = async () => { // add async
        try {
            await this.sendRequest(Events.onChange, {  // add await
                document: this.configuration.transformer.fromYdoc(data.document),
                documentName: data.documentName,
                context: data.context,
                requestHeaders: data.requestHeaders,
                requestParameters: Object.fromEntries(data.requestParameters.entries()),
            });
        }
        catch (e) {
            console.error(`Caught error in extension-webhook: ${e}`);
        }
    };
    if (!this.configuration.debounce) {
        return save();
    }
    this.debounce(data.documentName, save);
}

BTW, I found that other hooks actually have async/await, the miss is only in sendRequest.

uff. Thanks for reporting this! Will push a fix right away.


add description to every parameter in action.yml

Let's add descriptions, as it's done here: https://github.com/JamesIves/github-pages-deploy-action/blob/dev/action.yml

@yegor256 As I see descriptions are already written. Is there something else behind this issue or should it be closed?

@Legolass322 right, let's close


Prevent NavMenuGroup text from overlapping expand/collapse button - #493

Fixes #493
Two problems.
1: The menu group seems to be about 2px slightly more to the start (left) than menu link, but I don't know what margin-inline-start unit to use.
2: I cannot enable RTL mode for some reason, so I cannot check if the expand/collapse appears on the left in that mode.

Closing this as it targets the wrong branch


Capture cursor

I'd like a way to capture the cursor inside the window (to the center, like some games), is it possible for this feature to be added to minifb?

It's certainly possible, but not something I have planned to do. Would it be possible for you to do a PR with this feature?

Not sure, especially because I have to reboot to access other OSes and I'm currently focused on another project. I suppose I could just use click and drag instead of cursor locking.

So exactly what do you need to support this? a set_cursor_pos would do the trick?

I suppose it could be hacked together by setting the cursor position every frame, yeah. But, I would prefer to use the OS's cursor locking.

Also, I just ran into a weird bug where there is a blank space at the top of the window.

That has absolutely nothing to do with this, but here it is.

I think many games actually just set the cursor pos actually (that is because in an FPS you want to detect the direction the user switches the mouse and then you move it back to the center after detecting it)
Unsure what the issue above is, likely something with client area of the window being calculated wrong or something like.

I think many games actually just set the cursor pos actually (that is because in an FPS you want to detect the direction the user switches the mouse and then you move it back to the center after detecting it)

Well, it will probably work just fine :P

likely something with client area of the window being calculated wrong

The window's detected correctly, that's why the buffer is the right size, but the positioning of the buffer gets screwed up.

Is setting cursor position still unsupported?

Yes, it's still unsupported.


GrowBlockEvent was replaced by ChangeBlockEvent.Grow
MoveBlockEvent would be a two tick event and almost impossible to cancel. It is mostly replaced by ChangeBlockEvent.Multi

@gravityfox
You mean ChangeBlockEvent.Post.

@gravityfox
You mean ChangeBlockEvent.Post.

Closed in bleeding.

Closed in bleeding.

Erm ..., Can you remove these events from the event implementation progress tracker?

@Maxqia
Update the list for us and post as a comment on the tracker. We will then update the OP.


[Native] Warning for non supported fill modes

fixes https://github.com/BabylonJS/BabylonNative/issues/1216

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15300/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15300/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15300/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15300/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15300/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15300/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15300/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15300/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15300/merge/testResults/webgpuplaywright/index.html

You sure you want to log everyframe ?

You sure you want to log everyframe ?

I don't think the code does that

would it not log the warning every time draw is called so potentially every frame ?

stupid me lol I did not see this, I had a pavlov reflex of log in draw...


Validate that all virtual methods start with _

Follow-up² to #48746 (see https://github.com/godotengine/godot/pull/49546#issuecomment-860174491)

Thanks!


Rails 3.1

as requested my changes to get alchemy running on 3.1 (at least in development mode)

Nice! Everything works fine?
Also with fleximage?
Thanks i will pull this and test :)
Von meinem iPhone gesendet
Am 02.11.2011 um 04:46 schrieb Marc<EMAIL_ADDRESS>
as requested my changes to get alchemy running on 3.1 (at least in development mode)
You can merge this Pull Request by running:
git pull https://github.com/masche842/alchemy_cms rails_3.1
Or you can view, comment on it, or merge it online at:
https://github.com/magiclabs/alchemy_cms/pull/90
-- Commit Summary --

Update README.md
changed syntax in locales to be compatible with psych
resolved merge conflict in Readme
moved assets to 3.1 standard folder
masche's (my) forks of userstamp and fleximage to get rid of deprecation messages
moved assets around to be somewhat asset-pipeline compatible, updated asset_packages. Replaced static image-paths with sass's image-path
updated stylesheet_link- and javascript_include methods to fit the new (sprocket?) api
removed authlogic-version-requirement in favor of a manually git-statement in gemfile. Not sure if necessary but latest version is 3.1 anyway
moved assets around to be somewhat asset-pipeline compatible, updated asset_packages.
I was a bit too hasty. There are some issues left...
Together with the deprecation message I also killed the functionality of fleximage. Will dig a bit deeper.

One more thing: Since rjs has been removed, all render :update-calls fail.
Partially I was able to solve this, but I'm stuck with render_errors_or_redirect in alchemy-controller.
Is this a normal redirect or some javascript-stuff?

Ah, ok. We have to rewrite this. I rewrote a lot of old rjs templates, to .js.erb files. I will look into this.

So I pulled your request.
Why did you moved the tinymce plugins out of the tiny_mce folder? This is a very bad idea! Because TinyMCE depends on this folder to be there. And the tinymce_hammer gem takes these files and combines them into one tinymcehammer.js file.
And I don't think this is meant to by the asset pipeline. Putting TinyMCE into vendor/assets is correct, but not to take apart the library.
I will move it back into the correct folder then.

Ah, good that you've found that. It was a remaining of my first attempts to get everything working. I forgot to put it back where it belongs...
I fixed the specs, most of them are passing again. There are three failures in clipboard_spec though, I think you should have a closer look as I don't know enough about the implementation.

Ah, nice that you found a solution for that session bug.
I will take a look into the clipboard specs.
Thanks for your contributions!!

IMHO. Though .css.scss is the official naming at the moment, I stumbled over issues with that. After some research this has been solved by renaming to .scss
It's a bit ambigious, on one hand .scss has been the standard for sass and compass for a time now, on the other hand css.scss is more consistent with js.coffee and html.erb...
My feeling is to use .scss at the moment and to see what the future will bring...

And it does not work. As always. Rails development is like hunting bugs the whole day. Really annoying.
How do you resolved the css/scss problem? No background image gets loaded. Complaining about image-url beeing wrong css syntax. So the scss file does not get parsed via sass. It tried renaming the file in to .scss and back to css.scss. nothing helps. And yes, i have the sass-gems installed.


cmd/go: TestScript/build_trimpath_goroot fails during the release process

When run under x/build/cmd/release, the test fails in all longtest configurations:
--- FAIL: TestScript (0.01s)
    --- FAIL: TestScript/build_trimpath_goroot (0.04s)
        script_test.go:260:
            # Regression test for https://go.dev/issue/51461 and https://go.dev/issue/51483.
            #
            # When built with -trimpath, runtime.GOROOT() returned the bogus string "go"
            # if GOROOT was not set explicitly in the environment.
            # It should instead return the empty string, since we know that we don't
            # have a valid path to return.
            #
            # TODO(#51483): when runtime.GOROOT() returns the empty string,
            # go/build should default to 'go env GOROOT' instead. (0.044s)
            > env GOROOT=
            > env GOROOT_FINAL=
            > [trimpath] ! go env GOROOT
            > [trimpath] stderr '^go: cannot find GOROOT directory: ''go'' binary is trimmed and GOROOT is not set$'
            > [trimpath] stop
            > [short] skip
            > go run .
            [stderr]
            go: cannot find GOROOT directory: /usr/local/go
            [exit status 2]
            FAIL: testdata/script/build_trimpath_goroot.txt:21: unexpected command failure

FAIL
FAIL    cmd/go  244.852s

To reproduce, run something like:
release -target linux-amd64-longtest -user $USER -version go1.19beta1 -rev 957cc90c9b350a1d8ef90d992298dfe33cc13980

Worth noting that my redesigned release process in https://go.dev/cl/398694 doesn't exhibit the problem. I'd be interested to know if this indicates an improvement or a bug in the new process.

I can reproduce this fairly trivially by running:
GOROOT_FINAL=/bogus GOROOT=$(go env GOROOT) ../bin/go test cmd/go -run=TestScript/build_trimpath_goroot

#51504 is peripherally related, although this particular case is definitely a bug in the test (because it reproduces even if I set GOROOT explicitly to the correct value).

I'd be interested to know if this indicates an improvement or a bug in the new process.

I'm going with “probably bug”, because the fact that the test doesn't fail seems to imply that GOROOT_FINAL isn't set to its final value while the tests are being built and run.

Ah, yeah, that's correct -- I thought we wouldn't need to set GOROOT_FINAL while running the tests, but that was at least somewhat while I was hoping to use run.bash. OK, will fix.


Bug? Closure arguments not checked for variable number of arguments

Example:
class Biff {}
class Bang {}

interface Callback {
	(...biffs: Biff[]);
}

function foo(callback: Callback) {
	callback(new Biff(), new Biff());
}

foo((x: Biff, y: Bang) => console.log('whoa'));

foo((z: string, n: number) => console.log('huh'));

The closures passed as arguments in the two calls at the end, do not appear to get type-checked.
It's pretty plain that these closures will not be invoked with suitable arguments, yet the code appears to compile with no complaints.
How come this doesn't get type-checked?

Types are checked, but TypeScript has a structural type system. That means that some value is assignable to a type if it has (at least) all the properties of that type. Thus an object that has at least 0 properties is assignable to Biff. Every object has at least 0 properties, so the type Biff is basically equal to any. If you change Biff to:
class Biff {
  someProperty: number;
}

You will get errors.

Right, so even string and number

Right, so even string and number are assignable to a type with an empty interface - yeah, I forget that's how this works and get confused sometimes because the type system syntax is so similar to other languages where this is not the case. Thanks!


Skip processing of duplicate events

Don't create task_final events or syndicate events if the event just processed was evaluated to be a duplicate.


Add automation to file issues on workflow failures

Context
Recently, we added automation to our stack release repositories to file an issue and tag maintainers if the release workflow fails.
We added this because our automation fails silently, and it's easy to miss unless you look at the workflows.
Issue
We should consider adding a similar mechanism for our buildpack release workflows so that we're made aware of failures as they happen, rather than days to weeks later when we go to cut a release. It may also be helpful to implement this automation step for the push buildpackage workflows as well.
Outcome
A workflow step that will file an issue on release/buildpackage push failures for our buildpacks.

This would be especially useful for the builders repos, where the release and push steps are fully automated.


Approuter: remove the secret token on a 403

The rdns server can store the domain to etcd, each domain key has a TTL. If rdns-server didn’t get any update or renew request, the domain will be removed after 10 days.
From @ibuildthecloud :

The common situation is that I have a server I shutoff for >10 days.  When I start the server again I have a stale token and the client does not recover.

To fix this situation, we should remove the secret token on a 403.

We have move the rdns-client to rancher/rdns-server.
https://github.com/rancher/rdns-server/pull/6


Input width / Input Validation width

Select one ...  (check one with "x")
[x] bug
[ ] feature request
[ ] enhancement

Expected behavior
Expect the width of the input to be the same, both on an empty input element and an empty input element w/ validation.
Actual behavior
The input width of the validated element is longer than one without validation.
Reproduction of behavior
https://plnkr.co/edit/JIwRcgL6uQgyI7UrsOyb?p=preview
Environment details

clarity-angular: 0.10.13
angular: 4.3.0

Browser: all

@gnomeontherun
Does the new work on forms fix this issue?

This should be resolved because we reserve a 'space' for the icon regardless if it is used or not. Closing as fixed.


dead link

File: datacenter/dtr/2.4/reference/cli/index.md, CC @joaofnfernandes
https://docs.docker.com/datacenter/dtr/2.4/reference/cli/dumpcerts

Thanks for reporting this @martencassel. It seems that this command is not available anymore and we forgot to remove it from the docs.


Divi Builder Cache

Problem Description
When I change fields in "get_fields" method, they don't change in module builder.
I tryed to clear site cache, localStorage and change browser, but the error persists.
Steps To Reproduce

define fields in "get_fields"
open module in builder
add a field in "get_fields"
reopen module in builder

Hi @meronivalerio, I've tried this approach:
function test_clear_cache() {
	// clears the et_pb_test_module module from the Divi Builder cache
	echo '<script>
			localStorage.removeItem("et_pb_templates_et_pb_test_module");
		</script>';
}

function test_plugins_loaded() { //
	add_action('admin_head', 'test_clear_cache');
}
add_action('plugins_loaded', 'test_plugins_loaded');

but it doesn't work for me. Once the item is removed from localstorage I have to reload the page in order to open the module in the builder.
Any suggestions? Have you managed to make it work?
Thank you.

Hi @jcdotnet, I've tried already to clear localStorage (and cache) like you, but it doesn't work for me either.
For now I have not found a solution; I am changing the slug of the module every time I save, so the parameters are updated, but it is not a valid solution.
I hope someone can help us ;)

I've tried to set a timeout but I don't know at what point the localStorage must me cleared.
Afterwards I've realized that modules won't load until these messages are displayed in the console.
util.js:225 Google Maps JavaScript API warning: NoApiKeys
util.js:225 Google Maps JavaScript API warning: InvalidKey

Once I have these messages I can open any module, weird, so I'm not sure this is a localStorage issue...
@meronivalerio have a look at  \Divi\includes\builder\scripts, the et_builder_maybe_clear_localstorage() function might help you.
@lots0logs do you have a clue on how to solve my issue? I mean, I have to wait to the google messages to load any modules, both built-in and third-party ones. Fortunately there is no delay in most cases and it works as expected but if the google messages take some time the loader shows up and the module load fails.

That is very odd. Normally adding a window.localStorage.clear(); to a JS file, which is enqueued in the admin area, solves this issue

Having the same issue, Clearing localStorage does not help.

Having same issue.
I don't see my fields at all in backend. BUT all fields are accessible in Visual Builder.
Couple of times i saw that when i open module settings my fields (deinfed in get_fields) are quickly show and then dissapear.
Tried to clear wordpress cache and localStorage. Didn't help

Are you all using the new "show_if" conditional logic? I found that this causes fields to not show up on the BB. Also, using toggle slugs which I did not define myself sometimes caused issues.
If you rely on the BB, instead of show_if, you might want to fall back to the old "depends_show_if" conditional logic

@xxtesaxx thank you!
What helped me (i didn't have show_if):

Remove toggle_slug property from all fields (no idea what it does i just copypasted it from tutorial)
Remove all instances of your module from page
Do a localStorage.clear() in console
Reload page
Add your module (wait for it to load, it can take soem time)
It works!

@xxtesaxx I can confirm it works for me too. Thank you. So this example doesn't work anymore because of the 'main_content' toggle slug.

You can add the main_content toggle (or any other toggle you like) manually either in init() via $this->settings_modal_toggles or via the respective function get_settings_modal_toggles() and then it will work too.
Be aware the the Backend Builder caches the settings so after chaning them, you will need to run a localStorage.clear() and reload the page for them to show up. In my plugins, I have put this in a admin.js so I get the localStorage.clear() on every page load.

I have been working recently on the Divi custom modules and I could notice the problem that occurs when adding new fields.
In principle, as mentioned above, I tried to clean the site cache, the localStorage and even use different browsers without any success. I could also notice that if these steps had no effect, the problem should be properly in the plugin or in the internal logic used by WordPress.
This led me to a quick "solution": every time I made a change on the fields of the custom module, it is only necessary to deactivate and then activate again from the administrator the plugin created with npx create-divi-extension, refresh the page where we are testing the module and thus, the change in the fields can be seen in the frontend builder.

You can use ET_Core_PageResource::remove_static_resources from /core/components.PageResources.php

Is this still an issue? I haven't experienced issues with this in - well - years, actually. Just the other day I was changing the default value of a settings and simply reloading the page was enough to see the effects.

Is this still an issue? I haven't experienced issues with this in - well - years, actually. Just the other day I was changing the default value of a settings and simply reloading the page was enough to see the effects.

Can you please explain with some short example. It would be greateful.


Enzyme Internal Error: unknown node with tag 2

Current behavior
`    Enzyme Internal Error: unknown node with tag 2
  19 | ): ReactWrapper => {
  20 |   const el = (): JSX.Element => component;
> 21 |   return mount(<Provider store={store}>{el()}</Provider>);
     |          ^
  22 | };`

from:
import { shallow, ShallowWrapper, mount, ReactWrapper } from "enzyme";
import React, { ReactElement } from "react";
import { Provider } from "react-redux";
import { MockStoreEnhanced } from "redux-mock-store";

export const shallowWithStore = (
  component: ReactElement,
  store: MockStoreEnhanced
): ShallowWrapper => {
  const context = {
    store
  };
  return shallow(component, { context });
};

export const mountWithStore = (
  component: ReactElement,
  store: MockStoreEnhanced
): ReactWrapper => {
  const el = (): JSX.Element => component;
  return mount(<Provider store={store}>{el()}</Provider>);
};

I'm using react-i18next and suspect it could be to do with the export:
export default withTranslation()(
  withRouter(
    connect(
      mapStateToProps,
      mapDispatchToProps
    )(HomePage)
  )
);

Expected behavior
No error.
API

[ ] shallow
[x] mount
[ ] render

Version

library
version

enzyme
3.9.0

react
16.8.5

react-dom
16.8.4

react-test-renderer

adapter (below)
1.13.0

Adapter

[x] enzyme-adapter-react-16
[ ] enzyme-adapter-react-16.3
[ ] enzyme-adapter-react-16.2
[ ] enzyme-adapter-react-16.1
[ ] enzyme-adapter-react-15
[ ] enzyme-adapter-react-15.4
[ ] enzyme-adapter-react-14
[ ] enzyme-adapter-react-13
[ ] enzyme-adapter-react-helper
[ ] others ( )

What does withTranslation do?

<EMAIL_ADDRESS>looks like without the withRef option, it's just an SFC.
What's withRouter do?
This is particularly confusing because mount just uses normal React.
However, I do see that you have mismatched react-dom and react versions - what happens if you make both of them precisely identical?

react-dom has var IndeterminateComponent = 2; // Before we know whether it is function or class here, which is super strange.

We use withRouter to get the prop types:
https://github.com/ReactTraining/react-router/blob/master/packages/react-router/docs/api/withRouter.md
I updated both react & react-dom to 16.8.5 but still get the error.
The tests worked fine before I added the i18next framework.

A reproducible test case - without mountWithStore, and with no other deps besides perhaps i18next - would really help here.

Working on the repro test case.
I'll upload when I have it ready.

I figured out the cause. I was using Fetch from the backend to load the string resources. For some reason, this was causing the tests to fail. Running the tests using a local resources file worked fine.

I'm not sure what Fetch is (unless you mean fetch) - did this happen because you rendered a Promise?


Chars are not recognized as separate glyphs

Chars are not recognized as separate glyphs, the whole line only and it does not fit the screen. Is it possible to get smaller default gap value between chars?

Please attach the document.

is fixed in ec9d73ab7acd09c41f49df80946022dbb6adf2c0


evaluate_generator broken in 2.3

evaluate_generator will silently return the loss computed only on the first batch and not aggregate over batches. other metrics are ok, the problem is only with the loss.
it was ok in 2.2.5. and broken for 2.3.0, 2.3.1 due to this commit: https://github.com/keras-team/keras/commit/b938b91b527760cef3af29031e74ab181f2fc2b9
@pavithrasv

@gowthamkpr thanks but this is not a performance issue - this is a serious regression bug in evaluating the model (and in training if you use early stopping)

Thank you for reporting this issue @eyaler. If you know the fix for the issue, please feel free to send us a PR. Otherwise, I will look into the fix as soon as possible.

colab has not update from 2.2.5 to 2.3.1 and this will cause trouble

this is still a major issue. why is this closed?


A generic way for attestation on Paloma chain

Background
We should be able to attest to any arbitrary and generic event from the outside world.
Done when

[ ] there is a generic way to attest to something in chain

This is done.


Add an optional parameter to pass program directory to main and service commands.

Also did some modifications for ballerina-debug command to make it uniform with other parameters.
Currently you need to 'cd' to ballerina program directory to run a ballerina program ( with package name ). This optional parameter will allow you to run a program from any location without 'cd' to program directory.

will send this PR to vm branch


Xcode beta 11 requires adding signing certificate to each new flutter project

I am using Mac OS X 10.15 Catalina beta, with XCode beta 11. For each flutter app that I tried to run on an iOS device, I had to add my signing certificate in XCode. This was incredibly annoying to run, for example, the devicelab integration tests. Not sure if this is user error (or even if there's anything we can do about it), but I'm opening this to track it.
Doctor summary (to see all details, run flutter doctor -v):
[!] Flutter (Channel master, v1.8.2-pre.133, on Mac OS X 10.15 19A512f, locale en-US)
    ✗ Downloaded executables cannot execute on host.
      See https://github.com/flutter/flutter/issues/6207 for more information

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/setup/#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, set ANDROID_HOME to that location.
      You may also want to add it to your PATH environment variable.

[!] Xcode - develop for iOS and macOS (Xcode 11.0)
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install:
        sudo gem install cocoapods
        pod setup
[✓] Chrome - develop for the web
[!] Android Studio (not installed)
[!] VS Code (version 1.36.1)
    ✗ Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[✓] Connected device (2 available)

! Doctor found issues in 5 categories.

I'm going to close this as it seems like this behavior started in XCode 10 (and doesn't seem to be an issue to users?).

Turns out this was actually fixed by this: https://github.com/flutter/flutter/pull/37733


[Issue] Plugin version 1.1.0-alpha.3 on pub.dev doesn't include the new changes.

Pub.dev 1.1.0-alpha.3
Current Repo code

merge mistake, I will publish alpha.4


bug-4954  Polygon Data Downloader : Added Quote & Trade Tick Downloads with SIP Mapping

Description
Updated the language version in each project to be the LATEST C# version.  This was needed for coalescing null values
SIP Mapping logic needs work, but now adds Row Conditions and uses Suspicious Flag.
Trades and Quotes are now working
Added x64 for projects to eliminate out of memory errors
Added parallel foreach and commented out in downloader because Tick classes aren't thread safe on dictionary inserts
Extended HistoryRequest.cs for polygon parameters to flow through instead of config.json lookup which would vastly slow down the system
Extended Tick.cs for polygon specific parameters related to faster downloads
Related Issue

Motivation and Context
Added better logic and SIP mapping for polygon downloader
Requires Documentation Change
No documentation changes necessary
How Has This Been Tested?
Ran this many times and downloaded all of 2020 data and then verified the results manually
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] Refactor (non-breaking change which improves implementation)     Refactored the json downloader to handle null json values
[ ] Performance (non-breaking change which improves performance. Please add associated performance test and results)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Non-functional change (xml comments/documentation/etc)

Checklist:

[X] My code follows the code style of this project.
[ ] I have read the CONTRIBUTING document.
[X] I have added tests to cover my changes.   Just fixed the bug and verified the data
[ ] All new and existing tests passed.
[X] My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

Hey @TightLinesOfCode, thank you for the PR. We'll need to do substantial rework but can guide you through if you're up to it.
To start; strip all changes not in /Toolbox or  ToolBox/Polygon. Try and think of yourself as a heart surgeon, not a carpetener. Given the risks of interfering with live trading, we make all changes step by step.
Tick data is raw in lean; the filtering is at your algorithm level, not the infrastructure level. In separate PR's we'll need to review and discuss the addition of new properties and ensuring they're relevant and well-integrated first.
Anything not specifically Polygon related needs to be separate PRs.

Hey @TightLinesOfCode, thank you for the PR. We'll need to do substantial rework but can guide you through if you're up to it.
To start; strip all changes not in /Toolbox or  ToolBox/Polygon. Try and think of yourself as a heart surgeon, not a carpetener. Given the risks of interfering with live trading, we make all changes step by step.
Tick data is raw in lean; the filtering is at your algorithm level, not the infrastructure level. In separate PR's we'll need to review and discuss the addition of new properties and ensuring they're relevant and well-integrated first.
Anything not specifically Polygon related needs to be separate PRs.


Fixed duplicate/copy runset

added call to UpdateRunnersBusinessFlowRunsList function while saving runset as it making empty BusinessFlowRunsList while creating new Repo(like copy or duplicate)

Merged, please update the related ticket.


Add automatic check for updates

Closes #2.

This should print v2.1.5 is available: you have v2.1.3. Try "pip3 install --no-cache --user cs251tk" to update. when there is an update available.
This uses the new config module to store the last-checked-for-updates time, so it will only check every fifteen minutes.
I picked 15 minutes because the toolkit is either run very frequently within a short span of time (meetings) or once or twice over a longer time (homework checking).

Coverage decreased (-0.7%) to 56.937% when pulling ccb6d99ed5fd8f5d2b9e04e0772ddce3953d2b87 on check-for-updates into 4833c2dd3cdb02e8b42276372ec53b552927db81 on master.


Extension Feature Contributions is missing markdown styles

Example extension Markdown Checkboxes

Seems this extension contributes following which are not yet implemented to be rendered in this view
"contributes": {
		"markdown-markdownItPlugins": true,
		"markdown-previewStyles": [
			"checkboxes.css"
		]
	},

@mjbvz FYI

Is this issue up for grabs?

@sandy081 These extension points come from the built-in markdown extension, not core. I suggest that we show these in some way since a few other extensions also define their own extension points

Good to know. Not sure how can we represent contributions for extensions generically.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


HHH-13085 : Use FetchMode.JOIN to load OneToOne associations that hav…

…e the same AssociationKey as the entity owner
https://hibernate.atlassian.net/browse/HHH-13085
I still need to look at this more deeply and create a test case.

Closing this PR because https://github.com/hibernate/hibernate-orm/pull/3488 is a better solution.


Minimum size SKU required to deploy a cluster?

What is the minimum size SKU required to deploy a cluster? How difficult is that information to find?
Under the section, "Durability characteristics of the cluster" it says in the Bronze row, "VMs with at least 50 GB of local SSD." In regard to what SKU that requires, can that be clearer?
Under Virtual machine sizing it says, "For production workloads, the recommended VM size (SKU) is Standard D2_V2 (or equivalent) with a minimum of 50 GB of local SSD, 2 cores, and 4 GiB of memory." So, Standard D2_V2 is the recommended SKU size. Can you make that absolutely clear as described in the previous paragraph?
Like the standard system requirements we all see for installing software, please add a section named, "Minimum Cluster Requirements" or  "Cluster System Requirements" and then make it absolutely clear a Standard D2_V2 is recommended. If a smaller SKU is available that meets the minimum requirements then make that obvious.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 658de695-0927-175f-0419-dcf7b6bece9d
Version Independent ID: 8166322c-774b-9434-3df5-5501b7dd9cf1
Content: Service Fabric cluster capacity planning considerations - Azure Service Fabric
Content Source: articles/service-fabric/service-fabric-cluster-capacity.md
Service: service-fabric
GitHub Login: @erikadoyle
Microsoft Alias: pepogors

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. Though the scope of our feedback channel here on GitHub covers specific documentation fixes, we can help redirect you to the right support channel to get an answer to your question: [reference to alternative product/service or community endpoint]. Because this issue does not relate to documentation, we are closing this issue. #please-close


[Bitcoin] Refine inscribe inscriptions flow

Summary

Allow transfer and inscribe at the same transaction.
Confirm batch inscribe logic is correct handling.
There are three modes for Inscribe: SameSat, SeparateOutputs, SharedOutput
SameSat and SharedOutput mode have only one output.
When SeparateOutputs is used, the number of output and inscription is consistent.

Closes #issue

Case: Transfer and inscribe may happen at the same transaction
https://ordinals.com/tx/207322afdcca902cb36aeb674214dc5f80f9593f12c1de57830ad33adae46a0a


Interface infelicity - which device is syncing?

The attached screenshot shows, I think, that it's not always as clear as it should be which devices is syncing. (The red scribbles in the screenshot are for purposes of redacting personal information.)

This is a Syncthing issue, not a SyncTrayzor one. Please open an issue on the Syncthing issue tracker.


fix(fzf-lua): pass prompt option to fzf

Pass the prompt value given to vim.ui.select to fzf-lua
Context
Currently the select ui never shows any prompt for fzf-lua
See this example which should show the prompt "Code actions"

This is what I would expect (and how it looks with my changes):

Description
Check for a given prompt value in the opts parameter, otherwise fall back to "Select one of" (as vim.ui.select does by default). Instead of using a colon fzf usually uses the > as a delimiter for the prompt which is why I did so as well.

Just did so :+1:


[Feature Request] Parse YouTube Emoji

Currently custom emojis are not parsed, like I have said before in https://github.com/BanchouBoo/mpv-youtube-chat/issues/7 point number 3. Ideally the image of the emoji should be displayed, but converting these emojis to Unicode standard emoji might be also be an option.
I have scrapped official YouTube emojis to this CSV gist here. However this is not an exhaustive list as there are also channel-specific emojis, I'm not sure if there is any API available for this.

I have toyed around with this a bit in the past. Libass doesn't support colored emoji (https://github.com/libass/libass/issues/381). Theoretically I could render images, but as far as I'm aware there's no way to do that inline with the text. I'd love to get this working, but I don't see how I would.


glfwJoystickIsGamepad's return value is too vague and leads to redundant calls

If glfwJoystickIsGamepad returns false, we still have to call glfwJoystickPresent if we want to know that the joystick is connected. Having the function return one of 3 states instead of just GLFW_TRUE or GLFW_FALSE would allow us to prevent the redundant calls to _glfwPlatformPollJoystick.
Something like GLFW_GAMEPAD, GLFW_NOT_GAMEPAD, GLFW_NOT_PRESENT (not great names but I put it there to explain the idea).

You only need to poll with both functions at initialization. After that you have connection/disconnection events delivered via the joystick callback. Is the need for two calls at initialization a problem in practice on any system?


I do not understand how to execute the downloaded code unit 7

I am not a computer person, so I miss some basics. Concerning unit 7, AI vs AI, I have succeded to download the different files and get them organised as desired. For instance I have succeded following this instruction: At the end your executable should be in mlagents/training-envs-executables/SoccerTwos
I am using Windows10
But I do not know how to execute the code.
The instructions are
Depending on the executable you use (windows, ubuntu, mac) the training command will look like this (your executable path can be different so don’t hesitate to check before running).
which I do not fully understand.
This is what I obtain when trying:
(base) C:\Users\Paisa\ml-agents>mlagents-learn ./config/poca/SoccerTwos.yaml --env=./training-envs-executables/SoccerTwos.exe --run-id="SoccerTwos" --no-graphics "mlagents-learn" no se reconoce como un comando interno o externo, programa o archivo por lotes ejecutable.
I will appreciate any help.

#install conda
!pip install -q condacolab
import condacolab
condacolab.install()
!conda create --name rl python=3.9
!conda activate rl
#clone git branch
!git clone --branch aivsai https://github.com/huggingface/ml-agents
#install python package in ml/agents/ml-agents-envs and ml-agents
!pip install /content/ml-agents/ml-agents-envs
!pip install /content/ml-agents/ml-agents
#download the executable --this is for linux --if download according to your os
!gdown "https://drive.google.com/u/0/uc?id=1KuqBKYiXiIcU4kNMqEzhgypuFP5_45CL&export=download"
#unzip the files
!unzip SoccerTwos.zip -d SoccerTwos
#get the permission
!chmod -R 755 /content/SoccerTwos/SoccerTwos.x86_64
protobuf was throwing a version error, this fixes the error, and runs slower
import os
os.environ['PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION']='python'
#train dont forget to tune the hyperparameter
!mlagents-learn /content/ml-agents/config/poca/SoccerTwos.yaml --env /content/SoccerTwos/SoccerTwos.x86_64 --run-id="resnet" --no-graphics
#login
!huggingface-cli login
#push to hub --keep in mind the run-id
!python ml-agents/ml-agents/mlagents/utils/push_to_hf.py  --run-id="resnet" --local-dir=results/resnet --repo-id="username/poca-SoccerTwos" --commit-message="First Push"
hope it helps 🙂

Hey there 👋 it seems you didn't installed MLAgents please follow @dewasahu2003 solution (thanks @dewasahu2003 🤗).
I'm closing the issue

Hi, I had the same problem. Before even reading everything that @dewasahu2003 wrote, I just ran the install commands again and that solved the issue. However pip said, that nothing new was updated or installed, so I suppose its just some linking bug. Also make sure you are running the training from the correct folder.
cd ml-agents
pip install -e ./ml-agents-envs
pip install -e ./ml-agents


Change the pattern of --exclude option to Regular Expression

The minimatch glob pattern does not properly support multiple exclusions other than a simple (this|that) which is not enough if you need to exclude multiple directories and subdirectories. A good solution to this is to change from a minimatch glob pattern to a Regular Expression pattern which is more powerfull and can almost match anything.
Please consider accepting this as it doesn't affect anything other than this file matching and makes it considerably more powerful.

It does support it, you can do something like {this,that/bar/**} I believe. Either way, I'm 👎  on this PR as it'll break every current user using this feature and minimatch patterns as well-used and understood for CLI tools.

That's fine. If the globbing does support it then that is great but I couldn't manage to make it work and I've spent hours trying. I'll try it with your suggestion.

Success! Your suggestion worked well, thanks. I'll just close this now.


Audio set to loop infinitely stops, when next loop triggers while switched to another tab.

Version

Phaser Version: 3.60.0

Operating system: Win11

Browser: any / all (tested: Firefox Win11, Chrome Win11, Android, MacOS Safari)

Description

Posting following issue report as an info dump without source attached, cant create a repro due to time constraints (maybe later, seeing as i have 30days before closure..). Was able to work around the issue (@see below). Searched first, couldnt find a similar report, so here goes..
Im making a game, where if browser window goes out of focus, music mutes (not sfx, just music). For playing music only, i use separate 'audio' scene, which, when focus is lost, calls this.audioScene.sound.mute = true; and false, when focus is regained. Also i call this.audioScene.sound.pauseOnBlur = false; because the default focus handling / audio mute/unmute didn't always work as expected (all browsers/OSes..). So i funnel all the events (BLUR, FOCUS, HIDDEN, VISIBLE, PAUSE, RESUME) into 2 exit points onFocusGained and onFocusLost and handle the scene audio muting manually there.
Everything works wonders. Music is set to loop, loosing focus mutes it, regain of focus unmutes, works on all browsers/systems, all good. Except one strange issue - and this might be possibly unrelated to the muting code, it is likely related to the gain/loss of focus and how looping is handled internally by phaser during such situation (loss of focus as result of being switched to another tab during a loop trigger).
The issue is this: if i wait for a 1minute long infinitely looping music track to reach its end, and 5 seconds before it reaches its end (and Phaser makes the next loop of the music play), when i switch to another tab on FF/Chrome/MacOS Safari/whatever (this is 5 seconds before the end of the track, which is set to loop infinitely), and come back 10 seconds later (5 seconds after it should have been made to loop to the next loop), then the music unmutes and plays again (good), but 1 minute later, when the next loop should be started by Phaser, it wont start the next loop, and the music instead stops when the current loop finishes. This is unrelated to scene's audio's mute, since i can simply call resume() on the music track and it will start again; it simply will not be auto looped to next loop by phaser if it finishes its current loop. This auto looping bug only happens, when another tab is focused (game looses focus), while the next loop should start (and even then, it just stop looping after current music loop ends).
To work around this, i just opted to use pause/resume on currently playing music track, instead of scene.audio.mute {false|true}, when in|out of focus, that seems to 'work-around' well.
Example Test Code

WIP or none

I believe our team has also seen this issue multiple times on 3.7

Still happens on 3.85.0!

If it was fixed, this issue would be closed.

The AudioBufferSourceNode calls onended internally. This checks if the source is the same as the one in the WebAudioSound. If it is, it sets hasLooped = true and the update method recognises this and does the actual update. Of course, if the tab doesn't have focus, update doesn't run. Plus, if the tab doesn't have focus, the shape of the Source node changes, causing the type comparison check to fail, causing hasLooped to not be set.
Even better, looping is handled by setting the rate of the sound node to zero once the 'time' has expired. Then creating a brand new sound node. For every single iteration of the loop.
Annoyingly, the source node has a native, built-in loop property - if set to true the sound just keeps looping forever! Even when the tab isn't focused. And you can set the start/end points of the loop as well. As a bonus, you don't need to be constantly creating a brand new source node every time it loops.
However, there is no event or callback from web audio when a loop happens, which means no 'native' way to emit the Phaser LOOPED event. So that's wonderful, too. As it means the sound would actually loop, even out of focus, but the in-game events would not, so they'd be out of sync. Although I feel like this isn't really an issue, as you should account for it in your code.
So right now I'm tempted to use the native loop property of the source node, for seamless non-destructive looping. But it's a reasonably large internal change with unknown repercussions elsewhere. I'll have a think about it, but this may just need saving for our brand new Sound API.

Ok, have found a workaround for now and it's part of v3.85. The following test shows it working:
class Example extends Phaser.Scene
{
    preload ()
    {
        this.load.audio('theme', [
            'assets/audio/kyobi/wavs/nextLevel.wav'
        ]);
    }

    create ()
    {
        this.add.text(32, 32, 'Click to start music', { fill: '#ffffff' });

        this.sound.pauseOnBlur = false;

        this.input.once('pointerdown', () => {

            const music = this.sound.add('theme');

            music.play({ loop: true });

            this.add.text(32, 64, '5 seconds until loop', { fill: '#ffffff' });

        });
    }
}

const config = {
    type: Phaser.AUTO,
    parent: 'phaser-example',
    width: 800,
    height: 600,
    scene: Example
};

const game = new Phaser.Game(config);

(assets are in examples repo)


issue #9654

Scope & Purpose
Honor user-defined values for --rocksdb.max-write-buffer-number if within an acceptable range (i.e. 9 or greater). Ignore values that don't make sense.

[ ] Bug-Fix for devel-branch (i.e. no need for backports?)
[x] Bug-Fix for a released version (did you remember to port this to all relevant release branches?)
[x] The behavior in this PR can be (and was) manually tested (support / qa / customers can test it)

Related Information

[x] There is a GitHub Issue reported by a Community User: https://github.com/arangodb/arangodb/pull/9654

Testing & Verification
This change is a trivial rework / code cleanup without any test coverage.
https://jenkins01.arangodb.biz/view/PR/job/arangodb-matrix-pr/5764/

Tests blue


French translation of Intermediate Front End Development Projects

Closes #7639

@elazzabi It would be great if you can get this reviewed by a peer.
Refer : https://gitter.im/FreeCodeCamp/TranslationFrench

Already asked in the chatroom. Still waiting fir someone to check :)

@elazzabi We will merge within a week if no can check it. Thanks for your contribution!

https://gitter.im/FreeCodeCamp/TranslationFrench

https://gitter.im/FreeCodeCamp/TranslationFrench

https://gitter.im/FreeCodeCamp/TranslationFrench

#8725

closing as stale. Please feel free to reopen when ready.


Pokemon Info Evolution & Gradle Kotlin DSL

Adding Pokemon Info - Evolution and Implement Gradle Kotlin DSL

CC: @mrcsxsiq

@naufalprakoso Thanks for your contributions.

You're welcome 😁


OS X build broken: error: unknown warning group '-Werror', ignored [-Werror,-Wunknown-warning-option]

Looks like the current civetweb version breaks compilation on OS X.
Full build log can be found here:
https://travis-ci.org/alkurbatov/suvorov-bot/jobs/334184237
I managed to workaround this by editing s2client-api/contrib/civetweb/src/civetweb.c
and removing '#pragma GCC diagnostic ignored "-Werror"', string 17867.

I've sent path to Blizzard's civetweb regarding the issue, see https://github.com/Blizzard/civetweb/pull/1

Is there any progress on this issue?

Hi @sdy1130 No, it is not fixed, but you can try my fork or update the civetweb submodule manually to workaround the issue.


Fix many vt tests

This PR fixes many of the tests in escape-sequence-tests.js, see the commits for details on the individual changes. Previously 10 tests were passing, not 30+ are.
The major fixes:

CAN has been implemented which cancels any escape sequence, the corresponding test does not work still due to complex ignore logic in xterm that we don't have parity with
Index, LF/VT/FF, CUB, CUD, VPR behavior was change to stay at the end of the column if it was otherwise going to go to wraparound
When wraparound mode is disabled output will be cut off the end, no longer will the last character always be displayed in the last column
There was an error which cause causing our tests to compare only the first page of the buffer, not the viewport.
Expected output was trimmed as we don't care, at least not currently (we trim all output from xterm.js anyway).

There is a failing test:
1) xterm.js unicode - surrogates 2 characters per cell over line end without autowrap:

      AssertionError: expected 'a' to deeply equal '𐀀'
      + expected - actual

      -a
      +𐀀

This is likely because it relied on the previous wraparound disabled logic which I believe was wrong. @jerch thoughts since you added it?

This is the test case failing: https://github.com/sourcelair/xterm.js/blob/4d5046b5e7047bfcf1f1dba088acb343df1969bb/src/test/test.js#L598
Not sure which of these three expects fail. I think that you can just uncomment all your changes in inputhandler, parser and xterm and then by putting one piece at a time back in you will be able to find what broke this test.

Any way I can help?

It's the first expect and I think it's to do with https://github.com/Tyriar/xterm.js/commit/43c3349494c7b922577fa80d50a57b23896e0818
I believe it's verifying the wrong behavior.

@Tyriar Yeah the test checks that with wraparoundMode = false surrogate chars correctly end up the last cell. Since you changed that behavior to truncating the line, the test is not applicable anymore.
(Btw truncating the line is not conform to the old VTs, they were not able to move the cursor beyond the display, thats why they simply inserted the char at the last cell)

@jerch thanks for the clarification, I'll pull the test out :smile:

@parisk good to look at again


feat(apps/staging): add buildfarm app to research dev-test

May be we should call it "local test", currently we foucs on tidb with bazel toolchain.
usage: bazel build --remote_executor=grpc://buildfarm-server.apps.svc:8980 //pkg
Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/review default

/review default


How do I get VcXsrv to work with wayland-0

would like to get my full xfce4 desktop like I have with windows 10 and https://www.tinyurl.com/donofrioworkremmina2020 shows WSL1 working great.  In windows 11 I've yet to get WSL2 to work even when I do the display mod's for the env setup, but that was trying x11, now in Insider w11 dev channel I guess we now get Wayland native but how to get a full DE like xfce4 so far all I get is:
(xfwm4:3779): Gtk-WARNING **: 05:58:38.765: cannot open display: wayland-0

You don't need VcXsrv for WSLg feature. MSTSC is used with WSLg automatically.

WSLg doesn't sound like 2022 from what I've read it was how they did it in 2021?  I've just installed ubuntu in wsl2 on my w11 insider dev channel build then I tried mstsc /v:localhost:3390 but it doesn't connect?  why is it I can run xeyes but not the full DE?


AssertRequest.java:52-55: Figure out how to let the user...

The puzzle 38-1586cf52 from #38 has to be resolved:
https://github.com/amihaiemil/docker-java-api/blob/23aa92a82f3ee0f8fa258716bfee7fce889872a4/src/test/java/com/amihaiemil/docker/mock/AssertRequest.java#L52-L55
The puzzle was created by George Aristy on 13-Mar-18.
Estimate: 30 minutes, role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0crat in

@0crat in (here)

@amihaiemil Job #42 is now in scope, role is DEV

The job #42 assigned to @llorllale/z, here is why. The budget is 30 minutes, see §4. Please, read §8 and §9. If the task is not clear, read this and this.

@llorllale I think you can easily solve this by passing some sort of Conditions object in the ctor. This object would encapsulate that Collection of Predicate and also have a given failure message.
But this is just an idea, do it however you see more fit :D

@amihaiemil honestly, since this is just a support class for our tests I don't want to invest too much in the design. Maybe Condition can just be a carrier for a String and a Predicate<HttpRequest>, no?

@llorllale Yes, sure. But also start writing a test class for it and leave puzzlez for testing it if you don't have enough time.
Even though it is a support class, it needs to be tested itself so we are sure it works fine. Otherwise, bugs could be introduced into it and then all tests written with it become unreliable :)

The puzzle 38-1586cf52 has disappeared from the source code, that's why I closed this issue.

@llorllale I know, I also wasn't paid for the closed PRs, as ARC...
There is a bug in 0crat, discussed here: https://github.com/zerocracy/farm/issues/813
But it should make the payment tomorrow, I think.

@llorllale 0crat seems to be online now, did it pay you?

@amihaiemil it paid me yesterday at around 21:45 UTC


Add support for interface and record function results

This change allows functions returning an interface or record to work.

Can you please describe a use case?

Without these changes, while P4D does contain support for interfaces and records, it did not support them as parameters or result values for function calls.
Also, while it supported records, it did not support managed records.

Ok, we're going to try and merge this and get the P4D team to write some tests for it.

Tag @lmbelo to resolve conflicts on PR


Component name css selector

What problem does this feature solve?
In single file components, the styles section can possibly allow selectors with the same name as components used. Right now, if we need to style the component, it must be assigned to some class that may not be used elsewhere. Along with CSS scoping, I see this as a powerful solution of writing styles for nested components.
What does the proposed API look like?
This is what I am suggesting,
<template>
  <my-component />
</template>

I am not really sure how the loader would convert this to plain CSS using standard selectors. :thinking:

let's not reinvent the wheel, use a CSS class

+1
As we can already use the element selector...
<template>

Hello World

</template>

... why are we unable to do the same with components? Surely this seems like a logical extension of the standard. I wouldn't call it reinventing the wheel.

It’s easy to implement with Vue Loader but the compiled code will mess with selector specificity and lead to unexpected cascading result. Unless you output the root element of the component with an custom element, which works fine with semantic-less elements like
, while replacing other elements like <button> will change the actual behavior, which is also unacceptable.

I have created my own custom component library doing this sort of thing. For example
`

`
if you give the slot container a class name you can use that when creating the component. This one i can select using .cflex without needing to declare this where im creating the component instance. Making for clean and maintainable code. It also makes it rapid to develop some complex webpages once you have built out the component lib.


Check is config is ready before reading config

What does this PR do?
Check is config is ready before reading config

/merge


Trial permanent remove

@suhlrich and @AlbertoCasasOrtiz we decided to replace the word Remove by Trash everywhere in the code and to add an option Delete at the trial level for permanent removal (addressing #74). Trash therefore means "Send to recycle bin" and Delete means "Delete permanently". Could you confirm this is all good to you? Waiting for your green light to move on. Thanks.

Good with me
On Jun 13, 2023, at 2:46 AM, Antoine Falisse @.***> wrote:

@suhlrichhttps://github.com/suhlrich and @AlbertoCasasOrtizhttps://github.com/AlbertoCasasOrtiz we decided to replace the word Remove by Trash everywhere in the code and to add an option Delete at the trial level for permanent removal (addressing #74https://github.com/stanfordnmbl/opencap-viewer/issues/74). Trash therefore means "Send to recycle bin" and Delete means "Delete permanently". Could you confirm this is all good to you? Waiting for your green light to move on. Thanks.
—
Reply to this email directly, view it on GitHubhttps://github.com/stanfordnmbl/opencap-viewer/pull/143#issuecomment-1588941989, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKOYGJ5Y2FX6MZYO4HYMWA3XLAZG5ANCNFSM6AAAAAAZDKPUTM.
You are receiving this because you were mentioned.Message ID: @.***>


haskellPackages.megaparsec: disable tests on version 6.5

Motivation for this change
Disables broken tests for Haskell megaparsec package 6.5
Hydra failure: https://hydra.nixos.org/build/85949393/nixlog/1
Upstream: https://github.com/mrkkrp/megaparsec/issues/323
/cc ZHF #45960
Not tested, unsure how to do so.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I don't understand what happened but I guess somebody kicked Hydra at https://hydra.nixos.org/build/85949393/ ? The link to the log that's in my post no longer contains the error! Because the test failure has a message like "Randomized with seed xxx" I think the test only fails for some seeds. I'll close this in 24 hours if no one pipes up.


Toggle background color didn't function well

Is your feature request related to a problem? Please describe.

I gave a background-color to the toggle, but it didn't appear in the nobelium.

Describe the solution you'd like

Add the background-color to the toggle.

Additional context
The first img is what I write in the notion

the second img is what I see in nobelium

Hi there, sorry for the late reply. Unfortunately I couldn't find a good workaround about this at the moment. I might tweak about it in the future~

Hi! I know it’s so late but we have added the background support for toggle blocks! Hope you enjoy!


Add support for device: dreame.vacuum.p2029

its just like Z10 but without the bin for dust. :)

its just like Z10 but without the bin for dust. :)

Should be straight forward! Will add support for it in the next update!

Should be straight forward! Will add support for it in the next update!

How often do you do updates?

There is no schedule, usually when i think that there are enough changes then i update :) But the next update will be definitely next week, but not sure what day.
BTW, what is the exact name of your device?

Dreame Bot L10 Pro

BTW, i see that as you say the device is pretty much identical to the dreame.vacuum.p2028 (Z10 ), so if you cannot wait till next week you can force to use the Z10 spec by specifying the model manually in the config.json as dreame.vacuum.p2028 . That should make your device already work now, but just remember when the update comes out that you have to remove the model line so that the correct spec is used!

BTW, i see that as you say the device is pretty much identical to the dreame.vacuum.p2028 (Z10 ), so if you cannot wait till next week you can force to use the Z10 spec by specifying the model manually in the config.json as dreame.vacuum.p2028 . That should make your device already work now, but just remember when the update comes out that you have to remove the model line so that the correct spec is used!

I will give it a try tomorrow!

Support is now added as of version 0.9.10!

Hi! thanks!
seems like the "Charging" status does not work. when its connected to the dock its shows "Charging: no" in apple homekit and in homebridge

When it is connected to the dock and the battery is 100% the device will not charge so it will not charging.
The active mode is basic so the tile will be on even when the vacuum is in the dock, that is wanted behavior since you are able to switch modes when the vacuum is currently not cleaning.

Can i see the total cleaned area/brush life etc in apple homekit?

In HomeKit currently not, it is logged in the console. There is no accessory which can display such info. But I could actually use the lux sensor for that...


Crystal changed color.

Changing the JSON is nice but since it generates a different readme, why not update the readme also with the same PR ?

Updated the readme too!

Nope, still "Files changed: 1"


fix(deps): bump

Applied mkpr on undefined
/tmp/mkpr.sh

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

v1.29.26
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Syntax improvements for various syntaxes

Rebase

Improved rebase dashboard syntax + test
Changed rebase graph unicode characters to make it look better
The syntax still support old unicode characters

Graph syntax

more detailed scopes
added test
replaced * in graph with ●

Diff syntax

improved diff syntax and added test
Fixed a bug where diff can not detect new file where the header is like --- /dev/null

Show commit

improved show_commit syntax and added test
show commit buffer should be read-only

Make commit

make commit should use the GitSavvy diff syntax not the built-in diff syntax
improved make commit syntax and added test

Blame

improved blamed syntax and test

Interactive rebase

Improved interactive rebase syntax and added test

Here is how it looks. you can try it by using my custom color scheme

This is really great.  Would you mind rebasing and cleaning up the Git history somewhat?  I use git: generate changelog when announcing new versions, and this would add a bit of noise for our next release.  This is the preferred format, but just squashing related commits together would be more than enough.

As a separate thought, perhaps we should provide official GitSavvy color schemes and provide a mechanism to switch between them.  I like the colors you're using, and if we added a dark theme too, that might be received well.

Rebase is done.
Regarding default color scheme, I agree. my custom color scheme needs a lot of cleaning. The colors are borrowed from the offical git website. I'll send another pull request for the color scheme when I cleaned it up.

Sounds great - thanks again.


Kwargs of API client do not pass down correctly

Currently, kwargs are passed down as params, hence headers and request_timeout cannot be set.
Relevant code snippet.
def completion(self, **kwargs) -> dict:
        return self._request_as_json(
            "post",
            "/v1/complete",
            params=kwargs,
        )
	...

def _request_raw(
        self,
        method: str,
        path: str,
        params: dict,
        headers: Optional[Dict[str, str]] = None,
        request_timeout: Optional[Union[float, Tuple[float, float]]] = None,
    ) -> requests.Response:
	...

As shown above, if users set headers or requesst_timeout in completion method, it will be pass down as params. Headers and request_timeout will remain None.
It can be handled like below instead.
def completion(self, **kwargs) -> dict:
        return self._request_as_json(
            "post",
            "/v1/complete",
            **kwargs,
        )

def _request_raw(
        self,
        method: str,
        path: str,
        headers: Optional[Dict[str, str]] = None,
        request_timeout: Optional[Union[float, Tuple[float, float]]] = None,
        **params: dict,

    ) -> requests.Response:

Trying to fix this here:
https://github.com/anthropics/anthropic-sdk-python/pull/39

We've just released an entirely new version of the SDK, which should address this. Please take a look!


Can you please help me with proof keys implementation

I wanted to extend my sincere appreciation for the invaluable resource your repository has been during my WOPI work.
As I dive into implementing WOPI proof keys, I've noticed some commented-out sections in the code. Could you offer some guidance on how to proceed?

Can you point to which section of the code you're referring?


Mono support

Hello,
any mono experts out there? This implementation is working via Windows but not via Mono - In mono I got never a return value:
    public partial class Authenticator : Form, ITwoFactorCodeProvider
    {
        public Authenticator()
        {
            InitializeComponent();
        }

        public string otpAuth = string.Empty;

        public async Task<string> GetTwoFactorCodeAsync()
        {
            this.ShowDialog();
            return await Task.FromResult<string>(otpAuth);

            throw new NotImplementedException();
        }

        private void Login_Click(object sender, EventArgs e)
        {
            otpAuth = TXT_Auth.Text;
            this.Close();
        }
    }
}

Any ideas?

Nobody has an idea why the task will never finished?

You're on the UI thread, maybe it's deadlocking?
A couple of things you can try, but i haven't tested them.
public async Task<string> GetTwoFactorCodeAsync()
{
	this.ShowDialog();
	return await Task.FromResult<string>(otpAuth).ConfigureAwait(false);
}

private TaskCompletionSource<string> tcs = new TaskCompletionSource<string>();

public Task<string> GetTwoFactorCodeAsync()
{
	this.Show();
	return tcs.Task;
}

public void Login_Click(object sender, EventArgs e)
{
	tcs.SetResult(TXT_Auth.Text);
	this.Close();
}

Unfortunately the first one also never returns a value, the second one freezes the program even on Windows :(

Do you know how to debug in Mono without digging trough endless stack traces?

I gave it up :(


Update DG

Updated UML Diagrams in Developer Guide
Add Implementation details for marking a task as done

Codecov Report

Merging #90 (2c22871) into master (01fd7ce) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #90   +/-   ##
=========================================
  Coverage     41.50%   41.50%
  Complexity      382      382
=========================================
  Files           114      114
  Lines          2053     2053
  Branches        226      226
=========================================
  Hits            852      852
  Misses         1158     1158
  Partials         43       43

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


Exposing RoleAssignmentApproval APIs

Overview of Role Assignment Approval Scenario
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[x] SDK of Python
[x] SDK of Java
[x] SDK of Js
[x] SDK of Go
[x] PowerShell
[x] CLI
[x] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Add WaitForARMFeedback since new APIs added

Looks like there are many new API's introduced as part of this change. Please share the exact scenario and how these API's will be used and send it to armapireview alias for review to continue.

Filiz and I have added some comments. Please take a look.

Filiz and I have added some comments. Please take a look.

Thanks @majastrz and @filizt for your feedback. Appreciate it!
I've addressed your comments and updated the PR. Please take a look again.

Thanks @majastrz and @filizt for your feedback. Appreciate it!
I've addressed your comments and updated the PR. Please take a look again.

@yungezz can you please take a look again at this PR? Please don't merge it yet since we haven't deployed the APIs yet. I've added a donotmerge label(which I'll remove once we've deployed these APIs)

@yungezz can you please take a look again at this PR? Please don't merge it yet since we haven't deployed the APIs yet. I've added a donotmerge label(which I'll remove once we've deployed these APIs)

@yungezz can you please take a look again at this PR? Please don't merge it yet since we haven't deployed the APIs yet. I've added a donotmerge label(which I'll remove once we've deployed these APIs)

hi @shubhamguptacal I saw you removed Do Not merge label, does that mean the API change has been deployed to Prod? Pls ping me when the PR is ready to merge. thanks

CI passed in ADO https://dev.azure.com/azure-sdk//internal/_build/results?buildId=738937&view=logs&j=2be18a23-047c-5c9f-61d7-60a33437d1f1, but the status isn't synced back to github.


Update bylaws.md

Some changes to the wording for grammaticality, style, and clarity.

closed in 961f4a67b4a4f9da5f10d24e9ca71fa93db1a81b


[WIP] Handle unclean HTML textContent (with script or style tags)

Fixes https://github.com/scrapinghub/extruct/issues/22

Current coverage is 99.11% (diff: 100%)

Merging #23 into master will increase coverage by <.01%

@@             master        #23   diff @@
==========================================
  Files             2          2
  Lines           112        113     +1
  Methods           0          0
  Messages          0          0
  Branches         31         31
==========================================
+ Hits            111        112     +1
  Misses            0          0
  Partials          1          1

Powered by Codecov. Last update 0d215e2...1d35025


Docs: Explain how workspaces work

Add a page about what yarn, pnpm, and npm workspaces are and how they work. It should explain:

What are workspaces?
Why would you use them?
How do they work?
How do they relate to turbo?

Does this cover most things? https://turborepo.org/docs/guides/workspaces cc @mattpocock
First added in https://github.com/vercel/turborepo/pull/884

The handbook does a pretty good job I think: https://turborepo.org/docs/handbook/workspaces


not detect anything in the frames

Hi,
I downloaded the models from iro-cp/FCRN-DepthPrediction and Qidian213/deep_sort_yolov3. Then, I changed the video path to my own video and run python dmo.py. However, there is nothing been detected. The number of detected boxes is always 0 for every frame. Is there anything wrong with my setup?

Hey,
Did you download the weights file? Make sure to run deep_sort_yolov3 individually first. If the detection is a problem it's because you haven't configured deep_sort_yolov3 properly.

I found my problem. I copied yolo.py from Qidian213/deep_sort_yolov3. But it only draw boxes for class person because of line 100 and line 101 in yolo.py.
I commented these two lines then the detector works now.


atom-python-isort leaves behind abandoned status bar item

Running atom-python-isort successfully leaves behind a "isort: ✓" status bar item, which provides no additional information to the user once isort has run.
Since the status bar is generally used for important, persistent information (such as filename, language, interpreter, etc) I recommend that the status bar icon be removed entirely, and that errors only be displayed using the more canonical 'atom.notifications'.
As with #34, I'd be happy to update the package if nobody else is interested.

Personally I prefer the status bar notification. This package works similarly to a linter for me so I just need the status update and then I run the command manually. Having a notification popup in the top right hand corner of my editor would be a distraction. I can imagine this would be a useful option though, for people who want it!


Fix type in example features

Fix type in example features:

visting -> visiting

Good observation, thanks.


runtime certificates loading for truststore

HI there,
I've followed instructions at https://paketo.io/docs/howto/configuration/#ca-certificates to add certificates at runtime.
I can see that log at startup :
Added 3 additional CA certificate(s) to system truststore
But when I'm listing certificates from inside my app, I dont see the 3 added certificates...
It seems that build-time truststore is used....
I should have missed a step.
Can anyone help me, plz ?
Thanks
Ludo

@sdeleuze I think @neoludo wants to load the truststore at runtime, as recently added to graalvm:
https://www.graalvm.org/reference-manual/native-image/CertificateManagement/#run-time-options

Thanks for the link @matthyx, I was missed the fact it was properly documented now. I think such documentation on native image official documentation is good enough so I close this issue.

@sdeleuze hmm, not sure as it's not clear to me how you can specify javax.net.ssl.trustStore and javax.net.ssl.trustStorePassword to the native image from the container image.
One solution is to override the container entrypoint and/or arguments, but it's against the whole buildpack philosophy, isn't it?

Ok let's discuss that with @scottfrederick and @dmikusa-pivotal then.

Do you mean in this issue?

If you want to send additional arguments to the native-image command at build time, you can do that by setting $BP_NATIVE_IMAGE_BUILD_ARGUMENTS. Your arguments are augmented by the buildpack which adds the following information (you don't need/want to include this manually):

When on the tiny stack, -H:+StaticExecutableWithDynamicLibC will be appended to your custom argument list.
-H:Name= is appended to the custom argument list where the value is set as the full path to the start class
-cp is set with the full classpath
The start class is added to the end of the command

If you want to supply arguments at runtime, you can do that using these instructions.  This will let you pass additional arguments to the default command used to run your application.
In terms of javax.net.ssl.trustStore we don't generally recommend setting this value directly for Java applications running on a JVM. If you follow the recommended instructions for adding trusted CA certificates, this will result in the certificates being added to the default system truststore, i.e. /etc/ssl/certs/ca-certificates.crt and to the default truststore for the JVM.
In terms of native image builds, I think this advice would apply as well. The additional trusted certificates should be loaded into the JVM truststore when the JVM and native-image tool are installed. This means that they should be present, which according to the docs should be sufficient:

During the image building process, the native-image builder captures the host environment’s default TrustStore and embeds it into the native executable.

When I last checked this on https://github.com/paketo-buildpacks/graalvm/issues/66 things just worked. I haven't tried this lately though, so if it's not working. Let's take a look and see what's happening. To do that, please include the full output from your build plus any build config (i.e. env variables, buildpack ordering, that you're customizing).
That said, I see no reason why setting the arguments would be a problem. You'd need to use the options above to pass those additional arguments through at run/build time as required by native-image. Please give that a try and let me know how things go.

@dmikusa-pivotal please note we're talking about RUNTIME arguments.
I think @neoludo wants to avoid overriding ENTRYPOINT and CMD in the resulting image in case paketo sets something special during build.

@dmikusa-pivotal, after reading on my laptop (rather than my mobile) I see what you mean... apologizes.

If you follow the recommended instructions for adding trusted CA certificates, this will result in the certificates being added to the default system truststore, i.e. /etc/ssl/certs/ca-certificates.crt and to the default truststore for the JVM.

@neoludo already played with https://github.com/paketo-buildpacks/ca-certificates and at runtime it can add trusted certificates to the system truststore...
The issue is that in native mode it only creates /etc/ssl/certs/ca-certificates.crt and no JKS... which substrateVM doesn't handle (despite my suggestion).
So I think we need to modify the buildpack to generate a JKS and inject the required arguments for using it.

The issue is that in native mode it only creates /etc/ssl/certs/ca-certificates.crt and no JKS... which substrateVM doesn't handle (despite my suggestion).

I agree. I added an issue under the native image repo to track this for us. We'll look into this more.

Thanks @dmikusa-pivotal !

Should I close this issue ?

@sdeleuze I agree this issue is more related to a change in the buildpack... we can close this but let's hope @dmikusa-pivotal work goes to completion.


icon can not load in header

Thanks for your brilliant plugin!
I meet a problem when I installed the plugin, the icon was not showed in the header bar, and it appears after I reload several times, but the time doesn't parse, nothing found in the plugin window, and the icon disappeared after I reload again.
I use windows 11 and logseq APP 0.6.1.

might the internet problem, I see an error of loading the icon from a website

Hi, I cannot seem to reproduce this error.

Are there other plugins installed?
How did you install this plugin?

Yes, I installed some plugins. But the problem is still there after I close all the plugins.
I install it from the market.

Hmm, I am not sure how or why this happened. I am still not able to reproduce it.
Could you try to re-install the plugin using the below steps?

Uninstall plugin
Close Logseq
Delete logseq-calview-plugin.json from .logseq/settings folder
Open Logseq
Re-install plugin
Restart Logseq

it works, thanks a lot!😁


Sending data to Graylog is broken

Currently, when using a Graylog handler, no data is transferred. This has been introduced with #1600 and specifically with commit: b30b2509527dd06404c7aa17186d72ed5d1bb5a7

The problem is that with the changes introduced by #1600 we set up the extras inside the Formatter. This does not work for the pygelf handler, because it doesn't use a formatter at all.


Stablaization fixes for WSO2 SP 4.3.0

Description:
Following improvements should be added to SP 4.3.0

https://github.com/wso2/siddhi/issues/919 : NPE when we batch an empty response from the Query API

Fix a issues in incremental access loop controller (BackoffRetryCounter)

Add feature to wait table operation until connection recover

Add window.batch test cases and fix an issue in expired events process loop

Fix connection retry w/o backoff if connection callback error is called  : https://github.com/wso2/siddhi/issues/932

Fix NullpointerException in TimeWindowProcessor : https://github.com/wso2/siddhi/issues/260

Fix automatic retry functionality in distributed sink.

Fix issue where aggregation amounts doubling when queried : https://github.com/wso2/siddhi/issues/952

Remove unnecessary synchronization in siddhi

Fix blocking while publishing in In-memory broker

HA member change sink handlers activation loop Interrupt with Runtime Exceptions :  https://github.com/wso2/carbon-analytics/issues/1510

Fx the data bridge server hanging issue when we send event composites which is larger than the maximum size of the server.

Affected Product Version:
4.3.0

These fixes have been added to the master branch and PRs can be found at the issue links provided in the descriptions.
Hence, closing this issue.


Rendering on different computer - Unable to Render: File Not Found

We are loving nexrender and excited to implement it.
We are using nexrender with an AE project file created on a different machine, so the paths are different. If we open the project in AE, the relative paths seem to re-link and then everything is fine once we start using nexrender, but if we use nexrender first, then we get this error:
Unable to Render: File Not Found /path/to/file
I'm sure this is an issue with aerender and not nexrender, but i'm curious if you can think of any ways to not have to open the project in AE before rendering using the command line. It seems like aerender should be able to resolve the paths like the UI does.

Hey @byumark
Very glad to hear that!
Usually both arender and nexrender should be able to pick up new relative paths if they located at the same "subpath" relative to the project file you are working with. At least it was the case in older versions of aerender.
In case that doesn't work anymore, I would actually suggest manually settings paths to all assets in the job file description. You don't need to download them if they already exist locally, just use file:// protocol instead of http://. You can read more about that in the "asset substituion" part of readme.
Hope that helps!

Thanks, @inlife .
Are you referring to manually setting paths to all assets in the job json that we submit to nexrender? I can see how that would work, but we use a lot of assets that don't need to be dynamic so putting them all in there seems a bit tedious. Plus, would this still present a problem for cross-platform rendering between mac and windows since one uses a drive letter and the other does not? Or can you do relative paths in the assets section of the job json?

Well in a way yes. Although I think if automation of that part is possible and it might yield a better and more robust result.

so putting them all in there seems a bit tedious
That is unfortunately true. However, you have a benefit of telling AE exactly which files you want to use, and where are they located, preventing any possible bugs related to AE guessing wrong folders or taking wrong files.

Regarding relative paths - I'm not sure, that's something that needs to be tested.
Oh, just got an idea while writing this post. You can actually involve some of your own jsx scripting to solve that issue. The concept is to manually add code that would replace paths to your static assets to always verify they are properly located.
So nexrender uses this template script https://github.com/inlife/nexrender/blob/master/packages/nexrender-core/src/assets/nexrender.jsx
as a base for data injections (/USERSCRIPT/) is where the project-specific code will be put.
If you check out your nexrender's temp project folder, you will find a .jsx script there named after project unique id. And that file will contain a set of replacements for assets and data entries from your rules in job file.
You can add there something like that:
// mycustom.jsx
nexrender.selectLayersByName(null, "MyImageLayerName", function(layer) {
    nexrender.replaceFootage(layer, '/foo/bar/image.jpg')
})
nexrender.selectLayersByName(null, "MyImageLayerName2", function(layer) {
    nexrender.replaceFootage(layer, '/foo/bar/image2.jpg')
})
// ... and so on

Writing this script once, you can add it to every job you need. Honestly it kinda the same as adding those descriptions to the job definition, but it might work for you better in this form.

I am having this exact same problem. Are you saying I need to write a script to dynamically go through every image and repoint it to a new location?

Well, if after effects is not able to do it on its own, I'm afraid that there might be no other way. :(

Hi, i am facing this issue too, is there a solution which we can change all paths in the project programmatically? I also couldnt find any temp folder with unique projectId like suggested as well.

the discussion has moved to #238


Please include gradle-wrapper.properties

When gradle\wrapper\gradle-wrapper.properties and related files are not included in the repo, on first open, Android Studio will create one by itself, but with a wrong gradle version(gradle-6.1.1-all.zip on Android 4.0.1), causing:
Unable to find method 'org.gradle.api.file.ProjectLayout.directoryProperty(Lorg/gradle/api/provider/Provider;)Lorg/gradle/api/file/DirectoryProperty;'.
Possible causes for this unexpected error include:
Gradle's dependency cache may be corrupt (this sometimes occurs after a network connection timeout.)
Re-download dependencies and sync project (requires network)

The state of a Gradle build process (daemon) may be corrupt. Stopping all Gradle daemons may solve this problem.
Stop Gradle build processes (requires restart)

Your project may be using a third-party plugin which is not compatible with the other plugins in the project or the version of Gradle requested by the project.

In the case of corrupt Gradle processes, you can also try closing the IDE and then killing all Java processes.

Gradle version must match Android Gradle Plugin version. For com.android.tools.build:gradle:3.3.2, gradle-4.10.1-all.zip is required.
https://developer.android.com/studio/releases/gradle-plugin?buildsystem=ndk-build#updating-gradle

Yes, you are right. Gradle version should match Android Gradle Plugin version. On first open, we should set gradle version as you did.

For everyone cloning that project, with Android studio 4, I needed to :

Change  Android Gradle project version from 3.3.2 to 4.01 :  https://stackoverflow.com/a/63206864
Error was Unable to find method 'org.gradle.api.file.ProjectLayout.directoryProperty(Lorg/gradle/api/provider/Provider;)Lorg/gradle/api/file/DirectoryProperty;'.
Create gradle.properties in root project folder with : android.useAndroidX=true
Error was This project uses AndroidX dependencies, but the 'android.useAndroidX' property is not enabled. Set this property to true in the gradle.properties file and retry.

This is the SDK version, which is too old. You are advised to update it. For example:
For the project structure, set Android Gradle Plugin Version to 4.1.2  ，Gradle Version 6.5. sync project with gradle files


koord-manager: consider NodeReserved when calculate mid resource.

…n calculate mid resource.
Ⅰ. Describe what this PR does
Just like batch resources, we should consider SafetyMargin, NodeReserved, and SystemUsed when calculating mid resources.
Allocatable[Mid]' := min(Reclaimable[Mid], NodeAllocatable * thresholdRatio) + Unallocated[Mid] * midUnallocatedRatio
Unallocated[Mid] = max(NodeCapacity - NodeSafetyMargin - SystemUsed - Allocated[Prod], 0)

ref node resource model:
https://github.com/koordinator-sh/koordinator/blob/main/docs/images/node-resource-model.png
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in make test

PTAL @saintube thanks!

Allocatable[Mid]' := min(Reclaimable[Mid], NodeAllocatable * thresholdRatio) + Unallocated[Mid] * midUnallocatedRatio
Unallocated[Mid] = max(NodeCapacity - NodeSafetyMargin - NodeReserved - SystemUsed - Allocated[Prod], 0)

@tan90github Is it necessary to substitute the SystemUsed for the Unallocated[Mid]? The calculation of the mid-reclaimable should have included the prediction of the system usage.

When mid-reclaimable is not zero, it indeed includes system usage, so I shouldn't consider systemUsed.This is my mistake.
But when mid-reclaimable is zero, I think we should take into account these system occupations.
By the way, it seems that the systemUsed used in calculating midReclaimable doesn't include hostApplication  currently. How should we consider it? The same issue is also present in nodereserved.

What's your opinion? @saintube thx.

When mid-reclaimable is not zero, it indeed includes system usage, so I shouldn't consider systemUsed.This is my mistake.
But when mid-reclaimable is zero, I think we should take into account these system occupations.
By the way, it seems that the systemUsed used in calculating midReclaimable doesn't include hostApplication  currently. How should we consider it? The same issue is also present in nodereserved.

What's your opinion? @saintube thx.

@tan90github Yes, it is indeed an issue that the Mid reclaimable may be capped to zero and then ignore the system usage. IMO, we can take the system usage into account by substituting the max(systemUsed, nodeReserved).
Overall, we should derive the "unallocated" from the capacity (excluding the reserved) and the request, detouring the usage, since it is the duty of the reclaimable to concisely estimate and predict the unused allocatable. And double-substituting the usage can make the calculation of the Mid-tier confusing.

Yes, it is indeed an issue that the Mid reclaimable may be capped to zero and then ignore the system usage. IMO, we can take the system usage into account by substituting the max(systemUsed, nodeReserved).
Overall, we should derive the "unallocated" from the capacity (excluding the reserved) and the request, detouring the usage, since it is the duty of the reclaimable to concisely estimate and predict the unused allocatable. And double-substituting the usage can make the calculation of the Mid-tier confusing.

unallocated = nodeCapacity - max(systemUsed, nodeReserved) - sum(request[prod])

@saintube  is that right?

unallocated = nodeCapacity - max(systemUsed, nodeReserved) - sum(request[prod])

@saintube is that right?

@tan90github Yep.

I noticed that there is a prediction-safety-margin-percentparameter in prodReclaimable to control the safety threshold, and there is also a midUnallocatedPercent in Unallocated to control it, thus the midAllocatable resources are safe enough. Therefore, I did not consider NodeSafetyMargin when calculating Unallocated. Now the calculation formula for mid resources is as follows:
// Allocatable[Mid]' := min(Reclaimable[Mid], NodeAllocatable * thresholdRatio) + Unallocated[Mid] * midUnallocatedRatio
// Unallocated[Mid] = max(NodeCapacity - NodeReserved - Allocated[Prod], 0)

cc @saintube

@tan90github Agreed.

I noticed that there is a prediction-safety-margin-percentparameter in prodReclaimable to control the safety threshold, and there is also a midUnallocatedPercent in Unallocated to control it, thus the midAllocatable resources are safe enough. Therefore, I did not consider NodeSafetyMargin when calculating Unallocated. Now the calculation formula for mid resources is as follows:
// Allocatable[Mid]' := min(Reclaimable[Mid], NodeAllocatable * thresholdRatio) + Unallocated[Mid] * midUnallocatedRatio
// Unallocated[Mid] = max(NodeCapacity - NodeReserved - Allocated[Prod], 0)

cc @saintube

@tan90github Agreed.

I modified the code, PTAL@saintube thx.

Thanks for your reviews, @saintube . All problems have been fixed.

PTAL thx. @zwzhang0107 @hormes cc @saintube

Can we merge this PR now? I forgot to ask about it in yesterday's bi-weekly meeting. The PR has been LGTM for a while. Current mid-resource calculation logic is flawed (it doesn't take into account things like nodeReserved), and this PR addresses that issue, making the mid-resource calculation complete. If there are any further suggestions about this PR,  feel free to let me know. @saintube @zwzhang0107 thx.


ARROW-1046: [Python] Reconcile pandas metadata spec

Appveyor build: https://ci.appveyor.com/project/wesm/arrow/build/1.0.455

@wesm Thanks!


Added numberOfIslands file (graph application)

Leetcode #200 - Number of Islands is a standard application of graphs. Please find my executable code below along with comments.
Please accept my PR. Thanks!

Well done!