Add F5 pool monitoring scripts

The scripts f5-config.py and f5-monitor.sh are part of the juno branch
of os-ansible-deployment. The were removed in master because they are
Rackspace specific files.
This commit adds these scripts here so they are available for the kilo
release.
f5-monitor.sh was modified from the juno version, this was done because
keystone no longer supports XML. The script now parses the JSON
response for the auth token.
f5-config.py was modified from the juno version; references to
rpc_deploy and rpc_inventory have been replaced with openstack_deploy
and openstack_inventory.
Closes-Bug: https://github.com/rcbops/rpc-extras/issues/46
(cherry picked from commit<PHONE_NUMBER>4670921876943b3205076dfaa1ba44)

:-1: I don't think this has everything covered in the commit message, unless the files are already in the kilo branch and this is just a bug fix in which case the message should be  updated to reflect that fact.

It was cherry picked from that commit, this was the only thing not merged, kinda odd, but I can modify the commit message if you want

Yeah, can you modify the commit message just to people don't scratch their heads later.

-1
It looks like a mistake was made and the change was never merged into master. I think this commit is a regression. The change was introduced into kilo by [1]. [2] was meant to introduce the change into master but it looks like that was closed by mistake. [3] shows how [1] was done on the assumption the change was already introduced into master.
So it looks like master requires a patch to bring it in to line with kilo.
[1] - https://github.com/rcbops/rpc-openstack/pull/137
[2] - https://github.com/rcbops/rpc-openstack/pull/90
[3] - https://github.com/rcbops/rpc-openstack/issues/136

fair enough, gonna reverse this and submit a patch to master then


[2086] Bump azure-blob-storage-upload

Context
The job https://github.com/DFE-Digital/teacher-training-api/actions/workflows/database-restore.yml is failing
Changes proposed in this pull request
Bump the dependency azure-blob-storage-upload as the build is failing
Guidance to review
Run the job
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Fix Staging/QA DB


swap out Jasmine for Mocha

Jasmine doesn't natively support promises

Nevermind - Mocha also doesn't support promises natively. Either way a plugin is needed.


s/Travis CI/GitHub Actions

Also drop node v11 testing, and add node v12, v14, and v15.
Refs: https://github.com/elastic/apm-agent-nodejs/issues/1922

Note this also drops the "npm run coverage" mentioned in the the .travis.yml. I suspect that was copy pasta, because this is what Travis build logs showed for that step:
npm ERR! missing script: coverage

npm ERR! A complete log of this run can be found in:

npm ERR!     /home/travis/.npm/_logs/2020-11-19T12_48_15_906Z-debug.log

from https://travis-ci.org/github/elastic/node-traceparent/jobs/744646746


Write cicd_Repository

The CLI should write the cicd_Repository when artifact information is included such that an artifact source is not necessary when CI events are instrumented. When artifact information is omitted, the cicd_Repository should not be added to the event as the artifact is a dummy artifact that uses the vcs information.

bump the version?


[IDE integration] code not resolving under "transitive" mode

I have created a minimal repro at https://github.com/bazelbuild/rules_scala/pull/1358/files
3 targets: a -> b -> c. Temporarily setting the following for scala toolchain to show the point.
--- a/scala/BUILD
+++ b/scala/BUILD
@@ -23,7 +23,8 @@ toolchain(

 scala_toolchain(
     name = "unused_dependency_checker_error_toolchain_impl",
-    unused_dependency_checker_mode = "error",
+    unused_dependency_checker_mode = "off",
+    dependency_mode = "transitive",
     visibility = ["//visibility:public"],
 )

When importingtmp/src/scala/a into intellij with the bazel plugin:
directories:
  tmp/src/scala/a

# Automatically includes all relevant targets under the 'directories' above
derive_targets_from_directories: true

targets:
  # If source code isn't resolving, add additional targets that compile it here

additional_languages:
  scala

You can see that B can resolve correct, but C fails to resolve.
So was wondering if it's a rules_scala issue that it fails to let bazel plugin's aspect know about C, or a bazel plugin issue for asking for too little?
References

Importing tmp/src/scala would have C resolved.
building aworks just fine:

% bazel build tmp/src/scala/a
DEBUG: /private/var/tmp/_bazel_yic/2ceecb63e72239e50b9654ebbc6f2a73/external/bazel_toolchains/rules/rbe_repo/version_check.bzl:68:14:
Current running Bazel is ahead of bazel-toolchains repo. Please update your pin to bazel-toolchains repo in your WORKSPACE file.
DEBUG: /private/var/tmp/_bazel_yic/2ceecb63e72239e50b9654ebbc6f2a73/external/bazel_toolchains/rules/rbe_repo/checked_in.bzl:125:14: buildkite_config not using checked in configs; Bazel version 4.1.0 was picked/selected but no checked in config was found in map {"0.20.0": ["8.0.0"], "0.21.0": ["8.0.0"], "0.22.0": ["8.0.0", "9.0.0"], "0.23.0": ["8.0.0", "9.0.0"], "0.23.1": ["8.0.0", "9.0.0"], "0.23.2": ["9.0.0"], "0.24.0": ["9.0.0"], "0.24.1": ["9.0.0"], "0.25.0": ["9.0.0"], "0.25.1": ["9.0.0"], "0.25.2": ["9.0.0"], "0.26.0": ["9.0.0"], "0.26.1": ["9.0.0"], "0.27.0": ["9.0.0"], "0.27.1": ["9.0.0"], "0.28.0": ["9.0.0"], "0.28.1": ["9.0.0"], "0.29.0": ["9.0.0"], "0.29.1": ["9.0.0", "10.0.0"], "1.0.0": ["9.0.0", "10.0.0"], "1.0.1": ["10.0.0"], "1.1.0": ["10.0.0"], "1.2.0": ["10.0.0"], "1.2.1": ["10.0.0"], "2.0.0": ["10.0.0"], "2.1.0": ["10.0.0"], "2.1.1": ["10.0.0", "11.0.0"], "2.2.0": ["11.0.0"], "3.0.0": ["11.0.0"], "3.1.0": ["11.0.0"], "3.2.0": ["11.0.0"], "3.3.0": ["11.0.0"], "3.3.1": ["11.0.0"], "3.4.1": ["11.0.0"], "3.5.0": ["11.0.0"], "3.5.1": ["11.0.0"], "3.6.0": ["11.0.0"], "3.7.0": ["11.0.0"], "3.7.1": ["11.0.0"], "3.7.2": ["11.0.0"], "4.0.0": ["11.0.0"]}
INFO: Analyzed target //tmp/src/scala/a:a (0 packages loaded, 0 targets configured).
INFO: Found 1 target...
Target //tmp/src/scala/a:a up-to-date:
  bazel-bin/tmp/src/scala/a/a.jar
INFO: Elapsed time: 0.240s, Critical Path: 0.01s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action

Seems known at https://github.com/bazelbuild/intellij/issues/1825#issuecomment-629224211


prometheus boshrelease: bump to v27.0.0

What
https://www.pivotaltracker.com/story/show/180772978
Updated the upstream prometheus boshrelease to v27.0.0, bringing us prometheus 2.33.3 and a bunch of updated and new dashboards.
Updated the promtool our pre-merge CI uses to match this version. Looked at making it automatically extract the currently used version, but there doesn't seem to be an easy way of doing that.
How to review
Deploy to a dev env, take a look at the dev env's grafana and see its new "v2" dashboards. Take a look at its alertmanager. Take a look at its prometheus (ooh shiny!).

🚨⚠ Please do not merge this pull request via the GitHub UI ⚠🚨

Cool. Will deploy on Monday.


[SR-12672] Tailored diagnostics for missing member involving array literal default to any element

Tailored diagnostics for missing member involving array literal default to any element.

Resolves SR-12672.
cc @xedin @varungandhi-apple

@LucianoPAlmeida just a heads-up: I have a PR up which I'm going to merge soon-ish https://github.com/apple/swift/pull/31315. That change the name for the diagnostic, so you'll need to rebase this PR on top of that.

@LucianoPAlmeida just a heads-up: I have a PR up which I'm going to merge soon-ish #31315. That change the name for the diagnostic, so you'll need to rebase this PR on top of that.

No problem @varungandhi-apple :)

All adjusted @xedin  :)

@swift-ci please smoke test

@varungandhi-apple I was looking to the PR you merged seems like it doesn't affect this diagnostic message, so not sure if we need to rebase here ...

@LucianoPAlmeida oops, sorry about that! I got confused; I was working on changing the member inference diagnostic but that is still local and unfinished, not in a PR. Please go ahead and merge. 😄

@LucianoPAlmeida Let's get this into 5.3 branch as well.

Note that you'd have to remove castToExpr from there because 5.3 doesn't have that refactoring.

Sure @xedin! :)


Make exit available in scripts

This is purely recreational activity.
The need for exit arose when I tried to add Kamby to this polyglot.
It has a rule that ... the program must exit normally (e.g. by running off the end of the program, or via a command such as exit that performs normal program termination).
The problem is, without exit it is hard to check if this rule is applied.
I can easily check if the program prints the required number, but how do I know if the interpreter reached the end of the program or it encountered error somewhere after print statement?
The esiest way to ensure this is to call exit immediately after puts.
Another solution would be for interpreter to exit with nonzero error code if it encounters fatal error.


Add a qoscfg utility for setting a QoS forwarding group

Pull Request Functional Test Report for #1888 / 5e7edbadf39ef9b9265f57d757fe7bb479ab34c9
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

DP-1.9: WRR traffic testDP-1.3: QoS ECN feature configDP-1.4: QoS Interface Output Queue CountersDP-1.2: QoS policy feature configDP-1.14: QoS basic testDP-1.8: Two strict priority queue traffic testDP-1.11: Bursty traffic testDP-1.10: Mixed strict priority and WRR traffic testDP-1.7: One strict priority queue traffic test

Cisco 8000E

DP-1.9: WRR traffic testDP-1.3: QoS ECN feature configDP-1.4: QoS Interface Output Queue CountersDP-1.2: QoS policy feature configDP-1.14: QoS basic testDP-1.8: Two strict priority queue traffic testDP-1.11: Bursty traffic testDP-1.10: Mixed strict priority and WRR traffic testDP-1.7: One strict priority queue traffic test

Cisco XRd

DP-1.9: WRR traffic testDP-1.3: QoS ECN feature configDP-1.4: QoS Interface Output Queue CountersDP-1.2: QoS policy feature configDP-1.14: QoS basic testDP-1.8: Two strict priority queue traffic testDP-1.11: Bursty traffic testDP-1.10: Mixed strict priority and WRR traffic testDP-1.7: One strict priority queue traffic test

Juniper cPTX

DP-1.9: WRR traffic testDP-1.3: QoS ECN feature configDP-1.4: QoS Interface Output Queue CountersDP-1.2: QoS policy feature configDP-1.14: QoS basic testDP-1.8: Two strict priority queue traffic testDP-1.11: Bursty traffic testDP-1.10: Mixed strict priority and WRR traffic testDP-1.7: One strict priority queue traffic test

Nokia SR Linux

DP-1.9: WRR traffic testDP-1.3: QoS ECN feature configDP-1.4: QoS Interface Output Queue CountersDP-1.2: QoS policy feature configDP-1.14: QoS basic testDP-1.8: Two strict priority queue traffic testDP-1.11: Bursty traffic testDP-1.10: Mixed strict priority and WRR traffic testDP-1.7: One strict priority queue traffic test

