Field order affects object deserialization

Source/destination types
public class Message : MessageBase
{
    public long Value { get; set; }
}

public abstract class MessageBase
{
}

Test code
public class DeserializationTest
{
    [Fact]
    public void Should_DeserializeRequest()
    {
        var deserialized = JsonConvert.DeserializeObject<MessageBase>($@"{{
    ""Value"": 400,
    ""$type"": ""{typeof(Message).AssemblyQualifiedName}"",
}}", new JsonSerializerSettings
        {
            TypeNameHandling = TypeNameHandling.Auto
        });
        Assert.NotNull(deserialized);
    }

    [Fact]
    public void Should_DeserializeRequest2()
    {
        var deserialized = JsonConvert.DeserializeObject<MessageBase>($@"{{
    ""$type"": ""{typeof(Message).AssemblyQualifiedName}"",
    ""Value"": 400,
}}", new JsonSerializerSettings
        {
            TypeNameHandling = TypeNameHandling.Auto
        });
        Assert.NotNull(deserialized);
    }
}

Expected behavior
Both test passes
Actual behavior
First test fails
Explanation
As per JSON spesification (emphasis mine):

An object is an unordered collection of zero or more name/value
pairs, where a name is a string and a value is a string, number,
boolean, null, object, or array.
An array is an ordered sequence of zero or more values.
The terms "object" and "array" come from the conventions of
JavaScript.

Thus these objects are equal because they have same fields with same values. However, currently classes are failing to deserialize when $type field is not the topmost one.
I have an issue because I'm trying to interact with C# from Python where I use json.dumps(message) to create a JSON. As per spesification, it doesn't preserve the order of fields so $type field can appear anywhere in JSON.
I consider it's a bug becase specification says that they are both valid JSONs, while JSON.Net implementation expects object to be ordered.
Here is my repo with both tests implemented: https://github.com/Pzixel/Test.NetonsoftTest . You can easily clone, run and see that that behaviour persists even in the latest JSON.Net 11.0.2

Looks like a dupe #1716


vSphere: Deploy from content library template fails

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Create a vSphere node template: Select Creation Method -> Deploy From Template: Content Library and choose an existing Linux VM template from the library
Create cluster using the node template

Result:

Cluster creation fails.
Node status initially shows Waiting for machine to be running then throws error Timed out waiting for machine to be running

Other details that may be helpful:
Logs show the following error masked as INFO:

[INFO] [node-controller-docker-machine] (jan-mstr1) Unable to perform any actions, change flags and try again

Debug logs:
vsphere-deploy.log
Root cause:
https://github.com/rancher/machine/blob/master/drivers/vmwarevsphere/vsphere.go#L342
Environment information
Rancher 2.3.3
vSphere 6.7

rancher/rancher:v2.3-head
Reproduced this in our 6.7 vSphere environment.

rancher/rancher:v2.3-head 565ef1a85

Create node template to deploy from Content Library
ubuntu-19.0.4-server-cloudimg template
Template is copied from content library and cluster is created successfully

rancher/rancher:master-head 17045525a

Create node template to deploy from Content Library
ubuntu-19.0.4-server-cloudimg template
Template is copied from content library and cluster is created successfully


Tasks about updating links in spring.io

Context
We should update the link of  https://spring.io/projects/spring-cloud-azure#samples  when sample is adjusted.
Now the link in https://spring.io/projects/spring-cloud-azure#samples is azure-spring-boot-samples.

Task

Rename folder name from aad to azure active directory
now folder name aad is not consistant with offical doc, may be confusing.

[ ] rename folder @backwind1233
[x] Update spring reference yaml
[x] update links in samples, azure-sdk-for-java,azure-dev-docs-pr

[x] Update sample links in spring.io

@chenrujun I'm reassigning this to you since this is part of our samples visibility tasks

Closing this issue now.


Haxe parsing improvements

Generics in function types, and haxe metadata parsing.
Should fix issues #372, #379 and #1065.

Nice, even this works:

However, it doesn't deal very well with the incomplete version of it:

IMHO since Haxe syntax can be so varied I'd prefer to first get the parser for Haxe separated in its own file and look into some of the wrong cases. I'd consider this as a separate thing from the current issue at hand.

I've just found a parsing problem, I don't know if it's a regression with this PR, if it's something from my previous PR, or if it's something that has been present for some time but just noticed now. I'll look into it later with the corresponding unit test.

Just checked, it's not a regression with this PR. I also found that metadata in clases is still breaking other things, so I'll try to fix both problems tomorrow.

I also found that metadata in classes is still breaking other things, so I'll try to fix both problems tomorrow.

I didn't close the meta correctly... and I remember, FD adds the meta to the FileModel, IMHO that's completely wrong, the metadata is decorating the class, not the file. I already did this change in my MXML branch, and it allowed to remove a few calls and made it easier to improve some things, So I'll make a different PR with this change tomorrow.


The syntax for embedding a page slide in version 1.19 is not working.

{
      slide: [[slidesname]],
      page: 1
}

will be fixed in 1.20

will be fixed in 1.20

Still not working on version 1.20


Use Bootstrap in index.php

There was a lot of duplicate code between index.php and bootstrap ... this is a first pass at using a single bootup class for both SSI and "normal" elk to remove the duplication.
This may be a bit much for 1,1.1 ... if so we need to still make a couple of minor changes to allow SSI to load from inside ELK ... so things like portals work 😸
If we want to go this way i'll add in some quick docblocks, right now this was really a quick and dirty setup so things should work in 1.1.1 and we can do some proper things in 2.0.  Also if so PLEASE test this out, I have not checked the logs (site or server) for issues just yet.
If however we feel this is to much, I need to make a commit to fix the SSI loading form inside ELK, so let me know which way we feel is the right way (for 1.1.1 that is)

Neat! :smile_cat:

Less duplicate code means less bugs.

I'm a bit unsure about the last commit: 86e289111eb595090bea0bb47d1364f03274aff4
My original idea when I moved from SSI to bootstrap in these files was that bootstrap is the way to initialize the system, while SSI is a kind of demo showing how to use bootstrap and providing few custom functions to work with and since in these files the functions from SSI are not needed, they would just use bootstrap.

