WIP feat(material-experimental): add test harnesses for slide-toggle

Hey look I actually wrote some code for once
This is almost done, but for some reason the ng-invalid class is not
being added to the slide-toggle, making the isValid test fail.

@jelbourn That actually looks like a bug. I just reproduced it in a StackBlitz (see here). We need to provide the proper checkbox validator when required. The default required validator just asserts that the value is non-undefined.. which is always the case for a boolean state.


Support mixins for abstract classes

Search Terms
Related to #32122, but not the same.
abstract mixins classes
Suggestion
Right now we can't to extending abstract classes with mixins. So, it would be nice to create a solution for this case.
Use Cases
abstract class Test {
}

type Constructor = new (...args: any[]) => any;

function MyAwesomeMixin<T extends Constructor>(Constructor: Constructor) {
  abstract class MyAwesomeMixin extends Constructor {
  }
  return MyAwesomeMixin;
}

class A extends MyAwesomeMixin(Test) {
  /* Argument of type 'typeof Test' is not assignable to parameter of type 'Constructor'.
  Cannot assign an abstract constructor type to a non-abstract constructor type.(2345) */
}

Okay, try to create abstract constructor type:
type Constructor = abstract new (...args: any[]) => any;
/* Syntax error */

Hmm,..
Examples
Here is two way to create solution for this problem:

(simple) Allow create abstract constructor types

type Constructor = (new (...args: any[]) => any) | (abstract new (...args: any[]) => any);

(complex, but beautiful) Add new syntax to define mixins, and initially add support to extending abstract classes, like this:

abstract class Test {
   abstract mainMethod(): void;
}

mixin MyAwesomeMixin {
   public someMethod(): void {
      console.log('hello');
   }
}

class A extends MyAwesomeMixin(Test) {
   public mainMethod(): void {
      this.someMethod();
   }
}

// AND

abstract mixin MyAwesomeAbstractMixin {
    abstract someMethod(): void;
}

class B extends MyAwesomeAbstractMixin(Test) {
   public mainMethod(): void {
      this.someMethod();
   }
}

// Also this can add support to extending mixins without Mixin(Mixin2(Mixin3(Class)))

mixin MyAwesomeExtendedMixin extends MyAwesomeAbstractMixin {
   public someMethod(): void {
      ...
   }
}

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

Also realetd https://github.com/microsoft/TypeScript/issues/29653, IMO. I came to the same conclusion, that there is a need for a way to define a constructor that is explicitly abstract.

What would it mean to have an abstract constructor type?  You can’t actually use new with it, which I thought was the only purpose of a construct signature.

@fatcerberus you can't new but you can inherit and call it in the constructor of the derived class.
Currently if the base class signature returns an abstract class (ie. You are extending something of type new (...args :any) => AbstractClass) , it is assumed that this represents an abstract class and this forces you to implement the abstract methods, when in reality
new (...args :any) => AbstractClass could also represent the constructor of any derived class.

@fatcerberus mixins should input any classes (non-abstract & abstract).
If i define constructor type, like this:
type Constructor = new (...args: any[]) => any;

I give error: Argument of type 'typeof Test' is not assignable to parameter of type 'Constructor'.
Cannot assign an abstract constructor type to a non-abstract constructor type.(2345)
Playground
This is so uncomfortable, because you need to implement abstract class before extending with mixins.
Playground

The type system already tracks if a class is abstract. Can't it simply just return that type from a mixin function?

@trusktr If I completely trust the system, the result will be sad.
Playground

New thoughts about the new syntax:
Mixins should support generics like others
mixin A<T> {
   public method(value: T) {
      return value;
   }
}

Mixin can be abstract
abstract mixin Base {
   abstract protected config: any;

   public getConfig(): any {
      return this.config;
   }
}

mixin MyMixin extends Base {
   protected config = {};
}

Mixin can override methods from extended mixin
mixin Base {
   method() {
      console.log('base method call');
   }
}

mixin Extended extends Base {
   method() {
      super.method();
      console.log('extended method call');
   }
}

Mixins can't be have constructor.
mixin MyMixin {
   constructor() {} // Error - mixin can't be have constructor
}

### Mixin should infer constructor type from deriver classes

class BaseClass {
constructor(a: number, b: string) {}
}
mixin MyAwesomeMixin {
method() {};
}
class Extended extends MyAwesomeMixin(BaseClass) {
constructor() {
super(0, '');
}
}

The type system already tracks if a class is abstract. Can't it simply just return that type from a mixin function?

@trusktr I'm not sure what you mean. If I completely trust the system, the result will be sad.
Playground

I didn't mean that as something for you to do, I just meant maybe TypeScript can return the abstract class type back out of the mixin function, because TypeScript already knows that the mixin's abstract class is extending an abstract class, so I don't see any reason (other than it isn't implemented) that TypeScript cannot simply return an abstract class from the mixin. :)

