typeahead search not working

I am trying to get typeahead to work, it is working with $http.get but not with the odata library. Seems like it is a problem with returning the promise, not sure if I am missing something.
Works:
return $http.get("/odata/Plants?$filter=substringof('"+val+"',Name)").then(function (response) {
var data = response.data;
return data.value;
});
Returns data but typeahead doesn't get the return and hangs:
appDb.Plants.odata().filter(new $odata.Func('substringof',
new $odata.Value(val),
new $odata.Property("Name"))).query(function (data) {
return data;
});
returns data but typeahead does nothing, doesn't even trigger loading indicator:
return  appDb.Plants.odata().filter(new $odata.Func('substringof',
new $odata.Value(val),
new $odata.Property("Name"))).query(function (data) {
return data;
});

Hi @FarmSol, this is because your library expects a promise, and you're giving it an array. You should use $q and return a proper promise.

Or maybe something like :
var plants = appDb.Plants.odata().filter(...).query();
return plants.$promise;

Some libraries (like angular grids) work well and expect the resource object and know how to check resourceObj.$promise to know when resourceObj array will be populated and to populate the grid.  Others are just built around promises, so you'll have to implement some kind of adapter logic directing it to the $promise, I'm guessing the library would take the response from the properties on the $promise object rather than the resource object itself.
So just keep in mind, this library is a wrapper around the angular Resource object concept.

@devnixs Thanks!!  returning .$promise worked


Floor Button

It would be nice to have buttons that select the floor for the elevator to go to instead of having to press Z.
Since there's a way to place blocks in the elevator shaft, the button would stay there without anything getting in the way.
See Example:

But if this is the way the buttons are implemented, it would be nice if the panel didn't have depth, that it was like a "sheet of paper" glued to the wall, so it wouldn't have that elevation disturbing the vision as in the image above.
It could be something really simple, just with the buttons with the numbers inside them.
Suppose it has 5 floors.
The panel has 5 buttons.
If you are on Floor 3, Button 3 has a highlighted green light to indicate which floor you are on.
You press the 5 button, the green one goes out of the 3 button and the 5 button has a highlighted red light to indicate which floor you are going to.
Upon reaching the 5th floor, the light changes to green.

I like this idea!

Thanks for the suggestion. I think a separate menu has been created on purpose so that there wouldn't be any space constraint. Imagine if someone creates an elevator with 100 floors; I don't think it would look good to fit 100 buttons on a panel.

I understand your point, but I think it's still a valid idea, not for 100 floors as you said, but most players will create 10 floors at most, so it would be interesting to have a panel perhaps with a maximum of 10 floors or any other maximum, and then if the player wants more than the maximum, then it would be mandatory to use the menu!
But I, for example, would create a few elevators for my small base, just a few floors, and I'm sure many players would do the same!
And then a few more buttons would be better than opening the menu!
Obviously, I don't want to be so arrogant as to say what your mod needs or doesn't need, I'm just explaining my idea!


problem with "looking up region"