Openconfig Lemming

DP-1.9: WRR traffic testDP-1.3: QoS ECN feature configDP-1.4: QoS Interface Output Queue CountersDP-1.2: QoS policy feature configDP-1.14: QoS basic testDP-1.8: Two strict priority queue traffic testDP-1.11: Bursty traffic testDP-1.10: Mixed strict priority and WRR traffic testDP-1.7: One strict priority queue traffic test

Help


Option to select the variable with the highest median when making the index unique

Hi all,
when loading the data, in case of duplicates I usually choose the item with the highest median (e.g. the gene with the highest median signal). With a pandas DataFrame it can be done as easily as this:
df = df.T
df['Median'] = df.median(axis=1)
df = df.sort_values(by=['Median'], ascending=False, na_position='last')
df = df.drop(columns=['Median'])
df = df.groupby(level=1).first()
df = df.T

considering genes in columns and samples in index. I find it more useful than:
https://github.com/scverse/anndata/blob/1c052902a98e04502a36ce46a12b22500a785f50/anndata/utils.py#L10
since you don't change the name of genes. I don't know however how to implement it using AnnData. Are you interested in integrating it in AnnData, maybe also with different options (e.g. average, etc.)?
Thanks,
Francesco

No need to have special support, you can do it something like this:
# raw data
data = scipy.sparse.csr_matrix(...)
obs = pd.DataFrame(..., index=...)  # index has duplicates

# order and deduplicate
order = obs.assign(Median=data.median(axis=1)).idxmax()
adata = AnnData(data[order], obs[order])


Class instance can be passed to a class constructor that does not properly implement the declared interface

Bug Report
Within the following code, I am able to pass an instance of Foo to Bar1's constructor even though Foo's doSomething method is not compatible with the type declared by BarType1.  Effectively, this could be a runtime error if Foo.doSomething() were to expect property b to have been assigned:
interface TestInterface<T> {
	doSomething(param: T): void;
}

type FooType = { a: string; b: string };
class Foo implements TestInterface<FooType> {
	public doSomething(param: FooType): void {}
}

type BarType1 = { a: string }; // subset of FooType
class Bar1 {
	private test: TestInterface<BarType1>;

	public constructor(test: TestInterface<BarType1>) {
		this.test = test;
	}

	public anotherThing(param: BarType1) {
		this.test.doSomething(param);
	}
}

type BarType2 = { c: string }; // disjoint of FooType
class Bar2 {
	private test: TestInterface<BarType2>;

	public constructor(test: TestInterface<BarType2>) {
		this.test = test;
	}

	public anotherThing(param: BarType2) {
		this.test.doSomething(param);
	}
}

type BarType3 = { a: string; b: string; c: string}; // superset of FooType
class Bar3 {
	private test: TestInterface<BarType3>;

	public constructor(test: TestInterface<BarType3>) {
		this.test = test;
	}

	public anotherThing(param: BarType3) {
		this.test.doSomething(param);
	}
}

type BarType4 = { a: string; c: string } // contains some members of FooType as well as additional members
class Bar4 {
	private test: TestInterface<BarType4>;

	public constructor(test: TestInterface<BarType4>) {
		this.test = test;
	}

	public anotherThing(param: BarType4) {
		this.test.doSomething(param);
	}
}

const bar1 = new Bar1(new Foo()); // Shouldn't this be an error?  Property b is missing from BarType1 but it's required in FooType.  I would expect the same error as in the bar4 example below.
const bar2 = new Bar2(new Foo()); // Error "Type 'BarType2' is missing the following properties from type 'FooType': a, b" -- errors as expected
const bar3 = new Bar3(new Foo()); // No error but that seems okay -- the Foo.doSomething() would just get an extra property passed
const bar4 = new Bar4(new Foo()); // Error "Property 'b' is missing in type 'BarType4' but required in type 'FooType'" -- This error looks correct, but I'm not sure why this is an error when bar1 was not.

I'm guessing this is an undesirable although expected behavior in TypeScript since I am able to replicate it as a problem in all versions supported by the TypeScript playground.  However, this behavior seems incorrect to me since TS correctly detects the error in the Bar4 instance.
I reviewed the FAQ about strictFunctionTypes to see if that had any bearing, but the same non-error occurs when I use a function or arrow function which accepts a parameter that is of type TestInterface<BarType1>.
🔎 Search Terms

🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about strictFunctionTypes.

⏯ Playground Link
Playground link with relevant code
💻 Code

interface TestInterface<T> {
	doSomething(param: T): void;
}

type FooType = { a: string; b: string };
class Foo implements TestInterface<FooType> {
	public doSomething(param: FooType): void {}
}

type BarType = { a: string }; // subset of FooType
class Bar {
	private test: TestInterface<BarType>;

	public constructor(test: TestInterface<BarType>) {
		this.test = test;
	}

	public anotherThing(param: BarType) {
		this.test.doSomething(param);
	}
}

const bar = new Bar(new Foo()); // no error

🙁 Actual behavior
Construction of bar1 does not emit a type error.
🙂 Expected behavior
Construction of bar1 should emit Property 'b' is missing in type 'BarType1' but required in type 'FooType'

Methods are bivariant; this is in the FAQ. You can write
interface TestInterface<T> {
	doSomething: (param: T) => void;
}

to get the expected error

Method parameters are bivariant; this is in the FAQ. You can write

Thanks for the quick response.  I missed the word property in the FAQ and was testing arrow functions and function declarations, but they also had the same behavior because the interface remained unchanged.  In looking again at the FAQ I understand it more clearly now.


fctx_draw_commands starts failing sometimes

I have a piece of code that gets called on the minute, determines a resource ID, gets the path via fpath_create_from_resource then draws it using fctx_draw_commands.
sometimes the drawing part falls apart, even if the resource ID never changes.
Here's the piece of code
uint32_t weather_icon_resourceid = weather_icons_day[1].resourceid;
if (w_daynight == 1) {
fctx_set_offset(&fctx, FPointI(122+weather_icons_day[w_icon].offsetx-(temp_offset/30), weather_icons_day[w_icon].offsety));
weather_icon_resourceid = weather_icons_day[w_icon].resourceid;
} else {
fctx_set_offset(&fctx, FPointI(122+weather_icons_night[w_icon].offsetx-(temp_offset/30), weather_icons_night[w_icon].offsety));
weather_icon_resourceid = weather_icons_night[w_icon].resourceid;
}
fctx_set_scale(&fctx, FPointI(180,180), FPointI(168,168));
APP_LOG(APP_LOG_LEVEL_INFO, "weather icon resource id is %lu", weather_icon_resourceid);
weather_icon_path = fpath_create_from_resource(weather_icon_resourceid);
fctx_draw_commands(&fctx, FPointZero, weather_icon_path->data, weather_icon_path->size);
and here are two logs
[22:42:01] main.c:237> Redrawing screen
[22:42:01] main.c:339> weather icon resource id is 9
[22:42:01] main.c:388> Steps: 0
[22:43:01] main.c:237> Redrawing screen
[22:43:01] main.c:339> weather icon resource id is 9
[22:43:01] main.c:388> Steps: 0
[22:44:01] main.c:237> Redrawing screen
[22:44:01] main.c:339> weather icon resource id is 9
[22:44:01] main.c:388> Steps: 0
[22:45:01] main.c:237> Redrawing screen
[22:45:01] main.c:339> weather icon resource id is 9
[22:45:01] fctx.c:887> invalid draw command "
[22:45:01] main.c:388> Steps: 0
[22:46:01] main.c:237> Redrawing screen
[22:46:01] main.c:339> weather icon resource id is 9
[22:46:01] fctx.c:887> invalid draw command "
[22:46:01] main.c:388> Steps: 0
[22:47:01] main.c:237> Redrawing screen
[22:47:01] main.c:339> weather icon resource id is 9
[22:47:01] fctx.c:887> invalid draw command "
[22:47:01] main.c:388> Steps: 0
[22:48:01] main.c:237> Redrawing screen
[22:48:01] main.c:339> weather icon resource id is 9
[22:48:01] fctx.c:887> invalid draw command "
[22:48:01] main.c:388> Steps: 0
(once it starts happening this one will go on and on failing and not draw any icon, in the basalt emulator)
[22:47:04] main.c:237> Redrawing screen
[22:47:04] main.c:339> weather icon resource id is 9
[22:47:04] main.c:388> Steps: 21
[22:48:03] main.c:237> Redrawing screen
[22:48:03] main.c:339> weather icon resource id is 9
[22:48:03] main.c:388> Steps: 21
[22:49:03] main.c:237> Redrawing screen
[22:49:03] main.c:339> weather icon resource id is 9
[22:49:03] ault_handling.c:74> App fault! {b4d23fe5-030c-4a72-99e6-a18e84554cf4} PC: 0x1418 LR: 0x806b021
Program Counter (PC)    : 0x1418     /Users/peter/pebble/PJ-Business/watchface/build/../src/main.c:341
Link Register (LR)      : 0x806b021  ???
(this one sporadically happens and will restart the watchface, on actual basalt hardware with firmware 3.9.1)
both of these happen with SDK 3.9
the FPath has been defined at the top of main.c in this line
FPath *p_bt_icon, *p_tick_mark, *weather_icon_path;
attached is the fpath behind resource id 9
0910dn.fpath.zip

Nevermind. I'm a dummy and just now learned about fpath_destroy(). If that solves my problem I'm closing this issue

That was it, I'll show myself out


CSB-70 Develop camel-ftp