Fair enough, I actually had not tried any of those functions on just bootstrap.
I reverted them back to require bootstrap and then added a  new Bootstrap() call.
That call could also be added to the end of the bootstrap file, and if we do that probably should add in a big ass global blablabla statement as well so those are available when the require is done and not like what I did in subscriptions (which I think is necessary if I've got my head around the top level stuff)

Added a standalone param to the constructor, this should make new Bootstrap in theory just what it was/is in 1.0.  Calling it as new Bootstrap(false) should allow it to be used from index, loading just the basics that are common between normal and side entrance startups.  Calling SSI will call bootstrap assuming it has not been called before (allowing it to be included inside of ElkArte.  I suppose SSI could simply become an API to provide addon capability once ElkArte has been loaded.


Fix exhaustive test check

There were 3 issues:

The comparison between counts[...] and Combination() wasn't running at all ((i >> bits) is always 0 for i==0, so the loop would instantly exit).
The counts values were wrong (because it was counting a sum across all implementation), so if they would be compared to something, the test would incorrectly fail.
(i >> bits) is undefined when bits==64.

(1) would be fixed by changing i >> bits to (i >> bits) == 0, but that leaves issue (3), so introduce a mask and use a different way of writing the same, that keeps working for bits==64. (2) is fixed by only incrementing counts[...] for impl==0.
This was discovered through #50.

@gmaxwell @naumenkogs Care to review?

ACK 294b6c5a9cb6231738add963146f3e04bfe356d8
I wish we had a way to test that tests were run.

There is a way: introduce intentional bugs, and see if the tests catch it.


(Webhook) Send a discord message when a user creates a new claim replay request

Sends a message like this
"User morth1 has requested ownership of some replays. You can review this request at http://localhost:8000/users/claim/17 "
to discord when a new claim is created.
Using webhooks. If this is the better way to do things then I will close #402.

This change feels big, and I'm not sure how most of it relates directly to sending a discord message when a user creates a new claim replay request. I think you might be better off sending most of this as one PR, and then sending only the "discord webhook" part on a second PR.

Sorry, it should only be the final commit. The rest of the changes are from #400, which this depends on. Is there a good way to show that?

There's a trick I use to show a dependent PR: I set the merge target as the other PR it depends on. However I am not sure if there's a good way to do that when using a fork repo, since a PR from one of your repo's branches to another of your repo's branches would probably not actually show up here.


Sort by ID instead of alphabetically

Hi,
I have a problem showing the results of a function in the ps_facetedsearch module
Receives sorted functions by alphabetical name, or it is possible to sort by ID

This change is

Sorry @zafon18 , this PR does not include any contribution from your account. A merge from dev to master should be done manually.
I'm closing it as it can't be accepted as-is.
If you have an issue, please create one in the main repository https://github.com/PrestaShop/PrestaShop/issues/new/choose
Kind regards


Need to use safe accumulation for calculating the gradient of Embedding + Take

Description
Currently, the inner gradient accumulation method in Embedding and take is not based on safe accumulation, which means that we will lose precision in the fp16 case. Here's the example that amplified the issue:
import mxnet as mx
import numpy as np
mx.npx.set_np()

ctx = mx.gpu()
vocab_size = 8
embedding_dim = 1
index_num = 100000

dat = mx.np.random.randint(0, vocab_size, size=(index_num,), ctx=ctx)

for dtype in [np.float16, np.float32]:
    weight = mx.np.random.normal(0, 1, size=(vocab_size, embedding_dim), ctx=ctx, dtype=dtype)

    weight.attach_grad(grad_req='add')
    weight.grad[:] = 1.0
    with mx.autograd.record():
        out = mx.npx.embedding(dat, weight, input_dim=vocab_size, output_dim=embedding_dim) * 0.01
        out.backward()
    print('dtype=', dtype)
    print(weight.grad)

Output:
dtype= <class 'numpy.float16'>
[[32.]
 [32.]
 [32.]
 [32.]
 [32.]
 [32.]
 [32.]
 [32.]] @gpu(0)
dtype= <class 'numpy.float32'>
[[126.748665]
 [127.53883 ]
 [125.30836 ]
 [125.36837 ]
 [126.278564]
 [127.05873 ]
 [124.74824 ]
 [125.018295]] @gpu(0)

Also, the same happens for take
import mxnet as mx
import numpy as np
mx.npx.set_np()

ctx = mx.gpu()
vocab_size = 8
embedding_dim = 1
index_num = 100000

dat = mx.np.random.randint(0, vocab_size, size=(index_num,), ctx=ctx)
weight = mx.np.random.normal(0, 1, size=(vocab_size, embedding_dim), ctx=ctx, dtype=np.float16)

weight.attach_grad(grad_req='add')
weight.grad[:] = 1.0
with mx.autograd.record():
    out = mx.np.take(weight, dat, axis=0) * 0.01
    out.backward()
print(weight.grad)

Output:
dtype= <class 'numpy.float16'>
[[32.]
 [32.]
 [32.]
 [32.]
 [32.]
 [32.]
 [32.]
 [32.]] @gpu(0)
dtype= <class 'numpy.float32'>
[[125.44839 ]
 [126.68865 ]
 [126.62864 ]
 [125.44839 ]
 [127.028725]
 [126.38859 ]
 [125.108315]
 [125.32836 ]] @gpu(0)

Is stable sum enabled?

To clarify, it affects both nd and the new numpy (I'm testing with the numpy interface because I'd like to test the take operator).
This mimics the setting that appears in large-batch training of transformers, in which there will be way more tokens than the size of the vocabulary.

The simplest fix is to revise the kernel with safe accumulation, which means to cast float16 to float32 before accumulating. Also, I suggest that we should turn on  MXNET_SAFE_ACCUMULATION for float16 type in 1.7 (change the default behavior) so that float16 is accumulated via float32.
I think we should use the following approach for summing up a sequence of float16 numbers:

load them as half2,
cast half2 to float2
accumulate the numbers


JSONPb check enum int values

This checks enum integer values for JSONPb. Breaks one test that says this behavior is acceptable. However this caused a bug for me as I expected JSONPb to handled serialization to a know enum type.

\cc @cybrcodr

If I understand what you are attempting to do in this PR is that you'd like for the Unmarshaler to return error if enum int value is not within the defined set of enums, is that correct?
Per https://developers.google.com/protocol-buffers/docs/proto3

During deserialization, unrecognized enum values will be preserved in the message, though how this is represented when the message is deserialized is language-dependent. In languages that support open enum types with values outside the range of specified symbols, such as C++ and Go, the unknown enum value is simply stored as its underlying integer representation. In languages with closed enum types such as Java, a case in the enum is used to represent an unrecognized value, and the underlying integer can be accessed with special accessors. In either case, if the message is serialized the unrecognized value will still be serialized with the message.

I believe this is consistent with proto.Unmarshal.


Natural(-555) returns<PHONE_NUMBER>3709551061

Given the https://github.com/non/spire#number-types's explanation of Natural:

Natural unsigned, immutable, arbitrary precision integer

is this a bug?
scala> import spire.math.Natural
import spire.math.Natural

scala> Natural(-555)
res1: spire.math.Natural =<PHONE_NUMBER>3709551061

scala> Natural(BigInt(-555))
java.lang.IllegalArgumentException: negative numbers not allowed: -555

On a side note, I'm not sure which apply method is getting used. I don't see any methods accepting an Int - https://github.com/non/spire/blob/master/core/src/main/scala/spire/math/Natural.scala#L558-L583.
Perhaps the PositveInt macro would be useful to apply here per https://github.com/non/spire/issues/437?

Hi Kevin,
First of all, Int values will be promoted to Long values, so apply(Long) is being used.
This is not a bug (except in the sense that it wasn't documented properly). Since Scala (and Java) have no good way to provide literal unsigned numbers, programmers often have to use negative numbers as if they were their unsigned two's complement value.
Here -555 is being treated as a positive unsigned long value (using its two's complement value).
However, we don't want to do this for all values, only for Long values, since the only goal here is to allow programmers to treat a Long value as if it were unsigned -- BigInt is perfectly capable of representing<PHONE_NUMBER>3709551061 directly. Furthermore, since ULong and Long erase to the same type in many cases, we can't easily forbid Long without messing up the machinery for ULong.
I'm open to a better system, but it's important that we are able to accept any of the 2^64 possible unsigned long values.

I still think we should open a bug about poor documentation for Natural. I'll try to remember to do it tonight if you don't get to it first.


[HAMMER] [Changelog] Update for Hammer RC2

[skip ci]

@carbonin as time permits, can you review. Satoe will merge if approved.


Terrier Pipeline Performance Stage

I added a performance stage in the normal PR pipeline. This will allow anyone on CMU-DB team to check a PRs performance before merging, if that is a concern. Currently, to find out the information you will have to look at the stage output in Jenkins. However, in the future, we will make this easier to access. This will also be useful for when we build more automated checks.

This has changes to the Jenkins file I'm not sure if that means that @crd477 needs to do something special when merging this.


Inter-app recording

Hi, great project! I was wondering if it is possible for the screen recording to work with inter-app switching. When I switch to another app, it seems to stop the recording, and upon reopening from the screen-recording app from the background, Xcode spits out these error messages:

Error occured, status = 3, Operation Interrupted Optional(Error Domain=AVFoundationErrorDomain Code=-11847 "Operation Interrupted" UserInfo={NSLocalizedRecoverySuggestion=Stop other operations and try again., NSLocalizedDescription=Operation Interrupted, NSUnderlyingError=0x1c40430f0 {Error Domain=NSOSStatusErrorDomain Code=-16121 "(null)"}})

Is this not possible, or am I doing something wrong?

@Asleepace i am facing the same issue. Were you able to resolve it ?


Update SQLite3MultipleCiphers to 1.7.0 (SQLite 3.43.1)

Hi,
If you have time in near future, it would be good to update SQLite3 Multiple Ciphers to the latest version 1.7.0?
Thanks!

Native lib updated to 1.7.4 thanks to #95 ;
Released <IP_ADDRESS>


Does Azure AD B2C support Role Store?

I see AWS Cognito has role management, you just to add
services.AddCognitoIdentity();
what's the equivalent to AWS service?

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

ID: 0184d69e-7219-f4f0-6583-32f79f5a76e1
Version Independent ID: 49e5b0f0-69cb-4ab1-c8d9-62eb07778a88
Content: Application roles
Content Source: docs/multitenant-identity/app-roles.md
Service: guidance
GitHub Login: @MikeWasson
Microsoft Alias: pnp

@JipingWang are you referring to this newly released preview: https://azure.microsoft.com/en-us/updates/b2c-admin-roles-preview/

Hey @JipingWang We will now proceed to close this thread for now. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Is it possible to focus the parent window onclick of child window using javascript?

I need on click of button in child window parent window needs to be focused and child window go to background of parent window. I tried with below options but its didn't work as expected. Any suggestions greatly appreciated.
I tried below possible ways but those are not working  :
Window.blur();
Window.focus();
Window.opener.focus();

This is not an OpenLayers specific question. Closing


Series full of 'nan' strings or objects inferred as Datetime logical type

If a Series is passed to Woodwork that contains only "nan" strings, it gets inferred as Datetime logical type, filling the values with NaT.
  nans = pd.DataFrame({
        "nans":pd.Series(['nan', 'nan', 'nan', 'nan', 'nan']),
        "nan_strings":pd.Series(['nan', 'nan', 'nan', 'nan', 'nan'], dtype='string'),
        "nan_objs":pd.Series(['nan', 'nan', 'nan', 'nan', 'nan'], dtype='object'),
       })
  nans.ww.init()

assert nans.ww.logical_types['nans'] == ww.logical_types.Datetime()
assert nans.ww.logical_types['nan_strings'] == ww.logical_types.Datetime()
assert nans.ww.logical_types['nan_objs'] == ww.logical_types.Datetime()

Here's the DataFrame and its dtypes before and after initialization:

Datetime is definitely the incorrect logical type for these columns. What the correct logical type actually is may be up for discussion, but based off of the following behavior with np.nan and pd.NA inference, Categorical may be a good option that will allow for the most flexibility if values get added to the column in the future:
nans = pd.DataFrame({
                            "np_nan":pd.Series([np.nan, np.nan, np.nan, np.nan]),
                            "pd_na":pd.Series([pd.NA, pd.NA, pd.NA, pd.NA]),
                            "np_nan_objs":pd.Series([np.nan, np.nan, np.nan, np.nan], dtype='object'),
                            "pd_na_objs":pd.Series([pd.NA, pd.NA, pd.NA, pd.NA], dtype='object'),
                           })

nans.ww.init()

nans.ww

@tamargrey It seems to me that the logical type of a column with all missing values is ambiguous. Do you think it would be better to return the logical type a Unknown in this case (when that PR is implemented), rather than Categorical or some other logical type?

@thehomebrewnerd That makes a lot of sense to me. Are you thinking that a column of np.nan or pd.NA should also get inferred as Unknown? I feel like ideally, they'd all get inferred the same (though, np.nans having dtype float feels deliberate in a way that I'm not as certain we should change)

My initial thought is that any column that is all null (None, np.nan, or pd.NA) should get inferred as Unknown, regardless of the underlying series dtype. Even though a column of missing values with a float dtype seems deliberate, we won't know that for sure, so I would be in favor of not making an assumption in that case, and requiring the user to specify the logical type for that column if they really intend for it to be a Double.
String representations like "nan" or "None" seem a little tricker to me. My thought is that we would treat those the same as other strings and not assume they are intended to represent missing values.

Perhaps we could add a utility to Woodwork to help users replace nan and None with np.nan?

Moving to Icebox as we completed the situation with nan objects
See: https://github.com/alteryx/woodwork/pull/1043


Cannot access de.mrapp.util.datastructure.SortedArraySet

Hi.  I had a some error: Cannot access de.mrapp.util.datastructure.SortedArraySet. Can you help me?
This is my code:       double minSupport = 0.5;
Apriori apriori = new Apriori.Builder(minSupport).create();
Iterable<Transaction> iterable = () -> new DataIterator(new File("sad.txt"));
Output output = apriori.execute(iterable);
FrequentItemSets frequentItemSets = output.getRuleSet();

Hey, I just faced this problem while trying to use the library. It took a while but I figured out the problem, a workaround and (probably) the solution:
The problem
michael-rapp/Apriori internaly uses a SortedArraySet data structure, that resides in the de.mrapp.util.datastructure package. What took me a while to realize is that it's not the package at michael-rapp/Apriori, but at michael-rapp/JavaUtil.
The Apriori project depends on Michael's JavaUtil project, as you can see here: https://github.com/michael-rapp/Apriori/blob/2e8bb7df1b1e71c6bc78ff0528a480d7a55cbf50/build.gradle#L41
It you clone and build the Apriori project, everything works fine. The problem happens when you add it as a dependency on your project. In this case, output.getRuleSet() returns a FrequentItemSets, a class that inherits from SortedArraySet. Your project does not know about michael-rapp/JavaUtil, and complains that it can't access de.mrapp.util.datastructure.SortedArraySet.
The workaround
You can directly add michael-rapp/JavaUtil as a dependency on your project. The project's readme has the instructions both using Gradle and Maven.
The (probable) solution
I got confused on why the indirect dependecy was not being solved, since this seems to be the expected behavior. But then I found this great answer at stackoverflow: What's the difference between implementation and compile in Gradle? (the issue ended up being about Gradle's compile vs implementation and api).
As noted in The Java Library Plugin#API and implementation separation,

The plugin exposes two configurations that can be used to declare dependencies: api and implementation. The api configuration should be used to declare dependencies which are exported by the library API, whereas the implementation configuration should be used to declare dependencies which are internal to the component.

So the problem boils down to the usage of implementation instead of api here: https://github.com/michael-rapp/Apriori/blob/2e8bb7df1b1e71c6bc78ff0528a480d7a55cbf50/build.gradle#L41

I will submit a PR solving this. The alternative would be to add documentation to this project about explicitly including michael-rapp/JavaUtil as a dependency to your project, but I guess that's not the intended behavior.


increase logs

JIRA link (if applicable)
RWA-1615
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ X] No

stale and not required


deck dump error "Error: reading configuration from Kong: plugins: json: cannot unmarshal array into Go struct field Plugin.config of type kong.Configuration"

Hi there,
Hope you are doing well!
I was trying to use deck command to get a snapshot of what we have in our dev environment, the deck ping command run successfully (please see below).
deck ping --headers $ADMIN_TOKEN  --kong-addr https://****
Successfully connected to Kong!
Kong version:  <IP_ADDRESS>-enterprise-edition
When I run deck dump --headers $ADMIN_TOKEN  --kong-addr https://****, it gave me following error.
Error: reading configuration from Kong: plugins: json: cannot unmarshal array into Go struct field Plugin.config of type kong.Configuration
FYI:
We have developed some custom plugins, not sure if it's related.
when I ran deck version, it gave me following:
decK v1.12.2 (6811b77)

Could you please run deck dump with --verbose 2 flag and share the output here?

Closing for inactivity. Please feel free to re-open or comment back if needed.


Error Connecting on Mac os bigsur

I cannot connect properly to my device, commands seems to work sometimes, but I got no plotting at all.
this is one of the error I got from the terminal.
self.handle_received_data(reading.decode())
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x82 in position 2: invalid start byte

Hi @bgvsan sorry for the late answer, i was on holidays.
Did you fill this field ? With M or ay other character tat identifies your Commander interface?

thanks for the answer, yes the command was there.
strangoli i have to confess now the application looks  a little bit more stable and works, I moved to a new pcb and better connection that may could be the issue


Improve fullOpt behavior

Specifically, fix 2 issues in order to better mirror the behavior of "sbt fullOptJS"

Invoke the Google Closure Compiler after generating the optimized Javascript output

Set scala.scalajs.LinkingInfo.developmentMode = false (and productionMode = true) when
building fullOpt in order to allow Scala.js code to distinguish between development
and production mode.

Fixes #189

@Dennis4b some tests are failing. Maybe there is something with CI. can you push once again to trigger CI?

The PR iteself looks good. We can merge after CI goes green

Looks green-ish, merging it


Update renovatebot/github-action action to 32.236.2

This PR contains the following updates:

Package
Type
Update
Change

renovatebot/github-action
action
patch
v32.236.1 -> v32.236.2

Release Notes

renovatebot/github-action
v32.236.2
Compare Source
See the the changelog for changes in all releases.
32.236.2 (2022-10-13)
Bug Fixes

deps: update renovate/renovate docker tag to v32.236.2 (c671737)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

@balena-ci I self-certify!


Mounted twice when layout change

Version
v2.6.2
Reproduction link
https://codesandbox.io/s/wn7j570y85
Steps to reproduce
Simply click on button named "Mounted triggered twice, see console" and see in console.
(reproducable only with css:false for transition)
What is expected ?
mounted once
What is actually happening?
mounted twice
Additional comments?
The bug is still present on last release 2.7.1
It can be fixed quickly ? it's really a brake on client-side performance.
(Related : https://github.com/nuxt/nuxt.js/issues/5616)

This bug report is available on Nuxt community (#c9188)

I pulled this down locally and am only seeing a single mount.

no, you forgot something. The next page must have an other layout and css:false for transition. I have the same bug locally.
I have to make a video to prove it to you?

Someone is alive ? :)

Hi @usb248
This still happens in version 2.8.0
The problem is when transition: {css: false} it's defined

Any core team feels concerned ?

It's not just transition, still mounts twice on  this.$router.push

This seems to still be an issue, event with no transitions and two bare bones layouts, when you switch layouts the page gets double created/mounted

This still seems to be an issue when transitioning from one layout to another, could we get a re-open?

Happening with Nuxt version 2.10.0 as well. I am using nuxt generate. It only happens in the production build.

https://codesandbox.io/s/nuxt-vuetify-k7yv9 just hit the link and watch the console, you see created twitce

Hit the link and watch the console
https://codesandbox.io/s/nuxt-vuetify-k7yv9

It is still an issue:
https://codesandbox.io/s/nuxt-vuetify-k7yv9

I can confirm this does still happen with Nuxt 2.10.2 when page transitions are enabled.
I used to have these objects in my Nuxt config file in order to enable transitions:
layoutTransition: {
    name: 'layout',
    mode: ''
},

pageTransition: {
    name: 'page',
    mode: ''
}

I had to remove both of them to prevent pages components to be mounted/created twice.

As a temporary fix, I wrapped my code inside mounted in if condition. The condition is just to check if the layout name is the one I am expecting.
if (this.$nuxt.layoutName === 'myLayout') {
  ...
}

Don't know if this can help, but I managed to find my problem by putting breakpoints inside the fetch methode to realize the page was being loaded twice, by two different methods.
We had an el-menu made by Element-UI, that had inside of it, nuxt-links, so the route was loaded twice simultaneously.
I realize this was a coding error but this might help anyone looking for awnsers. Try to trace how the route gets changed !

My temporary workaround.
~/plugins/inject-layout.js
export default (context, inject) => {
  inject('layout', {

    // Layout name and page path of pages that is not dafault layout
    map: {
      'one-column': ['/signup/', '/login/'],
      'some-other-layout': ['/some/page/path/']
    },

    // Property that represents correct layout name of current page
    current: 'default'
  })
}

nuxt.config.js
export default {
  plugins: ['~/plugins/inject-layout.js']
}

~/mixins/layout.js
export default {

  // Set and keep correct layout name of current page
  layout({ app, route }) {
    let layoutName = 'default'
    for (const [key, value] of Object.entries(app.$layout.map)) {
      if (value.includes(route.path)) layoutName = key
    }
    app.$layout.current = layoutName
    return layoutName
  },

  computed: {
    // True while duplicated lifecycle hooks are running
    incorrectLayout() {
      return this.$nuxt.layoutName !== this.$layout.current
    }
  }
}

~/pages/some-page.vue
import layout from '~/mixins/layout'

export default {

  // Layout setting
  mixins: [layout],

  // Hooks will be executed twice
  mounted() {
    // Return immediately if duplicated runs
    if (this.incorrectLayout) return
    console.log('This is only displayed once.')
  }
}

This is still occurring for me as of today, in<EMAIL_ADDRESS>My Vue browser extension reports that in the default layout, there are two  views and thus everything occurs twice. This is in dev mode.
While running in dev mode, if I remove the  element from layouts/default.vue, the page structure is cleared. Adding it back, one  view appears (according to the Vue browser extension). After that, refreshing the page results in two  views, indefinitely.
Given that this is happening within Nuxt, I don't know of any workarounds or logical statements that can prevent it.

similar  to https://github.com/nuxt/nuxt.js/issues/6961

I have the same issue with twice mounted on nuxt, what I did is this.
<client-only>     <nuxt /> // On your layout <client-only>

I had something similar
the problem was in the incorrectly registered plugin

@tsarrB can you provide any details? examples?

@gerasimvol
my case is described here

That's still a huge problem, some components mounts twice - that's lead to visual bugs (for example, i have component that mount maps-script, and that leads for two maps visible).
I disabled SSR, cause site now in just demo mode, but I need SSR soon.
Any updates?

I'm facing the same issue! Using "nuxt": "^2.11.0". Need to check layout name before firing stuff inside created hook to prevent firing twice :(

Same issue. beforeDestroy fires twice too，and  I cannot get the right this.$nuxt.layoutName in beforeDestroy

Found another "fix" using Vuex / Nuxt Store.
By putting a 'v-if' statement on the  element on the layout itself you can prevent it from loading the content of the next route before the layout changes.
This way you won't need to deal with every 'created' or 'mounted' function on your page components.
What I did:
Step 1) create a pages.js store module to register the content status.
IMPORTANT: to use vuex modules like this, you need to create an index.js file on ~/store folder

~/store/pages.js

export const state = () => ({
  showLayoutContent: true // show by default in case of first access or refreshing the page
})

export const mutations = {
  TOGGLE_LAYOUT_CONTENT (state, isVisible) {
     state.showLayoutContent = isVisible
  },
}

Step 2) Create an app.js middleware to hide the content on every route change

