Errors in console and switching themes doesn't work anymore after upgrade to 1.3.1

Recently I wanted to upgrade from 1.2 to 1.3.1.

I deleted my MetroidvaniaSystem directory in my addons folder
downloaded the project zip from GitHub
extracted the MetroidvaniaSystem folder to addons.

When I now open my project:

errorrs show up in console when I open (screen 1)
same errors show up when I add a new room to the existing map everytime I do
if I switch the theme all my rooms are gone (screen 2)
on the current theme (SotN) the map looks different than before. The background of the rooms is now blue, it was black/translucent before (screen 3 & 4)
if I switch themes, click around and switch back, I get a map with mixed themes (screen 5)

Screen 1 - Errors:

Screen 2 - Switched Theme

Screen 3 - before:

Screen 4 - after:

Screen 5:

Additional Notes:

adding the sampleproject and extension directory don't change anything
the game still runs. The Map is now blue instead of black, though and I cannot switch themes
I haven't tried any of the new features like the new modern map

Were there any documented breaking changes from 1.2 to 1.3.1 that I missed?

Godot version?

Sorry I didn't notice earlier. Looks like you are using the version from GitHub. It's still experimental, get the addon from AssetLib for now.

That seemed to fix it, thanks

The only issue I have now is that whenever I start my game in the editor and then close it, it tries to tell me ther MapData.txt has changed.

Do you have any idea where this could be coming from?


customise subject email templates (otp code)

Hey everyone,
The OTP feature is working great and, I was thinking, it would be super helpful if we could customize the email template subject.
For example, it’d be amazing if the OTP code could be included right in the subject of the email. This would make it so much easier for users to spot their code at a glance without needing to open the email.

sorry, closed. posted on the wrong repo


Script updates

Includes:

fixes to CRY campaign
added extracted for CORSIKA
small fixes to scripts for reprocessing

Hi @sophiemiddleton,
You have proposed changes to files in these packages:

Scripts
CampaignConfig

which require these tests: build.
@Mu2e/fnalbuild-users, @Mu2e/write have access to CI actions on main.
:hourglass: The following tests have been triggered for ab14c3f93c00a731371dc3efbe8ccd04d6ac4646: build  (Build queue is empty)
About FNALbuild. Code review on Mu2e/Offline.

:sunny: The build tests passed at ab14c3f93c00a731371dc3efbe8ccd04d6ac4646.

Test
Result
Details

test with
:white_check_mark:
Command did not list any other PRs to include

merge
:white_check_mark:
Merged ab14c3f93c00a731371dc3efbe8ccd04d6ac4646 at 19eeac19f5f7d43b3ce1aa07d583c2a4af4f5ffd

build (prof)
:white_check_mark:
Log file. Build time: 11 min 03 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:white_check_mark:
Log file.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at ab14c3f93c00a731371dc3efbe8ccd04d6ac4646 after being merged into the base branch at 19eeac19f5f7d43b3ce1aa07d583c2a4af4f5ffd.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.

fixed the script


Add Schema Stitching

I've included lodash.merge for easy deep-merging of resolvers

Addresses https://github.com/gramps-graphql/gramps/issues/2

Have you tried this with actual data sources? I'll pull this down and play around to make sure it's doing what we expect, but at first glance this looks great.


connection: fix getpid() call on disconnect

Previous fix only handled the connections from Valkey class, which left classes like Sentinel broken because those are handled in AbstractConnection.
Also after merging previous fix I was pointed to the place in Python standard library where the similar issue was handled by having from os import getpid instead of having a local object assigned to os.getpid.
Fixes #158
Pull Request check-list

[ ] Do tests and lints pass with this change?
[ ] Do the CI tests pass with this change (enable it first in your forked repo and wait for the github action build to finish)?
[ ] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?
[ ] Is there an example added to the examples folder (if applicable)?

Description of change

Codecov Report
Attention: Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Project coverage is 59.32%. Comparing base (ef40c56) to head (a2c27a9).

Files with missing lines
Patch %
Lines

valkey/connection.py
85.71%
1 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (ef40c56) and HEAD (a2c27a9). Click for more details.
HEAD has 380 uploads less than BASE

Flag
BASE (ef40c56)
HEAD (a2c27a9)

381
1

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #166       +/-   ##
===========================================
- Coverage   76.23%   59.32%   -16.91%
===========================================
  Files         130      130
  Lines       33947    33913       -34
===========================================
- Hits        25879    20119     -5760
- Misses       8068    13794     +5726

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


THORN-2405 add MongoDB replicateSet + adminDatabase

Fix for:
https://issues.jboss.org/browse/WFNOSQL-32
https://issues.jboss.org/browse/WFNOSQL-33
Also to help with https://issues.jboss.org/browse/MWES-3171

Thanks!


The package fails to find the models?