About the mixin InvalidMixin { syntax, I don't think that is possible because TypeScript aims to provide types for JavaScript, but not invent new language features (other than ways to express types). This may not have been true in the past (f.e. namespace is not something in JavaScript), but I think it is more-so true today (mistakes of the past aside).

@trusktr If at the moment it is impossible to make a new syntax, at least make it possible to get the type of constructor of abstract classes.
See first example in main comment.

Starting with 4.2 we can declare "abstract" constructors with type AbstractCtor<T> = abstract new (...args: any[]) => T.
Thanks to that it's now possible to create mixins that extend abstract classes.
It works for me, until I try to use a generic class as the extension base.
I get a weird TS error that I just don't see how to fix.
I muted the error with //@ts-ignore-next-line and everything seems ok: both IDE types and runtime behavior.
Any idea? Am I doing something wrong or is this a TS limitation or bug?
abstract class A<P> {
  constructor(data: P) {
  }
}

type AbstractCtor<T> = abstract new(...args: any[]) => T;

function mixin<P, B extends AbstractCtor<A<P>>>(base: B) {
  abstract class M extends base {};
  return M;
}

// TS2344: A<number> does not satisfy the constraint AbstractCtor<A<number>>
class MixedA extends mixin<number, A<number>>(A) {
}

Playground:
https://www.typescriptlang.org/play?ts=4.2.3#code/IYIwzgLgTsDGEAJYBthjAgggHgAoD4EBvAWACgEkB7AO0igFd4qoAKAE2AmAC4FcAlMXLlKAX3ISy5CAE8ADgFMs4aHAgBhCC2wAVQgF4EoeuoQ1FAd1YA6O8CgBzMH2A1ZAbQC6Qg4V0A3CJkAGYMNPAAlrQIALaRAB6RNHgANAgAQgiKCRCKNOwYmKow8Fo6OAT4+KwgaIp8GUKkFMYlZihoGACy2bn5hQh1YMpEYkGtUIoQDFA0CN0TUuSd6AuJiuyYfXkFGPFJKTQMsSCKUOk4x6fn1ayYzcFiQA

Just after I wrote my previous comment, I eventually found a work-around, with an actually simpler declaration.
But it still fails because returning an abstract class from a mixin doesn't seem easy.
return class extends base fails because A doesn't implement abstract members.
It should be marked abstract but TS doesn't support abstract on class expressions.
👉 It would be nice if it did. Not sure if there are syntax ambiguities, but semantically seems ok.
abstract class M extends base is ok, but return M fails because the inferred returned type for the function exposes private name M.
I think you can work around that by explicitely declaring an interface for the mixin return type, but as your mixin grows in complexity, this becomes a copy-paste fest.
Also base & MixinInterface is not a perfect depiction of overriding classes, MixinInterface members do not hide base members, they intersect with them...
abstract class A<P> {
  constructor(public data: P) {
  }

  abstract x(): void;
}

abstract class AA<P> extends A<P> {
}

type AbstractCtor<T> = abstract new(...args: any[]) => T;

function mixin<P>(base: AbstractCtor<A<P>>) {
  // Fails because return type exposes private name M
  abstract class M extends base {};
  return M;

  // Fails because class does not implement x()
  // but TS doesn't support "abstract class" expressions
  return class extends base {};
}

class MixedA extends mixin<number>(AA) {
  x() {}
}

Playground:
https://www.typescriptlang.org/play?ts=4.4.0-beta&ssl=25&ssc=2&pln=1&pc=1#code/IYIwzgLgTsDGEAJYBthjAgggHgAoD4EBvAWACgEkB7AO0igFd4qoAKABwZGQEtYEAJsAjAAXAlwBKYuUoBfcrISh6cRAA9Wk8QDcqPAQG5yCsuRXQ1SVOiw4CCAKbqIjmgIz3CRSicVkIAE92RyxwS3gAYQgWbAAVQgBeZXCYeAQaRwB3VgA6fOAoAHMwcWAaQIBtAF1pRMI44zMyADMGGngeWgQAWx51Hho8fFYQNEdxTFS1aNivfGlSCgQAehWEADFgHmQMEEdYYAYwUKhHCAYoGgQgkKd1dioTjHYoHh1hUJpgHtCAWSUFjSiBQaAwf3urnce3GxDkTUoZwuVwQfyaSjWm22uwQ+0Ox1CoNsAiojgwNCoiB4PXYyEcvxoGi0GPWIAYiDiAGVBKSwDQAOSIMAMdiPKCIABEQKsRLAEvurzJYC6dCUSMu11lkLcHlxsKI8L8zS1f36jgEmG10N6-UG2BoDB6+ygI0wmEWSk0i1MciAA

@jods4  you want https://github.com/Microsoft/TypeScript/issues/21710
function subclass<T>(klasstype : T) : T {
  // This is our mixin, but for demo purposes just return the class
  return klasstype;
}

class DerivedPromise<T> extends subclass(Promise)<T> { }

The return value of calling mixin (or subclass above) is itself a (newable) function, which takes generic params, so you can say const Mixed = mixin(AA); class MixedA extends Mixed<number> { ... }.
Your error about exposing a private name is supposed to be fixed, for this exact use case, 4 years ago.  Take a look at tests/cases/compiler/declarationEmitLocalClassDeclarationMixin.ts in that PR.  I'm not sure why but your AbstractCtor type doesn't work while their Constructor interface does, even though they're very similar.  Take a look at them side by side on Playground.

This is already supported for abstract classes and can be closed.
See the AnyConstructor definition from lowclass:
https://github.com/trusktr/lowclass/blob/c182595253ee79f45e4770c97d1c55702e351866/src/Constructor.ts#L58-L60
And then use it like this:
function MyMixin<T extends AnyConstructor>(Base: T) {
  abstract class MyMixinClass extends Base {
    foo = 123 // non-abstract

    abstract someMethod(): whatever // abstract method, requires subclasses to implement
  }

  return MyMixinClass
}

class Foo extends MyMixin(SomeOtherClass) {
  // must implement:
  someMethod(): whatever {...}
}

Playground example

This is already supported for abstract classes and can be closed.

@trusktr It has been OK in the same project, but failed with external libraries. Such as:
function Mixin(Super: new (...data: any) => any) {
    abstract class MixinClass extends Super {
        abstract test: string
    }
    return MixinClass
}

class Test extends Mixin(class {}) {
    // should implements abstract members
}

https://www.typescriptlang.org/play/?#code/GYVwdgxgLglg9mABAWRgDxmAFAZRABwFMAnALkTEIHdEsA6BgEwEMpnzmwBPASkQF4AfIk69EAbwBQiGSIBGAZyjFm0RBAA2zBQpTpMAYS07EhNFEJhGuvEWITpsp80XLVURBaXklxTAHNHGQBfIMRiQigQYiRUDDAjbQVJUMlNJMQAFUIlU3NLaz14rHSTcWC+KScAemrEBQALOBANRkQYAFt8DUIOyyhdF193RD6OuRJk4KA
but if you copy the .d.ts generated in the right column of TS playground to the left column, the type alert disappears:
// lib.d.ts
declare function Mixin(Super: new (...data: any) => any): abstract new (...data: any) => {
    [x: string]: any;
    test: string;
};
declare const Test_base: abstract new (...data: any) => {
    [x: string]: any;
    test: string;
};
// app.ts
class Test extends Test_base {
    // no alert about missing abstract member implements
}

https://www.typescriptlang.org/play/?#code/PTAEBsEsCMDoBNYBcDOAoeBTAxuAhgE6agBmArgHbZKQD2FoAspAB6QUAUAymQA6YEAXKAqYA7qA6xp8PEjzC8FAJ4BKUAF4AfKCVrF0FEgJ5qI8ZOkI5C3SvXbQAbzSg3oANothRguwDmALqKKgDcru5ImEY+xgHhAL7hWLiExNj0RqAAKtFIAPrQeCiYBr6mSOYSUjI2IWqaOi7unt6gvgHBdsrhLVEx7XEU-onhILq8vMjoqSgoOXmgmCxRFPDzuUaFxcTN7uMUtLrgApV40LRklQC2kHMBuobGFaDXmNfQAqCQ17wnbxRUGgEmggA
then if you add abstract keyword to a class member in .d.ts, the alert comes back with a syntax error:
// lib.d.ts
declare function Mixin(Super: new (...data: any) => any): abstract new (...data: any) => {
    [x: string]: any;
    test: string;
};
declare const Test_base: abstract new (...data: any) => {
    [x: string]: any;
-    test: string;
+    abstract test: string;  // syntax error
};
// app.ts
class Test extends Test_base {
    // should implements abstract members
}

https://www.typescriptlang.org/play/?#code/PTAEBsEsCMDoBNYBcDOAoeBTAxuAhgE6agBmArgHbZKQD2FoAspAB6QUAUAymQA6YEAXKAqYA7qA6xp8PEjzC8FAJ4BKUAF4AfKCVrF0FEgJ5qI8ZOkI5C3SvXbQAbzSg3oANothRguwDmALqKKgDcru5ImEY+xgHhAL7hWLiExNj0RqAAKtFIAPrQeCiYBr6mSOYSUjI2IWqaOi7unt6gvgHBdsrhLXiGxhWgUTHtcRT+oW4g7coU8iygAgS0BGhJaDN4vLzI6KkoKDl5SyxRFPBHuUaFxcTN7jMoABa0ZODwoJAAtrzgmN9MPMjv1ymZAd9oAJ0AkgA

@TechQuery I see what you mean. It may be worth opening a new issue specifically for that as a declaration issue, linking to this one as related.


Fix #6130 objmode cache segfault

Fixes #6130
Thanks @juanjgalvez and @stuartarchibald for finding out the source of the problem on the lost env.
The patch makes sure the CompileResult acquires a reference to any referenced Environment objects. Changes to numba/core/compiler.py is the actual fix. The other non-test changes are for better error handling when an NULL pointer is encountered.

failing due to: https://github.com/numba/numba/pull/6168

/AzurePipelines run

CI passed, marked for merge


Make jawn-ast dependency for test only

Changed the jawn-ast dependency to test only as I think it was intended to be 😄

Thanks, @ghostdogpr !


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_32_19habla/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_32_19habla/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_32_19habla/clone/recipe/meta.yaml:27: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_32_19habla/clone/recipe/meta.yaml:33: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_32_19habla/clone/recipe/meta.yaml:40: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_32_19habla/clone/recipe/meta.yaml:40: missing_description: The recipe is missing a description
ERROR: /tmp/abs_32_19habla/clone/recipe/meta.yaml:40: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_32_19habla/clone/recipe/meta.yaml:40: http_url: http://github.com/google/jsonnet is not https
Errors were found


JBPM-5084: troubleshooting function for workitem

@mswiderski
It is update code, include RetryWorkItemCommand.
Thanks

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

ok to test

could be good to align with formatting, besides that looks good

@mswiderski
About code formatting , i use https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/tree/master/ide-configuration/eclipse-configuration/code-style,
and  formatted it , but also have some different .

jenkins retest this

jenkins retest this

+1


Issue #471 - Surface URI Click Callbacks for HTML Links

Issue Link -> https://github.com/MohamedRejeb/compose-rich-editor/issues/471
PR Description :
Added Callbacks for the Html link clicks so that it would be useful to handle clicks and tracking events etc, configurable to the dev
Code Changes
Added Callbacks for both BasicRichText and RichText to handle callback clicks

After the discussion on the issue, I will close this PR.


Initial PR: add internal, pkg,  cmd, makefile and build the project

see commits for details.

found issues in vendoring, but wasn't clear why as nothing seemed different (git diff branch-1 branch-2).
closing in favor #2


Please add changelog

Very time consuming to read every commit
Adding it from version 0.6.0 is good enough

:+1:

I guess it won't be done for 0.6.0, maybe 0.7.0 Q_Q?

Done for 0.7.0 :smiley:


DNS resolution is broken after ostree upgrade during OKD4 install

Describe the bug
While installing the bootstrap node from an ISO,
the first boot copies the base image onto the hard drive and reboots,
and the second boot downloads an upgraded ostree and reboots again.
Upon the second reboot, DNS resolution is broken,
and installation cannot proceed further.
Version
Occurs with stable and testing branches of Fedora CoreOS.
Version without the issue (from /etc/os-release)
NAME=Fedora
VERSION="33.20201201.3.0 (CoreOS)"
ID=fedora
VERSION_ID=33
VERSION_CODENAME=""
PLATFORM_ID="platform:f33"
PRETTY_NAME="Fedora CoreOS 33.20201201.3.0"
ANSI_COLOR="0;38;2;60;110;180"
LOGO=fedora-logo-icon
CPE_NAME="cpe:/o:fedoraproject:fedora:33"
HOME_URL="https://getfedora.org/coreos/"
DOCUMENTATION_URL="https://docs.fedoraproject.org/en-US/fedora-coreos/"
SUPPORT_URL="https://github.com/coreos/fedora-coreos-tracker/"
BUG_REPORT_URL="https://github.com/coreos/fedora-coreos-tracker/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=33
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=33
PRIVACY_POLICY_URL="https://fedoraproject.org/wiki/Legal:PrivacyPolicy"
VARIANT="CoreOS"
VARIANT_ID=coreos
OSTREE_VERSION='33.20201201.3.0'

and also
NAME=Fedora
VERSION="33.20201214.2.0 (CoreOS)"
ID=fedora
VERSION_ID=33
VERSION_CODENAME=""
PLATFORM_ID="platform:f33"
PRETTY_NAME="Fedora CoreOS 33.20201214.2.0"
ANSI_COLOR="0;38;2;60;110;180"
LOGO=fedora-logo-icon
CPE_NAME="cpe:/o:fedoraproject:fedora:33"
HOME_URL="https://getfedora.org/coreos/"
DOCUMENTATION_URL="https://docs.fedoraproject.org/en-US/fedora-coreos/"
SUPPORT_URL="https://github.com/coreos/fedora-coreos-tracker/"
BUG_REPORT_URL="https://github.com/coreos/fedora-coreos-tracker/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=33
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=33
PRIVACY_POLICY_URL="https://fedoraproject.org/wiki/Legal:PrivacyPolicy"
VARIANT="CoreOS"
VARIANT_ID=coreos
OSTREE_VERSION='33.20201214.2.0'

Version with the issue (from /etc/os-release)
NAME=Fedora
VERSION="33.20201209.10.0 (CoreOS)"
ID=fedora
VERSION_ID=33
VERSION_CODENAME=""
PLATFORM_ID="platform:f33"
PRETTY_NAME="Fedora CoreOS 33.20201209.10.0"
ANSI_COLOR="0;38;2;60;110;180"
LOGO=fedora-logo-icon
CPE_NAME="cpe:/o:fedoraproject:fedora:33"
HOME_URL="https://getfedora.org/coreos/"
DOCUMENTATION_URL="https://docs.fedoraproject.org/en-US/fedora-coreos/"
SUPPORT_URL="https://github.com/coreos/fedora-coreos-tracker/"
BUG_REPORT_URL="https://github.com/coreos/fedora-coreos-tracker/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=33
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=33
PRIVACY_POLICY_URL="https://fedoraproject.org/wiki/Legal:PrivacyPolicy"
VARIANT="CoreOS"
VARIANT_ID=coreos
OSTREE_VERSION='33.20201209.10.0'

Ignition configs generated from installer openshift-install-linux-4.6.0-0.okd-2020-12-12-135354.tar.gz
Installation is bare-metal (on libvirt) with static IP addresses passed on kernel command line via grub.
How reproducible
Failure happens 100% of the time.
Log bundle
Installation fails before OKD components are fully installed.
Preliminary debugging
During successful boots:
/etc/resolv.conf -> ../run/systemd/resolve/stub-resolv.conf -> resolv.conf

During unsuccessful boots:
/etc/resolv.conf -> (missing) -> resolv.conf

During unsuccessful boots /etc/systemd/resolved.conf.d/no-dns-stub.conf exists with the following contents:
[Resolve]
DNSStubListener=no

This causes systemd-resolved to skip creation of /run/systemd/resolve/stub-resolv.conf
It is unclear to me who is creating no-dns-stub.conf as it doesn't seem to be part of bootstrap.ign and is not part of the original CoreOS ISO.  However it appears here under a different filename: https://github.com/coreos/baselayout/commit/db31383ab708eb0b8a03547c2cf88536a3132391.  Also possibly related to https://github.com/coreos/fedora-coreos-config/blob/0ab295fe90b3024cf2bc4ba3279324334366ae96/manifests/fedora-coreos-base.yaml#L91; https://github.com/coreos/fedora-coreos-tracker/issues/649#issuecomment-736104003

Looks like a duplicate of https://github.com/openshift/okd/issues/440.


transport is undef in HttpConnection

Hi guys -- thanks for making all this public.
I'm trying to get a simple app working with update1. Server code is great, but the js is giving me issues (and I'm fairly new with modern javascript so still catching up):
let connection = new HubConnection('some url');
connection.start().then((tr, opt) => {
            debugger;
        },
        (err) => {
             debugger;
        });