~/middleware/app.js

export default function (context) {
  // hide the content
  context.store.commit('pages/TOGGLE_LAYOUT_CONTENT', false)
}

Step 3 Add the middleware reference to your nuxt.config.js to extend the router property

~/nuxt.config.js

//{...}
router: {
    middleware: ['app']
},
//{...}

Step 4 Enable and use the store on your layout.

In my case I've used the ```default.vue``` layout as a base of extension for my other layouts so the computed method is created

same here https://github.com/nuxt/nuxt.js/issues/6961
cause serious visual issues 🙁

Will look at it, sorry for the delay 😬

Update: the issue seems to come only when doing server-side rendering (no issue with nuxt dev --spa).
I looked at the Vue Devtools and indeed, the default layout looked different when server-rendered vs client-side (when navigating on error page for example)
SSR:

After navigating:

So it actually re-render the whole layout, causing the new mounted() hooks.
Keeping the investigation 🕵🏻♂

I found the issue, apparently the name of the layout is not defined, which cause a different name between server-side rendering and client-side (Default on SSR and layouts/default.vue on client).
As a quick workaround: Set a name for each of your layouts:
layouts/default.vue
<template>...</template>

I will make a PR to fix it in the next patch release of NuxtJS.
Thank you very much for reporting this and sorry for this outstanding delay 😞