Hello there!
I am on a M1 Mac (2020) and have python3, spacy and amrlib all installed through Homebrew.
LocalUser@LocalMac ~ % python3
Python 3.9.6 (default, Jun 28 2021, 19:24:41)
[Clang 12.0.5 (clang-1<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import amrlib
>>> stog = amrlib.load_stog_model()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/opt/homebrew/lib/python3.9/site-packages/amrlib/__init__.py", line 35, in load_stog_model
    stog_model = load_inference_model(model_dir, **kwargs)
  File "/opt/homebrew/lib/python3.9/site-packages/amrlib/models/model_factory.py", line 46, in load_inference_model
    raise FileNotFoundError(errno.ENOENT, os.strerror(errno.ENOENT), model_directory)
FileNotFoundError: [Errno 2] No such file or directory: '/opt/homebrew/lib/python3.9/site-packages/amrlib/data/model_stog'

Examining a little bit indicates decisively that the directory /opt/homebrew/lib/python3.9/site-packages/amrlib/ does not have a folder named data. Here's some sort of proof that python3 is the right one:
LocalUser@LocalMac ~ % which python3
/opt/homebrew/bin/python3

I'd appreciate your help!

Did you install the models there there or somewhere else?  If you installed them elsewhere, set a link from that location to where you installed them.  See https://amrlib.readthedocs.io/en/latest/install.html#install-the-models

RTFM works, what do ya know! I am truly sorry and thank you for your help!

Oh, yikes,  I spoke too soon! After setting a link, python fails with "no such file or directory"


Use RTK-query to get gene data for track panel

Description
Proposed as an alternative for https://github.com/Ensembl/ensembl-client/pull/552

Uses redux-toolkit query rather than Apollo client to fetch gene data from Thoas
The result of the query is currently used to display the tracks for the gene and its transcript; but there are other components on the genome browser page that can use this query (specifically, the top bar with a brief gene summary).

Questions

Should the query for the gene fetch the extra information that will appear in the drawer, or will we send a dedicated request when the user opens the drawer
What is the best name for the typescript type that describes the gene data that has been fetched? I've called it TrackPanelGene, but I don't particularly like this name, because the fetched data will be used in several components, not just in the track panel.

Future steps (in subsequent PRs)

Remove the bio_type and the  strand fields from following the EnsObjectGene type
Review and simplify EnsObjectTrack type
Figure out how to properly pass information about transcript from the track panel to the TranscriptSummary component in the drawer (we are currently losing this information)
Possibly rework the props of the TrackPanelListItem component to move from biological field names defined in the track prop to presentational field names, so that this component can support different track types
Remove data fetching via Apollo client and move to RTK-query

Deployment URL
http://track-panel-alternative.review.ensembl.org/
Views affected
Genome browser

@azangru I have a doubt after reviewing this PR. Aren't we going to rename EnsObject and/or move it to Genome Browser directory?

Aren't we going to rename EnsObject and/or move it to Genome Browser directory?

Not in this PR! I'd suggest moving towards the goal in multiple small steps (some of which are outlined in the Future steps section in the description for this PR).


New rule to flag links in text block missing inline

Relates to:

https://github.com/primer/eslint-plugin-primer-react/issues/182
https://github.com/github/accessibility/issues/6434

Context
This PR introduces a new lint rule, a11y-link-in-text-block. This rule will raise an error when a <Link> has been detected inside of a text block without having the inline attribute set. This ensures that color isn't the only means to communicate presence of a lInk - Understanding Success Criterion 1.4.1: Use of Color | WAI | W3C is respected. In dotcom, this ties into the link underline accessibility setting.
The logic essentially treats any Link that appears with a string literal on either side of it, as a link within a text block.
For example, the following are flagged:
<p>
  Learn more about <Link href={...}>GitHub</Link>.
</p>

<SomeComponent>
  <Link href={...}>GitHub</Link> is where you share code.
</SomeComponent>

There are a number of edge cases which this lint rule will avoid flagging in order to minimize false positives!
The edge cases that the linter skips will include:

<Link sx={{fontWeight:...}}> or <Link sx={{fontFamily:...}}> because these technically may provide sufficient distinguishing styling.
<Link> where the only adjacent text is a period, since that can't really be considered a text block.
<Link> where the children is a JSX component, rather than a string literal, because then it might be an icon link rather than a text link.
<Link> that are nested inside of headings as these have often been breadcrumbs, and are in weird territory.

Testing method
I've tested this branch against dotcom to make sure there's nothing unexpected! This flagged around 140 Link missing inline in dotcom. I've gone ahead and opened fixes for whatever was flagged by this rule across 8 PRs.
I broke this up into reviewable chunks so that codeowners can make sure to minimize false positives raised by this rule. (Related: https://github.com/github/accessibility/issues/6434)
Reviewers
I usually aim to minimize false positives and prefer being a bit more conservative with lint rules to avoid confusion.
I am open to any feedback on this logic! Is there anything I'm missing?
Should this rule be default to off and enabled in dotcom only?
Right now this rule will flag <Link underline> within a text block. I'm assuming that is the intention given underline is deprecated but let me know otherwise.
Note
While this lint rule will not catch everything, it should catch a decent number of links in text blocks!

I'm still in the process of getting feedback to make sure there's nothing unexpected with the instances flagged by this rule via PRs in dotcom but want to open this up for early review!

Makes sense to me on a theoretical level, hard to review without testing it. Can we run this once on gh/gh to see if it catches any false positives?

All PRs in dotcom have been reviewed and merged without any issues!
This is ready for a formal review! :D


how to write a schema to find a non-exist fields

e.g.
// schema.json
{
...
"type": [
"object"
],
"properties": {
"table": {
"type": "string"
},
"database": {
"type": "string"
}
}
}
// data.json
{
"abc" : "123"  <-- this field is not exist in schema. i want to find this error
}
somebody know how to write the schema?
Thanks

"additionalProperties" : false is your friend. You can find a bit more information here.


Added overload for setOnSelectedTab to not fire item selection

PR to resolve the #584

Coverage increased (+0.1%) to 55.973% when pulling 416679b7039fa9333373fc912d36404e2064ecc5 on leonardo2204:master into<PHONE_NUMBER>3355880215512e77f55a69884dc371 on roughike:master.


Proposal: Type-side instanceof keyword and functionality

🔍 Search Terms
type side instanceof keyword
mapping constructors
related issues:
#31311 #42534  #50714 #55032
#202 (seems indirectly related, but has been linked from #31311 #42534 #50714 #55032)
✅ Viability Checklist

[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, new syntax sugar for JS, etc.)
[X] This isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types
[X] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals

⭐ Suggestion
A new type-side keyword instanceof is proposed to allow an interface to be declared as mapping to a specific class or specific constructor variable.  When type-checking for assignability of said interface, the type checker would not rely on structural duck-typing, but instead rely on the tracked class/variable from which the object was constructed.
This proposal addresses runtime errors resulting from runtime use of run-side instanceof that are not caught by the type checker. It may also be useful for other reasons/purposes, as shown in "Motivating Examples and discussed in "Use Cases".
Of course absence of this feature is not a bug, as the overwhelming majority of TypeScript use cases require not using this proposed feature.  However, there are a minority of use cases where this feature would be essential and/or useful, and the absence of this feature is a limitation.
📃 Motivating Examples
Example 1.1:
Library side using type-side instanceof:
class FooPrivatish {
    x: number;
    constructor(x:number){
        this.x = x;
    }
    valueOf(){
        return this.x;
    }
}

// Notice `instanceof FooPrivatish` V V V V V V V V V V V
export interface FooInstanceofable instanceof FooPrivatish {
    valueOf(): number;
}
export interface FooInstanceofableConstructor {
    new(x: number): FooInstanceofable;
}
export var FooInstanceofable: FooInstanceofableConstructor = FooPrivatish;
export function processFooInstanceofable(foo: FooInstanceofable): number {
    if (!(foo instanceof FooPrivatish)) throw new Error('foo is not instanceof class FooPrivatish');
    return foo.valueOf()
}

Library side with existing behavior, doesn't use type-side instanceof:
class FooLikePrivatish {
    x: number;
    constructor(x:number){
        this.x = x+1;
    }
    valueOf(){
        return this.x;
    }
}
interface FooLike {
    valueOf(): number;
}
export interface FooLikeConstructor {
    new(x: number): FooLike;
}
export const FooLike: FooLikeConstructor = FooLikePrivatish;

function processFooLike(foo: FooLike): number {
    return foo.valueOf()
}

User side, shows difference between existing and proposed behavior, and how they coexist:
import {....} from '...';
import {....} from '...';

const fooInstanceofable = new FooInstanceofable(0);
const fooLike = new FooLike(0);

processFooInstanceofable(fooInstanceofable); // no ts error
processFooInstanceofable(fooLike); // ts error - fooLike is not an instanceof FooInstanceofable

processFooLike(fooInstanceofable); // no ts error (existing behavior)
processFooLike(fooLike); // no ts error

fooLike satisifies FooInstanceofable; // ts error
fooInstanceofable satisfies fooLike ; // no ts error (existing behavior)

const foo = Math.random() > 0.5 ? fooInstanceofable : fooLike;
foo satisifies FooInstanceofable; // ts error
if (foo instanceof FooInstanceofable) {
    foo satisifies FooInstanceofable; // no ts error
}

Example 1.2:
Unlike Example 1.1, this example shows intanceof being used to map to a constructor variable rather than an actual class.
Library side:
interface Boolean {
    /** Returns the primitive value of the specified object. */
    valueOf(): boolean;
}

interface BooleanConstructor {
    new (value?: any): Boolean;
    <T>(value?: T): boolean;
    readonly prototype: Boolean;
}

declare const Boolean: BooleanConstructor;  // No corresponding exported class.

// In `instanceof Boolean` the `Boolean` refers to the variable `Boolean` not the type `Boolean`
//                                 V  V  V  V  V  V  V
interface BooleanInstanceofable instanceof Boolean {
    valueOf(): boolean;
}

User or other library side:
interface Array {
  //...
  filter<T>(x:T, f: BooleanInstanceofable, ...): Exclude<T, 0|-0|typeof NaN|null|undefined|false|"">[];
  filter<T>(x:T, f: (x:T, f: ...)=>any, ...): T[];
  //...
}

Note that if the argument type of f were just Boolean, then anything that has a valueOf method would pass the type check, even if it were a different class with different behavior, and the narrowed output type would therefore be wrong.
💻 Use Cases

#42534 - in which a run time error is not caught by the type checker. That is working as expected but this proposal would allow a way for the user to make the type checker catch the error. (Also see comment).  Similar to #31311.

The workaround shown under #31311 requires a structural type check that in the general case has unknown complexity.  In contrast type-side instanceof type check is of known low fixed time complexity.

For an empty interface (representing a hidden class) which currently has no type checking at all, this proposal would allow the user to add type checking.

Example 1.2 above:  That can't be done presently because anything Boolean-like would pass the assignment test but it could be a different class with different behavior.

Implementation
Would require an internal mapping from an interface to its corresponding class or constructor variable.  In the example above, this would be a mapping from FooInstanceofable to FooPrivatish.
It may be that multiple interfaces declare instanceof, on the same target class or constructor variable, so there shold also be a map from targets to the set of interfaces that map to those targets.  That would allow instanceof equivalence of between interfaces that map to the same target.
Classes themselves are not affected by this proposal, e.g. the class FooPrivatish would not have any special behavior.  That allows for back compatibility.

Related Alternative or Auxilliary proposal: allow instanceof to qualify class and constructor variables
The main proposal showed how new interfaces can be declared to have a type-side instanceof relationship with a class.
This auxilliary proposal allows instanceof to be placed prior to class or constructor variable, acting a qualifier.
(Note: I now think this proposal might be better than the main proposal in terms of user ease and simplicity - while solving the same problems).
Example 2.1:
From the ts library
interface ArrayBufferConstructor {
    readonly prototype: ArrayBuffer;
    new (byteLength: number): ArrayBuffer;
    isView(arg: any): arg is ArrayBufferView;
}
declare const ArrayBuffer: ArrayBufferConstructor;

// This `ArrayBuffer`    V  V  refers to the declared variable `ArrayBuffer`, not a type
const arrayBuffer: instanceof ArrayBuffer = new Uint8Array() // Type-side `instanceof` would make this become a ts error.
const dataView = new DataView(arrayBuffer); // currently run time error because prior line (without `instanceof`) is not a ts error

See comment.
Example 2.2:
class Bar { x: number; constructor(x:number){ this.x=x; } }
class Baz { x: number; constructor(x:number){ this.x=x; } }

declare function acceptsOnlyBar(bar: instanceof Bar): void {
    if (!(bar instanceof Bar)) throw new Error('bar is not instanceof class Bar');
}
acceptsOnlyBar(new Bar(0)); // no ts error
acceptsOnlyBar(new Baz(0)); // ts error
acceptsOnlyBar({x:1}); // ts error

Example 2.3:
Compare to example 1.2:
interface Array {
  //...
  //                                 V V V  This `Boolean` is the declared variable, not the type.
  filter<T>(x:T, f: instanceof Boolean, ...): Exclude<T, 0|-0|typeof NaN|null|undefined|false|"">[];
  filter<T>(x:T, f: (x:T, f: ...)=>any, ...): T[];
  //...
}

This auxilliary proposal allows instanceof to be placed prior to class or constructor variable, acting a qualifier.
(Note: I now think this proposal might be better than the main proposal in terms of user ease and simplicity - while solving the same problems).

Agreed. I don't think restricting instanceof to only be useable with interfaces would be desirable or necessary. However, there are some interesting cases, particularly when considering using extends:
For example, using this setup:
class A { ... }
class B extends A { ... }
class C { ... }

// Basic examples of `instanceof ...` being used where you would use other types
const a1: instanceof A = new A()
const a2: typeof a1 = a1
type InstanceOfA = instanceof A
interface AlsoInstanceOfA extends InstanceOfA {
}
const a3: AlsoInstanceOfA = new A()

These accurately represent the runtime, so they probably should be allowed:
const b1: instanceof A = new B()
const b2: (instanceof A) & B = new B()
const b3: (typeof a1) & B = new B()
const b4: (instanceof A) & (instanceof B) = new B()
type AB = (instanceof A) & (instanceof B) // could be simplified under-the-hood to `instanceof B` because B extends A

And these are not possible, so they should be never:
type BC = (instanceof B) & (instanceof C)
type IndirectAC = AlsoInstanceOfA & (instanceof C)

Also, presumably mapped types reduce an instanceof type to a plain structural type. E.g.
type MappedA = { [K in keyof InstanceOfA]: InstanceOfA[K] }
const a1: MappedA = new A()
const a2: InstanceOfA = a1 // Error: MappedA matches the structure of InstanceOfA but may not be an instance of A

Side Note: Boolean
@craigphicks  I don't think your examples 1.2 and 2.3 do what you intend. My understanding is that would match filter(new Boolean()) (since new Boolean() instanceof Boolean === true) when I believe you are intending to match filter(Boolean) (but Boolean instanceof Boolean === false).

typeof instanceof A is not valid since instanceof A is a type, and typeof can only be used on a value. Assuming you meant something like:
const a: instanceof A
type X = typeof a

Then X would be instanceof A as a has a type of instanceof A

@craigphicks I think I understand your point: TypeScript currently only has structural types, so right now, typeof a can equally be interpreted as "the structural type of a" and "the entire type of a". So if this proposal is enacted, there has to be a decision on which of those two interpretations is correct. With that being said, my response is basically:

I think it would break a lot of things if typeof only retrieved the structural type information (see another example below)
I can't think of anything that wouldn't work as expected if typeof included all of the type information
I don't think it is confusing to developers if the official position was: typeof x is just the type of x, whatever that type may be (structural or non-structural)

Another edge case if typeof didn't support non-structural types:
function f<T>(foo: T): void {
  let temp: typeof foo = foo
  // Today this works. Would this stop working because `T` might be an `instanceof` type, like in the call below?
  foo = temp
}

f<instanceof A>(new A())

I would really appreciate this feature.
Using JS's instanceof is part of my workflow quite often. But when instanceof's are inside of functions, it's difficult or impossible to make an appropriate return type. Here's a very rudimentary version of the issue:
class Foo {
  baz: string
}

class Bar {
  baz: string
}

const something : Foo | Bar = new Bar()

// TypeScript clearly has the ability built-in to differentiate Foo from Bar under the hood when using instanceof
if (something instanceof Bar) {
  something // TypeScript correctly narrows to Bar
} else {
  something // Typescript correctly narrows to Foo
}

const errorIfBar<T>(obj : T) : ??? { // here something along the lines of `T excluding instanceof Bar` would suffice perfectly
  if (obj instanceof Bar) throw new Error("got a Bar!")
  return obj
}

Whatever TypeScript is doing under the hood from the if ( ... instanceof ... ) { .. } in terms of type narrowing, it would be very handy to be able to use that.
In the above example, when you want to explicitly specify a return type of a function that uses instanceof, structural typing doesn't work because, for example, Exclude<T, Bar> would exclude both because both Foo and Bar are the same shape.

Here's another use case I bumped into. I wanted to make an immutable map between class constructors and instances, with type-safe accessors:
class Foo {}
class Bar {}
class Baz {}

interface AnyClassInstance {
  constructor: Function;
};

class Collection<T extends AnyClassInstance[]> {
    private collection: Map<Function, any>;

    constructor(collection: T) {
        this.collection = new Map();
        for (const c of collection) {
            this.collection.set(c.constructor, c);
        }
    }

    get<U extends T[number]>(t: new (...args: any) => U) {
        return this.collection.get(t) as U;
    }
}

const collection: Collection<[Foo, Bar]> = new Collection([new Foo(), new Bar()]);

const foo = collection.get(Foo);
const bar = collection.get(Bar);
const baz = collection.get(Baz); // this should not be allowed, but it passes the typechecker!

Because of how aggressively structurally-typed TypeScript is, Baz is considered a subset of [Foo, Bar] here. There's no way to enforce that the U must be one of the class instances in T.


Failing test: Disk File Service move - directory - across providers (ubuffered - buffered)

I've skipped this test on Linux, as it has failed a build: https://github.com/microsoft/vscode/commit/074e1bb82ffd3ffa54fceff75786ad9ab94107db
https://dev.azure.com/monacotools/Monaco/_build/results?buildId=53792

Added retry support for file based tests.


0.60

New Platforms

Add iAlarm support (@RyuzakiKK - #10878) (alarm_control_panel.ialarm docs) (new-platform)
Add Alpha Vantage sensor (@fabaff - #10873) (sensor.alpha_vantage docs) (new-platform)
Add ADS component (@stlehmann - #10142) (ads docs) (binary_sensor.ads docs) (light.ads docs) (sensor.ads docs) (switch.ads docs) (new-platform)
Gearbest sensor (@HerrHofrat - #10556) (sensor.gearbest docs) (new-platform)
Add Ziggo Mediabox XL media_player (@b10m - #10514) (media_player.ziggo_mediabox_xl docs) (new-platform)
Meraki AP Device tracker (@masarliev - #10971) (device_tracker.meraki docs) (new-platform)
Added Vera scenes (@alanfischer - #10424) (vera docs) (scene.vera docs) (new-platform)
Add support for Canary component and platforms (@snjoetw - #10306) (canary docs) (camera.canary docs) (sensor.canary docs) (new-platform)
Add support for Logitech UE Smart Radios. (@GreenTurtwig - #10077) (media_player.ue_smart_radio docs) (new-platform)
Added support for cover in tellstick (@perfalk - #10858) (tellstick docs) (cover.tellstick docs) (new-platform)
Add a caldav calendar component (@maxlaverse - #10842) (calendar.caldav docs) (new-platform)
Refactor hue to split bridge support from light platform (@andreacampi - #10691) (hue docs) (light.hue docs) (breaking change) (new-platform)

Breaking Changes

Update snips to listen on new mqtt topic and utilize rawValue (@tschmidty69 - #11020) (snips docs) (breaking change)
Refactor hue to split bridge support from light platform (@andreacampi - #10691) (hue docs) (light.hue docs) (breaking change) (new-platform)
Use luftdaten module (@fabaff - #10970) (sensor.luftdaten docs) (breaking change)
ISY994 sensor improvements (@OverloadUT - #10805) (isy994 docs) (binary_sensor.isy994 docs) (breaking change)

All changes

Update ecobee version to fix stack-trace issue (@PhracturedBlue - #10894) (ecobee docs)
Pybotvac multi (@raymccarthy - #10843) (neato docs)
More declarative timeout syntax for manual alarm control panel. (@bonzini - #10738) (alarm_control_panel.manual docs) (alarm_control_panel.manual_mqtt docs)
Unpacking RESTful sensor JSON results into attributes. (@nickovs - #10753) (sensor.rest docs)
Bump dev to 0.60.0.dev0 (@fabaff - #10912)
Update eliqonline.py (@molobrakos - #10914) (sensor.eliqonline docs)
Add iAlarm support (@RyuzakiKK - #10878) (alarm_control_panel.ialarm docs) (new-platform)
Correction of Samsung Power OFF behaviour (@RiRomain - #10907) (media_player.samsungtv docs)
Add Alpha Vantage sensor (@fabaff - #10873) (sensor.alpha_vantage docs) (new-platform)
Don't repeat getting receiver name on each update / pushed to denonavr 0.5.5 (@scarface-4711 - #10915) (media_player.denonavr docs)
Add Min and Event Count Metrics To Prometheus (@bah2830 - #10530) (prometheus docs)
Update coveragerc (@balloob - #10931)
Report availability of TP-Link smart sockets (@DanNixon - #10933) (switch.tplink docs)
Set percent unit for battery level so that history displays properly; edited variable name for consistency (@nbougues - #10932) (sensor.tesla docs)
Export climate status and target temperature to Prometheus (@hudashot - #10919) (prometheus docs)
Tado ignore invalid devices (@dasos - #10927) (climate.tado docs)
don't ignore voltage data if sensor data changed (@PaulAnnekov - #10925) (xiaomi_aqara docs)
Add ADS component (@stlehmann - #10142) (ads docs) (binary_sensor.ads docs) (light.ads docs) (sensor.ads docs) (switch.ads docs) (new-platform)
Gearbest sensor (@HerrHofrat - #10556) (sensor.gearbest docs) (new-platform)
Add Ziggo Mediabox XL media_player (@b10m - #10514) (media_player.ziggo_mediabox_xl docs) (new-platform)
Generic thermostat initial_operation_mode (@ziotibia81 - #10690) (climate.generic_thermostat docs)
Use new build path for dev translations (@armills - #10937)
Add option to set default hide if away for new devices (@masarliev - #10762) (device_tracker docs)
Allow disabling the LEDs on TP-Link smart plugs (@DanNixon - #10980) (switch.tplink docs)
Meraki AP Device tracker (@masarliev - #10971) (device_tracker.meraki docs) (new-platform)
Update tradfri.py (@pvizeli - #10991) (light.tradfri docs)
webostv: Ensure source exists before use (@OddBloke - #10959) (media_player.webostv docs)
Ensure Docker script files uses LF line endings to support Docker for Windows. (@R1chardTM - #10067)
Added Vera scenes (@alanfischer - #10424) (vera docs) (scene.vera docs) (new-platform)
Fix Egardia alarm status shown as unknown after restart (@jeroenterheerdt - #11010) (alarm_control_panel.egardia docs)
Handle OSError when forcibly turning off media_player.samsungtv (@lwis - #10997) (media_player.samsungtv docs)
Shuffle support in Sonos (@mar-schmidt - #10875) (media_player.sonos docs)
Add support for Canary component and platforms (@snjoetw - #10306) (canary docs) (camera.canary docs) (sensor.canary docs) (new-platform)
Ignore Sonos players with unknown hostnames (@amelchio - #11013)
Support LIFX Mini products (@amelchio - #10996) (light.lifx docs)
Update snips to listen on new mqtt topic and utilize rawValue (@tschmidty69 - #11020) (snips docs) (breaking change)
Change default js version to auto (@andrey-git - #10999)
Serialize mochad requests (@mtreinish - #11029) (mochad docs) (light.mochad docs) (switch.mochad docs)
In dev mode expose only relevant sources (@andrey-git - #11026)
Added force_update for REST sensor (@foxel - #11016) (sensor.miflora docs) (sensor.mqtt docs) (sensor.rest docs)
Add support for Logitech UE Smart Radios. (@GreenTurtwig - #10077) (media_player.ue_smart_radio docs) (new-platform)
Make notify.html5 depend on config (@balloob - #11052) (notify.html5 docs)
This change fixes the error OSError: [WinError 193] on Windows debuggers (i.e. PyCharm) (@tringler - #11034)
Added support for cover in tellstick (@perfalk - #10858) (tellstick docs) (cover.tellstick docs) (new-platform)
Add a caldav calendar component (@maxlaverse - #10842) (calendar.caldav docs) (new-platform)
Add GPS coords to meraki (@addelovein - #10998) (device_tracker.meraki docs)
Refactor hue to split bridge support from light platform (@andreacampi - #10691) (hue docs) (light.hue docs) (breaking change) (new-platform)
add custom bypass status to total connect (@uchagani - #11042) (alarm_control_panel.totalconnect docs)
Volvo on call: Optional use of Scandinavian miles. Also add average fuel consumption property (@molobrakos - #11051) (volvooncall docs) (sensor.volvooncall docs)
Allow tradfri to read the available state of the device (@pvizeli - #11056) (light.tradfri docs)
Upgrade aiohttp to 2.3.6 (@fabaff - #11079)
Upgrade yarl to 0.16.0 (@fabaff - #11078)
Upgrade psutil to 5.4.2 (@fabaff - #11083) (sensor.systemmonitor docs)
Upgrade youtube_dl to 2017.12.10 (@fabaff - #11080) (media_extractor docs)
Upgrade shodan to 1.7.7 (@fabaff - #11084) (sensor.shodan docs)
Update tellcore-net to 0.4 (@pvizeli - #11087) (tellstick docs)
Bump pymusiccast to version 0.1.6 (@jalmeroth - #11091) (media_player.yamaha_musiccast docs)
Use luftdaten module (@fabaff - #10970) (sensor.luftdaten docs) (breaking change)
Bump pyatv to 0.3.9 (@postlund - #11104) (apple_tv docs)
Report availability for TP-Link smart bulbs (@DanNixon - #10976) (light.tplink docs)
Fix incorrect comment. (@emosenkis - #11111)
Update Warrant (@balloob - #11101) (cloud docs)
Fixed typo in automation.py (@ryanm101 - #11116) (config.automation docs)
Add media position properties (@pschmitt - #10076) (media_player.liveboxplaytv docs)
update pyripple (@nkgilley - #11122) (sensor.ripple docs)
Skip HASS emulated Hue bridges from detection. (@andreacampi - #11128) (hue docs)
Always consume the no_throttle keyword argument. (@andreacampi - #11126)
Add problem device class (@armills - #11130) (binary_sensor docs)
set default utc offset to 0 (@pollett - #11114) (sensor.efergy docs)
Allow using more than one keyboard remote (@BryanJacobs - #11061) (keyboard_remote docs)
ISY994 sensor improvements (@OverloadUT - #10805) (isy994 docs) (binary_sensor.isy994 docs) (breaking change)
Disable html5 notify dependency (@balloob - #11135) (notify.html5 docs)
Upgrade Homematic (@danielperna84 - #11149) (homematic docs)

Any chance PR #10675 can get merged? Thank you for considering it!


Bug when you decline backend choice

when you press "n" to decline backend, the sollievo continue a creation process instead of kill the process

bug fixed!


Editor - Support Github two factor authentication

What is the current behavior?
I have two factor authentication in github. Gitbook Editor native client failed to sync.
What is the expected behavior?
Https should support two factor auth in github.
Either support ssh or support two factor in https

If two factors auth is enabled for your GitHub account, you should use GitHub personal access token as the password.
Go to GitHub settings and create a new token for GitBook Editor. If your book repo is public, only selecting public_repo scope is enough. Then use your GitHub account name and this token as the password when syncing changes.

Can i suggest that the fix should be to have the Gitbook Editor UI remind users about Github's Personal Access Tokens?  Perhaps also a link to github's help page to assist them in creating it.
And another prompt if/when their login fails, as this is likely to be a common failure case.


[Bug] OAuth2 Plugin Validation Client Credential Flow

Summary
Kong does not always do proper validation on client credential flow for OAuth2.0 in specific scenario, specifically the empty creds passed:
POST https://gateway-dev.company.com/auth/oauth2/token HTTP/1.1
Accept-Encoding: gzip,deflate
Content-Type: application/x-www-form-urlencoded
Content-Length: 55
Host: gateway-dev.company.com
Connection: Keep-Alive
User-Agent: Apache-HttpClient/4.1.1 (java 1.5)

grant_type=client_credentials&client_id=&client_secret=

Response:
HTTP/1.1 500 Internal Server Error
Date: Mon, 17 Jun 2019 20:50:52 GMT
Content-Type: application/json; charset=utf-8
Connection: keep-alive
Content-Length: 42

{"message":"An unexpected error occurred"}

Expected Response(Or some variety of HTTP 400 bad request with a body):
HTTP/1.1 400 Bad Request
Date: Mon, 17 Jun 2019 20:50:52 GMT
Content-Type: application/json; charset=utf-8
Connection: keep-alive
Content-Length: 42

{"error_description":"Invalid client authentication","error":"invalid_client"}

Throws this in Kong logs too on error:
2019/06/17 20:50:52 [error] 37#0: *67100 [kong] access.lua:49 [oauth2] failed to get from node cache: [cassandra] length must be at least 1, client: 10.xx.xx.xxx, server: kong, request: "POST /auth/oauth2/token HTTP/1.1", host: "gateway-dev.company.com"

Additional Details & Logs

Kong version 1.2 (Not specific to this version, this bug has been around awhile I think)

@jeremyjpj0916 thank you for reporting, I'll try to do bunch of these OAuth2 card soon.

Closing this as it was merged. Will be picked up for 1.3.0 it seems.


chore: pinned Docker image to alpine-3.20

python:3.11-alpine used to be based on alpine:3.19 but the last week switched to alpine:3.21
alpine:3.19 depends on gcc 13.2.1, whereas alpine:3.21 depends on gcc 14.2.0
shapely package fails to compile under gcc 14.*
the bug is known and is expected to be delivered in shapely=2.1*
until then we need to go back to the apline with gcc version that is able to compile sharely without errors - alpine:3.20

/deploy-review


⚠ Bofrost.es has degraded performance

In 64456b2, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1418 ms

Resolved: Bofrost.es performance has improved in 88856f6 after 6 minutes.


Replace RG-Adguard.net with https://uupdump.ml/

The website RG-adguard.net looks generally fishy to new visitors, and it isn't clear where the binaries are downloaded from (e.g. piracy). I actually had a real argument with someone on the Raspberry Pi Forum who took one look at the RG-adguard.net site and thought we were doing piracy.
https://uupdump.ml/ is a similar website, but is easier to use, clear on where the files are downloaded from, no ads, and overall safer-looking. And not from Russia.

I prefer current one, your arguments seem to be pointless, but José will decide.

I actually @CornyjK had a real argument with somebody on the RPF Forum who concluded that the current rg-adguard.net looked a lot like piracy. I could hardly convince him otherwise.

looks fishy

So help the admins with a better template for the site. Out of irony, tb.rg-adguard.net/public.php gets the .iso files directly from MS Techbench Servers and even Windows Insider MVPs at answers.microsoft.com suggest and use the adguard site.
If you dare starting a download there, you'll see that the .iso comes from software-download.microsoft.com/some-text-here.
Adguard is an interface to MS downlaod servers.

It looks fishy

So help the admins with a better template for the site.
Out of irony, tb.rg-adguard.net/public.php gets the .iso files directly from MS Techbench Servers and even Windows Insider MVPs at answers.microsoft.com suggest and use the adguard site.
If you dare starting a download there, you'll see that the .iso file comes from software-download.microsoft.com/
Adguard is an interface to MS download servers.

Yeah and download it with a "downloader" exe file from uupdump- Yeah really not fishy at all.

@Atavic If you look at UUP Dump, it does the same.
@DrFrankReade Actually, RG-Adguard says to use a CMD file you downloaded. UUP Dump uses Aria2 for downloading, which is an open-source project on GitHub.

We can use both. Better to have more than 1 option.

@SuperJMN
That is master domain of https://uup.rg-adguard.net/ subdomain. IMHO people that are against Russian stuff are just stupid, what is actually bad on Russian stuff? Is it that Americans say it is bad (Kaspersky, etc.)? https://uup.rg-adguard.net/ served us for a long time and we are glad for that.

@SuperJMN TechBench by WZT has a language option on top right.


Question about the training cost

Thanks for your awesome work! Could you please describe the training costs (the type of GPU and the training time) of the three stages, respectively? Thanks in advance :)

Hi,
We train our model on 8 V100 GPUs (32 GB memory). Training clip captioning takes ~30 hours, segment description ~5 hours, and video summary ~5 hours.

Got it! Thanks for your reply :)


Move all contributing docs to this repo

Instructions for contributing to the documentation and docs engine are distributed between the "Contributing" section of the Teleport docs site and the README of this repo. This change makes this repo the single source of truth for docs contribution guidance.
Moving all docs contribution guidance into this repo has the following advantages:

It's easier to keep this information up to date.
We can free up a navigation sidebar section on the main docs site while still keeping this information discoverable.

Summary of changes:

Move all pages from the /docs/contributing/documentation section of the docs site into the docs-contributors directory of this repo. This required changing UI components in these pages to be compatible with GitHub Markdown.
Organize yarn information into a command reference and move the reference into docs-contributors.
Remove README info that overlaps with the UI reference.
Move docs site configuration information from the README into a separeate guide.
Remove instructions for running a dev server with Docker. These have often been a source of confusion for readers, and require the same initial setup as using yarn.

Reviewer note: once this is merged, I'll remove the "Contributing" section from the Teleport docs site.


[torch2.4] Fix sharded checkpointing backward compatibility issue

torch 2.4 breaks the sharded checkpointing backward compatibility. It changed how the save_planner and load_planner flatten the state dict keys. Torch issue: https://github.com/pytorch/pytorch/issues/133923 . So the new load planner can't load the checkpointing saved with old save_planner.
This PR monkey patches the load_planner if it failed to load an old checkpoint, then it removes the patch after loading the checkpointing.
test
daily test: https://github.com/mosaicml/composer/actions/runs/10480275642

LGTM! A few minor style notes before approval.
Can you also please list here: https://docs.google.com/document/d/1m-qWWN3mMTmQOePx0ip5dge1I71_RKXa5QmJm2eFwh4/edit#heading=h.aoxj18348987 under monkeypatches and link github issue?

added to the doc


Update d3-selection and d3-transition to modular implementation

d3 moved to a modular design, so it would be great if d3-transition could be updated to the 2.x line and the same with d3-selection

fixed with https://github.com/upsetjs/venn.js/releases/tag/v1.4.0


Accept strings containing nulls?

From what I've seen from testing on my own, the ScintillaNET control does not accept null characters in strings.
If I set the Text property to a value like "abc\0def", the control does not display anything beyond "abc".  If I open a file containing null bytes in Notepad++ and copy and paste to the ScintillaNET control, the NUL characters are replaced by spaces (0x20).
Is there a way to get ScintillaNET to work with null bytes in the content?

The native Scintilla control supports NUL characters and I thought we did also. What is probably happening is that the Text property internally uses one of the Scintilla APIs that sees NUL as the end of the string. Try another API such as AppendText or InsertText and let me know what you find.

AppendText does display the entire string, with the 0 byte displayed as NUL.
InsertText does not.  Also, copying and pasting a text block containing a null also does not preserve the null.  It is translated to a space.
Thanks for your help.

For future reference for anyone else (and myself), this boils down to how the Scintilla API is set up. In some instances it will accept C-style null-terminated strings. In others APIs, strings are length specified.
Example null-terminated APIs:

SCI_SETTEXT - which cooresponds to Text setter
SCI_INSERTTEXT - which cooresponds to InsertText

Example length specified APIs:

SCI_APPENDTEXT - which cooresponds to AppendText

There are others if anyone else wants to document them.
For loading large strings, the best (although most complicated) way is still probably Background Loading which has the benefit of being accessible from a background thread and supports null characters because it is length-based.


M-X sly hits Error running timer: (void-function set-up-sly-hippie-expand)

As the title suggests, set-up-sly-hippie-expand seems to be undefined.

Thanks — fixed now, I believe.


Determine set of initial labels

We need to set up some labels for this repo, to be canonicalized in settings.yml.  Here's my suggestion for some:

wg-whatever: each WG should have its own label
new-wg: proposals for not-yet-approved WGs
infra: issues/PRs related to github, chat, ML management
needs help: issues/PRs that are stuck and need new people
good first issue: the usual meaning
meeting: items that need to be discussed in the next SIG meeting

+1 with a special request: help-wanted instead of needs help


(chaos-exporter): Add Unit Test Cases for file chaos-exporter

UNIT TEST
What happened:

Write the unit-test

What you expected to happen:

Write the unit test cases for the chaos-exporter repo

Anything else we need to know?:

Issue related to Repository: https://github.com/litmuschaos/chaos-exporter
Reference:- https://github.com/litmuschaos/chaos-operator/blob/master/pkg/controller/chaosengine/chaosengine_controller_test.go

Does this file exist? I don't see it in chaos-exporter/pkg

@debojitkakoti It's moved to controller/scrape.go

Can I take on this one?

UNIT TEST
What happened:

Write the unit-test

What you expected to happen:

Write the unit test cases for the chaos-exporter repo

Anything else we need to know?:

Issue related to Repository: https://github.com/litmuschaos/chaos-exporter
Reference:- https://github.com/litmuschaos/chaos-operator/blob/master/pkg/controller/chaosengine/chaosengine_controller_test.go

Hello @Sanjay1611 I was going through this issue can you tell me which packages need unit test coverage here or do I need to add unit tests in all packages that have no unit tests present in -https://github.com/litmuschaos/chaos-exporter

@ispeakc0de have we resolved this issue?


TestInvoker#testSpeed() may fail on slow machines

Hi there,
we run into an issue where the following test in src/test/com/twitter/elephantbird/pig/piggybank/TestInvoker.java may fail on slow(er) machines.  In our case, for instance, this happens on our build server.  On my local dev box, the test sometimes passes and sometimes it doesn't (hmm).
    @Test
    public void testSpeed() throws IOException, SecurityException, ClassNotFoundException, NoSuchMethodException {
        EvalFunc<Double> log = new Log();
        Tuple tup = tf_.newTuple(1);
        long start = System.currentTimeMillis();
        for (int i=0; i < 1000000; i++) {
            tup.set(0, (double) i);
            log.exec(tup);
        }
        long staticSpeed = (System.currentTimeMillis()-start);
        start = System.currentTimeMillis();
        log = new InvokeForDouble("java.lang.Math.log", "Double", "static");
        for (int i=0; i < 1000000; i++) {
            tup.set(0, (double) i);
            log.exec(tup);
        }
        long dynamicSpeed = System.currentTimeMillis()-start;
        System.err.println("Dynamic to static ratio: "+((float) dynamicSpeed)/staticSpeed);
        assertTrue( ((float) dynamicSpeed)/staticSpeed < 5);
    }

The culprit is the last line in the method, where it checks whether the dynamic-to-static ration is smaller than "5" (see my question below -- I do not know why 5 in particular is used here.)
The "problem" with this test is that its success seems to depend on the build environment.  Even though a relative ratio is used, this alone apparently does not guarantee that the test completes successfully on any box that the build is being run on (e.g. it may pass if the build box is otherwise idle but it may fail if there are other tasks running in the background).  Also, this unit test is not a functional test but a performance test.
Hence I'd say that one can argue pro and con whether it should be a test that a) is enabled by default and/or b) whether failing this test should make the full build fail (maybe a warning would be more appropriate?).
On a related note:  Where does the value/number of 5 (last line in testSpeed() ) come from?  Was it arbitrarily chosen?  FWIW, on my dev box most dynamic-to-static ratios are in the 4.9-5.3 range, and on our build server they are in the 6.1-6.5 range.  At the moment, we have decided to @Ignore this test in our build for the time being because otherwise the build always fails due to this single test.

We should just remove these classes and their tests. They are now part of pig and don't need to be in EB.
On Feb 22, 2012, at 8:19 AM, "Michael G<EMAIL_ADDRESS>wrote:

Hi there,
we run into an issue where the following test in src/test/com/twitter/elephantbird/pig/piggybank/TestInvoker.java may fail on slow(er) machines.  In our case, for instance, this happens on our build server.  On my local dev box, the test sometimes passes and sometimes it doesn't (hmm).
   @Test
   public void testSpeed() throws IOException, SecurityException, ClassNotFoundException, NoSuchMethodException {
       EvalFunc<Double> log = new Log();
       Tuple tup = tf_.newTuple(1);
       long start = System.currentTimeMillis();
       for (int i=0; i < 1000000; i++) {
           tup.set(0, (double) i);
           log.exec(tup);
       }
       long staticSpeed = (System.currentTimeMillis()-start);
       start = System.currentTimeMillis();
       log = new InvokeForDouble("java.lang.Math.log", "Double", "static");
       for (int i=0; i < 1000000; i++) {
           tup.set(0, (double) i);
           log.exec(tup);
       }
       long dynamicSpeed = System.currentTimeMillis()-start;
       System.err.println("Dynamic to static ratio: "+((float) dynamicSpeed)/staticSpeed);
       assertTrue( ((float) dynamicSpeed)/staticSpeed < 5);
   }

The "problem" with this test is that its success seems to depend on the build environment.  Even though a relative ratio is used, this alone apparently does not guarantee that the test completes successfully on any box that the build is being run on.  Also, the test is not a functional test but a performance test.
Hence I'd say that one can argue pro and con whether it should be a test that a) is enabled by default and/or b) whether failing this test should make the full build fail (maybe a warning would be more appropriate?).
On a related note:  Where does the value/number of 5 (last line in testSpeed() ) come from?  Was it arbitrarily chosen?  FWIW, on my dev box most dynamic-to-static ratios are in the 4.9-5.3 range, and on our build server they are in the 6.1-6.5 range.  At the moment, we have decided to @Ignore this test in our build for the time being.

Reply to this email directly or view it on GitHub:
https://github.com/kevinweil/elephant-bird/issues/146

+1 for removing. I am pretty surprised by the high ratio, though, especially if you seeing it consistently. What JVM is that?
Looks like Dmitriy saw 1.3-1.8x slowdown when he implemented it.. see https://issues.apache.org/jira/browse/PIG-1354?focusedCommentId=12853421&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-12853421
Please send a pull request so  that you get the credit :) will keep this issue open for now.

I created a pull request:
https://github.com/kevinweil/elephant-bird/pull/148
FYI: I initially wanted to create the pull request to target kevinweil:eb-dev (your dev branch). But apparently the eb-dev branch is pretty out of date at the moment compared to kevinweil:master, so I decided to target kevinweil:master instead. Please let me know if that's ok.

@Raghu:
The JVM with which we build is from Sun JDK 1.6.x.
On my Ubuntu dev box:
$ java -version
java version "1.6.0_26"
Java(TM) SE Runtime Environment (build 1.6.0_26-b03)
Java HotSpot(TM) 64-Bit Server VM (build 20.1-b02, mixed mode)

On our RHEL5 build server:
$ java -version
java version "1.6.0_27"
Java(TM) SE Runtime Environment (build 1.6.0_27-b07)
Java HotSpot(TM) 64-Bit Server VM (build 20.2-b06, mixed mode)

Closing as it appears your pull request was merged to master. Thanks for the improvement!


[Feature request]  The recurrence fields should be modeled on those defined for triggering.

Hello,
I discovered LubeLogger, thank you for this application which allows me to delete my unsuitable spreadsheet.
When creating a reminder, it is possible to set a trigger based on :

a date or
a mileage or
both

It is also possible to define a recurrence but the user does not have the choice of the recurrence condition.
Both fields are mandatory, the user must define both fields when only one is probably useful.
This requires entering an improbable value for the unnecessary field.
The recurrence fields should be modeled on those defined for triggering.
Best regards

Released in 1.3.5


Category locale issue

Bug report
Title
Cannot edit category on other locale.
Issue Description
The error occurs when add new locale and try edit existing category using new locale, if we add category after create locale all work good.
Preconditions
1. Bagisto v1.0.0-BETA1
2. Laravel Framework 5.6.39

Steps to reproduce
1. Create category
2. Create new locale
3. Try edit category using new locale

Actual result

Hi @monter08
Bagisto v1.0.0-BETA1 version is compatible with Laravel Version 6.x.
https://github.com/bagisto/bagisto/releases/tag/v1.0.0-BETA1
And I checked this issue on Laravel 6, working fine.

I found the reason, the error occurs on php version 7.4

I am closing this issue, this issue has been already resolved in new version.


Safety of {int64,double}_value

The Tensor methods double_value() and int64_value() can possibly segfault if idx parameter is nonsense. For example:
let t: Tensor = (0.0).into(); // t is a scalar tensor
t.double_value(&[0]); // segfault!

The code segfaults due to irrelevant dimension index. The idx should be checked beforehand, or double_value() should be marked unsafe. It would help that we can identify errors by message rather than helpless segfaults.

Interestingly, this doesn't result in a segfault for me but in the expected panic (about applying select to a  0 dimension tensor).
Not sure what caused this difference of behavior, maybe there are some compile flags that turn on some boundary checks that I have enabled.
Anyway I added a test to check this, it's likely to segfault on your box, but I'll also push some changes so that the dimension are double checked and we exit with a panic in this case rather than segfaulting.


Document on how to run the e2e tests locally

Things to cover:

Creating & provisioning a 1-node cluster at Hetzner
Running the e2e tests
Which environment variables to set
How to run only specific tests (Like only Digitalocean)

Docs for this were added within #249


Setup rspec

Closes #6
Preemptively setup rspec in case it gets the go ahead, so I can get started with first milestone feature #5
In case it's decided to stay with minitest, we can just close this PR.

@cromulus from what I could gather speaking with Will, things are hectic for you on your. Going ahead and merging this to get started on #5

Excellent approach. I am paying attention, but deeply asynchronously. In
general, trust your instincts and move forward, similarly to what you've
done here.
Why don't I hop on a slack channel of yours or vice versa for the time
being and that should help with the communication.
On Jan 20, 2016 11:31 AM, "Oliver Azevedo Barnes"<EMAIL_ADDRESS>wrote:

@cromulus https://github.com/cromulus from what I could gather speaking
with Will, things are hectic for you on your. Going ahead and merging this
to get started on #5 https://github.com/BlueRidgeLabs/kimball/issues/5
—
Reply to this email directly or view it on GitHub
https://github.com/BlueRidgeLabs/kimball/pull/15#issuecomment-173262621.

Definitely, will do. Looks like Marco's added you to our slack already, I'll see you there on #kimball


Consume self-deploy's startAt attribute

As #157879 points-out, this attribute appears unused.
Fixes #157879

Good catch! Must have missed it when I rewrote the service for nixpkgs. I admit I'm happy to see actual use of the self-deploy service in the wild -- I've meant to write a blog post about it for ages but have never gotten around to it.
All looks good to me.

Looking back on the code, it may also be worthwhile to make the minor change of gating the service's systemd dependency on cfg.switchCommand being "boot" if only for correctness' purpose.


Add Security Policy

This is to add a Security Policy for the OCM projects. This should be similar to this SAP Securit Policy and the Flux one.

Required details:

What is the private reporting email / medium?
List of members?

Security Policy added by this PR from @Shegox. Guess this solves this issue.


JobLoadException when Marathon job is not on deployed on Marathon already

In my marathon definitions directory, i have a dirty git state right now, because i am preparing a new marathon definition for a new app:
➜  marathon-jobs git:(master) ✗ git st
On branch master
...
Untracked files:
  (use "git add <file>..." to include in what will be committed)

	X.json

nothing added to commit but untracked files present (use "git add" to track)

When i trigger a typical status call the JobLoadException will be thrown and i don't get results at all:
➜  chapi git:(master) bin/chapi status

  [Chapi\Exception\JobLoadException]
  Unable to load json job data from "/my/path/X.json". Please check if the json is valid.

status

Expected results would be that i get a listing of Chronos and Marathon tasks and when i have a dirty state (like X.json) i expect a info or warning note.

I guess the content of the X.json is not valid or empty!?

It contains invalid json.

I expect a normal status output with a warning


Storage shows strange info

This what Ward shows:

16 GiB Total

6 Disks

935 MiB Swap

However my Pi has only single card (1 disk) with 2 partitions on it:
$ lsblk

NAME         MAJ:MIN RM   SIZE RO TYPE MOUNTPOINT
mmcblk1      179:0    0  14.6G  0 disk
`-mmcblk1p1  179:1    0  14.4G  0 part /
mmcblk1boot0 179:32   0     4M  1 disk
mmcblk1boot1 179:64   0     4M  1 disk
zram0        252:0    0    50M  0 disk /var/log
zram1        252:1    0 935.6M  0 disk [SWAP]

$ df -h

Filesystem      Size  Used Avail Use% Mounted on
udev            801M     0  801M   0% /dev
tmpfs           188M  5.1M  183M   3% /run
/dev/mmcblk1p1   15G  2.6G   12G  19% /
tmpfs           936M     0  936M   0% /dev/shm
tmpfs           5.0M  4.0K  5.0M   1% /run/lock
tmpfs           936M     0  936M   0% /sys/fs/cgroup
tmpfs           936M   36K  936M   1% /tmp
/dev/zram0       49M  3.5M   42M   8% /var/log
tmpfs           188M     0  188M   0% /run/user/1000

Not a big deal, but seams strange.
PS. Thanks for persistent config!  Greatly appreciate.

BTW, the raid disk seems not to be recognized correctly.
here is the display: 14.5 TiB Total / 7 Disks
and here is the result of lsblk
sda                     8:0    0  3.7T  0 disk
├─sda1                  8:1    0  476M  0 part  /boot/efi
├─sda2                  8:2    0  477M  0 part
└─sda3                  8:3    0  3.7T  0 part
  └─md0                 9:0    0  3.7T  0 raid1
    ├─raid-root 253:0    0  3.6T  0 lvm   /
    ├─raid-swap 253:1    0 18.6G  0 lvm   [SWAP]
    ├─raid-var  253:2    0 18.6G  0 lvm   /var
    └─raid-tmp  253:3    0 18.6G  0 lvm   /tmp
sdb                     8:16   0  3.7T  0 disk
├─sdb1                  8:17   0  476M  0 part
├─sdb2                  8:18   0  477M  0 part
└─sdb3                  8:19   0  3.7T  0 part
  └─md0                 9:0    0  3.7T  0 raid1
    ├─raid-root 253:0    0  3.6T  0 lvm   /
    ├─raid-swap 253:1    0 18.6G  0 lvm   [SWAP]
    ├─raid-var  253:2    0 18.6G  0 lvm   /var
    └─raid-tmp  253:3    0 18.6G  0 lvm   /tmp


Update Plan and SimData to work with things that convert to Symbol

Codecov Report

Merging #6 into master will increase coverage by 0.16%.
The diff coverage is 73.91%.

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
+ Coverage   62.18%   62.34%   +0.16%
==========================================
  Files          19       19
  Lines        1153     1150       -3
==========================================
  Hits          717      717
+ Misses        436      433       -3

Impacted Files
Coverage Δ

src/Plans.jl
66.93% <62.50%> (+2.36%)
:arrow_up:

src/simdata.jl
95.50% <80.00%> (-0.50%)
:arrow_down:

Continue to review full report at Codecov.

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


DnsServerDsc: Replace ConvertTo-HashTableFromObject

Pull Request (PR) description

Replaced the private function ConvertTo-HashTableFromObject with the
public function ConvertFrom-DscResourceInstance (from DscResource.Common).

This Pull Request (PR) fixes the following issues
None.
Task list

[x] Added an entry to the change log under the Unreleased section of the
file CHANGELOG.md. Entry should say what was changed and how that
affects users (if applicable), and reference the issue being resolved
(if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Community Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[x] New/changed code adheres to DSC Community Style Guidelines.

This change is

Waiting for the command to be available in a full release.


When editing an existing, correct custom query, if the new query version is erroneous, the previous result stays on screen

Example SPARQL query used in the correct custom query:
SELECT ?s ?p ?o WHERE {
  ?s ?p ?o
}

Example erroneous SPARQL query used in the update (mind the ?o2):
SELECT ?s ?p ?o2 WHERE {
  ?s ?p ?o
}

The updated result view shows an error at the bottom indeed, but the previous result table is still visible:

The SPARQL query validation check is requested in #143.
Let's not use it to avoid the current issue.
We can use the current observation to understand why the displayed result table is out of sync, something that also happens randomly in other circumstances, where we see an old result that subsequently is updated with a new result.


Toggle Python interpreter characters in docs with button

To make it easier to copy and paste example code from the docs, it would be great if we could add a button to remove the >>> before each line of code.

https://sphinx-copybutton.readthedocs.io/en/latest/


[BUG]: Minor sentence edit in case filing acknowledgement page

Change "will be required to be signed" to something like "has to be signed"

Tested in QA Env, working as expected:

cc: @Ramu-kandimalla @rajeshcherukumalli


"Restart" button should stop current execution and restart

Currently, the "Restart" button simply starts a new version of the given execution (any currently running execution will likely fork to it's own branch). However, it should stop the current executing jobs and restart

It may be better to simply only allow the "restart" button when the execution isn't running and treat it like a "play" button


Increment version for azurestackhci releases

Increment package version after release of Azure.ResourceManager.Hci

Will update and merge to main branch.


[docs] Update Fontsource installation instructions

Fontsource went through a breaking change to make the library more versatile for international users.
This update aims to clean up all the old documentation to be more reflective of that and easier to understand for the reader. Also, the repository moved to an org so the main link was edited to not waste a redirect.

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

Shouldn't this be mentioned in the 4.x docs too? The 4.x mentions this:

⚠ Be careful when using this approach. Make sure your bundler doesn't eager load all the font variations (100/300/400/500/700/900, italic/regular, SVG/woff). Fontsource can be configured to load specific subsets, weights and styles. Inlining all the font files can significantly increase the size of your bundle. Material-UI default typography configuration only relies on 300, 400, 500, and 700 font weights.

I think that the 5.x docs helps the user much better that 4.x on this topic.

All ongoing work is on v5. If we cherry-pick this commit to v4, we would want to cherry-pick a lot more for consistency in the strategy. If you won't bother about breaking changes, you can use v5-alpha. We aim for the same level of quality as for a stable release.


Improve TypeScript support and definitions

Hello,
In my project I'm currently giving Apollo Server (express) a try and am having some difficulties finding how to write type-safe mutation resolvers.
Given the following schema:
type Query {
    ...
}

type Mutation {
    reserveDeal(input: ReserveDealInput!): ReserveDealPayload
}

input ReserveDealInput {
    dealId: String!
}

type ReserveDealPayload {
    result: Boolean
}

If I write the following then my code compiles and I do have some type safety with the reserveDeal input
const resolvers = {
    Query: {
        deals: () => dummyDeals
    },
    Mutation: {
        //{ input: ReserveDealInput }
        reserveDeal: (root: any, arg: { input: ReserveDealInput } | any): ReserveDealPayload => {
            console.log("Reserving deal: ", arg.input.dealId);
            return {result: true};
        }
    }
};

The problem is the | any that I had to add to work around the following compilation issue with makeExecutableSchema:
src/index.ts(81,59): error TS2345: Argument of type '{ typeDefs: string; resolvers: { Query: { deals: () => { id: string; title: string; description: ...' is not assignable to parameter of type 'IExecutableSchemaDefinition<any>'.
  Types of property 'resolvers' are incompatible.
    Type '{ Query: { deals: () => { id: string; title: string; description: string; }[]; }; Mutation: { res...' is not assignable to type 'IResolvers<any, any> | IResolvers<any, any>[] | undefined'.
      Type '{ Query: { deals: () => { id: string; title: string; description: string; }[]; }; Mutation: { res...' is not assignable to type 'IResolvers<any, any>[]'.
        Property 'includes' is missing in type '{ Query: { deals: () => { id: string; title: string; description: string; }[]; }; Mutation: { res...'.

Do you have any ideas/pointers to help me improve this code towards better type safety?

@sundeer: I'm getting the same, did you manage to solve that?

Similar issues: #372, #583 & DefinitelyTyped/DefinitelyTyped#21359.

Also related: #404

Updates?

Pull requests now welcome!

This looks potentially fixed in #1307, closing for now, please check out graphql-tools@next and reopen as necessary.


Probability of existance in the prediction step

Hello,
I'm curious about this line here:
https://github.com/yuhsuansia/Trajectory-PMB-EOT-BP/blob/0867c9b00d7aa9d0ca5d1ae6f7bd4292191cece8/TrajectoryPMB-EOT-BP/eotEllipticalShapeRFS.m#L65
Why should the probability of existance in the prediction be calculated like this?
I could find two equations in your paper "Poisson Multi-Bernoulli Approximations for Multiple Extended Object Filtering": 13d and 14d. It seems like 13d is used.
Thanks.

Hi,
Thanks for your question.
This line does not correspond to the prediction step. It is instead related to the initialization step of the loopy belief propagation described in the paper "[Trajectory PMB filters for extended object tracking using belief propagation]". You may still need to do some simple calculations based on the equation in the paper to obtain the equation in the code though.
If you are looking for the code of the prediction step of the trajectory PMB(M) filter, it would be better to have a look at my other TPMB(M) repo.
Best,
Yuxuan

Thanks for the quick reply!


License?

Noticed there's no license file, making this technically proprietary :P

Setting a license would I believe require the signoff of all prior contributors. Not a lawyer though


REPL: "noop" is not exported by<EMAIL_ADDRESS>

Full error:
"noop" is not exported by<EMAIL_ADDRESS>imported by<EMAIL_ADDRESS>
Happens e.g. here:
https://svelte.dev/repl/6dfa9e69a65243b0bbc44d66d91b8325?version=3.17.1
But not on newer versions:
https://svelte.dev/repl/6dfa9e69a65243b0bbc44d66d91b8325?version=4.2.11

This was fixed in Svelte 3.29.5 in https://github.com/sveltejs/svelte/pull/5624 over three years ago. I don't think we need to make the REPL's bundling code still accommodate the broken packages in older versions of Svelte 3.

@Conduitry  I see, however please note that this often means that older links just seem to be "broken", and there is no hint of "try again with newer version". E.g. google often still references links:

Or GitHub Discussions, e.g. basically all links used here: https://github.com/sveltejs/svelte/issues/4285
Perhaps as a simple workaround we could just automatically use the next nearest working svelte version in such cases?
It could (potentially) be interesting to support older versions when wanting to compare behavior changes of different svelte versions, though.


'QuadForm' object has no attribute 'P' with cvxpy 1.0.15

Hi Cvxpy devs!
Describe the bug
Since the last release (1.0.15), our project (DIPY) is failing due to the following error :
'QuadForm' object has no attribute 'P'.  How can I solve this problem?
Thank you for your feedback!
To Reproduce
import cvxpy
import numpy as np

data_norm = np.random.random(203)
M = np.random.random(203*50).reshape((203, 50))
c = cvxpy.Variable(M.shape[1])
lopt = 0
laplacian_matrix = np.ones((50, 50))

design_matrix = cvxpy.Constant(M)
objective = cvxpy.Minimize(
    cvxpy.sum_squares(design_matrix * c - data_norm) +
    lopt * cvxpy.quad_form(c, laplacian_matrix)
)
constraints = [(M[0] * c) == 1] # (K * c) >= -0.1]
prob = cvxpy.Problem(objective, constraints)
prob.solve()

Version

OS: Tested on Linux ,windows, mac
CVXPY Version: 1.0.15

It appears to work if lopt is not zero. A strange bug.

Indeed, really strange...
But Thanks for this information, a workaround is easy to implement until your fix and the new cvxpy release.

This is fixed in master.

Thank you @SteveDiamond!
By curiosity, which PR fixed it and what was the problem?

I fixed it directly in master. Basically if an expression was known to be zero, it was treated as constant. This is reasonable, but interacted badly with other aspects of the code base.

ok, great! Thank you for this information and the fast fix!


[simple-api] Support seek_to at nested levels

Problem solved by the enhancement
Currently the Simple API only supports seek_to at the top level. However, for example when trying to process all elements of an intermediate JSON array or object, this means you can only seek to that intermediate array or object and have to traverse the JSON data manually afterwards.
For example, consider this JSON path (where * represents all items):
first.second[*].third.fourth

Currently you can only use seek_to for the path first.second and then afterwards have to manually traverse third and fourth.
Enhancement description
seek_to should be available at all nesting levels. For non-top level values it should remember (based on the given JSON path) how many arrays and objects were opened, to then manually close them again after the value was consumed.
For the example above, this would then allow first seeking to first.second, and then for each array item seeking to third.fourth.
Alternatives / workarounds
The feature #48 might render this obsolete to some extend, but maybe there are still use cases where that feature cannot be used, e.g. when intermediate object member names are consumed in some way before traversing further down.
Therefore supporting seek_to for nested levels still seems to be useful.

Instead of having seek_to at nested levels, would probably need a read_seeked(path: JsonPath, f: FnOnce(ValueReader)) to make sure the caller consumed the value, or to skip the value otherwise.
Could then consider removing the existing top-level seek_to method (because read_seeked will be available there as well), or keep it for now.

The documentation for read_seeked should mention that: If the path matches a value, then f is called exactly once for reading that value. Otherwise if the path does not match, an error is returned and f is not called.
And similar to the other methods taking a Fn, if the value is not consumed explicitly it will be skipped implicitly.


Make an alias for parsing arguments

Should handle the whole boilerplate:
bb_parseargs "$@"
set -- "${BB_POSARGS[@]}"
unset BB_POSARGS

Added bb_processargs but you have to turn on expand_aliases to use it in a script.


Development build per branch versioning

:shipit: Mobile app from this branch has been deployed to:
https://expo.io/@thebrain/thebrain-mobile-fixDeployingToServerless
Download the Expo app and scan this QR code to get started!

Web is available here: http://thebrain-web-fixdeployingto-d31bdca.s3-website-us-east-1.amazonaws.com
And if you must, you can connect to the graphql server here: https://fdr6dzy326.execute-api.us-east-1.amazonaws.com/dev/graphql

Merging it for now but please move this to a script that we can run, so we don't duplicate such a complex command twice in the circleci (and in general I would rather try to avoid such complex commands in the circle config)

@lucetius ^


chore(rules): add confirmed rules

This new rule allow us to validate a field with a confirmation field.
The confirmation field should have the same name suffixed with _confirmation.
Example:

<form action="{{ 'users.store' | route }}" method="POST">

    <input name="password" type="password" placeholder="Mot de passe">

    <input name="password_confirmation" type="password" placeholder="Confirmez votre mot de passe">

</form>

// Controller

* store(request, response) {
  const rules = {
    'password': 'required|confirmed'
  }

  const validation = yield Validator.validate(rules, request.all())

  // ...

}

All should be ok now. 👍

Coverage increased (+0.004%) to 99.664% when pulling 4d44d93676f6f2ebbc7e2dbd73a991ba9279b086 on RomainLanz:develop into 63003edab532c8a901b75e1c7c4937de5e4ad3a6 on poppinss:develop.

Looks good! Going to run it on my machine once

Good :+1:
Once this has been merge I'll send a PR to the documentation to add this rule.


Initiate Workflow PutItem

Issue #, if available:
https://app.asana.com/0/1202938635037719/1204196022036762/f
Description of changes:
Initiate the workflow and store the JobID and ArchiveID to the DynamoDB table
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Please do git pull rebase locally, my code changes are showing in your branch and there are 3 commits.

Done.


libc/minimal: small fixes to malloc() and friends

When malloc() is called with a size of 0 we should not set errno to
ENOMEM as there is no actual allocation failure in that case. This
duplicates the realloc() behavior.
Check the return value of sys_mutex_lock() in the free() case to make
Coverity happy (issue #32938).
Replace all CHECKIF() by explicit assertion statements to uniformize
those checks with the free() case which can't return errors. In theory
sys_mutex_lock() will never return an error here anyway.

Superseded by #33630


Docs for HTTP interface to PFS

Files in PFS can be accessed via http://<IP_ADDRESS>:30652/v1/pfs/repos/<some-repo-name>/commits/<some-commit>/files/<path/to/some/file> (where <some-commit> is either a commit ID or a branch)
I don't think this is documented anywhere, but probably should be.
Added in #2043 though more useful if/when #2062 is implemented

I think we should not write docs for this, because this service is intended to be for internal use with the dash. For external use of PFS via HTTP, s3gateway is available.

Generally agreed, again, unless we actually follow through with a full HTTP API to Pach's APIs, which has been low-key on the roadmap for at least three years :P

The long-term goal is to phase out this service (#3791), so closing this out


Windows- or Installer-free version of JAR file

Describe the feature
A version of the lombok jar without Windows software.
The OWASP dependency check detects the two .dll files in the lombok installer.

lombok/installer/WindowsDriveInfo-i386.dll
lombok/installer/WindowsDriveInfo-x86_64.dll

On Linux that requires installation of a significant amount of extra software to scan those dependencies for vulnerabilities, and can raise questions with security teams. If these dependencies are not required on Linux, a Lombok jar without these dependencies would be helpful.
I'm not sure whether the installers aren't needed at all when you're using Lombok as a build dependency (as I am). In that case, then perhaps just a slim jar that doesn't contain the installers at all would be simpler.
Otherwise, a windows-free version so that use on Linux systems can avoid installing Windows software should work fine.
Describe the target audience
People who work on Linux and use the OWASP dependency check tool.
Additional context
I'm not sure I can use Lombok with these Windows dependencies, because I doubt I'll be allowed to install the necessary dependencies to scan those files in my target environment.

We're very hesitant on supplying different dependencies. Lombok is a compile-time dependency, and should not be included in the end product.
We prefer to generate one artifact. The current jar file is small, and it's easier for our maintainers and users to only have this one jar.
Would renaming the .dll inside the jar solve your problem? On Windows only, we extract the .dll file to a temp directory, and only when the installer is run.
I have no idea why you are referring to the OWASP tool, but if it complains about compile-time dependencies, you might want to file a bug report with them.

Thanks for the prompt update!
Regarding the compile-time nature of the dependency (certainly an important point), it still means that the dependency needs to be shipped into the infrastructure, specifically to the build server. In some high-security environments, that could be problematic - not sure if it is for me yet. I can suppress the alert, it came as something of a surprise to find Windows binaries present in my build!
Not sure if renaming the dll would help - even if it did, trying to hide the nature of the binaries doesn't sound like the right answer.

I'm not sure if the renaming helps, but I did it anyway, because it  was a possible solution and really simple to implement.

If the scanner still detects it, we can do some simple byte magic. I'm closing this issue, because we have no plans to ship platform dependent jar files, nor a separate installer.

Just want to reference this commit.
https://github.com/rzwitserloot/lombok/commit/75601240760bd81ff95fcde7a1b8185769ce64e8
It "fix" the dependency-check-maven's .NET Assembly Analyzer could not be initialized and at least one 'exe' or 'dll' was scanned. issue.
It took me a day try to find out why my spring project got .NET issue :D


Cannot use mapController.move while using animatedRotate

Hi @TesteurManiak,
I was testing the  animated rotation based on internal sensor bearing in flutter_map and I discovered that one cannot really zoom in when using the animatedRotate method. Do you know if there is anything that can be done about this? My animation duration is ~350ms and I believe the internal sensor sends data every ~370ms.
I was thinking to maybe not rotate if the change is < 10 degrees, which may give the other action time to take effect. This may be quite janky though.
Thanks :)

Is there any reason why you're not using the method animateTo to update the zoom and rotation simultaneously?

Hi @TesteurManiak,
Sorry for the delay, I've been away for work. The animation is triggered automatically as the user moves the device around and the zoom change occurs on button press. Therefore, I cannot see a way I can make this work simultaneously. Do you have any ideas?

I see, as of now the package allows only 1 animation at a time so I don't think you'll be able to implement this behavior with it for now. It would be needed to have a cancel mechanism for ongoing animation.

It is fine, for now I made it so that the animation is not triggered unless the bearing change is not greater than 10 deg. Thanks :)


SA1633 shows a code fix if the header is surrounded by a region, if regions are allowed

We use the header as follow:
#region Copyright © 2015 Company
// --------------------------------------------------------------------------------------------------------------------
// <copyright file="ViewModel.cs" company="Company">
//   Copyright © 2015 Company
// </copyright>
// <creator name="userid"/>
// --------------------------------------------------------------------------------------------------------------------
#endregion

SA1633 shows the code fix on the # of the #region pragma

The analyzers and code fixes for copyright headers currently have limited flexibility but do require the following:

The copyright text must appear in comments. #region directives are not comments, and we have no way to check (or insert) text like #region Copyright © 2015 Company.

The copyright header must be the first syntax trivia in the file, although it may be indented.

A tremendous amount of work went into making sure things work with the supported forms. The header shown above appears to require a large amount of additional work. It might be more appropriate to implement this as part of an independent project.

It is not possible to check if the first line is a #region pragma and the header starts in the second line? I have never worked with the roslyn SDK, so I have no idea what is possible and what not.

Roslyn makes just about anything possible. The majority of the current analysis for file headers was implemented by @vweijsters and @peteri. They might be able to provide more insight into this request.

I'll try and have a quick look at this tomorrow.
There are quite a few code paths around here and it's definitely not as simple as it sounds.

Even putting
#pragma warning disable SA1633 // FileMustHaveHeader
in the first line, the codefix is then reported on this line.

Sorry I've been busy at work the last couple of weeks but I'll have a proper look over the weekend.

Not sure #pragma works for stylecop warnings, normally you'd use a SupressMessage attribute on the code element but that won't work as it's at the top of the file.
Global suppressions like this:
[assembly: System.Diagnostics.CodeAnalysis.SuppressMessage("StyleCop.CSharp.DocumentationRules", "SA1633:File must have header", Justification = "Peters test", Scope = "namespace", Target = "~N:ClassLibrary2")]
do seem to work.
Still looking at modifying the existing rule.

@peteri #pragma warning was expanded in C# 6 to support diagnostic IDs reported by analyzers, in addition to the compiler-reported warning numbers supported in C# 5 and earlier.

@sharwell Didn't realise that (I really should read up on all the changes). I assume this is an issue with the underlying model in VS2015.

:thought_balloon: The issue might be this warning is reported on the first character in the file, and there is no way to place a #pragma line before the first character.

I think you're probably right. I suspect it should only be allowed for the specific warning rather than for all #pragmas  (are there any others that would get reported for a first line? I can't think of any apart from trailing spaces at the end of a file)


Mask API key for SerpApi.com tools

Description: Mask API key for SerpApi.com tools.

Mask SerpApi API key in SerpAPIWrapper and GoogleScholarAPIWrapper
Use SerpApiClient#pagination in the Google Scholar Search Tool (related to https://github.com/langchain-ai/langchain/issues/11513)

Issue: https://github.com/langchain-ai/langchain/issues/12165
Tag maintainer: @eyurtsev

@keenborder786, please review the changes in GoogleScholarAPIWrapper.

closing due to merge conflicts


Integration with another Draft.js plugin

There is another draft.js plugin that requires that they import an editor like this:
import Editor from 'draft-js-plugins-editor
But medium-draft also imports its own Editor
import Editor from "medium-draft";
If I use the Editor from medium-draft, the plugin doesn't work. If I use the plugin editor, medium-draft doesn't work. Is there a way around this problem?

You can use a named import. My recommendation would be something along the lines of the following:
import { Editor as DraftEditor } from 'draft-js'

You can import as
import MediumDraftEditor from 'medium-draft'

Not sure this answered his question. Is it possible to combine this editor with the draft js plugins one? Or would the best be to fork the plugins editor package to use medium-editor instead of draftjs editor?


REALMC-8832: Update command flag descriptions

worked with @kpham1231 on this one... things to look out for while reviewing:

consistency with capitalization ("Realm app", "Service", "Function") and punctuation (don't end usages with ., use ; b/w value descriptions, [] around the docs links)
consistency across commands for similar flags
typos or any other oddities


Add mutation testing

PR adds integration with Certora's Gambit mutation testing framework. Gambit receives Solidity file or list of files as input, and it will generate "mutants" as output. Those are copies of original file, but each with an atomic modification of original file, ie. flipped operator, mutated 'if' condition etc. Idea is to then run the test suite against the mutant. If all tests still pass, mutant survived, that's bad - either there's faulty test or there is missing coverage. If some test(s) fail, mutant's been killed, that's good.
gambitTester script works like this:

generates mutants for files specified in test-mutation/config.json
for each mutant do in parallel (in batches):

replace original file with mutant
re-compile and run foundry suite
track results

report results

How to use it:

update test-mutation/config.json with the list of solidity files to be tested
run yarn run test:mutation

Nice! I'm looking forward to trying this out on our other projects. Is there a way to ignore a certain mutation if we know it's not useful information?
It might be nice to add a section the readme about how to do the mutation testing, including where to install gambit

I've added new doc with all the steps: https://github.com/OffchainLabs/token-bridge-contracts/pull/66/commits/d3e5df6a9f0c4c6a797bc7274cfca5461e422f9d
In the JSON config file (which is the input to Gambit for generating mutants), you can specify which kind of "mutation operators" you want to applied. Here is the list of all supported operators. You can also ie. limit mutation operators to the specific functions that you're interested in testing. Once mutated files are generated, currently there is no way to tell the testing script to skip some of them. But we can add that feature if it would be useful.


Support for nD HDF5 files

The HDF5 dataset seems to still have an assumption of being 4D hardcoded.
As an example, libertem.io.dataset.hdf5.H5DataSet.get_partitions() works
with a 4D origin.
The same holds for libertem.io.dataset.base.DataSet.partition_shape() and
libertem.io.utils.get_partition_shape().
LiberTEM should support files with any shape and at least two dimensions.
See also #441
Thx @robertbuecker for reporting!
Contributor Checklist:

[x] I have added myself to the creators.json file
[ ] I have added a changelog entry for my contribution
[x] I have added/updated test cases

@uellue do you want to review? From my side, this is ready for check in

Thank you, fix confirmed! :-)


baseboxd: Bump version to 2.0.2

LGTM. Just checking that switch-image builds and runs before merging.


9893 xtuml2masl

Yes.
On Mon, Apr 1, 2019 at 2:29 PM Levi Starrett<EMAIL_ADDRESS>wrote:

@leviathan747 commented on this pull request.
In src/org.xtuml.bp.x2m/src/org/xtuml/bp/x2m/Xtuml2Masl.java
https://github.com/xtuml/bridgepoint/pull/660#discussion_r270995529:

@@ -128,8 +136,14 @@ public void xtuml2masl() throws IOException, RuntimeException {
// pipe inputs and outputs together
Path projectPath = new File(projectLocation).toPath();
String projectName = projectPath.getName(projectPath.getNameCount() - 1).toString();

   FileInputStream inputFile = new FileInputStream(

   FileInputStream inputFile;

   if (prebuild) {

       inputFile = new FileInputStream(
           projectLocation + File.separator + CODE_GEN_FOLDER + File.separator + projectName + ".sql");

When prebuild output is used, does the model data get loaded twice?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xtuml/bridgepoint/pull/660#pullrequestreview-221274585,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB5p7v228yPBDsKpDtEifD_zv67rf_V2ks5vck_4gaJpZM4cT3Fe
.


Fixing some pathing information for config.php and sqlite DB

We have the use case, where our Kanboard source tree is outside of the docroot.  As a result, our docroot contains only the following:
lrwxr-xr-x  1 www  www  13 Apr 13 17:39 assets@ -> ../git/assets
lrwxr-xr-x  1 www  www  16 Apr 13 17:39 index.php@ -> ../git/index.php
lrwxr-xr-x  1 www  www  18 Apr 13 17:39 jsonrpc.php@ -> ../git/jsonrpc.php

Prior to this patch, there also had to be a symlink for data, as well as the presence of a config.php.  With this patch, these resources can live in the source tree directory.  Check my math, however, and make sure that it won't break another use case.

I made a couple of improvements after merging your pull-request.
I also updated the installation instructions for people who needs a similar setup: https://github.com/fguillot/kanboard/blob/master/doc/installation.markdown#installation-outside-of-the-document-root
Thanks


Conditional media (plex_recently_added) no posters

I am running the latest Home Assistant, and been using your older dashboard for years and I love it.
Playing around with this one, but I cannot get the movies/series posters to show in the conditional media object.
It used to work, but I updated to the latest version of Home Assistant, and also to the latest version of plex_recently_added (https://github.com/custom-components/sensor.plex_recently_added) integration, I can see the images fine in the old dashboard but I cannot see them in ha-fusion.
Looking at the network inspection in chrome I can see it gives me a 404 and it points to:
http://IP_OF_HA_FUSION:5050/plex_recently_added?path=/library/metadata/19121/art/1716647748
If I change IP_OF_HA_FUSION to my HA instance, I can see the posters.
Any idea what has happened here?

Hey,
Here's some history:
Two years ago, maykar archived the plex_recently_added repository on GitHub, which automatically removed it from HACS. NemesisRE then forked it to so it could still be easily installed. Recently, mkanet took over the original repository and added new features.
I've noticed some issues when using it for a minute:

Setting max to 1 returns the recently added items for each library, whereas the old behavior returned only the single most recently added item.
There's no image resolution option, so you're essentially fetching a wallpaper-sized image each time you refresh the page.
Instead of saving the image to /www, it now fetches the image from /plex_recently_added?..., which is not proxied in Fusion.

The fork by NemesisRE still works fine and can be installed via HACS.
Here's my configuration:
sensor:
  - platform: plex_recently_added
    host: !secret nas_host
    port: !secret plex_port
    token: !secret plex_token
    max: 1
    img_dir: /
    image_resolution: 800

Yeah I figured all this, but I seemed to have some problems with latest version of Home Assistant and NemesisRE version (which I have been using all along prior). Which version of HA are you running?

2024.5.5

Alright yeah I am running the same version. But for some reason when using NemesisRE:s fork, it just does not fetch any data.
My config looks about the same:
sensor:

platform: plex_recently_added
host: !secret plex_host
port: !secret plex_port
token: !secret plex_token
ssl: true
max: 1
img_dir: /
image_resolution: 900

Only different is the ssl variable, which I think I need since I cant reach my plex host without tls, but it should not be a problem this has been working for 2 years prior. I'll try to dig deeper.

Definitely some weird issues surrounding my installation, but I just cant figure it out.
So I got it working, but I had to point towards a external domain name, which has a valid (and trusted) certificate.
This has never been an issue prior, but something must have happened during upgrading.
I have the exact same problem with the mkanet release (which I have a ticket for btw) where it does not matter if I untick the "Verify SSL" box, it still tries to verify the certificate given to you by plex remote access feature. Which is weird, I have never had this problem before.

Thanks for the update. Since the issue is on your side, I'm closing this.

If someone else struggles with this in the future I would like to add that this was caused by a locked pip package from another integration to a specific version, more specifically aiofiles. After removing that it started working again.


feat: add C implementation for math/base/special/rempio2

Description

What is the purpose of this pull request?

This pull request:

adds C implementation for math/base/special/rempio2.

Related Issues

Does this pull request have any related issues?

This pull request:

resolves a part of #649.

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

This is a draft PR, to discuss the implementation.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

So I was getting failure in some of the test cases. I have referred to FreeBSD version 12.2.0, as a reference.
rempio2: https://svnweb.freebsd.org/base/release/12.2.0/lib/msun/src/e_rem_pio2.c?revision=367086&view=markup
kernel-rempio2: https://svnweb.freebsd.org/base/release/12.2.0/lib/msun/src/k_rem_pio2.c?revision=367086&view=markup

@gunjjoshi I refactored the addon (see https://github.com/stdlib-js/stdlib/pull/2279/commits/0df78ac6dd54e79bdb066f7196fe127a92e0ec29). Tests now pass; however, results for C and JS differ. Currently, if provided 128.0, JS returns ~3.618e-17 and C returns -1.6380356615247201e-19. You'll want to investigate why this is the case.

@gunjjoshi I refactored the addon (see 0df78ac). Tests now pass; however, results for C and JS differ. Currently, if provided 128.0, JS returns ~3.618e-17 and C returns -1.6380356615247201e-19. You'll want to investigate why this is the case.

Great, thanks for the help! I'll go through the implementation once more to check everything.

@kgryte I was able to track down the problem. This was because in the C implementation, earlier the exponent mask the we used had a value of 0x7ff00000. But in the javascript implementation, we were using 0x7ff.
After the changes, both the implementations give the same result for 128.0, i.e., ~3.618e-17.
Also, the lint error that we currently have is arising due to two similar if conditions, but that is required for our implementation.

There is a lint error saying the arrays are uninitialized. So, should we initialize them with zeros at the time of declaration directly, or should we declare them first, then initialize them with zeros using a for loop later?

@gunjjoshi I'd go ahead and initialize with zeros at the time of declaration.

Thanks @Planeshifter @kgryte for helping in this.


I Want To Add Readme.md File Level Order Binary Tree Traversal in Tree Traversal

Description
Given a binary tree, print all nodes level by level, like print all nodes of level 1 first, then level 2 and so on… Printed nodes from left to right on any level...
Domain
Data Structure
Type of Contribution
Addition, Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

/assigned


[corechecks/networkpath] Add namespace

What does this PR do?
[corechecks/networkpath] Add Device Namespace

Motivation
This is needed for NDM Devices Resolution.

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge

/merge


Can't build Android exports

ArchLinux 64b godot3.0
error:
UnboundLocalError: local variable 'can_vectorize' referenced before assignment:
File "/root/Godot-nightly/godot/SConstruct", line 286:
detect.configure(env)
File "./platform/android/detect.py", line 95:
if (can_vectorize):

#9772 is merged recently. I think it supposed to fix this. Try pulling master branch and rebuild.

Yeah, this fixed by #9722.


Yatin: Watch viptip(?:@live.com)?

Yatin requests the watch of the watch_keyword viptip(?:@live\.com)?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

viptip(?:@live\.com)? has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by cigien in Charcoal HQ