Issue: https://issues.redhat.com/browse/CSB-70
Backport of 9e6d282c49c8427664e0cb0f57c23ed27b91df63
This PR contains upgrade of camel to version  3.14.2.redhat-00034, which contains 900bcc8ca5a1bb01e590c49758afb811c48cd574 (part of https://github.com/jboss-fuse/camel/pull/1956/), which is necessary for the tests for sftp.

Failed test is reported as https://issues.redhat.com/browse/CSB-457

I'm merging the PR because the failure is not caused by this PR.


Custom Seed Template from Config

Hello!
We are using Phinx for database migration and default template fits not so good in our corporate standards.
I'm aware of the following issue: seed template #1507, but the way of passing custom seed template as console argument is not so convenient. I would even call it redundant, because we have a single template for the whole project and so on.
I would like to try to add possibility to change the Seed Template File through Config.
I suggest to make the following priority of using the custom template:

Passed as a console option
If console option is not set, then attempt to take it from the Config. If set - then using console option
If nothing from Config either - then using Hardcoded Default Seed Template

One potential problem could be if I change src/Phinx/Config/ConfigInterface.php by adding a new method and change one class in the repository which implements it, project should not be broken, but the customers, if they for some reasons implemented that interface, their software could be broken until they implement it. To be honest, I don't see any reasons to implement this interface itself instead of Config-class.
The project is not designed to use custom Config-classes, so I see the only way to add this feature is to change ConfigInterface and Config, so just for circumstances I started this issue before I started the working on it, so please, give me short feedback about it while I'm implementing in order to increase the speed of accepting my Pull Request, which I will add as soon as I finish the changes.
Thank You very much for this nice project.

Implementation is done, just opened the pull request: https://github.com/cakephp/phinx/pull/2146

Ok, I don't understand how the review mechanism is working, so I decided to reopen this issue, maybe it will help somehow...
This is the pull-request: https://github.com/cakephp/phinx/pull/2146, which has been reviewed by @dereuromark, I made all changes as he suggested, but now I don't know what to do, I asked him to review again, but my re-review request is still pending...
Maybe someone else could review and accept it, please?


APEXMALHAR-1986 Change base for semantic version check to 3.3 release.

@tweise, @chandnisingh Please review and merge.

@vrosov please add this to the release process doc.


chore(buildpacks): update heroku-buildpack-java to v44

See https://github.com/heroku/heroku-buildpack-java/compare/v43...v44 and deis/deis#4972.

I tested this with the workflow-beta1 chart and example-play.


FirebaseObjectObservable is deprecated but it still is used on this doc page https://github.com/angular/angularfire2/blob/master/docs/rtdb/objects.md

it's confuse, since the documentation in the link on the issue titles it still is use FirebaseObjectObservable instead of  AngularFireList<T> as is mentioned on this link https://github.com/angular/angularfire2/blob/master/docs/version-5-upgrade.md
here an image what I'm getting when I try to use AngularFireList<T>  to retrive an object

please let me know how can I solve this issue?
thank you so much.

Let's try to make some points here:

When working with the real-time database, version 5 is not using FirebaseListObservable and FirebaseObjectObservable anymore, instead, now it uses AngularFireList for lists and AngularFireObject for objects.
db.object(.....) will give you back an AngularFireObject; db.list(.......) and AngularFireList. So you could do AngularFireObject<any> author = db.object('authors').
if you add ".valueChanges()" afterwards, directly in the same statement, you are basically getting the Observable (whether it is a list or an object) of that AngularFireObject/List to which you can subscribe then. Observable<any> author = db.object('authors').valueChanges()
On the other hand, If you just get 'authors', it seems to me like you want to get a List, not an object, the whole list of authors, as you are not specifying any authorId to get just one... AngularFireObject<any> author = db.object('authors/${authorId}') or AngularFireList<any> authorsList = db.list('authors')
It is also a good practice if you previously define an interface or a class "Author" with correspondent attributes so you can do AngularFireObject<Author> instead of <any>
Another suggestion just so you know, your lines 1 and 3 could be merged together as import { Component, OnDestroy } from '@angular/core';

I hope this answer is readable and helpful for you:)

export class AppComponent{
authors:AngularFireObject;
constructor(db:AngularFireDatabase){
this.authors=db.object('/authors/1')
}
}
{{ (authors | async | json)?.name }}
I am also having a similar problem, trying to display the name of the author but nothing gets displayed and i get the InvalidPipeArgument argument

    {{authors.name |async | json}}

I also tried this format but still get nothing and the documentation seems to be outdated when it comes to using AngularFireObject

Hi @kaygeeklaas,
Maybe in the "get started" section is not fully updated, but here you have the documentation in details, all fine :)
real time db docs updated
Anyway here is the solution to your problem:
db.object will return an AngularFireObject indeed, but what you need to display on the view is an Observable (or Observable in case you don't want to define the object); to get an observable out of an AngularFireObject you need to call .valueChanges(), then your binding to the view for the authors list will work with the async pipe correctly.
`export class AppComponent{
authors:Observable;
constructor(db:AngularFireDatabase){
     this.authors=db.object('/authors/1').valueChanges();
}

}
{{ authors | async }}`
I would encourage you to read a bit more of information about Observables in order to fully understand them :)

I've been struggling with firebase angular and observables a couple of days and this really helped me a lot.  Thank you so much @javieraviles!

Im not getting any output whatsoever from my database. Anything wrong im doing. i followed this documentation but to no avail: https://github.com/angular/angularfire2/blob/master/docs/rtdb/lists.md


add onPageUpperMargin

Closes #71

Implements a top margin when an additional page is triggered by a long table


Prepare <IP_ADDRESS>, extend CI

I guess we have to wait for ghcup to contain the GHC versions we want to test this again.


graph stops drawing sometimes

I'm often working with a sankey and sometimes it will just refuse to draw:

i can mess around with the params and eventually it will draw again - eg with 21 connections instead of 20. It seems to somehow give the renderer a kick.
This looks to me like the CSS isn't loading or something? I can only guess there's some loading order issue to the widget? Are you doing a server-side render of the whole D3 graph?
In this case when I save a png it also comes out the same.

I'm also getting the same error.
here is my code:
import os
import pathlib
import pandas as pd

import floweaver as fw
from ipysankeywidget import SankeyWidget

os.chdir(os.getcwd())

PATH_DATA = pathlib.Path(r'./storage/shared/Documents')
PATH_RESULTS = pathlib.Path(r'./storage/shared/Documents')

if not PATH_DATA.exists():
    raise ValueError("Data directory not found.")

df = pd.read_csv(PATH_DATA/'flow.csv', low_memory=False)

source_set = set(df['source'])
target_set = set(df['target'])
trunk = source_set - target_set
leaves = target_set - source_set
branches = (source_set | target_set) - trunk - leaves

SankeyWidget(links=df.to_dict('records')).auto_save_png(filename=str(PATH_RESULTS/'sankey_simple.png'))

the csv file:
source,AMOUNT,target
wages,75000,income
interest,30,income
other,5000,income

income,5,expenses
income,6000,liabilities
income,300,loans
income,200,savings

expenses,1000,Education
expenses,1999,Insurance
expenses,123,Medical
expenses,240,utilities

liabilities,745,Chase
liabilities,137,Discover

loans,2500,Education
loans,17000,Mortgage
loans,6000,Car

and the result


Improve error handling on client/server

Avoid using Fatalf and handle errors more gracefully
Refactor log.Info levels more appropriately

Changed error commit: 11db8690229889d79ca58b63f4e49deff67e6544


Uncaught Error: Could not find module ember-getowner-polyfill...

I get the following error after installing and making it to Step 5: Profit:

Could not find module ember-getowner-polyfill imported from ember-router-scroll/index

Once I manually installed ember-getowner-polyfill it worked fine.

https://github.com/dollarshaveclub/ember-router-scroll/blob/master/package.json#L75 << this should be in dependencies and not devDependencies. Would you mind creating a PR for that?

This seems to be fixed on master


Want to add contributors profiles and also update How to Contribute section in README.md

please assign me this task

Sure @Sowham-3098 , go ahead!


Adding the arabic keyboard (macos layout)

The Arabic keyboard layouts are different across major operating systems. This one is based on macos.

I signed it

@salsowelim, could you try out the arabic-mac extension I just added in the arabic-mac branch. I am not familiar with the Arabic language or keyboards so I do not know if the version that uses is exactly what you're looking for.
https://github.com/google/extra-keyboards-for-chrome-os/tree/arabic-mac/arabic-mac

@salsowelim did you ever try out the extension from the arabic-mac branch (664ba11f0ae580e35684b8c33712e0c818550bd9)? If I do not hear back soon I'll close this PR and merge that branch into the master branch.

I am going to close this as I believe PR #96 should work.

I am going to close this as I believe PR #96 should work.


Can not reveal evaluation result on detectron2

Hi, I am loaded YOLOX model and eval in detectron2, I found my result have a huge gap:
Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.313
Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=100 ] = 0.476
Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=100 ] = 0.339
Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.204
Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.363
Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.371
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=  1 ] = 0.260
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets= 10 ] = 0.389
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.398
Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.252
Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.442
Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.476
[08/11 13:19:02 d2.evaluation.coco_evaluation]: Evaluation results for bbox:
|   AP   |  AP50  |  AP75  |  APs   |  APm   |  APl   |
|:------:|:------:|:------:|:------:|:------:|:------:|
| 31.257 | 47.647 | 33.920 | 20.407 | 36.334 | 37.144 |

Result seems so different. Why?

@luohao123 I think you didn't correctly load all variables from YOLOX ckpt. Please check whether your variable definition is fully consistent with those in the original YOLOX and whether you left some important variables behind.
Also, you may try strict=True when calling model.load_state_dict().

Can you write your specific command of evaluation?

Hi, I want to learn about how to load YOLOX model in detectron2, I haven't found a YOLOX implementation in detectron2.


Add media description feature using Azure Content Understanding

Purpose
This PR adds a new optional feature that will extract figures in documents (using Azure Document Intelligence) figures output mode and send those figures to Azure Content Understanding (a new service that uses multimodal models) to generate a figure description. It will then insert that figure description into the content, which will then get sent for chunking. If the figure is of a graph or chart, it will include an HTML table with the data.
This gives developers a more lightweight approach to ingest media-rich documents and can be compared to the more heavyweight GPT-4-vision approach.
Does this introduce a breaking change?
When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.
[ ] Yes
[X] No

Does this require changes to learn.microsoft.com docs?
This repository is referenced by this tutorial
which includes deployment, settings and usage instructions. If text or screenshot need to change in the tutorial,
check the box below and notify the tutorial author. A Microsoft employee can do this for you if you're an external contributor.
[ ] Yes
[X] No