Fixed in V2.12.1 release 🔥

For me, it is still calls mounted twice.

@asolopovas Would you please fill in a new issue with reproduction code?

Still happens in v2.12.2

Still happens in v2.12.2

Then create a new issue and a reproduction CodeSandbox or repo 😉

@manniL why not reopen this issue? i have same issue

<template>

      <nuxt />

    <SignDialog />

</template>

<template>

    <nuxt />
    <SignDialog />

</template>

Same issue on 2.12.2

Yep, still happening on 2.12.2

still happening

Then create a new issue and a reproduction CodeSandbox or repo 😉

@Atinux
First of all, thanks for you effort and suggestion!
Using nuxt  2.12.2 and still have the problem
I am using both SPA and SSR

I found the issue, apparently the name of the layout is not defined, which cause a different name between server-side rendering and client-side (Default on SSR and layouts/default.vue on client).
As a quick workaround: Set a name for each of your layouts:
layouts/default.vue
<template>...</template>

I will make a PR to fix it in the next patch release of NuxtJS.
Thank you very much for reporting this and sorry for this outstanding delay 😞

tried this but have not resolved, any other suggestion :(

Hi - I am getting this error as well. I followed this pattern to create nested layout: https://constantsolutions.dk/2020/02/nested-layouts-in-nuxt-vue-js/ I am not sure if the other guys uses "slots" as well in their layout, and that's why it is happening?

As an example, in my pages/_slug.vue I have this:
mounted () {
    console.log('Mounted!', this.$nuxt.layoutName, this.$nuxt);
}

And in the console I get:

The strange thing is that no matter what I set in the layout name, it will always return "page" or "default". Changing page layout name to "name: 'APage'," it will still say it mounted "page".
I believe this could be the reason why my Nuxt application is slow when changing pages, despite having an API response time of 50ms.

reopen this issue?

Hi @manniL, I wanted to make a reproducible case, so I forked @amritk's codesandbox, and updated to Nuxt version 2.12.2 - I can confirm I didn't have to make any changes for the error to appear.
It writes "I have been mounted" twice in the console, see here: https://codesandbox.io/s/nuxt-vuetify-0xz91?file=/package.json. I even tried to give a name to the layout file.
When switching to the about page:
i have been created. Layout: default
i have been mounted. Layout: default
i have been created. Layout: other
i have been mounted. Layout: other

It's also weird it calls the layout "other" when I call this.$nuxt.layoutName, because I explicitly set the name to be otherlayout.
I think it would be appropriate to reopen this issue. Also I believe it causes page switches to take extra long time - we just launched our production site: https://thejewelleryroom.com and when switching pages to a page that has been cached by Cloudflare, and the response time is 50-100ms, it still takes 1-2 seconds for the page to switch. Perhaps it is because it mounts the page twice?

This is a very serious bug. I cannot not generate unique id for a client side only map container because it is calculated twice. I can not push a value to my empty-by-default array because it is pushed twice.

Issue reopened here: https://github.com/nuxt/nuxt.js/issues/7441

@melihplt you can make this workaround in your mounted/created hook.
if (this.$nuxt.layoutName === 'myLayout') {
    // will only fire once
}

However, it does not solve the performance issues it generates.

Any ideas? Having the same issue

I have the same issue with twice mounted on nuxt, what I did is this.
<client-only> <nuxt /> // On your layout <client-only>
hope it helps.

this works for me

This helped me :

Solution: After plucking some hair, I found that simply wrapping <Nuxt /> in any element fixed the issue. No idea why, but it works.
<template>

    <TheHeader />

        <Nuxt />

    <TheFooter />

</template>

But I don't understand why this ticket is "closed" ? The problem is still happening in 2.15.8 .

I can also confirm this as of 2.15.7 SSR Mode (from the previous comment 2.15.8 isn't any different). What helped was wrapping <Nuxt /> in a div (I also have a header and a footer inside my layout).
I have page transitions still enabled (transition: 'page-transition') so I guess it's not only tied to the transitions.

Solution: After plucking some hair, I found that simply wrapping <Nuxt /> in any element fixed the issue. No idea why, but it works.
<template>

    <TheHeader />

        <Nuxt />

    <TheFooter />

</template>

Same issue here

Why close this issue? It's still happen on 2.15.7

In my case, I have a component TheNavbar in the layout file.
<template>

      <slot>
        <Nuxt />
      </slot>

    <TheNavbar />

</template>

In this component, I have a computed that mapped from vuex store. And the navbar will render links dynamically based on that computed.
I have to put that template code that uses the vuex state within <client-only> component.
The state from my vuex is set when a client side plugin executing. If it was set from the server side this will not happen, I guess.

I am having this problem in version 2.14.3. When I get data from the cookie, the components are mounted twice.
I have a link list in footer. I need to show one of these links when language is 'tr' only.

        KVKK Başvuru Formu

In checkLanguage(), I am getting language data from Cookie.
checkLanguage () {
      return Cookie.get("i18n_redirected");
}

I don't get any errors while using them, but the components are mounted twice.
If I do not use the cookie in the checklanguage function and do it like this, the components do not mount 2 times
checkLanguage () {
      return 'tr'    //Cookie.get("i18n_redirected");
}

I am having this problem in version 2.14.3. When I get data from the cookie, the components are mounted twice.
I have a link list in footer. I need to show one of these links when language is 'tr' only.

        KVKK Başvuru Formu

In checkLanguage(), I am getting language data from Cookie.
checkLanguage () {
      return Cookie.get("i18n_redirected");
}

I don't get any errors while using them, but the components are mounted twice. If I do not use the cookie in the checklanguage function and do it like this, the components do not mount 2 times
checkLanguage () {
      return 'tr'    //Cookie.get("i18n_redirected");
}

Have you tried using v-show instead of v-if?

I am having this problem in version 2.14.3. When I get data from the cookie, the components are mounted twice.
I have a link list in footer. I need to show one of these links when language is 'tr' only.

        KVKK Başvuru Formu

In checkLanguage(), I am getting language data from Cookie.
checkLanguage () {
      return Cookie.get("i18n_redirected");
}

I don't get any errors while using them, but the components are mounted twice. If I do not use the cookie in the checklanguage function and do it like this, the components do not mount 2 times
checkLanguage () {
      return 'tr'    //Cookie.get("i18n_redirected");
}

Have you tried using v-show instead of v-if?

No, actually I solved my problem with computed variable.
But I just tried the above case with v-show. When I use v-show instead of v-if, it does not mount twice. Thanks.

Had the same issue! This thread was a huge help. For me the problem was having multiple elements directly in the page templates.
What I had before (triggered double mount):
<template>

First element

Second element

</template>

Fixed (single mount):
<template>

First element

Second element

</template>

I was under the impression you are allowed to have multiple elements in the page template as long as only 1 rendered. Apparently not 🤷♂  Hope this helps anyone

In my case, one page had an asyncData that was fetching some data from an API and adding that data to the store. But it WAS NOT RETURNING THAT DATA, the page was grabbing that data from the store instead (through a computed even).
Components stopped being mounted twice after returning that data directly from the asyncData method ! 🎉 (without going through the store)

none of the above highlighted solution worked for me on nuxt: 2.15.3

It's late but I hope this would be useful for some one.
I my case, the page mounted twice due to the build property in nuxt.config.js. If I remove build, page will only mount once.
Good luck.

@vanminhquangtri what do you have in the build property?
Some project have to configure the build extend or transpile modules, can't be removed easily just like that.

@trandaison usually because of the splitChunks property inside the build property. Just try to remove the build property to check whether the error is caused by it.

I faced with the same problem) are there any ideas ? I have build>transpile that i need )

I solved this problem by overriding context.app.data in the plugins
export default function (Vue, context) {
  const originData = context.app.data;

  context.app.data = function () {
    const data = originData.call(this);
    return {
      ...data,
      layout: 'div',
    };
  };
}

I solved this problem by overriding context.app.data in the client-side plugins
export default function (Vue, context) {
  const originData = context.app.data;

  context.app.data = function () {
    const data = originData.call(this);
    return {
      ...data,
      layout: 'div',
    };
  };
}

Can you say more about your fix?

This helped me :

Solution: After plucking some hair, I found that simply wrapping <Nuxt /> in any element fixed the issue. No idea why, but it works.
<template>

    <TheHeader />

        <Nuxt />

    <TheFooter />

</template>

But I don't understand why this ticket is "closed" ? The problem is still happening in 2.15.8 .

Thank you so much

This was a real head scratcher for us. This workaround worked though, thanks.

@danielroe , please help
Nuxt 3.12.4, trying to set layout with setPageLayout according on response from api and unfortunately faced double firing of onMount, onBeforeMount. Also tried to sey layout inside the middleware.
Everywhere I'm getting exact behavior: page loads - request - getting layout - setting it, fires onMount/onBeforeMount/middleware again.

If any with nuxt 2.x has this problem, try nuxt 2.17.4 or upper, it has fixes #10874 and #27484


Fix path to image in core-web-vitals dashboard

The image path was pointing to a non existing file.
/cc @kidk

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

LGTM 🎆  Thanks @adri


Algorithm 'ChaCha20Poly1305' is not supported on this platform

Error came up with old Intel 6700HQ processor.
We should check if CPU supports this implementation and if not, fallback to another implementation.

Fixed by replacing this part of algorithm with fully managed ChaCha20 implementation


prepping v1.8.0.dev0

