[Dialog] add support for position-top

In modal component, we would want to allow the following positioning -

position-center - currently the only option available
position-top - new request.

The request came from VR - VBC admin product for modal with changing sizes, so the position will stay fixed as the modal changes.

@jshenkman
Story-book:
https://tmp.dev.vivid.vonage.com/1226/index.html?path=/story/components-dialog--top-position&args=topPosition:true


Clean up redundant/dead CSS

There is still more that could be done, but it is pretty clean now.
The style.css is very close to the original master, and should not need additional editing
all overrides and customization should happen npstheme.css


Improve ux of adding signatories when creating new multisig

During multisig creation the next button becomes active once you have added 2 signatories.
Due to this, when entering the details of 3rd signatory, it's easy to hit the active next link instead of the + button, which then leads to proceeding without fully adding the signatory. This can lead to momentary user confusion on the next page when seeing fewer signatories than expected.
It would be good to improve the ux here to prevent / reduce the likelihood of that scenario.

I see 2 ways:

we make the new signatory field different, and not as aligned as it is right now, to make it clear visually that it's not part of the set yet
we add any valid account on the fly when clicking on "Next"

2 would require some gymnastic that may not be elegant in the code, while 1 isn't straight-forward without a design yet :D

I think making it visually different could be enough here, automatically adding it could also lead to a different set of users becoming confused.
Or a 3rd option, we could show a modal warning if next is clicked and there's an entered signatory that has not been added.

I'm not a huge fan of modals in general, and that's an error one, that no-one likes to see. I'll try to find a way with the design to make it obvious, it shouldn't be too hard. Also I think the "+" is too small and easy to forget, clicking the "Next" seem to come naturally.


"Hide the execution results page," then click on a step causes the "Widget is disposed" error

Resolved by 4281b47e11449037e0534642f1431472c450986d.


add actions and effects to Ostilli Host Archetypes

created Repel Ambient Magic and Spit Ambient Magic (requires automation) actions.
created effect for Repel Ambient Magic
added Repel Ambient Magic and Spit Ambient Magic to Ostilli Host Dedication.
created Administer Ambient Magic action (requires automation).
added Administer Ambient Magic to Soothing Pulse feat.
created Drape Ambient Magic action.
added Drape Ambient Magic to Cloaking Pulse feat.
created Turn Aside Ambient Magic action.
created Turn Aside Ambient Magic effect.
added Turn Aside Ambient Magic to Deflecting Pulse feat.
created Devour Ambient Magic action (requires automation).
added Devour Ambient Magic to Spell Swallow feat.

Hmm Going to redo this branch


Proposal: deprecate importsNotUsedAsValues and preserveValueImports in favor of single flag

Background: importsNotUsedAsValues
importsNotUsedAsValues was introduced alongside type-only imports in https://github.com/microsoft/TypeScript/pull/35200 as a way to control import elision. In particular, Angular users often experienced runtime errors due to the unintended import elision in files like:
import { MyService } from './MyService';

class MyComponent {
  constructor(private myService: MyService) { }
}

It appears to TypeScript as if the import declaration can be elided from the JS emit, but the ./MyService module contained order-sensitive side effects. By setting the new --importsNotUsedAsValues flag to preserve, import declarations would not be elided, and the module loading order and side effects could be preserved. Type-only imports could then be used to elide specific import declarations.
Background: preserveValueImports
preserveValueImports was added in https://github.com/microsoft/TypeScript/pull/44619 as a way to control elision of individual imported names so that symbols can be referenced from places TypeScript cannot analyze, like eval statements or Vue templates:
import { doSomething } from "./module";

eval("doSomething()");

Under default compiler options, the entire import statement is removed, so the eval’d code fails. Under --importsNotUsedAsValues preserve, the import declaration is preserved as import "./module" since the flag is only concerned with module loading order and potential side effects that may be contained in "./module". Under the new --preserveValueImports option, doSomething would be preserved even though the compiler thinks it is unused.
In the same release, the ability to mark individual import specifiers as type-only was added as a complement to --preserveValueImports.
User feedback
These two flags, along with type-only import syntax, were designed to solve fairly niche problems. Early on, I encouraged users not to use type-only imports unless they were facing one of those problems. But as soon as they were available, and consistently since then, we have seen enthusiasm for adopting type-only imports everywhere possible as an explicit marker of what imports will survive compilation to JS. But since the flags were not designed to support that kind of usage of type-only imports, the enthusiasm has been accompanied by confusion around the configuration space and frustration that auto-imports, error checking, and emit don’t align with users’ mental model of type-only imports.
Further, because the two flags were designed at different times to address different issues, they interact with each other (and with isolatedModules) in ways that are difficult to explain without diving into the background of each flag and the narrow problems they were intended to solve. And the flag names do nothing to clear up this confusion.
Proposal
We can solve the problems addressed by importsNotUsedAsValues and preserveValueImports with a single flag that is

easier to explain
less complex to implement
well-suited to users who want to use type-only imports for stylistic reasons

On the schedule of https://github.com/microsoft/TypeScript/issues/51000, I propose deprecating importsNotUsedAsValues and preserveValueImports, and replacing them with a single flag called (bikesheddable) verbatimModuleSyntax. The effect of verbatimModuleSyntax can be described very simply:

verbatimModuleSyntax: Emits imports and exports to JS outputs exactly as written in input files, minus anything marked as type-only. Includes checks to ensure the resulting output will be valid.

No elision without type
This is a stricter setting than either importsNotUsedAsValues or preserveValueImports (though it’s approximately what you get by combining both with isolatedModules), because it requires that all types be marked as type-only. For example:
import { writeFile, WriteFileOptions } from "fs";

would be an error in --verbatimModuleSyntax because WriteFileOptions is only a type, so would be a runtime error if emitted to JS. This import would have to be written
import { writeFile, type WriteFileOptions } from "fs";

No transformations between module systems
True to its name, verbatimModuleSyntax has another consequence: ESM syntax cannot be used in files that will emit CommonJS syntax. For example:
import { writeFile } from "fs";

This import is legal under --module esnext, but an error in --module commonjs. (In node16 and nodenext, it depends on the file extension and/or the package.json "type" field.) If the file is determined to be a CommonJS module at emit by any of these settings, it must be written as
import fs = require("fs");

instead. Many users have the impression that this syntax is legacy or deprecated, but that’s not the case. It accurately reflects that the output will use a require statement, instead of obscuring the output behind layers of transformations and interop helpers. I think using this syntax is particularly valuable in .cts files under --module nodenext, because in Node’s module system, imports and requires have markedly different semantics, and actually writing out require helps you understand when and why you can’t require an ES module—it’s easier to lose track of this when your require is disguised as an ESM import in the source file.

This is great. Simplifying the configuration to achieve the more explicit/understandable mental model will really help adoption.
As a thought exercise, I wonder if we can constrain the configuration even more to reduce combinatorial tsconfig complexity. Does this new flag need to be independent of isolatedModules? Would it ever make sense to use the option without isolatedModules?
Concretely, could we express the new option as "isolatedModules": "strict"?

Yeah, I think the flag as I’ve described it is a superset of isolatedModules, with the lone exception of a rule against accessing ambient const enums in isolatedModules. The const enum interaction with other flags has gotten incredibly messy and it’s not clear to me what’s needed there. But making this behavior a variant of isolatedModules seems doable. My only regret is that isolatedModules is itself not a very descriptive name, but I can live with that.

I'm very happy this is going ahead.
Given the new flag now obsoletes three existing flags, could we reduce config complexity by erroring if any of the existing flags are used in conjunction with the new flag regardless of their value?  Meaning this would force users to clean up and remove those old entries in tsconfig and would shortcut any discussion of how they interact.

Yeah, I think that’s reasonable.

Well, if we have multiple configs for multiple compilations, the one that has --module commonjs can simply not enable verbatimModuleSyntax. But ironically, the purported value of the flag is emit transparency, which is exactly what we don’t have by using a bundler, and what we might think valuable in our Node build 🙃

The flag will be called verbatimModuleSyntax, just because it’s more understandable than isolatedModules.

as a non-native, this is the first time I see the word "verbatim" so it's not so understandable to me 😂

We should make sure typescript-eslint can identify imports that can be import type but are not

consistent-type-imports does what you want for 99.9% of the cases.
There are some difficulties around decorators + emitDecoratorMetadata because TS has changed over time and does have some magic where cross-module types can impact the compiler output, which we cannot replicate as the rule is not type-aware.
Worth noting that we also have consistent-type-exports which enforces that types are exported with a specifier. It is type-aware so it can handle the export-with-source and import-then-export cases.
Together these rules should be enough to enforce that a module can be safely transpiled in isolation.

Is it possible to also get this added to the web playground?
I wanted to play around with it - but it's not listed as an option in the UI.
I could probably hack around to enable it - but it'd be good to see it added first-class to repro things easily.
Note that importsNotUsedAsValues was also not added to the playground web playground, but preserveValueImports was.

import fs = require("fs");

Then why not this?

const fs = require("fs");

@bradzacher You can control any option in the playground with a magic comment: // @verbatimModuleSyntax: true (note that it’s “sticky” and stays true even if deleted; you have to switch it to false to revert).
@Jack-Works a const declaration only declares a value, whereas an import alias can declare a value, type, namespace, or some combination of them. So without the import keyword, it would be impossible to, say, use the type meaning of a class:
const User = require("./models/user");
const u: User = new User();
//       ^^^^ `User` is only a value

We could say that you can use const when you only need a value and import when you need types, but it would be a massive breaking change to start resolving const/require in *.ts files. Lots of people use const/require as an escape hatch when they need to get a dependency whose typings are totally busted.
I do wish TypeScript had come up with a way to use totally standard CJS syntax and still be able to pass types around from the beginning, but (1) I have thought about this a lot and have never come up with a great alternative, and (2) I think the ship sailed too long ago to be worth trying to revise it now.

Whilst I love the idea behind this, the one part of it I simply cannot get behind is this behaviour:
import {type T} from 'mod';

// becomes

import 'mod';

I get that the idea is that TS will only operate on the specifier, but it just seems so wrong to me that an import that only imports types with inline specifiers would be left behind to create a runtime side-effect.
It feels wrong because that same import with the keyword shifted one token to the left would be entirely elided.
This seems like it's going to just exist as a foot-gun for users that will need to be solved via linting.
There are already requests to change/add lint rules to catch this case because some users can see that it's not the output they would want. (https://github.com/typescript-eslint/typescript-eslint/issues/6379)
Is there any specific reason that it was decided that TS wouldn't "roll-up" the elison?

You can think
import {type T} from 'mod';

// becomes

import {} from 'mod';

This is the same as import 'mod'. TS current behavior is correct IMO.