Type of change
[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Code quality checklist
See CONTRIBUTING.md for more details.

[ ] The current tests all pass (python -m pytest).
[ ] I added tests that prove my fix is effective or that my feature works
[ ] I ran python -m pytest --cov to verify 100% coverage of added lines
[ ] I ran python -m mypy to check for type errors
[X] I either used the pre-commit hooks or ran ruff and black manually on my code.

I need to do some cleanup, per the CI. This also needs a few more tests, as it currently only has a test for the changes to the splitting algorithm.

I have now completed both unit testing and manual testing, and this is good to merge. We will merge/release on Monday.


3.8.x continuous matrix testing

This is a patch to the 3.8.x branch to fulfill the goals for continuous matrix testing for the 4.0-era driver, which was 3.8.0.  Similar PRs will be done for the 4.2 and 4.4 era release branches.   What I've done here is two things:

Modify or skip tests that will not pass against later server versions.  Mostly because the features under test have been removed
In cases where the server made breaking changes to commands that prevents the driver code from working, I changed the driver code to make it work with the later server versions.  This seemed easier than disabling all the tests of that feature, though that is another option we could consider.  Look at the changes to MapReduceToCollectionOperation for an example of this.

There's nothing really new here.  All these changes have been applied already in one form or another to the master branch of the driver.  It was just a matter of finding them and copying them to this branch.
JAVA-3942
Full patch build: https://spruce.mongodb.com/version/60133b4d9ccd4e019373ad0d/tasks?sorts=STATUS%3AASC%3BBASE_STATUS%3ADESC

Pushed to https://github.com/mongodb/mongo-java-driver/tree/3.8.x-continuous-matrix-testing


cp: move copy_on_write funcs to platform module

Move the copy-on-write functions for cp to their own module. This provides a layer of indirection so that the cp.rs module need only use platform::copy_on_write(), and the platform module is responsible for providing the appropriate implementation for the current platform. This commit does not change the behavior of the code, just its organization.

Could you please fix the conflict? Sorry :(

It is updated.


False atom fix

Fixes issue #155  and adds a test case for it.
I short the false atom check did only validate the first 4 letters of false because it was using a 32 bit value to validate against. Another issue was that the false letters were padded with spaces, not nulls.
This approach happened to work for 'true' and 'null' by accident since they are 4 letters each and fit in a 32 bit value.
this PR changes the false parsing to use a 64 bit error and null out the data after the false letters instead of spacing them out.

Looks great... before I merge, can you fix the typo ("We have ot"). :-)

(Your code is better than the original btw, even this bug aside.)


AT_LEAST_ONE doesn't include any children.

I created a minimal repro:
https://gist.github.com/joelburget/39199fefa7401ad3ff4f2654c5b58393
Output:
parser.test.ts:
Warning: Warning: No LINE_BREAKS Found.
	This Lexer has been defined to track line and column information,
	But none of the Token Types can be identified as matching a line terminator.
	See https://chevrotain.io/docs/guide/resolving_lexer_errors.html#LINE_BREAKS
	for details.
{
  literal_list: [
    {
      name: "literal_list",
      children: {}
    }
  ]
}
69 |
70 |   return visitor.visit(cst);
71 | }
72 |
73 | test("expr", () => {
74 |   expect(() => parseExpr("1")).not.toThrow();
      ^
error: expect(received).not.toThrow()

Error name: "TypeError"
Error message: "undefined is not an object (evaluating 'ctx.literal_list.map')"

      at /Users/joel/code/del/parser.test.ts:74:2
✗ expr [1.98ms]

As you can see, children: {}. Shouldn't there be children?

Your literal_list rule is invalid:
-this.AT_LEAST_ONE(this.literal);
+this.AT_LEAST_ONE(() => this.SUBRULE(this.literal));

That should resolve the issue.


Allow for current user only installation (Install without Administrator access)

More people can use the program if this is implemented and it doesn't cause any issues.

Inno documentation specifies "Do not use this setting unless you are sure your installation will run successfully on unprivileged accounts." Can this be confirmed that Moonscraper won't have issues from this?
https://jrsoftware.org/ishelp/index.php?topic=setup_privilegesrequired
Alternatively has PrivilegesRequiredOverridesAllowed been considered? It results in this dialog instead which may be more preferred.

Inno documentation specifies "Do not use this setting unless you are sure your installation will run successfully on unprivileged accounts." Can this be confirmed that Moonscraper won't have issues from this? https://jrsoftware.org/ishelp/index.php?topic=setup_privilegesrequired
Alternatively has PrivilegesRequiredOverridesAllowed been considered? It results in this dialog instead which may be more preferred.

Sorry, I'm mostly unfamiliar with Inno setup, but I think the PrivlegesRequriedOverridesAllowed is a better choice.

This would need investigation into what exactly are the restrictions with unprivileged accounts before merging in to ensure that installation will be successful.

For an unprivileged account, they cannot edit or create any folders or files in the base Program Files/x86 and most subfolders. They may create or edit files/folders in their own AppData folder, so I suggest an unprivileged account to install in AppData/Local or AppData/Roaming (I don't remember the difference so I can't say such one to use for sure)


[e2e tests]Add Labels to expose dependencies of tests

In some use-cases a user might want to test
KubeVirt within a cluster that is different
from kubevirt-ci therefore we should expose
the tests dependencies.
Please note that this PR goal isn't to replace all the
skips in func tests but to take small steps to this goal.
Signed-off-by: bmordeha<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/approve
Thanks @Barakmor1, really nice to see this!

/retest-required

Great work @Barakmor1!
/lgtm
/retest-required

/cherrypick release-0.58


Develop

Changes made to the old BERT pretraining code

Hi team, I had to make changes to the BERT pretraining to work. I had to upgrade the mosaicml version, add mlflow and many minor code changes to match the signature


[Tooltip] Hide native title when disableHoverListener is true

[x] I have followed (at least) the PR section of the contributing guide.

Fixes #13346

@joshwooding Thank you :)


Support for setting WM_CLASS on a per-window basis

The current methods of setting WM_CLASS are:

Use the title set a creation time. Does not allow setting the name and class parts of WM_CLASS
Use an init hint. Does nto allow setting different WM_CLASS for different windows in the same application.

My proposed method addresses both od these shortcomings, by encoding the WM_CLASS and title values into a single string demarcated by ASCII start-of-header and record separator bytes (which do not occur in the utf-8 encoding). If the title does not start with start-of-header, the old behavior from (1) is used.

I believed we had agreed to the current glfwInitHintString solution. If that's insufficient then I need to remove it before 3.3 release and replace it with something else.
What are you planning to set WM_CLASS to that will vary between windows?

I just finished adding support for daemon mode to kitty. In which the first kitty window becomes the master and subsequent invocations cause the master to open new windows. The new windows can have different name/class values. I apologize for the change, but at the time that I thought the original fix was sufficient, I had not yet conceived of the daemon mode.

@kovidgoyal Would 79295048b2d049d54261eeffbd535749470908a0 work for you?

Looks good to me, thank you.

@Spasi How bad an idea would it be for me to merge 79295048b2d049d54261eeffbd535749470908a0 considering LWJGL is already shipping and exposing glfwInitHintString?

@elmindreda It's not a problem. GLFW functions are annotated with the version they were introduced. We're shipping a pre-release build of GLFW 3.3 and glfwInitHintString was added in that version, so we are not bound by any compatibility requirements.


Finish the 'Add Gradebook Item' window

We didn't have the backend services for this, nor the correct code. Hook this all up based on the edit page.

Done in PR #150 and #157.


support for version 2.x

Modified ConfigurationPanel
Added JenkinsVersion enum
Changed RequestManagerInterface.authenticate method singature - added JenkinsVersion parameter
Adapt Security package and JenkinsSettings, with changes

Looks good to me @dboissier


Add live tests that update the Named key value

Would it be possible to have a live test where we actually update the key after the client is already constructed and after we've already done some operations with the initial key? Maybe we could just update it to use the secondary key from the namespace.
Originally posted by @JoshLove-msft in https://github.com/Azure/azure-sdk-for-net/pull/19785#discussion_r600875650

We can use the ARM package to invalidate the primary key and regenerate keys as needed.

After 2 years, I think it's safe to assume that we don't plan on doing this.   closing it out.


Dockerfile instructions for run -i and -t

Right now every Dockerfile has to be distributed with a run script or a readme that documents whether it needs to be run with -it, -i or neither.
There should be a way to store this in the image by an instruction in the Dockerfile.
There was already an issue #1870 for this, but it was closed in favor of #4955 and then in turn #4882. I think this was by accident because the MR is not actually related to this feature, it just removes a buggy, incomplete implementation of a similar feature.

I understand your thinking, but I don't think we should do this. I think we should keep "runtime" configuration separate from the image. Although some images may require you to run them interactively, most images can actually be used in multiple ways. For example, the MySQL image can both be used as a server (non interactively), or as a client (interactively). The image should not dictate how to use it, and specifying that a TTY or "interactive" is required, would limit the usefulness of such an image.
Adding this would be a sliding slope; should we add Dockerfile instructions for the image requiring certain environment variables to be set, a network connection, a different container to be started first?
For runtime configuration, docker-compose can fulfill most of those things; providing a (sample) docker-compose.yml with the right settings, or just good documentation for the image is probably a better approach.

I see your point. The difference to network and dependency etc. options is that -i and -t feel like workarounds. Ideally they would just work automatically, but I'm not sure that's possible.
In fact I haven't looked into using docker-compose for this, I should do that. Maybe we should rename dockerto docker-client and docker-compose to docker then. :)
For now I have resorted to running an SSHd in my containers anyway, because the docker terminal is extremely unreliable, with its random crashes (#22345) and broken arrow keys (because the terminal ignores some ESC sequences) and such.

For now I have resorted to running an SSHd in my containers anyway, because the docker terminal is extremely unreliable

Yes, I think there's still issues around terminal emulation on Windows


Vision: Figure out how to use apriltag with realsense

[x] Setup camera node

[x] Run apriltag example

[x] Publish apriltag tf

[ ] Maybe: an example file on extrinsic calibration of the whole setup

target issue: https://github.com/wengmister/Apex-Putter/issues/3


Change background of site to be milkyway/nebula space/a void

Trying to use this website as a reference to include 3D models and more professional backgrounds: https://github.com/adrianhajdin/project_3D_developer_portfolio/tree/main


Run shellcheck on all shell scripts in scripts/

A bunch of changes to get shellcheck to not complain on common/ too much.
Currently I have only tested that make vault-init kept working
- Fix bash linting issues

Fix quoting issue in examples/kustomize-renderer/kustomize
Fix some shellcheck issues in scripts/secret.sh
Disable SC2181 shellcheck test
Drop unused ns and gitops vars
Quote password var
Fix a bunch of quoting in scripts/test.sh
Disable SC3037 in scripts/secret.sh
Fix SC2166 in scripts/test.sh
Shellcheck fixes for scripts/vault-token.sh
Fix most shellcheck warnings in vault-utils.sh
Disable SC2016 as we know what we're doing
Fix SC2013: To read lines rather than words, pipe/redirect to a 'while read' loop in vault-utils

Let's hold off on merging this until I tested a few more things.


Dev-server page isn't rendered

Bug description
After installing all dependencies and running dev-server, I get broken page at http://localhost:9000.
I've found the same very old issue #12040. The author supposed that it is specific to Chrome. But both Chrome and Firefox show broken page. Also it's worth mentioning that, as far as I see, there are no related errors in terminal or console output.
How to reproduce the bug

Install Superset without Docker

source venv/bin/activate
pip install -r requirements/development.txt
pip install -e .

cd superset-frontend
npm ci

Run Superset

export SUPERSET_CONFIG_PATH=./docker/pythonpath_dev/superset_config_local.py
superset run -p 8088 --with-threads --reload

Run development server

npm run dev-server

Navigate to http://localhost:9000

Screenshots/recordings

Superset version
master / latest-dev
Python version
3.11
Node version
18 or greater
Browser
Chrome
Additional context
node.js version: 18.20.4 (same error on 22.5.1)
Checklist

[X] I have searched Superset docs and Slack and didn't find a solution to my problem.
[ ] I have searched the GitHub issue tracker and didn't find a similar bug report.
[X] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

@dosu Nope, it didn't work.

不，它没有用。

I tried the same thing and it didn't work

@goldjee hi bro, I use safari and it's working fine

@monsterkai233 Hi! Good tip, Safari works. But I'm much more used to Chrome dev tools, so it would be great if maintainers could take a look.
@rusackas @michael-s-molina

Hi @goldjee. I also noticed this problem but haven't had the time to take a look. Hope other maintainers can fix it. In the meantime, I discovered that it works on Chrome if you use the IP address instead of localhost.

Let me tag @geido @rusackas as the leaders of dev experience group.

Hi @goldjee. I also noticed this problem but haven't had the time to take a look. I hope other maintainers can fix it. In the meantime, I discovered that it works on Chrome if you use the IP address instead of localhost.

Hi! Thank you for the suggestion. Just tried http://<IP_ADDRESS>:9000 and had the same result :(

Not <IP_ADDRESS>, your real IP such as 192....

Ah, I see. Yep, internal LAN IP worked.

I have got around that b6 downgrading to node 16.20. Superset isn't heavily tested on Node 18.

I have got around that by downgrading to node 16.20. Superset isn't heavily tested on Node 18.

@geido I'm able to reproduce this issue with Node v16.20.2

Node 20 doesn't seem to magically fix this. Still looking for a good fix...

Setting COMPRESS_REGISTER = False in superset config has helped some people getting around this issue.

Fixed! Closing this thanks to https://github.com/apache/superset/pull/30034


[WIP] Auracast support

I realize we can use Python 3.10 builtin type operators in Python 3.9 by importing future.__annotations__?
https://stackoverflow.com/questions/76579085/python-3-9-use-or-operator-for-union-types


Bug 1985125: fix(og): Fix missing MultiOperatorGroups condition in some cases (#2305)

In some special cases, the MultiOperatorGroups condition is missing
even though there are multiple OGs in the same namespace. The process
of adding this condition happens during syncNamespace which sometimes
doesn't happen if syncOperatorGroups fails prematurely due to other
errors. Moving the MultiOperatorGroups condition to syncOperatorGroups
to ensure it will be run everytime.
Upstream-repository: operator-lifecycle-manager
Upstream-commit: d140ef062304440b37712692a170a93b3dc76fba
Signed-off-by: Vu Dinh<EMAIL_ADDRESS>

/lgtm

/retest

/retest


teacher tool: persist user feedback

Store user feedback in state so that it isn't cleared when the feedback control unmounts for any reason.
Feedback state will be programmatically cleared when:

The criteria evaluation returns a different value for notes
The criteria fails to evaluate.

When feedback is given, the text label now changes to "Thanks for your feedback!". Because of this, I removed the toast that appears with the same message.

Fixes https://github.com/microsoft/pxt-microbit/issues/5815
Fixes https://github.com/microsoft/pxt-microbit/issues/5831

@thsparks I changed the behavior of the control to disallow multiple clicks. If the user clicks "helpful" they will no longer be able to change their response. Though it's fairly easy to reset manually. Running the AI criteria again will likely return slightly different results. Let me know if you preferred the other way. I can put it back.

@thsparks I changed the behavior of the control to disallow multiple clicks. If the user clicks "helpful" they will no longer be able to change their response. Though it's fairly easy to reset manually. Running the AI criteria again will likely return slightly different results. Let me know if you preferred the other way. I can put it back.

I think it's okay. I have a habit of accidentally clicking buttons like this and then wanting to reset/change it, which is why I kept the functionality. But it's probably not a universal experience :)


Cleanup makefile

📝 Summary

Add all and clean targets.

I'm sure I'm not the only one that's tried just running make to build things.
Using git clean -fdx instead of manually specifying files as we already depend on git for the version.

Set fake targets (happens to be all of them) as phony (see reference).

Not really that important, but I think it's good practice.

Delete run and run-dev rules. I don't think these make sense anymore.

mev-boost has required options (network) now.

Delete the generate-ssz rule. This has been moved to go-boost-utils (see reference).

Took me a few minutes to realize that the types directory moved repos.

Set dependencies for docker rules. Should be able to just do docker-push.

Didn't actually test this part, so please double-check.

⛱ Motivation and Context
I like Makefiles and I wanted to clean things up a bit.
📚 References

https://www.gnu.org/software/make/manual/html_node/Phony-Targets.html
https://github.com/flashbots/go-boost-utils/blob/417a64679a5d4a2c12b64ff7ee918243d4ab34eb/Makefile#L23-L25

✅ I have run these commands

[x] make lint
[x] make test
[x] make run-mergemock-integration
[x] go mod tidy

Thanks a lot, these are great! 👍
Left 2 suggestions, and since merging the other PR this needs to be rebased :)

Ah you're right. Thanks for pointing that out. Please review again.

very nice, thank you!


Add type="constant"

It would be great if we could add a node to the model without creating a bind.

type
label
default

constant
c
5

Right now to do this, we have to create a calculated item. We cannot directly add a constant to the model.
I think this might be primarily useful for developers, to create test forms.
Thoughts?
Should this feature never add a <bind> and therefore ignore all other XLSForm columns? A bind is not required according to the XForms spec.

This use case is already covered by type="hidden" (existing feature) as discovered by @alxndrsn


Implement math.ulp

CI fails with
error[E0463]: can't find crate for `serde_derive` which `num_bigint` depends on
Error:  --> bytecode/src/bytecode.rs:6:5
  |
6 | use num_bigint::BigInt;
  |     ^^^^^^^^^^ can't find crate

Is this related to anything I'm doing? It seems unlikely to me.

No, that's an existing problem related to cache stuff on macos; nothing to do with your changes.

Hmm, looks like some complex tests were enabled that were previously skipped by requires_IEEE_754. Do you want to add skips for those manually?

Done.
I think in general we should use expectedFailure instead of skip, since implementing feature A in one part of the codebase may cause tests for features B and C in other parts of the codebase to run successfully. If the tests are skipped, we may never know.

Oh wow, I just realized that you can still have expectedFailure if the test is throwing an error. But yeah I definitely agree, we should mention in the development document that it's preferred to do expectedFailure, if the test isn't hanging/panicking


Cursor alert invalid key

Start Server Command
OPENAI_API_KEY=sk-123456 deno run -A jsr:@ryoppippi/curxy
API ACCESS NORMAL
curl https://pn-circus-parental-should.trycloudflare.com/v1/chat/completions -H "Content-Type: application/json" -H "Authorization: Bearer sk-123456" -d '{
"messages": [
{
"role": "system",
"content": "You are a test assistant."
},
{
"role": "user",
"content": "Testing. Just say hi and nothing else."
}
],
"model": "qwen2.5-coder:7b"
}'
{"id":"chatcmpl-603","object":"chat.completion","created":1726741526,"model":"qwen2.5-coder:7b","system_fingerprint":"fp_ollama","choices":[{"index":0,"message":{"role":"assistant","content":"Hi"},"finish_reason":"stop"}],"usage":{"prompt_tokens":28,"completion_tokens":2,"total_tokens":30}}
Cursor ERROR

