libxc: 5.2.0 -> 5.2.2

Motivation for this change
bugfix release https://www.tddft.org/programs/libxc/changes/
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build cp2k octopus

@ryantm  We black listed this package in your auto updater a while ago. I think we remove it from the black list. The updates over the approximately last two years were much better behaved have rarely caused trouble anymore.

@markuskowa You're saying you think we can remove it now?

@ryantm Yes, you can remove it from the black list. This package is now suitable for automatic updates.


Planning sometimes changed when approaching an obstacle that failed to reach destination

Hi team,
I am using r7.0.0 to test the vehicle passing random obstacles. But I came across a case that cannot pass the obstacle to reach the destination.
In the beginning, it successfully planning the right route to destination.

But while approaching the object, it reset the route and get a stop signal with Obstacle is blocking the route, which made it stop beside the obstacle until timeout.

But in lgsvl, it seems still have space to crossover the obstacle and it didn't block the route

This case sometimes happened but sometimes with same obstacle the vehicle can crossover without the above situation.
I want to find the reason why it reset the route and stop? Is this issue caused by planning module or perception module? And how could I solve it? Thanks.

Try to lower static_obstacle_nudge_l_buffer.


testcase.yml with testcase.desc

I was confused with testcase.yml with testcase.desc, i 've found that there is no testcase.desc file in any other scenario folder.  but when i run "./deploy-test.sh" , the log "
grep: /Users/francis/workspc/agent-auto-integration-testcases/testcases/hessian-scenario-4.0.38/testcase.desc: No such file or directory" tells me that i need a testcase.desc.

This is a strategy problem. the testcase.desc file is auto-generated by the test case platform. Could you show me more detail about this problem?

i've finish build the agent.

This repo is closing, and moving into Apache repo.


Service account for ingress gateway and egress gateway

We have created two independent ServiceAccount for ingressgateway and egressgateway, but bind with special role.
https://github.com/istio/istio/blob/master/install/kubernetes/helm/istio/charts/ingressgateway/templates/serviceaccount.yaml
https://github.com/istio/istio/blob/master/install/kubernetes/helm/istio/charts/egressgateway/templates/serviceaccount.yaml
Can we replace them with default serviceAccount?
/cc @gyliu513 @rshriram @sdake

Why?
Are you asking for a helm option? If helm template is used, the yaml files can be customized before deployment, but I do not see why we would enable this.

@andraxylia just not clear why we create this service account without any clusterrolebindings.
/cc @rshriram who is the original author for this.

It's something we missed when Gateway was added, and an issue, since we do not want to let the default service account configure gateways. It is only the network operator who should do it. gyliu513 could you please fix this? And let's change the title of this issue.

@andraxylia Yes, I think we can use same clusterrole and clusterrolebindings as ingress, will post a PR soon.


feat: lutra query runner

Initial take on #3825. Inspired by prql-query.
Adds a query runner named Lutra that is basically a wrapper for prqlc compile, sqlite3 run and print(pl.DataFrame).
Connection parameters are defined within PRQL source with @lutra.sqlite annotation.
Uses connectorx for executing the query and converting to Apache Arrow. Uses polars to print the dataframe.
Next step is to create Python bindings so the resulting dataframe can be used with other existing tooling.

Before a few months, I built a tool called prqlite (I think you took a look at it). Is there any difference between it and Lutra?

Oops, this seems to depend on a very old version of polars.......

Before a few months, I built a tool called prqlite (I think you took a look at it). Is there any difference between it and Lutra?

There are. prqlite is primarily a TUI and it does that well. My intention with Lutra is to be able to call it from Python (and other languages) while providing it with only PRQL source.

Oops, this seems to depend on a very old version of polars

Yeah. It also won't compile for wasm32-unknown-unknown without a serious effort, as we'd have to compile polars and rusqlite too. That does not seem like a battle worth taking right now.

My intention with Lutra is to be able to call it from Python (and other languages) while providing it with only PRQL source.

Then, Lutra is something like a library or do I misunderstand it?

Then, Lutra is something like a library or do I misunderstand it?

Yes, but also with an option to invoke it from the command line.

Before a few months, I built a tool called prqlite (I think you took a look at it). Is there any difference between it and Lutra?

@Hulxv this is cool! Nice work. I don't think we knew about this...
(the goals seem to be slightly different to Lutra fwiw)

FYI can merge despite the failures — one is a cargo-audit which is fine and we can ignore; the other is the drop in test coverage

I'd like to replace connector-x with connector-arrow before merging. ConnectorX has stale dependencies and does a lot of things that are not needed in for lutra.


New hint: replace 'maybe Nothing f' with 'f =<<'.

What the title says.

I had to think about this for a while - it's more polymorphic which I'm not a huge fan of. But I decided to include it in the end. Thanks!


Angular 2 just went into beta

I hope I can update this part before Christmas :+1:

thanks i just made a small pull request too.


Cannot accessible when loading inside the iframe

https://www.npmjs.com/package/webcamjs
I installed the above and when i enable the camera using the below command:
        WebCamera.attach('.abc');

Here .abc is an element inside the iframe.
So it shows the error: WebCam.js error : Could not locate DOM element to attach to.
Any one please suggest.

You have to include the script inside the IFRAME.  It must live in the same DOM tree as the target element.


Azure API Management Sample

User Story
As an Azure APIM user, I want to be able to connect to the carbon aware SDK from my policy to make my implementation carbon aware.
Design Goals

Provide an example of how users can integrate the carbon aware SDK into APIM
Provide an example policy that uses the carbon aware SDK to make decisions about routing.

Task List

[ ] Add policy samples
[ ] Add readme docs

Acceptance Criteria

[ ] GettingStarted.md no longer exists in the root directory
[ ] User have a clear guide for starting out with the SDK
[ ] All unique pieces of information from the original GettingStarted.md can be found somewhere in /docs
[ ] Content touched is markdown-lint compliant

PR: Green-Software-Foundation/carbon-aware-sdk#257


Fix license in binary distribution

Motivation
versions were upgraded on some dependencies, but the license file is not updated.
Changes

fix the versions
drop the dependencies that don't exist
add the license for xz-java (public domain)

retest this please

retest this please


Universe error in IDE mode

Jay Bosamiya reports the following example that fails in the IDE mode:
module Bug

