@BeforeGroups called twice

public class testNG {

    @BeforeGroups(groups = {"smoketests", "functionaltests"})
    public void before() {
        System.out.println("Before Groups");
    }

    @Test(groups = {"smoketests", "functionaltests"})
    public void test() {
        System.out.println("Test");
    }

    @AfterGroups(groups = {"smoketests", "functionaltests"})
    public void after() {
        System.out.println("After Groups");
    }
}

Will display:
Before Groups
Before Groups
Test
After Groups

From: http://stackoverflow.com/q/38796087/4234729

@juherr I have tried fixing this issue. Please help review.

@krmahadevan Done

@juherr Thanks for helping with the review. I have fixed all the review comments.

@krmahadevan I'm getting the same issue with the 6.11 version.
<suite name="Test Suite">
    <test name="GroupTest">
          <groups>
              <run>
                <include name="sanity"/>
              </run>
          </groups>
          <classes>
              <class name="SampleTest"/>
          </classes>
    </test>
</suite>

Below is the test class:
public class SampleTest {
    @BeforeGroups(groups = {"sanity","regression"})
    void beforeGroup(){
        System.out.println("Before Group");
    }

    @AfterGroups(groups = {"sanity","regression"})
    void afterGroups(){
        System.out.println("After Groups");
    }

    @Test(groups = {"sanity"})
    void m1(){
        System.out.println("m1");
    }

    @Test(groups = {"sanity","regression"})
    void m3(){
        System.out.println("m3");
    }
    @Test(groups = {"sanity"})
    void m4(){
        System.out.println("m4");
    }
    @Test(groups = {"regression"})
    void m5(){
        System.out.println("m5");
    }
    @Test
    void m6(){
        System.out.println(Sample.class.getName());
    }
}

So when I am runnting the testng.xml, getting the following result:
Before Group
m1
m2
Before Group
m3
After Groups
m4
After Groups

Following is the result when I'm changing group name to regression in the suite.
Before Group
m3
After Groups
m5
After Groups

@BefroreGroups and @Aftergroups get executed for both the group entries.

@AdityaGaurav - The latest released version of TestNG is 6.14.3. Please retry using that. If you are still seeing the issue, please file a new issue with a standalone reproducible test


Support es2015+ in webpack config

babel-register might help before importing the user config.
https://twitter.com/rstacruz/status/872360854569832450

How would this work? It doesn't seem that just installing babel-register magically fixes it, nor does Phenomic pick up/enable babel support on webpack.config.babel.js

Oh ! we dont provide a babel config. Meanwhile you should provide yours.

Should works now, we provide babel-preset-env as a default!


Updating Docker URL to partnership URL

Updating Hub URL to unique URL for Docker/MSFT experience
cc @ghogen and @BigMorty

@nebuk89 Thanks for the update - can you explain this a little?  msftedge would seem to reference the Edge browser, but that doesn't make sense since users would be using any browser.

@ghogen This was overlooked in the pull request queue. Is the PR ready to merge? Thanks.

@ktoliver I have pinged the requestor over email along with a product PM in Visual Studio tools.

@ghogen Thanks; we'll add #hold-off for now.


i64 literal might not be allowing all values

I think 0x8000000000000000_i64 should evaluate to -9223372036854775808.  It seems like it does not check for sign and then fails because the max is positive number is<PHONE_NUMBER>854775807.
https://play.crystal-lang.org/#/r/7wmr

First sentence makes no sense to me already.

0x80000000_i32
Error:<PHONE_NUMBER> doesn't fit in an Int32

@wontruefree Number literals use regular math notation, not a binary data format. See https://forum.crystal-lang.org/t/negative-numbers-in-binary-and-hexa/1062/5?u=straight-shoota for the difference.

@straight-shoota that makes sense.  I was working on a c port and that is treated a little different.


feat: Improvements on securitykey.validate

useSessionKey=True is now default
pre_condition-parameter to auto-raise a PreconditionFailed() exception
Replaced and optimized calls in viur-core
100% backwards compatible to existing calls

I don't know why useSessionKey=True was always explicitly provided, as you can see below:

This can be seen as an improvement on #656

Let's talk about a decorator. I use this in ViUR2:
def skeyRequired(func=None, **decoratorKwArgs):
	"""Decorator that checks the skey before the method is called.

	Optional callable to pass keyword-arguments for the securitykey.validate-call.

	Example:
		>>> from server import exposed
		>>> @exposed
		>>> @skeyRequired
		>>> def yourMethod(self):
		>>>     return {"foo": "bar"}
	"""

	def outerWrapper(f):
		@functools.wraps(f)
		def wrapper(*args, **kwargs):
			if not securitykey.validate(kwargs.pop("skey", ""), **decoratorKwArgs):
				raise errors.PreconditionFailed("Missing or invalid skey")
			return f(*args, **kwargs)

		return wrapper

	if isinstance(func, (types.MethodType, types.FunctionType)):
		return outerWrapper(func)  # @skeyRequired
	else:
		return outerWrapper  # @skeyRequired() or @skeyRequired(**anyKwargs)

In today's viur-meeting, we decided to solve it by a decorator. I'm with you, but anyway would like to merge this PR as is and integrate the feature set into the code-base.

Closed as superseded by #764


Contentstack package depends on isomorphic-fetch, doesn't use it

In our test suite (running in node) it seems depending on contentstack creates a broken package dependency situation.
At https://github.com/contentstack/contentstack-javascript/blob/290cda73b1a9df0f697611cee28cc1992f2929b8/src/runtime/node/http.js#L2 the npm contentstack package imports node-fetch. However, it doesn't declare node-fetch among its dependencies.
The "contentstack" package correctly declares isomorphic-fetch among its dependencies. The isomorphic-fetch project allows packages to be defined which are neither node- or browser- specific, with the isomorphic-fetch package resolving to the appropriate packages at runtime.
However, because contentstack incorrectly imports node-fetch instead of isomorphic-fetch this forces projects to explicitly install node-fetch to prevent runtime errors when testing in node. This defeats the purpose of isomorphism, meaning that packages which are ONLY intended for client-side deployment end up needing a node dependency.

For reference this was detectable (and problematic) since node-fetch was resolved (again implicitly) to a version which was installed by another package in our monorepo. Unlike the node-fetch v2 dependency declared by isomorphic-fetch, the node-fetch v3 dependency which was then resolved (implicitly because it was undeclared by contentstack) turned out to be an ESM package, leading to the following error...
 RUN  v0.32.2 /Users/superman/workspace/scratch/contentstack-schema-iac/packages/stack-renderer

 ❯ test/transform.test.ts (0)

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Failed Suites 1 ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯

 FAIL  test/transform.test.ts [ test/transform.test.ts ]
Error: require() of ES Module /Users/superman/workspace/scratch/contentstack-schema-iac/node_modules/node-fetch/src/index.js from /Users/superman/workspace/scratch/contentstack-schema-iac/node_modules/contentstack/dist/node/contentstack.js not supported.
Instead change the require of index.js in /Users/superman/workspace/scratch/contentstack-schema-iac/node_modules/contentstack/dist/node/contentstack.js to a dynamic import() which is available in all CommonJS modules.
 ❯ 809 ../../node_modules/contentstack/dist/node/contentstack.js:1:37275
 ❯ r ../../node_modules/contentstack/dist/node/contentstack.js:1:37397
 ❯ 289 ../../node_modules/contentstack/dist/node/contentstack.js:1:36890
 ❯ r ../../node_modules/contentstack/dist/node/contentstack.js:1:37397
 ❯ 777 ../../node_modules/contentstack/dist/node/contentstack.js:1:4449
 ❯ r ../../node_modules/contentstack/dist/node/contentstack.js:1:37397
 ❯ 540 ../../node_modules/contentstack/dist/node/contentstack.js:1:11074
 ❯ r ../../node_modules/contentstack/dist/node/contentstack.js:1:37397
 ❯ 533 ../../node_modules/contentstack/dist/node/contentstack.js:1:29017
 ❯ r ../../node_modules/contentstack/dist/node/contentstack.js:1:37397
 ❯ 32 ../../node_modules/contentstack/dist/node/contentstack.js:1:2627
 ❯ r ../../node_modules/contentstack/dist/node/contentstack.js:1:37397
 ❯ ../../node_modules/contentstack/dist/node/contentstack.js:1:37423
 ❯ Object.<anonymous> ../../node_modules/contentstack/dist/node/contentstack.js:1:37446
 ❯ async /Users/superman/workspace/scratch/contentstack-schema-iac/packages/stack-iac/src/deliver.ts:3:31
 ❯ async /Users/superman/workspace/scratch/contentstack-schema-iac/packages/stack-iac/src/index.ts:3:31
 ❯ async /Users/superman/workspace/scratch/contentstack-schema-iac/packages/stack-renderer/src/transform.ts:2:31
 ❯ async /Users/superman/workspace/scratch/contentstack-schema-iac/packages/stack-renderer/test/transform.test.ts:2:31

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
Serialized Error: {
  "code": "ERR_REQUIRE_ESM",
}
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[1/1]⎯

 Test Files  1 failed (1)
      Tests  no tests
   Start at  14:36:06
   Duration  323ms (transform 92ms, setup 0ms, collect 0ms, tests 0ms, environment 0ms, prepare 77ms)

For reference a proven version of node-fetch that can be installed by a host project to make contentstack happy is "node-fetch": "^2.6.12"

Any updates on this issue?

Please make this package stable.

HI @cefn we will require some more time to check this and we will update you as soon as possible
cc: @netrajpatel


More painful than necessary to get started

Here was my experience this morning experimenting with butane.

Read the book, which pointed to version 6.
Read crates, which has 6.0 showing with an error and 6.1 preferred but 7 out.
Added version 7 to cargo and tried following the examples in the github repo.
Found that the example code failed with the published version 7 and tinkered around without success to get it succeeding.
Switched examples to the get started without async example.
Looked for version branches or links where I could conveniently get working, stable examples.
Guessed after failing to find any that while butane looks like exactly what I want it's not yet aimed at supporting external use.

I think all I was looking for was a published version for which I could easily find working examples with stable, published dependencies.