Issue #, if available:
Description of changes:
prepping v1.8.0.dev0
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: IDFModuleTestsInfrastructur-59JoFoVH64CI
Commit ID: bef8803b5e357fc6d8c75b97a4629712918a9e04
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Split backend classes

This branch encompasses work done to split the pipestat backend into classes (currently File and DB backends).

returning boolean on pipestat report function is not behaving as original branch (functionality loss?). Check logic for updated_ids vs returned_id after results are reported.
select_txt is not being tested
clear_status is not being tested.
select_distinct is not being tested.
_strict_record_id is hanging around in some functions. Revisit implementation.
Update doc strings.

Hmm, here's a thought: maybe we shouldn't write tests for specific back-ends at this point. I think we should just write tests, for now, for the primary interface functionality.

This is looking really great. Just a few more minor details below to address before we merge it:

[ ] select_txt should not be defined on on the PipestatManager object because it is specific to the DB Backend.
[ ] same thing for select() and select_distinct and _engine()... and maybe others. I think if someone wants to call a backend-specific function, they should do it by calling psm.backend.select().
[ ] _init_results_file and _load_results_file should be defined on the FileBackend class, not on the PipestatManager class.
[ ] PipestatManager is using both self[FILE_KEY] and self.file. I think this is the same thing. We should only be using a single paradigm. Please check this for all the possible CONST-based attributes and use them in the same way.
[ ] the properties use 3 different ways to retrieve the values to return. Can these be standardized to a single way? See these:

    def status_schema_source(self) -> Dict:
        """
        Status schema source

        :return dict: source of the schema that formalizes
            the pipeline status structure
        """
        return self.get(STATUS_SCHEMA_SOURCE_KEY)

    @property
    def schema_path(self) -> str:
        """
        Schema path

        :return str: path to the provided schema
        """
        return self._schema_path

    @property
    def config_path(self) -> str:
        """
        Config path. None if the config was not provided or if provided
        as a mapping of the config contents

        :return str: path to the provided config
        """
        return getattr(self, "_config_path", None)

[ ] On PipestatManager class, what do you think of organizing the order of the functions? I'd say something like First comes all the primary functions in alphabetical order, then comes all the private functions (_function), in alphabetical order, and finally, end with all the properties, in alphabetical order. Or, do you have an alternative suggestion? Right now it's hard to read because it seems random. Is there a PEP-specified guideline or recommended way on how to do this?

Regarding the property retrievals, I changed the ones that could be changed to align with the majority of the other retrievals. In the above example for config_path, I believe we need to leave that as is.

The order of methods in PipestatManager is still unintuitive to me. You checked this off, do you have a comment on why you wanted to leave it the way it is?

Overall everything looks great! I have just few suggestions about the code and structure:)

I'll check on _create_orms function; it may not have been migrated yet to the backend as desired (an oversight).
Regarding the pipestat property and method organization, I actually did move things around for both pipestat and backends. I attempted to group items based on similar operations and then alphabetize them. Is there something more specific that would make it more intuitive? Would simply alphabetizing after dividing them into properties and methods make more sense?

Functions/properties now organized -> public, then private, then property and within each group, alphabetized.


[ISSUE #7699] Add namespace v2 in client

Which Issue(s) This PR Fixes

Fixes #7699
Brief Description
Add namespace v2 in client

How Did You Test This Change?

Codecov Report
Attention: 114 lines in your changes are missing coverage. Please review.

Comparison is base (71a7a65) 43.17% compared to head (710d3bc) 43.15%.
Report is 15 commits behind head on develop.

Files
Patch %
Lines

...apache/rocketmq/remoting/rpc/RpcRequestHeader.java
27.27%
16 Missing :warning:

...ocketmq/client/consumer/DefaultMQPushConsumer.java
48.14%
14 Missing :warning:

...he/rocketmq/client/producer/DefaultMQProducer.java
43.75%
9 Missing :warning:

...g/apache/rocketmq/client/impl/MQClientAPIImpl.java
44.44%
5 Missing :warning:

...ocketmq/broker/processor/AdminBrokerProcessor.java
0.00%
4 Missing :warning:

...ocketmq/client/producer/TransactionMQProducer.java
33.33%
4 Missing :warning:

...ng/protocol/header/SendMessageRequestHeaderV2.java
55.55%
4 Missing :warning:

...g/apache/rocketmq/remoting/rpc/RequestBuilder.java
0.00%
4 Missing :warning:

...etmq/broker/processor/ConsumerManageProcessor.java
0.00%
2 Missing :warning:

...mq/broker/topic/TopicQueueMappingCleanService.java
0.00%
2 Missing :warning:

... and 44 more

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #7700      +/-   ##
=============================================
- Coverage      43.17%   43.15%   -0.02%
- Complexity      9797     9820      +23
=============================================
  Files           1165     1167       +2
  Lines          84477    84595     +118
  Branches       10966    10984      +18
=============================================
+ Hits           36474    36509      +35
- Misses         43473    43551      +78
- Partials        4530     4535       +5

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


Document v2.0.0  breaking changes in CHANGELOG.md

Expected behaviour

2.0.0 release signifies breaking changes from v 1.x.
Change log clarifies just what those breaking changes are and what they mean for adopters.

Actual behaviour

There's a 2.0.0 release
It has an entry in the change log
But it's not clear from the change log what the breaking change is and what it means for adopters.

Steps to reproduce the behaviour

View the change log
Try to understand what breaking change led to the major version rev
Be frustrated.

Guessing it's because Karma dropped support for Node.js 0.10 and 0.12.

The changelog error has been fixed.

I don't think this issue wrt the 2.0.0 changelog entry has been addressed.
Reading the changelog, looking at the 2.0.0 release, it still doesn't seem apparent what the breaking change was. Contrast the treatment in the 3.0.0 release, where breaking changes are called out thanks to #3109 .

Please open a PR to fix the log if you like.
The breaking change was
https://github.com/karma-runner/karma/commit/ec92ea9
but it was not marked so in the description and thus the script did not emit the changelog entry.
In any case the supported version is 3.0.

It's a little odd that https://github.com/karma-runner/karma/commit/ec92ea9 is the breaking change yielding 2.0.0 when that change is also in 1.7.1. Is this a matter of the 1.7.1 patch release inadvertently included a breaking change?


fix(table-version): table version is fixed to 11.2.3

@sanbornsen fabric8/fabric8-ui:SNAPSHOT-PR-2479-1 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2479-fabric8-planner.badger.fabric8.io


Debian Jessie install fails with Failed to fetch http://packages.treasuredata.com/2/debian/jessie/dists/jessie/contrib/binary-amd64/Packages  Hash Sum mismatch

I have an automated build for a Docker container image that we use for monitoring, and up until today it's been building fine (without any changes to our code).
We have a step that runs the following commands on a Debian Jessie container:
curl -L https://toolbelt.treasuredata.com/sh/install-debian-jessie-td-agent2.sh | sh
  && td-agent-gem install [plugin here]

When it's running the install, it has the error:
W: Failed to fetch http://packages.treasuredata.com/2/debian/jessie/dists/jessie/contrib/binary-amd64/Packages  Hash Sum mismatch
...
E: Unable to locate package td-agent

And then the next step runs and outputs:
/bin/sh: 1: td-agent-gem: not found

At this point, the build fails.
Looking at my build logs, the container builds worked fine (using the same commands) up to at least noon yesterday (ET). Is it possible that some internal build process failed?
I'm currently not able to install td-agent on a Debian Jessie instance (at least not using the official build scripts/repos).
(Also, hopefully this is the right issue queue for this; sorry if it isn't!)

Hmm, the last failed build was 2 hours ago. I just ran the same build in a different environment, and it seems to have succeeded there just now, so I'm going to do one more build in the original environment to see if it was just a temporal issue.

And now both builds are passing... so never mind!


CLI: implement install command

Implement the fontist install {name} command:
# install all fonts in formula
fontist install cleartype

# install one font from formula (all styles)
fontist install calibri

# install one font style from formula
fontist install calibri bold

From #90 .

Discussed to skip the style option, support only font and formula currently
Done in #148


fix #1794

Small fix for the mac MPS error #1794
It works perfectly as expected on my mac m1 with this fix.
Please note that MPS is not yet supported for all operations, and in #1794 it will be necessary to run the script with PYTORCH_ENABLE_MPS_FALLBACK=1

When this will be released? Otherwise im gonna just change my local code 🤔

@mattiasu96 please do :

git clone https://github.com/speechbrain/speechbrain.git
cd speechbrain
pip install -r requirements.txt
pip install --editable .

It will download and install the new SpeechBrain version with the fix.


Tinygrad import path changed in model conversion script

The latest tinygrad 0.7.0 moved state to under tinygrad.nn.state, which broke the conversion script. Updating the import path fixed the problem.

Updated.


POL-334 Azure Blob Service

Implemented the new Plugin service for Azure Blob Services
Description
POL-334 Azure Blob Service
Issues Resolved
[List any existing issues this PR resolves]
Contribution Check List

[ ] All tests pass.
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] New functionality has been documented in CHANGELOG.MD

          3 Errors

:no_entry_sign:
azure/rs_azure_blob_service/README.md:62: MD012 Multiple consecutive blank lines
azure/rs_azure_blob_service/README.md:36: MD022 Headers should be surrounded by blank lines
azure/rs_azure_blob_service/README.md:22: MD032 Lists should be surrounded by blank lines
A detailed description of the rules is available at https://github.com/markdownlint/markdownlint/blob/master/docs/RULES.md

:no_entry_sign:
README.md:7: MD022 Headers should be surrounded by blank lines
README.md:55: MD022 Headers should be surrounded by blank lines
README.md:8: MD032 Lists should be surrounded by blank lines
README.md:56: MD032 Lists should be surrounded by blank lines
A detailed description of the rules is available at https://github.com/markdownlint/markdownlint/blob/master/docs/RULES.md

:no_entry_sign:
File syntax check failed. azure/rs_azure_blob_service/azure_blob_services.rb.

  Generated by :no_entry_sign: Danger


[V4.0 WINCE] Update code from smartdevice475

Some modifications of CMakeLists.txt.
Add some code of dynamic library.
Some modifications of codes from unit test of win32 platform.

Fix several UT fails
Resolve code compile with VS2005
Add new feature to generate .dll dynamic library


[Kbtouch] Disregard touch screen calibration

... as to not risk having some characters be inaccessible.

The solution then I guess is to have sufficiently large borders where input is not needed for the apps functionality. These could be hardcoded or depend on the users touch screen calibration.
Another solution can be to remove the touch screen calibration on apps where access to the edges is needed and restore it when the app is closed.

Originally posted by @thyttan in https://github.com/espruino/BangleApps/issues/2457#issuecomment-1418022051

Let's discuss this on https://github.com/espruino/BangleApps/pull/2558

Closing this and #2558 - discussing on #2457


A question about your talk

Hey Peggy, I just watched your talk on YouTube - it was fantastic! Very inspiring.  I already love and use Apollo. Do you have an example boilerplate repo that shows everything wired up (web and react native with react-native-web and Apollo)?

Bonus points if the boilerplate also included expo and database => API wire up with graphcool/prisma

But yeah, amazing talk. Thank you so much for sharing!


Atom not honoring Exclude VCS Ignored Paths

I've updated Atom recently and notice no matter what I set Exclude VCS Ignored Paths to I can never see my /node_modules folder, if it is in my git ignore.
OS: OSX (yosemite)
Atom Version: 0.198.0

Please make sure Settings-->Packages-->Tree View-->Hide VCS Ignored Files is not checked.

@50Wliu Awesome! That was it, I must have changed it when rooting around in the settings. I was full sure it was the Exclude VCS Ignored Paths that controlled this.
Thanks for the quick response :+1:


expanded with more tests and slightly improved internal code

@haraldkubota Do you think you could run this version and confirm that it works? I don't have an SHT31 currently.

Tested and I cannot compile it:
harald@penguin:~/src/rust/rp2040/sht30$ cargo build    Compiling rp-sht30 v0.1.0 (/home/harald/src/rust/rp2040/sht30) error: no global memory allocator found but one is required; link to std or add #[global_allocator] to a static item that implements the GlobalAlloc trait
I use
#![no_std] #![no_main]
and Cargo.toml contains
sht31 = { git = "https://github.com/FloppyDisck/SHT31-rs.git", branch = "improvement" }

@haraldkubota Thanks for the help! I ended up adding an example for an esp32 for quick testing on future versions. You can always add an rp2040 example if youre interested.


Not all message-based events try to return the message.

Certain message events return a skeleton message. This should be fixed.

No complaints; closing.

message skeleton is still returned when the message isn't in the cache, rather than it being automatically fetched by the id. you might want to mark as 'wontfix' if this is intentional.

It is indeed.

maybe you could add a flag to DiscordMessage to reliably check if message body present or if it's just the skeleton.


Add blueprint and release notes

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: #134
Summary:
This PR adds the blueprint reference and release note docs to the repo.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[x] Other (docs)

Checklist:

[ ] My update follows the CONTRIBUTING guidelines of this project
[ ] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

:tada: This PR is included in version 6.3.7 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[SPARK-31632][CORE][WEBUI] Make the ApplicationInfo always available when accessed

What changes were proposed in this pull request?
This PR adds a while loop check for the ApplicationInfo returned in AppStatusStore.applicationInfo(). It eliminates the NoSuchElementError exception occasionally happens when a user accesses the Web UI during Spark startup.
Why are the changes needed?
During the initialization of SparkContext, it first starts the Web UI and then set up the LiveListenerBus thread for dispatching the SparkListenerApplicationStart event (which will trigger writing the requested ApplicationInfo to InMemoryStore). If the Web UI is accessed before this info's being written to InMemoryStore, the following NoSuchElementException will be thrown.
 WARN org.eclipse.jetty.server.HttpChannel: /jobs/
 java.util.NoSuchElementException
 at java.util.Collections$EmptyIterator.next(Collections.java:4191)
 at org.apache.spark.util.kvstore.InMemoryStore$InMemoryIterator.next(InMemoryStore.java:467)
 at org.apache.spark.status.AppStatusStore.applicationInfo(AppStatusStore.scala:39)
 at org.apache.spark.ui.jobs.AllJobsPage.render(AllJobsPage.scala:266)
 at org.apache.spark.ui.WebUI.$anonfun$attachPage$1(WebUI.scala:89)
 at org.apache.spark.ui.JettyUtils$$anon$1.doGet(JettyUtils.scala:80)
 at javax.servlet.http.HttpServlet.service(HttpServlet.java:687)
 at javax.servlet.http.HttpServlet.service(HttpServlet.java:790)
 at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:873)
 at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1623)
 at org.apache.spark.ui.HttpSecurityFilter.doFilter(HttpSecurityFilter.scala:95)
 at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1610)
 at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:540)
 at org.eclipse.jetty.server.handler.ScopedHandler.nextHandle(ScopedHandler.java:255)
 at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1345)
 at org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:203)
 at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:480)
 at org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:201)
 at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1247)
 at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:144)
 at org.eclipse.jetty.server.handler.gzip.GzipHandler.handle(GzipHandler.java:753)
 at org.eclipse.jetty.server.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:220)
 at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:132)
 at org.eclipse.jetty.server.Server.handle(Server.java:505)
 at org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:370)
 at org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:267)
 at org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:305)
 at org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:103)
 at org.eclipse.jetty.io.ChannelEndPoint$2.run(ChannelEndPoint.java:117)
 at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:698)
 at org.eclipse.jetty.util.thread.QueuedThreadPool$Runner.run(QueuedThreadPool.java:804)
 at java.lang.Thread.run(Thread.java:748)