abstract type seq (a:Type u#a) :Type u#a =
  | MkSeq: l:list a -> seq a

with:
(Error) ASSERTION FAILURE: Impossible: missing universe instantiation on length
F* may be in an inconsistent state.
Please file a bug report, ideally with a minimized version of the program that triggered the error.

Also heads up @cpitclaudel.

IDE mode always fails for explicit universe annotations, here is an even simpler example:
val foo (a:Type u#a) :Type u#a

Here is the reason:
Recently I added code to generalize annotated universes in ToSyntax.fs. So, the desugaring of the val above is supposed to be:
val foo<a> :a:Type (u@0) -> Type (u@1)

i.e. the universe a should be generalized and then all its occurrences should become de-bruijn. I added it so that the typechecker can now treat it as usual (this was done as part of 2-phase changes).
However, I added this generalization under the function desugar_module_common, which does not get called in the IDE mode for every decl (duh!). And so, the typechecker raises a legit error.
I will try adding this generalization code directly to desugar_decl. Hopefully that should fix it.

The fix is in master.


[@azure/arm-cosmosdb] Release GA Version 16.0.0 of the SDK

Packages impacted by this PR
@azure/arm-cosmosdb
Issues associated with this PR
https://github.com/Azure/sdk-release-request/issues/4555
Describe the problem that is addressed by this PR

The last release of @azure/arm-cosmosdb is of the version 16.0.0-beta.6 that has happened 4 months ago.
The last GA release of @azure/arm-cosmosdb is of the version 15.5.0 that has happened 5 months ago.
Now, the service team has come up with a new GA Version of the service and expect a new GA Version of the SDK. This PR is to release that new GA Version of 16.0.0

What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
There are no special design considerations required for regeneration. It is a straight forward regeneration using the autorest command. But, while working on this, I have found some issues with the test cases that are added for this SDK.
Issue #1: While executing the test cases (in live/record mode), the developer is supposed to do some manual work (running test cases separately in a specific order) which is not an acceptable task.
Issue #2: After the execution, it leaves out 2 accounts without deleting which is also not acceptable. Both these issues have been resolved in this PR.
Are there test cases added in this PR? (If not, why?)
No new test cases are added. The existing cases should be sufficient. But as I explained above the existing test cases have been fixed.
Provide a list of related PRs (if any)
https://github.com/Azure/sdk-release-request/issues/4572
Command used to generate this PR:**(Applicable only to SDK release request PRs)
autorest --typescript --license-header=MICROSOFT_MIT_NO_VERSION --typescript-sdks-folder=C:\Users\sarajama\Projects\azure-sdk-for-js --use=<EMAIL_ADDRESS>--version=3.9.3 --modelerfour.lenient-model-deduplication --azure-arm --head-as-boolean=true --generate-sample=true --generate-test  https://raw.githubusercontent.com/Azure/azure-rest-api-specs/c364b64a6b412ffd7507dea71ae53251d35748c1/specification/cosmos-db/resource-manager/readme.md
Checklists

[X] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[X] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.

Please hold on merging this PR until we figure out what's the right steps to take here. Thanks

Closing this issue in favor of https://github.com/Azure/azure-sdk-for-js/pull/27431


Cannot delete file using delete

Issue Type: Bug
Select a file and press delete, then delete has been caught by fitter. I have to delete a file by right click it and click del.
VS Code version: Code 1.36.0 (0f3794b38477eea13fb47fbe15a42798e6129338, 2019-07-03T13:20:56.241Z)
OS version: Linux x64 4.15.0-54-generic/Ubuntu 18.04
Disable all extension does not fix it.

/duplicate of #76514
Fixed in 1.36.1 version.


arm64 docker images

Currently available docker images only support amd64. It would be great if arm64 images could be provided as well!

I attempted to add it in https://github.com/protomaps/go-pmtiles/pull/138 but it looks like it requires a different build process:
ERROR: Multi-platform build is not supported for the docker driver.
Switch to a different driver, or turn on the containerd image store, and try again.

Can you investigate a PR for doing this? (and as a bonus, pushing to multiple registries without duplication)

Looks like it worked:
https://github.com/protomaps/go-pmtiles/pkgs/container/go-pmtiles
https://hub.docker.com/repository/docker/protomaps/go-pmtiles/general
Let me know if it solves it for you! Closing issue for now.

Works great! Many thanks!


Change ngFor in UserListComponent so that it is no longer binded directly to the filter function

Due to the way change detection works in Angular, if we generate our list with something like *ngFor="let user of filterUsers(param)", that filter function will be repeatedly called whenever any change whatsoever in the DOM is detected. This goes for clicking anything, scrolling (I think), or typing anything, even if our action has nothing to do with filtering, that function will be called over and over again hundreds of times. Filtering is an expensive operation, so this can be very problematic and is likely to blame for poor performance when transferring it over to the TodoListComponent, where there are hundreds of objects being dealt with. I'm assuming this is why the accordions and other things were so slow for the majority of the TodoLists this past lab.
What we want to be doing is to have the ngFor binded to just the filteredUsers array, and have the filter function only be called when things we care about change.

There seems to be some possibly useful information about filtering and data sources in the table documentation here: material angular table in the filtering section.

We have discussed this and determined that switching this to tables would be a good approach to try to handle it. We might be able to get some server side filtering by using pagination, but we will want to look at that. we will start by putting in tables and then go from there.

We will need to pull this change from lab 3 into lab 4.


Fixes

This adds the testing I introduced to the other plugins. It also adds some text regarding the Press This situation.

I am not sure why Travis is failing. Any opinions?

something with the unit tests and the database from looking at the logs https://travis-ci.org/indieweb/wordpress-indieweb-press-this/jobs/317437543

I see, but it is the same configuration I set up for the other plugins. I feel that I am missing something obvious

See https://github.com/indieweb/wordpress-indieweb-press-this/pull/20


Error object not properly filled out when submitting eth_subscribe request.

When I send the following request over the JSON-RPC:
{
	id: 1,
	jsonrpc: "2.0",
	method: "eth_subscribe",
	params: [ "newHeads", {} ]
}

Expected (per spec):
{
	...
	error: {
		code: -32601,
		message: "Method not found"
	}
}

Actually received:
{
	...
	"error":{
		"code":-32000,
		"message":"Error: HTTP Error: 405 on eth_subscribe\n    at Object.callback (chrome-extension://nkbihfbeogaeaoehlefnkodbefgpgknn/scripts/background.js:105443:18)\n    at cbOnce (chrome-extension://nkbihfbeogaeaoehlefnkodbefgpgknn/scripts/background.js:106012:21)\n    at loadFunc (chrome-extension://nkbihfbeogaeaoehlefnkodbefgpgknn/scripts/background.js:106079:16)\n    at XMLHttpRequest.readystatechange (chrome-extension://nkbihfbeogaeaoehlefnkodbefgpgknn/scripts/background.js:106018:13)"
	}
}

It looks to me like you're maybe hitting a managed RPC provider like Infura, which does not currently support stateful requests like eth_subscribe.
This is why the Filter Subprovider exists, it essentially polyfills this feature into a client-side polling layer.

If you confirm you're hitting Infura, we can close this issue.

I'm not sure how to tell.  I load my dApp in a webpage with the MetaMask extension installed and the send a JSON-RPC request using window.web3.currentProvider.sendAsync(payload, callback) with the payload I linked above.  callback is then called with a response that looks like the one I referenced in the issue description.
Also, note that my expectation here isn't that eth_subscribe actually works.  My expectation is only that I get a proper error message in response to it (-32601/Method not found).
If this is an Infura problem and MetaMask is just blindly forwarding the error to me, how does one go about reporting a bug to them?

Oh I see now.  This is an issue with how provider-engine forms and forwards error codes from an RPC, relative to the JSON RPC spec.
We would need to write a comprehensive mapping of RPC error codes to JSON RPC expected output, right about here.

This has re-surfaced over the past week or so.  (Or at least, that's when I first noticed it; it's possible it's been around much longer and just eluded my attention.)  I've been getting back a success (?) response (specifically, 0xb) from MetaMask when I try to send eth_subscribe requests, instead of the expected "Method not found" error.  Pinging @kumavis @danfinlay in case there's a regression on your end!


feat(apps/prod/tekton/configs/triggers/pingcap/tidb): support trigger buildings with enterprise profile

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


MSG (Outlook Emails) to PDF

Hi,
We have a repeated requirement with a lot of our clients to generate previews of MSG emails - the general use case is, they save their emails from Microsoft Outlook as MSG, and upload it to their systems. This is a propertiary Microsoft format, but many recent advancements in the https://github.com/TeamMsgExtractor/msg-extractor project look to make understanding and converting these files very simple.
Internally, this msg-extractor utility uses wkhtmltopdf, an abandoned HTML > PDF library - when the --pdf argument is specified - it isn't very good at all and has serious security flaws, alongside handling fonts badly (though this is probably fixable with the --wk-options argument?).
However, specifying the --html and --prepared-html options with the utility generate a folder of the same name of the MSG file provided to ingest, containing all image attachments, and an HTML file with relative links to the image files - and it looks great in Chrome. This would be absolutely perfect to pass to Gotenberg.
Setup:

Download latest release of TeamMsgExtractor/msg-extractor
Use w/ python -m extract_msg './test.msg' --html --prepared-html

I have tested this on some extremely large email chains, and it seems very reliable considering the nature of the format, and the project seems very active - though even if it wasn't, these files hardly have changed much and I'd argue there is a significant case to still render them.
To be clear, this isn't about converting .eml files, these are much simpler - this is about the propertiary Outlook Message Item format - the README of the extractor project contains much documentation about the format.
Happy to provide MSG files on request privately if you can't get access to these easily.
Would be interested to hear if anyone else has conquered this type of feature in their web applications, to preview these awkward file types? Sadly changing the system to force users to convert to PDF first isn't suitable for most of our clients as they have tens of years of MSG files in their system.

Hey @callumgarven 👋
To summarize, the idea is to have a new route /forms/chromium/convert/msg that takes a MSG file, and internally uses TeamMsgExtractor/msg-extractor to create an HTML file that then will be converted to PDF with Chromium.
Is that correct? 😄

Related #846.

to fix your trouble check this solution click
maybe this will solve your problem.

maybe this solution will solve your problem click


Values of private mutable variables are lost if modified from a base class constructor

Hi,
I am suffering an issue which prevents me from using F# on a .NET plugin for a third-party software.
To plug my code into a third-party library, I need to inherit a specific base class. This base class forces me to implement a method which will be directly called from its constructor :scream: .
The third-party code:
namespace ThirdPartyLib
{
    public abstract class ThirdPartyBaseClass
    {
        protected ThirdPartyBaseClass()
        {
            CalledFromConstructor();
        }

        protected abstract void CalledFromConstructor();
    }
}

A C# plugin:
namespace MyCSLib
{
    using ThirdPartyLib;

    public class MyCSClass : ThirdPartyBaseClass
    {
        private bool _myValue = false;

        protected override void CalledFromConstructor()
        {
            _myValue = true;
        }

        public bool MyValue
        {
            get { return _myValue; }
        }
    }
}

A F# plugin:
namespace MyLib

open ThirdPartyLib

type MyClass() =
    inherit ThirdPartyBaseClass()

    let mutable myValuePrivate = false

    override x.CalledFromConstructor() = myValuePrivate <- true

    member x.MyValue with get() = myValuePrivate

The problem here is that while the C# code keeps the modified value of myValuePrivate, the F# one equals to false when accessed with MyValue.
Here is a corresponding unit test:
using Microsoft.VisualStudio.TestTools.UnitTesting;
using MyCSLib;
using MyLib;

namespace ThirdPartyApp
{
    [TestClass]
    public class UnitTest
    {
        [TestMethod]
        public void TestFSharp()
        {
            var myClass = new MyClass();
            Assert.IsTrue(myClass.MyValue);
        }

        [TestMethod]
        public void TestCSharp()
        {
            var myCSClass = new MyCSClass();
            Assert.IsTrue(myCSClass.MyValue);
        }
    }
}

Does someone know why both plugin implementations are not reacting the same way?
A VS2013 solution demonstrating the problem can be cloned from https://github.com/bbenoist/fs-private-inheritance-bug . Feel free to submit your solution as a PR.

After reading "Explicit Fields: The val Keyword (F#)", I found a fix (https://github.com/bbenoist/fs-private-inheritance-bug/pull/1) by using the val keyword instead of let in the F# code:
namespace MyLib

open ThirdPartyLib

type MyClass() =
    inherit ThirdPartyBaseClass()

    [<DefaultValue(false)>] val mutable myValuePrivate : bool

    override x.CalledFromConstructor() = x.myValuePrivate <- true

    member x.MyValue with get() = x.myValuePrivate

What's wrong with the version using the let instruction?

The let binding is part of the primary constructor. That is, this F#
type A() =
    inherit B()
    let x = 1

is logically equivalent to this C#
class A : B {
    int x;
    public A() : base() {
        x = 1;
    }
}

Calling virtual methods from constructors is a notorious anti-pattern, for exactly this reason.  Many static analysis tools will warn you not to do it, see https://msdn.microsoft.com/en-us/library/ms182331.aspx and http://stackoverflow.com/questions/119506/virtual-member-call-in-a-constructor.
In C#, field initialization is done before invoking the base ctor, so in this particular problem is avoided.  F# and VB do the opposite - the base ctor is invoked first, then fields are initialized.  There are advantages to both approaches.  In F#/VB is it safe and legal to access base class members when initializing a derived class:
    public abstract class ThirdPartyBaseClass
    {
        protected int baseField = 42;
        public int BaseProp { get { return baseField; } }
    }

type MyClass() =
    inherit ThirdPartyBaseClass()
    let myValuePrivate = base.BaseProp + 10
    member __.MyValue = myValuePrivate

let x = MyClass()
printfn "%A" x.MyValue  // prints 52 as expected

Since C# runs initializers before calling base ctor, it is illegal to access base members during field initialization:
    public class DerivedClass : ThirdPartyBaseClass
    {
        // error: keyword 'base' is not available in the current context
        private int myValuePrivate = base.BaseProp + 10;
        public int MyValue { get { return myValuePrivate; } }
    }

@FunctionalFirst @latkin
Thank you for the explanations! I now clearly see what was the problem with my understanding.
@latkin

Calling virtual methods from constructors is a notorious anti-pattern, for exactly this reason. Many static analysis tools will warn you not to do it, see https://msdn.microsoft.com/en-us/library/ms182331.aspx and http://stackoverflow.com/questions/119506/virtual-member-call-in-a-constructor.

I totally agree but am unfortunately not responsible of the proprietary .NET software which executes my code. Thus, I have to deal with their poor design choices.

In C#, field initialization is done before invoking the base ctor, so in this particular case the problem you are seeing is avoided. F# and VB do the opposite - the base ctor is invoked first, then fields are initialized. There are advantages to both approaches. In F#/VB is it safe and legal to access base class members when initializing a derived class.

I was not aware of such differences, thanks for the info.


allow passing custom classes

This allows you to pass custom classes to main and children buttons. Especially handy if you work with https://github.com/css-modules/css-modules

A few comments inline.  Awesome PR!

Totally agree with the proposed changes. Originally, I was just trying to go with the current code style.

Published as 0.3.0.  Thanks!


Bumping guardrail

Actually, gonna wait to see how long it takes dependabot or whatever else to update this dep


Some minor inconsistency in the paper

According to your arxiv paper (https://arxiv.org/pdf/2310.12109.pdf), there is no activation in sequence mixing in formula (2). However, the appendix includes the code for MonarchMixerLayer and it includes the ReLU in the sequence mixing layer.

Plus, equation (3) seems to be a MLP operation (I might be wrong). I don't fully understand why it is said to be MLP-free in "The resulting architecture is entirely attention- and MLP-free."

Thanks for your questions!

it includes the ReLU in the sequence mixing layer

This is a typo - we used to say that the sequence mixing had an "optional" activation function that we would set to identity for the sequence mixer. We updated the equation but not the pseudocode -- will fix it the next time we update the arXiv!

equation (3) seems to be a MLP operation

Ah, this is helpful feedback! The distinction we intend to say is that an MLP is quadratic in $d$, and the M2 version is sub-quadratic.
Specifically -- in an MLP, there are linear layers that take quadratic compute ($O(d^2)$ for dimension $d$). In M2, we replace these linear layers with Monarch matrices, which can be computed in sub-quadratic time. So it has the similar structure of an MLP, but is sub-quadratic in $d$.
We'll clarify the language in the next arXiv update, thank you!


Update get_started.md

Two minor updates:

When using node 12 as described, building was throwing the following error:

ERROR: Your dev environment is running Node.js version v12.22.12 which does not meet the requirements for building this repository.  (The rush.json configuration requires nodeSupportedVersionRange=">=14.15.0 <15.0.0 || >=16.13.0 <17.0.0")

Quick note to let people know where to go to disable the git policy email error during the rush install step.

Thanks for the doc updates @benkeen! Looks good to me.


Add submodule auction_provider

Source code of Robonomics network watchdog to collect auctions bids.

Done. https://github.com/airalab/gaka-chu.online/commit/25651567a8b9016ad40b83661ac3876a6cd229d7


refactor: replace calculate_next_block_base_fee with alloy's builtin function

Fixes #7635 by replacing calculate_next_block_base_fee from https://github.com/paradigmxyz/reth/blob/d4ea41528a68c851b3d67c2a224cc2b225f65fef/crates/primitives/src/basefee.rs#L26-L26 with alloy's builtin function.

@mattsse

I'd prefer if we could entirely replace the function and use the re-exported alloy fn
this likely requires a few casts

how does the latest commit look?


Unexpected Cursor Offset when hitting Enter

Using GitHubs's new video upload feature, here's the bug. When pressing enter, for some reason the cursor skips a line. The expected behavior would be for the cursor to go to the next line and with a few spaces similar to when it's Prettified
https://user-images.githubusercontent.com/62767337/104938890-53fbf900-5975-11eb-959e-bf1842eb7dcc.mp4

Any idea what might be causing this?

Any idea what might be causing this?

I saw this bug for a bit, and then @yoshiakis and I couldn't figure out how to reproduce it.
Can you provide some details?

what version of graphiql npm package are you running?  Describe the setup briefly if you can.
what browser and version? (looks like firefox?)
what OS?

I saw this bug for a bit, and then @yoshiakis and I couldn't figure out how to reproduce it.
Can you provide some details?

what version of graphiql npm package are you running?  Describe the setup briefly if you can.
what browser and version? (looks like firefox?)
what OS?

Sure.

Running graphiql: ^1.3.2 on a React app. What's interesting is that at first that didn't happen for a while but then it started. At the same time, I did uninstall it and reinstalled it again in between I don't know if that could affect it.
Actually using Chrome. Version 88.0.4324.96 (Official Build) (64-bit)
Using Windows but @etienedi is using linux and the same happens to him.
No. It works fine on the Netlify app.

Sure.

Running graphiql: ^1.3.2 on a React app. What's interesting is that at first that didn't happen for a while but then it started. At the same time, I did uninstall it and reinstalled it again in between I don't know if that could affect it.
Actually using Chrome. Version 88.0.4324.96 (Official Build) (64-bit)
Using Windows but @etienedi is using linux and the same happens to him.
No. It works fine on the Netlify app.

thanks, all this info is really helpful! were you able to re-create the bug consistently once you discovered it, and then it went away when you rebuilt it?  if so, i wonder if it doesn't have to do with localstorage.
also, are you using any keyboard shortcuts? are you holding any keys down when hitting return, or using any other shortcuts?

thanks, all this info is really helpful! were you able to re-create the bug consistently once you discovered it, and then it went away when you rebuilt it?  if so, i wonder if it doesn't have to do with localstorage.
also, are you using any keyboard shortcuts? are you holding any keys down when hitting return, or using any other shortcuts?

No it was the other way around. When I first installed graphiql the bug wasn't there. Then after a few weeks it showed up and it's there consistently day in day out.

No shortcuts or anything. Just pressing enter. It only happens if Enter is pressed after { , ( etc. But if the cursor is at the beginning of the line with nothing typed, then it goes to the next line as expected without skipping a line.

No it was the other way around. When I first installed graphiql the bug wasn't there. Then after a few weeks it showed up and it's there consistently day in day out.

No shortcuts or anything. Just pressing enter. It only happens if Enter is pressed after { , ( etc. But if the cursor is at the beginning of the line with nothing typed, then it goes to the next line as expected without skipping a line.

what I'm wondering is whether something is happening when serializing the query from URL arguments that inserts weird characters that weren't there before?
we also have some logic specific to GraphiQL for eating whitespace characters that was introduced to fix similar issues - utilities/normalizeWhitespace, but that's only executed on paste. are you pasting graphql into the query editor? (most of us are)
we also have some custom logic for automatically filling out leaf fields like this on enter. GraphiQL.autoCompleteLeafs(). if no fields are required in the selection set you're expanding, it should lead to the behavior we're expecting. so, something with or regarding that logic I think could be failing, potentially related to many minor version updates to codemirror ?

what I'm wondering is whether something is happening when serializing the query from URL arguments that inserts weird characters that weren't there before?
we also have some logic specific to GraphiQL for eating whitespace characters that was introduced to fix similar issues - utilities/normalizeWhitespace, but that's only executed on paste. are you pasting graphql into the query editor? (most of us are)
we also have some custom logic for automatically filling out leaf fields like this on enter. GraphiQL.autoCompleteLeafs(). if no fields are required in the selection set you're expanding, it should lead to the behavior we're expecting. so, something with or regarding that logic I think could be failing, potentially related to many minor version updates to codemirror ?

It's gone. Thanks for the comments. They helped me think a little better of the issue. I had this additional toolbar button which copies the query to clipboard so I can share a link that directs the user straight into the graphiql page with the query already typed and prettified.
I had a ref wrapped around the additional toolbar button which I was using for toggling the component state. Anyway, once I removed the ref, the bug was gone and I found a work around for not using the ref anymore.

It's gone. Thanks for the comments. They helped me think a little better of the issue. I had this additional toolbar button which copies the query to clipboard so I can share a link that directs the user straight into the graphiql page with the query already typed and prettified.
I had a ref wrapped around the additional toolbar button which I was using for toggling the component state. Anyway, once I removed the ref, the bug was gone and I found a work around for not using the ref anymore.

very interesting! this is really good to know for solving the bug, thanks for all the details.
i just saw the issue using the default start-graphiql command, using our default example. no use of refs, but i wonder if something else is at play?

very interesting! this is really good to know for solving the bug, thanks for all the details.
i just saw the issue using the default start-graphiql command, using our default example. no use of refs, but i wonder if something else is at play?

Very strange. I thought I had fixed it because it worked locally when running it on localhost:3000, but when I pushed it to production, it didn't go away.
I've checked multiple times and it works fine locally but still skips a line in production. I thought the browser cache could be it, so I've cleaned it up but still skips the line.

Very strange. I thought I had fixed it because it worked locally when running it on localhost:3000, but when I pushed it to production, it didn't go away.
I've checked multiple times and it works fine locally but still skips a line in production. I thought the browser cache could be it, so I've cleaned it up but still skips the line.

innnteresting!
I wonder if you were to build the equivalent of the production dist locally, and then open that directly using npx serve ./dist?

innnteresting!
I wonder if you were to build the equivalent of the production dist locally, and then open that directly using npx serve ./dist?

So I just tried npm build And then did npx serve ./build to run the app with Webpacks production config and the cursor worked as expected. Yet in our production environment it doesn't. We will keep you posted if we find anything else that could be causing it.

So I just tried npm build And then did npx serve ./build to run the app with Webpacks production config and the cursor worked as expected. Yet in our production environment it doesn't. We will keep you posted if we find anything else that could be causing it.

ok thank you! all very helpful

ok thank you! all very helpful

Thanks so much for the help so far @acao!
To add to what our production config is:

We're copying the build folder into an nginx docker image (the contents of the build folder is the same that works with npx serve ./build)
The docker container runs on K8s (GKE) exposed through an Ingress with Google's load balancer with both HTTP and HTTPS. (The difference between HTTP and HTTPS makes no difference. I was suspecting that something might have been blocked by some odd mixed content errors, but there aren't any)
We do use Google Cloud CDN (on top of the LB), so we cannot (yet) rule out that something is cached that shouldn't. However, all other changes we deploy continuously are always present, making a caching issue less likely
The issue also occurs with a completely blank browser (to rule out local storage, browser caching, etc.)

We will also go through our build process step-by-step to see if we can identify anything that breaks it (e.g. running from a Docker container, serving the app through nginx, etc.) - but it will probably take a few days to get back on this.

Thanks so much for the help so far @acao!
To add to what our production config is:

We're copying the build folder into an nginx docker image (the contents of the build folder is the same that works with npx serve ./build)
The docker container runs on K8s (GKE) exposed through an Ingress with Google's load balancer with both HTTP and HTTPS. (The difference between HTTP and HTTPS makes no difference. I was suspecting that something might have been blocked by some odd mixed content errors, but there aren't any)
We do use Google Cloud CDN (on top of the LB), so we cannot (yet) rule out that something is cached that shouldn't. However, all other changes we deploy continuously are always present, making a caching issue less likely
The issue also occurs with a completely blank browser (to rule out local storage, browser caching, etc.)

We will also go through our build process step-by-step to see if we can identify anything that breaks it (e.g. running from a Docker container, serving the app through nginx, etc.) - but it will probably take a few days to get back on this.

@acao The cursor offset bug is happening again in the development environment. It's strange because I didn't touch that code from the time it was working properly to now.
I did update a few dependencies but not the graphiql. We will keep you updated. Please keep us updated as well if you are able to detect what's potentially causing it. Thanks.

@acao The cursor offset bug is happening again in the development environment. It's strange because I didn't touch that code from the time it was working properly to now.
I did update a few dependencies but not the graphiql. We will keep you updated. Please keep us updated as well if you are able to detect what's potentially causing it. Thanks.

ok! did you change graphql version by chance? or any other peer dependencies?

ok! did you change graphql version by chance? or any other peer dependencies?

No. Didn't touch graphiql or graphql. Thanks.

No. Didn't touch graphiql or graphql. Thanks.

Something interesting just happened to @etiennedi

started out on<EMAIL_ADDRESS>- issue occurs
downgraded to<EMAIL_ADDRESS>- issue gone
upgraded to<EMAIL_ADDRESS>- issue still gone
upgraded back to<EMAIL_ADDRESS>- issue still gone
rm -rf node_modules && npm install - issue still gone

However, when I tried to reproduce this, the issue was still there for me in all versions.

Something interesting just happened to @etiennedi

started out on<EMAIL_ADDRESS>- issue occurs
downgraded to<EMAIL_ADDRESS>- issue gone
upgraded to<EMAIL_ADDRESS>- issue still gone
upgraded back to<EMAIL_ADDRESS>- issue still gone
rm -rf node_modules && npm install - issue still gone

However, when I tried to reproduce this, the issue was still there for me in all versions.

ah! i wonder if it's because i incremented a dependency somewhere?

ah! i wonder if it's because i incremented a dependency somewhere?

SO weird haha

SO weird haha

@acao have you done anything to the code? It suddenly works in development and in production.

@acao have you done anything to the code? It suddenly works in development and in production.

the next time you catch it, see if deleting the localstorage entries and refreshing is enough for it to go away. also, see if ctrl+a and delete is enough to make it go away?

the next time you catch it, see if deleting the localstorage entries and refreshing is enough for it to go away. also, see if ctrl+a and delete is enough to make it go away?

Okay. I will let you know. So far so good.

Okay. I will let you know. So far so good.

I also experienced this issue.  Following @halvespereira 's instructions where I removed graphiql entirely, installed<EMAIL_ADDRESS>(which works), and then upgraded to<EMAIL_ADDRESS>fixed it.
I definitely suspect some sort of dependency issue is at play.

I also experienced this issue.  Following @halvespereira 's instructions where I removed graphiql entirely, installed<EMAIL_ADDRESS>(which works), and then upgraded to<EMAIL_ADDRESS>fixed it.
I definitely suspect some sort of dependency issue is at play.

yes i suspect it has to do with the version of codemirror, this confirms what i suggested earlier in this thread. thank you, will take a look this weekend!

yes i suspect it has to do with the version of codemirror, this confirms what i suggested earlier in this thread. thank you, will take a look this weekend!

hmm, no luck. the only things that have changed since 1.3.0 are internal dependencies such as codemirror-graphql. codemirror version upgrade for graphiql happened in 1.2.x

hmm, no luck. the only things that have changed since 1.3.0 are internal dependencies such as codemirror-graphql. codemirror version upgrade for graphiql happened in 1.2.x

Just to give an update. This issue is no longer happening for me and it hasn't for a few weeks now. Not sure if anything has been done to fix it but it appears to be gone.

Closing this due to inactivity and since the issue seems to have been resolved back in the day. There have been multiple new versions for both graphiql and codemirror since then, so given that there are no more comments here it's likely that this doesn't occur anymore in these newer versions.


As a user, I want the information on number of items in the collection to be displayed correctly

In the Search tool, for the nisv-catalogue-radio collection in the test environment, the number of items in the collection is shown to be -1. The number of results in the list is correct (405.526).

@mwigham will check this one out. Has something to do with my latest changes

@mwigham ok solved this one, this was a really stupid edge case, because of a sloppy way of error checking (my bad) if the collection metadata retrieved from the server is ok.
Closing this one instantly. This will be deployed on test soon.


EVM-850 Validator rootchain balance metrics

Description
Egde should periodically retrieve validator balance from the root chain and output that information as a Prometheus metric.
A new server flag --metrics-interval has been introduced. By default, this flag returns 8s for interval. A value 0s will disable metrics.
Also checkpoint_block_number metric is moved from checkpoint_manager to the polybft and it is being published by each validator node.
Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[ ] Hotfix (change that solves an urgent issue, and requires immediate attention)
[X] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

[X] I have assigned this PR to myself
[X] I have added at least 1 reviewer
[X] I have added the relevant labels
[ ] I have updated the official documentation
[ ] I have added sufficient documentation in code

Testing

[X] I have tested this code with the official test suite
[ ] I have tested this code manually

Late LGTM


[OTHER] remove stale.yml

What would you like to share?
Delete the file stale.yml
Path: .github/workflow/stale.yml
Additional information
No response

can I work on this if no one else wants to yet?


feat: refactor pedersen hash standard

WORK IN PROGRESS
Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

[ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
[ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to relevant issues (if any exist).

Benchmark results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit ba2f671c and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,444
179,588
716,132

l1_rollup_calldata_gas
222,972
868,076
3,448,064

l1_rollup_execution_gas
842,059
3,595,184
22,203,433

l2_block_processing_time_in_ms
:warning: 2,266 (+122%)
:warning: 8,808 (+129%)
:warning: 34,201 (+125%)

note_successful_decrypting_time_in_ms
:warning: 705 (+112%)
:warning: 2,501 (+144%)
:warning: 9,586 (+145%)

note_trial_decrypting_time_in_ms
:warning: 6.00 (-83%)
107 (-2%)
135 (-1%)

l2_block_building_time_in_ms
:warning: 15,604 (+75%)
:warning: 62,176 (+75%)
:warning: 249,135 (+59%)

l2_block_rollup_simulation_time_in_ms
:warning: 13,064 (+99%)
:warning: 52,059 (+99%)
:warning: 208,367 (+99%)

l2_block_public_tx_process_time_in_ms
2,501 (+10%)
9,993 (+9%)
:warning: 40,302 (-22%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
:warning: 25,061 (+115%)
:warning: 48,919 (+118%)

note_history_successful_decrypting_time_in_ms
:warning: 6,185 (+160%)
:warning: 12,234 (+163%)

note_history_trial_decrypting_time_in_ms
123
:warning: 148 (-27%)

node_database_size_in_bytes
1,665,543
1,203,677

pxe_database_size_in_bytes
27,188
54,187

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
:warning: 82.7 (+75%)
61,697
18,841

private-kernel-ordering
:warning: 50.7 (+117%)
24,233
8,089

base-rollup
:warning: 2,070 (+143%)
639,797
811

root-rollup
:warning: 89.9 (+131%)
4,072
1,097

private-kernel-inner
:warning: 59.2 (+41%)
81,504
18,841

public-kernel-private-input
46.7 (-2%)
41,455
18,841

public-kernel-non-first-iteration
29.2 (-6%)
41,497
18,841

merge-rollup
0.877 (+3%)
2,592
873

Miscellaneous
Transaction sizes based on how many contracts are deployed in the tx.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
8,723
29,391 (+7%)

Fixed master merge issues, reenabling auto-merge


Closes #1437: Support Round and Square stroke caps for vector progress bars

I can't tell what's different in the renders. I assume you'll add test cases for the butt/round cap distinction next. Can you also include some complex fills / patterns? (I'm also hopeful that these work as masks if there's something really complex to explore).

I can't tell what's different in the renders. I assume you'll add test cases for the butt/round cap distinction next. Can you also include some complex fills / patterns? (I'm also hopeful that these work as masks if there's something really complex to explore).
Edit: also, combinations with other effects like shadows and blend modes. I'm not sure what an inset shadow should do here.

I added three small curves to dials & gauges: one with no end cap, one with round, and one with square. I haven't added any tests for vectors that are not progress bar vectors because Figma already returns a fill vector path that includes the end caps. Or do you just mean I should add more complex cases for progress bar vectors?

Or do you just mean I should add more complex cases for progress bar vectors?

Yeah, I think it would be good to have some more complex examples in case some combinations reveal other issues. Maybe this is a different test section for combinations, though, because the existing tests are bad for testing too many things in one screenshot.

Added additional tests, and fixed a bug where turning progress vector off in the plugin did not work.


get_ebill_recipient_subscription_status_bulk doesnt work

I tried the code without converting the array of ids, removing the following 2 lines:
array_bill_recipient = self.client.get_type("ns2:ArrayOfBillRecipient")
recipients = array_bill_recipient(bill_recipient_id)
this removes the validation error, but weirdly, only returns one element. This route's signature also doesnt match the documentation (it says to use BillRecipientID, which results in an "unexpected argument error").
Is there any official explanation from Postfinance how to make this route work?

I have a possible fix here that you can use the "ns4:ArrayOfstring" converter instead of the the "ns2:ArrayOfBillRecipient"
apparently they changed this without documenting it properly, and the "ns2:ArrayOfstring" format is now correct?


Creating BC OnPrem based on image mcr.microsoft.com/businesscentral/onprem:1904-cu3-dk-ltsc2019 fails after 10 min

On Windows 10 version 1903 and running Docker Desktop <IP_ADDRESS> I run the following script:
New-NavContainer -isolation process -auth NavUserPassword -updateHosts -accept_outdated -enableSymbolLoading -doNotExportObjectsToText -includeCSide -containerName $ContainerName -imageName "mcr.microsoft.com/businesscentral/onprem:$(1904-cu3-dk)" -accept_eula -licenseFile $LicensFile.FullName -includeTestToolkit -restart no -useBestContainerOS
The scripts runs for approx 10 minutes and then it fails.
Below you will find the output data and I have verified that the license used has not expired and is for Business Central 365 OnPrem version 14:
Looking for update to module: NavContainerHelper
Import Module NavContainerHelper
Creating Container: SVS-14-CU3-DK
NavContainerHelper is version <IP_ADDRESS>
Host is Microsoft Windows 10 Pro - 1903
Docker Client Version is 19.03.1
Docker Server Version is 19.03.1
Pulling image mcr.microsoft.com/businesscentral/onprem:1904-cu3-dk-ltsc2019
1904-cu3-dk-ltsc2019: Pulling from businesscentral/onprem
65014b3c3121: Already exists
e7a74de96ddd: Pulling fs layer
pulling lines squashed in this output...
98b7f364024f: Pull complete
Digest: sha256:ce272724ce0ed6cf7ef526ac6c36f1bff1f8789d168fc2c180079bfb9af33e0a
Status: Downloaded newer image for mcr.microsoft.com/businesscentral/onprem:1904-cu3-dk-ltsc2019
Using image mcr.microsoft.com/businesscentral/onprem:1904-cu3-dk-ltsc2019
Creating Container SVS-14-CU3-DK
Using license file \ev-filsrv01\Navision\EV Internal Tools\Powershell\License\Udvikler MS Dynamics 365 BC_incl ISV_V14_050719.flf
Version: 14.4.35602.0-dk
Platform: 14.0.35570.0
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.678 (ltsc2019)
Host OS Version: 10.0.18362.295 (1903)
A better Generic Container OS exists for your host (1903)
Creating temp container from mcr.microsoft.com/businesscentral/onprem:1904-cu3-dk-ltsc2019 and extract necessary files
Extracting Service Tier and WebClient Files
Extracting Windows Client Files
Extracting Configuration packages
Extracting Test Assemblies
Extracting Test Toolkit
Extracting Upgrade Toolkit
Extracting Extensions
Extracting Applications
Extracting Files from Run folder
Extracting Database Files
Downloading prerequisites
Downloading C:\ProgramData\NavContainerHelper\14.4.35602.0-dk-Files\Prerequisite Components\IIS URL Rewrite Module\rewrite_2.0_rtw_x64.msi
Downloading C:\ProgramData\NavContainerHelper\14.4.35602.0-dk-Files\Prerequisite Components\Open XML SDK 2.5 for Microsoft Office\OpenXMLSDKv25.ms
i
Downloading C:\ProgramData\NavContainerHelper\14.4.35602.0-dk-Files\Prerequisite Components\DotNetCore\DotNetCore.1.0.4_1.1.1-WindowsHosting.exe
Performing cleanup
Removing temp container
Using generic image mcr.microsoft.com/dynamicsnav:generic-1903
Pulling image mcr.microsoft.com/dynamicsnav:generic-1903
generic-1903: Pulling from dynamicsnav
Digest: sha256:8ede5835811505be10d1bdc118a4c9281fdf6acb8900d356f45171cce5d416af
Generic Container OS Version: 10.0.18362.295 (1903)
Generic Tag of better generic: <IP_ADDRESS>
Using locale da-DK
Using process isolation
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Files in C:\ProgramData\NavContainerHelper\Extensions\SVS-14-CU3-DK\my:

AdditionalOutput.ps1
AdditionalSetup.ps1
license.flf
MainLoop.ps1
updatehosts.ps1
Creating container SVS-14-CU3-DK from image mcr.microsoft.com/dynamicsnav:generic-1903
705982e6f34cdd07dea1b9f5b46ec8b69abf43b90fdbeac1bdd28b383faec300
Waiting for container SVS-14-CU3-DK to be ready
Installing Business Central
Installing Url Rewrite
Installing OpenXML
Installing DotNetCore
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
Copying Web Client Files
Copying Windows Client Files
Copying PowerShell Scripts
Copying ConfigurationPackages
Copying Test Assemblies
Copying TestToolKit
Copying UpgradeToolKit
Copying Extensions
Copying ClientUserSettings
Copying ReportBuilder
Changing Database Server Collation to Danish_Greenlandic_100_CI_AS
SQL Server 2017 transmits information about your installation experience, as well as other usage and performance data, to Microsoft to help improv
e the product. To learn more about SQL Server 2017 data processing and privacy controls, please see the Privacy Statement.
Copying Cronus database
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Starting Business Central Service Tier
Installation took 115 seconds
Installation complete
Initializing...
Starting Container
Hostname is SVS-14-CU3-DK
PublicDnsName is SVS-14-CU3-DK
Using NavUserPassword Authentication
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 0A9997D50C9F0F623641AAB23AB24FAA75147CAB
Modifying Service Tier Config File with Instance Specific Settings
Restarting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Using license file 'c:\run\my\license.flf'
Import License
The server "NAV" was unable to process the request. The application will close.
at , C:\Run\SetupLicense.ps1: line 29
at , C:\Run\navstart.ps1: line 156
at , C:\Run\start.ps1: line 121
at , : line 1Error
Installing Business Central
Installing Url Rewrite
Installing OpenXML
Installing DotNetCore
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
Copying Web Client Files
Copying Windows Client Files
Copying PowerShell Scripts
Copying ConfigurationPackages
Copying Test Assemblies
Copying TestToolKit
Copying UpgradeToolKit
Copying Extensions
Copying ClientUserSettings
Copying ReportBuilder
Changing Database Server Collation to Danish_Greenlandic_100_CI_AS
SQL Server 2017 transmits information about your installation experience, as well as other usage and performance data, to Microsoft to help improv
e the product. To learn more about SQL Server 2017 data processing and privacy controls, please see the Privacy Statement.
Copying Cronus database
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Starting Business Central Service Tier
Installation took 115 seconds
Installation complete
Initializing...
Starting Container
Hostname is SVS-14-CU3-DK
PublicDnsName is SVS-14-CU3-DK
Using NavUserPassword Authentication
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 0A9997D50C9F0F623641AAB23AB24FAA75147CAB
Modifying Service Tier Config File with Instance Specific Settings
Restarting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Using license file 'c:\run\my\license.flf'
Import License
The server "NAV" was unable to process the request. The application will close.
at , C:\Run\SetupLicense.ps1: line 29
at , C:\Run\navstart.ps1: line 156
at , C:\Run\start.ps1: line 121
at , : line 1

An Error Occurred:
Initialization of container SVS-14-CU3-DK failed
Press Key To Continue!

Container Event Log attached
SVS-14-CU3-DK.zip

If you leave out the licensefile parameter - does it work then?

It works without the license parameter, but just gives some warnings about incompatible license in the container. See details below.
BC License details:
Microsoft Software License Information
Copyright (C) 2007-2018 Microsoft Corporation. All rights reserved

VOICE Kontonummer       : 4944540
Licensed to             : Elbek & Vejrup A/S
Tangen 6
                      rhus N
                      84
                      8200
                      Denmark

Produktlinie            : Microsoft Dynamics 365 Business Central on premises
Produktudgave           : Unspecified
Produktversion	        : 14
Land                    : Denmark
Sprog                   : Danish
Oprettet Dato           : 05-07-2019 00:37:55 (-07:00)
Udl>ber                 : 03-10-2019
Konfiguration           : BC365

Navn                                                                   Antal

Country Code: Denmark                                                      1
Demo/Dev Named User                                                      150
Dyn365 Business Central Team Members                                     300
Partner Demo/Dev Module                                                    1
Product Specialist                                                         1
Solution Developer                                                         1

Container details (take 2 without license parameter):
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 7BA0A3F7245F60E89636413FBDCAA9B661277540
Modifying Service Tier Config File with Instance Specific Settings
Restarting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Creating http download site
Setting SA Password and enabling SA
Creating krj as SQL User and add to sysadmin
WARNING: This license is not compatible with this version of Business Central.
Creating SUPER user
WARNING: This license is not compatible with this version of Business Central.
WARNING: This license is not compatible with this version of Business Central.
Container IP Address: <IP_ADDRESS>
Container Hostname  : SVS2-14-CU3-DK
Container Dns Name  : SVS2-14-CU3-DK
Web Client          : http://SVS2-14-CU3-DK/NAV/
Dev. Server         : http://SVS2-14-CU3-DK
Dev. ServerInstance : NAV
Files:
http://SVS2-14-CU3-DK:8080/al-3.0.154634.vsix
Initialization took 97 seconds
Ready for connections!
Reading CustomSettings.config from SVS2-14-CU3-DK
WARNING: This license is not compatible with this version of Business Central.
WARNING: This license is not compatible with this version of Business Central.
Creating Desktop Shortcuts for SVS2-14-CU3-DK
Importing Objects from C:\TestToolKit\CALTestCodeunits.DK.fob (container path)
Importing Objects from C:\TestToolKit\CALTestLibraries.DK.fob (container path)
Importing Objects from C:\TestToolKit\CALTestRunner.fob (container path)
Adding SQL-Server credentials for authentication
Creating application symbols in SVS2-14-CU3-DK
[0] [29-08-19 14:11:42] The command completed successfully in '371' seconds.
Symbols successfully generated
TestToolkit successfully imported
Container SVS2-14-CU3-DK successfully created
Clone: https://bitbucket.org/ElbekVejrup/svane-shipping.git
Copy Add-ins
Welcome to the Server Admin Tool Shell!
For a complete list of Server cmdlets type
Get-Command -Module Microsoft.Dynamics.Nav.Management, Microsoft.Dynamics.Nav.Apps.Management
VERBOSE: Performing the operation "Set-NAVServerInstance" on target "ServerInstance = MicrosoftDynamicsNavServer$NAV, Start = False, Stop = False,
Restart = True".
VERBOSE: Attempting to stop service MicrosoftDynamicsNavServer$NAV
VERBOSE: Service MicrosoftDynamicsNavServer$NAV is stopped
VERBOSE: Service HTTP is already running
VERBOSE: Attempting to start service MicrosoftDynamicsNavServer$NAV
VERBOSE: Service MicrosoftDynamicsNavServer$NAV is running
Open VS Code
Done

How do you specify the license parameter?
Is it a URL or is it a local file?

It will be copied from UNC Share to local folder and used as parameter, See full script here and check about line 70-76 for details:
$ContainerName = 'SVS2-14-CU3-DK'
$BCTag = '1904-cu3-dk'
$GitRepository = 'https://bitbucket.org/ElbekVejrup/svane-shipping.git'
if (-not ([Security.Principal.WindowsPrincipal][Security.Principal.WindowsIdentity]::GetCurrent()).IsInRole([Security.Principal.WindowsBuiltInRole]::Administrator)) {
$Arguments = "& '" + $myinvocation.mycommand.definition + "'"
if (!$myinvocation.MyCommand.Definition.StartsWith("\")) {
$DriveInfo = New-Object System.IO.DriveInfo($myinvocation.MyCommand.Definition)
$DriveInfo.drivetype | Out-Null
If ($DriveInfo.drivetype -ne "Fixed") {
$DrivePrefix = $myinvocation.MyCommand.Definition.Substring(0, 2);
$ManagementObject = New-Object System.Management.ManagementObject("Win32_LogicalDisk='$($DrivePrefix)'")
$UNCPrefix = $ManagementObject["ProviderName"].ToString()
$Arguments = $Arguments.Replace($DrivePrefix, $UNCPrefix)
}
}
Start-Process powershell -Verb runAs -ArgumentList $Arguments
Break
}
Clear-Host
$ContinueInstall = $true
Try {
$ContainerIds = docker ps -a --filter name=$ContainerName --format '{{.Names}};{{.ID}};{{.Status}}' --no-trunc
$ContainerIds | ForEach-Object {
if ($.Split(';').Item(0).ToUpper() -eq $ContainerName.ToUpper()) {
$ContainerId = $.Split(';').Item(1)
$IsRunning = ($_.Split(';').Item(2).SubString(0,2).ToUpper() -eq "UP")
}
}
if ($ContainerId) {
Write-Host
Write-Host -ForegroundColor Red "Container " -NoNewline
Write-Host -ForegroundColor Green $ContainerName -NoNewline
Write-Host -ForegroundColor Red " is allready created do you want to overwrite (Y/N) " -NoNewline
$ContinueInstall = (Read-host).ToUpper() -eq 'Y'
if (!$ContinueInstall) {
if (!$IsRunning) {
Write-Host
Write-Host -ForegroundColor Green "Start Container: $ContainerName"
docker start $ContainerId
}
}
}
if ($ContinueInstall) {
$Module = "NavContainerHelper"
$ModuleExist = Get-Module -Name $Module -ListAvailable
if (!($ModuleExist)) {
Write-host
Write-Host -ForegroundColor Green "Install $Module (Y/N)" -NoNewline
$Install = (Read-host).ToUpper()
if ($Install -eq 'Y') {
Write-Host
Write-Host -ForegroundColor Green "Installing $Module"
Install-Module -Name $Module -Force
Import-Module -Name $Module
}
}
else {
Write-Host
Write-Host -ForegroundColor Green "Looking for update to module: $Module"
$NewVersion = $(Find-Module $Module) | Select-Object Version
$CurrentVersion = $ModuleExist[0] | Select-Object Version
if ($NewVersion.Version -gt $CurrentVersion.Version) {
Write-Host
Write-Host -ForegroundColor Green "Update $Module from: $($CurrentVersion.Version) to: $($NewVersion.Version)"
Update-Module $Module -Force
Get-InstalledModule $Module -RequiredVersion $CurrentVersion.Version | Uninstall-Module
}
Write-Host
Write-Host -ForegroundColor Green "Import Module $Module"
Import-Module -Name $Module -WarningAction SilentlyContinue
}
Write-Host
$LicensFile = Get-Item "\ev-filsrv01\Navision\EV Internal Tools\Powershell\License*BC*.flf" | Sort-Object LastWriteTime -Descending | Select-Object -First 1
if ($LicensFile) {
Copy-Item -Path $LicensFile -Destination (Join-Path "$env:ProgramData\NavContainerHelper" "BC.flf") -Force -Recurse
} else {
$LicensFile = Get-Item "$env:ProgramData\NavContainerHelper\BC.flf" -ErrorAction SilentlyContinue
}
if (!$LicensFile) {
Throw "No Licens File Exist! ($($env:ProgramData)\NavContainerHelper\BC.flf)"
}
Write-Host -ForegroundColor Green "Creating Container: $ContainerName"
# New-NavContainer -isolation process -auth NavUserPassword -updateHosts -accept_outdated -enableSymbolLoading -doNotExportObjectsToText -includeCSide -containerName $ContainerName -imageName "mcr.microsoft.com/businesscentral/onprem:$($BCTag)" -accept_eula -licenseFile $LicensFile.FullName -includeTestToolkit -restart no -useBestContainerOS
New-NavContainer -isolation process -auth NavUserPassword -updateHosts -accept_outdated -enableSymbolLoading -doNotExportObjectsToText -includeCSide -containerName $ContainerName -imageName "mcr.microsoft.com/businesscentral/onprem:$($BCTag)" -accept_eula -includeTestToolkit -restart no -useBestContainerOS
}
if (($GitRepository -ne '') -and ($GitRepository)) {
$LocalRepoFolder = (Join-Path $([Environment]::GetFolderPath("MyDocuments")) "Al")
if ((Test-Path $LocalRepoFolder) -ne $true) {
New-Item -Path $LocalRepoFolder -ItemType Directory
}
$RepoFolder = (Join-Path $LocalRepoFolder $([System.IO.Path]::GetFileNameWithoutExtension($($GitRepository.Split('/') | Select-Object -Last 1))));
if ((Test-Path $RepoFolder) -ne $true) {
Push-Location $LocalRepoFolder -StackName "Default"
Write-Host
Write-Host -ForegroundColor Green "Clone: $GitRepository"
Start-Process Git -ArgumentList "clone $GitRepository" -Wait
Pop-Location -StackName "Default"
}
Push-Location $RepoFolder
if (Test-Path .\App\Add-ins) {
Write-Host
Write-Host -ForegroundColor Green "Copy Add-ins"
$CopySession = New-PSSession -ContainerId (Get-NavContainerId $ContainerName) -RunAsAdministrator
$AddinsFolder = Invoke-Command -Session $CopySession -ScriptBlock {
return Get-Item 'C:\Program Files\Microsoft Dynamics NAV*\Service\Add-ins'
}
Copy-Item -ToSession $CopySession -Path ".\App\Add-ins*" -Destination $AddinsFolder -Recurse -Force
Invoke-Command -Session $CopySession -ScriptBlock {
Import-Module 'C:\Program Files\Microsoft Dynamics NAV*\Service\NavAdminTool.ps1' | Out-Null
$CurrentServerInstance = Get-NAVServerInstance -ServerInstance NAV
$CurrentServerInstance | Set-NAVServerInstance -Restart -Verbose
}
Remove-PSSession $CopySession
}
    Write-Host
    Write-Host -ForegroundColor Green "Open VS Code"
    if (Test-Path (Join-Path $RepoFolder App)) {
        Code (Join-Path $RepoFolder App)
    }
    else {
        Code $RepoFolder
    }
}

} Catch {
if ($CopySession) {
Remove-PSSession $CopySession
}
Write-Host
Write-Host -ForegroundColor Red "An Error Occurred:"
Write-Host -ForegroundColor Red $_
Write-Host
Write-Host -ForegroundColor Green "Press Key To Continue!" -NoNewline
Read-Host
}
Write-Host
Write-Host -ForegroundColor Green "Done"
Start-Sleep -Seconds 2

Could the file be marked as blocked?
before new-navcontainer you could call unblock-file $licensefile.fullname  - don't know if that changes anything

The file is not in quarantine by Windows and the GPO for preserving zone info when files are copied over the network is disabled on my pc.
When I copy the file manually and check the file properties on the destination it is not blocked either.
I will implement your suggestion in our script and manually install our dev license into the container.
I will be out of office until tuesday, but have enough input/inspiration to handle this situation. Thank for your input.

Just as information.
The issue I had with using the license file from UNC share was not an issue my colleagues had even though we have the same permissions to the UNC share and are running same version of NAV-containerhelper.
Anyways; it works for everybody if we copy the license file from the UNC share to $env:ProgramData\NavContainerHelper\BC.flf and pass on this local path as the parameter for the container creation.


MODLD-582: LCCN Validation | Ensure uniquely assigned

Hi @AndreiBordak,
Let us make following 2 changes in the Module descriptor.

Add search.instances.collection.get permission to the modulePermissions of POST /resource and PUT /resource/id API. (As GET /search/instance API is protected by this permission. See here
Add following to the "requires" array of module descriptor
{
"id": "search",
"version": "1.3"
}

Hi @AndreiBordak,
There are some sonar violations. Please take a look

There are 4 New issues according to SQ report above. Let's fix'em at once please?

4 New issues

fixed all except:

4 New issues

fixed all except:

Let's ignore it via SuppressWarnings too, it's definitely a false-positive


New line spaces and markdown issue templates returning {}

If using this in a private repository, we have to use the markdown issue templates vs. issue forms that's in beta only for public repositories.
If we have a markdown response that doesn't have new lines between the heading and response, it returns as an empty JSON response {}. An example of is below:
### Repo name
repo-name

### team
team-name

If we ensure that there are spaces before the question/answer, then it works. Like so:
### Repo name

repo-name

### team

team-name

Yeah this is a known limitation. Reach out to GitHub and ask them to enable the beta for your org. I'm pretty sure they will.

@stefanbuck can’t we maybe parse for a single new line vs 2 new line characters?
Not sure what your implementation is looking at though.


sample rate

Why did I set the sample rate of SpeedInsights to 0.01, but it seems that the number of data points did not become one percent of the original?

Hey! Did datapoints decrease or did nothing change at all for you?

Datapoints has decreased significantly.

What is the issue then? I am not sure if I can follow

When I set the sampling rate to 0.01, I expected the number of data points to decrease to approximately 1% of the original quantity. However, it only decreased to 20% of the original quantity. After checking and calculating, I believe this may be due to the statistical reference data not covering the entire day or normal fluctuations. We can consider this matter resolved. Thank you for your assistance.


ARROW-9146: [C++][Dataset] Lazily store fragment physical schema

The physical schema is required to validate predicates used for filtering row groups based on statistics.
It can also be explicitly provided to ensure that if no row groups satisfy the predicate no I/O is done.

I suppose we don't / don't want to support files being changed after the dataset object has been constructed anyways?

No, we don't support this at all. IMHO if fragments are modified that calls for the viewing Dataset to be reconstructed. @fsaintjacques

The test is failing on windows. https://ci.appveyor.com/project/ApacheSoftwareFoundation/arrow/builds/33736283/job/mshkl837y3b5v5u6

This has a python lint failure: https://github.com/apache/arrow/pull/7526/checks?check_run_id=811470274#step:7:1377

I'll merge after Appveyor passes, ignoring Travis


Login for main page

Hi, how can I make users have to login to see the status page? We dont want the page being shared!
Thanks

You can configure apache's basic HTML authentication


Add changes needed to use rustfmt as a subtree in rust-lang/rust

These changes can be replicated by checking out https://github.com/rust-lang/rust/pull/82208 and then running git subtree push --prefix=src/tools/rustfmt <some remote here> subtree. That command took only a few seconds to run, I didn't run into any issues with repo size like Clippy has in the past.
This shouldn't be merged until there's some sort of decision on https://github.com/rust-lang/rust/pull/82208.

I don't think that PR is the right place for decisionmaking, I think we should file a separate issue and close that PR

The repo size issues will occur on the subtree pull, pulling changes back in from Rust.

What clippy typically does is pin a rust nightly via rust-toolchain, and update it when we do syncs.

What clippy typically does is pin a rust nightly via rust-toolchain, and update it when we do syncs.

Yup, that was my plan :) if you look at rust-toolchain I changed it to add the rustc-dev component.

Know the question about beta in CI has primarily been discussed elsewhere but posting here given context and this being the main PR left on this side of the equation.
I don't see any issues with dropping beta from our CI jobs that would warrant preventing moving forward with trying the subtree approach. The only active use cases are tests that verify the ability/inability to use certain formatting config options on nightly/non-nightly, and I can't think of nor find any motivations for CI on beta that predate my involvement with the project either.

Makes sense, and I'll point out that rustc will be CIing beta rustfmt with beta rustc anyway


Add new target 'json'

Is your feature request related to a problem? Please describe.
Currently 'updatecli' only works for YAML target files and it would be nice to support file format like JSON

This library may be helpful
-> https://github.com/itchyny/gojq#usage-as-a-library

Available since v0.34.0


Diode Insertion Problem at TritonRoute

Description
To reduce antenna violations, I tried the configuration DIODE_INSERTION_STRATEGY = 5. When I did that, I got an error message in detailed routing phase. The log file is full of these warning messages, which can be seen in attachments.
...
[WARNING DRT-0246] ANTENNA__17693__A/DIODE from _00054_ has nullptr as prefAP.
[WARNING DRT-0246] ANTENNA__17694__A2/DIODE from _00054_ has nullptr as prefAP.
[WARNING DRT-0246] ANTENNA__17695__A/DIODE from _00054_ has nullptr as prefAP.
[WARNING DRT-0246] ANTENNA__17703__A/DIODE from _00054_ has nullptr as prefAP.
[WARNING DRT-0246] ANTENNA__17711__A/DIODE from _00054_ has nullptr as prefAP.
[WARNING DRT-0246] ANTENNA__23141__S/DIODE from _00116_ has nullptr as prefAP.
...

Flow successfully finishes when DIODE_INSERTION_STRATEGY = 3. In that case there is no diode usage. It seems like the problem is with diode usage, but I have no clue what is wrong.
Reproduction Material
There is a tarball file with a source folder and configuration file in. This can be reproduced in openlane bash.
[sources.tar.gz](https://github.com/The-OpenROAD-Project/OpenLane/files/7386389/sources.tar.gz)

Logs
[24-tritonRoute.log](https://github.com/The-OpenROAD-Project/OpenLane/files/7381832/24-tritonRoute.log)

What version of OpenLane are you using?

@vijayank88 - Can you run this test case and check to see if the antenna violations persist with the latest OL Tag?

@dralabeing yes with latest OL tag 2021.10.25_20.35.00-11-g18a17f4 got same issue as user reported.
@BYCakar  with default DIODE_INSERTION technique passed flow successfully, why you're trying DIODE_INSERTION_STRATEGY = 5 again. Can your share number of antenna violation left with DIODE_INSERTION_STRATEGY = 3 and flow_summary.log to understand your issue?

I've updated my design but same problem still exists. Thus I uploaded my latest run's antenna report and flow summary files.
antenna_and_summary.zip

This still needs a packaged test case in order to make any progress.  Logs & reports are not enough.


Aggregate values

When N pipes are connected to same input we can aggregate the values coming from all upstream components and thus do not loose any data. The aggregate can also contain metadata, which will be useful to understand where from each signal comes from

Agregate signals are implemented in xp4


Cannot download video from Facebook

Checklist

[x ] I'm reporting a broken site support
[x ] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ x] I've searched the bugtracker for similar issues including closed ones

Verbose log
PASTE VERBOSE LOG HERE
aa@tcm73:~$ youtube-dl "https://www.facebook.com/watch/live/?ref=watch_permalink&v=279718980981125" --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.facebook.com/watch/live/?ref=watch_permalink&v=279718980981125', '--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.6.9 (CPython) - Linux-5.3.0-62-generic-x86_64-with-LinuxMint-19.3-tricia
[debug] exe versions: avconv 3.4.6, ffmpeg 3.4.6, ffprobe 3.4.6
[debug] Proxy map: {}
[facebook]<PHONE_NUMBER>81125: Downloading webpage
[facebook]<PHONE_NUMBER>81125: Downloading webpage
ERROR: Cannot parse data; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper
return func(self, *args, **kwargs)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
ie_result = self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/facebook.py", line 680, in _real_extract
return self._extract_from_url(real_url, video_id)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/facebook.py", line 594, in _extract_from_url
raise ExtractorError('Cannot parse data')
youtube_dl.utils.ExtractorError: Cannot parse data; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Description
WRITE DESCRIPTION HERE
Cannot download the video from Facebook. The video plays well in the browser.
Thanks.

This content isn't available right now


Add feature to show resulting config

Currently the default template is included in limactl info, but in a different format (JSON instead of YAML)
It would be more useful if it was shown in the same format, without the comments and with values replaced...
limactl config [FILE.yaml...]

Also add a small helper function, to compare a template against the default.
This is more handy than dumping two yaml files and running diff on them.
limactl config --diff FILE.yaml...

Currently the default template is included in limactl info, but in a different format (JSON instead of YAML)

And I have wondered what the use case was for that as well: https://github.com/lima-vm/lima/pull/280#discussion_r721628426

It would be more useful if it was shown in the same format, without the comments and with values replaced...

Useful for what? Just removing the comments?
$ yq eval '... comments=""' /usr/local/share/doc/lima/examples/default.yaml
arch: "default"
images:
  - location: "~/Downloads/impish-server-cloudimg-amd64.img"
    arch: "x86_64"
  - location: "~/Downloads/impish-server-cloudimg-arm64.img"
    arch: "aarch64"
  - location: "https://cloud-images.ubuntu.com/impish/current/impish-server-cloudimg-amd64.img"
    arch: "x86_64"
  - location: "https://cloud-images.ubuntu.com/impish/current/impish-server-cloudimg-arm64.img"
    arch: "aarch64"
cpus: 4
memory: "4GiB"
disk: "100GiB"
mounts:
  - location: "~"
    writable: false
  - location: "/tmp/lima"
    writable: true
ssh:
  localPort: 0
  loadDotSSHPubKeys: true
  forwardAgent: false
containerd:
  system: false
  user: true
firmware:
  legacyBIOS: false
video:
  display: "none"
networks:
propagateProxyEnv: true
useHostResolver: true

I would find the config command more useful (after #436 is done) if it displayed the merged config, and not just the original YAML without comments.

limactl config [FILE.yaml...]

Also add a small helper function, to compare a template against the default.
This is more handy than dumping two yaml files and running diff on them.

Maybe, but not really by a lot.

limactl config --diff FILE.yaml...

This I find somewhat useful, except I would want to be able to compare any 2 config files, and not just one file against the default.
However, you can mostly already do this with yq as well; it just doesn't fill in the default values, so they show up in the diff if just one of the files specifies them.
$ diff -u <(yq eval -P '... comments=""' examples/docker.yaml) <(yq eval -P '... comments=""' examples/podman.yaml)

But even then I see very little benefit over just running diff -u between the 2 files, at least for the config files I used for testing.
My Main point
I'm concerned about "littering" limactl with too many subcommands that are most likely useless for the vast majority of users. They will present a cognitive barrier for new users to getting started with lima. If nothing else, it is more and more stuff to read through, just to find out it is not relevant to you.
Not sure what the right level of compromise is, but I feel like both config and du are not really important subcommands (and yes, I've mostly felt like that about info too, but I can see it being useful as we add more information to it).
To me the barrier to inclusion in limactl should be: "What end-user problem is this solving, and how?". There should be a compelling answer to this before we add anything.
Maybe the commands that don't clear the barrier can be moved to a separate utility/helper tool?

Maybe we can have config and du as subcommands of limactl debug. info can be perhaps moved/aliased there too.

I was looking at the similar config command for compose, that is why.
But it can be done in the shell instead, using yq and friends, so closing.

It would be more useful if it was shown in the same format, without the comments and with values replaced...

Useful for what?

I meant that it could be fed back into the tool, as config for another instance. Or diffed. Harder to do, with json vs yaml ?

I'm concerned about "littering" limactl with too many subcommands that are most likely useless for the vast majority of users.

Good thing I didn't include the "color diff" then, which was mostly left out due to adding dependencies (and thus some extra bloat)


No map display, no any errors in any consoles

Vaadin: 24.1.4
Flow: 24.1.5
Java: Oracle Corporation 19.0.2
OS: x86_64 Mac OS X 13.4.1
Browser: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Live reload: Java active (Spring Boot Devtools): Front end active
Browser console log, attached.
localhost
Screen shot
GoogleMapsView

Works well :) Screen
Solution is : set true in properties vaadin.productionMode = true :)
In vaadin debug mode it doesn't work :(

Hi @XandrMaster if you're using spring you need to add com.flowingcode to the vaadin.whitelisted-packages property, see here.

Hey @paodb , it works good 👍 , thank you for help me :) , I would to read Special configuration for Spring before start , sorry .


yaml file location

Hi,
what is the right location for the yaml files for the 'from_file_keyword'?  What if I want to ship them as a package?
cheers,
Martin

+1 to this. Would love to know the answer or solutions that you may have found.

Relative path is also OK.
The root path is your project path


metrics fmeasure and matthews_correlation don't work batchwise

Hello,
In my opinion, the metrics fmeasure, matthews_correlation, precision and recall all don't work batchwise. In general, this is the case for all metrics which incorporate true/false positives/negatives.
Here is a small and easy counterexample:
Let's assume we have just 4 samples: two negatives and two positives. Also, our batch size is 2:

Batch
Label
Prediction

1
0
0

1
0
1

2
1
0

2
1
1

Now we want to calculate the recall aka. TP-rate in batchwise manner. For the first batch, we have a TP-rate of 0 (since we don't have any true positives). For the second batch, we have a TP-rate of 0.5. Finally, we take the mean over all batches and end with recall=TP-rate=mean(0 + 0.5) = 0.25.
But: as we easily can see the correct recall is 0.5. The problem with the batchwise calculation is that we wrongly incorporated the first batch.

I think I stumbled upon the same problem while experimenting with an unbalanced dataset. Here is an example I wrote to isolate the problem:
from keras.models import Sequential
from keras.layers import Dense
import numpy as np

np.random.seed(1380)

input_data = np.random.choice(2,10000, p = [0.99,0.01])

model = Sequential()
model.add(Dense(1, input_dim=1, init='uniform', activation='sigmoid'))

model.compile(loss='binary_crossentropy',
              optimizer='adam',
              metrics=['accuracy', 'precision', 'recall'])

model.fit(input_data, input_data, nb_epoch=2, batch_size=2, class_weight = {0:0.01, 1:0.99})

Output:

Epoch 1/2
10000/10000 [==============================] - 12s - loss: 0.0130 - acc: 0.7067 - precision: 0.0187 - recall: 0.0218
Epoch 2/2
10000/10000 [==============================] - 11s - loss: 0.0100 - acc: 1.0000 - precision: 0.0230 - recall: 0.0230

In this simple example the classification is always correct, yet the precision and recall is almost 0. It goes up when the batch size is increased. It roughly follows the probability of getting a random batch sized subset of the data where at least on sample is 1. (In this example 1-(.99**2) = 0.019)
That is why I think that this is the same issue the OP described.

I'm also using a binary classifier and seeing some suspicious metrics reported. Namely ever epoch reports the exact same value for accuracy, precision and recall over hundreds of thousands of epochs. This occurs in both training and evaluation.
4736/4789 [============================>.] - ETA: 1s - loss: 0.5149 - acc: 0.8566 - precision: 0.8566 - recall: 0.8566
4768/4789 [============================>.] - ETA: 0s - loss: 0.5152 - acc: 0.8565 - precision: 0.8565 - recall: 0.8565
4789/4789 [==============================] - 166s - loss: 0.5153 - acc: 0.8563 - precision: 0.8563 - recall: 0.8563 - val_loss: 0.6714 - val_acc: 0.7749 - val_precision: 0.7749 - val_recall: 0.7749
Epoch 4/100
  32/4789 [..............................] - ETA: 162s - loss: 0.4381 - acc: 0.8750 - precision: 0.8750 - recall: 0.8750
  64/4789 [..............................] - ETA: 167s - loss: 0.5680 - acc: 0.8125 - precision: 0.8125 - recall: 0.8125
  96/4789 [..............................] - ETA: 164s - loss: 0.5151 - acc: 0.8438 - precision: 0.8438 - recall: 0.8438
 128/4789 [..............................] - ETA: 161s - loss: 0.4999 - acc: 0.8516 - precision: 0.8516 - recall: 0.8516```

@laxatives I am seeing the exact same thing also.

@laxatives I have this same problem with a binary classifier with a balanced dataset.  I use a generator and have verified that it gives balanced data each call to next().
@DepthFirstSearch @sietschie Have you figured out the issue?

@laxatives @isaacgerg I don't think you have the same issue as @sietschie or me.
As I stated in my initial post, I don't think it's possible to compute fmeasure, precision etc. in a batch-wise manner (see my example). That's the whole issue here.

@DepthFirstSearch During training these metrics are meant to be computed for each batch, not for all the batches.  In any case, for a binary classifier, the metrics are still being reported incorrectly even on a per batch basis.  See #5400.


Data: project addition by Deepanshu #981

Related Issue
This is issue number #981
Description
Adde new project of Google keeper clone using ReactJs
Screenshots

Changed the title


Dragging to far on Android tablet makes item disappear

Greetings..
Your sortable JS works great on desktop browser.  But if you use an android tablet and you drag items to far they disappear.  Not sure if this is by design or not.
Thank you in advance for any advice you my offer.
Cheers
Mark..

Hey, this is definitely not by design. 😉  Anyway if you can provide any information or even better a fix that would be very appreciated.

It is reproducable on edge. Just take the example file, take the first item of the first list, drag up, and poof!

This appears to be the same bug as #424


Link outdated in Learn Web Development - Floats

One of the first links in https://developer.mozilla.org/en-US/docs/Learn/CSS/CSS_layout/Floats points to a section of a previous article that no longer exists (or not there anyway): Fancy paragraph example

Thanks for noticing! I've updated the link to one that works — a CSS Tricks article on drop caps.


Packaging

This work reorganizes the Janus repository so that it becomes a PyPI package.
The main change is that the Socrates radiation code is now put out of the repository for clarity. It is assumed that is is preinstalled before installing and using Janus (see new README.md).
Module sources are now in the src folder, as well as all data files (spectral files and luminosity tracks). Other utility scripts are in the tools folder. Demo scripts are in the test folder for future GH actions to be set up.

Just for confirmation: you specifically selected Harrison and Stef to review this, right? Fine for me, just to avoid confusion. :)

@lsoucasse Just for confirmation: you specifically selected Harrison and Stef to review this, right? Fine for me, just to avoid confusion. :)

Yes, because it is software-oriented and not model-oriented. But I am happy to add you as well.

It now installs fine. To get it working, I removed importlib from the pyproject.toml file, and also installed a version of setuptools packaged by conda rather than by pip. It was also missing the dependency f90nml.
With these changes, the model output looks good. Do you want me to push these to this branch, @lsoucasse ?

It now installs fine. To get it working, I removed importlib from the pyproject.toml file (as suggested here), and also installed a version of setuptools packaged by conda rather than by pip. It was also missing the dependency f90nml.
With these changes, the model output looks good. Do you want me to push these to this branch, @lsoucasse ?

Yes, please do.

I agree. Maybe we could pull the spectral files from OSF using this library?

Hi @lsoucasse , looks good, it seems to install fine with pip install -e . and pip install . on python 3.12. It's a bit difficult to test everything, because there are no tests. I tried to import some files and it seems mostly fine.
Couple of notes:
* pyproject.toml is missing metadata, like the required python version, licence information, urls, etc: https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#a-full-example

* Missing import: f90nml, phys, seaborn

* The package is very large with a 230 MB download, and 873 MB unpacked. Nearly all of this is about the spectral files that are stored as plain text (872 MB). This is something we could think of optimizing.

Thanks Stef!

Licence information and project url are already in the toml file. What do you want specifically to add? For the minimum python version requirement do you have any suggestion @nichollsh?
The "phys" module is actually an in-house one in janus.utils. There are some ambiguous imports I will fix. Maybe worth considering renaming this module?
For the data management, we have to define a strategy for the whole PROTEUS platform. Maybe it is relevant to have some spectral data in the package (for basic usage, tests) but I agree it is better to have most of them stored in a dedicated platform.

The code certainly requires Python >= 3.10, because it includes some match statements which were only introduced in that version. I've been using 3.11 recently, but I think 3.10 is a reasonable requirement.
I agree that we should leave at least 1 spectral file in for tests, and that we should think about the rest.

All good, I was thinking of being explicit about the licence like so:
[project]
license = {text = "Apache 2.0 License"}

This will show up nicer in pip show and other places, because it does not copy the entire file 😅
And also keywords which are useful for pypi:
[project]
keywords = ["egg", "bacon", "sausage", "tomatoes", "Lobster Thermidor"]

@timlichtenberg and @nichollsh, do you have any relevant keyword in mind for the Janus package? I would suggest "exoplanet" and "atmosphere".


Update Register 2, 4, 6 values min_max with checking Register 2 min/max which depends on emmission type installed

hinot exactly:)the value range depends on emission type is huge:)eg user have emission type radiant so he isn’t allowed to put value below 18 but without this user can send value 5 to heat pump -so in return on next read he theoretically get this minimal value of 18 -even he send 5.so this part adjust and limits to send incorrect -not accepted value to heat pump. user feedback is similar with this part of code like without. these hard coded values are max and min values accepted by heat pump. there isn’t possibility in this moment with esp home to send this info for user -ideally would be to hardcode this to get min and max value -but for this moment esp home don’t support lambda calls in min and max and only solution is build for these sensors own full part of code in loop section…this in future could be more help to add lambdas for min and max and then it could be removed from lambda write part;)but tyou are owner of basic code -so if you want it could be removed -no problem, but also very wide range of temperatures persist 5-60personally i use 5 deg in summer and in winter time 27-44deg ;)PozdrawiamRyszard GwizdakMARM.pl Sp. z o.o.Wiadomość napisana przez Richard Arends @.***> w dniu 26.10.2024, o godz. 17:43:
@Mosibi commented on this pull request.
In heatpump.yaml:

@@ -3189,23 +3078,49 @@ number:
value_bytes[0] = value >> 8;     // high byte (zone 2)
value_bytes[1] = value & 0x00FF; // low byte (zone 1)

 // ESP_LOGI("","Zone 1 is %d", value_bytes[1]);

 // ESP_LOGI("","Zone 2 is %d", value_bytes[0]);

 auto operation_state = id(${devicename}_operational_mode).state;

Rereading your code helped 😉 What you do is set the limit based on the emission type. I see two issues with this:
The user sets a value, but the system changes that value without feedback to user. If the user does not verify, another value can be used then he/she wanted.
The limits are hard coded and are easily forgotten if these are changed somewhere else.
I rather keep it simple and only define the upper and lower limit, thus as it already is.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

I will keep this in mind, but for now let's move it out and revert it.
Maybe in the (near) future we can introduce a global var that defines the limits for the specific emission types. Then we can use those vars in both the default range and in the code. That way we only have to change it at one spot.

oki will revert nowno you can’t neither set min_value and max_value values. it should be done by make pull request for new feature to esp home -you can’t update these values by lambdaPozdrawiamRyszard GwizdakMARM.pl Sp. z o.o.Wiadomość napisana przez Richard Arends @.***> w dniu 26.10.2024, o godz. 19:44:
I will keep this in mind, but for now let's move it out and revert it.
Maybe in the (near) future we can introduce a global var that defines the limits for the specific emission types. Then we can use those vars in both the default range and in the code. That way we only have to change it at one spot.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

@rysiulg merged. Again thank you for your contribution !


Release azure-cli-batch 2.0.3

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Started releasing azure-cli-batch 2.0.3...

Release of 'azure-cli-batch' with version '2.0.3' successful. View at https://pypi.python.org/pypi/azure-cli-batch/2.0.3.

GitHub release created. https://github.com/Azure/azure-cli/releases.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3210   +/-   ##
=======================================
  Coverage   70.35%   70.35%
=======================================
  Files         381      381
  Lines       24777    24777
  Branches     3785     3785
=======================================
  Hits        17431    17431
- Misses       6237     6240    +3
+ Partials     1109     1106    -3

Impacted Files
Coverage Δ

...-cli-role/azure/cli/command_modules/role/custom.py
35.74% <0%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
68.61% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
34.69% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Using arguments according to README fails.

Cookbook version
3.0.0
Chef-client version
12.19.36
Platform Details
debian/ubuntu
Scenario:
Using chef_handler resource with arguments to pass ap ath according to your README.  Switching to a custom resource from LWRP seems to have tightened the property checking.
Changing to make the parameter explicitly an array seems to fix:
 chef_handler 'Chef::Handler::JsonFile' do
    source 'chef/handler/json_file'
    arguments [path: '/var/chef/reports']
    action :enable
  end

Perhaps this is just a documentation issue.
Steps to Reproduce:
Use chef_handler according to the README"
 chef_handler 'Chef::Handler::JsonFile' do
    source 'chef/handler/json_file'
    arguments path: '/var/chef/reports'
    action :enable
  end

Expected Result:
Have it converge.
Actual Result:
....
Chef::Exceptions::ValidationFailed
  ----------------------------------
  Property arguments must be one of: Array, String!  You passed {:path=>"/var/chef/reports"}
....

This should be fixed in 3.0.1. Thanks for the report


Open access url validation fix

Closes #459

@ajkiessl I took my user-agent changes back since it did not work, I am just pushing the change for the spec for now and we can keep the other issue in the backlog to visit later.