Thank you for the feedback. I'm fixing your first point in #285.
The remaining difficulty you had may stem from using the published 0.7 crate but looking at the examples from latest master. To see the code corresponding to the 0.7 release, you can check out to the 0.7 tag in git (https://github.com/Electron100/butane/releases/tag/0.7)
The current head of master has some significant revisions for the upcoming 0.8 release which makes some breaking changes in order to add optional async support.
That said, if you're interested in sharing more about the specific errors you encountered, I'm certainly happy to see what I can improve.


Ajay tripathy nosec

Minor logging improvement
Explanations of static analysis security audit comments.

Is this still something we want to review and merge?


slight cleanup in fips enabled tests

Cleanup a small clippy warning
Allow tests to pass in a non FIPS build. Right now the tests will fail with --no-default-features. Is this intentional?

@stevenctl: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
3be0bbab3a0533b3a25809f022679e15cf4151dc
link
true
/test test

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


Infer instance type arguments as constraint

In JavaScript, there are several runtime checks that can determine if an object is an instance of a specific class. However, due to the fact that TypeScript type parameters only exist at the type level, there are limited options for how to handle these when building the class instance reference type. At the moment, there are two different approaches used by the two different kind of checks: one defaults to any and the other re-uses the type argument from the enclosing class. The first could be stricter and the second is possibly too strict in some cases.
The proposed change is to modify the way TypeScript handles these checks such that the type arguments are inferred as the constraint from the corresponding type parameter. This would default to unknown if no constraint was specified as that is the "default constraint".
Fixes #17473 - Infer constrained generic parameters after instanceof check
For instanceof checks, TypeScript mirrors the underlying behavior of JavaScript and looks at the Class.prototype property. This property is computed dynamically in the checker and is then used to narrow the target of the instanceof expression. If the class in question has any type parameters, TypeScript returns a type reference where all of the type arguments are any. This is documented as being part of the "TypeScript 1.0 spec" so it is very long-standing behavior. The proposed change in behavior would constitute a breaking change as it would narrow some types from any to the constraint type or unknown.
class Box<T> { value: T }
Box.prototype; // changes from `Box<any>` to `Box<unknown>`
declare const box: any;
if (box instanceof Box) {
    box; // changes from `Box<any>` to `Box<unknown>`
}

class StringBox<T extends string> { value: T }
StringBox.prototype; // changes from `StringBox<any>` to `StringBox<string>`
declare const stringBox: any;
if (stringBox instanceof StringBox) {
    stringBox; // changes from `StringBox<any>` to `StringBox<string>`
}

Fixes #46668 - Private field check narrows generic class too far
For #brand in obj checks, TypeScript can infer that if an object has a private identifier then it must be an instance of the enclosing class. If the class in question has any type parameters, TypeScript sets all of the type arguments of the inferred type to match the type arguments of this (see thread for potential issues). This behavior was only introduced recently via ergonomic brand checks for private fields. The proposed change in behavior is technically a breaking change, but less so than the instanceof check as in some cases it would loosen type arguments to match their constraint.
class PrivateBox<T> {
    #brand;
    value: T;
    copyValueFrom(other: object) {
        if (#brand in other) {
            other; // changes from `PrivateBox<T>` to `PrivateBox<unknown>`
        }
    }
}

class PrivateStringBox<T extends string> {
    #brand;
    value: T;
    copyValueFrom(other: object) {
        if (#brand in other) {
            other; // changes from `PrivateStringBox<T>` to `PrivateStringBox<string>`
        }
    }
}

Breaking Changes
In order to deal with these breaking changes, this change should be gated behind a new strict compiler option inferInstanceTypeArgumentsAsConstraint. This would be similar to the useUnknownInCatchVariables option that narrowed catch variables from any to unknown (#41013).
For individual instances of this issue, it should be possible to re-widen the narrowed type with a cast:
if (obj instanceof Box) {
    someCodeThatWorksWithAny(obj.value); // now fails with Box<unknown>
    someCodeThatWorksWithAny((obj as Box<any>).value);
}

Addendum on Name: I am aware that this new compiler option name is very verbose, and could potentially be improved. I considered something that referenced "runtime instance checks" but it's hard to include that verbiage in the name and not make it potentially confusing. (ex: Would someone see strictRuntimeInstanceChecks and then assume that it somehow changes runtime behavior?) Also considered useConstraintAsDefaultInstanceTypeArgument or strictDefaultInstanceTypeArguments.

Here's a playground link that contains both code samples from the description, for convenience: 4.8.0-pr-49863-9

I've been thinking more about this issue, specifically in the context of type parameter variance when a custom extends is specified. (The following comments do not effect the base case <T> where any switches to unknown)
For <T extends U>, if T was contravariant or invariant, than it would be unsafe to choose U as the type argument.  Therefore, it is really only safe to choose U as the type argument if T is covariant. How should we account for this? If we use the variance annotations we can make some decisions:

<out T extends U> type argument should default to U
<in T extends U> type argument should default to unknown
<in out T extends U> type argument should default to unknown

We also then need to decide what happens in the absence of those annotations <T extends U>. The safest option would probably be to widen to unknown, but keeping it as-is (U) may also be a reasonable strategy. It would let more people use this feature immediately and the variance annotations could act as a way to formally opt-in (or opt-out, depending on perspective).
What does everyone think?

<in T extends U> type argument should default to unknown

This should be never, right? If you have
declare class HasAction<T> {
  readonly fn: (x: T) => void;
}

then you want the most restrictive instantiation of HasAction<T> to instantiate T with its lower-bound.

This should be never, right?

Ah, never instead of unknown makes sense. So that would make it:

<out T extends U> defaults to U
<in T extends U> defaults to never
<in out T extends U> defaults to never
<T extends U> defaults to never (or maybe U for usability?)

And then for <T> it would be the same for the above except U = unknown.

In this latest commit, I've updated the behavior such that the presence of an in variance annotation on the type parameter will force the inferred type argument to be never. This does not effect the default case (no variance annotations) which is unchanged, for now.

<T extends U> defaults to U
<out T extends U> defaults to U
<in T extends U> defaults to never
<in out T extends U> defaults to never

Rebased, but the license/cla check is incomplete where it wasn't previously.

@microsoft-github-policy-service agree [company="Bloomberg"]

@microsoft-github-policy-service agree company="Bloomberg"

Seems like the CLA bot is broken 😢

I made a PR to test out what happens when this is always enabled by default: https://github.com/microsoft/TypeScript/pull/53300

@typescript-bot pack this
@DanielRosenwasser wanna re-merge/do your PR and rerun the community tests with the flag locked to on?

What about instantiation expressions? I think (StringBox<'foo'>).prototype should be StringBox<'foo'> instead of StringBox<string>.

@typescript-bot pack this
@typescript-bot test this
@typescript-bot test top100
@typescript-bot user test this
@typescript-bot user test tsserver
@typescript-bot test tsserver top100
@typescript-bot run dt
@typescript-bot perf test this

Following up on this. I saw some of the failures from the top-repos tests, but not sure how those should be interpreted. Most of the errors feel like correctly replacing a loose any with a strict unknown, but on the other hand it does feel quite strict.
If there's any conclusion that can be drawn from these results, or any further tests to be run, let me know if I can push this PR along in any way. Thanks!

Tested this out some more, and there might be a potential issue when a type parameter constraint refers to some other local type parameters (playground link)
declare class Box<T> { value: T }
declare class BoxBox<T, B extends Box<T>> { box: B }
declare const bb: any;
if (boxbox instanceof BoxBox) {
    bb; // BoxBox<unknown, Box<T>>
}

The new type of the class after the instanceof check is BoxBox<unknown, Box<T>>, but ideally it should be BoxBox<unknown, Box<unknown>>, right? Accessing bb.box.value returns a value with type T, but it doesn't seem like that should be allowed outside the context of the BoxBox definition.
I'll treat this as a bug for now and attempt to implement a solution.

@molisani @DanielRosenwasser @jakebailey It looks like this PR needs some fixes before it's ready. Do you want to keep working on it? Otherwise I like to close stale PRs after a few months.

@molisani, have you had a chance to come back to this PR?

I haven't, but I'm still optimistic that I'll be able to tackle this at some point. For now I'll close it out and if I can pick it up again I'll open a new PR with everything refreshed.

For anyone subscribed to this PR, I have recreated it with my latest comments resolved here: https://github.com/microsoft/TypeScript/pull/58028


Run eshop in k8s ,get  [xxx pubsub already exists.  error message .]

[06:45:38 ERR] A default subscription to topic UserCheckoutAcceptedIntegrationEvent on pubsub pubsub already exists.
[06:45:38 ERR] A default subscription to topic OrderStockConfirmedIntegrationEvent on pubsub pubsub already exists.
[06:45:38 ERR] A default subscription to topic OrderStockRejectedIntegrationEvent on pubsub pubsub already exists.
[06:45:38 ERR] A default subscription to topic OrderPaymentSucceededIntegrationEvent on pubsub pubsub already exists.
[06:45:38 ERR] A default subscription to topic OrderPaymentFailedIntegrationEvent on pubsub pubsub already exists.
[06:45:38 ERR] A default subscription to topic OrderStatusChangedToSubmittedIntegrationEvent on pubsub pubsub already exists.
[06:45:38 ERR] A default subscription to topic OrderStatusChangedToAwaitingStockValidationIntegrationEvent on pubsub pubsub already exists.
[06:45:38 ERR] A default subscription to topic OrderStatusChangedToValidatedIntegrationEvent on pubsub pubsub already exists.
[06:45:38 ERR] A default subscription to topic OrderStatusChangedToPaidIntegrationEvent on pubsub pubsub already exists.
[06:45:38 ERR] A default subscription to topic OrderStatusChangedToShippedIntegrationEvent on pubsub pubsub already exists.

Is this on Dapr 1.5 or higher?
There was an issue in the .NET SDK (see https://github.com/dapr/dotnet-sdk/pull/761), but that should've been fixed in 1.5.


Matcher should always call describeMismatch of a nested macther

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?

Create a Matcher with has a describeMismatch method
Create a falling unittest with the Matcher in the isNot-Matcher: isNot(myMatcher)

What is the expected output? What do you see instead?
Expected:

Expected: not <my description>
  Actual: <value>
   Which: <my mismatch description>
Actual:

Expected: not <my description>
  Actual: <value>
What version of the product are you using? On what operating system?
Dart VM version: <IP_ADDRESS>_r28355 (Tue Oct 08 06:29:27 2013) on "windows_ia32"
Please provide any additional information below.
All matcher that contain other matcher should add the nested description. At least isNot is missing that.

This issue has been moved to dart-lang/matcher#16.


Blank shields are 20x20

This looks weird, there's a lot of empty space:

Reason is the road has two empty routes on it, which render as 20x20 transparent:

And because US:NY:STE ranks lower in the tile generation.
Would be preferable to not generate these entirely blank shields:

(Note that this logs a warning, once, when we silently refuse to load an image.)
Or generate a 1x1 transparent, which only adds a little bit of unnecessary space:

I think it's better to not generate the shields, even if it logs a warning. The spacing with the 1x1 blank shield just looks like a rendering error, and doesn't give much in the way of mapper feedback.


Uncaught (in promise) TypeError: Cannot read property 'addEventListener' of null at new Tracker (tracker.ts:91) at HTMLElement.updated (app-datepicker.ts:671)

Description
Have a problem with app-datepicker. Calendar doesn't show up. Instead of it I have some class in calendar window as text. Part of this class:
...
{ if (!(part instanceof NodePart)) { throw new Error('cache can only be used in text bindings'); } let templateCache = templateCaches.get(part); if (templateCache ...
In console I have errors:
Uncaught (in promise) TypeError: Cannot read property 'addEventListener' of null
at new Tracker (tracker.ts:91)
at HTMLElement.updated (app-datepicker.ts:671)
at HTMLElement.performUpdate (updating-element.ts:215)
at HTMLElement._enqueueUpdate (updating-element.ts:717)
Expected outcome
Calendar should show up
Actual outcome
Calendar doesn't show up. Instead of it I have some class in calendar window as text. Part of this class:
...
{ if (!(part instanceof NodePart)) { throw new Error('cache can only be used in text bindings'); } let templateCache = templateCaches.get(part); if (templateCache ...
In console I have errors:
Uncaught (in promise) TypeError: Cannot read property 'addEventListener' of null
at new Tracker (tracker.ts:91)
at HTMLElement.updated (app-datepicker.ts:671)
at HTMLElement.performUpdate (updating-element.ts:215)
at HTMLElement._enqueueUpdate (updating-element.ts:717)
Live Demo
Steps to reproduce

Browsers Affected

[x ] Chrome/ Opera
didn't checked other browsers

Do you have a reduced test case for this problem? What's the version of lit-element you are using?

"lit-element": "^2.2.0",
"lit-html": "^1.1.0"

To be honest I don't know how to reproduce test case. It works for some people but doesn't work for other. I tried to use previous version of datepicker and it works as it should. But v4 doesn't work for me.

Thank you for the response. I'll keep this opened for a while and see if that happens again before closing this.

I was getting this problem too for a minute. I was using the newest chrome, lit, and app-datepicker. Once I restarted my node server this error went away and the datepicker started functioning as normal

This should have been resolved by this e3cc4494917cfc2f0e30d0ee5792b0b2832638ed commit. Please give it a try and see if that resolves on your side.

Closing this


Add issues template for documentation issues

I would like to add a issues template for documentation issues in this repository. I believe that having a standardised template will help streamline the contribution process, ensuring that all necessary information is included and making it easier for maintainers to review issues.
Could you please assign this issue to me under GSSOC'24.

I am a GSSoC '24 Contributor.
kindly ,@Ijaiswalshivam assign me this issue to start work on it .


PHP 8.2 build seems out of date or something?

Describe the bug

Version

[ ] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems

ubuntu-latest
PHP versions

8.2
To Reproduce

      - name: Set up php ${{ matrix.php-versions }}
        uses: shivammathur/setup-php@v2
        with:
          php-version: ${{ matrix.php-versions }}
          tools: phpunit
          extensions: mbstring, iconv, fileinfo, intl, sqlite, pdo_sqlite, gd
          coverage: none

      - name: PHPUnit
        working-directory: tests
        run: phpunit --configuration phpunit-autotest.xml

Expected behavior

I should get the same errors as when running phpunit locally with php 8.2 built from git, especially a lot of errors and warnings about Using ${var} in strings is deprecated, use {$var} instead at and Creation of dynamic property Class::$property is deprecated at.
Screenshots/Logs

I get almost no more errors than with php 8.1.
Additional context

How is the PHP 8.2 nightly built? Is it from master branch?
Are you willing to submit a PR?

@come-nc This looks like basically a duplicate of https://github.com/shivammathur/setup-php/issues/469 where I asked the same regarding PHP 8.1.
The short of it is that the default php.ini is the production ini file and that if you want to see deprecations, you need to either set the ini-file option to develop or set (a variation of) the following ini-values: zend.assertions=1, error_reporting=-1, display_errors=On.
Hope this helps ;-)

Oh and before I forget: also make sure you have convertDeprecationsToExceptions="true" set in your phpunit.xml.dist file as the default value for that attribute was changed to false in PHPUnit 9.5.10, which means that builds won't fail on deprecations anymore.


Help with CW

[ X ] I've verified and I assure that I'm running youtube-dl 2016.04.06

Before submitting an issue make sure you have:

[ X ] At least skimmed through README and most notably FAQ and BUGS sections
[ X ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ X ] Bug report (encountered problems with youtube-dl)
[ X ] Question

[user@1 :: ~/downloaders]$ youtube-dl -v http://www.cwtv.com/shows/jane-the-virgin/chapter-thirty-eight/?play=ce23e0a2-89d5-4dc3-810d-a4076beade5f
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.cwtv.com/shows/jane-the-virgin/chapter-thirty-eight/?play=ce23e0a2-89d5-4dc3-810d-a4076beade5f']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.04.06
[debug] Python version 2.7.6 - Darwin-14.1.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.0.1, ffprobe 3.0.1, rtmpdump 2.4
[debug] Proxy map: {}
[CWTV] ce23e0a2-89d5-4dc3-810d-a4076beade5f: Downloading JSON metadata
[CWTV] ce23e0a2-89d5-4dc3-810d-a4076beade5f: Downloading m3u8 information
[debug] Invoking downloader on u'http://hlsioscwtv.warnerbros.com/hls/2016/03/26/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps.m3u8'
[download] Destination: Chapter Thirty-Eight-ce23e0a2-89d5-4dc3-810d-a4076beade5f.mp4
[debug] ffmpeg command line: ffmpeg -y -headers 'Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7
Accept-Language: en-us,en;q=0.5
Accept-Encoding: gzip, deflate
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0) Gecko/20150101 Firefox/44.0 (Chrome)
' -i http://hlsioscwtv.warnerbros.com/hls/2016/03/26/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps.m3u8 -c copy -f mp4 -bsf:a aac_adtstoasc 'file:Chapter Thirty-Eight-ce23e0a2-89d5-4dc3-810d-a4076beade5f.mp4.part'
ffmpeg version 3.0.1 Copyright (c) 2000-2016 the FFmpeg developers
  built with Apple LLVM version 7.0.2 (clang-700.1.81)
  configuration: --prefix=/usr/local/Cellar/ffmpeg/3.0.1 --enable-shared --enable-pthreads --enable-gpl --enable-version3 --enable-hardcoded-tables --enable-avresample --cc=clang --host-cflags= --host-ldflags= --enable-opencl --enable-libx264 --enable-libmp3lame --enable-libxvid --enable-vda
  libavutil      55. 17.103 / 55. 17.103
  libavcodec     57. 24.102 / 57. 24.102
  libavformat    57. 25.100 / 57. 25.100
  libavdevice    57.  0.101 / 57.  0.101
  libavfilter     6. 31.100 /  6. 31.100
  libavresample   3.  0.  0 /  3.  0.  0
  libswscale      4.  0.100 /  4.  0.100
  libswresample   2.  0.101 /  2.  0.101
  libpostproc    54.  0.100 / 54.  0.100
Input #0, hls,applehttp, from 'http://hlsioscwtv.warnerbros.com/hls/2016/03/26/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps.m3u8':
  Duration: 00:42:38.70, start: 10.000000, bitrate: 0 kb/s
  Program 0
    Metadata:
      variant_bitrate : 0
    Stream #0:0: Video: h264 (Constrained Baseline) ([27][0][0][0] / 0x001B), yuv420p, 1280x720 [SAR 1:1 DAR 16:9], 23.98 fps, 23.98 tbr, 90k tbn, 47.95 tbc
    Stream #0:1: Audio: aac (LC) ([15][0][0][0] / 0x000F), 44100 Hz, stereo, fltp, 5 kb/s
Output #0, mp4, to 'file:Chapter Thirty-Eight-ce23e0a2-89d5-4dc3-810d-a4076beade5f.mp4.part':
  Metadata:
    encoder         : Lavf57.25.100
    Stream #0:0: Video: h264 ([33][0][0][0] / 0x0021), yuv420p, 1280x720 [SAR 1:1 DAR 16:9], q=2-31, 23.98 fps, 23.98 tbr, 90k tbn, 90k tbc
    Stream #0:1: Audio: aac (LC) ([64][0][0][0] / 0x0040), 44100 Hz, stereo, 5 kb/s
Stream mapping:
  Stream #0:0 -> #0:0 (copy)
  Stream #0:1 -> #0:1 (copy)
Press [q] to stop, [?] for help
[mp4 @ 0x7fd2540b0e00] Non-monotonous DTS in output stream 0:1; previous: 20480, current: 0; changing to 20481. This may result in incorrect timestamps in the output file.
frame=  362 fps=361 q=-1.0 size=    3297kB time=00:00:15.05 bitrate=1793.9kbits/s speed=
**_PERSONAL NOTE: At this point it pauses without any output or updates for a few minutes and then refreshes with the time= part changed by about 30 seconds and then the process continues like this.._**
[http @ 0x7fd253d3cc80] Stream ends prematurely at 47691, should be 2493072bits/s speed=0.164x
    Last message repeated 1 times
[mpegts @ 0x7fd254801000] PES packet size mismatch
[crypto @ 0x7fd253c341c0] Unable to open resource: http://hlsioscwtv.warnerbros.com/hls/2016/03/26/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps-4.ts
[hls,applehttp @ 0x7fd25400da00] Failed to open segment of playlist 0
frame=  904 fps=6.0 q=-1.0 size=    8811kB time=00:00:57.43 bitrate=1256.8kbits/s speed=0.382x

**^C^C**[crypto @ 0x7fd253c35740] Unable to open resource: http://hlsioscwtv.warnerbros.com/hls/2016/03/26/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps-6.ts
av_interleaved_write_frame(): Immediate exit requested
Error writing trailer of file:Chapter Thirty-Eight-ce23e0a2-89d5-4dc3-810d-a4076beade5f.mp4.part: Immediate exit requestedframe=  959 fps=1.1 q=-1.0 Lsize=    9498kB time=00:00:59.83 bitrate=1300.2kbits/s speed=0.0656x
video:8888kB audio:615kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: unknown

ERROR: Interrupted by user
Exiting normally, received signal 2.

And on my other computer, same command, same versions, it runs mostly okay - a few errors but appears to have downloaded in full:
[user@2 :: /tmp]$ youtube-dl -v http://www.cwtv.com/shows/jane-the-virgin/chapter-thirty-eight/?play=ce23e0a2-89d5-4dc3-810d-a4076beade5f
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.cwtv.com/shows/jane-the-virgin/chapter-thirty-eight/?play=ce23e0a2-89d5-4dc3-810d-a4076beade5f']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.04.06
[debug] Python version 2.7.10 - Darwin-15.3.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.0.1, ffprobe 3.0.1, rtmpdump 2.4
[debug] Proxy map: {}
[CWTV] ce23e0a2-89d5-4dc3-810d-a4076beade5f: Downloading JSON metadata
[CWTV] ce23e0a2-89d5-4dc3-810d-a4076beade5f: Downloading m3u8 information
[debug] Invoking downloader on u'http://hlsioscwtv.warnerbros.com/hls/2016/03/26/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps.m3u8'
[download] Destination: Chapter Thirty-Eight-ce23e0a2-89d5-4dc3-810d-a4076beade5f.mp4
[debug] ffmpeg command line: ffmpeg -y -headers 'Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7
Accept-Language: en-us,en;q=0.5
Accept-Encoding: gzip, deflate
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0) Gecko/20150101 Firefox/44.0 (Chrome)
' -i http://hlsioscwtv.warnerbros.com/hls/2016/03/26/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps.m3u8 -c copy -f mp4 -bsf:a aac_adtstoasc 'file:Chapter Thirty-Eight-ce23e0a2-89d5-4dc3-810d-a4076beade5f.mp4.part'
ffmpeg version 3.0.1 Copyright (c) 2000-2016 the FFmpeg developers
  built with Apple LLVM version 7.3.0 (clang-703.0.29)
  configuration: --prefix=/usr/local/Cellar/ffmpeg/3.0.1 --enable-shared --enable-pthreads --enable-gpl --enable-version3 --enable-hardcoded-tables --enable-avresample --cc=clang --host-cflags= --host-ldflags= --enable-opencl --enable-libx264 --enable-libmp3lame --enable-libxvid --enable-vda
  libavutil      55. 17.103 / 55. 17.103
  libavcodec     57. 24.102 / 57. 24.102
  libavformat    57. 25.100 / 57. 25.100
  libavdevice    57.  0.101 / 57.  0.101
  libavfilter     6. 31.100 /  6. 31.100
  libavresample   3.  0.  0 /  3.  0.  0
  libswscale      4.  0.100 /  4.  0.100
  libswresample   2.  0.101 /  2.  0.101
  libpostproc    54.  0.100 / 54.  0.100
Input #0, hls,applehttp, from 'http://hlsioscwtv.warnerbros.com/hls/2016/03/26/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps/JaneTheVirgin-216-ChapterThirtyEight-CW-Stereo_a3a2bb56f_2100kbps.m3u8':
  Duration: 00:42:38.70, start: 10.000000, bitrate: 0 kb/s
  Program 0
    Metadata:
      variant_bitrate : 0
    Stream #0:0: Video: h264 (Constrained Baseline) ([27][0][0][0] / 0x001B), yuv420p, 1280x720 [SAR 1:1 DAR 16:9], 23.98 fps, 23.98 tbr, 90k tbn, 47.95 tbc
    Stream #0:1: Audio: aac (LC) ([15][0][0][0] / 0x000F), 44100 Hz, stereo, fltp, 5 kb/s
Output #0, mp4, to 'file:Chapter Thirty-Eight-ce23e0a2-89d5-4dc3-810d-a4076beade5f.mp4.part':
  Metadata:
    encoder         : Lavf57.25.100
    Stream #0:0: Video: h264 ([33][0][0][0] / 0x0021), yuv420p, 1280x720 [SAR 1:1 DAR 16:9], q=2-31, 23.98 fps, 23.98 tbr, 90k tbn, 90k tbc
    Stream #0:1: Audio: aac (LC) ([64][0][0][0] / 0x0040), 44100 Hz, stereo, 5 kb/s
Stream mapping:
  Stream #0:0 -> #0:0 (copy)
  Stream #0:1 -> #0:1 (copy)
Press [q] to stop, [?] for help
[mp4 @ 0x7fc30a005c00] Non-monotonous DTS in output stream 0:1; previous: 20480, current: 0; changing to 20481. This may result in incorrect timestamps in the output file.
[mp4 @ 0x7fc30a005c00] Non-monotonous DTS in output stream 0:0; previous: 3731228, current: 3731228; changing to 3731229. This may result in incorrect timestamps in the output file.
[mp4 @ 0x7fc30a005c00] Non-monotonous DTS in output stream 0:0; previous: 229500521, current: 229500521; changing to 229500522. This may result in incorrect timestamps in the output file.
[mp4 @ 0x7fc30a005c00] Non-monotonous DTS in output stream 0:0; previous: 229571843, current: 229571843; changing to 229571844. This may result in incorrect timestamps in the output file.
frame=61346 fps=307 q=-1.0 Lsize=  630511kB time=00:42:38.69 bitrate=2018.7kbits/s speed=12.8x
video:589100kB audio:39206kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: 0.350914%
[ffmpeg] Downloaded 645643208 bytes
[download] 100% of 615.73MiB

Description of your issue, suggested solution and other information
I have been using youtube-dl to get videos from CWTV (and others) and it's been working very well for the most part. But lately, and just with one computer, CW has been failing miserably. As you can see above, the same command works on one computer but not the other. So it's probably not the fault of youtube-dl by itself, but I'm really out of ideas!
Any thoughts of things I can try?  Thanks much.

It's not the same command. Post the command with the same URL.

Thanks for noticing that! For the record, neither URL had been working on computer 1 for the past few days. I've updated the original post.
Also I highly doubt this is geo restricted, so I'd appreciate removing that label. My two computers are on the same home network (in USA), literally 15 feet away from each other. That's some pretty crazy geo-restriction!

It's georestricted.

Sorry, this content is not available in your location

As of the first log you've interrupted ffmpeg by yourself:

ERROR: Interrupted by user
Exiting normally, received signal 2.

Post the log of failure without manual interruption.

Well I have no idea but I repeat the command (copy/pasted exact same) and now it works. I didn't reboot or anything. This has been plaguing me for days. Just needed to post about it. If it happens again I'll report back.
Re: geo-restricted label, my sincere apologies I thought the label was implying that my problem was due to geo-restriction.
Thank you again for your time!!


Why are span's display: inline-block?

The comment reads:
/**
 * All styled spans inside terminal lines should be inline-blocks,
 * in orde to achieve better height adjustment.
 */
.terminal .xterm-rows span {
	display: inline-block;
}

As far as I can see it only messes with how ::selection works:

This is with span's display:inline:

This is with the ::selection changes I've made for vscode which are too cumbersome to do without a JS API.

The main thing I'm curious about is why inline-block's (span) would be useful in between just floating uncoloured text nodes.

Another example with a different font-size and line-height:

vs inline:

Note these examples are using electron (chromium).

:+1:


Problem: Caused by: java.lang.IllegalStateException: You need to use a Theme.AppCompat theme (or descendant) with this activity.

Fix, by replacing:
<EMAIL_ADDRESS>on lines 36 and 61 in AndroidManifest.xml with:
<EMAIL_ADDRESS>or update styles.xml

I fixed this issue with commit https://github.com/acchampion/TicTacToeNew/commit/90ff834263ceded49e5d5820a3a5dc5d805cf1fc . Let me know if you have any other issues.


gs1.1: chore: misc fixes

Miscellaneous fixes found in the course of writing tests

Codecov Report

Merging #115 into gsv1.1 will increase coverage by 9.84%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           gsv1.1     #115      +/-   ##
==========================================
+ Coverage   81.81%   91.66%   +9.84%
==========================================
  Files           1        2       +1
  Lines          11       24      +13
==========================================
+ Hits            9       22      +13
  Misses          2        2

Impacted Files
Coverage Δ

.aegir.js
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Capitalization not standardized for command line help text.

/kind bug

Description
Command line help text does not look to be standardized in regards to capitalization.

Steps to reproduce the issue:

$ sudo podman --help

Describe the results you received:
$ sudo podman --help
NAME:
   podman - manage pods and images

USAGE:
   podman [global options] command [command options] [arguments...]

VERSION:
   0.7.3

COMMANDS:
     attach           Attach to a running container
     commit           Create new image based on the changed container
     container        container COMMAND
     build            build an image using instructions from Dockerfiles
     create           create but do not start a container
     diff             Inspect changes on container's file systems
     exec             Run a process in a running container
     export           Export container's filesystem contents as a tar archive
     history          Show history of a specified image
     image            image COMMAND
     images           list images in local storage
     import           Import a tarball to create a filesystem image
     info             display system information
     inspect          Displays the configuration of a container or image
     kill             Kill one or more running containers with a specific signal
     load             load an image from docker archive
     login            login to a container registry
     logout           logout of a container registry
     logs             Fetch the logs of a container
     mount            Mount a working container's root filesystem
     pause            Pauses all the processes in one or more containers
     ps               List containers
     pod              Manage pods
     port             List port mappings or a specific mapping for the container
     pull             pull an image from a registry
     push             push an image to a specified destination
     restart          Restart one or more containers
     rm               podman rm will remove one or more containers from the host.  The container name or ID can be used.
                                  This does not remove images.  Running containers will not be removed without the -f option.
     rmi              removes one or more images from local storage
     run              run a command in a new container
     save             Save image to an archive
     search           search registry for image
     start            Start one or more containers
     stats            Display percentage of CPU, memory, network I/O, block I/O and PIDs for one or more containers
     stop             Stop one or more containers
     tag              Add an additional name to a local image
     top              Display the running processes of a container
     umount, unmount  Unmounts working container's root filesystem
     unpause          Unpause the processes in one or more containers
     version          Display the PODMAN Version Information
     wait             Block on one or more containers
     varlink          Run varlink interface
     help, h          Shows a list of commands or help for one command

Describe the results you expected:
All the help text begin with lower case?
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
0.7.3

I think this also extends beyond just the helptext for just podman - I tried podman attach --help and the --latest flag had different capitalization from the others.

@mheon I think you are wrong
./bin/podman stop --help | grep latest
   --latest, -l                             act on the latest container podman is aware of
 ./bin/podman attach --help | grep latest
   --latest, -l         act on the latest container podman is aware of

@preethit Just about every tool I looked at Capitalized these descriptions, So I capitalized them.

@rhatdan Thank you for looking at other tools before making the change. I did not know what the standard is in other tools. I just looked at rpm and saw that it was lowercase and assumed that is the norm.

I looked at git, docker, systemctl.
I guess the bottom line is they should be consistent.


fix: watch without namespace path if watching all namespaces

resolves #634

ugh I'll update the tests


[DOCS] Complete all community guidelines

Description
Let's add missing docs.
https://github.com/ykdojo/defaang/community

Reopening this because #78 is yet to be merged.

Somebody should probably drag this back under "In progress"...?

done! thanks @veganaize


Error after upgrading to v0.2.0

First: Big thanks for an awesome plugin. My client loves it!
Today I upgraded to Version 0.2.0 and it produces the following error
Fatal error: Class 'Opengraph\Opengraph' not found in /var/www/craft/app/framework/YiiBase.php on line 213

Thanks

Hey @dommmel, thanks!
I can't seem to reproduce this error. It seems like the Opengraph PHP library isn't being found for some reason. Can you check if the directory craft/plugins/embeddedassets/vendor/euskadi31 exists? Don't know why it wouldn't be there, but might as well rule out the obvious.

Thanks for the swift reply.
Yes, the directory craft/plugins/embeddedassets/vendor/euskadi31 exists.
Yes, I am using the previous version and it works just fine. Just wanted to let you know.

I get the same error as of v0.2.0, so it isn't a one off.

@dommmel @andyharris sorry for the inconvenience! Currently at work but will look into it tonight. Little baffled as to what it could be though as I can't seem to reproduce it. Maybe I'll bypass composers autoload and require the libraries directly. Thanks for your patience fellas

Finally got around to fixing this!

Thanks!

Great, thanks.


Compiling Order

It doesn't work with Twitter Bootstrap.
Bootstrap requires some order to compile its files. It would be interesting to be able to define the compiling order of every less file.
(y)

This program is abandoned, I can recommend prepros.io @YOoSlim.


Win32: Use a dedicated background thread for each remote UI thread.

This works independent of COM marshaling, which is not implemented in Mono.

I think most of those were mismarked, but GetMinimumIncrementAsync probably should be CommandThreadPriority.User, since that's used in adjust_scrollbars.


ResponseStream ending before completion of sending data

gRPC Version 1.7.1
OS: Windows 10
Using Visual Studio
I am trying to chunk and send a large byte array across gRPC using a server-side streaming call. I am using Nunit to test the implementation and cannot get my tests to pass. I am reading in the data from one stream and then writing it to the gRPC response stream.
This is a simplified snippet of how I am doing my gRPC calls:
SERVER SIDE
var imageStream = storage.GetImageStream(imageId);
var sendByte = new byte[1024 * 16];
int bytesRead;

while ((bytesRead = await imageStream.ReadAsync(sendByte, 0, sendByte.Length)) > 0)
{
	var message = new ImageResponse
	{
		ImageExtension = fileExtension,
		Image = ByteString.CopyFrom(sendByte, 0, bytesRead)
	};
	await responseStream.WriteAsync(message);
}

CLIENT SIDE
using (var call = _textExtractorClient.GetImage(imageRequest, callOptions))
{
	var responseStream = call.ResponseStream;
	while (await responseStream.MoveNext())
	{
		var imageExt = responseStream.Current.ImageExtension;

		Buffer.BlockCopy(responseStream.Current.Image.ToByteArray(), 0, imageData, offset, call.ResponseStream.Current.Image.ToByteArray().Length);
		offset += responseStream.Current.Image.ToByteArray().Length;
	}
}

It is difficult to debug due to the streams timing out and sending the test into teardown.
I expect to have an byte array of about 0.5Mb size or higher, but instead it only sends about half of the byte array or less. When I did some logging I found the exception that is occurring in the implementation is this:
EXCEPTION HAPPENED == Response stream has already been completed.
I notice that the problem becomes worse as the total size of the byte array that I am trying to pass increases. The outcomes of my tests are also inconsistent. Different amounts of data are being passed each time that I run it.
I am looking for some consistency from the code and would love any help that could be offered. I couldn't find anything online about this exception.

I experiencing some troubles with response stream: 'Response stream is already completed',
when it is just starting.
The scenario is hen the await responseStream.WriteAsync() is inside a Task.Run(),
If not all work fine.
Please help

Sorry for this not getting attention. Since this looks like more of a usage question, if it's still relevant can you please post this to grpc-io or stack overflow?


Proposal: diverging colormaps for B0 chapter

Following up on #12, I'm suggesting to switch the B0 maps from grayscale to a diverging colormap (I have a preference of blue-white-red since it's intuitive), with white set to 0, blue negatives, red positives. I'm open to a discussion, how are they being displayed in Shimming-Toolbox?
Here's what they look like with my changes,
Figure 5.1

Figure 5.3

Figure 5.9

It all looks great to me, thanks!
Before merging, wanted to give you heads up that the deploy workflow on main sets mb/dev as trigger ref
https://github.com/qMRLab/mooc/blob/f40f0c15caf44d803c26f6b52f82d574779cb03e/.github/workflows/deploy.yml#L5

Yep thanks! I was about to change it to main again last night but fell asleep haha - will do asap

Thanks all! I was about to merge when I noticed the colors are going the wrong way (blue should be negative and red positive) - I think plotly has two different definitions for this diverging colormap - I'll look up the other one and fix it prior to merge!

the colors are going the wrong way

Good catch!


cherry-picking 1.1.x commit for adding lombok.config into 1.1.10

Description
Cherry picking commit from 1.1.x into 1.1.10
https://github.com/openanthem/nimbus-core/pull/282
Overview of Changes

N/A
Type of Change

N/A
Test Details

N/A
Additional Notes

N/A

You did it @swethavemuri-anthem!
Thank you for signing the Anthem Opensource Software CLA.


ActiveType.custom not working for multiple word in a text

let CustomTerm = ActiveType.custom(pattern: "\sterms & conditions\b")// not working
let customPrivacy =  ActiveType.custom(pattern: "\sprivacy policy\b")// working
messageLabel.text = NSLocalizedString("Please review our terms & conditions and privacy policy before setting up your wallet", comment: "")

I need both as a active label TERMS & CONDITION  and PRIVACY POLICY


Says incompatible with another mod

Says that with my other mods and without them. Even tried removing all mods except this one from the directory and get same result. Only thing that seems to work is toggling the traffic lights. Any ideas? Tried downloading the source and recompiling but my virtual studio always has errors. And visual studio can't load the tmf playground? Required for building?

Can you give me a link to an older version to try. Unsure why this mod won't work. Running snowfall.

I also don't know why it is not working for you. Do you have got a player_log.txt and TMPE.log on your hand?

Its is working now. Don't know what was going on. Keep up the good work.
On Aug 13, 2016 11:21 PM, "Victor-Philipp Negoescu" <
<EMAIL_ADDRESS>wrote:

I also don't know why it is not working for you. Do you have got a
player_log.txt and TMPE.log on your hand?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/VictorPhilipp/Cities-Skylines-Traffic-Manager-President-Edition/issues/11#issuecomment-239653720,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATzPA4DSLlywZ-emblbCNec67y3i-991ks5qfombgaJpZM4JZStn
.


Attempting to export from an Importer that has failed Entries throws error

Error occurs here: https://github.com/samvera-labs/bulkrax/blob/master/app/parsers/bulkrax/csv_parser.rb#L101
Because the Entry failed to create a Work, attempting to look up a Work's id through the Entry#identifier fails with a undefined method on nil error.
ActiveFedora::SolrService.query(query, fl: 'id', rows: 1)
=> []
ActiveFedora::SolrService.query(query, fl: 'id', rows: 1).first
=> nil
ActiveFedora::SolrService.query(query, fl: 'id', rows: 1).first['id']
=> undefined method `[]' for nil:NilClass

Proposed fix: Before running the exporter, filter out the Importer's entries that contain errors.
NOTE: The above proposed fix is a little band-aid-y. Consider a more long-term solution.

I'm not sure I'd say that's band-aid-y ... because we're exporting WORKS, we only want the entries that will have works attached, hence filtering out errors is the obvious way to do that.

Commit to proposed fix: https://github.com/samvera-labs/bulkrax/pull/68/commits/ea029495b349c983dd53a69dc1567b31285aca55


Keep Code 4 IBM i LIBL and CURLIB in synch with the Project Explorer LIBL

Today when an Code for IBM i action, it uses the Code for IBM i library list which is not the same as the one defined by the iproj.json from the project.
Hence when the Run Action is executed from the project

or active editor

the wrong library is used as CURLIB or to look up DB Refs or includes

The solution is to maintain the .env variables CURLIB and LIBL whenever iproj.json curlib and preUsrlibl and postUsrlibl and reference variables change.
@SanjulaGanepola Sanjula could you tackle this?

A few points to consider before doing this:

We'll need new commands in Code for i to do this. Changing current library is already possible, changing the entire library list is not.
Project Explorer should not change the default library list but instead create a dedicated profile (always the same; so, create it if needed), make it active and change its library list. This may require new commands too (to be checked).
This way, the users won't loose their default library list when using Project Explorer.

@edmundreinhardt I will be a bit busy for the next two weeks, so I likely won't be able to get a fix in until then. And as Sebastien mentioned, this will require changes in Code for i

I mentioned this to Edmund, I don't believe changing Code for IBM i is the right way to do this. I think, per project, should be using the .env file, which would be automatically updated when the project library list changes.

Note that this is a duplicate of #254

@worksofliam @sebjulliand
It is not clear from Seb's comments whether it is sufficient to update the LIBL variable in .env in order to affect the library list used when running Code for IBM i actions.  Could you please clarify?


Update to latest leveldown-hyper

https://github.com/Level/level-hyper/commit/9ef263254ef8b209e9e9679de21ffa4fdd60e4e6


Add $ back for GetTotalExpenditure

As approved by prof, see: https://github.com/nus-cs2103-AY2324S1/forum/issues/421

No worries at all! Definitely something that was hard to notice at first glance


commit merge and post_apply can make tikv panic

Bug Report

What version of TiKV are you using?
v4.0.9
Because on_ready_commit_merge will remove source read delegate,
https://github.com/tikv/tikv/blob/bf92a4a9b56f410f3c0eefe8056c22923a08ceee/components/raftstore/src/store/fsm/peer.rs#L2774
and post_apply will update read_delegate
https://github.com/tikv/tikv/blob/bf92a4a9b56f410f3c0eefe8056c22923a08ceee/components/raftstore/src/store/peer.rs#L2094-L2095
. If post_apply on source is executed after on_ready_commit_merge on target, and before on_merge_result, TiKV will panic as it can't find the read delegate.
/cc @gengliqi @NingLin-P

The target region will not execute the CommitMerge command until the source region catches up logs.

target region apply CommitMerge and wait for the source region's LogsUpToDate message
source region
a. catch up logs and apply PrepareMerge
b. enter on_ready_result and then send LogsUpToDate
c. return from on_ready_result and on_apply
target region execute the CommitMerge command and then on_ready_commit_merge

The case may occur if step 3 happened between 2.b and 2.c, but this should be very rare.

run tipocket testcase tipocket-bank2-tiflash1 occur this issue，stack as follow:
[2021/09/14 07:38:39.905 +00:00] [FATAL] [lib.rs:465] ["called Option::unwrap() on a None value"] [backtrace="stack backtrace:\n   0: tikv_util::set_panic_hook::{{closure}}\n             at components/tikv_util/src/lib.rs:464\n   1: std::panicking::rust_panic_with_hook\n             at library/std/src/panicking.rs:626\n   2: std::panicking::begin_panic_handler::{{closure}}\n             at library/std/src/panicking.rs:517\n   3: std::sys_common::backtrace::__rust_end_short_backtrace\n             at library/std/src/sys_common/backtrace.rs:141\n   4: rust_begin_unwind\n             at library/std/src/panicking.rs:515\n   5: core::panicking::panic_fmt\n             at library/core/src/panicking.rs:92\n   6: core::panicking::panic\n             at library/core/src/panicking.rs:50\n   7: core::option::Option::unwrap\n             at rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/core/src/option.rs:722\n      raftstore::store::peer::Peer<EK,ER>::post_apply\n             at home/jenkins/agent/workspace/build_tikv_multi_branch_master/tikv/components/raftstore/src/store/peer.rs:2197\n      raftstore::store::fsm::peer::PeerFsmDelegate<EK,ER,T>::on_apply_res\n             at home/jenkins/agent/workspace/build_tikv_multi_branch_master/tikv/components/raftstore/src/store/fsm/peer.rs:1268\n      raftstore::store::fsm::peer::PeerFsmDelegate<EK,ER,T>::handle_msgs\n             at home/jenkins/agent/workspace/build_tikv_multi_branch_master/tikv/components/raftstore/src/store/fsm/peer.rs:618\n   8: <raftstore::store::fsm::store::RaftPoller<EK,ER,T> as batch_system::batch::PollHandler<raftstore::store::fsm::peer::PeerFsm<EK,ER>,raftstore::store::fsm::store::StoreFsm>>::handle_normal\n             at home/jenkins/agent/workspace/build_tikv_multi_branch_master/tikv/components/raftstore/src/store/fsm/store.rs:896\n   9: batch_system::batch::Poller<N,C,Handler>::poll\n             at home/jenkins/agent/workspace/build_tikv_multi_branch_master/tikv/components/batch-system/src/batch.rs:310\n  10: batch_system::batch::BatchSystem<N,C>::start_poller::{{closure}}\n             at home/jenkins/agent/workspace/build_tikv_multi_branch_master/tikv/components/batch-system/src/batch.rs:422\n      std::sys_common::backtrace::__rust_begin_short_backtrace\n             at rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/std/src/sys_common/backtrace.rs:125\n  11: std::thread::Builder::spawn_unchecked::{{closure}}::{{closure}}\n             at rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/std/src/thread/mod.rs:476\n      <std::panic::AssertUnwindSafe as core::ops::function::FnOnce<()>>::call_once\n             at rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/std/src/panic.rs:347\n      std::panicking::try::do_call\n             at rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/std/src/panicking.rs:401\n      std::panicking::try\n             at rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/std/src/panicking.rs:365\n      std::panic::catch_unwind\n             at rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/std/src/panic.rs:434\n      std::thread::Builder::spawn_unchecked::{{closure}}\n             at rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/std/src/thread/mod.rs:475\n      core::ops::function::FnOnce::call_once{{vtable.shim}}\n             at rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/core/src/ops/function.rs:227\n  12: <alloc::boxed::Box<F,A> as core::ops::function::FnOnce>::call_once\n             at rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/alloc/src/boxed.rs:1572\n      <alloc::boxed::Box<F,A> as core::ops::function::FnOnce>::call_once\n             at rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/alloc/src/boxed.rs:1572\n      std::sys::unix::thread::thread::new::thread_start\n             at library/std/src/sys/unix/thread.rs:91\n  13: \n  14: clone\n"] [location=/home/jenkins/agent/workspace/build_tikv_multi_branch_master/tikv/components/raftstore/src/store/peer.rs:2197] [thread_name=raftstore-4-0]
tikv version:
[2021/09/14 06:35:54.353 +00:00] [INFO] [lib.rs:80] ["Welcome to TiKV"]
[2021/09/14 06:35:54.353 +00:00] [INFO] [lib.rs:85] ["Release Version:   5.3.0-alpha"]
[2021/09/14 06:35:54.353 +00:00] [INFO] [lib.rs:85] ["Edition:           Community"]
[2021/09/14 06:35:54.353 +00:00] [INFO] [lib.rs:85] ["Git Commit Hash:   d377cfda49c0f2fbd5e7b4501dfdfbe310f1fe90"]
[2021/09/14 06:35:54.353 +00:00] [INFO] [lib.rs:85] ["Git Commit Branch: master"]
[2021/09/14 06:35:54.353 +00:00] [INFO] [lib.rs:85] ["UTC Build Time:    Unknown (env var does not exist when building)"]

Duplicated with #11852.


Convert Musicgen to MLX

I would like to express my gratitude for your hard work on this project! I am interested in converting Musigen from the Meta team to MLX. To optimize the model for my limited RAM, I am considering using the smaller-model version (https://huggingface.co/facebook/musicgen-stereo-small).
Could you please suggest a good starting point for this conversion? From the Hugging Face implementation, I can see that it uses the T5 encoder for text encoding, which is already available in this repository, and Encodec from Meta team for audio encoding.
Thank you for your assistance!

That would be awesome! For conversions usually the best place to start is a reference PyTorch implementation. This is a slightly bigger project since it involves multiple models (encodec, the music generator and the LM).
We already have a T5 example as you mentioned.
It might make sense to have Encodec as a standalone example since it's useful for lot's of downstream audio generation. For example, I was looking recently at converting a different TTS model which also uses it. Maybe that is a good place to start?

Yes, I was thinking the same of Separating Encodec as a different module as it could be used individually and in many TTS systems like VALLE and VITs.

Is anyone working on a port of Encodec? If not I might take a stab myself as I'm interested in getting some audio generation up and running!

@awni  I just started yesterday night and there are modules which are directly available in torch like LSTM and sequential layers which are used in ENcodec but not available in mlx directly. I started from encodec main repo as it was pretty simple. I will be getting time only over weekends as i have my org works too.  And i really want audio generation up and running.

Got it. Ok let me know what's missing in terms of layers etc that should be in mlx.nn and we can prioritize getting them in. For example there is a PR for RNNs/LSTMs out now that we can try to get merged sooner.
If you think it will take a while, you can always start a draft PR and we can collaborate on it!

Yes i checked that PR today morning. it has most of the things. i will go through Encodec code and come back with more details. Maybe by tomorrow. @awni  just a suggestion, can we keep discussions instead of issues as most of the issues reported here are only enhancements as mlx is still growing and wrt to performance i haven't got any issues till now.

@awni
below modules will be required in mlx and some existing PRs and issues have already been addressed.

Setting dilation, groups in convolution layer - https://github.com/ml-explore/mlx/issues/100
Addition of sequential layers like lstm, rnn, gru - https://github.com/ml-explore/mlx/pull/268
torch has a function for full layer normalisation which will be helpful here - https://pytorch.org/docs/stable/_modules/torch/nn/utils/weight_norm.html#weight_norm

Rest of the items seems easily portable. for point 1 i have tried using cnn without dilation and group params as the values set did not have major impact when i went through pytorch code. it's default only.

I'm digging into the C++ for it https://github.com/pytorch/pytorch/blob/834c7a1d3ea07878ad87d127ee28606fc140b552/aten/src/ATen/native/WeightNorm.cpp#L50
I'm fine with C++ but not configured to build the MLX project.. questioning motivation on a Saturday night haha

We have a discord link here: https://github.com/ml-explore/mlx/discussions/733
You shouldn't need to implement weight norm in C++. That can all be done in Python using existing ops.

That's great, and thanks a lot for the input @awni. Part of me was thinking that too.

This was done. Check it out here.


Make the output channel generic

Currently, event listening via the watch command does not support generic channel management. The user is forced to choose between the tokio feature or the standard Rust channel (see: src/watch/mod.rs#L38-L46). The purpose of this issue is to investigate whether it is possible to elegantly support any type of channel.
TODO

[ ] Investigate

The less than elegant solution is to rely on flume or another crate supporting both sync and async at the same time. If you think it is acceptable I can provide a PR

@lu-zero this could save a feature. However, the goal here is to find a way to be compatible with all types of channels to avoid using multiple types of channels in a crate.

There isn't a common trait across all the channel implementation so there isn't a way even to support alternative implementations of sync channels like flume vs crossbeam vs std or async channels such as flume vs tokio vs async-channel ^^;

Yes, you are right, there won't be any generics in terms of Rust declarations. However, we could instead find a way to make it "generic" by using a Fn (that takes the event as an argument), which will be defined by the users, as a parameter and is called every time an event is sent.

Storing Fn requires a generic and tends to be annoying in my experience though...

Storing Fn requires a generic

@lu-zero How does this imply a generic declaration? Here's what I imagined, what's your point of view?
// /!\ Pseudo code
let (sender, receiver) = mpsc::channel();
db.watch(move |event| {
    // Send the event message into the channel
   sender.send(event).unwrap();
});

As you can see, you can put whatever you want in the closure. This does not imply a generic.

@lu-zero Can you provide a detailed example of a situation where this would pose a problem?

It is easier to write down the signature of your pseudocode I guess :)
// `clo` is generic over anything implementing `FnMut`
fn watch(&mut self, clo: impl FnMut(EventTy)) {

// and if we want to save it someplace we have to box it
self.closure = Box::new(clo);

...
}

This should be a way to make it work for sync, for async it gets more annoying since we do not have yet async closures.

@lu-zero Yes, that's exactly the idea! And indeed, for async, we'll have to wait a bit ;)
If you need support for flume or something else in the meantime, you can add it to the two existing channel supports, and I will merge it.

I do not need it myself, but since it is just few minutes to implement it using flume, it could solve the use-case of keeping the same API across features.


HandleManager::destroy() undeclared identifier using MT 5

Returning compilation error using MT5 MetaQuotes/MetaEditor Version5.00 build 2375 31/Mar/2020 into file MQL5\Include\Mql\Lang\GlobalVariable.mqh into lines 228:
if(!m_cs.isValid()) {HandleManager::destroy(m_ref); return;}
and line 233:
HandleManager::destroy(m_ref);

(Windows 10/64bits)
The same project does not return compilation error using version MT 5 MetaEditor Version 5.00 build 2363 13/Mar/2020 AND version MT 5 MetaEditor Version 5.00 build 2361 08/MAr/2020

Same issue after updating MT5 to v5.00 build 2380 2/Apr/2020. (Windows 10/64bits).

I'm commenting this lines at this time and no bad effects until now..

@a2gsdev as long as you don't use multiple contexts, this should be fine.
But I think this is really a bug in the MQL compiler, since a typename parameter should not be checked for existence of member functions.
template<typename A>
class Context {
public:
	Context() {
		A::create();
	}

	~Context() {
		A::destroy();
	}
};

class B: public Context<B> {
	static void create();
	static void destroy();
};

This is a common pattern in C++ (which MQL imitates).
MetaQuotes is known to introduce bugs as their MQL implementation is not well documented and full of incompatibilities. I have to come up with a solution to work around this problem.

@a2gsdev as long as you don't use multiple contexts, this should be fine.
But I think this is really a bug in the MQL compiler, since a typename parameter should not be checked for existence of member functions.
template<typename A>
class Context {
public:
	Context() {
		A::create();
	}

	~Context() {
		A::destroy();
	}
};

class B: public Context<B> {
	static void create();
	static void destroy();
};

This is a common pattern in C++ (which MQL imitates).
MetaQuotes is known to introduce bugs as their MQL implementation is not well documented and full of incompatibilities. I have to come up with a solution to work around this problem.

Thank you @dingmaotu for your work! And sorry change issue status :(  my error...

Hi fellas!
I found better solution on russian MQL5 forum
template<typename T>
class GetClassType{
public:
   class type : public T{};
};

class Foo {
public:
    static void Start() {
        Print("Hallo!");
    }
};

template<typename T>
class Bar {
public:
    Bar() {
        GetClassType<T>::type::Start();
    }
};

void OnStart() {
    Bar<Foo> var;
}

So you have just use GetClassType everywhere instead of making base classes.

@aisurfer Nice solution and it is a surprise that they started to support nested class definition! I think a lot of mql-lib code can be refactored using this feature.
But the code does not work on my MT4 (build 1260). If we don't use class inheritance, we can directly reference T instead of a sub type of T, which I think is cleaner:
class Foo {
public:
    static void Start() {
        Print("Hallo!");
    }
};

template<typename T>
class Bar {
public:
    Bar() {
        T::Start();
    }
};

void OnStart()
  {
   Bar<Foo> var;
  }

Yes, this snippet is exact example that does not work on recent MT5 :) (as you know already!)
About nested class solution. I forgot that your library is cross-MT-platform, so yes, let's go on with current fix, it's okay!


Some packages are not ingested with correct construct major version

Example: https://constructs.dev/data/cdktf-aws-secure/v2.0.1/metadata.json
This package takes<EMAIL_ADDRESS>as a dependency, so it should be labeled with the construct framework majorVersion 0, just like this one is: http://constructs.dev/data/cdktf-snowflake/v0.0.4/metadata.json

From https://www.npmjs.com/package/@cdktf/provider-aws:

This project is explicitly not tracking the Terraform aws Provider version 1:1. In fact, it always tracks latest of ~> 3.0 with every release.
...
While the Terraform Engine and the Terraform aws Provider are relatively stable, the Terraform CDK is in an early stage. Therefore, it's likely that there will be breaking changes.

Based on this I think we should just cherry-pick the framework version based on cdktf (so in practice, all packages will be marked as v0). I don't think displaying a version based on cdktf-provider versions will be reliable since I believe it's possible for community constructs to depend on multiple providers.


Add ability to get file as resource

There should be ability to get file from bucket as a resource, e.g. via fopen().
Amazon SDK supports this via Aws\S3\StreamWrapper.

Resolved by 62fee832db5b482b830a79c99206e02f825d83db


nixos/bind: add directory config option

Motivation for this change
On computers that establish their own Internet connection (e.g. a home router), the managed-keys file is corrupt if the Internet is not established when BIND starts up. With this change the file could be stored in a persistent directory.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

@ofborg test bind


[KM] OEN payment api

OEN payment api connnected workflow

It takes 6 hours.
https://github.com/CAFECA-IO/KnowledgeManagement/blob/master/Research/payment_service_oen.md
done


[MsTeams][Mobile][iOS][Action.showcard button not getting displayed]

Platform
What platform is your issue or question related to? (Delete other platforms).

iOS

Author or host
MsTeams
#Details
HI Team,
I have trying to show two different form in adaptive card by using action.Showcard property
this worked fine on desktop and android device but when same thing I'm trying to get on iOS device it not get rendered.
in case of android device I'm getting card like

but In case of iOS device buttons are not getting displayed
please help me on this
PFA sample adaptive card json
ActionShowcard.txt
same concern i have raised in issue https://github.com/microsoft/AdaptiveCards/issues/3890.
As per @nesalang comment on issue raising this issue separately.

Thanks for reporting this @Sujit-Gaikwad ... @nesalang can you help take a look please? Thanks!

@Sujit-Gaikwad
Hello,
Thanks for sharing the issue with us.
Can you share what version of iOS Teams app you are using?
@Aditya0603
Hi Aditya,
Can you take a look at this issue?
On SDK side, this card works as below.

@nesalang : this version of iOS and teams app we are using
iphone 6
version - 12.4.5
ms teams app version - 2.0.2
ipad mini
version - 12.4.5
ms teams app version -2.0.2

@Sujit-Gaikwad
Hello,
Thanks for sharing the issue with us.
Can you share what version of iOS Teams app you are using?
@Aditya0603
Hi Aditya,
Can you take a look at this issue?
On SDK side, this card works as below.

@shalinijoshi19 can you help me forward this issue to Teams?

@shalinijoshi19 can you help me forward this issue to Teams?

any updated  on this??

@vikrantsingh01 I think @jwoo-msft has a question here for your team?

This is working fine on the latest Teams iOS store build. @Sujit-Gaikwad please check it out.

@adumrewal  Thanks for an update

@adumrewal closing this out per @adumrewal's update above. Do reach out if you run into any other issues.


Statistical analysis of dataset

I can't calculate the statistical relationship between the quantity and distance of each category. Isn't the distance 256?

Distance bin size is 256. The actual maximal range and range resolution have to be computed with radar configurations.
I had the scripts for statistical analysis be4, but I re-installed the system on my laptop.........


[ISSUE-841][REFACTOR] Migrate shuffle client side conf to Celeborn Configuration System

…
[BUG]/[FEATURE] title
What changes were proposed in this pull request?
Why are the changes needed?

What are the items that need reviewer attention?
Related issues.
close #841
Related pull requests.
How was this patch tested?
/cc @related-reviewer
/assign @main-reviewer

ping @waitinfuture @FMX @pan3793


Provider exec

Thank you for this great provider! I have been sampling different Terraform providers which utilize Kustomize and stumbled across this one. I would like to request that exec block be available in the provider configuration. The Helm provider has this available.

Would you like to contribute to this feature? Any contributions is welcome!😁

Definitely! I'll have some time to dedicate to this around next month

Thanks for showing enthusiasm for this project. Have a nice day!


Calling FindNextElement with Direction as Next or Previous on FocusManager with FindNextElementOptions throws exception

Describe the bug
When trying to find the next focusable element using FindNextElement on FocusManager, you can't provide FindNextElementOptions with a SearchRoot if using Next or Previous as the Direction.
Steps to reproduce the bug

Populate a FindNextElementOptions structure with a SearchRoot
Call FindNextElement on FocusManager, provide the options and specify Next or Previous as the Direction

Expected behavior
It should return the next element that can receive focus in that direction
Version Info
Windows 10, version vb_release.191004-1432
Additional context
Throws an unexpected exception:
Platform::InvalidArgumentException ^ at memory location 0x000000AECB0FCF20. HRESULT:0x80070057 The parameter is incorrect.
WinRT information: Focus navigation directions Next and Previous are not supported when using FindNextElementOptions.

As the FindNextElement docs state:

Tab navigation (FocusNavigationDirection.Previous and FocusNavigationDirection.Next ) cannot be used with FindNextElementOptions. Only directional navigation (FocusNavigationDirection.Up, FocusNavigationDirection.Down, FocusNavigationDirection.Left, or FocusNavigationDirection.Right ) is valid.

So not a bug.

My team added this API and it's a bug that we don't support these two directions, moving this to my team.  We'll look at addressing this in WinUI 3+.


Disable Pester test colors

Description
Disable Pester test colors
Related issue:
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2022,macos12,ubuntu2204


arXiv:2411.04274v1 [cs.IT] 06 Nov 2024

Description
html document is incomplete
(Optional:) Please add any files, screenshots, or other information here.
No response
(Required) What is this issue most closely related to? Select one.
Choose One
Internal issue ID
be250e6a-ecd8-4cb3-8a8c-b9bf7c8969b6
Paper URL
https://arxiv.org/html/2411.04274v1
Browser
Firefox/132.0
Device Type
Desktop

@dublv which parts are missing? It looks like a finished article to me.

The html doc is looking fine now. This was not the case a few weeks ago.
thanks for looking into this.
Prof. Vinay A. Vaishampayan
Electrical Engineering
CUNY-CSI

From: Deyan Ginev @.>
Sent: Friday, November 22, 2024 3:12 PM
To: arXiv/html_feedback @.>
Cc: Vinay Vaishampayan @.>; Mention @.>
Subject: Re: [arXiv/html_feedback] arXiv:2411.04274v1 [cs.IT] 06 Nov 2024 (Issue #2571)

This email originates from a sender outside of CUNY. Verify the sender before replying or clicking on links and attachments. *

@dublvhttps://urldefense.com/v3/__https://github.com/dublv__;!!Jg5O23S-og4!7-hxx3el_QBUG2Z1H6ZZbFAbkK6OQSqE8G-EvLTrcDKJ7LuQaLxslC9zdw6rQqYRa41fCInY6v8ChEKACioW0uiTf__BBPd8EInF$ which parts are missing? It looks like a finished article to me.
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/arXiv/html_feedback/issues/2571*issuecomment-2494718477__;Iw!!Jg5O23S-og4!7-hxx3el_QBUG2Z1H6ZZbFAbkK6OQSqE8G-EvLTrcDKJ7LuQaLxslC9zdw6rQqYRa41fCInY6v8ChEKACioW0uiTf__BBNikk0Fk$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AMUQCUS3X243U354K66FALT2B6F4RAVCNFSM6AAAAABRSOZTVSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJUG4YTQNBXG4__;!!Jg5O23S-og4!7-hxx3el_QBUG2Z1H6ZZbFAbkK6OQSqE8G-EvLTrcDKJ7LuQaLxslC9zdw6rQqYRa41fCInY6v8ChEKACioW0uiTf__BBKpYTXn7$.
You are receiving this because you were mentioned.Message ID: @.***>

Thank you, that points to a very specific technical issue we are aware of. Much appreciated.


Error during ParallelLoop or parallel returns unhelp error message

🐛 Bug Report
If an op fails (for whatever reason) inside a parallel loop, the return error is child call failed but does not provide any traceback or other information regarding the cause of the fail. This makes development on parallel ops frustrating as one has to refactor to a serial loop in order to determine the cause of the failure.
There's a slack message outlining a way to debug but doesn't actually fix the problem.
To Reproduce

Write an op with a parallel loop that one of the child steps fails
Will result in child call failed error

Expected behavior
An op with a failure inside a parallel loop would function like other failurs: output the cause of the error to the command line.
Actual Behavior
An op inside a parallel loop outputs child call failed error regardless of cause.

Release 0.1.50 included some fixes for error logging. Could you see if you're still hitting this with latest? If so could you do us a favor and post an op that repros this behavior? If not we'll close this out as resolved.


Format of AWS profiles has changed in version 2.8.0

We've received reports from developers that the format of AWS profiles has changed from this:
aws-account-/role_name
to this:
aws-account-role_name
This has caused some frustration because documentation and automation scripts reference the format prior to 2.8.0. When I look at the release notes for 2.8.0, I'm not seeing an explanation of why this change was made (maybe I missed it, please share if you can).
Expected Behavior

Profile names to match this format aws-account-/role_name, not the new one that was introduced in 2.8.0.
Current Behavior

Profile names are being written out as aws-account-role_name which is a deviation from the previous format.
Possible Solution

Steps to Reproduce (for bugs)

Context

This change has caused unexpected problems in the way some of our automation scripts work because they assume the format that existed in versions prior to 2.8.0 was stable.
Your Environment

App Version used:

Environment name and version:

Operating System and version:

App Version used: <IP_ADDRESS>

Environment name and version: N/A

Operating System and version: MacOS Sonoma 14.2.1, Apple Silicon

This is fixed in master - When reading the config file, True and False were read in as Strings instead of Booleans.  That was changed in 2.8.0 but the comparisons for include_path and resolve_alias were missed.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1452 ms

Resolved: Bofrost.at performance has improved in 07a59e0.


fix spelling in log.debug

"update" instead of "uldate"

Coverage remained the same at 90.476% when pulling 7a3100839685afe30df6975542a0e04f20483902 on fix-spelling-log-debug into 1bf4a3484e9bee956648cc027e4dd73e9f11159d on dev.


Add p_median to math library

This Issue tracks my contribution of p_median() to the math library.

Possible duplicate of #54

CC @olajep

Indeed, although there were no implementation of p_median when I started coding.
However the current implementation is O(n log n) because it sorts the whole array, and uses the stack instead of the heap (stack generally has little space available). My solution have a expected time of O(n) (uses QuickSelect) and uses heap.

While I was trying to perform tests between the current and mine implementations I realized the current is totally wrong and fixed it in the first commit 718a9ba.
After some improvements in my code it performed 5x faster than the current one for 1024x1024 arrays random filled, and it replaces the old implementation in the second commit cacf8f5.

What is the purpose of the rand function in this context?

The same when used in quicksort, create an "expected running time". However for simplicity I removed the rand function. I had forgotten the srand call but already removed too.

@WesleyCeraso thanks for pointing out that the wrong array was being used in the original implementation (the median_task example uses arrays that are sorted, and so was hiding the issue). Unfortunately, your fix in 718a9ba is incorrect, line 28 should not have a -1 in it.
Line 28 is executed when there is an odd number of elements, so only the division / bit shift is needed. For example, if n is 5, the bit shift reduces this to 2, which is the third (and middle) element of the array.

@6thimage Thanks for your comment. However, if you remeber that the c language is 0-based, you will realize that my implementation is correct.

@6thimage Thinking once more about it you are absolutely right, I`ll fix that asap.

@6thimage Its fixed now. Im going to run more test on my implementation, but I think it`s ok now. By the way, do you know what should we do when the input is zero length? Should we let it crash or what?

@WesleyCeraso That is a good question - crashing is bad, as it will likely just hide the issue. Ideally, we need to report the error - unfortunately, the only output of the function is the returned array. Obviously, we can't set the returned value to zero to indicate an error, but perhaps it could be set to +/- inf or nan? Although this still has the effect of hiding the origin of the problem (if it returns nan, was the data nan before it entered the function?).
I think the best way to do this would be to have a global error variable for the math library, which we can set to a specific 'invalid input length' value to indicate an error, whilst returning nan from the function. This would then allow a program to check the cause of the nan. However, to keep all math functions similar, it would require that the majority of math functions update this error variable (of course only setting errors and not removing them). Which would be quite a large fundamental change to the workings of the math library, perhaps @aolofsson would like to weigh in on this.

@aolofsson My change is now quite smaller and faster than previous and is tested.
Regarding the conversation with @6thimage, couldn't we return 0/1 for no_error/error or something like that?

@WesleyCeraso None of the other math functions return values, so for this function to return a value seems odd and against the rest of the library.
Thinking about it a bit more, I prefer the global variable approach as it would allow functions that don't care too much about their input to still be defined as void (saving operations and memory). For example, the mean function should have an output of zero for no input, and so it returning a constant no error value seems like a waste. Where as, the global variable approach allows functions to opt in for error reporting.


Warning when actions have same pattern

Address #466 by providing a user-visible warning when two actions have patterns with the same signature (tested and having actions with the same signature no longer messes up the book).

send it


DL: Improve performance for predict

JIRA: MADLIB-1343
Performance improvements

Using SD to cache the model and set the weights only once for the
first row for each segment. This also meant that we had to clear the SD
for the last row for each segment.

We replaced PythonFunctionBodyOnly with
PythonFunctionBodyOnlyNoSchema in the internal keras predict sql.
Using PythonFunctionBodyOnly made the query much slower because it
added the overhead of executing the schema query for every row in the
test table. We don't really need to know the schema name for the
internal UDF so now we use PythonFunctionBodyOnlyNoSchema instead.

Additionally:

Replace the use of predict_classes and proba with predict since non
sequential models do not support predict_classes.
Modify the internal keras predict query to not join the test table
and the model table because it caused weird inconsistencies with
the segment id due to which SD was not getting set/cleared properly.
Add try catch in the internal predict UDF so that we can clear out
the SD in case of an error.
Reorder arguments for fit and evaluate UDA

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/913/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/914/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/917/

(1)
model_size is not computed correctly
DROP TABLE IF EXISTS model1, model1_summary;

SELECT madlib.madlib_keras_fit('mnist_train_packed',  -- source_table
                               'model1', -- model
                               'model_arch_library',  -- model_arch_table
                               1,  -- model_arch_id
                               $$ loss='categorical_crossentropy', optimizer='adam', metrics=['acc'] $$,  -- compile_params
                               NULL,  -- fit_params
                               5,  -- num_iterations
                               0,  -- gpus per host
                               NULL,  -- validation_table
                               NULL,  -- metrics compute frequency
                               'Frank',  -- name
                               'A test model'  -- description
                              );
madlib=# select model_size from model1_summary;
 model_size
------------
         43
(1 row)

model_simple = Sequential()
model_simple.add(Dense(100, activation='relu', input_shape=(784,)))
model_simple.add(Dense(10, activation='softmax'))

model_simple.summary()

Layer (type)                 Output Shape              Param #
=================================================================
dense_1 (Dense)              (None, 100)               78500
_________________________________________________________________
dense_2 (Dense)              (None, 10)                1010
=================================================================
Total params: 79,510
Trainable params: 79,510
Non-trainable params: 0
_________________________________________________________________

Should be the size in bytes of the bytea storage of the model weights.

LGTM
(0) timings on a 2 segment GP5 cluster on CPU
10K predictions
Time: 6077.533 ms

50K predictions
Time: 26784.859 ms

This is much better than before, so looks OK for now.
(1) response
DROP TABLE IF EXISTS mnist_test_prediction;

SELECT madlib.madlib_keras_predict('model1', -- model
                                   'mnist_test',  -- test_table
                                   'id',  -- id column
                                   'x', -- independent var
                                   'mnist_test_prediction'  -- output table
                                   );

SELECT round(count(*)*100/10000.0,2) as test_accuracy_percent from
    (select mnist_test.y as actual, mnist_test_prediction.estimated_y as estimated
     from mnist_test_prediction inner join mnist_test
     on mnist_test.id=mnist_test_prediction.id) q
WHERE q.actual=q.estimated;

 test_accuracy_percent
-----------------------
                 97.18
(1 row)

OK, accuracy as expected
(2) prob
DROP TABLE IF EXISTS mnist_test_prediction;

SELECT madlib.madlib_keras_predict('model1', -- model
                                   'mnist_test',  -- test_table
                                   'id',  -- id column
                                   'x', -- independent var
                                   'mnist_test_prediction',  -- output table
                                   'prob'
                                   );

SELECT * FROM mnist_test_prediction LIMIT 10;

 id |    prob_0     |     prob_1     |    prob_2     |    prob_3     |    prob_4     |    prob_5     |     prob_6     |    prob_7     |    prob_8     |     prob_9
----+---------------+----------------+---------------+---------------+---------------+---------------+----------------+---------------+---------------+----------------
  2 |  9.880677e-07 |   0.0010419372 |     0.9986218 |   6.84927e-05 | 1.2382638e-12 | 0.00024825096 |   6.948901e-07 | 6.1711205e-13 | 1.7804521e-05 |  6.0889234e-14
  4 |     0.9999366 |  2.8256621e-09 | 6.6410103e-06 | 8.4593694e-07 | 7.4489925e-08 | 3.2383898e-05 |  2.9831072e-06 |  9.729254e-06 |  5.108415e-08 | 1.07344185e-05
 12 |  0.0011051608 |  4.6904443e-08 |  3.491906e-05 | 2.4090486e-05 | 2.4900895e-05 |  0.0010728389 |     0.99681705 | 1.4937936e-07 | 0.00092049537 |  2.8245486e-07
 14 |     0.9999608 |  1.6933312e-08 |  3.732735e-06 | 5.4127717e-08 | 9.5467556e-08 | 4.5280945e-06 |   3.560989e-06 | 4.5551647e-06 |  2.301657e-06 |  2.0342342e-05
 20 | 2.2352947e-07 |    9.69698e-09 | 1.6347584e-06 |  1.932719e-06 |     0.9992822 | 3.8802187e-05 |  4.0969678e-07 |  3.759048e-05 | 3.6773997e-06 |   0.0006335543
 22 | 4.0293417e-05 |   6.249896e-09 | 2.8016773e-06 |  7.212842e-07 | 4.9328148e-05 |  0.0026104918 |      0.9971624 | 1.2448003e-08 | 0.00012794712 |  5.9117415e-06
 26 |     0.9999931 | 1.24117915e-14 | 2.2590778e-09 | 1.8620958e-12 |  6.718903e-08 | 2.8648715e-11 |   5.973386e-06 | 4.3913162e-10 | 2.1172746e-09 |  8.0753307e-07
 28 |  1.634239e-07 |   3.018619e-10 |  3.811128e-06 |  7.628142e-08 |     0.9992101 |  5.254892e-05 |  1.4238293e-06 |  5.970224e-06 | 3.5786666e-06 |   0.0007222573
 34 |   0.033928625 | 1.19020935e-08 |  7.676703e-05 | 1.4824612e-07 |    0.94482905 |  0.0005254827 |    0.020210627 | 0.00023137043 |  7.968004e-05 |  0.00011828837
 36 | 2.8703305e-06 |   0.0022232432 |     0.9966936 |  0.0010433431 |  5.737119e-11 | 1.1605422e-05 | 1.04220526e-08 | 3.3659873e-08 | 2.5263162e-05 |   8.308043e-09
(10 rows)

OK
(3) ask for a GPU but there isn't one avail
DROP TABLE IF EXISTS mnist_test_prediction;

SELECT madlib.madlib_keras_predict('model1', -- model
                                   'mnist_test',  -- test_table
                                   'id',  -- id column
                                   'x', -- independent var
                                   'mnist_test_prediction',  -- output table
                                   NULL,
                                   1 -- GPU's per segment host
                                   );

ERROR:  plpy.SPIError: plpy.Error: Cannot assign a device for operation Placeholder: node Placeholder (defined at /home/gpadmin/.local/lib/python2.7/site-packages/keras/backend/tensorflow_backend.py:2464) was explicitly assigned to /device:GPU:0 but available devices are [ /job:localhost/replica:0/task:0/device:CPU:0, /job:localhost/replica:0/task:0/device:XLA_CPU:0 ]. Make sure the device specification refers to a valid device. The requested device appears to be a GPU, but CUDA is not enabled. (plpython.c:5038)  (seg1 slice1 <IP_ADDRESS>:40001 pid=14544) (plpython.c:5038)
DETAIL:

	 [[node Placeholder (defined at /home/gpadmin/.local/lib/python2.7/site-packages/keras/backend/tensorflow_backend.py:2464) ]]

Caused by op u'Placeholder', defined at:
  File "<string>", line 1, in <module>
  File "<string>", line 8, in __plpython_procedure_internal_keras_predict_66207
  File "/home/gpadmin/madlib/build/src/ports/greenplum/5/modules/deep_learning/madlib_keras_predict.py", line 165, in internal_keras_predict
    set_model_weights(model, device_name, model_data, model_shapes)
  File "/home/gpadmin/madlib/build/src/ports/greenplum/5/modu
Traceback (most recent call last):
  PL/Python function "internal_keras_predict", line 7, in <module>
    return madlib_keras_predict.internal_keras_predict(**globals())
  PL/Python function "internal_keras_predict", line 210, in internal_keras_predict
PL/Python function "internal_keras_predict"
CONTEXT:  Traceback (most recent call last):
  PL/Python function "madlib_keras_predict", line 28, in <module>
    gpus_per_host)
  PL/Python function "madlib_keras_predict", line 112, in predict
PL/Python function "madlib_keras_predict"

OK

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/921/