CURSOR INFO
Version: 0.40.4
VSCode Version: 1.91.1
Commit: 63dae81aea2c3fc079420fe72578a8eef26ba870
Date: 2024-09-05T02:52:05.660Z
Electron: 29.4.0
ElectronBuildId: undefined
Chromium: 122.0.6261.156
Node.js: 20.9.0
V8: 12.2.281.27-electron.0
OS: Darwin arm64 24.1.0
THX!

Confirmed. Cursor uses HTTP OPTIONS requests to identify supported HTTP methods, which are not supported by Ollama. This issue needs to be addressed in Curxy, as the original problem https://github.com/ollama/ollama/issues/3746 persists.

@pedh Thank you for your debugging!


Implement address scanning on initial load

On first load on a new device, check for addresses in batches of 5 to look for addresses that have a balance. The search continues until we find a batch where all the addresses in it have no balance.
Fixes #8
Preview URL: https://kasvault-git-init-scan-coderofstuffs-projects.vercel.app/
https://github.com/user-attachments/assets/b63d5574-243e-49e4-90b9-fc2d3da1e3eb


Fix/isbasemodel

Adding simple variable check.
This check will fail if initialValue is null, when it tries to retrieve _isBaseModel of null.
typeof null
// object

can you please merge upstream or something... it says 40 commits. it's probably just 1?


Backport: Fixed #29182 -- Fixed schema table alteration on SQLite 3.26+.

SQLite 3.26 repoints foreign key constraints on table renames even when
foreign_keys pragma is off which breaks every operation that requires
a table rebuild to simulate unsupported ALTER TABLE statements.
The newly introduced legacy_alter_table pragma disables this behavior
and restores the previous schema editor assumptions.
Thanks Florian Apolloner, Christoph Trassl, Chris Lamb for the report and
troubleshooting assistance.
Backport of c8ffdbe514b55ff5c9a2b8cb8bbdf2d3978c188f from master.

This patch doesn't qualify for a backport. Please check also the discussion starting at https://github.com/django/django/pull/10733#issuecomment-456731434.

FWIW, this issue makes it impossible to test libraries supporting both Django 1.11 and Django 2.2 on a single Centos server:

Centos 7 supports sqlite3 3.7.17 (see EL-7 here)
Centos 8 supports sqlite3 3.26.0 (see EL-8 here

Our team runs internal libraries on Centos servers using tox to test against various versions of dependencies. The current version incompatibilities mean that Django 1.11 tests of a library must run on Centos 7 and tests for Django 2.2 must run on Centos 8.

Applied at phpMyAdmin on our GitHub actions CI !
Thank you so much for the patch !
See: https://github.com/phpmyadmin/website/commit/644136cf6675bd9cf03ef3756c4bcfe4ed7dd1ac
Logs: https://github.com/phpmyadmin/website/runs/1835844702#step:5:16


6.16.0 releases is buggy - Please take a look

Related

Thanks for the report, we'll be looking into it shortly

Releasing v6.16.1 until we actually get the time to figure this out.

There is now an open PR for react-native.
https://github.com/facebook/react-native/pull/12550
with 7.0 we reintroduced the change that was breaking react-native.


[FEATURE] Support for Reciprocal Rank Fusion Retriever with Azure OpenAI's GPT-3.5 Turbo Instruct in Flowise

Describe the feature you'd like
I am requesting support for implementing Reciprocal Rank Fusion Retriever using Azure OpenAI's GPT-3.5 Turbo Instruct model within Flowise. The current Azure OpenAI Node in Flowise does not list the GPT-3.5 Turbo Instruct model, which is essential for advanced retrieval-based tasks and has replaced the text-davinci-003 model.
Additional context
The GPT-3.5 Turbo Instruct model is specifically designed to efficiently handle instruction-based prompts and deliver concise responses, which are key capabilities for the Reciprocal Rank Fusion Retriever methodology. Enabling Chat completion with this model would greatly improve the information retrieval process within Flowise, allowing users to develop more complex and effective AI-powered solutions.
Updating Flowise to include Azure OpenAI's GPT-3.5 Turbo Instruct model is critical to ensure that users can leverage the most up-to-date AI functionalities. This update would not only keep Flowise in line with Azure's evolving AI services but also expand the platform's capabilities in terms of retrieval accuracy and performance.
I believe that adding this support is crucial for users looking to utilize state-of-the-art AI models for sophisticated information retrieval and would appreciate your consideration in making this feature available.
Thank you for evaluating this feature request.

its already available here - https://github.com/FlowiseAI/Flowise/tree/main/packages/components/nodes/retrievers/RRFRetriever

I know, however that's seems not work correctly with Azure Chat models.

I checked that the reciprocal rank fusion retriever runs correctly on Azure LLM with GPT3-instruct, not Azure Chat Modules with any other model.


Add header to image request

Is there a way to add a header value to the WebImage component? To get the images from my server, I must pass some info in there
Thanks in advance

Already supported and mentioned on Readme:

URL Request / Response Modifier, provide custom HTTP Header

See the wiki, you need to pass the context option .requestModifier, or you can setup a common header, in the SDWebImageDownloader.shared.
Wiki : https://github.com/SDWebImage/SDWebImage/wiki/How-to-use#use-request-modifier-50
I don't want to add many code sample or details in readme, since it's for beginner and quick guide for a framework. For details, the docuementation and wikis is always the better place to check.

You can also just use another API, but this will apply to all image request, and does not support a block-based API (Which means, it's static, not dynamic). But it's simpler:
https://sdwebimage.github.io/Classes/SDWebImageDownloader.html#/c:objc(cs)SDWebImageDownloader(im)setValue:forHTTPHeaderField:

The wiki exemple worked perfectly!
Thank you!

@vipatronon I update the readme with a new chapter called Customization and configuration setup, make it easier for new user to find the place to do something setup about HTTP headers, coder plugins. Hope this can help for you.
https://github.com/SDWebImage/SDWebImageSwiftUI#customization-and-configuration-setup

@dreampiggy thank you for updating the readme, I helped me for sure and I think this will help new users too

@dreampiggy I still don't get it. How can you add a header for a single request? Where / how do you passt the requestModifier for a single request?
All I can see is changing default headers.

Since SD pass you a URLRequest value, you can return back another URLRequest.
URLRequest (actually the mutable one) has a property called allHTTPHeaderFields
You know how to use... Though this API not really convenient for simple use case. You can wrap it into a category for convenient

let requestModifier = SDWebImageDownloaderRequestModifier { (request) -> URLRequest? in
    if (request.url?.host == "foo") {
        var mutableRequest = request
        mutableRequest.setValue("foo=bar", forHTTPHeaderField: "Cookie")
        return mutableRequest
    }
    return request
};
imageView.sd_setImage(with: url, options: [], context: [.requestModifier] = requestModifier]) { }

Thank you so much, this clears it up a lot for me! :)