so it connects to the client.
But I came across with a problem:
-level=error msg="failed looking up region" backoff=16ms err="context deadline exceeded" key=""row1"" table=""emp""
and
-level=error msg="failed looking up region" backoff=16ms err="context deadline exceeded" key="""" table=""hbase:meta""
any idea how to solve this? thanks in advance

+1

after split region,region info is not realtime .

import "github.com/sirupsen/logrus"
logrus.SetLevel("debug")

Open debug log and check details

+1

add hbase hosts in local /ect/hosts

+1

Still facing the same issue even after adding ip:hostname mapping.

I use remote zookeeper. Sets zookeepers address as input fix my issue.

Still facing the same issue even


Team description issue

Log ID: 4ca2529cf3b6d6585e3b481c
[~]$ keybase team settings lgbt --description='LGBT friendly group.'
▶ ERROR PushHandler: failed to run teamHandler handler: json: cannot unmarshal number into Go struct field TeamChangeRow.implicit_team of type bool [tags:GRGIBM=...

That failure might not have caused any problems (though we'll look into it). Did the update actually work?

It looks like the update to the team description did go through.

Ah cool, we'll look into that error...


CupertinoScrollbar shouldn't require a long press to drag when using a mouse

Initiating a drag on a CupertinoScrollbar currently requires a long press on the scrollbar.  This makes sense in a touch-screen interface, but it doesn't make sense when using a physical mouse.

I think this is a duplicate of the more general issue: https://github.com/flutter/flutter/issues/44840


Make elm-package help identifying forks and dupes

Intro
One of the best things about Elm is the ecosystem. In no small part, this is due to the tooling:

the author is part of the package, which makes it easy to say "oh, I've used another high quality package by abc, so I'll go with that one"
Semver is strictly enforced
packages are encouraged to have descriptive names. something-extra for extensions to existing functionality, -exploration for experimental overhauls
etc.

Problem statement
However, there are some painpoints:

We have 5 packages that have something to do with your keyboard, excluding elm-lang/keyboard. Of these 5, 3 are called keyboard-extra. Note that, after the release of the most recent of those, @eeue56 intervened and helped out in getting the package author to communicate with @ohanhi. Eventually, changes were made to ohanhi/keyboard-extra that made the new package redundant, and it was unreleased.
We have 2 elm-bootstrap packages, and it's been mentioned on Slack that the second one is a fork which came about without any communication with the original maintainer.

Note that even though licensing allows this, and there is no inherent problem with forks; I do feel like the tooling could work harder to prevent splintering and duplication.
How to handle this
I propose three possible improvements.

elm package publish could check if the package name was already in use. This could just be a are you sure? y/n type thing, but pointing it out would at least prevent it from happening by accident.
elm package publish could check if the API of a certain package has significant overlap, and mark it as a fork. When it does, it could ask if any attempt was made to communicate with the original maintainer - if not, point out how it could help the ecosystem more than by just forking and splintering. Either way, having a description of why a package was forked could be helpful.
On package.elm-lang.org, there is no way of identifying forks easily. Adding that, in addition to displaying the reason for the fork, could help people selecting which package they're really after.

Note that the elm-bootstrap fork might just be to enable testing some changes, for personal use or whatever. And that's fine, but if it's not meant for general consumption, elm package could provide some pointers on how to test a package without publishing it (git submodules, etc).

For the last point (forks not being easy to recognize), I made this quick mock up of how I think it could look like:

Hello,
I'm the scrub dev who forked elm-bootstrap and then published it.  To anyone interested, I opened elm-lang/package.elm-lang.org#227 to see if I can get my fork removed from the package manager.
FWIW, I agree wholeheartedly with the solution proposed in this issue.  Whatever message is displayed when elm-package finds an identical package name should be extremely strongly worded so that would-be publishers are made extremely aware that publishing a fork without the original maintainer's blessing is a major insult to that developer's efforts.

I don't think your last half-sentence is consense.


Expose Blazor components dpi

Description of Change
It would be useful to expose the dpi field in the SKCanvasView and SKGLView components.
Bugs Fixed
fixes #1831

Related to issue #

API Changes
Added on SKCanvasView :

public double DPI => dpi;

Added on SKGLView :

public double DPI => dpi;

Behavioral Changes
PR Checklist

[ ] Has tests (if omitted, state reason in description)
[ ] Rebased on top of main at time of PR
[ ] Changes adhere to coding standard
[ ] Updated documentation

The previous commit applies the rename @robloo

Please merge into main finally :(

@dotnet-policy-service agree

Thanks for staying with us :) I know it has been a long time.


[SPARK-18134][SQL] Orderable MapType

What changes were proposed in this pull request?
We can make MapType orderable, and thus usable in aggregates and joins.
How was this patch tested?
Unit tests.

Test build #82106 has started for PR 19330 at commit 2e2b98d.

It seems https://github.com/apache/spark/pull/15970 is not being worked.
I resolved conflicts and add some tests in this pr.

Test build #82107 has started for PR 19330 at commit bd50495.

Test build #82106 has finished for PR 19330 at commit 2e2b98d.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

@jinxing64 thanks for taking over. I have glanced over the PR, and I miss the explicit sorting of maps. We can't assume that maps are sorted out of the box, for example the following example should evaluate to true but it won't in your PR: map(1, 2, 3, 4) = map(3, 4, 1, 2).

Test build #82107 has finished for PR 19330 at commit bd50495.

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

Merged build finished. Test PASSed.

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

@hvanhovell : based on your comment over the jira, it seemed that the approach to be used is yet to be finalised. Are we moving ahead with this approach ?

@hvanhovell
Thanks a lot for comment.
I got you point. I will refine soon.

Test build #82173 has started for PR 19330 at commit 33e532b.

Test build #82174 has started for PR 19330 at commit 0385487.

Test build #82174 has finished for PR 19330 at commit 0385487.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Test build #82173 has finished for PR 19330 at commit 33e532b.

This patch fails due to an unknown error code, -9.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Build finished. Test FAILed.

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

Merged build finished. Test FAILed.

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

Jenkins, retest this plesase.

Test build #82196 has started for PR 19330 at commit 5279338.

Test build #82196 has finished for PR 19330 at commit 5279338.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Merged build finished. Test FAILed.

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

It seems the failed SparkR unit tests is not related.
In current change, I added trait OrderSpecified, expressions(BinaryComparison, Max, Min, SortArray, SortOrder) using ordering of data type extends OrderSpecified. During analyzing, expressions with map type output will be wrapped in OrderMaps , thus changed to be an ordered map.
Expressions with map type output in Aggregate, Distinct and SetOperation will also be wrapped in OrderMaps.

Jenkins, retest this please

Test build #82232 has started for PR 19330 at commit 5279338.

Test build #82232 has finished for PR 19330 at commit 5279338.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Merged build finished. Test FAILed.

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

Conflicts resolved.

Test build #82242 has started for PR 19330 at commit e071892.

Test build #82242 has finished for PR 19330 at commit e071892.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Merged build finished. Test FAILed.

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

Test build #82264 has started for PR 19330 at commit 0b2b52c.

Test build #82264 has finished for PR 19330 at commit 0b2b52c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #82272 has started for PR 19330 at commit 80b6238.

Test build #82272 has finished for PR 19330 at commit 80b6238.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #82282 has started for PR 19330 at commit a9d09f6.

Test build #82282 has finished for PR 19330 at commit a9d09f6.

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

Merged build finished. Test PASSed.

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

@kiszk
Thanks a lot for comments. Tests passed now. In current change ordered is included in jsonValue. But I'm not sure it is appropriate.
Thanks again for taking time looking into this :)

@jinxing64 I would like to hear opinions from other reviewers whether ordered should be included in these methods like ToString, buildFormattedString, and jsonValue.
IMO, I think we should make these methods consistent, ordered is included or not. The method prints the status of MapType

Test build #83892 has started for PR 19330 at commit 5445834.

Test build #83893 has started for PR 19330 at commit a472dae.

Test build #83892 has finished for PR 19330 at commit 5445834.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Merged build finished. Test FAILed.

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

Test build #83893 has finished for PR 19330 at commit a472dae.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Merged build finished. Test FAILed.

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

Test build #83926 has started for PR 19330 at commit b1886f6.

Test build #83926 has finished for PR 19330 at commit b1886f6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Merged build finished. Test FAILed.

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

Test build #83929 has started for PR 19330 at commit 6aa4b8b.

Test build #83929 has finished for PR 19330 at commit 6aa4b8b.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Merged build finished. Test FAILed.

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

Jenkins, retest this please.

Test build #83936 has started for PR 19330 at commit 6aa4b8b.

Test build #83936 has finished for PR 19330 at commit 6aa4b8b.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Merged build finished. Test PASSed.

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

Does the community currently have a join and group by code that supports mapType?

@xxzzycq
Currently no

I think we can resume this pr for v3.0. Are u still there and can you resume this? @jinxing64
Also, can you add Closes #15970 in the PR description?

@maropu
Thanks, and yes I'm still here and I can keep going if this pr is interested.
I will update this pr this weekend.

Thanks!

Build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4200/
Test PASSed.

Test build #97739 has started for PR 19330 at commit 6aa4b8b.

Build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4222/
Test PASSed.

Test build #97739 has finished for PR 19330 at commit 6aa4b8b.

This patch fails due to an unknown error code, -9.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Build finished. Test FAILed.

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

Build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4238/
Test PASSed.

Test build #97751 has started for PR 19330 at commit 6aa4b8b.

Test build #97767 has started for PR 19330 at commit 6aa4b8b.

Test build #97751 has finished for PR 19330 at commit 6aa4b8b.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Build finished. Test FAILed.

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

Test build #97767 has finished for PR 19330 at commit 6aa4b8b.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
case class Max(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Min(child: Expression) extends DeclarativeAggregate with OrderSpecified
case class Least(children: Seq[Expression]) extends Expression with OrderSpecified
case class Greatest(children: Seq[Expression]) extends Expression with OrderSpecified
case class OrderMaps(child: Expression) extends UnaryExpression with ExpectsInputTypes
class InterpretedOrdering(orders: Seq[SortOrder]) extends Ordering[InternalRow]
case class In(value: Expression, list: Seq[Expression]) extends Predicate with OrderSpecified
case class InSet(child: Expression, hset: Set[Any])
abstract class BinaryComparison extends BinaryOperator with Predicate with OrderSpecified

Build finished. Test FAILed.

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

Test build #97817 has started for PR 19330 at commit 6aa4b8b.

Build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4272/
Test PASSed.


Question about using bosh-init as deployment tool.

Hey, all.
I am very interested in knowing your plans on future of this project. Particularly if you plan to switch to using bosh-init as a main deployment tool?
Thank you.

My main goal that drove me to write bosh-bootstrap and keep supporting it is to make it super easy to get started with bosh.
Figuring out how to construct a micro-bosh.yml wasn't ever trivial; and you didn't learn anything about using bosh from the activity (diff schema from a bosh manifest)
With bosh-init we now have a schema that is closer to what a normal bosh manifest looks like - so on one side, automatically generating it for someone seems to take away from the learning process.
But on the other side, a bosh-init manifest is longer and currently more complicated than micro-bosh.yml so I can imagine as a user I would want a way to simplify how to get started.
In production we're now generating micro-bosh.yml using spiff and/or spruce and deploying via concourse.ci pipelines. In future it would generate bosh-init manifests.
So it's possible that nextgen of bosh-bootstrap is a different type of tool; with a different short and long term goal for a user.
But the future is bosh-init
On Fri, Sep 25, 2015 at 8:23 AM, Alexander Lomov<EMAIL_ADDRESS>wrote:

Hey, all.
I am very interested in knowing your plans on future of this project. Particularly if you plan to switch to using bosh-init as a main deployment tool?
Thank you,
Alex L.
Reply to this email directly or view it on GitHub:
https://github.com/cloudfoundry-community/bosh-bootstrap/issues/318

Thank you for the answer. I was wondering because I wanted to update terraform-openstack-cf-install, which uses bosh-bootstrap by adding the latest cf and possibility to set AZ. Still using bosh-init here seems more natural for me and I will switch to using it.

If started spiking out bosh-init support in terraform-aws-bosh repo at the start of the year. Perhaps have a look there too. It's not as supported as terraform-aws-cf-install (it's my play area)
On Fri, Sep 25, 2015 at 9:16 AM, Alexander Lomov<EMAIL_ADDRESS>wrote:

Closed #318.
Reply to this email directly or view it on GitHub:
https://github.com/cloudfoundry-community/bosh-bootstrap/issues/318#event-419348739


Feature Request: Disable scrolling stretch-and-squish (V2)

I'm running V2.0.0 Android 9 on a Samsung S8, and the first thing I did after updating was look for an in-app option to disable the stretchy scrolling animation. The second thing I did was to toggle "System Settings -> Accessability -> Visability enhancements -> Disable Animations", but that didn't do it. The third thing I did was to unlock the developer settings and set all of "Window animation scale", "Transition animation scale", and "Animator duration scale" to "Off" (not my first rodeo :p). This also had no effect, so I figure these four settings must only apply to "system" activities or components vs whatever stack is in-use? Not my area of expertise.
Aside from this and gestures (which I belive I saw an existing issue for, will check out action buttons when they hit fdroid), the app still looks great! Congrats c:

One of the downsides of using Flutter as the development framework for this app is that it doesn't respect system settings as much as native apps. To solve the overscroll effect not matching the expected animation on older Android versions I've overridden the default overscroll indicator to the glowing indicator for Android versions below 12 in 10ddeee4c0ae716f8257419949bfd3a07cc9662a. Will be part of the next release.


Enable configurable timeouts for aws opensearch as well

This PR should resolve this issue https://github.com/toluaina/pgsync/issues/187


Mixer validator

This validator ensures that the Mixer job configuration is correct, data is properly aligned, and filters are valid before starting the actual Mixer job.

Here's the ticket
Here's a sample output

Configuration Validation

Loads and validates the configuration file structure
Checks for required fields and correct data types in the config

S3 Path and Permission Validation

Validates S3 paths for documents, attributes, and output
Checks permissions (read/write) for the specified S3 paths
Verifies the existence of parent directories for output paths

Stream Filter Validation

Validates filter expressions in the configuration
Checks for syntax errors and provides warnings for potential issues

Document and Attribute Alignment

Samples a specified number of documents and their corresponding attribute files
Validates the alignment between document and attribute files:
- Checks that document and attribute files have the same number of lines
- Verifies that both document and attribute files are valid JSONL
- Ensures required fields are present in both document and attribute files

Filter Execution Simulation

Executes filter commands on a sample of attribute files
Reports on the number of lines that pass or are excluded by the filters
Identifies any errors encountered during filter execution

Attribute Name Validation

Cross-references attribute names used in filters with those found in attribute files
Identifies missing attributes and potential typos in attribute names
Provides a comprehensive list of all attributes found in the files

File Sampling and Analysis

Downloads and analyzes a sample of files from the specified S3 paths
Counts lines in both document and attribute files
Validates the structure and content of sampled files

Reporting and Logging

Provides detailed console output for each validation step
Warns about potential issues or misconfigurations
Summarizes the results of each validation phase

Error Handling and Cleanup

Handles errors gracefully and provides informative error messages
Cleans up temporary files and directories after validation

You're going to need to add this file's dependencies to pyproject.toml in order for it to run in a clean environment

You're going to need to add this file's dependencies to pyproject.toml in order for it to run in a clean environment

Addressed this one and added dependencies

The warnings produce a lot of noise, it'd be good to accept a 'verbose' flag and only log the warnings if it's set to true.

The warnings produce a lot of noise, it'd be good to accept a 'verbose' flag and only log the warnings if it's set to true.

Added --verbose flag and hid most of the print statements in it


Added latest gpt models, mistral local and mistral api

In this updated version, I integrated GPT Turbo by making some modifications to the models/gpt.py file. Additionally, I incorporated two methods for utilizing Mistral to fully explore its capabilities: The local model and the Mistral API. The "mistralai" Python package has been included in the install.sh script.
Within the Mistral API, users have the option to choose from three models: tiny, small, and medium. The NLL for Mistral 7b in the local setup was obtained from llama, while it has not yet been implemented for the Mistral API.
In future iterations, the Mistral embedding API can be utilized for tokenization. Although the current version utilizes the existing tokenization method, an update using the Mistral embedding API may be implemented soon.

any question please contact me at<EMAIL_ADDRESS>

LGTM. Thanks for the contribution!


feat: Merge combobox and dropdown

Description
Merge Combobox and Dropdown together. The dropdown has a new attribute "filterable" that allows users to query options.
Fixes #1202
Type of change

[x] Bug fix for current dropdown (keyboard behaviors & width)
[x] New feature (Combobox)
[x] Storybook updated
[x] Add arialabelledby for "input" and "dropdown"
[x] Add "id" for "form-item"

How Has This Been Tested?

[ ] focus should be active after selecting the option.
[ ] unable to type in combo-box when NVDA is active.
[ ] left padding on dropdown should be 12px, currently 8px. (it should use --goa-spacing-s)

[x]  focus should be active after selecting the option.
[ ]  unable to type in combo-box when NVDA is active.
[ ]  left padding on dropdown should be 12px, currently 8px. (it should use --goa-spacing-s)

I tested NVDA on windows 11, looks like it is working. Maybe you have something set up on NVDA different from mine?
https://github.com/GovAlta/ui-components/assets/120135417/328f6aa8-49a0-43d6-972c-d6dee69ba3af

[x]   focus should be active after selecting the option.
[ ]   unable to type in combo-box when NVDA is active.
[ ]   left padding on dropdown should be 12px, currently 8px. (it should use --goa-spacing-s)

I tested NVDA on windows 11, looks like it is working. Maybe you have something set up on NVDA different from mine?
combobox-nvda.mov

[ ] Issue-1: Requires 2 'esc' to remove the content from input box
[ ] Issue-2: After removing content with esc key, unable to type in input box

Steps to Reproduce:

Turn on NVDA.
type 'red' and select 'RED' from the suggestion box.
press 'esc' key and remove the content [ Issue-1 ]
type anything in input box [ Issue-2 ]

Hi @Radha-Alberta
I can re-produce the above issue with the standard example of combobox from w3:
https://www.w3.org/WAI/ARIA/apg/patterns/combobox/examples/combobox-autocomplete-list/
See the below video:
https://github.com/GovAlta/ui-components/assets/120135417/1230994c-4bd9-49f5-8d98-e4c9c44b3bfd
I checked the below issue from Microsoft: https://learn.microsoft.com/en-us/answers/questions/1289668/web-client-escape-key-intercepted-by-web-browser-i
The Escape key is misled by Chrome browser (and Edge too). I will suggest to close this comment as we cannot fix it for now.

:tada: This PR is included in version 2.5.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.14.0-alpha.2 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.14.0-alpha.2 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Winds: initial winds implementation

This is based on this: http://www.dungeonleague.com/2010/03/28/wind-direction/
This PR is based on top of #214
Example of generated winds:

This is based on #214 , right? I will wait until that one is polished and merged before looking at the code in here then.
The map looks interesting. :)


rename src folder to test

The confusing part here is that the tests are in src/ and not in test/. It took me a while to look into that directory, it's not an intuitive place. Is it possible to put the tests in test/ like every JS project?
https://github.com/ipfs/js-ipfs-api/issues/325#issuecomment-243344037 @haadcode

We have had the tests in src because this is a 'module', but if it makes sense for everyone, we can have the folder being exposed by the package.json being the test folder, avoiding the confusion.
👍 by me, anyone has anything against?

I find it confusing if it is in the test folder in here, given that there is no source. So I am :-1: on this change.

I would really like see this to change. src is not the source for this module. It only has tests, so test is the natural place for that code.
Why I brought this up is that every time I can't find documentation for a function, I go and look at the tests. Finding the tests for js-ipfs-api is extremely frustrating as it is now. I've done this multiple times and it happens every time. Mentioning it in the README ("Look for the tests in the src/ directory") is not a solution either as it'll be buried in the README.
Always think: What is the shortest path to the information being looked for?

I am 👍 for this. As @dignifiedquire, there is no source; putting tests in src just to satisfy that requirement seems backward logic for me. Not all modules have to be the same.

This module is a test suite, the source of the test suite sits on its src folder, there are no tests to test the test suite. Hope this clarifies, closing.


[PE-D][Tester E] Date does not allow for single day stay for hotel guests

Description:
A product designed for hotels may be expected to interpret the event of a 1-day stay, but the current product may not be able capture this event properly.
Adding a guest with a duration of 13/12/25 - 13/12/25 cannot be accepted as the second date must be strictly greater than the first date.
Steps to reproduce:

Go to the command box and add a new guest with the command

add n/Betsy Crowe p/82297553<EMAIL_ADDRESS>rm/Aloha5 dr/10/01/12 - 10/01/12 ng/1 rq/Extra towels

Observe that this is not allowed

Suggestion:
Allow the second date to be greater than or equals to the first date instead

Labels: severity.Medium type.FeatureFlaw
original: rui-han-crh/ped#1

This is not a bug it is an intended feature as the check in time is usually before the check out time, hence a guest should not be able to check out on the same day of checking in.


Added Link

Added link to the document where we have further details on how to block the files.
Problem: https://github.com/MicrosoftDocs/windows-itpro-docs/issues/3073

@mjcaparas - can you please assist to check, approve and merge the content for this PR based on issue #3073?
Thank you.


Habitat 0.9.0 support

See individual commit messages for full details.
/cc @jtimberman @metadave @rhass @wrightp

Re-opening this review properly on delivery.chef.co.


[Manual] - The "Based on who you follow" section header disappears after following or unfollowing user on "Search" screen

Commit number: 0e8e7126865d5ac106e04482326f059a9af96579
Preconditions: test user is signed in
Steps to reproduce:

Open "Search" screen
Open "People" tab
Tap on "Following" or "Follow" button for an user from the default list

Actual Result: The "Based on who you follow" section header disappears and empty space is presented instead of the header
Expected Result: The "Based on who you follow" section header is displayed

Verified on a build from commit number 7f3d09256e9b78b6e4cd0207ad47b75f4691d65e


cannot import name 'six' from 'django.utils'

Trying to install Papermerge the bare metal way on my raspberry pi. when i installed everything according to the guide here https://papermerge.readthedocs.io/en/latest/setup/manual_way.html  i get this.
./manage.py migrate
Traceback (most recent call last):
File "./manage.py", line 24, in
execute_from_command_line(sys.argv)
File "/home/pi/Papermerge/.venv/lib/python3.7/site-packages/django/core/management/init.py", line 401, in execute_from_command_line
utility.execute()
File "/home/pi/Papermerge/.venv/lib/python3.7/site-packages/django/core/management/init.py", line 377, in execute
django.setup()
File "/home/pi/Papermerge/.venv/lib/python3.7/site-packages/django/init.py", line 24, in setup
apps.populate(settings.INSTALLED_APPS)
File "/home/pi/Papermerge/.venv/lib/python3.7/site-packages/django/apps/registry.py", line 114, in populate
app_config.import_models()
File "/home/pi/Papermerge/.venv/lib/python3.7/site-packages/django/apps/config.py", line 211, in import_models
self.models_module = import_module(models_module_name)
File "/usr/lib/python3.7/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1006, in _gcd_import
File "", line 983, in _find_and_load
File "", line 967, in _find_and_load_unlocked
File "", line 677, in _load_unlocked
File "", line 728, in exec_module
File "", line 219, in _call_with_frames_removed
File "/home/pi/Papermerge/.venv/lib/python3.7/site-packages/knox/models.py", line 5, in
from knox import crypto
File "/home/pi/Papermerge/.venv/lib/python3.7/site-packages/knox/crypto.py", line 7, in
from knox.settings import CONSTANTS, knox_settings
File "/home/pi/Papermerge/.venv/lib/python3.7/site-packages/knox/settings.py", line 5, in
from rest_framework.settings import APISettings, api_settings
File "/home/pi/.local/lib/python3.7/site-packages/rest_framework/settings.py", line 27, in
from django.utils import six
ImportError: cannot import name 'six' from 'django.utils' (/home/pi/Papermerge/.venv/lib/python3.7/site-packages/django/utils/init.py)

@NovaMoon
That guide you mentioned about is for Ubuntu 20.04 LTS or Debian compatible system.
I know that there are persons who successfully run Papermerge on raspberry pi.
This issue is definitely not a bug.

@NovaMoon
That guide you mentioned about is for Ubuntu 20.04 LTS or Debian compatible system.
I know that there are persons who successfully run Papermerge on raspberry pi.
This issue is definitely not a bug.

@NovaMoon You probably had installed Django's RestFramework previously on your Pi, so you are stuck with an old version incompatible with Django 3.1.
I would suggest removing your current Papermerge installation and following the tutorial again, running python3 -m venv .venv instead of python3 -m venv .venv --system-site-packages.
I believe this should be changed in the setup guide as well @ciur. There are very few reasons why a virtualenv should be initialised with system packages, and it usually causes more problems than it solves by summoning old cruft from the system.

@NovaMoon You probably had installed Django's RestFramework previously on your Pi, so you are stuck with an old version incompatible with Django 3.1.
I would suggest removing your current Papermerge installation and following the tutorial again, running python3 -m venv .venv instead of python3 -m venv .venv --system-site-packages.
I believe this should be changed in the setup guide as well @ciur. There are very few reasons why a virtualenv should be initialised with system packages, and it usually causes more problems than it solves by summoning old cruft from the system.

@NovaMoon You probably had installed Django's RestFramework previously on your Pi, so you are stuck with an old version incompatible with Django 3.1.
I would suggest removing your current Papermerge installation and following the tutorial again, running python3 -m venv .venv instead of python3 -m venv .venv --system-site-packages.
I believe this should be changed in the setup guide as well @ciur. There are very few reasons why a virtualenv should be initialised with system packages, and it usually causes more problems than it solves by summoning old cruft from the system.

this was the solution and everything works now. thanks a lot! i was indeed summoning old cruft.

@NovaMoon You probably had installed Django's RestFramework previously on your Pi, so you are stuck with an old version incompatible with Django 3.1.
I would suggest removing your current Papermerge installation and following the tutorial again, running python3 -m venv .venv instead of python3 -m venv .venv --system-site-packages.
I believe this should be changed in the setup guide as well @ciur. There are very few reasons why a virtualenv should be initialised with system packages, and it usually causes more problems than it solves by summoning old cruft from the system.

this was the solution and everything works now. thanks a lot! i was indeed summoning old cruft.


Include Drive Scope with BigQuery OAuth connection

Include Drive Scope with BigQuery OAuth connection
Issue description
When using OAuth with BigQuery dbt will fail when working with external federated table (Google Sheets/Drive files). The issue with OAuth and external tables seems to be that the Drive API is not explicitly set as the scope of the gcloud SDK.
Google docs with the code sample:
https://cloud.google.com/bigquery/external-data-drive#enable-google-drive
Some extra discussions:
https://github.com/googleapis/google-cloud-python/issues/3067
Results
Running a simple dbt model select * from external_table will produce the following error: Access Denied: BigQuery BigQuery: No OAuth token with Google Drive scope was found.
System information
The output of dbt --version:
installed version: 0.13.1
   latest version: 0.13.1

The operating system you're running on:
macOS 10.13.6
The python version you're using (probably the output of python --version)
3.7.3
Steps to reproduce
profiles.ymlsetting:
bigquery:
  target: dev
  outputs:

    dev:
      type: bigquery
      method: oauth

Thanks for the report @PowdyPowPow. Copy/pasting the results of my investigation from Slack:
This came up internally last week and I took another look at the state of the world. We were able to get this working using separate service accounts, but I agree, it would be great if this worked over OAuth!
We’re actually (trying) to set scopes when we construct the BigQuery client: https://github.com/fishtown-analytics/dbt/blob/dev/wilt-chamberlain/plugins/bigquery/dbt/adapters/bigquery/connections.py#L63-L65
A lot of the examples in this issue aren’t actually that helpful. This one is for service accounts only, for instance: https://github.com/googleapis/google-cloud-python/issues/3067#issuecomment-364103069
I’d definitely love to get this working, but I’m pretty stumped!! If anyone has any experience setting up sheets access over oauth, I'd love to hear about it

You can add Drive scope via OAuth. The Drive scope needs to be added at the end of the OAuth command.
Use:
gcloud auth application-default login --scopes=https://www.googleapis.com/auth/userinfo.email,https://www.googleapis.com/auth/cloud-platform,https://www.googleapis.com/auth/drive.readonly
Notice the addition of “- -scopes” flag and “https://www.googleapis.com/auth/drive.readonly” at the end. That’s the additional scope. The other two scopes are the default scopes.
Now if your user has view access to a sheets file you should have no trouble running don’t locally.
I learned about the “- -scopes” flag here: https://cloud.google.com/sdk/gcloud/reference/auth/application-default/login
I found the Drive scope here: https://developers.google.com/identity/protocols/googlescopes#drivev3
@PowdyPowPow - Happy to help you debug if you run into any issues. Good luck.
@drewbanin - Could be useful to include this in dbt docs as another OAuth command (and potentially the default recommendation).

Thanks @ciscodebs! I just updated the dbt documentation with this information :)

Thanks @ciscodebs
I was actually playing with the scopes but almost certainly never used the same combination as you suggest. I tried it just now and can confirm it works. Thank you.

@drewbanin - since this was added to docs should it remain open?

@ciscodebs yep! Thanks for the heads up :)

I have used the commands to oauth for BigQuery. I can create data sets and tables in UI but when I run dbt then I get access denied. I just upgraded to 0.17 version. Any idea what could go wrong?

hey @LubiBel - did you follow the steps shown here? https://docs.getdbt.com/docs/supported-databases/profile-bigquery/#oauth-authorization
What does your access denied error message look like?

hey @LubiBel - did you follow the steps shown here? https://docs.getdbt.com/docs/supported-databases/profile-bigquery/#oauth-authorization

This link is broken @drewbanin

@techadmin-getsales Looks like it's here now: https://docs.getdbt.com/reference/warehouse-profiles/bigquery-profile/#oauth-authentication

I am also receiving the same error. Any suggestions please?

Based on https://stackoverflow.com/questions/68487890/google-cloud-bigquery-client-ignoring-provided-scopes-resulting-in-permission, I tried:
gcloud auth application-default login --scopes=\
openid,\
https://www.googleapis.com/auth/userinfo.email,\
https://www.googleapis.com/auth/cloud-platform,\
https://www.googleapis.com/auth/drive,\
https://www.googleapis.com/auth/bigquery

This worked for me.


Crash on starting a TtA world - NPE

Since the last stable we seem to have broken TtA once more. I verified the master branch is fine. It might be all the refactoring for commands, maybe something permission related, I saw some more detailed errors at once point but now get a very vague one:
23:07:47.556 [main] INFO  o.t.w.b.internal.BlockManagerImpl - Registered Block Core:DoorBottom.BACK with id 16
23:07:47.565 [main] ERROR o.t.p.serializers.PrefabSerializer - Prefab contains unknown component 'Miniaturizer'
23:07:47.623 [main] ERROR o.t.p.serializers.PrefabSerializer - Prefab contains unknown component 'UnbreathableBlock'
23:07:49.482 [main] ERROR o.t.e.event.internal.EventSystemImpl - Invalid event handler method: componentActivated
23:07:49.489 [main] ERROR o.t.e.event.internal.EventSystemImpl - Invalid event handler method: componentActivated
23:07:50.440 [main] ERROR o.t.module.sandbox.ModuleClassLoader - Denied access to class (not allowed with this module's permissions): org.terasology.world.sun.OnMidnightEvent
23:07:51.297 [main] WARN  o.t.engine.internal.TimeBase - Delta too great (3613), capping to 1000
23:07:52.585 [main] INFO  o.t.e.m.l.InitialiseWorld - World seed: "3h3XSDUGZxwVEM9V6UNjkk9DH3l1RGCh"
23:07:53.301 [main] ERROR o.terasology.engine.TerasologyEngine - Uncaught exception, attempting clean game shutdown
java.lang.NullPointerException: null
	at org.terasology.engine.modes.loadProcesses.InitialiseWorld.step(InitialiseWorld.java:144) ~[classes/:na]
	at org.terasology.engine.modes.StateLoading.update(StateLoading.java:235) ~[classes/:na]
	at org.terasology.engine.TerasologyEngine.mainLoop(TerasologyEngine.java:440) ~[classes/:na]
	at org.terasology.engine.TerasologyEngine.run(TerasologyEngine.java:371) ~[classes/:na]
	at org.terasology.engine.Terasology.main(Terasology.java:137) [classes/:na]
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[na:1.7.0_55]
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57) ~[na:1.7.0_55]
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[na:1.7.0_55]
	at java.lang.reflect.Method.invoke(Method.java:606) ~[na:1.7.0_55]
	at com.intellij.rt.execution.application.AppMain.main(AppMain.java:134) [idea_rt.jar:na]
23:07:53.301 [main] INFO  o.terasology.engine.TerasologyEngine - Shutting down Terasology...

However it is possible the earlier error was just some sort of dirty workspace issue.
The org.terasology.world.sun.OnMidnightEvent permission error is not new - TtA has worked with that in place before, although it certainly seems like something we need to fix.
The componentActivated errors on the other hand I haven't noticed before. That method appears to only be in use in two classes from the ClimateConditions module, which is part of TtA, so maybe that's the culprit. Not sure why they are getting flagged as invalid event handlers.

Linking to #1375 since it is similar functionality, might be worth looking at together with this (no direct relation that I know of though)

The org.terasology.world.sun.OnMidnightEvent was fixed by #1485 but that indeed didn't fix this crash for me :-(

The mystery thickens.
TTA works as-is for @Limeth on Win7 from source with just the main TTA module present in source form.
It fails for me set up like that, in my mega-integration workspace with everything in source, in the latest Omega zip, and even on my tablet with the very latest Omega zip, or on my wife's PC.
Maybe it is machine specific. Tagging as support in that case.

I get a different NPE - PluggableWorldGenerator initializes the world differently from the other world gens. The world gen is not yet initialized in InitialiseWorld which tries to get the sea level height from it. Will look into that (on Wednesday probably)


Understanding RLDS and translating JAT datasets to TFDS format

Adding a Notebook that details how to view and understand RLDS and JAT datasets. It also includes code to translate a HuggingFace JAT dataset to the TFDS format. It can be seen towards the end of the Notebook that the MuJoCo datasets from both sources are in the same format and have very similar representations of actions, observations, rewards.

Looks good to me! Great work on this @pranavguru


Versions + Branches

Node.js - I'm confused.
In your dist directory under nodejs there is a uws.js file.
That file is dated yesterday. And shows you've bumped the version to 0.12.0.
I install uws with npm, specifying 0.12.0 and I get a uws.js dated 3 Dec 2016. The api offered by this uws is nothing like that illustrated in your examples. For example uws.http.createServer is met with an error on uws.http being "undefined".
Am I doing something completely stupid? Or would it make sense to move your node.js code into its own repo and put it under version control?

As is written in the post, version 0.13.0 is not yet released to NPM. As is also written, you need to compile the addon youtself for now. I am working on it.

Ah cool, thx.
Great work by the way - I'm testing w/ the existing version for now and the performance / stability seem excellent.


[OpenBit] Add support for Syscoin NEVM network

If this is your first time integrating, the following information is required

Whitepaper: https://syscoin.org/research-whitepapers
Social media links (website)
Any other useful material so we can get to know you better: https://docs.syscoin.org/docs/intro/nutshell/
Github: https://github.com/syscoin
Faucet: https://faucet.syscoin.org/

If you want to integrate a chain, you would need to provide

Name of the blockchain: Syscoin NEVM
Type of the blockchain: EVM
Blockchain providers: https://rpc.syscoin.org
Metadata of the blockchain (chainId, paraId, addressPrefix, symbol,...)
Please check here: https://docs.syscoin.org/docs/guides/nevm/metamask/#mainnet
Logo of the blockchain: https://www.figma.com/design/kx0dqXMzHmODOfWm9WMys0/02.OpenBit---Logo-Project?node-id=2439-156&t=OXDzDSYV2GeL6WJ0-4
Block explorers: https://explorer.syscoin.org/

https://explorer.syscoin.org/accounts


[Release-1.21] etcd-snapshot subcommand does not out debug logs even when --debug flag is provided

Backport #4920 to release-1.21

Verified in k3s Release 1.21 using commit 900e5ff519cc4e5c1e699714f253fbbe68c742a1 and observed the console log below:
sudo k3s etcd-snapshot save --debug
INFO[2022-01-18T20:36:59.717465196Z] Managed etcd cluster bootstrap already complete and initialized
W0118 20:36:59.742036    7749 warnings.go:70] apiextensions.k8s.io/v1beta1 CustomResourceDefinition is deprecated in v1.16+, unavailable in v1.22+; use apiextensions.k8s.io/v1 CustomResourceDefinition
W0118 20:36:59.758809    7749 warnings.go:70] apiextensions.k8s.io/v1beta1 CustomResourceDefinition is deprecated in v1.16+, unavailable in v1.22+; use apiextensions.k8s.io/v1 CustomResourceDefinition
W0118 20:36:59.764893    7749 warnings.go:70] apiextensions.k8s.io/v1beta1 CustomResourceDefinition is deprecated in v1.16+, unavailable in v1.22+; use apiextensions.k8s.io/v1 CustomResourceDefinition
DEBU[2022-01-18T20:36:59.772833430Z] Attempting to retrieve extra metadata from k3s-etcd-snapshot-extra-metadata ConfigMap
DEBU[2022-01-18T20:36:59.779465326Z] Error encountered attempting to retrieve extra metadata from k3s-etcd-snapshot-extra-metadata ConfigMap, error: configmaps "k3s-etcd-snapshot-extra-metadata" not found
INFO[2022-01-18T20:36:59.788966289Z] Saving etcd snapshot to /var/lib/rancher/k3s/server/db/snapshots/on-demand-ip-172-31-44-58-1642538219
{"level":"info","msg":"created temporary db file","path":"/var/lib/rancher/k3s/server/db/snapshots/on-demand-ip-172-31-44-58-1642538219.part"}
{"level":"info","ts":"2022-01-18T20:36:59.791Z","caller":"clientv3/maintenance.go:200","msg":"opened snapshot stream; downloading"}
{"level":"info","msg":"fetching snapshot","endpoint":"https://<IP_ADDRESS>:2379"}
{"level":"info","ts":"2022-01-18T20:36:59.819Z","caller":"clientv3/maintenance.go:208","msg":"completed snapshot read; closing"}
{"level":"info","msg":"fetched snapshot","endpoint":"https://<IP_ADDRESS>:2379","size":"2.2 MB","took":"36.274525ms"}
{"level":"info","msg":"saved","path":"/var/lib/rancher/k3s/server/db/snapshots/on-demand-ip-172-31-44-58-1642538219"}
INFO[2022-01-18T20:36:59.836975416Z] Reconciling etcd snapshot data in k3s-etcd-snapshots ConfigMap
DEBU[2022-01-18T20:36:59.837022214Z] core.Factory is initialized
DEBU[2022-01-18T20:36:59.838729043Z] Attempting to reconcile etcd snapshot data for configmap generation 0
DEBU[2022-01-18T20:36:59.839083173Z] Updating snapshot ConfigMap (k3s-etcd-snapshots) with 2 entries
INFO[2022-01-18T20:36:59.842330241Z] Reconciliation of snapshot data in k3s-etcd-snapshots ConfigMap complete


report.ejs and disclaimer.ejs translations.

Before merging please look at lines #36 and #139. I dynamically filled in the hostname instead of using the translations to allow easier support of sites for different countries with the same language (i.e. uk and usa).

You could also consider using <%= locals.baseUrl %>, https://github.com/BustByte/coronastatus/blob/master/app/views/partials/head.ejs#L6

copied this over into the branch manually due to merge conflicts. Thanks!


Update fastlane to remove Xcode 13 use

Pull request
Description
This PR updates fastlane to remove the need to use Xcode 13 for TestFlight binary submission.
Changes made

Update fastlane and Fastfile.

Checklist

[x] Your branch has been rebased onto the main branch.
[x] APIs have been properly documented (if relevant).
[x] The documentation has been updated (if relevant).
[x] New unit tests have been written (if relevant).
[x] The demo has been updated (if relevant).

🌙 Available in TestFlight nightlies
The most recent state can be tested in the following builds:

Platform
Version

iOS
-

tvOS
1.0.0 (91)


RiotTS does not work with typescript 1.6

I am running typescript 1.6.2 and it fails to compile with your code. It seems that typescript 1.6 no longer support @template annotation. Can you fix that?

strange, I am sure I use typescript@next builds, but will check it.

Update: I checked and it works with TS 1.6 (typescript@next).
More likely what you are referring to is the --experimentaDecorators flag (I am guessing).
Since @decorators are not a stable TypeScript feature they must be "enabled". If you are using a VS solution, edit your .csproj this way:
  <PropertyGroup Condition="'$(Configuration)' == 'Debug'">
    <TypeScriptExperimentalDecorators>true</TypeScriptExperimentalDecorators>   // <-- add this line
  </PropertyGroup>
  <PropertyGroup Condition="'$(Configuration)' == 'Release'">
    <TypeScriptExperimentalDecorators>true</TypeScriptExperimentalDecorators>   // <-- add this line
  </PropertyGroup>
``

Please let me know if this fixes your problem.

I am working on Mac and cannot have .csproj file to edit. If @decorators is not a stable feature, I think they might deprecate it in the future. Have you thought about that?

Decorators is a feature that is going to be added to the ES7 standard, but since it's not official yet, they added the "experimentalDecorators" flag as a reminder for the developer (as they might change some implementation details).
I don't think they'll drop it as everybody are using decorators; consider for example that the Angular2 team switched from AtScript to TypeScript once it started to support decorators.
Back to your issue:

if you compile from the command line, just add the switch --experimentalDecorators.

if you use a tsconfig.json file (highly recommended), just add the line "experimentalDecorators": true under compilerOptions, e.g.

"compilerOptions": {
    "experimentalDecorators": true
  }

Can you create an example on github of how to use your life without VS? I was trying to compile one of the example file and got an error: error TS2304: Cannot find name 'template'.. My command is tsc --experimentalDecorators my-element.ts.
My code is very simple:
@template("my-element.html")
class MyElement extends Riot.Element
{
}

// after the element is defined, we register it in Riot
MyElement.register();

If you clone this repo and go in the Test sub-directory you can build the "specRunner" (the page that runs the tests) from the command line without Visual Studio.
Just launch tsc with no parameters, e.g.
$ cd Test
$ tsc

That will produce specRunner.js that is used by specRunner.html (which you can open it in the browser).
This is the tsconfig.json file that makes it possible:
{
    "compilerOptions": {
        "removeComments": true,
        "out": "specRunner.js",
        "target": "ES5",
        "experimentalDecorators": true,
        "sourceMap": true
    },
    "files": [
        "bower_components/riot-ts/riot-ts.d.ts",
        "elements/specElements.ts",
        "specRunner.ts"
    ]
}

Note the experimentalDecorators flag and the inclusion of riot-ts.d.ts among the files to compile.
Let me know if this is enough for you to get started.

:+1: Solved my issue. I like your lib. It makes my development much easier. I hope you can invest more time to make this a more mature lib.


Java Resources not included from jvmMain/resources

@jonamireh reported that if you have a proguard file in src/jvmMain/resources/proguard/rules.pro, it won't be in the resulting JAR.
We need to make sure we also map the resource directories over along with the java and kotlin directories.

Fixed in the following PRs:

https://github.com/handstandsam/kmp4free/pull/13 by @jonamireh
https://github.com/handstandsam/kmp4free/pull/14 by @handstandsam


Removed unused code from sample code in docs

This is now consistent with the live code in the sample app

Codecov Report

Merging #63 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #63   +/-   ##
======================================
  Coverage    91.7%   91.7%
======================================
  Files          46      46
  Lines        4784    4784
======================================
  Hits         4387    4387
  Misses        397     397

Continue to review full report at Codecov.

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

LGTM. @dustin-shean-linkedin can you take a look/merge?

Thanks @katalisha for the PR :)


Remove the need to use Connection to talk to internal LSPs

SourceKitServer now talks to the SwiftLanguageServer and
ClangLanguageServerShim directly through the LanguageService
protocol.
I have some TODOs left in the code to double check the threading
as I'm not quite sure how it should behave. Should each internal
LSP be responsible for handling the threading?

Should each internal LSP be responsible for handling the threading?

I think this is the only way to handle it. While we currently always initiate the calls from the SourceKitServer's queue, the async callbacks from e.g. sourcekitd can still come in anytime, and some of those require access to shared state.  So I  think we still want them all to have their own queue, but now you would need to use that queue in each of these individual methods if you're not funneling through the code that used to do that.
It's unfortunate that we don't have better language support for this and will need boiler plate around the queue handling, but I don't really see an alternative if we don't have a single entry point to dispatch through anymore.

Ok, believe I've fixed the threading issues by having all requests/responses in SwiftLanguageServer move to its own queue as well as handling callbacks from the secondary LSPs in SourceKitServer's own queue. Let me know if there's any place where I should swap from strong -> weak references for the queue async blocks.

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

Looks good, thanks!  For additional confidence, I ran the  tests a few dozen times with thread sanitizer enabled and didn't see any issues.


Update regex for AWS secret key (issue #672)

Description:
resolves #672
Checklist:

[x] Does your PR pass tests?
[x] Have you written new tests for your changes?
[x] Have you lint your code locally prior to submission?

The test cases did all exist already, they just missed detecting the AWS secrets.

I was debugging a false positive and found this new change to the rule. I wonder if there was a particular reason we make '=' optional?
In my case, the false positive was from terraform:
data "aws_iam_policy" "AmazonDynamoDBFullAccesswithDataPipeline" {
  ...
}

Hello @khanhldt ,
I don't know the reason, however, you can force to use v7.6.1 instead of latest commit in order to make the false positive go away.
Will ping @zricethezav and @dirkaholic in order to review this PR again.
Thank you 😄


790 popup message delete super event

Description
Added deletion message in popup window. Checks isSuperEvent flag and display deletion message accordingly
Fixes #790
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[x] Within a week
[ ] When possible

Type of changes
Please select a relevant option:

[ ] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[ ] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[ ] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

can you make a branch from develop branch? and refer to the implementation of https://github.com/rokwire/events-manager/pull/793?

can you make a branch from develop branch? and refer to the implementation of #793?

reverted to develop branch

@minump please fix the conflicts, then we will merge it. thanks.


Ajax requests are not considered

When using ajax requests these requests are not integrated into debugbar, because they are not considered.
This Pullrequest fixes this ( PR 40 ).
Do you have a timeframe when this will be merged and released?

Its rendered in my application, the bars are stacked one on top of the other so you end up with one for the original request, then it stacks one for each ajax request or at least that is the behavior I was getting.

Yes, correct. But this is faulty and this PR fixes that. As the debugbar module can handle ajax requests, but this bridge does not use that funcionality. The PR adds that needed parameters.

[offtopic]
I thought it was ironic we used the same exact variable name: isAjax.
[/offtopic]

The PR is not from me, but yes I did notice that. ;-)


⚠ 366.ru has degraded performance

In 91464ff, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 1020 ms

Resolved: 366.ru performance has improved in f0195e5.


Most recent code not available through npm

Last recent version of vue-drag-resize is 1.3.2. But this version does not contain the snapToGrid feature which is documented in the docs. Could you release a new version with most up to date code?

Yeah,please update your newest version code,add snapToGrid feature to it.

@kirillmurashov Could you please publish the new version?

Today is April 1, 2020, and April Fool's Day, is this reponsitory abandoned?

fix


we need a announcement for python-wechaty plugin system

As mentioned by #69 huan, we need a blog post or even a video seminar to announce our Python Wechaty Plugin system so that more developers will get to know and use it!
Are there are some volunteers to complete that ?  @huangaszaq @fish-ball

Yes, it's very valuable to make those article and events because it is s great way to let others know more about us.
I'm looking forward to seeing it!


[Bug] SPA Mode (Popup not showing)

In SPA mode, the popups won't appear again after the first time you interact with the page.

Was just about to open an issue for this :) Do you know of any workarounds?

Turn off spa locally for the time being.

Keeping it on has helped me catch a few edge cases with SPA mode. No worries, will keep an eye out for updates.


STM32L072KBU6 BOM issue

@tomipiriyev Do you have any idea to redesign the Air Unit schematic by using either RP2040 (Raspberry pi Pico) or ESP32 or both since the STM32L072KBU6 controller will not be available atleast until next year February.
TIA.

@asprakash RP2040 cannot be used in this project,  even at deep sleep it has high power consumption.
STM32L0 we have supplier

What provider recommend me for the STM32L0 and if all the pieces in the same supplier are possible?

@josmelmesa  I used to buy 95% from http://lcsc.com.  but now because of silicon shortage it is hard to source from 1 supplier.


Webhook subscription is not sent for already started service

Description:
import githubwebhook3;
import ballerina/http;
import ballerina/io;
import ballerina/oauth2;
import ballerina/websub;

githubwebhook3:Listener githubListener = new (8080);

// following globle variables are added as a workaround until annotaions can take local variables
string _topic = "";
string _secret = "";
boolean isStarted = false;

oauth2:OutboundOAuth2Provider _oauth2Provider = new ({
    accessToken: ""
});
http:BearerAuthHandler _oauth2Handler = new (_oauth2Provider);

public function startWebhook(string topic, string secret, string token) {

    _topic = topic;

    oauth2:OutboundOAuth2Provider oauth2Provider = new ({
        accessToken: token
    });
    http:BearerAuthHandler oauth2Handler = new (oauth2Provider);

    _oauth2Handler = oauth2Handler;

    service ss = @websub:SubscriberServiceConfig {
        path: "/",
        subscribeOnStartUp: true,
        target: [githubwebhook3: HUB, _topic],
        secret: "<SECRET>",
        callback: "https://00025cd0.ngrok.io",

        hubClientConfig: {
            auth: {
                authHandler: _oauth2Handler
            }
        }
    }
    @websub:SpecificSubscriber
    service {
        resource function onPush(websub:Notification notification, githubwebhook3:PushEvent event) {
            io:println("[onPush] pushed by: ", event.sender);
        }
    };

    checkpanic githubListener.__attach(ss);
    if (isStarted == false) {
        isStarted = true;
        checkpanic githubListener.__start();
    }

}

public function main() {
    io:println("before");
    startWebhook("https://github.com/my/myRepo/events/*.json", "", "mytoken");
    io:println("after 1");
    startWebhook("https://github.com/my/my2ndRepo/events/*.json", "", "mytoken");
    io:println("after 2");
}

All code lines in above main method are getting executed but the subscription request is not sent for the 2nd repo.
The requirement is to sent multiple subscriptions to multiple repos over the time while listening for callbacks.
Affected Versions:
0.992.0-m2-SNAPSHOT

Currently if subscribeOnStartup is set to true in the annotation, we send a subscription request when .__start() is called on the listener the service is attached to.
Since attaching is done here after the listener is started, the subscription request would also have to be sent programatically. This can be done using the hub client - https://github.com/ballerina-platform/ballerina-lang/blob/v1.0.0-alpha/examples/websub-hub-client-sample/subscription_change_client.bal#L18.
Please reopen this issue if the solution doesn't address the concerns.


src: update func go runtime dep

:broom: updates func runtime dependency
/kind cleanup

/lgtm

@lkingland it looks like newer version of runtime is not Windows compatible. lkingland/func-runtime-go@0fd4ffe -- the signal is not defined on windows.

Thanks! I updated the filter to only apply for linux, and to compare by hex

@lkingland  The TestBuilder_Concurrency test is failing too often at Windows, isn't there really a bug?

@lkingland I believe there is really a bug on Windows. I can reproduce it.

a) It looks like "normal" user cannot create symlink:
=== RUN   TestBuilder_Concurrency
mkdir -p C:\Users\mvasek\AppData\Local\Temp\TestBuilder_Concurrency1529383575\001\.func\builds\by-hash\b097429205e010f2b4a96d9a80860c0335bd0d961
f761ead9e3ba79396ace1ce
mkdir -p C:\Users\mvasek\AppData\Local\Temp\TestBuilder_Concurrency1529383575\001\.func\builds\by-pid
ln -s ..\by-hash\b097429205e010f2b4a96d9a80860c0335bd0d961f761ead9e3ba79396ace1ce C:\Users\mvasek\AppData\Local\Temp\TestBuilder_Concurrency1529
383575\001\.func\builds\by-pid\8220
test build error symlink ..\by-hash\b097429205e010f2b4a96d9a80860c0335bd0d961f761ead9e3ba79396ace1ce C:\Users\mvasek\AppData\Local\Temp\TestBuil
der_Concurrency1529383575\001\.func\builds\by-pid\8220: A required privilege is not held by the client.panic: test timed out after 10s

b) even when running as an admin there is another issue:
=== RUN   TestBuilder_Concurrency
mkdir -p C:\Users\mvasek\AppData\Local\Temp\TestBuilder_Concurrency1347840251\001\.func\builds\by-hash\d9434b8b4ec1e5e485daa9a248a3c59272b36f0c8
cce8b41d58ef305bae69c56
mkdir -p C:\Users\mvasek\AppData\Local\Temp\TestBuilder_Concurrency1347840251\001\.func\builds\by-pid
ln -s ..\by-hash\d9434b8b4ec1e5e485daa9a248a3c59272b36f0c8cce8b41d58ef305bae69c56 C:\Users\mvasek\AppData\Local\Temp\TestBuilder_Concurrency1347
840251\001\.func\builds\by-pid\19300
ΓåÆ /func
ΓåÆ /func/README.md
ΓåÆ /func/func.yaml
ΓåÆ /func/go.mod
ΓåÆ /func/handle.go
ΓåÆ /func/handle_test.go
mv .\datalayer.tar.gz .\oci\blobs\sha256\2a2c63a63996284d9070f746d9225c6db3f10040404a318d315410e9990a2e01
ΓåÆ /etc/ssl/certs/ca-certificates.crt
ΓåÆ /etc/pki/tls/certs/ca-certificates.crt
mv .\certslayer.tar.gz .\oci\blobs\sha256\3650df3d1bb6e357be8bca5bdc04df36f9e3b973fb221d9cdce3193ae10a08c7
rm -rf C:\Users\mvasek\AppData\Local\Temp\TestBuilder_Concurrency1347840251\001\.func\builds\by-hash\d9434b8b4ec1e5e485daa9a248a3c59272b36f0c8cc
e8b41d58ef305bae69c56
test build error remove C:\Users\mvasek\AppData\Local\Temp\TestBuilder_Concurrency1347840251\001\.func\builds\by-hash\d9434b8b4ec1e5e485daa9a248
a3c59272b36f0c8cce8b41d58ef305bae69c56\config.json: The process cannot access the file because it is being used by another process.mv .\config.j
son .\oci\blobs\sha256\f72f75e96f760d68fafd1c64a01752c66f18c5a6a5791f938927b833750d944e
rm C:\Users\mvasek\AppData\Local\Temp\TestBuilder_Concurrency1347840251\001\.func\builds\by-pid\19300
rm C:\Users\mvasek\AppData\Local\Temp\TestBuilder_Concurrency1347840251\001\.func\builds\by-hash\d9434b8b4ec1e5e485daa9a248a3c59272b36f0c8cce8b4
1d58ef305bae69c56
test build error rename C:\Users\mvasek\AppData\Local\Temp\TestBuilder_Concurrency1347840251\001\.func\builds\by-hash\d9434b8b4ec1e5e485daa9a248
a3c59272b36f0c8cce8b41d58ef305bae69c56\config.json C:\Users\mvasek\AppData\Local\Temp\TestBuilder_Concurrency1347840251\001\.func\builds\by-hash
\d9434b8b4ec1e5e485daa9a248a3c59272b36f0c8cce8b41d58ef305bae69c56\oci\blobs\sha256\f72f75e96f760d68fafd1c64a01752c66f18c5a6a5791f938927b833750d9
44e: The system cannot find the path specified.panic: test timed out after 10s

It's probably not introduced by this PR but still needs fixing.

@lkingland I believe there is really a bug on Windows. I can reproduce it.

That's great news you can reproduce!

/approve
/lgtm

/override "Unit Test (1.20.2, 17, windows-latest)"


Added ability to specify page language and add classes to the  tag

@benhurrell - CC'ing you on this PR, so you don't go ahead and implement!


Insert Step from Visualization

Add the ability to insert a new Step between two existing Steps from the Visualization. There are two ways the user should be able to do this:

The user clicks a "+" button on the last step, which inserts a slot between the two steps. The user can then drag a step from the Catalog onto the slot.
The user clicks a "+" button on the last step, which inserts a slot between the two steps. A mini catalog (#216 ) opens up, and the user can then choose the step to be aggregated/appended.

It's not entirely clear if both ways will be possible or intuitive to the user. A story should be created for each to see which option would be best to start with.
See this issue for more information: #218 <-- we should be following the same decision for both ways of adding a step

Steps

[ ] Add validation for inserting a step
[ ] Add custom edge
[ ] Add logic to show a slot
[ ] Create stories for both scenarios described above
[ ] Implement changes onto Visualization

Shouldn't this be closed with #216 ?

#342 seems closed. This should unblock the issue, right?

#342 seems closed. This should unblock the issue, right?

@lhein sheesh it was just merged today 😂

#342 seems closed. This should unblock the issue, right?

@lhein kidding aside, yes this should unblock it for @mmelko

:D I did not check dates. Was just curious ;)

:partying_face: :balloon:


Extension Markdown pages are rendering  with Courier font

Issue Type: Bug
When viewing ReadMe's of various extensions, if the ReadMe doc is using  tags to render Keyboard shortcuts, the font used is less-than-desirable Courier New.
A good repro to look at would be the C# Extension's Readme within VS Code, and the "Found a bug?" section near the bottom.

When I copied/pasted this text as raw HTML, I noticed that there was no [font-family] css style set for the  elements, unlike the HTML from  tags.
 What do you propose as the alternative?

Perhaps could use the same font used in other fixed-width Markdown styles, like the `backtick code` format that's seen in the same screenshot (but using the text's default color).
Even GitHub itself right here uses a decent KBD tag style format:
Ctrl+C   ===>  Ctrl+C .
My request for the bug is to only set the font to use the same ones as code blocks.  The existing boundary outline style, text color and background colors are perfect.
This is what the <kbd style= info looks like in HTML:  (...no font style info)
Ctrl

To compare: here's what the <code style= info looks like in HTML:
CSharp: Report an issue

Yes, the  tag also defines Courier New, but it's way down the list.  Consolas has the priority (on Windows anyways).
I looked at this file: /src/vs/workbench/contrib/markdown/browser/markdownDocumentRenderer.ts, and it didn't define any style for kbd, though it defined some for code.
The files: index.html and index-no-csp.html in path /src/vs/workbench/contrib/webview/browser/pre/index.html, appear to define the kbd style there.
Hope this helps.

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


按照示例使用useTrackAnimation时at init (useTrackAnimation.js:28:48)报错，Uncaught TypeError: Cannot read properties of undefined (reading 'getMapInstance')

Describe the bug A clear and concise description of what the bug is.

same as https://github.com/yue1123/vue3-baidu-map-gl/issues/54


What should be the internal representation of DerivationPath?

Firstly, I'd love to just not have it since it's the same as slice/vec containing sufficiently specific types. But we probably really need it to implement FromStr and Display. For Display we'd ideally have an unsized version too. (or rather have DerivationPath and DerivationPathBuf)
But there's one more thing. With the addition of NormalDerivationPath there's optimization opportunity to not reallocate when converting between the paths. If DerivationPath contained u32s that are converted on the edges we could make one conversion zero-cost (just unsafe cast the types) and the other would only cost checking the numbers. But to do this we must lose AsRef<[ChildNumber]> and Index impls. We can still have accessors of individual numbers (just converting) and with the unsized type it shouldn't be a big deal. (But we're doing the exact opposite of what I'd prefer ideally.)
I wonder if it's worth it.

I think losing AsRef<[ChildNumber]> is probably fine. This functionality can mostly be emulated if we have a way to iterate over DerivationPaths yielding the various childnumber types, and if we implement ops::Index<usize> to return a ChildNumber (maybe doing some conversions to do so).
We won't be able to implement ops::Index<ops::Range*<usize>> for the same reasons we can't implement AsRef but I think this is a pretty rare thing for people to want.

if we implement ops::Index<usize> to return a ChildNumber (maybe doing some conversions to do so).

That's impossible because we have to return a reference.

We won't be able to implement ops::Index<ops::Range*<usize>>

We could if we had unsized DerivationPath.

if we implement ops::Index<usize> to return a ChildNumber (maybe doing some conversions to do so).

That's impossible because we have to return a reference.

Damn. I should've checked that before posting. Frustrating.
We can provide our own .get or .nth function I guess.

We won't be able to implement ops::Index<ops::Range*<usize>>

We could if we had unsized DerivationPath.

If we had unsized DerivationPath and if whatever childnumber types we might want to return were #[repr(transparent)] wrappers on usize? I like this idea but it's a pretty strong API limitation. I think it's fine for our childnumbers to be repr(transparent) but making DerivationPath unsized also means that it basically can't have any non-zero-sized fields execept the unsized one. So we can't put any extra metadata or anything into it. Maybe that's ok.
(Technically Rust allows structs with arbitrary fields to be unsized; you just make the last field be [u8] or dyn Trait or whatever. But good luck constructing such a thing. AFAICT you can't do it without wading waaay into the weeds of manual memory allocation and messing with Layout and so on. I tried this in rust-simplicity with our Value type recently and couldn't do it.)

We can provide our own .get or .nth function I guess.

Yes. Also maybe one day Index will support non-references since we have GAT today.

whatever childnumber types we might want to return were #[repr(transparent)] wrappers on usize?

u32 rather.

making DerivationPath unsized also means that it basically can't have any non-zero-sized fields execept the unsized one

I'd want to have *Buf types too. (so what is currently DerivationPath would become DerivationPathBuf - exactly what we did with Script.)

But good luck constructing such a thing. AFAICT you can't do it without wading waaay into the weeds of manual memory allocation and messing with Layout and so on.

Actually it can be very simple if you know the length statically - just create a generic struct, initialize it with sized array and then castt it. But that's not possible for custom types. IIRC I've seen some advanced hacks around this that I was considering for ControlBlock.

All sounds great to me. And looking forward to seeing any advanced hacks you can find in the future.

If DerivationPath contained u32s that are converted on the edges we could make one conversion zero-cost (just unsafe cast the types) and the other would only cost checking the numbers. But to do this we must lose AsRef<[ChildNumber]> and Index impls.

I think its a great trade-off, i would argue that access to internals, besides being unsafe, certainly pass an idea that the developer using the lib might need it, we are not offering all the methods needed to use the implementation. If any feature, that we dont have, needs some access to internals, sure thing that we will be notified with a opportunity to improve the lib


Added test_duckdb.ipynb

Description of what I changed

Fixes #987
E2E test

TESTED:
Built the docker image locally, i.e., from the query/ dir:
docker build -t my-fhir-views .

then ran and tested the created image:
docker run -p 10002:8888 --network cloudbuild -v "[PATH]/docker/dwh:/dwh" --name fhir-views my-fhir-views

Checklist: I completed these to help reviewers :)

[x] I have read and will follow the review process.

[x] I am familiar with Google Style Guides for the language I have coded in.
No? Please take some time and review Java and Python style guides.

[x] My IDE is configured to follow the Google code styles.
No? Unsure? -> configure your IDE.

[ ] I have added tests to cover my changes. (If you refactored existing code that was well tested you do not have to add tests)

[x] I ran mvn clean package right before creating this pull request and added all formatting changes to my commit.

[x] All new and existing tests passed.

[x] My pull request is based on the latest changes of the master branch.
No? Unsure? -> execute command git pull --rebase upstream master

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

Project coverage is 50.60%. Comparing base (498d7c1) to head (34e124b).

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1064   +/-   ##
=========================================
  Coverage     50.60%   50.60%
  Complexity      673      673
=========================================
  Files            91       91
  Lines          5505     5505
  Branches        707      707
=========================================
  Hits           2786     2786
  Misses         2457     2457
  Partials        262      262

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update inscriptions.json for ordinal-cubes-by-haus-hoppe

All new mints (total of 112 cubes now). Thanks! 😊👍

Closing this PR in favour of https://github.com/ordinals-wallet/ordinals-collections/pull/3974 (118 cubes).