Does this PR introduce any user-facing change?
No
How was this patch tested?
Manually tested.
This can be reproduced:

./bin/spark-shell
at the same time, open http://localhost:4040/jobs/ in your browser with quickly refreshing.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Just 2 cents as I'm not committer as well as not an expert of web UI area.
I see what's happening, but I'm not sure the fix is on the right way. I feel it's probably better to wait from other place (instead of AppStatusStore which is simply serving the information), even better if we can serve web UI once such precondition is fulfilled.
Having empty ApplicationInfo (and showing properly) might be also one of valid options.

Hi @HeartSaVioR, thanks for your suggestion.
Yes, I'm also not quite sure if doing a while-loop-check is suitable here. I also considered the solutions you mentioned, e.g., making the AppStatusStore.applicationInfo() return an optional ApplicationInfo or constructing an "empty" ApplicationInfo when it's unavailable in the KVStore. I made the current choice because after checking all the invokers of this method, I found they all expect to get a valid ApplicationInfo. If this is the so-called contract and we keep the current initialized logic, this fix could just work. However, I can definitely make another choice if we reach a consensus.

Hi @Ngone51 and @HyukjinKwon , do you have any suggestions on fixing this?

I've took a look at those invokers. It seems we still need some retry logic where the end user can not help. If that's the truth, I'd prefer to keep retry with a timeout in applicationInfo().

As there're no more other comments, I just updated the PR as @Ngone51 suggested.
Please take a look when it's convenient for you. Thanks.

ok to test

Test build #122472 has started for PR 28444 at commit 0951f3d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27133/
Test PASSed.

Test build #122472 has finished for PR 28444 at commit 0951f3d.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122472/
Test FAILed.

Hi @HyukjinKwon, I've updated this PR. Now the exception looks like below.

Test build #122566 has started for PR 28444 at commit e633d1f.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27228/
Test PASSed.

Looks good, please also update the PR description.

Test build #122566 has finished for PR 28444 at commit e633d1f.

This patch fails build dependency tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122566/
Test FAILed.

Thanks for your reminder @jiangxb1987. Done.

Thanks @xccui and all. Merged to master, branch-3.0 and branch-2.4.

Thanks all for your suggestions and feedbacks.


Update repositories.txt to add lautisilber/SmartComm library

Thank you dearly for all your work maintaining the Arduino project!

What about now? (Sorry for the inconvenience btw)


Refactor zkapp generators

Explain your changes:
This PR cleans up zkapp_command_generators a little bit. Instead of using a bunch of optional argument, I created a record that collects those and make it an argument to the generator function.
Explain how you tested your changes:
*
Checklist:

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

LGTM, but please revert the changes to the crypto submodules

Sure.

!ci-build-me

!ci-build-me


Build next ObsPy release (do not merge yet)