There may be a chance for SD to provide better convenient for common Swift users. And this repo can be merged into SDWebImage Core repo (current Objective-C only)
I've have a plan (and demo in (dreampiggy/SDWebImage/swift fork branch)[https://github.com/SDWebImage/SDWebImage/compare/master...dreampiggy:SDWebImage:swift]) to make SDWebImage 6.0 become Swift-Objc-mixed and provide massive better support like enum-associated-object/concurrency/result type, etc). But seems has no extra time to push it recently..


metric add SPI

https://github.com/apache/dubbo/issues/10555
metric add spi

Codecov Report

Merging #10622 (1813639) into 3.1 (ddd1786) will decrease coverage by 0.35%.
The diff coverage is 88.48%.

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

@@             Coverage Diff              @@
##                3.1   #10622      +/-   ##
============================================
- Coverage     65.16%   64.81%   -0.36%
+ Complexity      493      430      -63
============================================
  Files          1336     1340       +4
  Lines         56874    56994     +120
  Branches       8421     8406      -15
============================================
- Hits          37063    36938     -125
- Misses        15842    16054     +212
- Partials       3969     4002      +33

Impacted Files
Coverage Δ

...apache/dubbo/common/constants/CommonConstants.java
100.00% <ø> (ø)

...org/apache/dubbo/metrics/filter/MetricsFilter.java
83.33% <0.00%> (-10.42%)
:arrow_down:

...e/dubbo/metrics/service/DefaultMetricsService.java
0.00% <ø> (ø)

.../config/metrics/DefaultMetricsServiceExporter.java
80.55% <80.55%> (ø)

...bootstrap/builders/ExportServiceConfigBuilder.java
91.01% <91.01%> (ø)

...ubbo/config/deploy/DefaultApplicationDeployer.java
72.91% <91.66%> (+0.44%)
:arrow_up:

...ava/org/apache/dubbo/config/ApplicationConfig.java
85.71% <100.00%> (+0.52%)
:arrow_up:

.../metadata/ConfigurableMetadataServiceExporter.java
67.56% <100.00%> (-10.79%)
:arrow_down:

...mmon/serialize/fastjson2/FastJson2ObjectInput.java
0.00% <0.00%> (-62.50%)
:arrow_down:

...mon/serialize/fastjson2/FastJson2ObjectOutput.java
0.00% <0.00%> (-56.25%)
:arrow_down:

... and 75 more

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

Can you please explain the design of the configuration of the metrics-related properties, I noticed there are metrics properties defined in both ApplicationConfig and MetricsConfig, when should we need to set properties in ApplicationConfig and when should we set MetricsConfig?

metricsConfig中的协议，只是一种实现方式的声明,比如声明为 prometheus，而内部导出的服务，以什么样的形式提供调用，允许定义不同的协议调用方法

后续最好能将所有配置统一到 MetricsConfig 中进行配置

已处理


Copy Pasting comments broken in 5.x

Figured I'd report the issue you can no longer copy paste comment nodes in latest 5.x (using that currently to try out the addon stuffs)
Seems like a simple whoops quickly looking at the code seems to filter purposefully only flow nodes.

Sorry it took so long, but the issue is now fixed :)


(fix TypeError): Cannot read property 'timer' of undefined