@Jack-Works to clarify, i fully understand the steps to get to a side-effect import. The crux of it is that TS elided inline specifiers without touching the declaration.
The question I'm asking is why it was chosen that TS wouldn't roll up the elison to the declaration like all the prior art does eg babel (which is the same result for TS and flow), TS's default behaviour, swc.

Some users need control over whether the declaration is preserved for module-loading side-effects. That’s why importsNotUsedAsValues was made, so its successor needs to have feature parity. Also, the point of the flag is to be obvious and explainable. We only mess with the parts that have type on them. No more magic.
When designing this feature, I fully expected linters to pick up the job of making sure import declarations are marked as type-only when possible for users who want to ensure that no side-effect imports remain unnecessarily in their emit, as I said in the proposal:

We should make sure typescript-eslint can identify imports that can be import type but are not

and I was pleasantly surprised when you commented earlier that consistent-type-imports would be sufficient here.

Also, it doesn’t really matter for what people care about here, but the blog is currently incorrect. import {type T} from 'mod' becomes import {} from 'mod', not import 'mod'. I clarify this just to reinforce the point that the syntactic transformation is dead-simple, and does what it looks like it would do, in my opinion.

I was pleasantly surprised when you commented earlier that consistent-type-imports would be sufficient here

Reading the OP I didn't see any mention of this behaviour and given that the base behaviour of TS is to not do this, I didn't know that importsNotUsedAsValues worked this way (I've never used the flag, personally).
consistent-type-imports will enforce that any import specifier used only in a type location must be marked with a type specifier. That is all that it does. It does not encode logic to ensure that an import that has no value specifiers should use a top-level qualifier because as far as we knew this was a purely stylistic choice.
From talking to the community it seems most people still believe it is (which is technically correct... In some cases).
There are other lint rules that can enforce whether you use top-level or inline specifiers, but none so far that enforce that allow inline specifiers, but enforce that a top-level qualifier be used if there are no value specifiers.

but none so far that enforce that allow inline specifiers, but enforce that a top-level qualifier be used if there are no value specifiers

Would you be open to including such a rule in typescript-eslint?

Discussing the best course of action here:
https://github.com/import-js/eslint-plugin-import/issues/2676#issuecomment-1407107260

Side-effecting imports, by almost universal convention, do not export anything - as such, it's advisable to assume that there are only side effects if the file is imported without bindings, including types. An import statement that only brings in types should mean that the entire import statement is removed, so that the file isn't included at all.

Just to look at this at another point of view; if we use the same sort of type-erasure scheme as the "type in JS" ECMAScript proposal uses, this code:
import { type Foo } from "bar";

Would be read by the runtime as:
import {} from "bar";

Because only the types get erased.
That means the question "should this module be loaded?" becomes "does the import declaration have a type keyword?". To me this is a simple rule, compared to "and also, if all specifiers have type, then also do nothing", and less likely to break things (as mentioned in the original PR description).

@jakebailey i agree the most conservative approach would do exactly this - but I don't think it's actually beneficial for the ecosystem to tacitly endorse modules with exports also having side effects, by being concerned with breaking those use cases.

Just exploring the auto-removal-if-no-value-bindings suggestion, what runtime behavior would we expect from these files? Should they match?
// index.js
import {} from "path"

// index.ts
import {} from "path"

Those specific two, sure. But import { type x } from 'path' is NOT simply import {} from 'path' with a type import added on, it's "i like that style better and i only added this import statement to get the type".

For a bit of historical context, the pattern that motivated importsNotUsedAsValues was from an older version of Angular, where it was indeed very common to export things and have side effects in the way of DI registration. TypeScript has always preserved side-effect-only imports (ones with no import clause whatsoever); the complaint was that code like this was needed in order to ensure SomeService got registered in the right order or something:
import { SomeService } from "./service"; // gets removed completely
import "./service";                      // always stays

export class SomeComponent {
  constructor(service: SomeService) {}
}

it's "i like that style better and i only added this import statement to get the type".

Completely disagree. Different syntax was added for different use cases and has different behavior.
Also, just to make sure everyone realizes, none of this is new with verbatimModuleSyntax. Elision has always worked this way with importsNotUsedAsValues and preserveValueImports.

@andrewbranch why the syntax was added is never guaranteed to be the same reason someone chooses to use it.
I would argue that the two-line approach is hugely preferable, and even better would be avoiding that hazardous design pattern in angular itself (which "older" implies, they did move to avoid?)

I used to work on a big web app with an absolute spaghetti of cyclic dependencies where everything constructed singleton classes at load time that all depend on each other. Having been scarred for life, I don’t need convincing that the pattern is bad 😄. But Angular version whatever-point-oh was pretty popular, and it was one of our most upvoted issues.
But I don’t think we have to justify past motivations to make sense of this. verbatimModuleSyntax is an opt-in flag that makes imports and exports emit as close to what you wrote as possible. If you want imports to be elided wherever possible, that’s literally the default. If you want that, but also want to be transpiler-friendly, that is and always has been isolatedModules. Neither of these modes are going away. 99% of people who were using importsNotUsedAsValues were using it for linty reasons. People who want that can use a linter.
My top priority is to keep verbatimModuleSyntax simple and explainable, which means not changing the emit back to do magic elision. What I’m potentially more willing to budge on is whether there should be a way of keeping the error that importsNotUsedAsValues had that makes sure you write import type (fully elided import declarations) whenever possible. I thought that concern was kind of linty, and if we issued an error for it predicated on verbatimModuleSyntax, we would be un-fixing the old Angular-inspired issue we fixed. Those users would have to go back to using double imports, or // @ts-ignore the error. We could error under a separate flag, but I really hoped to reduce the complexity matrix here. While I haven’t really heard any compelling reasons why this isn’t a lint issue, I’m open to feedback on that.

Yep that’s what the PR says. It’s also what the new in-progress module docs say.

In TypeScript 5.0, a new compiler option called verbatimModuleSyntax was introduced to help TypeScript authors know exactly how their import and export statements will be emitted. When enabled, the flag requires imports and exports in input files to be written in the form that will undergo the least amount of transformation before emit. So if a file will be emitted as ESM, imports and exports must be written in ESM syntax; if a file will be emitted as CJS, it must be written in the CommonJS-inspired TypeScript syntax (import fs = require("fs") and export = {}). This setting is particularly recommended for Node projects that use mostly ESM, but have a select few CJS files. It is not recommended for projects that currently target CJS, but may want to target ESM in the future.

Hi,
I am really not sure if this is the right issue for this, but it is what I was able to trace using --isolatedModules in the switch from 4.9.5 to 5.0.x.
Given this file:
// I am aware that either is enough to raise an error up to 4.9.5
interface Foo {
  bar(): void;
}

function foo() {}

When using 4.9.5 it causes this, expected, error:
foo.ts:1:1 - error TS1208: 'foo.ts' cannot be compiled under '--isolatedModules' because it is considered a global script file. Add an import, export, or an empty 'export {}' statement to make it a module.

1 interface Foo {
  ~~~~~~~~~

Found 1 error in foo.ts:1

So far so good. Now with the same tsconfig.json, upgrading to anything on the 5.x.x range, has stopped raising the error, and the code compiles normally.
tsconfig
The tsconfig.json file used is the tsc --init default with this modification:
+ "isolatedModules": true,

Perhaps any of you can recognise this change in behavior? Perhaps it is expected to begin with? Thanks!


bug(datepicker): legacy datepicker module

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
We are migrating our library from angular v14 to latest v15 rc.
We have styled all the angular components based on our requirement including datepicker component.
We currently do not have enough capacity to migrate all the component to mdc components. So we are planning to use legacy components for a while.
While migrating, I can see datepicker component does not have legacy version of the same.
Is this expected?
Reproduction
NA
Expected Behavior
There should be a legacy datepicker component.
Actual Behavior
There is no legacy datepicker component.
Environment

Angular:  15.0.0-rc.4
CDK/Material: 15.0.0-rc.3
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows

The datepicker works both with the legacy and non-legacy versions of mat-form-field.

@crisbeto We have extended MatDatepickerInput and created a timepicker in my library. So when we do super inside constructor of custom directive, it expects us to pass MatFormField.
So after migrating to 15rc, If we pass legacy MatFormField instead, it throws below error -
Error: libs/**/time-picker.directive.ts:130:52 - error TS2345: Argument of type 'MatLegacyFormField' is not assignable to parameter of type 'MatFormField'.
  Type 'MatLegacyFormField' is missing the following properties from type 'MatFormField': _textField, _iconPrefixContainer, _textPrefixContainer, _floatingLabel, and 24 more.

130     super(_elementRef, _dateAdapter, _dateFormats, _formField);
                                                       ~~~~~~~~~~

× Failed to compile.

@crisbeto Also there are some styling related changes in datepicker component which forces us to revisit styles for datepicker component unlike for other components which are having legacy component. This is the 2nd reason why I asked for legacy datepicker component.

I've opened a PR to address the typing issue. Until it gets released you should be able to safely cast to any in the constructor.
As for the styling, I believe it should be identical, judging by https://github.com/angular/components/pull/25648


No Metrics Showing

Does not show performance graphs. Just the tabs for them.

Looks like you have a typo in the BATCH_INSIGHTS_DOWNLOAD_UR env variable(missing the L

Hi typo fixed but still no graphs.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Timothee Guerin<EMAIL_ADDRESS>Sent: Tuesday, June 11, 2019 11:06:30 AM
To: Azure/batch-insights
Cc: Sharmaine Popov; Author
Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
Looks like you have a typo in the BATCH_INSIGHTS_DOWNLOAD_UR env variable(missing the L
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWRBHV4VATH4OV275ILPZ65PNA5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXNOGHA%23issuecomment-500884252&data=02|01|Sharmaine.Popov%40microsoft.com|add2fc12a79644471e4708d6ee7e62c9|72f988bf86f141af91ab2d7cd011db47|1|0|636958623926293780&sdata=wYJUu1zQjkQ%2BvPNKPuhO9wd4PY9YUFj%2FJrWnC9oewIY%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWSPCLB2PRG4VWIYDETPZ65PNANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|add2fc12a79644471e4708d6ee7e62c9|72f988bf86f141af91ab2d7cd011db47|1|0|636958623926303780&sdata=1CsNEosEgjKogYGU94xQHDRHB2dyHFsO94%2B%2FUgciS0k%3D&reserved=0.

...also same errors.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Timothee Guerin<EMAIL_ADDRESS>Sent: Tuesday, June 11, 2019 11:06:30 AM
To: Azure/batch-insights
Cc: Sharmaine Popov; Author
Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
Looks like you have a typo in the BATCH_INSIGHTS_DOWNLOAD_UR env variable(missing the L
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWRBHV4VATH4OV275ILPZ65PNA5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXNOGHA%23issuecomment-500884252&data=02|01|Sharmaine.Popov%40microsoft.com|add2fc12a79644471e4708d6ee7e62c9|72f988bf86f141af91ab2d7cd011db47|1|0|636958623926293780&sdata=wYJUu1zQjkQ%2BvPNKPuhO9wd4PY9YUFj%2FJrWnC9oewIY%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWSPCLB2PRG4VWIYDETPZ65PNANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|add2fc12a79644471e4708d6ee7e62c9|72f988bf86f141af91ab2d7cd011db47|1|0|636958623926303780&sdata=1CsNEosEgjKogYGU94xQHDRHB2dyHFsO94%2B%2FUgciS0k%3D&reserved=0.

hhm, Same error in stderr.txt as well? I guess you rebooted the node(s) after the change?

Oh the reboot. Tripped over that before. Ok let me do that.
Sharmaine Popov
Cloud Solution Architect |Microsoft Canada Inc. | sharmaine.popov<EMAIL_ADDRESS>|<PHONE_NUMBER>
From: Timothee Guerin<EMAIL_ADDRESS>Sent: Tuesday, June 11, 2019 12:34 PM
To: Azure/batch-insights<EMAIL_ADDRESS>Cc: Sharmaine Popov<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
hhm, Same error in stderr.txt as well? I guess you rebooted the node(s) after the change?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWWHFGVGKHN3WG4BJCLPZ7HWBA5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXNXOZY%23issuecomment-500922215&data=02|01|Sharmaine.Popov%40microsoft.com|f3cc331e286d4e80d5c708d6ee8a8dc2|72f988bf86f141af91ab2d7cd011db47|1|0|636958676182457686&sdata=1%2BjM93ZdorcuYarirTD1KjrLjFVUWmz0%2FUzTEn1NE3Q%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWWGSBE3FRTVZMVR5U3PZ7HWBANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|f3cc331e286d4e80d5c708d6ee8a8dc2|72f988bf86f141af91ab2d7cd011db47|1|0|636958676182467693&sdata=ZkHZX272cI4s5aAh0NhMSfPmZTPykqu3hFIGsO623N8%3D&reserved=0.

In Batch Explorer after you update the start task you get a notification that lets you reboot all the nodes at once too.

? I have never noticed that.
Sharmaine Popov
Cloud Solution Architect |Microsoft Canada Inc. | sharmaine.popov<EMAIL_ADDRESS>|<PHONE_NUMBER>
From: Timothee Guerin<EMAIL_ADDRESS>Sent: Tuesday, June 11, 2019 12:36 PM
To: Azure/batch-insights<EMAIL_ADDRESS>Cc: Sharmaine Popov<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
In Batch Explorer after you update the start task you get a notification that lets you reboot all the nodes at once too.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWWRPZZ5J5NXADBJG7LPZ7H77A5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXNXXTA%23issuecomment-500923340&data=02|01|Sharmaine.Popov%40microsoft.com|b0413dac1d684ba622b108d6ee8aec93|72f988bf86f141af91ab2d7cd011db47|1|0|636958677780173497&sdata=pcpL1sjpEmALqrhM04wWxT4LSEkUA1l7xxGfrE3x1g0%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWSGL3PAZ2NXXFVTCTDPZ7H77ANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|b0413dac1d684ba622b108d6ee8aec93|72f988bf86f141af91ab2d7cd011db47|1|0|636958677780173497&sdata=uHML3G%2BPDqofoBb5NE6get8bBb9zZ%2FA4SH24X%2BXfSmc%3D&reserved=0.

Should look like this

Ok! No errors but no graphs either 😊 ☹
Sharmaine Popov
Cloud Solution Architect |Microsoft Canada Inc. | sharmaine.popov<EMAIL_ADDRESS>|<PHONE_NUMBER>
From: Timothee Guerin<EMAIL_ADDRESS>Sent: Tuesday, June 11, 2019 12:36 PM
To: Azure/batch-insights<EMAIL_ADDRESS>Cc: Sharmaine Popov<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
In Batch Explorer after you update the start task you get a notification that lets you reboot all the nodes at once too.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWWRPZZ5J5NXADBJG7LPZ7H77A5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXNXXTA%23issuecomment-500923340&data=02|01|Sharmaine.Popov%40microsoft.com|b0413dac1d684ba622b108d6ee8aec93|72f988bf86f141af91ab2d7cd011db47|1|0|636958677780173497&sdata=pcpL1sjpEmALqrhM04wWxT4LSEkUA1l7xxGfrE3x1g0%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWSGL3PAZ2NXXFVTCTDPZ7H77ANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|b0413dac1d684ba622b108d6ee8aec93|72f988bf86f141af91ab2d7cd011db47|1|0|636958677780173497&sdata=uHML3G%2BPDqofoBb5NE6get8bBb9zZ%2FA4SH24X%2BXfSmc%3D&reserved=0.

Do you have a working demo? Would like to show my client something in about 15 minutes 😊
Sharmaine Popov
Cloud Solution Architect |Microsoft Canada Inc. | sharmaine.popov<EMAIL_ADDRESS>|<PHONE_NUMBER>
From: Timothee Guerin<EMAIL_ADDRESS>Sent: Tuesday, June 11, 2019 12:40 PM
To: Azure/batch-insights<EMAIL_ADDRESS>Cc: Sharmaine Popov<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
Should look like this
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F1031227%2F59290196-db777700-8c2c-11e9-8c2b-30a30bc0fc81.png&data=02|01|Sharmaine.Popov%40microsoft.com|e921f855256943c3203708d6ee8b73b9|72f988bf86f141af91ab2d7cd011db47|1|0|636958680058844890&sdata=7gozNRvp8XsOCoB5iWblyrrFAKtObYiqP5zfO9qvjSQ%3D&reserved=0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWVOGYYKJQVZJZTFDVDPZ7IOFA5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXNYDUI%23issuecomment-500924881&data=02|01|Sharmaine.Popov%40microsoft.com|e921f855256943c3203708d6ee8b73b9|72f988bf86f141af91ab2d7cd011db47|1|0|636958680058844890&sdata=YO555DFmtTbKZnDCAEZSJ4tTuRz4EiVx1YFQ8Fhmc%2BI%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWUHNRVK67JQE3ORQYTPZ7IOFANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|e921f855256943c3203708d6ee8b73b9|72f988bf86f141af91ab2d7cd011db47|1|0|636958680058854897&sdata=%2FVHrgNkyRM7JkduppkKzwGLBJ%2FpeMfvbA7L7Yra62Lw%3D&reserved=0.

Ok! Got graphs. Is there a way to persist this data or they can just look at the graphs??
Sharmaine Popov
Cloud Solution Architect |Microsoft Canada Inc. | sharmaine.popov<EMAIL_ADDRESS>|<PHONE_NUMBER>
From: Timothee Guerin<EMAIL_ADDRESS>Sent: Tuesday, June 11, 2019 12:40 PM
To: Azure/batch-insights<EMAIL_ADDRESS>Cc: Sharmaine Popov<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
Should look like this
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F1031227%2F59290196-db777700-8c2c-11e9-8c2b-30a30bc0fc81.png&data=02|01|Sharmaine.Popov%40microsoft.com|e921f855256943c3203708d6ee8b73b9|72f988bf86f141af91ab2d7cd011db47|1|0|636958680058844890&sdata=7gozNRvp8XsOCoB5iWblyrrFAKtObYiqP5zfO9qvjSQ%3D&reserved=0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWVOGYYKJQVZJZTFDVDPZ7IOFA5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXNYDUI%23issuecomment-500924881&data=02|01|Sharmaine.Popov%40microsoft.com|e921f855256943c3203708d6ee8b73b9|72f988bf86f141af91ab2d7cd011db47|1|0|636958680058844890&sdata=YO555DFmtTbKZnDCAEZSJ4tTuRz4EiVx1YFQ8Fhmc%2BI%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWUHNRVK67JQE3ORQYTPZ7IOFANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|e921f855256943c3203708d6ee8b73b9|72f988bf86f141af91ab2d7cd011db47|1|0|636958680058854897&sdata=%2FVHrgNkyRM7JkduppkKzwGLBJ%2FpeMfvbA7L7Yra62Lw%3D&reserved=0.

App insights has a delay in the metrics ~5min so would make sense nothing is there yet. I can give you read permission to an account which has test pool but graphs will look like that

All this data in in application insights which persit for 90 days I believe. You can query it yourself and get with specific parameters. See this doc for it https://github.com/Azure/batch-insights/wiki/Query-application-insights

Wait have graphs – but no CPU?
Also do you know if code can be instrumented wit App Insights to get the task run data?  I went through this article, with the accompanying code but never got anything in App Insights.
https://docs.microsoft.com/en-us/azure/batch/monitor-application-insights
Sharmaine Popov
Cloud Solution Architect |Microsoft Canada Inc. | sharmaine.popov<EMAIL_ADDRESS>|<PHONE_NUMBER>
From: Timothee Guerin<EMAIL_ADDRESS>Sent: Tuesday, June 11, 2019 12:40 PM
To: Azure/batch-insights<EMAIL_ADDRESS>Cc: Sharmaine Popov<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
Should look like this
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F1031227%2F59290196-db777700-8c2c-11e9-8c2b-30a30bc0fc81.png&data=02|01|Sharmaine.Popov%40microsoft.com|e921f855256943c3203708d6ee8b73b9|72f988bf86f141af91ab2d7cd011db47|1|0|636958680058844890&sdata=7gozNRvp8XsOCoB5iWblyrrFAKtObYiqP5zfO9qvjSQ%3D&reserved=0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWVOGYYKJQVZJZTFDVDPZ7IOFA5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXNYDUI%23issuecomment-500924881&data=02|01|Sharmaine.Popov%40microsoft.com|e921f855256943c3203708d6ee8b73b9|72f988bf86f141af91ab2d7cd011db47|1|0|636958680058844890&sdata=YO555DFmtTbKZnDCAEZSJ4tTuRz4EiVx1YFQ8Fhmc%2BI%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWUHNRVK67JQE3ORQYTPZ7IOFANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|e921f855256943c3203708d6ee8b73b9|72f988bf86f141af91ab2d7cd011db47|1|0|636958680058854897&sdata=%2FVHrgNkyRM7JkduppkKzwGLBJ%2FpeMfvbA7L7Yra62Lw%3D&reserved=0.

CPU should be here, what do you see? For the task specific metrics. this tool unfortunately doesn't provide this functionality. For this you'd need to have something run for every task.

<EMAIL_ADDRESS>Sharmaine Popov
Cloud Solution Architect |Microsoft Canada Inc. | sharmaine.popov<EMAIL_ADDRESS>|<PHONE_NUMBER>
From: Timothee Guerin<EMAIL_ADDRESS>Sent: Tuesday, June 11, 2019 12:53 PM
To: Azure/batch-insights<EMAIL_ADDRESS>Cc: Sharmaine Popov<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
CPU should be here, what do you see? For the task specific metrics. this tool unfortunately doesn't provide this functionality. For this you'd need to have something run for every task.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWXG3MES4O45TTZIKCDPZ7KAPA5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXNZQHI%23issuecomment-500930589&data=02|01|Sharmaine.Popov%40microsoft.com|0a2b149d92aa4fc2027a08d6ee8d53aa|72f988bf86f141af91ab2d7cd011db47|1|0|636958688096609800&sdata=LYGAgWWFXXKg1jq1MBmdP23tcUU5XuRhWEbwI%2Bzmb%2F0%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWTCFUCEQSOZVOTOUCLPZ7KAPANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|0a2b149d92aa4fc2027a08d6ee8d53aa|72f988bf86f141af91ab2d7cd011db47|1|0|636958688096609800&sdata=Qq%2FChMFu%2BEGogqaFNDI5aT0iymjSJ%2FfcIu%2BCkxSgtkc%3D&reserved=0.

The image doesn't show up on github or the email. Could you try uploading via github?

Hhm and no errors in nodestats.log or nodestats.err.log? Which OS is the pool running on?

Nope, I’ll send the out log later.
Can we get per node stats in log analytics
Sharmaine Popov
Cloud Solution Architect |Microsoft Canada Inc. | sharmaine.popov<EMAIL_ADDRESS>|<PHONE_NUMBER>
From: Timothee Guerin<EMAIL_ADDRESS>Sent: Tuesday, June 11, 2019 1:02 PM
To: Azure/batch-insights<EMAIL_ADDRESS>Cc: Sharmaine Popov<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
Hhm and no errors in nodestats.log or nodestats.err.log? Which OS is the pool running on?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWT5HWATAAZKPQJ2YHTPZ7LCDA5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXN2NQA%23issuecomment-500934336&data=02|01|Sharmaine.Popov%40microsoft.com|e6f9347c8b58468a530008d6ee8e93fa|72f988bf86f141af91ab2d7cd011db47|1|0|636958693469122823&sdata=SRBbkd26vocGAE%2F8V3Ai4%2B3AE7B8974a9xnE%2B2nwEPE%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWWOZQ4R5N6GPGWNOWTPZ7LCDANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|e6f9347c8b58468a530008d6ee8e93fa|72f988bf86f141af91ab2d7cd011db47|1|0|636958693469122823&sdata=TaTVoonvDoJlDjM06qp2TCjsuwSDMJrFHmGuJJfu05E%3D&reserved=0.

Yes, you can query using the roleInstance property
https://github.com/Azure/batch-insights/wiki/Query-application-insights#basic-list-of-all-metrics-for-a-node

Hi Thanks for all the help, I think the client can use this. Do you know what the data collection interval is for the performance counters?
Sharmaine Popov
Cloud Solution Architect |Microsoft Canada Inc. | sharmaine.popov<EMAIL_ADDRESS>|<PHONE_NUMBER>
From: Timothee Guerin<EMAIL_ADDRESS>Sent: Tuesday, June 11, 2019 1:07 PM
To: Azure/batch-insights<EMAIL_ADDRESS>Cc: Sharmaine Popov<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
Yes, you can query using the roleInstance property
https://github.com/Azure/batch-insights/wiki/Query-application-insights#basic-list-of-all-metrics-for-a-nodehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fwiki%2FQuery-application-insights%23basic-list-of-all-metrics-for-a-node&data=02|01|Sharmaine.Popov%40microsoft.com|a62d858e05714b4cc38808d6ee8f329e|72f988bf86f141af91ab2d7cd011db47|1|0|636958696135982526&sdata=5WMt4%2BVpoT3svmcMpgALA4OuhbujPxhwGJ3nN0XwZ%2B4%3D&reserved=0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWT7YKLIOO2DWFN4LTDPZ7LSXA5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXN22PQ%23issuecomment-500935998&data=02|01|Sharmaine.Popov%40microsoft.com|a62d858e05714b4cc38808d6ee8f329e|72f988bf86f141af91ab2d7cd011db47|1|0|636958696135982526&sdata=ow2Vx7T4QCiRMgS1JSS1yBqOxt%2BRGNXfoSZBk%2Beh1yo%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWVPPZ6GF33RIJAM3SLPZ7LSXANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|a62d858e05714b4cc38808d6ee8f329e|72f988bf86f141af91ab2d7cd011db47|1|0|636958696135982526&sdata=A3v%2Fy3iUd5JBn0TtvW%2BMgT5%2Bg8zERy5iKS46raiCQf8%3D&reserved=0.

You can check the config options here https://github.com/Azure/batch-insights/blob/master/docs/configuration.md
One is the local aggregation interval. By default it collect metrics every 5s, aggregate and uploads every minute(which is the minimum app insights will aggregate on. That way it reduce the amount of data uploaded to app insights and so the cost)

Thanks again. Any chance we can make the aggregation interval less than 1 minute - or otherwise get at the raw data?
My client is a research lab and they are going to be doing a bench mark paper, and they were hoping they could get the data the need via a cloud service rather than trying instrument the OS.
Sharmaine Popov
Cloud Solution Architect |Microsoft Canada Inc. | sharmaine.popov<EMAIL_ADDRESS>|<PHONE_NUMBER>
From: Timothee Guerin<EMAIL_ADDRESS>Sent: Tuesday, June 11, 2019 7:15 PM
To: Azure/batch-insights<EMAIL_ADDRESS>Cc: Sharmaine Popov<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
You can check the config options here https://github.com/Azure/batch-insights/blob/master/docs/configuration.mdhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fblob%2Fmaster%2Fdocs%2Fconfiguration.md&data=02|01|Sharmaine.Popov%40microsoft.com|1939d660dc94463053a608d6eec2a0d7|72f988bf86f141af91ab2d7cd011db47|1|0|636958917051368341&sdata=qRwRlxbN%2Bkx6mW2Iz9OBBFEiFP310boFehtaxRpUVLQ%3D&reserved=0
One is the local aggregation interval. By default it collect metrics every 5s, aggregate and uploads every minute(which is the minimum app insights will aggregate on. That way it reduce the amount of data uploaded to app insights and so the cost)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWURRBYOVL7ZMRUSODTP2AWXJA5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXOYVVA%23issuecomment-501058260&data=02|01|Sharmaine.Popov%40microsoft.com|1939d660dc94463053a608d6eec2a0d7|72f988bf86f141af91ab2d7cd011db47|1|0|636958917051368341&sdata=9u27buOetT7Zw%2B7oJ9LPoqDikjR5cbBKUVaK6VDTlss%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWXYRUFDXUO52HYC77TP2AWXJANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|1939d660dc94463053a608d6eec2a0d7|72f988bf86f141af91ab2d7cd011db47|1|0|636958917051524596&sdata=94C3PsGaDeVrGI5nwTMo28NDcNlHGmj2FQleeC2Rytc%3D&reserved=0.

Unfortunately even if batch-insights upload every 5s, I believe application insights will only aggregate the data in a minimum of a minute interval. You have to be carful with the cost of application insights which can become quite high with large pools if you upload too much data. You could always take this project and change it to upload to another  system(like graphana) and get the raw data this way

Ok – where would I start. Is this developer stuff?
Sharmaine Popov
Cloud Solution Architect |Microsoft Canada Inc. | sharmaine.popov<EMAIL_ADDRESS>|<PHONE_NUMBER>
From: Timothee Guerin<EMAIL_ADDRESS>Sent: Wednesday, June 12, 2019 11:11 AM
To: Azure/batch-insights<EMAIL_ADDRESS>Cc: Sharmaine Popov<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/batch-insights] No Metrics Showing (#39)
Unfortunately even if batch-insights upload every 5s, I believe application insights will only aggregate the data in a minimum of a minute interval. You have to be carful with the cost of application insights which can become quite high with large pools if you upload too much data. You could always take this project and change it to upload to another system(like graphana) and get the raw data this way
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fbatch-insights%2Fissues%2F39%3Femail_source%3Dnotifications%26email_token%3DAIIVNWRI6TGKB5QFC5PYZKTP2EGWXA5CNFSM4HW2AXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXQYEHQ%23issuecomment-501318174&data=02|01|Sharmaine.Popov%40microsoft.com|d45f58d18eae4787dd1808d6ef481f63|72f988bf86f141af91ab2d7cd011db47|1|0|636959490375791671&sdata=5NIxf2%2BHN%2F%2BRVHIq9FaKgnVPDpa8N2LoQqqhfWz0TYU%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIIVNWT6GP43QGGGDTO63X3P2EGWXANCNFSM4HW2AXFA&data=02|01|Sharmaine.Popov%40microsoft.com|d45f58d18eae4787dd1808d6ef481f63|72f988bf86f141af91ab2d7cd011db47|1|0|636959490375791671&sdata=Ey%2FBB7LqqL29oeA9wd93yTZiKGh43QFSjpOBAcbUEcg%3D&reserved=0.

Yes you would need to fork the project and implement the data egress for another system than application insights.


HHH-11147 (5.3)

https://hibernate.atlassian.net/browse/HHH-11147
Supersedes: https://github.com/hibernate/hibernate-orm/pull/2867
Backport of https://github.com/hibernate/hibernate-orm/pull/2846 to 5.3

Superseded by https://github.com/hibernate/hibernate-orm/pull/2905


Update backup_restore.md referencing matching PSQL versions

Added note in Restore section to state that PSQL backup versions must match restore destination versions. Close #3518 when merged.

Thats not true, postgres build in function can cause errors, thats why we use our backup solution. This is the only way to update to new postgres version without issues. Backup, update postgres and restore.


[Installer not starting]

So i wanted to repair BetterDiscord since it wasn't launching with discord anymore and downloaded the installer. But i cannot even launch it, it just doesn't want to.
I just need to double click on the installer and it doesn't even start. There is the loading logo but it never launched
(I also checked if it would appear on the task manager and it doesn't).
So is there a way to fix that?

Discord version: Stable

Have a look at my comment on https://github.com/BetterDiscord/Installer/issues/113.

Please see the above. That is the location for installer issues as well.


python3 not supported

Hello, i have an issue with synolopy using python3
You are using Queue but in python3 this module has been rename "queue"
it's could be useful to migrate from python2 to python3 and thus changing Queue in queue

Hi @D3m0t3p. You're right, Python 3 isn't supported yet.
Not sure there is a lot to do to make it compatible though. I'll work on that.

@thavel any news for the availble version for python3 ?


How to preserve ordering

Category

[x] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

I am wrting a litte tool https://github.com/mudox/tav
I want to perserve the item ordering when filtering, but specifying --tiebreak=index seems of no use.
echo 'xxxaaa
aaa
a' | fzf --tiebreak=index

gives me
aaa
a
xxxaaa

with input a

The result is correct, 1. fzf ranks aaa and a higher than xxxaaa, 2. aaa and a are tied, 3. and aaa comes before a.
You can completely turn off sorting with --no-sort.

@junegunn thank you for the quick reply.
Is it possbile to set the index as the  highest ranking criteria.
The position relationship realy matters in my tool.

Is it possbile to set the index as the highest ranking criteria.

Isn't that what --no-sort does?

My bad, not perform a clean build.
-not-sort works perfectly


Magic Auth not working on Microsoft Edge, OK on Chrome/Firefox/Safari/Vivaldi

✅ Prerequisites

[ ] Did you perform a cursory search of open issues? Is this bug already reported elsewhere?
[ ] Are you running the latest SDK version?
[ ] Are you reporting to the correct repository (magic-sdk)?

🐛 Description
Hey Magic Team,
We have a very standard flow in our app that we use to sign users in using a magic link. It currently works for users on chrome/safari/vivaldi/firefox. Recently we onboarded a customer who uses edge, and they have been having issues.
The observed pattern in Edge is that after entering their email, verifying from their email, our web app moves them to a new page(the cookies get set), but then throws the following errors from magic.link:

The payload of the failed HTTP requests in the screenshot are the following:
{"data":{},"error_code":"auth_relayer/UNABLE_TO_REFRESH_SESSION","message":"Unable to refresh user session. Please log in using the magic link flow to enable session persistence.","status":"failed"}
{"data":{},"error_code":"AUTH_PASSWORDLESS_LOGIN_NOT_VERIFIED","message":"Passwordless login is not verified yet","status":"failed"}
if the user refreshes, then everything works alright, but after they login & are redirected, but before they have refreshed the page, the authentication fails. We again only see this on Edge.
With some more testing realized the following, everything works if the Edge user has basic mode selected, but if they have the middle or right mode, it doesn't work

edge://settings/privacy (or type this into edge)
🧩 Steps to Reproduce

[First Step]
[Second Step]
[and so on...]

🤔 Expected behavior
[What you expected to happen?]
😮 Actual behavior
[What actually happened? Please include any error stack traces you encounter.]
💻 Code Sample
[If possible, please provide a code repository, gist, code snippet or sample files to reproduce the issue.]
🌎 Environment

Software
Version(s)

magic-sdk
8.1.1

Browser
Edge Version 110.0.1587.41 (Official build) (x86_64)

yarn
8.15.0 (NPM) 1.22.10 (YARN)

Operating System
MacOS, although bug is happening on Windows

Hello,
Thank you for posting this issue.
The errors and warning you shared are expected when the user is not yet logged in. Checking for a valid session when there isn't one will produce the 400 on the session/refresh endpoint because a session doesn't exist. The warnings are fine, you can't refresh the session because there isn't a valid session.
The final series of errors logged to the console are from polling the status endpoint while the app waits for the user to finish authentication. That means the user already received the login email with a magic link or OTP and the app is waiting for the user to enter the OTP or click the magic link.
By the way, I tested out session persistence on Microsoft Edge and sessions persisted normally.


kubeconfig can be specified dynamically as a file

This PR should not be merged. It's just to track the changes we have on top of linkyard/concourse-helm-resource. There's a linkyard/concourse-helm-resource#46 to add this upstream, but it's unlikely to get merged.
Signed-off-by: Dan Wendorf<EMAIL_ADDRESS>

I've also moved our previous changes to this repo to https://github.com/opendoor-labs/concourse-helm-resource/tree/old-tweaks and force-pushed master to track upstream master.


add option for Context Specific spacebar behavior addressing issue #925

Added an additional option to the spacebar behavior pref panes as detailed here https://github.com/quicksilver/Quicksilver/issues/925

Hey guys, any chance of this PR making it into an upcoming release?

100% chance, but I don’t know about timing. I’m hoping to take a week off just to work on QS this summer.

Merged this by hand. It didn’t close, I think because I didn’t have the last commit, but we can live without that.

Cool, does that mean this will make an appearance in the next release?

Yes! Thanks for the contribution.
I added some enhancements that I thought I had a pull request for, but apparently, I never submitted it.

I’m updating the credits and I see that we didn’t add you to the list of contributors, @brightOrigin. First, I’m sorry about that, but I’m going to add you now. How would you like to be listed? Your name isn’t attached to your GitHub profile.

Thx Rob.
Full name is Tony Papale.
On Oct 22, 2015, 6:17 AM -0700, Rob<EMAIL_ADDRESS>wrote:

I’m updating the credits and I see that we didn’t add you to the list of contributors,@brightOrigin(https://github.com/brightOrigin). First, I’m sorry about that, but I’m going to add you now. How would you like to be listed? Your name isn’t attached to your GitHub profile.
—
Reply to this email directly orview it on GitHub(https://github.com/quicksilver/Quicksilver/pull/1803#issuecomment-150220645).


Creating a project with the same name as an earlier project takes over the earlier projects awards

It seems to create a new token with the old project awards.
Checkout Citizen Code edgebot 3D Printer Drone design.

on  edgebot


Frostbalance

Automerge has a problem with this? Huh

This is being moved to a separate project, using BotManager as a library


Playground: Moving rectangle demo not working

The rectangle doesn't respond to keyboard input.

Weird

Which demo do you mean? The one in samples/rect allows you to move with the mouse.

I mean the web playground.

Ah

Seems to be a Sokol issue, if you use isPressedRepeatc instead of isPressedc it will work

Or rather an emscripten issue (https://github.com/emscripten-core/emscripten/pull/22201), I can't see why it would behave this way, I know that it sends key_repeat all the time, instead of just after the first time, which registers it incorrectly. I will make a temporary hot fix for this until they fix emscripten.

Can you reproduce it with a local build? I couldn't. Everything was OK when I built it and ran on localhost.

Can you reproduce it with a local build? I couldn't. Everything was OK when I built it and ran on localhost.

Yeah, it seems to be related to some recent changes in emscripten - https://github.com/emscripten-core/emscripten/blob/main/ChangeLog.md#3162---070224
I specified a version that should work.

I see. I have 3.1.56 on my laptop.

Downgrading worked.


Wind API is bad

This API convention doesn't follow any of the React, metaversefile, or Webaverse conventions:
  getWinds() {
    return world.winds;
  },
  setWinds(wind) {
    world.winds.push(wind);
  },

This probably requires totum updates.


Feat(MOSSMVP-247): Set up backend theming logic

Change the color value format for theme files. Now it support various various formats like:

"primary": "rgba(0, 0, 0, 1)"
"sideBar.background": "#000000"
"toolBar.background": "hsl(240, 100%, 50%)"

They will be parsed on the backend, and the frontend will receive the corresponding rgba value.
2. Move the parsing and validation logic for theme files to the backend. Now the fetch_all_themes command will iterate over the theme directory and find all .json file, and read_theme command will return a parsed Theme object instead of a raw string.
3. Refactor uses of the the manually implemented Theme and Colors type in moss-models to the auto-exported type in desktop-models. This ensures that the typings are always kept in sync between the frontend and backend.

pls, name the pull request in accordance with the established naming conventions, and the PR title should match the ticket title


Style _includes/fundamentals/section_toc.liquid

Should look something similar to related_guides.liquid but needs to fit at the bottom of each section.

Just committed this in https://github.com/google/WebFundamentals/commit/f63475294f4e138d68070b2798c6f32214248033
Not sure what to put as the title (If I should put a title at all).
I've also had to drop the section heading from related guides because it doesn't make much sense for section table of contents (it'll be the same everywhere).

Looks good!


Key Binding Firing On Other pages

The following JavaScript file is used for key binding:
document.addEventListener('keydown', function(event) {
  // Check for a specific key, e.g., 'G' key (case-insensitive)
  if (event.key === 'c' || event.key === 'C') {
    window.location.href = 'https://github.com/LineIndent/buridan-ui'; // Redirect to your desired URL
  }
  if (event.key === 'x' || event.key === 'X') {
    window.location.href = 'https://github.com/reflex-dev/reflex'; // Redirect to your desired URL
  }
});

The issue is the script fires OK on landing page, but it also fires on other pages as well.
Either remove key bindings all together or add two-key-binding using CMD + Key

Removed key binding functionality from site in v0.3.2.


Wish to grab Premium Version. Little Clarification Required

Hello Sir,
I want to buy your premium vr. but I am really keen to know how I will install the plugin after payment.
Secondly I want to know about the accuracy in android. Is it better than before? In this matter I want to inform you I am very new to mobile app development and workin for android app.

You install the plugin as you do any cordova plugin:
cordova plugin add
https://github.com/christocracy/cordova-background-geolocation.git
The github repo above is private, of course, with access granted after
payment.
The accuracy in Android is as good as iOS.
However, most important for Android is the improved battery performance
(5-10 times better).
The new Android version is better than iOS in its ability to very quickly
determine when device is moving since it implements the Android
ActivityDetection API.  This low-power api constantly samples the device’s
accelerometer & gyroscope and outputs an answer such as “STILL”, “RUNNING”,
“BICYCLE”, “IN_VEHICLE”, “WALKING”.  The sample-rate of this system can be
configured (every second, 10s, 30s, etc).
With iOS, the device must move ~3 blocks beyond its stationary position
before it turns the GPS on.  With Android, it takes just several meters.
Similary for stop-detection, iOS takes exactly 15min to determine device
has stopped — Android can detect you’ve stopped within the sample-rate of
the ActivityDetection API (e.g, 10s, 30s, etc).
On Fri, Mar 20, 2015 at 10:12 AM, riddhimcc<EMAIL_ADDRESS>wrote:

Hello Sir,
I want to buy your premium vr. but I am really keen to know how I will
install the plugin after payment.
Secondly I want to know about the accuracy in android. Is it better than
before? In this matter I want to inform you I am very new to mobile app
development and workin for android app.
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/156
.

--
Chris Scott
Transistor Software http://www.transistorsoft.com

Thanks again for your reply,
Is it possible to do some background geolocalisation with your plugin when user force-close the application ?
On ios, «  privacy > location services » , I see « allow location access » set to « always » when I use your plugin. Does it means you handle geolocation refresh if app is force-closed (not in background) ?
But I don’t see it in « general > background app refresh »
Is there an equivalence on Android ?
—
David
--
David Barré
Le vendredi 20 mars 2015 à 15:37, Chris Scott a écrit :

You install the plugin as you do any cordova plugin:
cordova plugin add
https://github.com/christocracy/cordova-background-geolocation.git
The github repo above is private, of course, with access granted after
payment.
The accuracy in Android is as good as iOS.
However, most important for Android is the improved battery performance
(5-10 times better).
The new Android version is better than iOS in its ability to very quickly
determine when device is moving since it implements the Android
ActivityDetection API. This low-power api constantly samples the device’s
accelerometer & gyroscope and outputs an answer such as “STILL”, “RUNNING”,
“BICYCLE”, “IN_VEHICLE”, “WALKING”. The sample-rate of this system can be
configured (every second, 10s, 30s, etc).
With iOS, the device must move ~3 blocks beyond its stationary position
before it turns the GPS on. With Android, it takes just several meters.
Similary for stop-detection, iOS takes exactly 15min to determine device
has stopped — Android can detect you’ve stopped within the sample-rate of
the ActivityDetection API (e.g, 10s, 30s, etc).
On Fri, Mar 20, 2015 at 10:12 AM, riddhimcc<EMAIL_ADDRESS>(mailto:notifications@github.com)>
wrote:

Hello Sir,
I want to buy your premium vr. but I am really keen to know how I will
install the plugin after payment.
Secondly I want to know about the accuracy in android. Is it better than
before? In this matter I want to inform you I am very new to mobile app
development and workin for android app.
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/156
.

--
Chris Scott
Transistor Software http://www.transistorsoft.com
—
Reply to this email directly or view it on GitHub (https://github.com/christocracy/cordova-plugin-background-geolocation/issues/156#issuecomment-84033366).

If user force-closes the app while the plugin is enabled, the app will be
auto-booted again.

But I don’t see it in « general > background app refresh »

Background app refresh is a different thing
http://www.scottyloveless.com/blog/2014/background-app-refresh-explained,
nothing to do with geolocation.
I made a separate plugin
https://github.com/christocracy/cordova-plugin-background-fetch
implementing the “background refresh api”.
Background Refresh is an iso-only system that awakes your app about every
15min (this interval is modified by iOS automatically based upon the user’s
behaviour (e.g. when the user is presumed to be sleeping,
background-refresh interval might occur every hour instead).
With my background-fetch plugin, when a background-refresh event occurs, my
plugin will execute your configured javascript callback — you have exactly
30s of running time before iOS suspends your app again.
On Fri, Mar 20, 2015 at 12:38 PM, devteamshippeo<EMAIL_ADDRESS>wrote:

Thanks again for your reply,
Is it possible to do some background geolocalisation with your plugin when
user force-close the application ?
On ios, « privacy > location services » , I see « allow location access »
set to « always » when I use your plugin. Does it means you handle
geolocation refresh if app is force-closed (not in background) ?
But I don’t see it in « general > background app refresh »
Is there an equivalence on Android ?
—
David
--
David Barré
Le vendredi 20 mars 2015 à 15:37, Chris Scott a écrit :

You install the plugin as you do any cordova plugin:
cordova plugin add
https://github.com/christocracy/cordova-background-geolocation.git
The github repo above is private, of course, with access granted after
payment.
The accuracy in Android is as good as iOS.
However, most important for Android is the improved battery performance
(5-10 times better).
The new Android version is better than iOS in its ability to very quickly
determine when device is moving since it implements the Android
ActivityDetection API. This low-power api constantly samples the device’s
accelerometer & gyroscope and outputs an answer such as “STILL”,
“RUNNING”,
“BICYCLE”, “IN_VEHICLE”, “WALKING”. The sample-rate of this system can be
configured (every second, 10s, 30s, etc).
With iOS, the device must move ~3 blocks beyond its stationary position
before it turns the GPS on. With Android, it takes just several meters.
Similary for stop-detection, iOS takes exactly 15min to determine device
has stopped — Android can detect you’ve stopped within the sample-rate of
the ActivityDetection API (e.g, 10s, 30s, etc).
On Fri, Mar 20, 2015 at 10:12 AM, riddhimcc<EMAIL_ADDRESS>(mailto:notifications@github.com)>
wrote:

Hello Sir,
I want to buy your premium vr. but I am really keen to know how I will
install the plugin after payment.
Secondly I want to know about the accuracy in android. Is it better
than
before? In this matter I want to inform you I am very new to mobile app
development and workin for android app.
—
Reply to this email directly or view it on GitHub
<
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/156

.

--
Chris Scott
Transistor Software http://www.transistorsoft.com
—
Reply to this email directly or view it on GitHub (
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/156#issuecomment-84033366
).

—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/156#issuecomment-84064199
.

--
Chris Scott
Transistor Software http://www.transistorsoft.com

Before I buy would like to clear the following points:
I have used your plugin (free vr.) to develop an android app to track user silently from background.
A. I find that when the device is stationary the app is returning co-ordinates which ideally it should not.
B. Some times the co-ordinates returned by the app is fluctuating within a range of few Kms. Same thing is happening when the device is stationary.
C. The frequency in wchich co-ordinates are being returned by the app is varrying drastically. SOme times in a minute I am getting 3 - 4 co ordinates and sometimes one co-ordinate in 15 minutes. Same thing is happening when the device is stationary or moving.
D. I developed the app in Cordova 2.9.1. Is your Premium Vr. compatible with this cordova vr.?
E. After buying Plugin, in case I face any problem, will I get Support from your end?
F. I have free account in GitHub, will I still use your Premium Plugin after purchase?

Please find below the code what I have  developed for my app. Kindly Suggest whether code is ok or I am missing out something due to which correct position are not being returned.

window.navigator.geolocation.getCurrentPosition(onFrontGeoSuccess, onFrontGeoFail);
var bgGeo = window.plugins.backgroundGeoLocation;
bgGeo.configure(onBackGeoSuccess, onBackGeoFail, {
url: 'http://myurl/BackgroundLocationTrack.aspx',//''http://only.for.android.com/update_location.json', // <-- Android ONLY:  your server url to send locations to
params: {
userId: localStorage.getItem("userId"),
uuid: device.uuid    //  <-- Android ONLY:  HTTP POST params sent to your server when persisting locations.
//, foo: 'bar'                              //  <-- Android ONLY:  HTTP POST params sent to your server when persisting locations.
},
headers: {                                   // <-- Android ONLY:  Optional HTTP headers sent to your configured #url when persisting locations
"X-Foo": "BAR"
},
        desiredAccuracy: 0,
        stationaryRadius: 20,
        distanceFilter: 30,
		notificationTitle: 'Checking Background tracking', // <-- android only, customize the title of the notification
        notificationText: 'ENABLED.....', // <-- android only, customize the text of the notification
		//activityType: 'AutomotiveNavigation',
		debug: false, // enable this hear sounds for background-geolocation life-cycle.
		stopOnTerminate: false // enable this to clear background location settings when the app terminates
	});
	var onBackGeoSuccess = function(location) {
		console.log('[js] BackgroundGeoLocation callback:  ' + location.latitude + ',' + location.longitude);
		// Do your HTTP request here to POST location to your server.
		//
		// Ajax Call
		//
		//ajaxCallback.call(this);
	};
	var onBackGeoFail = function(error) {
		//console.log('BackgroundGeoLocation error');
		//bgGeo.finish();
	}

	// Turn ON the background-geolocation system.  The user will be tracked whenever they suspend the app.
	bgGeo.start();

	// bgGeo.stop();

	function ajaxCallback(){
		// on Server update success or failure, run the following

		//bgGeo.finish();
	}

A. I find that when the device is stationary the app is returning co-ordinates which ideally it should not.
Premium version has completely refactored Android.  When Android is
stationary it doesn’t sample the GPS anymore.  Premium version uses
ActivityRecognition API to determine when device is moving.
ActivityRecognition API uses accelerometer & gyroscope.  Problem solved.

B. Some times the co-ordinates returned by the app is fluctuating within a range of few Kms. Same thing is happening when the device is stationary.
Solved as in previous.

C. The frequency in wchich co-ordinates are being returned by the app is varrying drastically. SOme times in a minute I am getting 3 - 4 co ordinates and sometimes one co-ordinate in 15 minutes. Same thing is happening when the device is stationary or moving.
Android has been completely refactored to use Android
FusedLocationProvider
http://www.kpbird.com/2013/06/fused-location-provider-example.htmlfrom
Google Play Services, a more efficient and advanced LocationProvider than
in the free-version.

D. I developed the app in Cordova 2.9.1. Is your Premium Vr. compatible with this cordova vr.?
You need to upgrade to 3.x.  The new Cordova plugin system, upon which my
plugin is based, wasn’t released until 3.0.

E. After buying Plugin, in case I face any problem, will I get Support from your end?
You will have access to my private repo along with all the other paying
customers.  You may post issues as normal.

F. I have free account in GitHub, will I still use your Premium Plugin after purchase?
My checkout system, after completing the transaction, will automatically
add your github user to my private repo.

Please find below the code what I have  developed for my app. Kindly Suggest whether code is ok or I am missing out something due to which correct position are not being returned.
If you want me to look at code, please ensure it’s formatted properly.
Post a gist https://gist.github.com at least.

window.navigator.geolocation.getCurrentPosition(onFrontGeoSuccess,
onFrontGeoFail);
var bgGeo = window.plugins.backgroundGeoLocation;
bgGeo.configure(onBackGeoSuccess, onBackGeoFail, {
url: 'http://myurl/BackgroundLocationTrack.aspx',//''
http://only.for.android.com/update_location.json', // <-- Android ONLY:
your server url to send locations to
params: {
userId: localStorage.getItem("userId"),
uuid: device.uuid // <-- Android ONLY: HTTP POST params sent to your
server when persisting locations.
//, foo: 'bar' // <-- Android ONLY: HTTP POST params sent to your server
when persisting locations.
},
headers: { // <-- Android ONLY: Optional HTTP headers sent to your
configured #url when persisting locations
"X-Foo": "BAR"
},
    desiredAccuracy: 0,
    stationaryRadius: 20,
    distanceFilter: 30,
    notificationTitle: 'Checking Background tracking', // <-- android only, customize the title of the notification
    notificationText: 'ENABLED.....', // <-- android only, customize the text of the notification
    //activityType: 'AutomotiveNavigation',
    debug: false, // enable this hear sounds for background-geolocation life-cycle.
    stopOnTerminate: false // enable this to clear background location settings when the app terminates
});
var onBackGeoSuccess = function(location) {
    console.log('[js] BackgroundGeoLocation callback:  ' + location.latitude + ',' + location.longitude);
    // Do your HTTP request here to POST location to your server.
    //
    // Ajax Call
    //
    //ajaxCallback.call(this);
};
var onBackGeoFail = function(error) {
    //console.log('BackgroundGeoLocation error');
    //bgGeo.finish();
}

// Turn ON the background-geolocation system.  The user will be tracked whenever they suspend the app.
bgGeo.start();

// bgGeo.stop();

function ajaxCallback(){
    // on Server update success or failure, run the following

    //bgGeo.finish();
}

—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/156#issuecomment-84263046
.

--
Chris Scott
Transistor Software http://www.transistorsoft.com


The translation of theme 'Light' is not correct under View menu

Storage Explorer Version: 1.10.1
Build: 20191012.1
Branch: master
Platform/OS: Windows 10/ Linux Ubuntu 19.04/macOS High Sierra
Language: Chinese(zh-TW)
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Launch Storage Explorer.
Open Settings panel -> Regional Settings -> Select Chinese (traditional) as application language -> Restart Storage Explorer.
Click the localized View menu -> Click the localized Theme.
Check the translation of theme 'Light'.

Expect Experience:
The theme 'Light' is translated to ‘淺色'.
Actual Experience:
The theme 'Light' is translated to '光'.

More Info:

This issue doesn't reproduce in Chinese(zh-CN).
This issue doesn't reproduce on Setting panel.

On review.
File: loc\en\src\Standalone\resources\Shell.json
ID: 0; "MenuThemeLight"
SRC: Light
TGT: 光

This issue has been fixed on build 20191017.3. So we close it.


about code config

Hello, inside the code implementation, there is a parameter that is the point cloud data range, x_min=0, that is, only the target in front of the car is detected. Can you set x_min<0, so that the network can detect all targets around the car?
change  point_cloud_range:[0, -39.68, -3, 69.12, 39.68, 1] to [-69.12, -39.68,-3, 69.12, 39.68, 1]

Hello, inside the code implementation, there is a parameter that is the point cloud data range, x_min=0, that is, only the target in front of the car is detected. Can you set x_min<0, so that the network can detect all targets around the car?
change point_cloud_range:[0, -39.68, -3, 69.12, 39.68, 1] to [-69.


Fix Automatic Download

If the installation can't find the MSI package, then should the installation automatic try to download the package from IBM's FTP Site.

Start working on this part, but need to solve #15 first to make sure we have the download links in JSON format.


cann't find plugin from gradle plugin repo

searched 'com.ms.gradle.application' from https://plugins.gradle.org/ but seems nothing's there.

Correct, the plugin has not yet been published to Gradle Plugin Portal. I'm currently working on some pre-requisite administrative tasks.

Correct, the plugin has not yet been published to Gradle Plugin Portal. I'm currently working on some pre-requisite administrative tasks.

I just see a new release, is it published to the repo now?

It's pending approval by the maintainers of Gradle Plugin Portal.
See: https://blog.gradle.org/plugin-portal-validations

It is now available on Gradle Plugin Portal, just click the badge at the top of the main README. :)


WOTLK - Error when loading the addon on a druid

Seems to happen with the scripted outfits being enabled:

I can't see enough of the error to really know if it's fixed, but I just realesed <IP_ADDRESS> with a lot of updates.


[YUNIKORN-2586] Set GOROOT to avoid tool failure when using newer go compilers

What is this PR for?
Compiling with newer go compilers (tested on go 1.22.2) fails when generating 3rd party licenses. This was traced back to GOROOT not being set. Set it in the Makefile to avoid this issue.
What type of PR is it?

[x] - Bug Fix
[ ] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2586
How should this be tested?
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

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

Project coverage is 66.07%. Comparing base (db33f10) to head (07cbdb3).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #827      +/-   ##
==========================================
- Coverage   66.09%   66.07%   -0.03%
==========================================
  Files          69       69
  Lines        7522     7522
==========================================
- Hits         4972     4970       -2
- Misses       2339     2340       +1
- Partials      211      212       +1

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

@craigcondit just curios. how to reproduce the error? There is no GOROOT and then I run make clean && make go-license-generate. It works well.

Not sure but it's 100% reproducible for me without the patch.

Apparently go-license sometimes has difficulty detecting GOROOT properly under certain cases. There an open issue on it, but the symptom is that it misdetects standard library packages as non-modules and breaks the build.


Bug in bundling resources of private npm packages (containing @)

I'm submitting a bug report

Library Version:
0.5.0

Please tell us about your environment:

Operating System:
Windows 10

Node Version:
6.2.0

NPM Version:
3.8.9

JSPM OR Webpack AND Version
JSPM 0.16.47

Language:
ESNext

Current behavior:
adding resource of private npm package to bundle configuration, for example:
[
    @example/my-package,
    @example/my-package/my.resource.css!text   // this line throws the error
]

throws exception: "Error: A version conflict was found among the module names specified in the configuration for..."
The error is thrown because the getFullModuleName does not match correctly private packages.
First the package tries to exact match, so this works for something like @example/my-package, but if you try to add a resource like @example/my-package/my.resource.css!text, exact match wont work and the cleanName function is used to clean the module name.
The result of running the cleanName function on @example/my-package/my.resource.css!text is empty string and no match is found.
To reproduce, from command line:
node -e "console.log('@example/package/my.resource.css!text'.replace(/^.*:/<EMAIL_ADDRESS>''));"

What is the expected behavior?

@example/my-package/my.resource.css!text should match @example/my-package module.

What is the motivation / use case for changing the behavior?

bundling of private npm repositories with resources.
Consider a repo that contains number of css styles (themes) in addition to the js code.
The user of the repo want's to chose the required style for his project and bundle it as part of the application.

@genadis I am not sure how a private npm package is different here. It should work either way. Can you create a map in config.js (systemjs config file) for the private package you are using and use that in bundle config and see how that goes?

@ahmedshuhel Thank you for quick response
I already did (actually I'm giving jspm install do the dirty work, but usually check the map: [] of the generated config.js for correctness if there are issues)
Below I've posted the configuration I'm using. But first allow me to point out that I have the same package in the public npm called encapsulated-mdl which works just fine. And the private package when configured with bitbucket worked for me as well. The issue started when I switched to private npm.
For example in my map I have the following:
map: [
"@annoto/mdl": "npm<EMAIL_ADDRESS> //...
]

@annoto/mdl has no dependencies so it's the only record in the map.
In the project I have a directory for the package:
jspm_packages
   -- npm
      -- @annoto
         --<EMAIL_ADDRESS>           -- material.js
           -- material.cyan-blue.min.css
           -- material.deep_orange-pink.min.css
           -- ....

When the bundle configuration includes only this:
"includes": [
        "@annoto/mdl",
        // ...
]

(I have the package.json of the @annoto/mdl configured correctly to point to material.js as the main)
All is good because the getFullModuleName function of the bundler has an exact match of the module here: Line 189.
When I try adding a css:
"includes": [
        "@annoto/mdl",
        "@annoto/mdl/material.cyan-blue.min.css!text",
        // ...
]

I get the mentioned error. As I've pointed out, I believe it is because of the cleanName let cleanName = (n: string) => n.replace(/^.*:/<EMAIL_ADDRESS>''); on Line 188.
I understand it's purpose is to clean the versions of the module which use the @ character, but as a side effect it deletes the private module name as well as it starts with @ returning empty string for all private modules. It should recognize that if @ is the first character it is private module and not a version indicator and not clean it.
Thanks!

@ahmedshuhel @EisenbergEffect is there any update on this?
If it's not a bug, and there is issue with the configuration I've posted above I would really appreciate pointers to solving it. Thank you in advance!
It really blocks us from using private npm with aurelia-bundler, for the time being we are using bitbucket.

I will look into it and let you know.On Dec 31, 2016 6:57 PM, Genadi<EMAIL_ADDRESS>wrote:@ahmedshuhel @EisenbergEffect is there any update on this?
If it's not a bug, and there is issue with the configuration I've posted above I would really appreciate pointers to solving it. Thank you in advance!
It really blocks us from using private npm with aurelia-bundler, for the time being we are using bitbucket.
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.

Hi guys, I have this same issue.
With my config as....
"@timbailey/control-dashboard",
"@timbailey/control-dashboard/dashboard.html!text"

The message I get is:
Error: A version conflict was found among the module names specified in the configuration for '@timbailey/control-dashboard/dashboard.html!text'. Try including a full module name with a specific
version number or resolve the conflict manually with jspm.
    at getFullModuleName (C:\Projects\timbailey\Source\myApp\node_modules\aurelia-bundler\dist\bundler.js:232:9)
    at C:\Projects\timbailey\Source\myApp\node_modules\aurelia-bundler\dist\bundler.js:55:12
    at Array.map (native)
    at createBuildExpression (C:\Projects\timbailey\Source\myApp\node_modules\aurelia-bundler\dist\bundler.js:54:40)
    at Object.bundle (C:\Projects\timbailey\Source\myApp\node_modules\aurelia-bundler\dist\bundler.js:109:25)
    at _bundle (C:\Projects\timbailey\Source\myApp\node_modules\aurelia-bundler\dist\index.js:78:18)
    at C:\Projects\timbailey\Source\myApp\node_modules\aurelia-bundler\dist\index.js:53:18
    at Array.forEach (native)
    at Object.bundle (C:\Projects\timbailey\Source\myApp\node_modules\aurelia-bundler\dist\index.js:46:24)
    at Gulp.<anonymous> (C:\Projects\timbailey\Source\myApp\build\tasks\bundle.js:13:18)
    at module.exports (C:\Projects\timbailey\Source\myApp\node_modules\orchestrator\lib\runTask.js:34:7)
    at Gulp.Orchestrator._runTask (C:\Projects\timbailey\Source\myApp\node_modules\orchestrator\index.js:273:3)
    at Gulp.Orchestrator._runStep (C:\Projects\timbailey\Source\myApp\node_modules\orchestrator\index.js:214:10)
    at C:\Projects\timbailey\Source\myApp\node_modules\orchestrator\index.js:279:18
    at finish (C:\Projects\timbailey\Source\myApp\node_modules\orchestrator\lib\runTask.js:21:8)
    at cb (C:\Projects\timbailey\Source\myApp\node_modules\orchestrator\lib\runTask.js:29:3)
    at finish (C:\Projects\timbailey\Source\myApp\node_modules\run-sequence\index.js:60:5)
    at runNextSet (C:\Projects\timbailey\Source\myApp\node_modules\run-sequence\index.js:88:5)
    at Gulp.onTaskEnd (C:\Projects\timbailey\Source\myApp\node_modules\run-sequence\index.js:75:5)
    at emitOne (events.js:101:20)
    at Gulp.emit (events.js:188:7)
    at Gulp.Orchestrator._emitTaskDone (C:\Projects\timbailey\Source\myApp\node_modules\orchestrator\index.js:264:8)
    at C:\Projects\timbailey\Source\myApp\node_modules\orchestrator\index.js:275:23
    at finish (C:\Projects\timbailey\Source\myApp\node_modules\orchestrator\lib\runTask.js:21:8)
    at C:\Projects\timbailey\Source\myApp\node_modules\orchestrator\lib\runTask.js:52:4
    at f (C:\Projects\timbailey\Source\myApp\node_modules\end-of-stream\node_modules\once\once.js:17:25)
    at DestroyableTransform.onend (C:\Projects\timbailey\Source\myApp\node_modules\end-of-stream\index.js:31:18)
    at emitNone (events.js:91:20)

As its complaining about "versions" I think that the leading @ is causing the "startswith" version check to trigger.
Thanks

Further to my comment, I think I've resolved with a change to the "cleanName" function....
export function getFullModuleName(moduleName: string, map: any) {
  let cleanName = (n: string) => {

    // strip leading 'registry' prefixes
    let result = n.replace(/^.*:/, '');

    // strip trailing version info
    if (result.charAt(0) === '@') {
      result = '@' +<EMAIL_ADDRESS>'');
    } else {
      result =<EMAIL_ADDRESS>'');
    }

    return result;
  }
...
...

There might be a neater way to handle this using regexes?
I can submit a PR if required.

@ahmedshuhel any updates on this. seems like @TimBailey-pnk posted the solution (Thanks! by the way)

Am I right this issue can now be closed thanks to the linked PR?

I believe so.


Method GuzzleHttp\Stream\Stream::__toString() must not throw an exception

At some point between request creation and actually sending the request my resource becomes stale. Every step shows the type as a stream until the __toString call the type is unknown throwing an exception when performing stream operations.

Ah, I see. It's happening when my signature subscriber clones the stream.  What is a good way to get the stream contents to hash the body on a stream that is not seekable? Wrap the stream get contents in stream factory if it is not and set the body to the new stream?

What do you mean by "the type is unknown"? Is there an exception that's thrown or a warning message?

On Apr 1, 2015, at 9:32 PM, Steve Buzonas<EMAIL_ADDRESS>wrote:
At some point between request creation and actually sending the request my resource becomes stale. Every step shows the type as a stream until the __toString call the type is unknown throwing an exception when performing stream operations.
—
Reply to this email directly or view it on GitHub.

var_dump($this->stream) tells me resource, the identifier, and tells me stream... after I clone it var_dump($this->stream) tells me resource, identifier doesn't change, but the type says Unknown  I'm getting an exception that stems from the stream operations on the resource. I haven't checked to see if it was an error translated to an exception.

What is a good way to get the stream contents to hash the body on a stream that is not seekable?

Reading from a stream that is not seekable will consume the stream such that you cannot subsequently read from the stream (i.e., reading to hash the stream will prevent you from reading from the stream to send it in an HTTP request).
You'll need to either load the stream into memory or decorate the stream with a CachingStream so that previously read bytes are stored in a buffer and recalled later if necessary: https://github.com/guzzle/streams/blob/master/src/CachingStream.php

Ah, decorating it works. Thank you.


Refresh DEVELOPER.md

Refresh DEVELOPER.md to make it more clear.

TOC contents reorg
Updated headings
Improved command description and results
rm $ from each command for copy/paste
general copy editing

Addressed all comments.


Error when behind corporate firewall

This seems to fail when behind a corporate firewall.

Same problem here. We need some way to configure json schema to use proxy. Maybe grab the one from npm / apm settings?

+1

+1

+1


[FEATURE] Add possibility to run in alpine container as docker in docker

It maybe me, but i was unable to run divy under alpine container. Using strace it looks like it is missing something.
Is there a possibility to add package for alpine to releases page, or, there is something wrong with alpine based container and to run divy there I am missing some packages?
My Dockerfile is quite simple
FROM alpine:latest
ENV KUBE_LATEST_VERSION=v1.15.1
ENV HELM_VERSION=v2.14.2
ENV HELM_FILENAME=helm-${HELM_VERSION}-linux-amd64.tar.gz
LABEL version="v1.3.4"
RUN apk add --no-cache --update -t deps ca-certificates curl bash gettext tar gzip openssl python python-dev py-pip && pip install yamllint \
&& curl -L https://storage.googleapis.com/kubernetes-release/release/${KUBE_LATEST_VERSION}/bin/linux/amd64/kubectl -o /bin/kubectl && chmod +x /bin/kubectl \
&& curl -L https://storage.googleapis.com/kubernetes-helm/${HELM_FILENAME} | tar xz && mv linux-amd64/helm /bin/helm && rm -rf linux-amd64 && chmod +x /bin/helm
CMD busybox

Look for this change in the upcoming v0.8.0 release


update - from patch to fully replace

Delete prepareItemForUpdate and use prepareItemForInsert instead.
For collection with this schema: _id, fieldA, fieldB, update request with this item:
{_id: '..', fieldA: 'whatEver'}, will become in prepareItemsForInsert to :
{_id: '.., fieldA: 'whatEver', fieldB: null}
Until now, this kind of request would only update the fields that the object contains and not override the others.
fixing this issue: https://github.com/wix-private/auto-views/issues/7095

Will merge after merging #393 to avoid conflicts.


[SR-6671] Fix Decimal implementation of FloatingPoint constants

This PR is a reprise of #1386, which has long ago gone stale. There are several issues with Decimal implementations of FloatingPoint constants:
First, as reported in SR-6671,  the leastNormalMagnitude and the leastNonzeroMagnitude actually aren't. This is corrected in both the overlay and in swift-corelibs, with amendments to the corresponding tests.
Second, as pointed out by @stephentyrone in #1386, the implementation of leastFiniteMagnitude is incorrect. (It is also not a constant required by FloatingPoint, but with API stability it's too late now to remove from Decimal's API. It is left to the code owners as a separate matter whether they wish to deprecate this API.) Apple's documentation of the API states that it is "the decimal that contains the smallest possible non-infinite magnitude for the underlying representation." Magnitude is never negative, and the correct value here is zero. In this PR we make this API an explicit synonym for .zero. This is corrected in both the overlay and in swift-corelibs, with amendments to the corresponding tests.

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test macOS

@swift-ci test

@swift-ci test

(Just back from vacation, sorry for delay in reading this)
I'm worried about changing the value of leastFiniteMagnitude introducing subtle bugs to existing code that "works" with the current (incorrect) value. My advice would be instead to deprecate the property while keeping its value as is.

@stephentyrone Yeah, I as well. On the other hand, the value of the property is currently documented in a way that emphasizes its "magnitudeness" and there may be current code subtly broken relying on that. I think this should be deprecated regardless with a good error message, but it's not clear to me which value of the property is best in its deprecated state.

Default assumption should be that already-compiled code is happy with the existing behavior (wrong though it may be), and we shouldn't break them. New code will get a deprecation warning, so we can catch them there. @spevans, how do you want to handle this?

#3071


[BUG]

Failed to parse. Text: "
json
{
"action": "web-browser",
"action_input": "https://www.google.com/search?q=Fayette+Maine+store+phone+number&rlz=1C5CHFA_enUS747US747&source=lnms&tbm=isch&sa=X&ved=0ahUKEwiQ2Jvh0JDjAhUMwFkKHQEADQQ_AUIBigB&biw=1366&bih=657&dpr=2.34&sz=1280x800#imgrc=>_",
}". Error: SyntaxError: Expected double-quoted property name in JSON at position 262

It was the model


[Insiders] installing an extension via command line failure

Version: 1.28.0-insider (user setup)
Commit: fe8d9d2470ddffb9dc2da604b23f5bd8a185cd66
Date: 2018-09-25T05:31:33.193Z
Electron: 2.0.9
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64
OS: Windows 10
Steps to Reproduce:

Attempt to install the VS Live Share extension via the command line in insiders (our CI tests use this set up) fails with the following:

code-insiders --install-extension "C:\Users\jasonra.REDMOND\Desktop\vsliveshare-0.3.733-insiders.vsix"
diagnostic_channel_1.channel.unsubscribe is not a function
It installs successfully on stable:

code --install-extension "C:\Users\jasonra.REDMOND\Desktop\vsliveshare-0.3.733-insiders.vsix"
[createInstance] extensionManagementService depends on downloadService which is NOT registered.
Extension 'vsliveshare-0.3.733-insiders.vsix' was successfully installed!

Duplicate of #59212


Uncaught TypeError: Cannot read property 'destroy' of undefined(…) mediaelement-and-player.min.js:25

After I migrate the mediaelement from 2.16.4 to 3.0.0,the video can play,but the chrome browser console the error "Uncaught TypeError: Cannot read property 'destroy' of undefined(…) mediaelement-and-player.min.js:25 ".
is my use method error or is it a bug?
Thank you very much!

I initialize the meidaelement like this:
new MediaElementPlayer("#video-player", {
success: function (mediaElement, originalNode) {
videoPlayer = mediaElement;
videoPlayer.setSrc(videoUrl);
}
});

I replace the  mediaelement-and-player.min.js file with mediaelement-and-player.js,the error  printscreen is like this:

Please download the latest version; your error was fixed in 3.1.1

It solves my problem.Thank you very much!

You are welcome


Respond to "unclassified" messages

If users type a message back to greetbot, we should respond and inform them that they're typing to a bot.

Will need to do a bit more research here, but generally, I think we'll need to do the following:

Subscribe the bot_user to app_mention in the Slack /events API (See: https://api.slack.com/bot-users)
Add the incoming message for the bot to the generic method that parses /events in the CodeBuddies app
Add a method to respond to the message, based on it's flag or content or other criteria.  In this case, it would be an "unclassified" message - so maybe this can be set up as a default case in a switch statement that handles any messages addressed directly to the bot.
A basic bot reply will be required to open a websocket with the RTM API by sending an authenticated call to the rtm.connect API method (See:  https://api.slack.com/rtm).   chat.postEphemeral might also be useful in this context (https://api.slack.com/methods/chat.postEphemeral).  If it's decided that a more complicated (or emoji-tastic ) response is needed, then the bot will need to use the Web API method chat.postMessage (https://api.slack.com/methods/chat.postMessage).

As for messages.  Something along the lines of:
Thanks for your reply, but I'm only a humble bot.  Please post in #codebuddies-meta or #general to interact with human community members.  ❤


Use time_machine module for deliver schedule messages tests.

Based on @timabbott's comment https://github.com/zulip/zulip/pull/25440#discussion_r1188968553, we should time_machine for tests test_scheduled_messages.py where we test delivery of scheduled messages.

Hey I am interested in solving this issue. Guide me how to get started.

@zulipbot claim

@amanagr I have used the time_machine travel function as decorator and commented out the use of mock, please let me know if this would work?

@amanagr I used the time_machine.travel function and commented out the usage of mock function, please can you verify if that is what was supposed to be done?
Thank you

Looks like it, we should also use with time_machine... here.

@zulipbot claim

@zulipbot claim