This is just preparation to build the next ObsPy release, currently only contains reverting the pinning of conda-build (so we don't forget about it later) that was temporarily needed to get the first build and won't be necessary once conda-build 1.21.11 is released.
DO NOT MERGE YET

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Oh no! Looks like you forgot to fork, @megies. 😱
Just as FYI, all branches on the feedstock automatically deploy (even if they are unmerged PRs). It would be best to close this one out an open a fresh PR from a fork. Unlike the main feedstock, forks do not deploy anything. PRs from forks to the main feedstock will only deploy when merged.

Should add that conda-build version 1.21.11 does appear to be out.

Oh no! Looks like you forgot to fork, @megies. :scream:

Uhm.. I did fork, but for certain-to-get-merged topics I usually I find it much more convenient to work on branches in the main repo, so that all maintainers can edit the branch without "if you add me to your fork i can help" chit-chats.

Just as FYI, all branches on the feedstock automatically deploy (even if they are unmerged PRs).

Ok, good to know. Didn't know, obviously. If I was to decide, I would only deploy from master. But I really am light years away from complaining as this really is a huge effort you're tackling with conda-forge. :smile:

It would be best to close this one out an open a fresh PR from a fork.

Sure will. But still I'd vote for a change so that only commits to master deploy. I think that would save you and others some surprises in the future.

Unlike the main feedstock, forks do not deploy anything. PRs from forks to the main feedstock will only deploy when merged.

Oh no! Looks like you forgot to fork, @megies. :scream:

Uhm.. I did fork

Meant this branch didn't come from your fork. In any event, it seems this point is clear.

But still I'd vote for a change so that only commits to master deploy.

Unfortunately it is not that simple. Some feedstocks do need multiple branches for deployments (e.g. many active versions).

I think that would save you and others some surprises in the future.

Trust me. I have been having this conversation with new maintainers for several months. It is no longer surprising. Merely am trying to be informative.

As for the rest of this, there's nothing new under the sun. 🌞 Everything you have said is a good point that has been raised before. See issue ( https://github.com/conda-forge/conda-forge.github.io/issues/87 ). At this point, it is a matter of doing something about it. At least on my end, there are simply many more pressing issues at conda-forge that already absorb my time. Though I do appreciate the value of this sort of change.

Meant this branch didn't come from your fork. In any event, it seems this point is clear.

Yup, it's clear now, after explanation. I think this needs to be clarified in the README.md, I'm proposing a change to the template with conda-forge/conda-smithy#246.

Unfortunately it is not that simple. Some feedstocks do need multiple branches for deployments (e.g. many active versions).

Yeah, that's a valid reason and it makes sense to me now.

Trust me. I have been having this conversation with new maintainers for several months. It is no longer surprising. Merely am trying to be informative.

Yes, I bet you had the same questions a bazillion of times. Therefore my shot at making this clearer with conda-forge/conda-smithy#246. Just hoping to save you guys some time explaining..

there are simply many more pressing issues at conda-forge that already absorb my time

.. I bet!

That's totally reasonable. We should definitely improve on the docs. Thanks for taking that one on.


Null safety issue in icon

Icon should be optional:
Previous code:
icon initialization  final Widget icon;
icon usage icon: icon ?? Icon(             Icons.expand_more,           ),

Thank @Rashid-Khabeer for the issue and relevant pull request!


Command Palette should steal focus from active webview...

hey there :wave:
i ran into a weird one. i have an extension which presents a webview with a number of html button elements. when these buttons are clicked, they often run a command via executeCommand which in turn prompts the user via showInputBox.
when a user clicks, the button is focused, command is run, and the input box is shown prompting the user to enter some info. they enter text and press the <enter> key to submit.
since the button on the webview is still focused, it receives the <enter> key press and re-triggers the command. iow, <enter> in the input box is re-routed to the focused html button element.
ideally, focus should be removed from a webview (or any of it's children elements) once the user opens the command palette.
for now, i just call el.blur() in my webview's event handler which fixes the issue.

VSCode Version: v1.28.2
OS Version: macOS v10.13.6 (high sierra)

The QuickPick should be able to request focus when being opened via command execution.

But you can type in the input box? So it must have focus. Is there anything special in that regard about webviews?

I'm hitting the same issue with this. We have a webview that hosts react components, two of which trigger export and import commands (which then use the open file and save file dialogs). These commands can be triggered from the react webview buttons or the command palette. The following steps trigger this pretty easily.

Click the export command webview button.
Hit esc to close the export dialog (focus is still on the export webview button)
Hit Ctrl-Shift-P and select the import command in the command palette and press enter
First the export dialog pops up again (due to the enter key on the focused web element)
Next the import dialog pops up (due to the enter key in the command palette)

From an accessibility standpoint I think that we would probably want the command palette to override current focus when popped up and then restore focus when closed but even stealing the focus and not giving it back after would be better than keyboard key going to both the webview and the command palette.


EmulationStation and ssh

So I want to start EmulationStation from ssh, not to play on my computer but on my tv. The thing is that i have two controllers and a Wifi dongle and i can't connect a keyboard. So my question is, can i through ssh tell my raspberry pi to start EmulationStation? The other reason that I would like to do this this way is because when i let EmulationStation connected to long it just freezes and i have to restart the whole machine.
Thank you

This works:
su -l pi -c emulationstation


(update) Abort Documentation

Add more information about Abourt process to advise about why there is not an instant "stop" to an abort call.
Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: _____ Add more information about Abourt process to advise about why there is not an instant "stop" to an abort call.
Jira/GitHub Issue numbers (if any): ______ https://harness.atlassian.net/browse/PL-56864
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66e09d35fde060007e10458d--harness-developer.netlify.app


Make nd-data-transfer library pip-installable

User story
As a user, I want to install nd-data-transfer via pip, so I can easily run the code on local machines.
Acceptance criteria

[ ] The nd-data-transfer library should be released to pypi?

Sprint Ready Checklist

[x] Acceptance criteria defined

[x] Team understands acceptance criteria

[x] Team has defined solution / steps to satisfy acceptance criteria

[x] Acceptance criteria is verifiable / testable

[x] External / 3rd Party dependencies identified

[x] Ticket is prioritized and sized

Notes
Add any helpful notes here.

PyPI doesn't allow for publishing of direct dependencies. We'll need to resolve those before this ticket can be completed.


Beef up the hydra

This PR enhances the hydra's abilities. It won't affect the single hydra hand placed in the temple (that apparently is in a .mob file, and has a bunch of its stats overridden from the proto), but I think it might affect a random encounter containing hydras (I've seen it in the files, but I don't remember where it occurs). The changes are as follows:

Fixed fast healing. Some other enemies already had this (like quasits), but it was unlikely to do anything.
Enhanced the full attack cost logic to allow taking full attacks as a standard action based on a query. The hydra has a condition that allows this, so it can move and full attack. Seemed like the easiest way to accomplish this aspect.
Enhanced charge attacks in a similar way, so that based on a query, a charge puts you in the full attack state at the end. Should be useful for the Pounce ability as well. I think the AI uses this correctly, but I'll double check.
Tweaked the combat reflexes logic to give hydras an extra AoO for each one of its (current) heads, instead of based on dexterity. (Atari bug 275)
Made a python condition for hydras that hooks into the above.
Added a proto line override in the co8 directory. Gives the hydra fast healing 15 instead of regeneration 5 (or poison, from the original game), and gives them the hydra condition with 5 heads. I also gave them the boar AI so that they charge at people.

I think the 5 headed hydra is still not very dangerous, even with all these fixes. I've been testing with 9 and 12 headed hydras, and those are pretty dangerous; they basically take out one character per round if they can reach them, and they're harder to tie up. Unfortunately we don't have models for those. (If someone could put that together, with ways of dynamically varying the number of heads, that'd be pretty sweet for the future.)

I'm getting a link error against the python library on my fork. Do I need to update something?

That is odd... I wouldn't expect you to get any link errors.  What exactly are you seeing?

That actually does look like the typical "CI upgraded something" error.

Well, in the mean time, I was wrong about the AI actually charging with a full attack (it seems like it never makes use of left-over attacks). So I'll see if I can do something about that. Otherwise I'll get rid of the boar AI, because charging is worse than the hydra's normal move-and-attack.

OK cool.  It may be tomorrow before I get my VS updated and I can see if I need to rebuild the dependencies package.

Never mind. I was running the wrong build. The AI does charge+full attack, so no fix is necessary. (The other stuff I was seeing is a separate issue, I guess.)
Take your time, though.

I'd like to add certain things to the table:

There is extraordinary monster ability Pounce which allows full attack after move. Which brings to second point.

MM1 pg 313, Pounce (Ex): When a creature with this special attack makes a
charge, it can follow with a full attack—including rake attacks if
the creature also has the rake ability.

You need to know D&D 3.5 rules. This is one of the reasons why I personally became Dungeon Master - to understand and practice rules.
The Hydra does not have Pounce, therefore this fix should not be here.
But I support creating possibility for Pounce. As there are multiple ways to achieve this marvelous ability: Claws of the Leopard from CAd, or CCh Spiritual Totem Lion Totem.
Hydra's Combat Reflexes is indeed problematic. It's either Head Count * Dex Mod and once per aoo, or during single aoo strike Head Count times. I would settle for first for now.
You need to in-depth know how tactics works in ToEE.

Huh, indeed, thanks!
Then it is sort of Pounce. In that case we should implement Pounce.
It's a pity that there is no room in D20CAF.

The rules for the hydra's combat reflexes are not super spelled out, so some interpretation is necessary. I think the people who think it's supposed to mean that a single AoO uses all the heads at once (which you often see on internet forums) are not thinking it through.
One of the big mechanics for hydras is chopping their heads off with sunder attempts. Unless you have improved sunder, each one of those provokes an AoO from the hydra. However, if it only had 1 AoO per round, this would make beheading it pretty safe. Even if it got 2 with combat reflexes (because it only has 12 dex), you could still get multiple free sunder attempts at the higher end of things. So, instead it gets a number of (otherwise normal) AoOs proportional to its current heads, so sever attempts are unlikely to ever be free (without improved sunder).
At higher levels, a full suite of attacks from a hydra can completely kill even a robust character, so the other version would turn into sacrificing a character to get free attempts from others, or just being completely unable to make attempts without improved sunder. I don't think that's the point of the ability.
I'm not in inclined to multiply the heads by the dex bonus, because there's no indication that it's supposed to work that way. I think it's good if it's just using the head count in place of the dex bonus, under the premise that each head is acting sort of independently (same as for the other abilities), and the combat reflexes feat is just to help motivate and explain the ability. That's why I made it a number of extra AoOs equal to the head count, too. One for each head doing its own thing, plus the normal one.
Pounce can be added later. It's easy to do with the way I set this up (just make a condition that responds to the right signal). The instances I can recall enable extra attacks beyond the normal full attack, which requires extra work. And it's not work that has anything to do with hydras.

CI should be good now thanks to help from @shartte !

Okay. Branch builds, and seems to work without my dropping the files into an override.

Looking good now... any way to override the mob file and have this work for the the single hydra in the temple?  Also, is the fast healing fix going to work correctly with all feat healing abilities?

I have no idea how mob files work. They're binary, so you need one of the modding tools to edit them. I don't know if Temple+ is set up to pull them from override directories or not. Apparently even the Co8 mod isn't overriding it, because it still has poison damage instead of the Regeneration 5 condition in the protos.
I'm not sure what feats you mean. Fast healing is supposed to be like natural healing, except you heal every round instead of every day (or hour or whatever for nonlethal). I don't know what feats it would interact with.

Option A: add first heartbeat to proto and setup conditions dynamically.
Option B: use Ed to edit Hydra mob on the map. But saving the map could introduce a lot of changes. So backup the mob somewhere, restore whole folder and overwrite hydra mob file only.

I don't think A will work. I tried changing one of the scripts in the proto, and it didn't change on the placed creature. So, I think the mob file is overriding all the proto scripts. Probably not surprising, because the placed hydra is not hostile on sight (overriding the KOS script), and I bet you have to either override none or all of those scripts.
Does the editor tell you which mob file is which placed object? The file names are like meaningless UUIDs. The only clue is which map it's in.
Anyhow, this isn't something I can easily fix. If someone else wants to pick that up, that'd be cool. The relevant differences I know of are:

Monster Melee Poison with argument 4 should instead be Monster Fast Healing with argument 15
It should have an additional condition: Monster Hydra with argument 5

The simplest way would be to make the mob file pull the conditions from the proto, but I don't know if that's possible without messing something else up.

My first pet project for toee was creation of mob parser in c#. I have cmd which converts mob to xml and back.
I will take care of the mob.
In my modules I never use monster mobs. There is only one on the map, which create all others in Py, based on Sitra's daemon idea.

I said feats but I guess it is more spells that I was thinking about.  Sorry for the confusion.  Sounds like your fast healing change won't cause any issues.
@anatoliy-savchak Can you submit another PR with the mob file fix?  That probably isn't appropriate for this PR.  I am going to go ahead and merge it.  Thank you @dolio !


[feat] add causalvae ✨

Display
Image Reconstruction
Original image:

Reconstructed image:

Video Reconstruction
Raw reconstructed video here: 1, 2.
https://github.com/PKU-YuanGroup/Open-Sora-Plan/assets/88202804/c100bb02-2420-48a3-9d7b-4608a41f14aa
https://github.com/PKU-YuanGroup/Open-Sora-Plan/assets/88202804/8aa8f587-d9f1-4e8b-8a82-d3bf9ba91d68
Model Structure

Currently, we are still in the stage of exploring and rapidly iterating the training of the model. Our training configurations are not yet stable, so they are not public at the moment. The specific training configurations and model weights will be open-sourced together after the completion of the t2v model training.


Updated notebooks after recent tests for CSHS

Overview
This PR updates notebooks that were not compatible with the recent versions of xclim and ravenpy.
Also used the opportunity to clean the notebooks, removing all pre-run code, so they are all in a clean, new state.

I just realized I had used the wrong initial versions before doing my changes. I have reimported the new ones and updated them accordingly, and now all outputs are preserved. Thanks for the feedback!


E2E Tests: Improve flakey Publishing Flow test

Task Description
The following test regularly fails:
https://github.com/google/web-stories-wp/blob/08b68ab3879cf33c1d046f881e2adeb0165d7bce/tests/e2e/specs/editor/publishingFlow.js#L58-L115
  ● Publishing Flow › should guide me towards creating a new post to embed my story

    TimeoutError: Text not found "Publishing Flow Test"

      waiting for function failed: timeout 500ms exceeded

  ● Publishing Flow › should guide me towards creating a new post to embed my story

    expect(jest.fn()).not.toHaveErrored(expected)

    Expected mock function not to be called but it was called with:
    ["Failed to load resource: the server responded with a status of 404 (Not Found)"]

      at Object.assertExpectedCalls (../../node_modules/@wordpress/jest-console/build/@wordpress/jest-console/src/index.js:36:4)

Still quite flakey
https://github.com/google/web-stories-wp/runs/2366358638#step:14:17


Tests need to be updated and following best practices

https://www.browserstack.com/guide/top-react-testing-libraries
https://www.testquality.com/blog/tpost/tf857kfsl1-react-testing-best-frameworks-libraries
https://fe-tool.com/react-testing-frameworks

So according to the first three sources I find on the web, Jest appears to be the most used testing framework for React. Mocha could be interesting to look into also as it is quite popular too and I know that is used by Hardhat for example. So maybe it could be more suited for the blockchain aspect of it.
For now, as I don't know any of both and as we already have tests written with Jest I would suggest that we start by updating them and understand the possible limitations of the framework before digging into another one.


linksynergy.com

Energized Protection - block ⚡

Let's make an annoyance free, better open internet, altogether!

Issue Submit Form

  Provide the following infos properly, which will help us to resolve your issue quickly.

Issue(s):
Type x in between [ ] and make sure there isn't any space between brackets. Example; for Your Selected Issue(s), type like this - [x]
You can select more than one category of issues if you need to!

[x] Whitelist
[ ] Blacklist
[ ] App Broken
[ ] Website Broken
[ ] Request
[ ] Other Issue

Pack(s):
Write the name of the pack(s) you are using.

Pack(s) Name: blu

Extension(s):
Also name the extension(s) if you are using any. If you don't then leave it blank.

Extension(s) Name:

Domain(s):
If you are submitting this issue for whitelist/blacklist issue, send us the domain(s) for whitelisting/blacklisting here. Kindly use the Code Tag to prevent tracking.

Domain(s):

click.linksynergy.com
linksynergy.com

Your Config:
Just to ensure there is no issues or conflicts with other app/software/magisk module/extension/source list.
Make sure you are running Energized Protection Service only.

Client:

Version:

Details:
Write us a lil bit more about your issue or query. You can attach any screen shot or log of the issue or advert, this will help to highlight it.

Your Issue Detail:
Blocks internal links here:
https://www.cnn.com/2019/08/18/success/sothebys-first-porsche-auction/index.html
Google news
http://click.linksynergy.com/deeplink?id=je6NUbpObpQ&mid=42390&u1=jzhobx2cgf02i8on0mg38&murl=https%3A%2F%2Fwww.sothebys.com%2Fen%2Fvideos%2Fporsche-type-64-an-origin-story

Thank you for making Energized Protection great with your kind help!

A project by Ador with ❤

This is an advertisement site. It's better not to let it open. May cause advertisements pop up on several sites.


why all my output results of the fcn32 is ZERO

after the fcn32 was trained for 9 epochs, the output value score of it is still zero.
however, the weight of each layers are not zero. i am wondering if somebody has suggestions to fix this problem.

I have the sane problem with fcn8s, help anyone?

after the fcn32 was trained for 9 epochs, the output value score of it is still zero.
however, the weight of each layers are not zero. i am wondering if somebody has suggestions to fix this problem.

Hi,Did you solved this problem?I trained 50 epoch,but when I use the trained model to test,the output is zero.

I have the sane problem with fcn8s, help anyone?

Hi,Did you solved this problem?I trained 50 epoch,but when I use the trained model to test,the output is zero.

For training fcn8s, you first start training fcn32s and fcn16s.

I have the sane problem with fcn8s, help anyone?

For training fcn8s, you first start training fcn32s and fcn16s.

Hi,I trained the fcn32 network,but I also has this problem,the output of the network is always zero.

I have the sane problem with fcn8s, help anyone?

Hi,Did you solved this problem?I trained 50 epoch,but when I use the trained model to test,the output is zero.

I met the same problem, have you solved it?

I found that the intermediate output of the network yields to zero after conv1_1, which uses a padding=100. Does anyone knows why this strange padding setting is used?

I found that the intermediate output of the network yields to zero after conv1_1, which uses a padding=100. Does anyone knows why this strange padding setting is used?

I found that in the initialization function, weights are initialized to zero, which may cause the problem. Does anyone knows why the fcn8s need to initialize all the weights to zero?

Because of this https://github.com/wkentaro/pytorch-fcn/issues/40

I didn't solve this problem yet, for these days I was tracked by some trifles, I am just going to resume this work. I do think this problem might caused by the initial strategy. i will response here as soon as i solve this problem.

I didn't solve this problem yet, for these days I was tracked by some trifles, I am just going to resume this work. I do think this problem might caused by the initial strategy. i will response here as soon as i solve this problem.

Hi，Did you solved this problem?Thanks!

I have the same problem with fcn32s, all output results are zeros. Anyone can help ?

I have the same problem, all output results are zeros. Anyone can help ?

I have same problem, did you solve it? @daiyp3360xxxxxisxxxxxxmine

can you run the code?i have lots of problems about this code.Could you please share me your code?My<EMAIL_ADDRESS>you very much!!!


v-bind="$attrs" not override default attributes

Version
2.6.11
Steps to reproduce
Just create nensted input:
Example component

      <input
         v-bind="$attrs"
         :value="value"
         type="text"
      >

What is expected?
type override default type
What is actually happening?
If add add to this component attribute
<input-component type="email" />

type="email" not override type="text" this really uncomfortable

This is expected behavior for Vue 2 (see https://github.com/vuejs/vue/issues/7914). Please search existing issues before opening new ones and remember to use the forum or the Discord chat to ask questions!

Try something like this:

      <input
         v-bind="$attrs"
         :value="value"
         :type="'type' in $attrs ? $attrs.type : text"
      />


WebSiteClientManagementClient.webApps.update[Slot] does not update scmSiteAlsoStopped property

Package Name: @azure/arm-appservice
Package Version: 6.1.0
Operating system: Macos
[x] nodejs

version: 12.13.0

[ ] browser

name/version:

[x] typescript

version: 4.1.2

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
When attempting to update the scmSiteAlsoStopped property using WebSiteClientManagementClient.webApps.update or WebSiteClientManagementClient.webApps.updateSlot scmSiteAlsoStopped is not updated.
To Reproduce
Steps to reproduce the behavior:

Create credentials (const creds = await AzureCliCredentials.create())
Create new WebSiteManagementClient (const client = new WebSiteManagementClient(creds, subscriptionId);)
Create SitePatchResource (const patch: SitePatchResource = { scmSiteAlsoStopped: true })
Update slot

slot = await client.webApps.updateSlot('myGroup', 'myApp', patch, 'test')
console.log({ name: slot.name, state: slot.state, scmSiteAlsoStopped: slot.scmSiteAlsoStopped })

// { name: 'myApp/test', state: 'Running', scmSiteAlsoStopped: false }

I also tried passing all the settings rather than just the scmSiteAlsoStopped. I did this by getting the slot first, updating the return object and passing that in:
let slot = await client.webApps.getSlot('myGroup', 'myApp', 'test')
slot.scmSiteAlsoStopped = true
slot = await client.webApps.updateSlot('myGroup', 'myApp', slot, 'test')
console.log({ name: slot.name, state: slot.state, scmSiteAlsoStopped: slot.scmSiteAlsoStopped })

// { name: 'myApp/test', state: 'Running', scmSiteAlsoStopped: false }

I ran a seperate test to determine if this happened on the primary app, and got the same result.
Expected behavior
I expected the scmSiteAlsoStopped flag to be set on the resource
Additional context
This seems to be a broader problem with Azure APIs (See https://github.com/Azure/azure-powershell/issues/13227), but is also occuring via the JS sdk.

After trying to update scmSiteAlsoStopped directly by rest api instead of sdk, I get the same result and the scmSiteAlsoStopped flag cannot be set. we need help from service team to check why it cannot be set. Thanks

This issue still exisits on latest api version 2021-02-01.