fix (#141 )

Are these two separate changes?

wait im wrong something on this commit

Why did you make it only ignore bots when their IDs are in Client#ownerID?

sorry its just my personal use

i forgot that ive created a pull request


Proper way to handle exceptions in persistor.load?

    redux_persist_flutter: ^0.8.0-rc.0
    redux_persist: ^0.8.0-rc.0

The Readme has no indication on how to handle an exception on persistor.load, so I wrote this try catch because when there is an exception on persistor.load, the app freezes and stays white unless I put in the try catch. If there is a better way to write this, please let me know I would really appreciate it.
void main() async {
  final persistor = Persistor<AppState>(storage: FlutterStorage(), serializer: JsonSerializer<AppState>(AppState.rehydrate));

  var initialState;
  try {
    initialState = await persistor.load();
  }
  catch (e) {
    initialState = null;
  }

  List<Middleware<AppState>> createMiddleware() {
    return <Middleware<AppState>>[
      thunkMiddleware,
      persistor.createMiddleware(),
      LoggingMiddleware.printer(),
    ]
      ..addAll(createHomeMiddleware())
      ..addAll(createMeMiddleware());
  }

  final store = Store<AppState>(
    appReducer,
    initialState: initialState ?? AppState(),
    middleware: createMiddleware(),
  );

  runApp(Main(store: store));
}

Your JSON decoder function (AppState.fromJson) must be able to accept null, you'll most likely want to return an empty state.

A try...catch is the proper way to handle this.


fail to load more than 100 inventory (SignatureDoesNotMatch error)

Dear Bhushankumar,
I tried this example below with some codes changed,
https://github.com/bhushankumarl/amazon-mws/blob/master/examples/javaScript/fulfillmentInventory/listInventorySupply.js
We have 160+ ASINs in our store,  but I was only able to fetch 100 of them before getting an error.
below is the code, please can you help check?
//----------------------code starts-----------------------------
var auth = require("./auth");
var amazonMws = require('amazon-mws')(auth.accessKey,auth.secretKey);
var  query = {
'Version': '2010-10-01',
'Action': 'ListInventorySupply',
'SellerId': auth.sellerID,
'MWSAuthToken': auth.token,
'MarketplaceId': auth.marketplaceID,
'QueryStartDateTime': new Date(01, 04, 2018)
}
var nextQuery = {
'SellerId': auth.sellerID,
'MarketplaceId': auth.marketplaceID,
'Version': '2010-10-01',
"AWSAccessKeyId": auth.token,
'MWSAuthToken': auth.token,
'Path': "/FulfillmentInventory/2010-10-01"

}
function listInventory(query){
amazonMws.fulfillmentInventory.search(query, function (error, response) {
    if (error) {

    	console.log(nextQuery);

        console.log('error ', error);
        return;
    }else{

    	// console.log(response);

    	var members = response.InventorySupplyList.member;

    	console.log("------------------------------");
    	console.log(members.length);
    	console.log("------------------------------");

    	for (var i = members.length - 1; i >= 0; i--) {
    		console.log(members[i]);
    	}

    	try{

			var nextToken = response.NextToken;

			console.log("------------------------------");

			console.log(nextToken);

			console.log("------------------------------");

			nextQuery["NextToken"] = nextToken;
			nextQuery["Action"] = "ListInventorySupplyByNextToken";

			listInventory(nextQuery);

		}catch(e){

			console.log("All Fetched.");
		}

    }

});

}
listInventory(query);
//----------------------code ends-----------------------------
and here is the error msg I get:
error  { Type: 'Sender',
Code: 'SignatureDoesNotMatch',
Message: 'The request signature we calculated does not match the signature you provided. Check your AWS Secret Access Key and signing method. Consult the service documentation for details.',
I appreciate your kind attention to this issue. Thank you so very much!

@wewework
If you look at the example properly. You will found that 'Path' parameter is not required.
That's the reason you are facing an error on 'nextQuery'.


on_delete will be required when defining ForeignKey in Django 2.0

on_delete=models.CASCADE is the default behaviour on Django 1.7 -> 1.9 and will be a required arg in Django 2.0

Merged, thanks.


bug: Unsupported type for spread_zip

using Kaskada main, with beepGPT FineTuning_v1.ipynb, the code that used to function now errors.
messages = await kd.sources.Parquet.create(
    path = "slack-export.parquet",
    time_column = "ts",
    key_column = "channel",
    time_unit = "s",
)

# Messages come from Slack in chronological order, mixing concurrent conversations together.
# Let's re-group messages by thread and/or channel.
messages = messages.with_key(kd.record({
    "channel": messages.col("channel"),
    "thread": messages.col("thread_ts"),
}))

# To understand a conversation we need more than the most recent message.
# We build a rolling window of messages over the most recent messages.
conversations = (
    messages
    .select("user", "ts", "text", "reactions")
    .collect(max=20)
)

# We want to know who's interested in a conversation, but we only learn that later.
# Shift the conversation forward in time so we can associate it with reactions that happen during that time.
shifted_conversations = conversations.shift_by(timedelta(minutes=5))

shifted_conversations.preview(5)

Error:
{
	"name": "RuntimeError",
	"message": "error in kaskada Rust code
├╴at src/error.rs:54:21
│
├─▶ execute query
│   ╰╴at /Users/eric.pinzur/src/github.com/kaskada-ai/kaskada/crates/sparrow-session/src/session.rs:543:28
│
├─▶ internal compute error: failed to join compute threads
│   ╰╴at /Users/eric.pinzur/src/github.com/kaskada-ai/kaskada/crates/sparrow-runtime/src/execute/compute_executor.rs:192:22
│
├─▶ internal compute error: no additional context
│   ╰╴at /Users/eric.pinzur/src/github.com/kaskada-ai/kaskada/crates/sparrow-runtime/src/execute/operation.rs:345:58
│
├─▶ internal compute error: no additional context
│   ╰╴at /Users/eric.pinzur/src/github.com/kaskada-ai/kaskada/crates/sparrow-runtime/src/execute/operation/shift_to.rs:638:14
│
╰─▶ zip for shift_to
    ├╴at /Users/eric.pinzur/src/github.com/kaskada-ai/kaskada/crates/sparrow-runtime/src/execute/operation/shift_to.rs:637:14
    ╰╴Unsupported type for spread_zip: List(Field { name: \"item\", data_type: Struct([Field { name: \"ts\", data_type: Float64, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: \"user\", data_type: Utf8, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: \"text\", data_type: Utf8, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: \"reactions\", data_type: List(Field { name: \"item\", data_type: Struct([Field { name: \"count\", data_type: Int64, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: \"name\", data_type: Utf8, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: \"users\", data_type: List(Field { name: \"item\", data_type: Utf8, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }), nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }]), nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }), nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }]), nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} })",
	"stack": "---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
/var/folders/_7/czrlt9jx79n78l_zsyfwycs40000gp/T/ipykernel_39086/3983930945.py in <cell line: 0>()
     19
     20
---> 21 shifted_conversations.preview(5)

~/.pyenv/versions/3.11.4/lib/python3.11/site-packages/kaskada/_timestream.py in preview(self, limit, results)
   1068             results: The results to produce. Defaults to `History()` producing all points.
   1069         \"\"\"
-> 1070         return self.to_pandas(results, row_limit=limit)
   1071
   1072     def to_pandas(

~/.pyenv/versions/3.11.4/lib/python3.11/site-packages/kaskada/_timestream.py in to_pandas(self, results, row_limit)
   1091         \"\"\"
   1092         execution = self._execute(results, row_limit=row_limit)
-> 1093         batches = execution.collect_pyarrow()
   1094         table = pa.Table.from_batches(batches, schema=execution.schema())
   1095         table = table.drop_columns([\"_subsort\", \"_key_hash\"])

RuntimeError: error in kaskada Rust code
├╴at src/error.rs:54:21
│
├─▶ execute query
│   ╰╴at /Users/eric.pinzur/src/github.com/kaskada-ai/kaskada/crates/sparrow-session/src/session.rs:543:28
│
├─▶ internal compute error: failed to join compute threads
│   ╰╴at /Users/eric.pinzur/src/github.com/kaskada-ai/kaskada/crates/sparrow-runtime/src/execute/compute_executor.rs:192:22
│
├─▶ internal compute error: no additional context
│   ╰╴at /Users/eric.pinzur/src/github.com/kaskada-ai/kaskada/crates/sparrow-runtime/src/execute/operation.rs:345:58
│
├─▶ internal compute error: no additional context
│   ╰╴at /Users/eric.pinzur/src/github.com/kaskada-ai/kaskada/crates/sparrow-runtime/src/execute/operation/shift_to.rs:638:14
│
╰─▶ zip for shift_to
    ├╴at /Users/eric.pinzur/src/github.com/kaskada-ai/kaskada/crates/sparrow-runtime/src/execute/operation/shift_to.rs:637:14
    ╰╴Unsupported type for spread_zip: List(Field { name: \"item\", data_type: Struct([Field { name: \"ts\", data_type: Float64, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: \"user\", data_type: Utf8, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: \"text\", data_type: Utf8, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: \"reactions\", data_type: List(Field { name: \"item\", data_type: Struct([Field { name: \"count\", data_type: Int64, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: \"name\", data_type: Utf8, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }, Field { name: \"users\", data_type: List(Field { name: \"item\", data_type: Utf8, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }), nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }]), nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }), nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} }]), nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} })"
}

example data can be found here (for now): https://github.com/kaskada-ai/beep-gpt/blob/esp/generate_slack/slack-export.parquet
the example data is currently on a branch and will be available in main soon.

Hmm. @jordanrfrazier the only thing that I could see changing is that somehow the bug you fixed (about determining what is continuous or not) has changed in some way, causing it to try to spread_zip a list.

Observation: shift_by may be continuous even if shift_to isn't. Specifically, an aggregation being shifted forward by a fixed amount is still continuous.

Observation: shift_by may be continuous even if shift_to isn't. Specifically, an aggregation being shifted forward by a fixed amount is still continuous.

Yeah, I think agreed on this. Unfortunately, making this change isn't a one-line change since shift_by is implemented with shift_to:
shift_by(value) => shift_to(add_time(delta, time_of(value)), value)

Observation: shift_by may be continuous even if shift_to isn't. Specifically, an aggregation being shifted forward by a fixed amount is still continuous.

Yeah, I think agreed on this. Though, what about shift_by(Table.time)? That may re-order rows, or cause rows to shift to the same time. We could still be continuous, but it's something to think about. Making shift_by(literal) continuous and shift_by(column) discrete is an option, but one I think would cause more confusion.
Unfortunately, making this change isn't a one-line change since shift_by is implemented with shift_to:
shift_by(value) => shift_to(add_time(delta, time_of(value)), value)

Also agreed. Probably worth a separate issue to track that, since it seems unrelated to this bug.


Switch aria-describedby pointing to unexistant ID

As @noetibbl reported to me, the aria-describedby in the switch doesn't always point to the right state label.

Issue actually occurs when you perform a toggle like click by clicking the "on" section of the switch button whenthe page loads to switch off, and without moving your cursor away, click the "on" section again to toggle back on.


⚠ Minecraft has degraded performance

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

HTTP code: 200
Response time: 1098 ms

Resolved: Minecraft performance has improved in 07baa48.


Block parent not adding in parent block

Before 1.4.2, block inheritance with parent would work correctly on a simple example, but at 1.4.2 (currently latest) when I use parent it does not add the parent template anymore. I also tried installing from Github directly and it didn't work so didn't notice a fix.

An example where I expect to have two links declared but does not have the parent declared.
layout.swig

<!doctype html>
<html>
  <head>
    ...
    {% block styles %}<link rel="stylesheet" href="/styles/index.css">{% endblock %}
  </head>

  <body>
    ...
  </body>
</html>

page.swig

{% extends 'layout.swig' %}

{% block styles %}
  {% parent %}
  <link rel="stylesheet" href="/styles/page.css">
{% endblock %}


Add an auto reconnect

Hello !
The old implementation of reconnect in the persistentSubscription wasn't working.
One of the actual workaround to handle NotMasterException can be:
// You only need to set the default scheduler once
Scheduler::setDefaultFactory(
    function () {
        // The getLoop method auto start loop
        return new Scheduler\EventLoopScheduler(EventLoop::getLoop());
    }
);

$dsn = 'tcp://admin:changeit@toto.com:1113';

$observer = new CallbackObserver(
    function($e) {
        var_dump($e);
    }
);

$this->eventStore = new EventStore(EventLoop::getLoop());

$connect = function($dsn, $stream, $group) {
    return $this->eventStore
        ->connect($dsn)
        ->flatMapTo($this->eventStore->persistentSubscription($stream, $group));
};

$connect($dsn, $stream, $group)
    ->catch(function (\Throwable $e) use ($connect, $dsn, $stream, $group) {
        $this->eventStore = new EventStore(EventLoop::getLoop());
        $credentials = parse_url($dsn);
        if ($e instanceof NotMasterException) {
            $dsn = 'tcp://'.$credentials['user'].':'.$credentials['pass'].'@'.$e->getMasterIp().':'.$e->getMasterPort();
            return $connect($dsn, $stream, $group);
        }
        throw $e;
    })
    ->flatMap($this->adapter)
    ->subscribe($observer);

But it's not a solution when you want to handle multiple \Throwable before the message consumption. The catch method only executed once.
Probably the connect method can handle internally this logic, but actually i'm a little lost.

Connect is an independent stream, so the best solution
I've made an exemple in a previous release that can be adapted
<?php
use EventLoop\EventLoop;
use Ramsey\Uuid\Uuid;
use Rx\Observer\CallbackObserver;
use Rx\Scheduler\EventLoopScheduler;
use Rxnet\EventStore\Data\WriteEventsCompleted;
use Rxnet\EventStore\EventRecord;
use Rxnet\EventStore\EventStore;
use Rxnet\EventStore\NewEvent\JsonEvent;
use Rxnet\Httpd\Httpd;
use Rxnet\Httpd\HttpdEvent;
use Rxnet\Operator\OnBackPressureBuffer;
require '../vendor/autoload.php';
$eventStore = new EventStore();
// Wait connexion to be up before starting
\Rxnet\awaitOnce(
    $eventStore->connect()
        ->doOnError(function (\Exception $e) {
            echo "got an error {$e->getMessage()} \n";
        })
        // on error try to reconnect until end of world
        ->retryWhen(function (\Rx\Observable $errors) {
            // Wait 5s between reconnection
            return $errors->delay(5000)
                ->doOnNext(function () {
                    echo "Disconnected, retry\n";
                });
        })
);
// When http is faster than write
// buffer in memory, could have been redis buffer
$memoryBuffer = new OnBackPressureBuffer(
    10000,
    function () {
        echo "Buffer overflow !";
    },
    OnBackPressureBuffer::OVERFLOW_STRATEGY_ERROR
);
// Write to this stream id
// API part :
// Forward to event store when an event is received
$httpd = new Httpd();
$httpd->listen(8082)
    // Don't filter anything and answer ok
    // in real life that would be
    ->map(function (HttpdEvent $event) use ($memoryBuffer) {
        $id = Uuid::uuid4()->toString();
        echo "Received HTTP request {$id} on /test, save an event \n";
        // Answer we handle it
        $event->getResponse()
            ->json(compact('id'));
        // Transform it to an event (do what you want)
        return new JsonEvent(
            '/test/asked',
            ['i' => microtime(true)],
            $id
        );
    })
    // here we buffer until commit finished
    ->lift($memoryBuffer->operator())
    // Write to event store
    ->flatMap(function ($data) use ($eventStore) {
        return $eventStore->write('category-test', $data);
    })
    // Ask for next element in buffer
    ->doOnNext([$memoryBuffer, 'request'])
    // Output some debug when write is complete
    ->subscribe(
        new CallbackObserver(
            function (WriteEventsCompleted $eventsCompleted) {
                echo "Event saved with number {$eventsCompleted->getLastEventNumber()} on commit position {$eventsCompleted->getCommitPosition()} \n";
            }
        ),
        new EventLoopScheduler(EventLoop::getLoop())
    );
echo "HTTPD server listening on http://localhost:8082/test\n";
// Listener part :
// listen for events on given stream and to projection
// persistent subscription is better here
$eventStore->volatileSubscription('category-test', true)
    ->subscribeCallback(function (EventRecord $record) {
        echo "Event received {$record->getNumber()}@{$record->getStreamId()} {$record->getType()} with ID {$record->getId()}\n";
    });
// Make the event loop run
EventLoop::getLoop()->run();

For await you can replace it by https://github.com/RxPHP/RxAwait

@Th3Mouk is it ok with this example ?

Yes thank you, I didn't want to use an await, but for this kind of thing it's acceptable.
The timeout will be handled after the connect, and the NotMasterException after the subscription.
The problem with the NotMasterException is that it needs a brand new connection and EventStore object currently. I will probably have a look to propose a solution.


[FEATURE] Rework current dir usage in wash-lib helpers

Affected project(s)

[ ] documentation
[ ] examples
[ ] wasmCloud host
[X] wasmCloud CLI (wash)
[ ] wasmCloud dashboard UI (washboard)
[ ] capability providers
[ ] provider bindgen
[ ] control interface client
[ ] other / not sure

Is your feature request related to a problem? Please describe.
Currently some helper code in wash-lib is quite unergonomic/likely to be problematic in the future with it's use of std::env::current_dir(). It would be great to refactor this code (and it's usage points) to be more explicit about the intended directory, and hopefully to make the code easier to reuse without global side effects/forcing callers to resolve exactly which directory they should be using.
See Taylor's comments. It's not clear exactly what the best approach is here, but the goals are to avoid having this "utility" method just decide the project dir at the same time as it does other work.
We don't have to worry about breaking changes too much here, as it seems that the main/only user of wash-lib is wash-cli, and we can evolve them in tandem.
Describe the solution you'd like
Ideally, we'd have much cleaner code in the common.rs file that doesn't try to call out to std::env::current_dir(), even if that means just making the determination of project directory more explicit (i.e. moving that out into a separate helper function).
Refactoring common.rs into a module with different submodules for helpers might be nice.
Describe alternatives you've considered
No response
Additional context
No response

Hi @vados-cosmonic
I would like to have a look at this [FEATURE]
Based on what you mentioned I can think of 2 soln paths as of now (would appreciate your view on those):
Soln1: Directory Context Way

Creates a dedicated ProjectContext struct to handle all directory-related operations. The context is created explicitly and passed around, removing global state dependencies. Basic implementation would look like this:

#[derive(Debug, Clone)]
pub struct ProjectContext {
    project_dir: PathBuf,
    dev_dir: PathBuf,
}

impl ProjectContext {
    pub fn new(project_dir: impl Into<PathBuf>) -> Self {
        let project_dir = project_dir.into();
        let dev_dir = project_dir.join(".wash").join("dev");
        Self { project_dir, dev_dir }
    }
    ........

Soln2: Config-based Approach

It uses a central configuration object that holds all directory information and passes it through the application. Directory paths are configured once and accessed through this object. Basic implementation would look like this:

#[derive(Debug, Clone)]
pub struct WashConfig {
    directories: DirectoryConfig,
    // Other config options...
}

#[derive(Debug, Clone)]
pub struct DirectoryConfig {
    project_dir: PathBuf,
    ..................
}

impl WashConfig {
...................
}

Hey @PsychoPunkSage thanks for taking this on!
I'm trying to take a look through the solutions, and I have to admit what I was thinking was something much simpler! If we could just determine the "project directory" and make sure that we referenced that in every function (i.e. taking it explicitly instead of calling current_dir() in random functions) that would be a huge step forward I think .
I think that Solution 2 that you mentioned looks the most aligned and least intrusive to implement, but what I'd love to see as well is starting with an analysis of where we use current_dir/depend on knowing the project (or some other) directory that we can figure out once.

OK, should I begin by making a draft PR for Soln2? Can you assign this issue to me?

Absolutely -- thanks for the putting up a PR!


Rename mod to HAMi

Yes, that's needs to be done, thanks for fixing, have you tested the "dev-image"?

Yes, that's needs to be done, thanks for fixing, have you tested the "dev-image"?

Yes.


Removing Attachments

In the example removing attachments you declare rev to 0
What is rev?
I am attempting to use the samples but my rev 8 appears to be related to revisions(history), not attachments.

You will need to get the revision value . then use that value + 1 in your work item update call to remove the attachment.
https://docs.microsoft.com/en-us/rest/api/vsts/wit/work items/update?view=vsts#remove_an_attachment


DialogModal misaligns child RadioButton text

Issue
When a <RadioButton /> component is placed inside a <DialogModal /> the text will be misaligned. Example:
import React from "react";
import { DialogModal, RadioButton } from "radiance-ui";

function App() {
  return (
    <DialogModal>
      <RadioButton type="primary" checked>
        This is a primary radio button
      </RadioButton>
    </DialogModal>
  );
}

export default App;

Will result in:

Cause
This is caused by the margin-bottom styles applied to all p & p:last-of-type elements inside <DialogModal />; including nested p elements inside child components.  This is the responsible styling.
Workaround
This can be temporarily fixed by using emotion to inject a css style to override the margin styling:
import React from "react";
import { DialogModal, RadioButton } from "radiance-ui";
import { css } from "@emotion/core";

function App() {
  return (
    <DialogModal>
      <RadioButton
        type="primary"
        checked
        css={css`
          & > p:last-of-type {
            margin-bottom: 0;
          }
        `}
      >
        This is a primary radio button
      </RadioButton>
    </DialogModal>
  );
}

export default App;

Will result in:

@morgan-sam Thank you for pointing this out! It looks like our usage (outside of radiance-ui) has a regular pattern of nesting paragraph tags like so:
<DialogModal title="We changed things up to give you more control">
  <p>
    Example Text
  </p>
  {...rest}
</DialogModal>

Based on your example code, I think an appropriate course of action is to codify the convention of a paragraph tag child as a prop:
<DialogModal title="We changed things up to give you more control" message="Example Text">
  {...rest}
</DialogModal>

We can then wrap the message in a special styled component and use that as the selector instead of p tags, generically:
export const Message = styled.p``;

export const ModalBody = styled.div`
  ${Message} {
    margin-bottom: ${SPACER.large};
    &:last-of-type {
      margin-bottom: ${SPACER.xlarge};
    }
  }
`;

That way these styles won't bleed over to other components, unnecessarily.

I'll update this issue once the PR is ready, for tracking purposes. Again, thank you for the issue!

Encountered this issue while creating a custom Modal. IMO it seems strange that the "Container" (ModalBody) is styling its child elements. Perhaps there is a good reason for this that I don't know yet.
But leaving the container responsible for its own styles and behavior and letting the engineer compose spacing and styling for the child elements strikes me as a more standard pattern for UI libraries.

This will be fixed in v18.0.0, PR: https://github.com/curology/radiance-ui/pull/833
Closing. Thanks again for pointing this out @morgan-sam 🙏


Running in a docker container

I'm pretty familiar with docker, but I'm having a hard time building an image with ark-deployer included.  Most of the problems seem to be node and gyp related.
While running bridgechain.sh install-node ... there are quite a few  errors spamming the terminal.  The first one is:

tput: unknown terminal "unknown"

I could see it being beneficial for the ark team to officially support this?

Thanks for opening an issue. I've no experience of Docker, which is why vagrant was used. I'm happy for anyone in the community to do this, but it's not a high priority for the team to do it currently.
I'll leave the issue open though until it is done

@alexbarnsley I have a working Dockerfile for deployer. I would need to clean it up a bit. But, would a PR for this issue still be accepted or are we waiting for core v2 first?

@alexbarnsley @rodxavier I want to start deployer-v2 into Docker container. Is someone working on it yet or can I create a PR ?

@Nigui no there's not, but I would advise waiting until deployer v2 is finished

waiting until deployer v2 is finished

Any idea when it will be finished? In a month or two, or even much later?
Thank you

This issue has been closed. If you wish to re-open it please provide additional information.


Fix app_name incorrect extraction

Appname will be empty string in this situation

Seems interesting ! Could you sync with latest commit and remove chinese comments to english one ?


Update smart margin deployment

Update addresses and ABIs to latest deployment
Description

[x] Update addresses
[x] Update Account ABI
[x] Update AccountFactory ABI
[x] Update Settings ABI

This PR seems obsolete if we simply merge perps-v2-dev with the caph release updates into cm-v2 ?


Dockerise website codes

Dockerise codes
Should dockerise website codes to make deployment easier

Website codes has been dockerise. There fore i will be closing this issue


WIP facets/sorter: initial prototype of facets and sorting for REST API (very much WIP)

(addresses inveniosoftware/invenio-search#52)

(addresses inveniosoftware/invenio-search#53)

sort field names has to be fixed from "field/-field" to "field:asc/fied:desc" (impacts invenio-search-js)

Ping @egabancho - ready for review.

Should we also update the example app? otherwise :shipit:

I've opened an issue for it #37 :-) Will do it as part of the facets :-)


Add Light Car Company Rocket from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of Moncasta, to add Light Car Company Rocket to CKAN.
Please direct questions about this pull request to Moncasta.
Mod details:
name = /mod/2333/Light%20Car%20Company%20Rocket
author = Moncasta
abstract = Light Car Company Rocket is a racing vehicle for KSP.
license = GPLv2
Homepage = https://moncasta.wordpress.com/2020/02/14/light-car-company-rocket-launch-film/
description =
Light Car Company Rocket is a racing vehicle for KSP. Designed for TR-2L wheel. Craft file inclued

See #7670.


Question about WS subscription

Hi Semashkinvg,
In my code I did this:
bitmexApiSocketService.Subscribe(BitmexApiSubscriptionInfo<IEnumerable<OrderBookDto>>.Create(SubscriptionType.orderBookL2, a => { foreach(var d in a) Console.WriteLine(d.ToStr()); })); }

This gave me the output:
BITMX Entry: XBTUSD id:15599217800 side:Sell size:10000 price:7822
BITMX Entry: XBTUSD id:15599217900 side:Sell size:3130 price:7821
BITMX Entry: XBTUSD id:15599218000 side:Sell size:800 price:7820
BITMX Entry: XBTUSD id:15599218050 side:Sell size:50 price:7819.5
BITMX Entry: XBTUSD id:15599218250 side:Sell size:575 price:7817.5
BITMX Entry: XBTUSD id:15599218300 side:Sell size:30 price:7817
...
BITMX Entry: XBTUSD id:15599227400 side:Sell size:6792 price:0
BITMX Entry: XBTUSD id:15599231250 side:Sell size:9864 price:0
BITMX Entry: XBTUSD id:15599235100 side:Sell size:6867 price:0
BITMX Entry: XBTUSD id:15599238900 side:Sell size:6592 price:0

Questions:

There is no indication of whether the action is "partial", "update", "delete" or "insert". Am I supposed to make the guess that if price is 0 then it is an "update", and if size is 0 then it is a "delete" etc?

Also according to https://www.bitmex.com/app/wsAPI#subscriptions I should discard all data before "partial" but how can I know whether a message is "partial" or "insert"?

If I subscribe to orderBook10, all I get are zero entries. Why?

BITMX Entry: EOSU18 id:0 side: size:0 price:0
BITMX Entry: ETHU18 id:0 side: size:0 price:0
BITMX Entry: ETHUSD id:0 side: size:0 price:0
BITMX Entry: LTCU18 id:0 side: size:0 price:0
BITMX Entry: TRXU18 id:0 side: size:0 price:0
BITMX Entry: XBTJPY id:0 side: size:0 price:0
BITMX Entry: XRPU18 id:0 side: size:0 price:0
BITMX Entry: XBTKRW id:0 side: size:0 price:0
BITMX Entry: XBTZ18 id:0 side: size:0 price:0
BITMX Entry: TRXU18 id:0 side: size:0 price:0
BITMX Entry: TRXU18 id:0 side: size:0 price:0

hi @GitteryCoder,
I think the question you raised is quite important and it's definitely worth investigating it carefully. It's crucial not just to know if these are bugs or not, but also to add some description of the API to this repository. I know that handling order book changes is a key point in the most of the trading strategies, so I will come back to you soon. All the exchanges provide order book updates with a different way, so,  I think it's important to describe what's the best way to handle order book using Bitmex API
Thanks,


[Bug] Resolve "Corepack is about to download"

Is there an existing issue for this?

[X] I have searched the existing issues

What product(s) are you seeing the problem on?
Sage Monorepo
Current behavior
The image of the devcontainer has recently been updated. The following message is now shown when running workspace-install for the first time in a new devcontainer.
vscode@1a5d528bb7c5:/workspaces/sage-monorepo$ workspace-install
Corepack is about to download https://repo.yarnpkg.com/3.6.1/packages/yarnpkg-cli/bin/yarn.js.

Do you want to continue? [Y/n]

Ideally, Yarn should already be installed as part of the dev container, or at least the developer should not be prompted by this message.
Expected behavior
No response
Anything else?
No response
Commit ID
63af8d378f214a211187ece95f73684d934504c8
Are you developing inside the dev container?

[X] I am using the dev container

Code of Conduct

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

Yarn is no longer pre-installed in the devcontainer:
vscode@b515be0087ab:/workspaces/sage-monorepo$ yarn --version
Corepack is about to download https://repo.yarnpkg.com/3.6.1/packages/yarnpkg-cli/bin/yarn.js.

Do you want to continue? [Y/n]


loadingView position in placeSubviews

I got a problem is :
When I subclass MJRefreshNormalHeader and override the function placeSubviews, I change the arrowView's position by directly setting frame or constraint, the loadingView's position does not changed with it.
I look into the original code, the loadIngView's center is aligned with arrowView. But the loadView's position is already set before the subclass's function placeSubviews is called. So how can I solve the problem rather than forking the repository and changing the code?
thanks!

- (void)placeSubviews
{
    [super placeSubviews];
    self.arrowView.frame = ...;
    self.loadingView.frame = ...; // or self.loadingView.center = ...;
}

Yes, that is the way to do it.
In MJRefreshNormalHeader.m:
@property (weak, nonatomic) UIActivityIndicatorView *loadingView;
The the loadingView property is internal property of Header. I can't visit it, so if possible to put the property in .h