this results in the following error (and the promise's error callback being invoked):

Error: Failed to start the connection. TypeError: Cannot set property 'onreceive' of undefined

calling /negotiate manually seems to work:
{
  "Url": "\/signalr",
  "ConnectionToken": "Z3z1w03ekE9MTiVft7UzfvgxJIe6UjVngyb0SI21SE10QSifO6C\/h5XB1p6cosKGFBVI7F6HsdMVh8nswZOy3Dkl1fLol2U19y1Nw0ejlJgKJgNKmOaqhl9aVLXyJPWq",
  "ConnectionId": "edf55548-cc7f-4633-b9f7-3fd439f32641",
  "KeepAliveTimeout": 20,
  "DisconnectTimeout": 30,
  "ConnectionTimeout": 110,
  "TryWebSockets": true,
  "ProtocolVersion": "1.2",
  "TransportConnectTimeout": 5,
  "LongPollDelay": 0
}

Looks like you're trying to use the SignalR Core client with a 2.X Server, that is unsupported. The new client is not compatible with the old server, you have to use the new server.

Ok, thank you; I'll switch out the server and retry and update the thread shortly.

Wonderful, it starts! Thanks again.


Handle AzureException during storage upload

Backport #3930

(DCOS-40442) AzureException during storage upload

@mesosphere-mergebot label ready for review

@mesosphere-mergebot merge-it


Pass args to grlc Dockerfile

For example, the sparql_endpoint value is currently hard-coded in config.ini. One could use ARG to pass the value to Dockerfile.

True -- but this does not matter too much as the sparql_endpoint in config.ini is the value set by default. But all our queries can take the endpoint as a parameter.

fixed in clariah/grlc:latest (https://github.com/candYgene/docker-images/pull/8)

fixed in clariah/grlc:latest (https://github.com/candYgene/docker-images/pull/8)


REQUEST: Add docker support

Please provide a way to pull and run pgweb image by docker.

https://github.com/sosedoff/pgweb/wiki/Installation#docker
https://github.com/sosedoff/pgweb/wiki/Docker


Small knob polishing and amend duplicated attempts to add undetectable images to pickle

Hope you don't mind some small adjustments
Fixes #1030
Comments welcome.

LGTM


As a user I want to be able to open an IDE or text editor easily.

Suggested new feature or improvement
From within the Dev Home app, I should be able to open an external Visual Studio or VS Code, IntelliJ, notepad++, notepad, vim, what have you. Give me a widget for running a batch command. (These are all examples, but just generally we should be able to set up something like a button that runs an executable.
Scenario
I'm a developer, I'm using this new product "Dev Home" and I have a stream lined experience interacting with my a database or application instance, etc.
Additional details
No response

related items: https://github.com/microsoft/devhome/issues/1984 https://github.com/microsoft/devhome/issues/2043

/dup https://github.com/microsoft/devhome/issues/2462


Refund after fulfillment

POSTRefund has the following piece of code:
if state != pb.OrderState_FUNDED {
	ErrorResponse(w, http.StatusBadRequest, "order must be funded before refunding")
	return
}

What if the vendor has fulfilled the order (e.g. sent the goods) but the goods were damaged or the buyer was not satisfied, and the vendor wants to make a refund?
Should the vendor be able to make a refund if the order state is FULFILLED?

The code snippet above only checks that it was funded. Should still work if it's fulfilled.

As I understand, if the state is FULFILLED then if state != pb.OrderState_FUNDED fails, so the function returns, and i.node.RefundOrder doesn't get called.
if state != pb.OrderState_FUNDED {
	ErrorResponse(w, http.StatusBadRequest, "order must be funded before refunding")
	return
}
err = i.node.RefundOrder(contract, records)

Looks like you're right.


Arguments for Powershell in integrated terminal not working as expected

When using feature to start PowerShell with arguments (terminal.integrated.shellArgs.windows ), the terminal closes.  Argument "-noexit" need to be part of the arguments send as part of the command.
Steps to Reproduce:

Set configuration

    "terminal.integrated.shell.windows": "C:\\windows\\System32\\WindowsPowerShell\\v1.0\\powershell.exe",
    "terminal.integrated.shellArgs.windows": ["dir"],

Then this will launch the terminal but it is going to close.  If terminal.integrated.shellArgs.windows option is deleted then the powershell will start as normal.
2.- Workaround, add the option -noexit to force PowerShell to remain open.
    "terminal.integrated.shell.windows": "C:\\windows\\System32\\WindowsPowerShell\\v1.0\\powershell.exe",
    "terminal.integrated.shellArgs.windows": ["-noexit", "dir"],

This may be not an issue but it was not expected by me.

Does this issue occur when all extensions are disabled?: Yes

This is working as expected, shellArgs is an advanced level feature that allows the user finer control, hardcoding -noexit when the shell is detected as powershell or something would be frail and take away control from the user.


US890043: Added Java21 images and moved NodeJS18 to NodeJS20

Ticket: https://internal.almoctane.com/ui/entity-navigation?p=131002/6001&entityType=work_item&id=890043
Moved Java 21 images into this repository and changed NodeJS18 image to NodeJS20 image as nodejs18 is not available on Leap 15.6

A developer build has not yet been created for this branch. Click here to go ahead and create the build...


subscribe never failed when sentinel master switched

Looking for thoughts here -- it's possible I'm mis-using a portion of the library.  I've got a go program that ties two different redis instances together via pub/sub:

Redis A, sentinels, psubscribe for published events
Redis B, AWS elasticache, publish transformed events

.. the other day we had a sentinel failover in Redis A; the event count dropped to 0, no events were being published to B, cue disaster music.  The particular redis instance we were connected to never died, but no events were being published to it after sentinel failover.
The library may well -- probably did -- receive the +switch-master notification, but that never bubbled up to me, and the existing connection was happy.
How would you handle this case?  Is there an intention to bubble sentinel events up to clients in some way?

What currently happens is that go-redis tries to close free connections to the old master. But PubSub connections are never free and therefore that connection is never closed. I think the proper fix would be to close all connections after master address is changed.

Sentinel should be fixed in v6.5.1. Keeping this open as reminder to do something similar for Redis Cluster

I've similar issue (should I open another?), when Redis Cluster failed beside blocking and non-blocking operations, and some master changes its state:
CLUSTERDOWN The cluster is down
MOVED 5441 x.x.x.x:yyyy

The cluster resurrects, but several connections to the old masters stays established.


Improve contributing/dev setup guide

While I was working on #185 I noticed that vl-convert-python#development-setup mentions using conda for an env.
I thought this was strange as the .gitignore seems to imply there may be a pixi config in-use.
https://github.com/vega/vl-convert/blob/8f5200cf6d92d8cf712fe097d128b8f1d7b48a88/.gitignore#L17-L21
I'm curious about contributing more in the future, and a project-style config/setup guide would be super helpful in making sure everything is set up correctly.
@jonmmease are you using pixi?
From my understanding, hatch (used in https://github.com/vega/altair and https://github.com/vega/sphinxext-altair) wouldn't be a good fit due to non-python dependencies.
Side note
Not everything from there is transferable, but I used parts of https://docs.pola.rs/development/contributing/#setting-up-your-local-environment to help with setting up the rust side of things.
Maybe there are parts of that guide that could be drawn from?

Hi @dangotbanned, I'm using pixi locally and would like use it officially here as I do in VegaFusion.
A nice thing about using pixi is that we can install Rust inside the environment, so that it's not necessary to have a system installation of Rust.

Hi @dangotbanned, I'm using pixi locally and would like use it officially here as I do in VegaFusion.
A nice thing about using pixi is that we can install Rust inside the environment, so that it's not necessary to have a system installation of Rust.

Perfect!
I've not used pixi before, but if you're already using it then hopefully resolving this won't be too much work

Yeah, I'll try to add the pixi config soon

Working on pixi migration in https://github.com/vega/vl-convert/pull/187, but ran into some Rust build issues on CI that I haven't gotten to the bottom of yet.


[Citations] Implement Citations list page & block

Reference: https://www.moleculardevices.com/citations
Content: https://main--moleculardevices--hlxsites.hlx.live/citations
Data: https://main--moleculardevices--hlxsites.hlx.live/resources/citations/query-index.json?sheet=citations

Index to get citation list from: https://main--moleculardevices--hlxsites.hlx.page/resources/citations/query-index.json


downloader.js in npm wrong executable

These lines in downloader.js
19:  //download the correct version of the binary based on the platform
20:  url = exec(url);

seem to be missing in the npm module.
So it will always download the linux version of the executable.

Hey great find, kinda tedious changing the executable every install

This should be fixed, consider comment if you see strange behavior there


Persistent array: values reset to 0

In the example gif below, the value is reset by simply opening and closing the editor, but it seems other events can reset it too.

Here's another way how to reset values by simply opening and closing the editor (without editing anything). To me it looks like it's connected with the cursor position:


I don't know what's wrong with my terminal today. Where is my dollar sign???

ADD ISSUE DESCRIPTION HERE
Version: 1.69.0-insider
Commit: fbf1cf3832d43088e27837dbb68d24ab65a098c1
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

I think the problem I have stated at stackchange with pictures will be more straightforward.
https://cs50.stackexchange.com/questions/42819/need-help-with-my-terminal-i-have-been-strugggling-with-it-one-day
Thank you for help!!!!!!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


[ISSUE] Lots of missing Optionals in the type hinting of WorkspaceClient

Description
In lots of places the SDK's type hints look like:
foo: str = None

This typing is incorrect and should be:
foo: str | None = None

To be honest, this project doesn't make use of mypy typechecking quite yet. We've been meaning to add support for it, and I think we'd be open to discussion about whether enabling --implicit-optional is preferable or not for this project.

I think that using explicit Optional is probably preferable given that not using it requires downstream consumers to have to configure their type checker correctly vs just using the defaults.

This seems reasonable to me, especially as this seems to be the direction that mypy and the ecosystem are moving towards.

This seems reasonable to me, especially as this seems to be the direction that mypy and the ecosystem are moving towards.

This might not be the right place to raise this but it would be nice if this could be changed/updated in databrick-connect too. I'd open a PR but unfortunately the source isn't available anywhere other that PyPi.


Test app for locust

Created this based on the slack conversation here
The request is to create a test app that can be used by new users to experiment and play around with locust. In addition to the documentation, this can potentially increase the level of understanding even further.
The test app could be a simple flask app mimicking a real-world API like weather etc to make it a bit more realistic to work with. However, this is something to think about.
Most new users might not check out the locust repo on their local so it wouldn't make sense to bundle it up with the existing repo. Alternatively, a separate PyPi package would be easier to install and consume i.e. pip install locust-testapp (or something similar).

Would be nice to have! Do you think you'll have a look at it at some point?


e2 file funcs malfunctioning?

This was in singleplayer/lan...
@persist FilePath:string
@trigger none

if (first()) {
    FilePath = "myfile.txt"
    fileLoad(FilePath)
    runOnFile(1)
}

if (fileClk()) {
    local File = fileRead()

    if (File != "") {
        print(File)
    } else {
        print("Empty...")
        fileWrite(FilePath, "someContents")
    }
}

It works properly the first time you spawn the script, but any subsequent spawns and it will not trigger fileClk until you reload expression2 (wire_expression2_reload). Once you reload expression2, if the file exists, it seems to trigger just fine.

Any errors in console?


Update get-package-sas-uri.md

Updated limitations and response.

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/5901


Add vertical bar between "Edit" and "Delete" buttons and arrow in List Cells

commit: https://github.com/johneastman/todo/commit/0b30d2bc65362c8e274283330ff1334fe99c9641


More default FormFields

Use case
I'm trying to create a more complex form using PopupMenuButton, Checkbox and CheckboxListTile. But I have to rely on packages for "Form validations" in those fields (for example, accepting EULA is necessary). E.g.: https://pub.dev/packages/checkbox_formfield and https://pub.dev/packages/select_form_field
Proposal
I would like to see some of these features implemented by Flutter with widgets like TextFormField or similar. I would like my project to have the same pattern for dealing with form fields and the same look to the outcome of errors. So I leave here my suggestion for the widgets that I'm using and maybe even some others like DateFormField or DateRangeFormField

You might be happy with the reactive_forms package.
I am using it for all my forms, they also have it modular, so when I created my own input widget, I was able to implement it to be compatible with this package. (I'm not affiliated with the author)

Hello @FMorschel. This looks like an issue solved by loads of packages that the community has already built. Although Flutter comes in-built with a lot of widgets, expected all of them to be there is infeasible. Also, the use case of accepting EULA is a relatively simple one to validate using Flutter's in-build Form.
For the above reasons, I am closing this issue. If you disagree, please comment and I can reopen it. Thank you.


variable "strategy" is not defined in CandlestickFinder.js

Here is the stack trace:
ReferenceError: strategy is not defined
    at AbandonedBaby._getLastDataForCandleStick (...\node_modules\technicalindicators\lib\candlestick\CandlestickFinder.js:51:75)
    at AbandonedBaby.hasPattern (...\node_modules\technicalindicators\lib\candlestick\CandlestickFinder.js:45:43)
...
...

Hmmm, Do not use candlestick patterns yet, it is not complete, I'll add example in runkit once it is done, This is very basic there will be a lot of changes when we add more candlestick so the api may break.

Hmmm, Do not use candlestick patterns yet, it is not complete, I'll add example in runkit once it is done, This is very basic there will be a lot of changes when we add more candlestick so the api may break.

I think you are using the old version, has pattern should work in latest version, all tests are passing. It should fail only in getAllPatternIndex.

I think you are using the old version, has pattern should work in latest version, all tests are passing. It should fail only in getAllPatternIndex.

I don't sure whether old version is used, but variable strategy cannot be found.
Even the pattern check haven't been fully implemented, I really can't wait for this interesting feature. Please let me put some comments here:
If array cannot fulfill requiredCount, exception should not be thrown. It is because logically it should be grouped into "pattern not found".
It is no doubt that your approach is right, just because I have written code as follow as a trial run::
var input = {
  date: [..., ..., ..., ...],
  open: [..., ..., ..., ...],
  close: [..., ..., ..., ...],
  high: [..., ..., ..., ...],
  low: [..., ..., ..., ...]
}
for(i =0 ... length) {
  let candlePattern;
  if(TI.AbandonedBaby.hasPattern(input)) candlePattern = ", AbandonedBaby";
  else if(TI.BearishEngulfingPattern.hasPattern(input)) candlePattern = ", BearishEngulfingPattern";
  else if(TI.BullishEngulfingPattern.hasPattern(input)) candlePattern = ", BullishEngulfingPattern";
  else if(TI.DarkCloudCover.hasPattern(input)) candlePattern = ", DarkCloudCover";
  else if(TI.Doji.hasPattern(input)) candlePattern = ", Doji";
  else if(TI.DownsideTasukiGap.hasPattern(input)) candlePattern = ", DownsideTasukiGap";
  else if(TI.DragonFlyDoji.hasPattern(input)) candlePattern = ", DragonFlyDoji";
  ...
  //output when pattern is found
  ...
  input.date.shift();
  input.open.shift();
  input.close.shift();
  input.high.shift();
  input.low.shift();
}

As candle patterns have no fixed requiredCount, it seems that the simple way to achieve what I want is to catch the exceptions.
Anyway, that is just my thought. Thanks for your great code.

I don't sure whether old version is used, but variable strategy cannot be found.
Even the pattern check haven't been fully implemented, I really can't wait for this interesting feature. Please let me put some comments here:
If array cannot fulfill requiredCount, exception should not be thrown. It is because logically it should be grouped into "pattern not found".
It is no doubt that your approach is right, just because I have written code as follow as a trial run::
var input = {
  date: [..., ..., ..., ...],
  open: [..., ..., ..., ...],
  close: [..., ..., ..., ...],
  high: [..., ..., ..., ...],
  low: [..., ..., ..., ...]
}
for(i =0 ... length) {
  let candlePattern;
  if(TI.AbandonedBaby.hasPattern(input)) candlePattern = ", AbandonedBaby";
  else if(TI.BearishEngulfingPattern.hasPattern(input)) candlePattern = ", BearishEngulfingPattern";
  else if(TI.BullishEngulfingPattern.hasPattern(input)) candlePattern = ", BullishEngulfingPattern";
  else if(TI.DarkCloudCover.hasPattern(input)) candlePattern = ", DarkCloudCover";
  else if(TI.Doji.hasPattern(input)) candlePattern = ", Doji";
  else if(TI.DownsideTasukiGap.hasPattern(input)) candlePattern = ", DownsideTasukiGap";
  else if(TI.DragonFlyDoji.hasPattern(input)) candlePattern = ", DragonFlyDoji";
  ...
  //output when pattern is found
  ...
  input.date.shift();
  input.open.shift();
  input.close.shift();
  input.high.shift();
  input.low.shift();
}

As candle patterns have no fixed requiredCount, it seems that the simple way to achieve what I want is to catch the exceptions.
Anyway, that is just my thought. Thanks for your great code.

This is fixed on 0.2.9.
sample

Sorry may I know where is strategy defined? I saw it be used in CandlestickFinder.js but I am not able to find where it is declared. Thanks

CandlestickFinder is the parent class for all candlestick patterns, every candlestick pattern has a method logic, which is the strategyFn that is used. You can see this
https://github.com/anandanand84/technicalindicators/blob/master/lib/candlestick/CandlestickFinder.js#L32
and
https://github.com/anandanand84/technicalindicators/blob/master/lib/candlestick/BearishEngulfingPattern.js#L8

Please try to change the input as follows in the sample to:
var twoDayBearishInput = {
  open: [21.44],
  high: [25.10],
  close: [23.25],
  low: [20.82]
}

Then you will get what I mean.
Thanks!

Good catch!! It was due to dangerous copy paste. I have fixed it and added a test for it. Thanks. It should be available in version 0.2.10


Is it possible to migrate from "access" token to "sliding" token in existing app?

as in the question, we are using TokenObtainPairView ,
Is it possible to migrate to TokenObtainSlidingView?

I wouldn't recommend it.  I plan to eventually remove support for sliding tokens.


Optional inputs / generative tasks

This is low priority for v0, but I think it is worth assessing level of effort.
Would it make sense to make all inputs optional?
I understand we currently provide as an input everything, always: the image (with masked out patches), location, time, instrument, ...
I want to understand if we can drop some of those inputs, specifically the image one. These are some applications where this can unlock a lot of value:

If we don't provide an image it would be extremely valuable to make a finetuned application to generate the expected image. In essence creating a "semantically compressed" model of Earth. To the degree that this works, it could provide e.g. cloudfree RGB images, synthetic SAR, extremely fast approximation of a basemap in device...
If we don't provide a location we can still try to infer semantics. This is useful to leverage images without locations information. We can also finetune to predict locations, which can help locate, or fix locations of images.

I can see implementing this making the MAE 100% of the image some of the times, and/or masking other inputs (and adding a flag). I expect the loses much greater in those cases, but it can hopefully force the model to learn absolute anchors in the model, instead on relative to the available patches on the image.
Does this make sense?

That makes sense for a foundation model, it should be able to handle many different kinds of inputs and missing data etc. I think that also relates to time series of imagery.

We could envision a system that has "slots" for many different satellites systems and data sources, but not expecting all of them at once.
When going into time series data, we will have missing time steps as well, or "alternating" data providers that are not simultaneous but could be input in sequence.

This can be integrated in the masking strategy, a great example for a masking strategy that takes these things into account is NASA Havest's Presto model.
I agree though that for v0 its not the main focus.


Compass lesson is not labelled as a compass lesson, and it has a bug

microbit.org-ticket: 1459
I'm trying to migrate a blocks user who has found a bug in the blocks compass lesson over to PXT, but the PXT compass lesson that I found in the pxt/lessons page doesn't seem to be described as a compass lesson? It refers to 'creating a die on the micro:bit'.
https://pxt.microbit.org/lessons/compass

It is a compass lesson, but also note that it suffers from the same bug as the original compass lesson in blocks that the user is complaining is wrong
https://www.microbit.co.uk/blocks/lessons/compass

Is it possible for someone to fix this lesson please, so that we can at least try to migrate this user over to the new editor?
Thanks

Fixed.

https://github.com/Microsoft/pxt-microbit/commit/a2a28391d0f17bdb37adaa5780aaef7ecb9aba8e
https://github.com/Microsoft/pxt-microbit/commit/b3ad4e5b4dd9499dd988b438952858e49f34b394


feat: endpoint for group enrolment/withdraw for a user

TIS21-3897
TIS21-3901

@Judge40 Thanks for taking a look.
Haha, the generic endpoint came as I was worried about the upcoming beta group too much. XD
Just wondering would the Beta Private or other groups later be in use at the same time?

@Judge40 Thanks for taking a look. Haha, the generic endpoint came as I was worried about the upcoming beta group too much. XD Just wondering would the Beta Private or other groups later be in use at the same time?

Potentially yes, we might be best off with a new UserGroupResource with endpoints like /user-group/dsp-consultants/enroll /user-group/dsp-consultants/withdraw /user-group/dsp-private-beta/enroll /user-group/dsp-private-beta/withdraw


Circular reveal animation too fast when hiding sheet

On a Nexus 5 with android 6.0.1, the circular reveal works fine when clicking on the fab but plays too fast when it's reversed to hide the sheet.
I first saw that behavior in the app I'm coding using your great library but could witness it in the sample applicaiton as well. Maybe that's the intended behavior but it doesn't match the 'preview' visible in the readme.

The hide animation does look a bit fast but I'm not sure if I intentionally made it this way or if something changed in Android 6.0.1. Is the animation any different on older versions of Android?
The gif in the readme is also a bit laggy so it might just appear to be slower than it actually is.

It looks better on a Galaxy Nexus under 4.3, though a tad fast to my liking ^^
What strikes me is that I can't seem to see the circular unreveal on 6.0.1, looks like the sheet disapears and the button pops in a few frames.
Maybe you could expose an interface so we could fiddle with the values and not bother you ?

Any thoughts on how to make the closing animation as smooth as the opening animation?

The library that was implemented to make the animation (CircularReveal) only works when a layout is opening, it doesn't animate on the closing. So do you have any thoughts on how to animate it while closing?

@f8full
Do you happen to be using Proguard? The animations can be incredibly choppy if the Proguard rules in the readme aren't added.
I've also noticed that when there are very complex view layouts on screen, the animation can lag a bit. Can you try the sample app on your devices and see if the animation is any different? If it's not laggy on the sample app, then you might want to try optimizing your layouts and see if that helps.
@joaosardinha
It's already animating when closing. It's just that the sheet's visibility is being hidden partway through in an attempt to blend the multiple animations together.

Well yea it's animating but looks really weird >.<

Alright, after more investigation, my client code was faulty. Sorry about that.
Your library is great, I even learnt how to circular reveal by reading it :+1:

Had a big brain fart : I thought my problem was in the sheet reveal, and not hiding.
I can confirm the sample app displays the same "faulty" behavior on closing : circular unreveal last for a few frames and button pops. This is weird because the reveal from fab to sheet just looks perfect.
I don't use proguard and yeah, my layouts are a bit complex. I'll make sure to investigate deeper after the all nighters and release :)

The animation values may need some tweaking. I don't have time to look at it right now but feel free to play around with it and submit a PR if you think it looks good!

Sure thing ! And maybe, I'd code an interface to allow client code to tweak it. Just maybe.


Handle x86 and x64 bit version of raylib

Currently using a custom build of raylib for x64 bit but also want to support x86.
Need a way for the bindings to use both. They currently look for a file called raylib.dll in the same directory as the .exe.
Maybe copy the dll at build or runtime based on the architecture used?

look at this https://github.com/dotnet/standard/issues/584


Add real python blog about polars

Link to suggestion
https://realpython.com/polars-python/#reader-comments
Targeted section
Blog posts
Describe your suggestion
Good news from python world
Checklist

[X] checked if your suggestion is not already listed in Awesome Polars list.
[X] checked if it can fit in an existed section or suggest a new section and explain why your item does not fit into one and what would be the interest of your new section.
[X] checked when it's a repository if it's maintained (archived repository will be rejected).

@hfhoffman1144 your great tutorial has been added to the Awesome Polars List


Group Extras javadoc together in Guide

The documentation for various Extras modules is taking up a bit too much space in the Guide.  We want it there, but in a single item until expanded.

Is this how you imagined it?

Yes, that looks great.  I would probably want to change the structure to put the non-extras reference at the top level, but I assume you are reading the structure from the codeworld.shelf file?
Can it be nested more than two levels?  That doesn't need to be possible; I'm just curious.

I assume you are reading the structure from the codeworld.shelf file?

Yes, I believe that's where the structure is coming from.

Can it be nested more than two levels?

Sure thing, it can be nested as many times as needed.

That's great!  That should scale to much more documentation in the future.  For example, we could add a teacher's guide with tips on presentation, on building customized activities, on configuring the requirements checker, etc.  I've been unable to add some of these things because they would overwhelm students, but if advanced documentation is hidden under one section, a lot more becomes possible.

Do you want start reviewing changes already or I should wait with the PR until you adjust the structure?

I'm not sure what you mean.  I don't think I can change the structure until this change is in.  Right now, there is no way to nest documentation at all.

To clarify what I meant, the current codeworld.shelf file is at https://github.com/google/codeworld/blob/master/web/help/codeworld.shelf.  It is a JSON file that describes the help window organization.  Right now, inside the "named" object, the key/value pairs are the collapsible sections.
{
  ...
  "named": {
    "Section 1": "help/Section1.md",
    "Section 2": "help/Section2.md",
    "Section 3": "help/Section3.md",
    ...
  }
}

I think you should modify the code to accept entries in that object whose values are nested objects.  Something like this:
{
...
"named": {
"Section 1": "help/Section1.md",
"Section 2": "help/Section2.md",
"Section 3": {
"overview": "help/Section3.md",
"named": {
"Part 1": "help/Section3Part1.md",
"Part 2": "help/Section3Part2.md",
...
}
},
...
}
}

I'm not attached to this specific format, but hopefully you get the idea.  In your initial pull request, it's fine to modify `codeworld.shelf` to just group all of the appendices the way you have them now.  I just wanted to be sure that it will be easy to change that.

Thinking about this in detail does bring up a question.  When you open a section with nested subsections, what happens?
My example format above assumes that there is an overview document that is loaded.  Since there's no place to put the table of contents for that overview document, I assume you'd just skip that part, and the overview should be short enough not to require it.
Another alternative would be to automatically load the first child document, so expanding a section of the accordion would also expand its first child.  But that could be a weird experience, and make it harder to navigate to later child sections.  So I'm not in favor of this.  I prefer the overview approach.

Also: I haven't seen your code yet, but one thing to keep in mind is that a viewer's current location in the help is saved in local storage, and restored when help is reopened.  That is something else that will have to be updated.

When you open a section with nested subsections, what happens?

Right now the section just gets expanded, section on the right stays intact. I'm hesitant about the possibility to load an overview doc as accordion's markup has to be structured in a very specific way. Is it really important? If so, I'll try to find a workaround.

one thing to keep in mind is that a viewer's current location in the help is saved in local storage, and restored when help is reopened

At first glance, this might prove tricky to achieve with nested accordions as we'd have to manually determine indices (locations) in all parents to reveal article's title in the navbar. I'll look into it next week.

I'm hesitant about the possibility to load an overview doc as accordion's markup has to be structured in a very specific way. Is it really important? If so, I'll try to find a workaround.

I guess my question is: when I select a section that has nested subsections, what loads?  If it is the first subsection, that's fine and the overview isn't needed.  But if it's a blank page, that would be unfortunate.

We definitely need the saved location restore to keep working.  That makes the guide a lot more usable.  (Without it, you become scared to click on a code sample, because reopening the guide loses your place.)

when I select a section that has nested subsections, what loads?

Accordion's children, content on the right hand side stays intact (we load intro by default after initializing the guide). I can arrange for loading the first item in the list, but you pointed out earlier that it's undesirable and I tend to agree with you.

Oh, I see.  Well that's probably okay.


Issue launching LNBits

Hi guys,
I have a node that has been up for couple of years now and today I wanted to add the LNBits service. It installed just fine but I'm getting the following error message during the service launch:
Jan 02 21:53:03 raspberrypi systemd[1]: Started lnbits.
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.57 | INFO | Starting LNbits
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.57 | INFO | Host: <IP_ADDRESS>
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.57 | INFO | Port: 5000
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.57 | INFO | Debug: False
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.57 | INFO | Site title: LNbits
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.58 | INFO | Funding source: LndRestWallet
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.58 | INFO | Database: SQLite
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.58 | INFO | Data folder: /mnt/hdd/app-data/LNBits
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.58 | INFO | Git version: 3b98a10c9e65ea35e4e0fc3ff4f417954860f13a
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.58 | INFO | Started server process [55199]
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.58 | INFO | Waiting for application startup.
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.65 | ERROR | Traceback (most recent call last):
Jan 02 21:53:08 raspberrypi sh[55199]:   File "/home/lnbits/lnbits/venv/lib/python3.9/site-packages/starlette/routing.py", line 635, in lifespan
Jan 02 21:53:08 raspberrypi sh[55199]:     async with self.lifespan_context(app):
Jan 02 21:53:08 raspberrypi sh[55199]:   File "/home/lnbits/lnbits/venv/lib/python3.9/site-packages/starlette/routing.py", line 530, in aenter
Jan 02 21:53:08 raspberrypi sh[55199]:     await self._router.startup()
Jan 02 21:53:08 raspberrypi sh[55199]:   File "/home/lnbits/lnbits/venv/lib/python3.9/site-packages/starlette/routing.py", line 612, in startup
Jan 02 21:53:08 raspberrypi sh[55199]:     await handler()
Jan 02 21:53:08 raspberrypi sh[55199]:   File "./lnbits/app.py", line 108, in check_wallet_status
Jan 02 21:53:08 raspberrypi sh[55199]:     logger.error(
Jan 02 21:53:08 raspberrypi sh[55199]:   File "/home/lnbits/lnbits/venv/lib/python3.9/site-packages/loguru/_logger.py", line 1989, in error
Jan 02 21:53:08 raspberrypi sh[55199]:     __self._log("ERROR", None, False, __self._options, __message, args, kwargs)
Jan 02 21:53:08 raspberrypi sh[55199]:   File "/home/lnbits/lnbits/venv/lib/python3.9/site-packages/loguru/_logger.py", line 1954, in _log
Jan 02 21:53:08 raspberrypi sh[55199]:     log_record["message"] = message.format(*args, **kwargs)
Jan 02 21:53:08 raspberrypi sh[55199]: KeyError: '"code"'
Jan 02 21:53:08 raspberrypi sh[55199]: 2023-01-02 21:53:08.65 | ERROR | Application startup failed. Exiting.
Any idea what I did wrong by any chance ?
Thanks a lot !

Can you try an sd card update to the latest RaspiBlitz v1.9.0rc1 which contains the latest LNbits including bugfixes?
https://github.com/rootzoll/raspiblitz/tree/v1.9#downloading-the-software
MAINMENU > UPDATE > RELEASE

@VeniVidiViciLA  is this still an issue with 1.9.0?
Closing this for now, please re-open if it still persists.


Crash during npm building

Hi ;)
npm building crash on Gentoo Linux with:
+node ./bin/npm.js config set "prefix=/usr"
+node ./bin/npm.js config set "build=i686-pc-linux-gnu"
+node ./bin/npm.js config set "host=i686-pc-linux-gnu"
+node ./bin/npm.js config set "mandir=/usr/share/man"
+node ./bin/npm.js config set "infodir=/usr/share/info"
+node ./bin/npm.js config set "datadir=/usr/share"
+node ./bin/npm.js config set "sysconfdir=/etc"
+node ./bin/npm.js config set "localstatedir=/var/lib"
+node ./bin/npm.js config set "prefix=/var/tmp/portage/sys-apps/npm-1.0.26/image//usr"

+node ./bin/npm.js config ls

npm ERR! prefix directory not found
npm ERR! Error: ENOENT, No such file or directory '/var/tmp/portage/sys-apps/npm-1.0.26/image/usr'
npm ERR! Report this *entire* log at:
npm ERR!     <http://github.com/isaacs/npm/issues>
npm ERR! or email it to:
npm ERR<EMAIL_ADDRESS>npm ERR!
npm ERR! System Linux 3.0.7-tuxonice
npm ERR! command "node" "/var/tmp/portage/sys-apps/npm-1.0.26/work/npm-1.0.26/bin/npm.js" "config" "ls"
npm ERR! cwd /var/tmp/portage/sys-apps/npm-1.0.26/work/npm-1.0.26
npm ERR! node -v v0.4.12
npm ERR! npm -v 1.0.26
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /var/tmp/portage/sys-apps/npm-1.0.26/work/npm-1.0.26/npm-debug.log
npm not ok

I can't be responsible for every random package manager under the sun and what it tries to install.
Use the instructions in the npm readme.  If those don't work, please let me know, and I'll help you.

You're right and all works well with a "normal" installation, sorry for the convenience ;)


Added two more developers

@ankitashukla Thanks for this. It has some conflicts can you correct them?

@tapasweni-pathak Resolved! :)

@ankitashukla This seems to be a mess now. Can you please rebase?

Please reopen if updated or create a new one.


Suggestion: Make the branch regex configurable

Thanks for a great plugin!
Can you make the regex for finding the ID from the branch to be configurable?
Also add possibility for prefix?
Example
I've the branch "sprint/123456-WithSomeText" and I would love to have my commit messages prefixed with:
#123456:
is the prefix and : is the sufix. 123456 is found from banch name by configurable regex

Branch "sprint/123456-WithSomeText" does not work because the current version of the plugin only supports ticket systems like Jira that start the ticket ID with a project key followed by a dash. For example, ABC-1234 is a valid Jira ticket ID, but 1234 is not.
In my team we ran into this issue when we moved from Jira to Azure Boards. Therefore, we have made a change to the plugin to also support tickets that consist entirely of digits (like 1234). Branch "sprint/123456-WithSomeText" then works too. I am currently preparing a PR to hopefully get this merged and available to everybody.

Thank you both for your suggestions. I will not be able to look at it in
june. But i will give it a try in july/august.
Thank you for using my plugin and feel free to make a PR 🙂
OrtecJohnB @.***> schrieb am Di., 11. Juni 2024, 09:54:

Branch "sprint/123456-WithSomeText" does not work because the current
version of the plugin only supports ticket systems like Jira that start the
ticket ID with a project key followed by a dash. For example, ABC-1234 is a
valid Jira ticket ID, but 1234 is not.
In my team we ran into this issue when we moved from Jira to Azure Boards.
Therefore, we have made a change to the plugin to also support tickets that
consist entirely of digits (like 1234). Branch "sprint/123456-WithSomeText"
then works too. I am currently preparing a PR to hopefully get this merged
and available to everybody.
—
Reply to this email directly, view it on GitHub
https://github.com/thomasrepnik/idea-commit-prefix-plugin/issues/27#issuecomment-2160039700,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALMGMYANI5RI23UUP2FOJXDZG2UKVAVCNFSM6AAAAABIE5OR4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRQGAZTSNZQGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>

Thank you both for your suggestions. I will not be able to look at it in june. But i will give it a try in july/august. Thank you for using my plugin and feel free to make a PR 🙂 OrtecJohnB @.***> schrieb am Di., 11. Juni 2024, 09:54:

PR https://github.com/thomasrepnik/idea-commit-prefix-plugin/pull/28 is open, looking forward to hearing from you!

Thanks for a great plugin! Can you make the regex for finding the ID from the branch to be configurable? Also add possibility for prefix?
Example
I've the branch "sprint/123456-WithSomeText" and I would love to have my commit messages prefixed with: #123456:
is the prefix and : is the sufix. 123456 is found from banch name by configurable regex

If you want to give my change a try, you can clone the repo, check out my branch, build the plugin locally and install it from disk. You will need to have the Plugin DevKit plugin and follow these instructions: https://plugins.jetbrains.com/docs/intellij/publishing-plugin.html#building-distribution
Note that I have already incremented the version number to 1.4.0, so when using that version I am not sure you will get an update notification if Thomas publishes an update with that number or lower.

Thanks for the PR 🙌
Since I won't have my notebook with me for the next two weeks, I won't be able to try it out.
Will check it as soon as possible 👍


Can't authenticate to AAD

I'm following the CNTK-GPU-Python notebook. The call to utilities.create_batchai_client fails with this error:
AuthenticationError: , InvalidClientIdError: (invalid_request) AADSTS90002: Tenant  not found. This may happen if there are no active subscriptions for the tenant. Check with your subscription administrator.
I was able to fix it by editing utilities.py so the AAD URI domain was https://login.windows.net, and it worked fine.

Hi Paul, Thanks for reporting the issue. We have just pushed the fix.


Hey :)

Using dark theme on my blog, if I view on mobile, some of my code blocks, pictures escape the dark theme, and propagate half my page white, I have tried changing the body to black in the sass file but unsure what the issue is really....Thanks for you time. https://ryku-1.github.io
(cheers for the theme btw it is awesome)

Sorry for the late response. I checked your blog and the problem seem solved, isn't it?

Sorry for the late response. I checked your blog and the problem seem solved, isn't it?

Hey!! No problem, Yeah it has seemed to resolve itself, thank you!

I'm glad to hear that 😄
I'm closing the issue now. See you!


border around the applet

Requested from speedsolving.com
Please make no border around the applet (white or tranpared border are ok)

The border was just for my testing to see the size of the cube. I will remove it.

When you do it, please let me/us know.

it was just solved yesterday!

sorry forgot to upload to /scripts. It was only on github

Interestingly enough, on galaxy ace 3 the border is still visible and the algorithm area is still empty (as mentioned in another issue). I will look at it on computer on monday...
offtopic: did you know my Blonk number is 2 via several people? :-)

haha well a blonk number is usually not that big.... :-)
BTW what is your site?

Yeah but it is not your fault. The reason is well-travelled cubers like Ron or Lars Vandenberg.
hlavolam.maweb.eu - but it is only in Czech

Ah so that is your website. I see you're using AnimCube. I am targeting a lot of his features, but my main animation has always been rubikplayer.  Well done.

" I see you're using AnimCube."
For 3x3x3 (and derived puzzles like Void cube, Rubik´s domino etc.), yes. Main reasons:

same nationality = no language barrier problem (in 2004, when I started making 3x3x3 tutorial, Josef really helped me with running it on my website and I understood him (back then I didn´t understand Werner because of my bad English))
move counter available - I use it a lot to describe what is really going on on the cube
edit parameter and show/hide buttonbar available - I use those parameters a lot too

For Barrel, 2x2x2, 4x4x4 and 5x5x5 I use Werner´s applets (obvious). For Square-1, Pyraminx and Megaminx I use applets made by Lucas and Jeremy Fleischman (Lucas plans to convert it into JS).
For some puzzles (mostly denoted by upper index 1 in my home page) I use scanning of computer screen instead, leading in a video output (Boris´ applets are beautiful, for example: http://pMetro.su/pCubes.zip).

I like pcubes, very nice. I'll get to edit parameter, and move counter too. Show/hide buttonbar is easy, you can simply remove the whole html section in order to remove the buttonbar. Or just remove whatever you want from the html.

As we are out of the original topic already, I must recommend you to go ahead and look at some other applets if you like pCubes: http://twistypuzzles.com/forum/viewtopic.php?f=8&t=24876
I wonder if you can solve other puzzles than 3x3x3 and Rubik's magic as your wca profile says?

Nice page. I don't have other puzzles, but at one point I could do the 4x4, 5x5 and megaminx. And of course the 2x2 LOL. I understand commutators so many puzzles are within reach.

Aha, in that case I demand to know if you can solve the Dino cube (it should not take so much time to you). Next step could be shapeshifters - Fisher cube seems like a nice challenge for you. Last but not least, you can already try the Void cube in your GLube simulator.
I am just letting you know that I noticed that java-based applets seems to be working as well. For more information see speedsolving.com thread.

I just checked Ryan Heise's site http://www.ryanheise.com/cube/edges_first.html and it seems like he actually ported animcube completely to javascript using GWT. That's quite a feat! If you like you can ask Ryan for the source code and usage. On his site the code is obfuscated (not open source). Of course I will keep improving my own cube as well, but perhaps you can already start by using this one. Things are busy here, so development of GLube is currently slow.

I tried 2 years ago to contact him via e-mail. Without success - he is apparently not active anymore (seems like since 2008). Maybe my e-mail ended up in his spam folder, I don´t know.
I tried to somehow run it even without his help, but I didn´t succeeded (the code is very complex and unreadable for me)... So you say without his help the usage is forbidden to others? :-(
I would love to use his system as a replacement of my current system. Subsequently, I plan to replace it by your or Werner´s simulator.
Thanks for the heads-up about development. I was thinking you were either on vacation or busy beacause of examination period (as a teacher)...

Yeah we're wrapping up the school year, but I also have new website development projects running. So it's busy all over. In a week it's summer break, I'll have more time to devote.
I think Ryan Heise's code is usable, but without the source it will be difficult to figure out how to set parameters. I think it looks good, though not as good as AnimCube itself.
I am planning to do something similar with GLube where the buttons are simply generated by the script, so you don't actually have to include them. That way it's easier to plug in cubes with just an algorithm and you get the buttons automatically.

Mabye I found the source code. See a speedsolving.com thread for more if you are interested.

And first issue found ;-) https://github.com/molarmanful/AnimCubeJS/issues/1

solved Jun 11, 2015


WIP: Add Rails/LeftJoin and Rails/LeftOuterJoin cops

This PR adds two cops: Rails/LeftJoin and Rails/LeftOuterJoin which look for .joins("LEFT JOIN ...") and .joins("LEFT OUTER JOIN ...") and propose to use the .left_join("...") and .left_outer_join("...") methods introduced in Rails 5.0.
Closes #5812

Before submitting the PR make sure the following are checked:

[ ] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[ ] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[ ] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[ ] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

@bbatsov what's the policy of Rubocop about adding similar cops like in this PR?
.joins('LEFT JOIN ...) -> .left_join(...)
.joins('LEFT OUTER JOIN ...) -> .left_outer_join(...)
Should I create two cops per each method above?
Or should I just use one cop, say Rails/LeftJoin, and check both cases in it?
:slightly_smiling_face:

@Drenmi maybe you could give me a tip (see the previous comment above), please? :slightly_smiling_face:

@blackst0ne Sorry about the radio silence - I was crazy busy lately. Given the fact that the two checks are pretty similar it seems to me that one cop will be fine. I don't that someone would like to enable one check and disable the other.

What's the status here? I see the build is till failing.

@bbatsov sorry, I was busy last weeks.
I am going to finish this PR soon. :slightly_smiling_face:

@bbatsov specs are green, code is updated.
Could you review, please? :slightly_smiling_face:

This should be moved to rubocop-rails. We won't be accepting any new Rails cops in RuboCop itself anymore.

@blackst0ne Would you mind reopening this at rubocop-rails?


CreateScreenshotUsingRenderTarget crash with Babylon 4.2 in WebGL1 and not working properly with Babylon 5.0.0-alpha 11

Repeated raw calls to CreateScreenshotUsingRenderTarget in Babylon 4.2.0 (without doing anything with the data parameter on its callback) crashes in WebGL1 devices. This was tested with different iOS versions (crashes around 100 calls) and with Safari in MacOS (crashes around 600 calls in my tests).
Also went to explore existing playgrounds using this API function, and found the following two (written by another person):

https://www.babylonjs-playground.com/#HJI09Z#1
https://www.babylonjs-playground.com/#HJI09Z#2

Although these do not seem to crash on WebGL1, they add a full screen flickering that makes the whole scene useless.
It seems that now BabylonJS PlayGrounds are using Babylon 5.0.0-alpha11, and thus I replaced the engine that I was using on my own app with 5.0.0-alpha11. The result was that it does not crash anymore, but it introduced now the same flickering as can be seen on the Playgrounds mentioned above.
So it seems that there are 2 separate issues with CreateScreenshotUsingRenderTarget:

On WebGL1 devices and Babylon 4.2.0, there is a crash. I tried doing the call slower (from about 180 ms between calls to around 450 ms, which anyways is non acceptable on my use case), and still crashes. However, it works perfectly on WebGL2 devices.

On either WebGL1 or WebGL2 devices, but with Babylon 5.0.0 Alpha 11, it introduced the full screen flickering. No crashes, but not usable anyways.

why are you not even reading the issue template?
Please discuss the issue on the forum first

Sorry, I read it, but my understanding of a what a "confirmed bug is" seems to be different, so that's why I reported here (as I do with any other GitHub project): We were doing extensive testing and I know that this is not just confirmed (from my humble point of view), but that can be replicated as easily as going to one of the 2 playgrounds that I added.
I will post in the forum anyways.

thank you very much! @Popov72 will look into it

Thank YOU!
I added a forum entry, after searching that the bug was not already reported there:
https://forum.babylonjs.com/t/taking-screenshot-crashes-with-webgl1/18694


#3224 Fix extra connections populating MRU List

This is a fix for an issue where connections were being added to the MRU list that really shouldn't be  - such as opening a new query window.
The fix is to modify where we add active connections so that we can use the saveTheConnection option to decide whether to add a connection to the MRU. This was necessary because the old location was called from onConnectionComplete which is sent by the SqlToolsService - and that doesn't get any UI related information like the options. The new location is still only called after the connection completes and will be added only if the connection succeeds.
So now only calls to connect that specify saveTheConnection as true will be added to the MRU. Currently this is limited to just initial connections in the OE dialog and connections made through the connection dialog. Others can be added later if we deem that necessary.
Added new test and updated existing tests to handle new logic (plus a bit of async-refactoring).

The issue we have right now is that since we're adding every connection it's populating with a lot of "extra" connections that the user isn't specifically making. For example if they right click on a DB and select New Query then the connection to that DB will show up in the MRU. This means the MRU gets very cluttered very quickly and its usefulness is severely diminished.
There is a potential for us to add logic back in later though for exposing a user option that lets users specify which connections are added to the MRU though - then they could select just the ones they want. But initially I decided to keep this more targeted and then follow up with further discussion (since there's other MRU related things we should look into fixing, for example deciding what makes a connection "unique" in the MRU, something we don't handle very cleanly right now)

@Charles-Gagnon if this is targeting March we should try to get merged as soon as it's ready to start including in test passes.


Update dependency esbuild to v0.17.14

This PR contains the following updates:

Package
Type
Update
Change
Pending

esbuild
dependencies
patch
0.17.10 -> 0.17.14
0.17.15

Release Notes

evanw/esbuild
v0.17.14
Compare Source

Allow the TypeScript 5.0 const modifier in object type declarations (#3021)
The new TypeScript 5.0 const modifier was added to esbuild in version 0.17.5, and works with classes, functions, and arrow expressions. However, support for it wasn't added to object type declarations (e.g. interfaces) due to an oversight. This release adds support for these cases, so the following TypeScript 5.0 code can now be built with esbuild:
interface Foo { <const T>(): T }
type Bar = { new <const T>(): T }

Implement preliminary lowering for CSS nesting (#1945)
Chrome has implemented the new CSS nesting specification in version 112, which is currently in beta but will become stable very soon. So CSS nesting is now a part of the web platform!
This release of esbuild can now transform nested CSS syntax into non-nested CSS syntax for older browsers. The transformation relies on the :is() pseudo-class in many cases, so the transformation is only guaranteed to work when targeting browsers that support :is() (e.g. Chrome 88+). You'll need to set esbuild's target to the browsers you intend to support to tell esbuild to do this transformation. You will get a warning if you use CSS nesting syntax with a target which includes older browsers that don't support :is().
The lowering transformation looks like this:
/* Original input */
a.btn {
  color: #333;
  &:hover { color: #444 }
  &:active { color: #555 }
}

/* New output (with --target=chrome88) */
a.btn {
  color: #333;
}
a.btn:hover {
  color: #444;
}
a.btn:active {
  color: #555;
}

More complex cases may generate the :is() pseudo-class:
/* Original input */
div, p {
  .warning, .error {
    padding: 20px;
  }
}

/* New output (with --target=chrome88) */
:is(div, p) :is(.warning, .error) {
  padding: 20px;
}

In addition, esbuild now has a special warning message for nested style rules that start with an identifier. This isn't allowed in CSS because the syntax would be ambiguous with the existing declaration syntax. The new warning message looks like this:
▲ [WARNING] A nested style rule cannot start with "p" because it looks like the start of a declaration [css-syntax-error]

    <stdin>:1:7:
      1 │ main { p { margin: auto } }
        │        ^
        ╵        :is(p)

  To start a nested style rule with an identifier, you need to wrap the identifier in ":is(...)" to
  prevent the rule from being parsed as a declaration.

Keep in mind that the transformation in this release is a preliminary implementation. CSS has many features that interact in complex ways, and there may be some edge cases that don't work correctly yet.

Minification now removes unnecessary & CSS nesting selectors
This release introduces the following CSS minification optimizations:
/* Original input */
a {
  font-weight: bold;
  & {
    color: blue;
  }
  & :hover {
    text-decoration: underline;
  }
}

/* Old output (with --minify) */
a{font-weight:700;&{color:#00f}& :hover{text-decoration:underline}}

/* New output (with --minify) */
a{font-weight:700;:hover{text-decoration:underline}color:#00f}

Minification now removes duplicates from CSS selector lists
This release introduces the following CSS minification optimization:
/* Original input */
div, div { color: red }

/* Old output (with --minify) */
div,div{color:red}

/* New output (with --minify) */
div{color:red}

v0.17.13
Compare Source

Work around an issue with NODE_PATH and Go's WebAssembly internals (#3001)
Go's WebAssembly implementation returns EINVAL instead of ENOTDIR when using the readdir syscall on a file. This messes up esbuild's implementation of node's module resolution algorithm since encountering ENOTDIR causes esbuild to continue its search (since it's a normal condition) while other encountering other errors causes esbuild to fail with an I/O error (since it's an unexpected condition). You can encounter this issue in practice if you use node's legacy NODE_PATH feature to tell esbuild to resolve node modules in a custom directory that was not installed by npm. This release works around this problem by converting EINVAL into ENOTDIR for the readdir syscall.

Fix a minification bug with CSS @layer rules that have parsing errors (#3016)
CSS at-rules require either a {} block or a semicolon at the end. Omitting both of these causes esbuild to treat the rule as an unknown at-rule. Previous releases of esbuild had a bug that incorrectly removed unknown at-rules without any children during minification if the at-rule token matched an at-rule that esbuild can handle. Specifically cssnano can generate @layer rules with parsing errors, and empty @layer rules cannot be removed because they have side effects (@layer didn't exist when esbuild's CSS support was added, so esbuild wasn't written to handle this). This release changes esbuild to no longer discard @layer rules with parsing errors when minifying (the rule @layer c has a parsing error):
/* Original input */
@layer a {
  @layer b {
    @layer c
  }
}

/* Old output (with --minify) */
@layer a.b;

/* New output (with --minify) */
@layer a.b.c;

Unterminated strings in CSS are no longer an error
The CSS specification provides rules for handling parsing errors. One of those rules is that user agents must close strings upon reaching the end of a line (i.e., before an unescaped line feed, carriage return or form feed character), but then drop the construct (declaration or rule) in which the string was found. For example:
p {
  color: green;
  font-family: 'Courier New Times
  color: red;
  color: green;
}

...would be treated the same as:
p { color: green; color: green; }

...because the second declaration (from font-family to the semicolon after color: red) is invalid and is dropped.
Previously using this CSS with esbuild failed to build due to a syntax error, even though the code can be interpreted by a browser. With this release, the code now produces a warning instead of an error, and esbuild prints the invalid CSS such that it stays invalid in the output:
/* esbuild's new non-minified output: */
p {
  color: green;
  font-family: 'Courier New Times
  color: red;
  color: green;
}

/* esbuild's new minified output: */
p{font-family:'Courier New Times
color: red;color:green}

v0.17.12
Compare Source

Fix a crash when parsing inline TypeScript decorators (#2991)
Previously esbuild's TypeScript parser crashed when parsing TypeScript decorators if the definition of the decorator was inlined into the decorator itself:
@(function sealed(constructor: Function) {
  Object.seal(constructor);
  Object.seal(constructor.prototype);
})
class Foo {}

This crash was not noticed earlier because this edge case did not have test coverage. The crash is fixed in this release.

v0.17.11
Compare Source

Fix the alias feature to always prefer the longest match (#2963)
It's possible to configure conflicting aliases such as --alias:a=b and --alias:a/c=d, which is ambiguous for the import path a/c/x (since it could map to either b/c/x or d/x). Previously esbuild would pick the first matching alias, which would non-deterministically pick between one of the possible matches. This release fixes esbuild to always deterministically pick the longest possible match.

Minify calls to some global primitive constructors (#2962)
With this release, esbuild's minifier now replaces calls to Boolean/Number/String/BigInt with equivalent shorter code when relevant:
// Original code
console.log(
  Boolean(a ? (b | c) !== 0 : (c & d) !== 0),
  Number(e ? '1' : '2'),
  String(e ? '1' : '2'),
  BigInt(e ? 1n : 2n),
)

// Old output (with --minify)
console.log(Boolean(a?(b|c)!==0:(c&d)!==0),Number(e?"1":"2"),String(e?"1":"2"),BigInt(e?1n:2n));

// New output (with --minify)
console.log(!!(a?b|c:c&d),+(e?"1":"2"),e?"1":"2",e?1n:2n);

Adjust some feature compatibility tables for node (#2940)
This release makes the following adjustments to esbuild's internal feature compatibility tables for node, which tell esbuild which versions of node are known to support all aspects of that feature:

class-private-brand-checks: node v16.9+ => node v16.4+ (a decrease)
hashbang: node v12.0+ => node v12.5+ (an increase)
optional-chain: node v16.9+ => node v16.1+ (a decrease)
template-literal: node v4+ => node v10+ (an increase)

Each of these adjustments was identified by comparing against data from the node-compat-table package and was manually verified using old node executables downloaded from https://nodejs.org/download/release/.

Configuration
📅 Schedule: Branch creation - "before 3am on the first day of the month" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Self-Hosted Renovate Bot. Find all logs here.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.

thunk_bundle changes detected but no repro locally, can't seem to make it update


Meteo france

Description:
Move the Meteo France integration to a directory.
Follow-up of #18404
Example entry for configuration.yaml (if applicable):
meteo_france:
  - city: 'paris'
    monitored_conditions:
      - rain_chance
      - freeze_chance
      - thunder_chance
      - snow_chance

CC @victorcerutti
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[x] New dependencies are only imported inside functions that use them ([example][ex-import]).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

We can merge after the conflicts are resolved.


Post Archive

Howdy, I'm really enjoying the ghost platform. However there is a major feature that I feel is absent: a post archive. This archive would utilise the 'date published' of a post. As a result you would easily be able to filter down posts to a particular year, month etc. (from my experience, the majority of blogging platforms will simply conjoin the month and the year for filtering i.e. July 2015, August 2015)...
Here are some examples of post-date archives / filtering:

http://www.parkandcube.com/site-archives/
http://shinebythree.com/archive/2015/

p.s. for what it's worth, it would be awesome if you could make an archive page that uses it's own, stylised (more thumbnail-y I imagine) post 'loop'.
Thanks again!

Hi there @edwardessing. In my opinion, specifically date-based archives are quite an old fashioned blog concept, and more importantly something that is too wide in scope to be built as a specific feature in Ghost core. There's just too many possibilities in terms of how you categorise or filter the posts, what URLs to use, etc. I don't think there's one obvious and clear way to do it that would serve 90% of people's needs.
Instead we're working on flexible tools to allow you to extend your blog in different ways like channels and the get helper. These tools are intended to give the widest set of possibilities and make them super easy to work with, so that you can completely customise the way your blog's content is displayed.
Nonetheless, I may be wrong, but unfortunately GitHub is not a good place to get feedback and traction for an idea. Recently we launched a new wishlist for feature requests. This wishlist is accessible from inside your Ghost admin panel and reaches a much wider audience, so why not add your suggestion there?

Hmmm I'm quite surprised that you would think date-based filtering is old-fashioned – I'm currently migrating a client from Blogger over to Ghost and she's indicated that it is a blogging standard in her circles (fashion blogging [which, incidentally is a massive portion of web blogging imo] ) . Anyway the channels sounds interesting, and thank you for the reply!


Missing Brazil SouthEast region

[Enter feedback here]
It is missing the Brazil SouthEast region latency statistics.

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

ID: 311b6efb-2d93-6130-96a9-867fad56bd52
Version Independent ID: a17faabf-04c6-c3da-9a37-a86b0d00cbfd
Content: Azure network round-trip latency statistics
Content Source: articles/networking/azure-network-latency.md
Service: virtual-network
GitHub Login: @mbender-ms
Microsoft Alias: mbender

@marcusgaspar
Thanks for your feedback! We will investigate and update as appropriate.

@mbender-ms
Can you please check and add your comments on this doc update request as applicable.

@marcusgaspar
I'm going to assign this to the document author so they can take a look at it accordingly

A request has been put into the team that manages this, and it's added to my backlog.
#Please-close


[Workflow] Pre-Commit Linting and Formatting

Implemented https://github.com/georgian-io/LLM-Finetuning-Hub/issues/103

Using ruff for linting and formatting

Added instructions to add pre-commit hooks for local development

Enabled automatic linting and formatting before each local commits

Closed as using github action is a better solution (doesn't require additional dev environment setup).


We need to make it clear how a simulation is started

Every time I try to start a simulation I have trouble finding the correct button ...

Yes... ;-)

If the annotation is valid, enabled a button to the panel which runs it. Hopefully these panel buttons don't keep stacking up...
| Start | Annotate | Clone | Download |                          | Delete | Terminate |

Looks good, it didn't make sense that you have to go the the annotate page in order to run the simulation.


could not build Objective-C  module 'IGListKit'

Hi
I'm handling weird problem, i have started new project in xcode8 swift3, and in the beginning every thing works good, but after several pod installs (was need to add other pods) I'm getting the error
could not build Objective-C  module 'IGListKit'
And also error:
'IGListAdapter.h' file not found
But after delete the import IGListKit line the errors disappear and project complied
There's someone have any idea what could cause this problem?
I tried pod's version 1.0.0 & 1.1.0 xcode8 & xcode8.1 nothing help

Hey @dekelm8, bummer. I'm not totally sure what could be going wrong. Do you have an example project we could take a look at? What does your Podfile look like?

Latest CocoaPods is 1.1.1

pod file:
`platform :ios, '9.0'
target 'WallaHamal' do
use_frameworks!
pod 'Firebase/Core'
pod 'Firebase/Database'
pod 'Firebase/Messaging'
pod 'Firebase/Auth'
pod 'IGListKit'
pod 'Alamofire', '~> 4.0'
end`

Ok so after install cocoapods 1.1.1 still not working, but after return back to 1.1.0 it's work weird (But at least its working now :) )
Thanks for the support

😕 weird. I'll go ahead and close this out for now and consider it an Xcode fluke. Will definitely keep an eye on these if they keep popping up.

Hi,
I had the same problem, weird. There's someone have any idea what could cause this problem?
Attach File is my project
IGListDemoSwift.zip


Merge function to use ABAPGit in a defined namespace

Hi all,
Currently ABAPGit can be used as Z objects in a ABAP system. Would it be possible to merge/extract the ABAPGit sources in a defined namespace like /blabla/CL_ABAPGIT...., so that it can be transported to other systems by SAP transportation management tools?
Best regards,
Thomas

I think that's a good idea.
Maybe duplicate of #1178 ? There are some workarounds listed there.

Try out my tool for ABAP namespace refactoring: ABAPSpace

Also take note of
ZCL_ABAPGIT_PERSIST_MIGRATE->TABLE_CREATE and LOCK_CREATE which creates a DDIC object and enqueue object when ZABAPGIT starts the first time

think this is outside the scope of abapGit, can be solved with projects like https://github.com/SAPAssets/ABAPSpace


hide/show the legend entries line by line

Describe the Enhancement
"We were making some plots in METviewer that use a lot of lines for ensemble members .
It ends up making a very large legend which then distorts the plot. It would be ideal to only include a representative member of each set of lines in the legend (i.e. 3 legend entries instead of ~30, so maybe the option to hide/show the legend entries line by line in the "Series Formatting" pane?). As a simpler solution, it would also suffice in this particular case to simply turn the legend off. "
Corresponding issue in METplotpy:
https://github.com/dtcenter/METplotpy/issues/355
Time Estimate
Estimate the amount of work required here.
Issues should represent approximately 1 to 3 days of work.
Sub-Issues
Consider breaking the enhancement down into sub-issues.

[ ] Add a checkbox for each sub-issue here.

Relevant Deadlines
List relevant project deadlines here or state NONE.
Funding Source
Define the source of funding and account keys here or state NONE.
Define the Metadata
Assignee

[ ] Select engineer(s) or no engineer required
[ ] Select scientist(s) or no scientist required

Labels

[ ] Select component(s)
[ ] Select priority
[ ] Select requestor(s)

Projects and Milestone

[ ] Select Repository and/or Organization level Project(s) or add alert: NEED CYCLE ASSIGNMENT label
[ ] Select Milestone as the next official version or Future Versions

Define Related Issue(s)
Consider the impact to the other METplus components.

[ ] METplus, MET, METdataio, METviewer, METexpress, METcalcpy, METplotpy

Enhancement Checklist
See the METplus Workflow for details.

[ ] Complete the issue definition above, including the Time Estimate and Funding Source.
[ ] Fork this repository or create a branch of develop.
Branch name: feature_<Issue Number>_<Description>
[ ] Complete the development and test your changes.
[ ] Add/update log messages for easier debugging.
[ ] Add/update unit tests.
[ ] Add/update documentation.
[ ] Push local changes to GitHub.
[ ] Submit a pull request to merge into develop.
Pull request: feature <Issue Number> <Description>
[ ] Define the pull request metadata, as permissions allow.
Select: Reviewer(s) and Development issues
Select: Repository level development cycle Project for the next official release
Select: Milestone as the next official version
[ ] Iterate until the reviewer(s) accept and merge your changes.
[ ] Delete your fork or branch.
[ ] Close this issue.

The backwards compatibility of config files was discussed with users during RRFS Agile Framework Meeting (Nov.1, 2023).
Based on users recommendations the application should throw an error message if the required parameter is missing  in the user config files. The application should not use default values to define a missing parameter.


Add the text"Thank you for visiting Freecodecamp" to article.

Pre-Submission Checklist

[ ] Your pull request targets the staging branch of freeCodeCamp.
[ ] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[ ] You have only one commit (if not, squash them into one commit).
[ ] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[ ] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[ ] Tested changes locally.
[ ] Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #XXXXX
Description

Unsubscribe me
On Mon, Oct 1, 2018, 4:30 AM giriprasath2000<EMAIL_ADDRESS>wrote:

Pre-Submission Checklist

Your pull request targets the staging branch of freeCodeCamp.
Branch starts with either fix/, feature/, or translate/ (e.g.
fix/signin-issue)
You have only one commit (if not, squash
http://forum.freecodecamp.org/t/how-to-squash-multiple-commits-into-one-with-git/13231
them into one commit).
All new and existing tests pass the command npm test. Use git commit
--amend to amend any fixes.

Type of Change

Small bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds new functionality)
Breaking change (fix or feature that would change existing
functionality)
Add new translation (feature adding new translations)

Checklist:

Tested changes locally.
Addressed currently open issue (replace XXXXX with an issue no in
next line)

Closes #XXXXX
Description
You can view, comment on, or merge this pull request online at:
https://github.com/freeCodeCamp/freeCodeCamp/pull/18205
Commit Summary

Add the text"Thank you for visiting Freecodecamp" to article.

File Changes

M .github/ISSUE_TEMPLATE.md
https://github.com/freeCodeCamp/freeCodeCamp/pull/18205/files#diff-0
(1)

Patch Links:

https://github.com/freeCodeCamp/freeCodeCamp/pull/18205.patch
https://github.com/freeCodeCamp/freeCodeCamp/pull/18205.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/pull/18205, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AQgjcTvxn9IDXHGCREuZbDEI3y0AV_zFks5ugdK2gaJpZM4XBl5T
.


Command 'Quarto: New Notebook (ipynb)' when run as a first command after install

I really like the idea of quarto.When try to initialize quarto after installing with new ipbnv notebook  I got
Command 'Quarto: New Notebook (ipynb)' resulted in an error (command 'quarto.newNotebook' not found)

However after creating qmd document I was able to  create ipynb notebooks  normally.
Quarto 1.12.1 vscode  1.66.1

Thank you for reporting this! Should be fixed here: https://github.com/quarto-dev/quarto-vscode/commit/f268ab0f88d135312ebd4ae58c2039c185f5ceaf


Test if existing fields are implemented on page

Hey guys,
after knowing, how to use all standard objects in my test codeunit, I would like to test, if my implemented fields on the page "item card" exist and in that case, the ouput should be true. I am doing this for my extension, I have developed. For that kind of testing I would like to use the standard codeunit "Assert". Should I use the function IsTrue?
Or do you have another idea, how to test that?

This repository no longer handles general questions and is only dedicated to bug reports and feature suggestions for the AL compiler and developer tools.
To get help for your question please use one of the following resources:
Business Central Community
mibuso forum
Dynamics User Group