1.12.1 update version

Update 1.12.1 version that. Port https://github.com/dcos/dcos/pull/3955 to the minor branch that was missed.

@mesosphere-mergebot label ready for  review

@mesosphere-mergebot bump-ee


Add getter with out params

This left out materials getter and also index data getters which are a bit more complex

Please modify all original getters to re-use getX(), so that we don't duplicate code especially if we have an issue somewhere.

@SignorPipo I shouldn't have asked you to modify get/set extent actually, for backward compatibility, we need them exactly like before without using getExtents. People might be using them like that:
component.extents[0] = 1;

And that's not going to work with getExtents

Merci for this contribution!


set :default_builder inside Padrino.configure_apps doesn't work

Basically, below code does not work as expected.
Padrino.configure_apps do

  # [...]

  set :default_builder, Utils::FormBuilder

end

Application is still using default StandardFormBuilder. Of course it works when in app.rb directly, but all of my apps will be using custom form builder so puting it in config/apps.rb seems quite natural.

Thank you for your report.


Add loop support

I know that it can be easily added by using
var player = document.querySelectorAll(".player")[0].plyr;

player.media.addEventListener('ended', function() {
  player.play();
});

but a shortcut as part of the configuration would be handy?

Hey @jofrysutanto ,
Did you try the loop attribute? I tested that today and it seemed to work ok?
http://www.w3.org/TR/html-markup/video.html
Cheers

@SamPotts , blimey, how did I missed that.
Though adding up on this, as this is what i end up doing:
plyr.setup({
    click: false
});
player = document.querySelectorAll(".player")[0].plyr;
player.media.loop = true;

It'll be awesome to do this as part of setup:
plyr.setup({
    click: false,
    media: {
        loop: true
    }
});

Unless i'm missing something here.
Awesome library!
Cheers.

I meant you can add it to the <video> or <audio> element. e.g....
<video src="path/to/video.mp4" controls autoplay loop>
</video>

Ahh silly me. Oh well, you learn something new everyday!

No worries :+1:

Hello,
Is it possible to loop videos from vimeo?
cheers

@magicspon word for me by setting the loop functionality within vimeo, in the embed section of the video

@benbonnet you mean the embed settings of the individual videos? Because I can't seem to find a loop option.


Missing about configs for pretrained model

Hello guys, i want to use the pretrained model for music(can be download from https://huggingface.co/lukewys/laion_clap/blob/main/music_audioset_epoch_15_esc_90.14.pt), but i do not know which is the config about this pretrained model(eg: the name of amodel and tmodel). I think i need the config files for every pretrained model.
And further, i try to guess some information about the config through model params. For the music model metioned above maybe  have amodel of htsat-base and robert, but i am not sure if i miss some other important params. Because i test some music data from youtube, the map@10 is very low(about only 0.03).
Who can give me a config file about this model??
Thank for the CLAP team and it is a GREAT work !!

Hi, you can refer to this folder to get the configuration.
music_audioset model uses the htsat-based.json (if my memory serves me well), and they are trained without feature fusion.

is there a table of pretrained models vs config json?


Added  clean to Makefile for FreeBSD

Hello , i added clean for FreeBSD to example Makefile

Thanks for the update @Martinfx!


[BUG]: Admin | Sync Template v2 (Model Driven Apps) always fails

Describe the bug
Admin | Sync Template v2 (Model Driven Apps) always fails
Component (please tell us which flow or app you are experiencing issues with):

Solution Core
Version [e.g. 1.29]
Flow - Admin | Sync Template v2 (Model Driven Apps)

To Reproduce
Steps to reproduce the behavior:

Go to "Last Run"
Click on 'Edit'
Scroll down till "Get Model Driven Apps fails - Error Handling"
See error

Expected behavior
We expect this flow to complete successfully .
Additional context
We also need understanding around what this flow is expected to do as we don't find any details here :https://docs.microsoft.com/en-us/power-platform/guidance/coe/core-components

Good catch on the documentation here, we missed this flow. I have added it to our local copy and it will be present in the next release. Thanks for helping make CoE great!
Note that this flow is the same as the other sync flows but it does it for Model Driven Apps:
Runs when an environment is created or modified, and gets model driven app information.

Can you please go into the block above "Get Model Driven Apps and store in CoE CDS Entity" for the failing record and see what happens in that block?

Can you confirm that the two scopes "Get Model Driven Apps and store in CoE CDS Entity2 and "Get Model Driven Apps fails - Error Handling" are connected by a red dotted line and not a straight black line, see below

If they are connected by a straight black line, please click on the ... next to "Get Model Driven Apps fails - Error Handling" select "Configure run after" and ensure only has failed is checked here

This was missed in a previous version of the CoE Starter Kit and has been fixed since, but based on your comment it looks like you are using an older version of the CoE Starter Kit.

Hi Manuela,
Yes, it succeeded this time . I will monitor it for a day or two to confirm if runs are successful . Can you tell me one thing ? We installed the solution in the month of January but never received any updates. What’s the best approach to remain updated ?
Regards,
Vimmi
From: Manuela Pichler<EMAIL_ADDRESS>Sent: Thursday, April 30, 2020 6:56 PM
To: microsoft/powerapps-tools<EMAIL_ADDRESS>Cc: Rawat, Vimmi<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [External] Re: [microsoft/powerapps-tools] [BUG]: Admin | Sync Template v2 (Model Driven Apps) always fails (#248)
This message is from an EXTERNAL SENDER - be CAUTIOUS, particularly with links and attachments.

Can you confirm that the two scopes "Get Model Driven Apps and store in CoE CDS Entity2 and "Get Model Driven Apps fails - Error Handling" are connected by a red dotted line and not a straight black line, see below
[image]https://urldefense.proofpoint.com/v2/url?u=https-3A__user-2Dimages.githubusercontent.com_55705669_80715407-2D3f798700-2D8aee-2D11ea-2D8101-2D7abfd6abb9e2.png&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=RxdAoBPFsyo_ysmJ6Zhb1B-Y7gjfjYc0tMXS6bZlNCM&m=CKZtO8tTy1vFkuavZ1cTO9ouoAQAEaBUkGChc_ziBB8&s=C1dadyWPCMUxJrfwpF6R1gTm3oTgA3NploPB2HzCnjc&e=
If they are connected by a straight black line, please click on the ... next to "Get Model Driven Apps fails - Error Handling" select "Configure run after" and ensure only has failed is checked here
[image]https://urldefense.proofpoint.com/v2/url?u=https-3A__user-2Dimages.githubusercontent.com_55705669_80715515-2D5ae49200-2D8aee-2D11ea-2D9359-2D004708564750.png&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=RxdAoBPFsyo_ysmJ6Zhb1B-Y7gjfjYc0tMXS6bZlNCM&m=CKZtO8tTy1vFkuavZ1cTO9ouoAQAEaBUkGChc_ziBB8&s=ZURRQEbgCRD0SAjeKtZy0NrJx9GbEGzYag14_6fT3KE&e=
This was missed in a previous version of the CoE Starter Kit and has been fixed since, but based on your comment it looks like you are using an older version of the CoE Starter Kit.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_microsoft_powerapps-2Dtools_issues_248-23issuecomment-2D621833930&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=RxdAoBPFsyo_ysmJ6Zhb1B-Y7gjfjYc0tMXS6bZlNCM&m=CKZtO8tTy1vFkuavZ1cTO9ouoAQAEaBUkGChc_ziBB8&s=Pgoc1LsZuRn0P7xGuRAvtGmrT9jU-jtnt6xVYaWX9UY&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AO54P65DNMG6OTDRVD6VIXLRPF35LANCNFSM4MVNYK6Q&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=RxdAoBPFsyo_ysmJ6Zhb1B-Y7gjfjYc0tMXS6bZlNCM&m=CKZtO8tTy1vFkuavZ1cTO9ouoAQAEaBUkGChc_ziBB8&s=SUoDQJEILSZfNUtsSAahMZjSm6ZivoKbSmWX76uRvZ4&e=.

This message is for the designated recipient only and may contain privileged, proprietary, or otherwise confidential information. If you have received it in error, please notify the sender immediately and delete the original. Any other use of the e-mail by you is prohibited. Where allowed by local law, electronic communications with Accenture and its affiliates, including e-mail and instant messaging (including content), may be scanned by our systems for the purposes of information security and assessment of internal compliance with Accenture policy. Your privacy is important to us. Accenture uses your personal data only in compliance with data protection laws. For further information on how Accenture processes your personal data, please see our privacy statement at https://www.accenture.com/us-en/privacy-policy.

www.accenture.com

Hi,
Followed below and looks like it is working fine now . We will monitor it for a few days .
Two scopes "Get Model Driven Apps and store in CoE CDS Entity2 and "Get Model Driven Apps fails - Error Handling" were connected by a red dotted line and not a straight black line, see below
[image]https://urldefense.proofpoint.com/v2/url?u=https-3A__user-2Dimages.githubusercontent.com_55705669_80715407-2D3f798700-2D8aee-2D11ea-2D8101-2D7abfd6abb9e2.png&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=RxdAoBPFsyo_ysmJ6Zhb1B-Y7gjfjYc0tMXS6bZlNCM&m=CKZtO8tTy1vFkuavZ1cTO9ouoAQAEaBUkGChc_ziBB8&s=C1dadyWPCMUxJrfwpF6R1gTm3oTgA3NploPB2HzCnjc&e=
Click on the ... next to "Get Model Driven Apps fails - Error Handling" select "Configure run after" and checked “has failed” as below :
[image]https://urldefense.proofpoint.com/v2/url?u=https-3A__user-2Dimages.githubusercontent.com_55705669_80715515-2D5ae49200-2D8aee-2D11ea-2D9359-2D004708564750.png&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=RxdAoBPFsyo_ysmJ6Zhb1B-Y7gjfjYc0tMXS6bZlNCM&m=CKZtO8tTy1vFkuavZ1cTO9ouoAQAEaBUkGChc_ziBB8&s=ZURRQEbgCRD0SAjeKtZy0NrJx9GbEGzYag14_6fT3KE&e=
Regards,
Vimmi
From: Jenefer Monroe<EMAIL_ADDRESS>Sent: Thursday, April 30, 2020 6:44 PM
To: microsoft/powerapps-tools<EMAIL_ADDRESS>Cc: Rawat, Vimmi<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [External] Re: [microsoft/powerapps-tools] [BUG]: Admin | Sync Template v2 (Model Driven Apps) always fails (#248)
This message is from an EXTERNAL SENDER - be CAUTIOUS, particularly with links and attachments.

Can you please go into the block above "Get Model Driven Apps and store in CoE CDS Entity" for the failing record and see what happens in that block?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_microsoft_powerapps-2Dtools_issues_248-23issuecomment-2D621826990&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=RxdAoBPFsyo_ysmJ6Zhb1B-Y7gjfjYc0tMXS6bZlNCM&m=r1lJx99NrOdeGGin0QhJxpKorXVIUmdgtA8NktWurHM&s=IVy021cDOunI3IrJS9D0rEyu3WttKmQnl2lxDaULNNw&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AO54P63E56UVDQ4ZJWGJLDLRPF2QVANCNFSM4MVNYK6Q&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=RxdAoBPFsyo_ysmJ6Zhb1B-Y7gjfjYc0tMXS6bZlNCM&m=r1lJx99NrOdeGGin0QhJxpKorXVIUmdgtA8NktWurHM&s=OgbKPIuol4MKcdwsFus6-h2uz0SMe1UHXLP5R6sAkfk&e=.

This message is for the designated recipient only and may contain privileged, proprietary, or otherwise confidential information. If you have received it in error, please notify the sender immediately and delete the original. Any other use of the e-mail by you is prohibited. Where allowed by local law, electronic communications with Accenture and its affiliates, including e-mail and instant messaging (including content), may be scanned by our systems for the purposes of information security and assessment of internal compliance with Accenture policy. Your privacy is important to us. Accenture uses your personal data only in compliance with data protection laws. For further information on how Accenture processes your personal data, please see our privacy statement at https://www.accenture.com/us-en/privacy-policy.

www.accenture.com

Awesome, thanks for confirming!
Good question, at the moment we don't have a good way of notifying users when a new update is released - we do document new releases here under Updates https://github.com/microsoft/powerapps-tools/tree/master/Administration/CoEStarterKit
We are looking at a better way for users to subscribe to updates!
I will close this issue for now, but please let us know if you experience any further issues.


License file?

Is there any intention of adding any form of FOSS license for this project?

Well this project was actually more about keeping a trace this tool I wrote for back-porting c++17 to c++14 in case I need It again. But yes I'll add a MIT license.
Thanks for the interest


Intellij/General IDE Enhancements

Allow generating both buildscript project and main project project together.
Allow auto updating classpath when buildscript changes maybe use https://plugins.jetbrains.com/plugin/7177-file-watchers or write a custom plugin?
Add source jars to intellij buildscripts

when

when the

when the :flushed:


Create Institution > Indicator for Required Fields

Steps to Reproduce

Go to https://collect.earth
Sign in/Create account
Go to "Create an Institution"
Fill in Title field, but not Description
"Create Institution" button is disabled, but there are no warnings/indicators that Title AND Description are required fields.

Screenshots

I prefer not to disable buttons, but instead show an error message when clicked.  Feel free to make that change here.


Viet - Add facility status to backend, and update filter navbar and CR facilities

Description

Types of changes

Checklist

[ ] I have read the CONTRIBUTING doc
[ ] I have checked that unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

CCFRI-2584, CCFRI-2670, CCFRI-2682


Theme not changing

I used that app too many months after updating Gboard i couldn't change themes when i change it reverts back to system default theme
My Gboard version: version <IP_ADDRESS>7749191-beta-arm64-v8a

Ask in the support group
Provide more Infos like your os, phone manufacturer and model name, location (some locations block certain things)

I don't have telegram i don't want to give my phone number to telegram.
Device pixel 3a XL
Android 11

strange that it happens on stock, might be that you have to fix your flags.
Fix flags
Download link from message
if this doesnt help, try turning "use magisk" off in rboard settings

Sorry for not telling first

i was able to use it with that rom
After the gboard update it's gone

try no magisk mode

There is only 1 magisk module that i never tried with it: Safety Net Fix i added that recently can that be a problem?

maybe

i dont know much what can cause problems or can be a fix

Tried ut without magisk module and its working right now (except auto dark and light themes). Please fix that on new update

you cant use auto dark / light without magisk. nothing to fix


⚠ Samsung has degraded performance

In 01b0a58, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2235 ms

Resolved: Samsung performance has improved in 3c824e5.


Add lost section for loki

Added lost section for loki as discussed

Change of plans as discussed: base this commit on latest tag


Entity.builder().set(String, long) doesn't exist?

I have the following code:

entity = Entity.builder(key)
  .set("name", "John Doe")
  .set("age", 30L) // I also tried 30 the int value, which failed similarly.
  .set("updated", false)
  .build();

This fails with the following error:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.0.2:compile (default-compile) on project hello-datastore: Compilation failure: Compilation failure:
[ERROR] /usr/local/google/home/jjg/gcdjavasample/hello-datastore/src/main/java/org/geewax/hellodatastore/HelloDatastore.java:[18,58] error: incompatible types
[ERROR]
[ERROR] could not parse error message:   required: KeyFactory
[ERROR] found:    Builder
[ERROR] /usr/local/google/home/jjg/gcdjavasample/hello-datastore/src/main/java/org/geewax/hellodatastore/HelloDatastore.java:29: error: no suitable method found for set(String,long)
[ERROR] .set("age", 30L)
[ERROR] ^
[ERROR]
[ERROR] method Builder.set(String,Blob) is not applicable
[ERROR] (actual argument long cannot be converted to Blob by method invocation conversion)
[ERROR] method Builder.set(String,List) is not applicable
[ERROR] (actual argument long cannot be converted to List by method invocation conversion)
[ERROR] method Builder.set(String,FullEntity) is not applicable
[ERROR] (actual argument long cannot be converted to FullEntity by method invocation conversion)
[ERROR] method Builder.set(String,Key) is not applicable
[ERROR] (actual argument long cannot be converted to Key by method invocation conversion)
[ERROR] method Builder.set(String,DateTime) is not applicable
[ERROR] (actual argument long cannot be converted to DateTime by method invocation conversion)
[ERROR] method Builder.set(String,boolean) is not applicable
[ERROR] (actual argument long cannot be converted to boolean by method invocation conversion)
[ERROR] method Builder.set(String,String) is not applicable
[ERROR] (actual argument long cannot be converted to String by method invocation conversion)
[ERROR] method Builder.set(String,Value) is not applicable
[ERROR] (actual argument long cannot be converted to Value by method invocation conversion)
[ERROR] /usr/local/google/home/jjg/gcdjavasample/hello-datastore/src/main/java/org/geewax/hellodatastore/HelloDatastore.java:[32,15] error: cannot find symbol
[ERROR]

If I take away the .set("name", "John Doe") I get a different error...

You can put this in EntityTest and see that it works (I just did).
Entity entity = Entity.builder(KEY1)
.set("name", "John Doe")
.set("age", 30) // I also tried 30 the int value, which failed similarly.
.set("updated", false)
.build();
what java version are you using? Also, might be related to the error above it...
Can you provide the whole file?

Java version:
$ java -version
openjdk version "1.7.0-google-v6"
OpenJDK Runtime Environment (build 1.7.0-google-v6-86819614-77620724)
OpenJDK 64-Bit Server VM (build 24.76-b04, mixed mode)

The whole file is:
package org.geewax.hellodatastore;

import com.google.gcloud.datastore.DatastoreService;
import com.google.gcloud.datastore.DatastoreServiceFactory;
import com.google.gcloud.datastore.DatastoreServiceOptions;
import com.google.gcloud.datastore.Entity;
import com.google.gcloud.datastore.Key;
import com.google.gcloud.datastore.KeyFactory;

public class HelloDatastore {
  private static final String DATASET = "gcloud-datastore-demo";

  public static void main(String[] args) {
    DatastoreServiceOptions options = DatastoreServiceOptions.builder().dataset(DATASET).build();
    DatastoreService datastore = DatastoreServiceFactory.getDefault(options);
    KeyFactory keyFactory = (KeyFactory) datastore.newKeyFactory().kind("Person");
    Key key = keyFactory.newKey("Jimmy");

    System.out.println("Trying to get the entity by its key!");

    Entity entity = datastore.get(key);

    if (entity == null) {
      System.out.println("Entity not found! Creating it!");
      entity = Entity.builder(key)
          .set("age", 30L)
          .build();
    }
  }
}

What I'm seeing from my end is that chaining things together seems to break...
So for example, the following compiles without any issues:
      Entity.Builder builder = Entity.builder(key);
      builder.set("age", 30L);
      builder.set("updated", false);//.set("name", "John Doe");
      entity = builder.build();

But the following fails:
      Entity.Builder builder = Entity.builder(key);
      builder.set("age", 30L);
      builder.set("updated", false).set("name", "John Doe");
      entity = builder.build();

The trace is:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.0.2:compile (default-compile) on project hello-datastore: Compilation failure
[ERROR] /usr/local/google/home/jjg/gcdjavasample/hello-datastore/src/main/java/org/geewax/hellodatastore/HelloDatastore.java:[29,35] error: no suitable method found for set(String,String)
[ERROR] -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.0.2:compile (default-compile) on project hello-datastore: Compilation failure
/usr/local/google/home/jjg/gcdjavasample/hello-datastore/src/main/java/org/geewax/hellodatastore/HelloDatastore.java:[29,35] error: no suitable method found for set(String,String)

        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:213)
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:153)
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:145)
        at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:84)
        at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:59)
        at org.apache.maven.lifecycle.internal.LifecycleStarter.singleThreadedBuild(LifecycleStarter.java:183)
        at org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:161)
        at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:320)
        at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:156)
        at org.apache.maven.cli.MavenCli.execute(MavenCli.java:537)
        at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:196)
        at org.apache.maven.cli.MavenCli.main(MavenCli.java:141)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:289)
        at org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:229)
        at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:415)
        at org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:356)
Caused by: org.apache.maven.plugin.CompilationFailureException: Compilation failure
/usr/local/google/home/jjg/gcdjavasample/hello-datastore/src/main/java/org/geewax/hellodatastore/HelloDatastore.java:[29,35] error: no suitable method found for set(String,String)

        at org.apache.maven.plugin.AbstractCompilerMojo.execute(AbstractCompilerMojo.java:516)
        at org.apache.maven.plugin.CompilerMojo.execute(CompilerMojo.java:114)
        at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:101)
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:209)
        ... 19 more

Similarly, with the KeyFactory, I need to do some magical casting to make this all work. For example:
The following fails
KeyFactory keyFactory = datastore.newKeyFactory().kind("Person")

However the following compiles without issue
KeyFactory keyFactory = (KeyFactory) (datastore.newKeyFactory()).kind("Person")

Is there some magical thing that makes type casting "just work" ? Or am I just doing something really really stupid.... ?

Taking this down to the simplest example I can construct....
This compiles:
package org.geewax.hellodatastore;

import com.google.gcloud.datastore.Entity;
import com.google.gcloud.datastore.Key;

public class HelloDatastore {
  public static void main(String[] args) {
    Key key = Key.builder("dataset", "Person", "Jim").build();
    Entity.Builder builder = Entity.builder(key);
    System.out.println(builder.set("f", 30L));
  }
}

When run, it outputs
[INFO] --- exec-maven-plugin:1.3.2:java (default-cli) @ hello-datastore ---
[WARNING] Warning: killAfter is now deprecated. Do you need it ? Please comment on MEXEC-6.
com.google.gcloud.datastore.Entity$Builder@79089b11

This fails to compile (the only addition is .set("name", "Jim") chained onto things, which should be possible give the previous object was an Entity.Builder according to the prior output.
package org.geewax.hellodatastore;

import com.google.gcloud.datastore.Entity;
import com.google.gcloud.datastore.Key;

public class HelloDatastore {
  public static void main(String[] args) {
    Key key = Key.builder("dataset", "Person", "Jim").build();
    Entity.Builder builder = Entity.builder(key);
    System.out.println(builder.set("f", 30L).set("name", "Jim"));
  }
}

With the following trace:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.0.2:compile (default-compile) on project hello-datastore: Compilation failure
[ERROR] /usr/local/google/home/jjg/gcdjavasample/hello-datastore/src/main/java/org/geewax/hellodatastore/HelloDatastore.java:[19,44] error: no suitable method found for set(String,String)
[ERROR] -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.0.2:compile (default-compile) on project hello-datastore: Compilation failure
/usr/local/google/home/jjg/gcdjavasample/hello-datastore/src/main/java/org/geewax/hellodatastore/HelloDatastore.java:[19,44] error: no suitable method found for set(String,String)

        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:213)
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:153)
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:145)
        at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:84)
        at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:59)
        at org.apache.maven.lifecycle.internal.LifecycleStarter.singleThreadedBuild(LifecycleStarter.java:183)
        at org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:161)
        at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:320)
        at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:156)
        at org.apache.maven.cli.MavenCli.execute(MavenCli.java:537)
        at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:196)
        at org.apache.maven.cli.MavenCli.main(MavenCli.java:141)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:289)
        at org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:229)
        at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:415)
        at org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:356)
Caused by: org.apache.maven.plugin.CompilationFailureException: Compilation failure
/usr/local/google/home/jjg/gcdjavasample/hello-datastore/src/main/java/org/geewax/hellodatastore/HelloDatastore.java:[19,44] error: no suitable method found for set(String,String)

        at org.apache.maven.plugin.AbstractCompilerMojo.execute(AbstractCompilerMojo.java:516)
        at org.apache.maven.plugin.CompilerMojo.execute(CompilerMojo.java:114)
        at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:101)
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:209)
        ... 19 more

OK I think I found the problem.
BaseEntity.Builder is a package-private class. So the tests would work (as they're in the same package) but something from another package (like anyone's project....) won't work :(
Can we make BaseEntity.Builder public?

I don't think this is a problem in this case.
Any public method M in a packaged scope class B that is extend by a public class A in the same package would be visible to all via A.
However, because indeed unit-tests in Java are typically using the same package as the class they test such errors could happen (which should raise the priority for examples/integration-test...).
I am still not sure what is the problem is in your case (though I start to wonder if the gcloud-java version that is being pulled and used in your example is not the version we are expecting 0.0.3)....
Running maven in verbose mode (-X) could help with that.
Also, I added examples package to gcloud-java, so maybe you want to give it a try (or improve on it)...

I made the change locally, and the code compiles.
I think the issue is in the generic casting situation, because as I mentioned, when I cast the value to an Entity.Builder after calling .set(), everything works just fine.
When I don't cast and the BaseEntity.Builder class is private, code doesn't compile. When it's public, code compiles.
Can you try a test using gcloud-java from a different package and seeing where I'm coming from here? This is how our users will interact with the library, so if you can't reproduce there, I'll keep digging for a better solution and call this "works-for-me".

The problem was that @jgeewax was using an older version of maven (< 3.0) which was using (by default) an older version (<3.0) of maven-compiler-plugin (< 2.3). the maven-compiler-plugin version 2.2 and before is configured to use java source before 1.7 but gcloud-java is using java 1.7 features.
Possible solutions (any one should work):

use a newer version of maven (3.0 or above)
use a newer version of maven-compiler-plugin (2.3 or above)
explicitly specify source/target with "1.7" for the maven-compiler-plugin configuration.


Change Set to UniqueSeq

Create my own collection or use an existing collection that accepts distinct elements while being covariant.

or a vector with distinct


Error during setup of component xiaomi_miot

After upgrade xiaomi_miot to 0.4.14  integration doesn't work with multiple errors in the log:
Logger: homeassistant.setup
Source: custom_components/xiaomi_miot/core/xiaomi_cloud.py:409
Integration: Xiaomi Miot Auto (documentation, issues)
First occurred: 12:33:01 (1 occurrences)
Last logged: 12:33:01

Error during setup of component xiaomi_miot
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 174, in _new_conn
    conn = connection.create_connection(
  File "/usr/local/lib/python3.9/site-packages/urllib3/util/connection.py", line 73, in create_connection
    for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
  File "/usr/local/lib/python3.9/socket.py", line 954, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -3] Try again

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 699, in urlopen
    httplib_response = self._make_request(
  File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 382, in _make_request
    self._validate_conn(conn)
  File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 1010, in _validate_conn
    conn.connect()
  File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 358, in connect
    conn = self._new_conn()
  File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 186, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7f42e8bc92e0>: Failed to establish a new connection: [Errno -3] Try again

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/requests/adapters.py", line 439, in send
    resp = conn.urlopen(
  File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 755, in urlopen
    retries = retries.increment(
  File "/usr/local/lib/python3.9/site-packages/urllib3/util/retry.py", line 574, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='ru', port=443): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f42e8bc92e0>: Failed to establish a new connection: [Errno -3] Try again'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/setup.py", line 229, in _async_setup_component
    result = await task
  File "/config/custom_components/xiaomi_miot/__init__.py", line 211, in async_setup
    hass.data[DOMAIN]['devices_by_mac'] = await mic.async_get_devices_by_key('mac') or {}
  File "/config/custom_components/xiaomi_miot/core/xiaomi_cloud.py", line 248, in async_get_devices_by_key
    dvs = await self.async_get_devices(renew=renew) or []
  File "/config/custom_components/xiaomi_miot/core/xiaomi_cloud.py", line 222, in async_get_devices
    dvs = await self.hass.async_add_executor_job(self.get_device_list)
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/xiaomi_miot/core/xiaomi_cloud.py", line 181, in get_device_list
    rdt = self.request_miot_api('home/device_list', {
  File "/config/custom_components/xiaomi_miot/core/xiaomi_cloud.py", line 151, in request_miot_api
    rsp = self.request(self.get_api_url(api), params)
  File "/config/custom_components/xiaomi_miot/core/xiaomi_cloud.py", line 409, in request
    response = self.session.post(url, data=post_data, timeout=self.http_timeout)
  File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 590, in post
    return self.request('POST', url, data=data, json=json, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 655, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/requests/adapters.py", line 516, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='ru', port=443): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f42e8bc92e0>: Failed to establish a new connection: [Errno -3] Try again'))

I restored from backup version 0.4.9 files, and now all is OK.

Please upgrade to master branch and try again.

Please upgrade to master branch and try again.

After this upgrage the problem appears again:

Logger: homeassistant.setup
Source: custom_components/xiaomi_miot/core/xiaomi_cloud.py:410
Integration: Xiaomi Miot Auto (documentation, issues)
First occurred: 10:44:25 (1 occurrences)
Last logged: 10:44:25

Error during setup of component xiaomi_miot
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 174, in _new_conn
    conn = connection.create_connection(
  File "/usr/local/lib/python3.9/site-packages/urllib3/util/connection.py", line 73, in create_connection
    for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
  File "/usr/local/lib/python3.9/socket.py", line 954, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -3] Try again

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 699, in urlopen
    httplib_response = self._make_request(
  File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 382, in _make_request
    self._validate_conn(conn)
  File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 1010, in _validate_conn
    conn.connect()
  File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 358, in connect
    conn = self._new_conn()
  File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 186, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7fdd477cd430>: Failed to establish a new connection: [Errno -3] Try again

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/requests/adapters.py", line 439, in send
    resp = conn.urlopen(
  File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 755, in urlopen
    retries = retries.increment(
  File "/usr/local/lib/python3.9/site-packages/urllib3/util/retry.py", line 574, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='ru', port=443): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fdd477cd430>: Failed to establish a new connection: [Errno -3] Try again'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/setup.py", line 229, in _async_setup_component
    result = await task
  File "/config/custom_components/xiaomi_miot/__init__.py", line 211, in async_setup
    hass.data[DOMAIN]['devices_by_mac'] = await mic.async_get_devices_by_key('mac') or {}
  File "/config/custom_components/xiaomi_miot/core/xiaomi_cloud.py", line 248, in async_get_devices_by_key
    dvs = await self.async_get_devices(renew=renew) or []
  File "/config/custom_components/xiaomi_miot/core/xiaomi_cloud.py", line 222, in async_get_devices
    dvs = await self.hass.async_add_executor_job(self.get_device_list)
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/xiaomi_miot/core/xiaomi_cloud.py", line 181, in get_device_list
    rdt = self.request_miot_api('home/device_list', {
  File "/config/custom_components/xiaomi_miot/core/xiaomi_cloud.py", line 151, in request_miot_api
    rsp = self.request(self.get_api_url(api), params, **kwargs)
  File "/config/custom_components/xiaomi_miot/core/xiaomi_cloud.py", line 410, in request
    response = self.session.post(url, data=post_data, timeout=timeout)
  File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 590, in post
    return self.request('POST', url, data=data, json=json, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 655, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/requests/adapters.py", line 516, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='ru', port=443): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fdd477cd430>: Failed to establish a new connection: [Errno -3] Try again'))

https://github.com/al-one/hass-xiaomi-miot/blob/11e403742e3d77ca53ec1f8bc5f89620cd2564d4/custom_components/xiaomi_miot/core/xiaomi_cloud.py#L398-L399
Please modify your component local file, like this:
    def request(self, url, params, **kwargs):
        _LOGGER.warning('xiaomi miot request: %s', [url, params])
        self.session = self.api_session()

And give me the logs after restarting HA.

In fact, the correct error report should be like this: HTTPSConnectionPool(host='ru.api.io.mi.com', port=443), but the host in your log is ru.

Please check your configuration:
# configuration.yaml
xiaomi_miot:
  username: xiaomi_username
  password: xiaomi_password
  server_country: ru

Not:
# configuration.yaml
xiaomi_miot:
  username: xiaomi_username
  password: xiaomi_password
  server_country: http://ru # ❌  Error example

In fact, these configurations are optional. You can integrate devices through your account without yaml.

The problem disappeared after deleting of these entries in configuration.yaml
xiaomi_miot:
  username<EMAIL_ADDRESS>  password: xxxxxxxxxxx
  server_country: ru
  language: zh
  translations:
    Silence: Бесшумный
    Stanrd: Стандарт
    Middle: Средний
    Enchance: Турбо

Just delete these entries:
  username<EMAIL_ADDRESS>  password: xxxxxxxxxxx
  server_country: ru


MAINT: remove error throw in binned_statistic_dd() on non-finite values or samples

Reference issue
Closes gh-11365
What does this implement/fix?
When I added an error throw in gh-10664 it wasn't the right fix and broke the the lightkurve package, see e.g.
https://github.com/KeplerGO/lightkurve/pull/661
and
https://github.com/KeplerGO/lightkurve/issues/663
The proposal from @barentsen as a fix is confirmed here and the test for non-finite values is repurposed to check and raise a ValueError if the bins arg is int type and there are any non-finite values.
Additional information
The error is raised in binned_statistic_dd() so it is straightforward to the user but the underlying issue is that _bin_edges() is not robust to a specified number of bins and any non-finite values. A follow up PR may want to provide a more permanent fix to _bin_edges()-if possible.

Removing the "needs decision" tag, there seems to be general agreement this was an unintential defect that I introduced. Agreement is to backport in 1.4.2 and include in 1.5.0 release too.
See e.g. discussion in associated issue.

@rlucas7, could you incorporate the changes that I suggested in https://github.com/scipy/scipy/pull/10664 into this PR?  The function should not reference values when statistic is 'count'.
Also, double-check that the code you add complies with PEP-8 (max. line length 79, spaces after commas, etc.).  I often temporarily rename tox.ini to something like tox.ini.save, and then run pycodestyle on the files that I'm editing to ensure that I don't introduce any new PEP-8 violations.

EDIT
@rlucas7, could you incorporate the changes that I suggested in #10664 into this PR? The function should not reference values when statistic is 'count'. Sorry, I see that the pull request already fixes that, by simply not checking values for finiteness. I'll look into gh-11365 before commenting further.
Also, double-check that the code you add complies with PEP-8 (max. line length 79, spaces after commas, etc.). I often temporarily rename tox.ini to something like tox.ini.save, and then run pycodestyle on the files that I'm editing to ensure that I don't introduce any new PEP-8 violations.

I'll check the PEP 8 and update the 2d binned statistic docstring example you found on the original PR when I get home.

@WarrenWeckesser I reverted the docstring changes in the binned_statistic_2d function, the second instance of x in the function call is now reverted to None in both instances. I also made the PEP 8 changes. There where E501 issues in the doctest results, the BinnedStatisticResult return values were all on 1-line. I've continued them across 2 (or 3 if necessary) lines. Hopefully this doesn't make the doctests fail in the build, I tested them locally after the chnages via python  scipy/stats/_binned_statistic.py  which gave no output so hopefully these are fine.
Also, thanks for the tip on the tox file. Indeed that has a setting to ignore the E501 in pycodestyle setting. I didn't realize there were pycodestyle ignore flags inside that file-good to know.

@WarrenWeckesser I've addressed the PEP8 issues in doctests and those are not failing on latest build. The 2 travis failures appear to be related to a curvefit covariance test failing, nothing that I've changed. Would be great if you could take another look at the PR when you have time.

The previous test errors were unrelated (special and linalg tests) and the branch was about 10 days old so I rebased.

looks like the tests all pass in travis on rebase.
Only windows failure is:
https://dev.azure.com/scipy-org/SciPy/_build/results?buildId=5316&view=logs&j=689af343-93ec-58dc-880e-8a2e7b5f2ce5&t=0daec287-0840-5bd9-6022-6f148a04400b&l=478
and this PR does not touch the differential evolution solver.
Leaving as is until @WarrenWeckesser has time to review the changes he suggested.

@rlucasy, finally got back to this.  Looks good.
The master branch now has a couple lines added to the unit test to avoid making the inplace change of self.u visible after the test is finished.  That could be simplified by using u = self.u.copy(), and then use the code in this PR for the rest of the test.  Or do the swapping that is in master--either is fine.

@WarrenWeckesser I added the changes you've suggested. Thanks for catching that one.
I only added a test case within the existing non-finite test and only for np.int64.
I'm guessing np.int is a wrapper for python's int because that was the only one I tested that seemed to pass the check. Although, np.int8, np.int32 etc. all would've failed I figured testing only one of the int types from numpy should be sufficient. If you want me to add the others in I can-just seems like overkill.

I'm guessing np.int is a wrapper for python's int

np.int is the Python int object, and that has occasionally been a source of confusion over the years.  This aliasing of Python's builtin types in the numpy namespace may soon be deprecated: https://github.com/numpy/numpy/pull/14882

Although, np.int8, np.int32 etc. all would've failed I figured testing only one of the int types from numpy should be sufficient.

Yup, that's fine.

@rlucas7, thanks! I removed a couple comments that weren't really needed, fixed a line length, squashed and merged.

Thanks Warren, LGTM.
Sincerely,
-Lucas Roberts

On May 11, 2020, at 12:53 AM, Warren Weckesser<EMAIL_ADDRESS>wrote:

@rlucas7, thanks! I removed a couple comments that weren't really needed, fixed a line length, squashed and merged.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Adding Dark Mode to the Website

I am Praful Nigam. Under GSSOC'24, I would like to work on this issue. Pls assign me this!!

I would like to work on this issue. Pls assign me this!!

already taken


Client does not paginate through all account domains

When I configure to update sub.domain.com it comes back with:
The domain sub.domain.com doesn't appear to be one of your CloudFlare domains. We only found [...] and the list of TLDs it finds (.com, .co.uk, .uk) is a fraction of what I have in my CF account.  Is there some restriction on TLDs that are pulled back?

This library does not place any restrictions on the available TLDs, as far as I am aware. Maybe the API is enforcing this.
There's not much I can do here without a minimally reproducible example.

It looks like it's down to the default setting of 20 domains per page.  If I increase this to the max of 50, then I see the domains that I need.  It doesn't look like cloudflare-ddns-client is paginating, so it only pulls back the first 20 domains in the account.
per_page | Number of zones per page 20 | default value: 20 min value:5 max value:50
https://api.cloudflare.com/#zone-list-zones

Nice catch! I'm happy to accept a PR to fix this behavior.

I'll push a PR to increase to the max of 50 as temporary fix, and once I get chance I'll try and look into the pagination.


Perplexity Numbers Reported for Fan et al

Are the reported perplexities calculated with respect to the NLTK tokenization or the BPE encoding scheme used by the model or at some word by word level?

In the paper "Hierarchical Story Generation", we did not use BPE. The perplexity is reported at the word level, for the vocabulary size reported in the paper.


Incorrect output when $clog used in nested function calls

Version
Yosys 0.25+3 (git sha1 2677569d4, gcc 9.3.0-22 -fPIC -Os)
On which OS did this happen?
Linux
Reproduction Steps
top.sv:
module top (o);
	output wire signed [31:0] o;
	parameter signed [31:0] X = 3;
	function automatic signed [$clog2(X) - 1:0] narrow;
		input reg signed [$clog2(X) - 1:0] inp;
		narrow = inp;
	endfunction
	function automatic signed [31:0] to_int;
		input reg signed [31:0] inp;
		to_int = inp;
	endfunction
	assign o = to_int(narrow(X));
endmodule

Executed commands:
$ yosys
yosys> read_verilog top.sv

This issue doesn't occur when to_int is removed or when $clog2 is replaced by another function which just returns a value like:
	function automatic signed [1:0] get_3;
		get_3 = 3;
	endfunction

so use of $clog2 might be relevant here.
Expected Behavior
Produced o should be 32-bit wide and its value should be 3, e.g.:
output signed [31:0] o;
assign o = 3;

Actual Behavior
Produced o is 32-bit wide but its value is -1, generated -dump_vlog2:
Dumping Verilog AST after simplification:
    module top(o);
      /** AST_PARAMETER **/
      output signed [31:0] o;
      /** AST_FUNCTION **/
      /** AST_FUNCTION **/
      assign o = -1;
    endmodule
--- END OF AST DUMP ---

I believe Yosys's behavior on this particular example is standards-compliant. Commercial simulators and iverilog yield -1, too.
narrow(3) is effectively 2'(3), which is 2'sb11. to_int(2'sb11) then simply sign-extends, yielding a regular size-32 -1.
Perhaps you have found an issue, but this example doesn't quite replicate it?

I analyzed this example once again and it looks like I somehow misinterpreted $clog2(3) to be 3 instead of 2 which confused me because I was getting different results for "the same value". -1 is of course correct, sorry for the confusion.


does not work on visual studio

I cannot reproduce the bug description. TrackballScroll works fine with Visual Studio 2019 v16.9.3.

My version of vs is the same and does not scroll in any file... I do not know if I have to configure anything

Does scrolling work for you in any other program, e.g. the file explorer? If not, make sure the trackball buttons emit the the default mouse events.
E.g. for the Logitech Marble Mouse, the settings in the "SetPoint" driver software are set to forward/backward initially, which is not captured by TrackballScroll.

Works on all programs except VS 2019...

This is a wild guess but do you start VS 2019 as administrator? TrackballScroll cannot send signals to software running with different access permissions.


Another try for auto-publish to gh-pages

This is a #PATCH#

Not sure why coveralls push is failing, but it seems related to: https://github.com/lemurheavy/coveralls-public/issues/487


File still opened after calling Stop method

I try to delete a file I've opened with xZune.Vlc immediately after I've stopped its playback.
This is my code:
VlcPlayer.BeginStop(() =>
{
RecycleOption recycleOption = RecycleOption.DeletePermanently;
if (SettingsViewModel.SendToRecycleBin)
recycleOption = RecycleOption.SendToRecycleBin;
FileSystem.DeleteFile(_imagePathList[_currentImageIndex].FullName,UIOption.OnlyErrorDialogs,
recycleOption);
});
But Windows Explorer tells me that the file is still opened somewhere.

I have fixed this problem with the use of Tasks


[UI] Review Upper - background color based on srs rank

Color the background based on srs rank.
Pink Purple DarkBlue LightBlue Black

scratch that, too distracting.


Keyboard manager does not disable hotkeys when they are "rolled"

Microsoft PowerToys version
v.058.0
Running as admin

[X] Yes

Area(s) with issue?
Keyboard Manager
Steps to reproduce

set up a remap to win + E and set that to disable on all apps
set up a remap to win + V and set that to disable on all apps

A quick definition of how I'm using the term "rolled". While holding down lwin, Hold E then hold V,
When doing the lwin + E OR lwin + V shortcut it correctly clears the hotkey. However if you "roll" the hotkey by holding down lwin + E + V you can see it no longer overriding it with left click oemclear anymore.
In the attached video you can see that at around 4 seconds the clipboard history fires off (lwin + v) and right after the explorer fires off (lwin+E)
The keyboard overlay on the left side also shows the key press sequences and the vs code window has a summary of what I'm trying to demonstrate.
https://user-images.githubusercontent.com/13132548/170129123-7314910c-3b1a-4740-bbd1-f93655376925.mp4
✔ Expected Behavior
I had expected it to not allow those hotkeys through when rolled. OR to allow definitions of LWIN+E+V to be defined so that it can also be set to disabled. The second ask might be a weird one because it's not often that people chord two or more keys after holding down a modifier.
❌ Actual Behavior
fires off the disabled hotkeys when you roll two keys together even though these are keys set to disabled.
Other Software
No response

Reproduced Issue
Case

Shortcuts not rolled.

winkey + e
winkey + v

-> Nothing happens, as expected.

Shortcut rolled

winkey + e + v
winkey + v + e

-> Both shortcuts are registered.

Interesting. Thanks for the report. I can reproduce this.

@ujl123 What is the status in the latest version?

@ujl123 What is the status in the latest version?
this has not been resolved yet.

Would this fix https://github.com/microsoft/PowerToys/issues/10393 as well?


[chip-test] chip_sw_sysrst_ctrl_input

Test point name
chip_sw_sysrst_ctrl_input
Host side component
Rust
OpenTitanTool infrastructure implemented
No
Silicon Validation (SiVal)
Yes
Emulation Targets

[ ] None
[ ] CW310
[x] Hyperdebug + CW310

Contact person
No response
Checklist
Please fill out this checklist as items are completed. Link to PRs and issues as appropriate.

[x] Check if existing test covers most or all of this testpoint (if so, either extend said test to cover all points, or skip the next 3 checkboxes)
[ ] Device-side (C) component developed
[ ] Bazel build rules developed
[ ] Host-side component developed
[x] HJSON test plan updated with test name (so it shows up in the dashboard)
[x] Test added to dvsim nightly regression (and passing at time of checking)
[ ] For SiVal test cases, test is running relevant FPGA or silicon regression

estimate 3

Fixed #20820

Needs the Bazel test point merged in the testplan. @pamaury . May already have merged in - but not visible from an update today.


trigger field validation

I'm setting certain field contents with code.
Is there functionality to trigger the field validation programatically, so I can have it show a warning or error?

Whoops... Should have looked better.
Basically its just try? field.validate()


Remove "sequelpro" command (for now).

What happened (or feature request):
As per the decisions in https://github.com/drud/ddev/issues/28#issuecomment-286201906, we are removing the sequelpro as a top level command within ddev until we can determine a better home and/or solution.
What you expected to happen:
I should no longer see "sequelpro" as an option when running "ddev" from the command line.
Anything else do we need to know:
We don't want to completely destroy this, but rather return to it at the appropriate time. We can split out into a future ticket for shortcuts and services.
Related source links or issues (like source JIRA issue):
https://github.com/drud/ddev/issues/28#issuecomment-286201906

What we've discussed outside the issue queue is to hide this command for Linux and keep it for macOS.


Halt DataDog::Client after errors

This will fix #6 and mostly based on the internal work done by @s-nez.

Looks all good now


Fix delta_secs issue when not compiling with physics_avian feature

This one's pretty self explanatory. One change was just missed. 😄

My bad, forgot to build with non default feature flags😅 Thanks for the PR!


chocolatey package for Rocket.Chat.Electron is deprecated

I was the maintainer for the chocolatey package for Rocket.Chat.Electron.
I am deprecating the chocolatey package for Rocket.Chat.Electron because I no longer have time to maintain it.
I have requested for another package maintainer here: https://github.com/chocolatey/chocolatey-package-requests/issues/414
The chocolatey package page for Rocket.Chat.Electron : https://chocolatey.org/packages/rocketchat
Sorry for all the inconvenience I have caused.
(This issue is created automatically, some of the links may be wrong, please inform me if You want me to change them.)

Please test if the problem persists with our latest release. I will be closing the issue but if there is any problem yet, we will open it again.


Dialog can not have hasClose when hasHeader is false

Hello,
Apparently you can not have the close cross when you don't display the header.
However, this is I'd like to have. 😬

Hello, is this still relevant? I would like to work on this issue.

It is. :)

@brandongregoryscott hello, can you assign me to this issue, if that's not a problem?


Replace gRPC/protobuf by FlatBuffer

https://google.github.io/flatbuffers/
https://reasonablypolymorphic.com/blog/protos-are-wrong/index.html

Evaluate other options like msgpack, as @wrobell pointed.


Some videos not playing on Chrome Works on FF

I am developing a streaming application using Clapper.js . Some HLS streams that are directly converted from DVB UDP Stream using FFMPEG (Not encoded) is not playing, or takes too much time to start playing. But these streams are playing as quickly as possible on latest version of Firefox and old version of Chrome (Version 69). This makes inconsistency between browsers.
I am using basic configuration taken from Clapper.js documentation. NB: Streams encoded with FFMPEG works without any problem on both FF and Chrome. Also using latest version of clapper.js.

Any information ?


Problem when opening multiple windows at the same time

Hello, I am getting a null reference error when doing something like this:
urls.forEach(url => {   blankshield.open(url, '_blank'); });
Looks like this is caused by the child.opener = null in iframeOpen function, preventing to execute this line for the second time and onwards.

This is because most browsers won't let you open windows in a loop, so the child window ends up being null for every subsequent invocation of window.call. For example, this will fail to open more than 1 window:
['https://github.com', 'https://github.com'].forEach(url => { window.open('https://www.github.com', '_blank') });

Will update the code to prevent throwing an error :)


Merges develop latest changes back into release/10.0.0

Merges develop latest changes back into release/10.0.0
Also ran a local build after the rebase and had to fix some packaging issue with the new Aperture Theme.

This is a release management task, as per our policy, we are self-approving it.


[HotChocolate] Emitting state to new subscribers causes a race condition

Describe the bug
c.f., Slack message.
Often, we want to fetch the initial state of an object while simultaneously subscribing to changes. Attempting to do this as two discrete queries leads to a race condition wherein the object may be mutated between the query and the subscription (it's also very clunky for the clients). However, it appears there is no way to respond to a new subscription by sending the initial state of the subscribed object (ideally, ONLY to the new client, not to previous subscribers).
To Reproduce
Consider a typical subscribe pattern:
  [ExtendObjectType(Name = OpenControllerSchema.Subscription)]
  public class PortsSubscription {
    [Subscribe(With = nameof(SubscribeToAllPortState))]
    [AuthorizeReadControllers]
    public SystemPort OnPortChange([EventMessage] SystemPort port) => port;

    public ValueTask<IObservable<SystemPort>> SubscribeToAllPortState(
      [Service] ControllerManager controllerManager,
      CancellationToken ct
    ) => controllerManager.Ports.SubscribeToAll(PortTopic.State, ct);

There is no hook (AFAIK) in between the two methods. The latter is called implicitly, but the subscription is not active until it returns. Therefore, any attempts to emit a change in the controllerManager.Ports.SubscribeToAll method is invalid. That is, the IObservable has no observers yet because Subscribe has not yet been called.
Expected behavior
I would like to emit the initial state of the object back to ONLY the client which is newly subscribed. I have no strong opinions how to achieve this.

Hmm... it looks like implementing my own IObservable class can solve this. Specifically, I'm overriding Subscribe to emit the last message to any new subscriber...
public class SubscribableTopic<T> : IObservable<T> where T : ITopicMessage {
    private readonly List<TopicSubscription<T>> _subscriptions = new();

    private T? _lastMessage;

    public virtual IDisposable Subscribe(IObserver<T> observer) {
      TopicSubscription<T> sub = new TopicSubscription<T>(this, observer);
      if (_lastMessage != null) sub.observer.OnNext(_lastMessage);
      return sub;
    }

    public virtual void Emit(T obj) {
      lock (_subscriptions) {
        for (int i = 0; i < _subscriptions.Count; i++) {
          _subscriptions[i].observer.OnNext(obj);
        }
      }
      _lastMessage = obj;
    }

... but I'd be interested in any feedback or ideas how this could be improved.

An Observable is a good solution. If you have an Idea for how to make this a simple thing I am all yours :)


Use local Unix socket instead of BSD socket

We BSD is vulnerable (and possibly slower) than Unix socket.

On the Lisp side, simply replace usocket with iolib.
Can Electron create Unix sockets?
As far as I understand, it's enough to replace the port by a path:
https://nodejs.org/api/net.html

Node js is capable of anything (I guess making coffee too 😨)

No longer true since https://github.com/usocket/usocket/commit/7a24c571541b1d046976f0c175f1f2658dbdf0f7.


Feature Request: Get a svelte repl in a preview panel

Coding in VScode.dev and results in a preview panel. Not sure how to do it tho.

Rendering this? https://github.com/sveltejs/svelte-repl

Sounds like a great idea, but I'm not sure it should go into this extension - it's currently a port of the official extension which doesn't have a preview panel... I'll keep the issue open for now, let's think about it...

Yeah, I forgot about the official extension.


vscode-extensions.equinusocio.vsc-material-theme: 33.8.0 -> 34.3.1

Description of changes
https://github.com/material-theme/vsc-material-theme/releases/tag/v34.3.1
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Looks like there's a hash mismatch now...

good that you checked. I am confused...


fix: update initial schema for pg14

What kind of change does this PR introduce?
partial fix #401
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 29.677%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1461

Relevant Lines:
4923

💛  - Coveralls

superseded by #408


Payments are always registered double

Hello
For some strange reason, all payments are always registered double on the orders page.
In the example below, the order amount was 199,99 EUR, but it says 399,98 € was paid which is not true.
The client paid 199,99 EUR via Mollie (payment method Mistercash).
I also double check if it's a certain method, but no it happens with all of them.
Except in some cases there is no method available. Also very strange. (example in attachment too)
Did somebody encounter this issue too? Any ideas or fixes to get rid of this annoying issue?
Does it has anything to do with the order status that might register a payment double?
I enclose a screenshot from the settings if anybody needs to see it.
Please let me know if this is a known (new) issue or if we are using some kind of wrong parameter or configuration, but I can not see anything wrong in the configuration section.
Thanks!

Hello, anybody alive here?

Hi @7GRAFIX,
We are currently investigating this with our Prestashop partner and will let you know as soon as we can provide you with an update. For now, apologies for the issues and delay.

Hello @7GRAFIX,
Could you please let us know if you still need support? Otherwise, we have to close the issue.
Regards

Hello,
Sorry for late reply.
We are running version <IP_ADDRESS>. and mollie version 1.3.0 with below server details.
Servergegevens Linux #1 SMP Debian 3.16.39-1+deb8u2 (2017-03-07) x86_64
Server softwareversie nginx/1.10.3
PHP versie 7.0.9
Geheugenlimiet 512M
Maximale uitvoeringstijd 3000
MySQL versie 10.0.30-MariaDB-0+deb8u2
Is that Mollie version you sent newer than the one we already using?

Hi @7GRAFIX ,
The plugin I sent is only compatible with Prestashop 1.7 and up. I sent it because your problem, as described, is a know problem in Prestashop 1.7. It's not compatible with Prestashop 1.6.
I see you are using PHP 7.0.9. Is it possible to lower your PHP version to version 5.6.x (or lower, minimum version: 5.2) and do a test payment to see if the problem still occurs? PHP 7.0.x is known to cause problems because it contains bugs, so this could be the cause of your problem as well.
You can also disable your other (paid) Modules (the ones which you have added afterwards and which are not standard) and keep the Mollie Module enabled. Do a test payment to see if one of your other modules is conflicting with the Mollie Module.
Regards

Hi @7GRAFIX ,
Have you had a chance to try our suggestions? Please let us know so we can solve this problem.
Regards

Hi @7GRAFIX ,
We are closing this issue. You can still place comments to let us know if you still need support. We will close comments after 1 week.
Regards

@Trendweb
Sorry, I just saw this message. I changed recently company name and domain.
We have changed PHP to 5.6 and are still investigating the issue.
There is/was also an issue with another module - EffectConnect - to sync product data with Bol.com plaza which also conflicted due it's own payment settings.
We have disabled this plugin too and we are now monitoring all activity and orders to see if the issue is solved permanently.

Hi @7GRAFIX ,
Thank you for informing us. Please let us know if the issue has been solved perminently, if not, we will offer you further support to resolve this. Just a suggestion, are you currently using a "clean URL module" to remove ID numbers from the URL's? Some of these cause problems with other modules. Perhaps this could be (part of) the problem as well.
Regards

@Trendweb
We have been debugging for the past days but the problem is not solved.
Payments are still being registered double.
We are running PHP 5.6 now for 2 weeks.
EffectConnect has been disabled the whole week and still not solved.
We are not using any modules that touch the URL or permalinks.
Any other suggestions that might cause this?

Hi @7GRAFIX ,
Can you please contact me at<EMAIL_ADDRESS>. To solve this, we will have to debug ourselves / recreate the error so we can find out what is causing the problem.
Regards


In Information entry => Language proficiency information page, data is not saving.

It's working properly.


nextjs build+export

(node:35459) UnhandledPromiseRejectionWarning: ReferenceError: window is not defined
    at doloadGPTScript (/Users/jonatan/projects/react-dfp-surge/node_modules/react-dfp/lib/utils.js:9:3)
    at /Users/jonatan/projects/react-dfp-surge/node_modules/react-dfp/lib/utils.js:29:5
    at new Promise (<anonymous>)
    at Object.loadGPTScript (/Users/jonatan/projects/react-dfp-surge/node_modules/react-dfp/lib/utils.js:28:10)
    at EventEmitter.getGoogletag (/Users/jonatan/projects/react-dfp-surge/node_modules/react-dfp/lib/manager.js:132:42)
    at EventEmitter.init (/Users/jonatan/projects/react-dfp-surge/node_modules/react-dfp/lib/manager.js:92:12)
    at EventEmitter.doLoad (/Users/jonatan/projects/react-dfp-surge/node_modules/react-dfp/lib/manager.js:155:10)
    at EventEmitter.load (/Users/jonatan/projects/react-dfp-surge/node_modules/react-dfp/lib/manager.js:148:26)
    at Module.<anonymous> (/Users/jonatan/projects/react-dfp-surge/.next/server/static/i0iUOVpMWpyfeE9xt0Qwf/pages/basic.js:230:54)
    at __webpack_require__ (/Users/jonatan/projects/react-dfp-surge/.next/server/static/i0iUOVpMWpyfeE9xt0Qwf/pages/basic.js:23:31)

@rhaze are u still experiencing issues?, keep in mind that calls to load() and refresh() need to be made only after the component is mounted at browser side, so you typically will have to place them into componentDidMount() { }.

We're using react-dfp with Next.js v9 with no issues. For client side operations don't forget to make use of if (process.browser) {...}


ci: fix release action

Codecov Report

Merging #208 (a83645a) into develop (22bd131) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff            @@
##           develop     #208   +/-   ##
========================================
  Coverage    19.53%   19.53%
========================================
  Files           15       15
  Lines         1065     1065
  Branches       373      373
========================================
  Hits           208      208
  Misses         834      834
  Partials        23       23

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Event: appland.appmap/plugin/appmap:upload

:tada: This issue has been resolved in version 0.18.0 :tada:
The release is available on:

v0.18.0
GitHub release

Your semantic-release bot :package::rocket:


Unexpected spacing within Edit Preferences Drawer

The Add Option button seems too close to the questions below it.

Wrong repo :(


Turn on snippets for additional context surrounding data loss prevention rule violations

Admins can now view ‚ÄúSensitive Content Snippets‚Äù for data loss prevention (DLP) rules. This applies to DLP events for Drive, Chat, and Chrome. When turned on, snippets will log the matched content that triggered a DLP violation in the security investigation tool. Admins can use the information captured in the snippet to better identify actual security risks, determine whether a false positive was returned, and decide on an appropriate course of action.
Publish Date: Wednesday, December 6, 2023
Update Blog Link

This is being deferred to the hardening guide. No changes required to baseline.


Pass opening files to a reader thread_worker when opening list of paths

Description
This PR would be one option for addressing the second part of #3125 by handing off reading of layers to a thread_worker when the user has a large list of files to open, using the current mechanisms we have in place.
I've refactored _add_layers_with_plugins so that we can use it for adding stacks (which I'm not passing off to a thread_worker, though we could) and so we can compare the difference in the opening methods.
The thread_worker gets created by get_reader_worker which parameterizes the worker with its total number of files (for the progress bar) and a yield callback (to add the layer to the viewer on yield). This could also just be an inner declaration in viewer.open though.
The major issue with this approach (and I think any approach we could take with the mechanisms we have currently available, though @tlambert03 might be able to correct me on this), is that we can no longer have viewer.open return the list of layers it has added, which is a pretty big API change and not something I think we should necessarily move forward with?
However, if we had some form of Futures object as proposed in #2581 and #2593, we would be able to populate our returned list with these objects instead. I don't think this PR in its current form should be merged necessarily, but I think it's good to open a discussion about how we could support threaded layer reads. I'd appreciate any thoughts people have (@jni, @sofroniewn) - maybe we can pick up the idea of a Futures object again?
Type of change

[ ] Bug-fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

References

How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

The major issue with this approach (and I think any approach we could take with the mechanisms we have currently available, though @tlambert03 might be able to correct me on this), is that we can no longer have viewer.open return the list of layers it has added, which is a pretty big API change and not something I think we should necessarily move forward with?

this really is a big challenge and a major difference between the needs of the interactive/command line user and the GUI user.  The ipython user is generally quite accustomed to a synchronous execution model, and more than happy to wait for the results of the last command if it's "reasonably" short.  The GUI user, however, expects the gui to never freeze up, and recognizes that the layer will "get there when it gets there"... while they are free to do other things.  The GUI user doesn't need a pointer to the new layer (since they can access the new layer in the GUI) while the ipython user needs that pointer.
This is a really fundamental conflict that we're unlikely to solve in a "general" way (i.e. both users are unlikely to be able to use the same API without frustrating one of the two).
I can (currently) think of two general patterns that we might use, but maybe there are others:

Introduce a general pattern of adding an asynchronous parameter to a lot of our functions that toggles whether the user should expect a Future object, or the actual result.  It would default to asynchronous=False (i.e. block and wait for the result), to match the current API, but internally we could use asynchronous=True wherever we wanted to.
Have a general proxy API that different users (i.e. plugin writers vs end-users vs internal napari) are expected to use.  This is a lot more complicated on the dev side, and would probably require some tricky changes... but might ultimately make the distinction between our public APIs and our internal business logic clearer.  If you look through the vscode extension API, you see a ton of functions that return Promises (the javascript equivalent of the Future), (but javascript users are all familiar with promises, since it's all over the web).  There, it's almost a completely different API that the extensions are expected to use from the internal stuff.

probably others?  definitely tricky... still need to ponder

definitely tricky... still need to ponder

Indeed sounds it. I'm not sure here either, maybe @jni has ideas

This PR is being closed in favor of tracking discussion and progress on this subject in issue #3570.


Add properties

This addresses #65.
It isn't as clean as a pure @property approach, but it turns out you can't use @property and @staticmethod at the same time. So to keep the Config functioning as a namespace like it is now (instead of requiring Config to be created as an object), I had to add the property functionallity to a MetaClass.

@alxwrd Perfect thanks, this has taught me lots!


Version of esmvaltool is 2.6.1 when installing esmvalcore in development mode

This is because we have a hard(-ish) pin on esmvalcore in the env file at 2.7.x but the development version of esmvalcore is 2.8.1dev

In te most recent CircleCI development installation test run it seems to work fine:
ESMValCore: 2.7.1
ESMValTool: 2.7.1.dev32+g9d3a65b1e

(copied from here)

Or do you mean installing both ESMValTool and ESMValCore from their current main branches?

Or do you mean installing both ESMValTool and ESMValCore from their current main branches?

yes this one, but even so I can't manage to get 2.6.1 for Tool again like I did on me laptop at home - just gonna close this and forget about it, if we notice it any way we can take it from there :+1:


Starting with tsx does not execute correctly.

@JounQin
// main.ts
import { createSyncFn, TsRunner } from 'synckit'

const syncFn = createSyncFn(require.resolve('./worker.ts'), {
    tsRunner: TsRunner.TSX,
})

syncFn(1)

// worker.ts
import { runAsWorker } from 'synckit'

console.log('worker started')

runAsWorker(async function main(...args) {
    console.log(args)
    return 'hello'
})

After starting with yarn tsx main.ts, work is not executed.

Please provide a runnable reproduction. Maybe you want require.resolve('./worker.js') instead.

I use tsx, shouldn't require a js file? I did not compile the ts file

Sorry for the delay.
require.resolve('./worker.js') will resolve .ts file inside.
Do you have a reproduction if you have any issue?

maybe you need install tsx pkg to your project.

Close due to inactive.


Windows 11 Support

Could we get this page updated with activation support for Windows 11? Thanks!

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

ID: c3b24b69-ab33-c1b1-b8e2-51c23bbba31d
Version Independent ID: ebc3123c-57f7-0d2c-f875-9b6ac6c7a319
Content: Key Management Services (KMS) activation planning for Windows Server
Content Source: WindowsServerDocs/get-started/kms-activation-planning.md
Product: windows-server
Technology: getting-started
GitHub Login: @dknappettmsft
Microsoft Alias: daknappe

Thanks for the feedback. This article has been updated for Windows 11 and the changes should be visible later today.
#please-close


Update README.md

made it clear where to clone the open_pdks repo

@mattvenn can you please modify it to: cd ../pdks instead of cd pdks
Because we are working on the assumption that the user will follow the previous instructions as well, and this way he should already be inside pdks.
Also please submit the PR to the develop branch.


Add informing NSE and Client of local mechanisms provisioned

Once dataplane completed provisioning LocalMechanism for a client and nse, the relevant information
get sent to both. In case of NSE not finding provisioned LocalMechanism, an error gets returned to the client.
Signed-off-by: Serguei Bezverkhi<EMAIL_ADDRESS>

@fkautz when I had interface in the name @edwarnicke came up with a concern that interface is too overloaded term and he came up with Mechanism term, so I think you guys needs to be consistent how you want to name things and not change that depending on a season.

@fkautz also I am fine to close this pr if you plan to address these things in yours.

@sbezverk I get it now... so 'interface_name' is a perfectly fine parameter for the kernel interface LocalMechanism type... the problem is insisting that every LocalMechanism has an interface_name (because they don't).
That parameter "interface_name" for a LocalMechanism of kernel interface type should already be set when it comes back from the Dataplane... just pass through the NSM on its way to the NSE :)
Different LocalMechanisms will have different parameter keys, example:
Kernel Interface:

interface_name

Memif:

socketfile
master
slave

etc.  If the NSE in question only understands the Kernel Interface LocalMechanism type (which we can tell because it only advertises that type of LocalMechanism) then any LocalMechanism we send it would have a 'interface_name' (or whatever name we settle on to hold the name of the interface) parameter (plus whatever other parameters are needed to 'fully specify' a LocalMechanism of that type).
Apologies if I was less than clear, and seemed to be advocating for strange names in a situation like this.

@edwarnicke As yo ucan see this PR is closed, no further discussion is required..


allow #[trigger] on FnSpec call and inside a lambda

#[trigger] in a lambda adds the expression to the lambda apply axiom.
This enables:
#[verifier(external_body)]
proof fn something(fn1: FnSpec(S)->bool, fn2: FnSpec(S)->bool)
ensures forall|s: S| #[trigger] fn1(s) ==> fn2(s) { }

proof fn foo(s: S) {
  something(|s1| #[trigger] prop_1(s1), |s1| prop_2(s1));
  assert forall|s: S| prop_1(s) implies prop_2(s) by {
    assert(prop_1(s));
    assert(prop_2(s));
  }
}

where both #[trigger] annotations are necessary for assert(prop_2(s)); to pass.
This is a pattern encountered by @marshtompsxd as described in https://github.com/verus-lang/verus/discussions/326

[ ] automatically select triggers for quantifier with calls to FnSpecs?

You can also do this without the trigger annotation on the lambda:
#[verifier(external_body)]
proof fn something(fn1: FnSpec(S)->bool, fn2: FnSpec(S)->bool)
ensures forall|s: S| #[trigger] fn1(s) ==> fn2(s) { }

proof fn foo(s: S) {
  something(|s1| prop_1(s1), |s1| prop_2(s1));
  assert forall|s: S| prop_1(s) implies prop_2(s) by {
    assert((|s1| prop_1(s1))(s));
  }
}

Thank you @utaal ! This works well in my local test


Output to file not working in >5.4.0

Bug Report

TSLint version:  5.4.2
TypeScript version: 2.3.4
Running TSLint via:  CLI

TypeScript code being linted
tslint -p tsconfig.json -c tslint.json -o tslint-output.json

with tslint.json configuration:
{
    "rules": {
    	"no-var-keyword": true
    }
}

Actual behavior
With a versions start from 5.4.0 tslint-output.json file is empty. Works with 5.3.2, 5.3.0.
Expected behavior
tslint-output.json file has information about errors.
Note
I added only one rule in tslint.json just for sample purpose. It doesn't work with any other rule too.

Seems like a race condition between asynchronously writing to file and the call to process.exit(code) after linting.
PR incoming.


Translation to russian

Hello!
Why you do not want to add Russian translation for your plugin?

Sorry about that, just haven't found the time to do it yet...

Thx


The following are the requested features....

Context menus need to be consistent. if possible, make them like these:

and also the login scree, profile image is too huge.
proposed design:

@m2058 This repo is not the place for random Windows 10 feedback.


Responsive design support

This function solves the problem (这个功能解决的问题)
It is currently very difficult to manipulate components based on screen resolution for things like mobile/tablet/desktop. There is no service that helps identify which platform the app is running on. Therefore making a responsive app with Naive UI is also very difficult.
Expected API (期望的 API)
Provide an API like Vuetify: https://vuetifyjs.com/en/features/display-and-platform/ This allows triggering display states based on device and screen resolution in both CSS and JS.

Any progress on this?

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem


Refactoring the public TD management

Description
Please include a brief summary of the change that this pull request proposes. Include any relevant motivation and context. List any dependencies required for this change.

Storing the total difficulty inside the best block at run time to reduce potential mismatches caused by calling separate functions for the best block and total difficulty.

Fixes Issue # .
Type of change
Insert x into the following checkboxes to confirm (eg. [x]):

[ ] Bug fix.
[x] New feature.
[ ] Breaking change (a fix or feature that causes existing functionality to not work as expected).
[ ] Requires documentation update.

Testing
Please describe the tests you used to validate this pull request. Provide any relevant details for test configurations as well as any instructions to reproduce these results.

Existing suite of unit tests.

Verification
Insert x into the following checkboxes to confirm (eg. [x]):

[x] I have self-reviewed my own code and conformed to the style guidelines of this project.
[x] New and existing tests pass locally with my changes.
[ ] I have added tests for my fix or feature.
[x] I have made appropriate changes to the corresponding documentation.
[x] My code generates no new warnings.
[x] Any dependent changes have been made.

Possible for AionBlockchainImpl to be at an inconsistent state until a new block is imported:
In the previous setup on boot, we load the blockchain and that involves setting the total difficulty, with these changes what used to be pubBestTD will now be block.getDifficulty() until the first new block is imported, after which bestBlock() will be set with the correct totalDifficulty(). Suggestion is to set totalDifficulty() when setting the best block.

@yulongaion available to review?


cpackget fails to list packs while csolution succeeds

$ csolution list packs
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>frq09468@LMECWL0888 MINGW64 /c/Packs
$ echo $CMSIS_PACK_ROOT
/c/CMSIS_PACKS
frq09468@LMECWL0888 MINGW64 /c/Packs
$ cpackget pack list
I: Using pack root: "C:/CMSIS_PACKS"
I: Listing installed packs
E: Pack vendor "" does not match ^[0-9a-zA-Z_-]+$
E: A pack in the cache folder has malformed pack name: C:.CMSIS_PACKS.ARM.CMSIS-Driver.2.6.1.pack
E: unknown behavior

@fred-r  as a temporary workaround could you please set the CMSIS_PACK_ROOT variable:
export CMSIS_PACK_ROOT=c:\\CMSIS_PACKS

Works indeed, thanks !
frq09468@LMECWL0888 MINGW64 /c/Packs
$ export CMSIS_PACK_ROOT=c:\\CMSIS_PACKS

frq09468@LMECWL0888 MINGW64 /c/Packs
$ cpackget pack list
I: Using pack root: "c:\CMSIS_PACKS"
I: Listing installed packs
I: ARM.CMSIS-Driver.2.6.1
I: ARM.CMSIS.5.7.0
I: ARM.CMSIS.5.8.0
I: ARM.mbedTLS.1.7.0
I: ARM.PSA.1.0.0
I: Keil.ARM_Compiler.1.6.3
I: Keil.MDK-Middleware.7.12.0
I: Keil.MDK-Middleware.7.13.0
I: Keil.NUCLEO-L552ZE-Q_BSP.1.2.0
I: Keil.STM32F4xx_DFP.2.15.0
I: Keil.STM32H7xx_DFP.2.8.0
I: Keil.STM32L4xx_DFP.2.6.1
I: Keil.STM32L552E-EV_BSP.1.1.0
I: Keil.STM32L562E-DK_BSP.1.3.0
I: Keil.STM32L5xx_DFP.1.4.0
I: Keil.STM32L5xx_TFM-PF.1.1.0
I: Keil.STM32NUCLEO_BSP.1.8.0
I: Keil.STM32U5xx_DFP.1.1.0
I: STMicroelectronics.board_resources_B-U585I-IOT02A.0.0.1
I: STMicroelectronics.Example_HAL_GPIO_Toggle.0.0.1-alpha1
I: STMicroelectronics.STM32U5xx_DFP.0.1.0-alpha1
I: STMicroelectronics.STM32U5xx_Drivers.0.0.1-alpha1
I: STMicroelectronics.STM32U5xx_Drivers.0.1.0-alpha1

@fred-r, Thanks for opening the issue!
Which terminal/bash are you using?

By default, CMSIS_PACK_ROOT environment variable is defined in a Windows environment as C:\SOME_PATH. Both bash and cmd work fine with that approach.
In git-bash/mingw, if the user tries to export CMSIS_PACK_ROOT=C:\SOME_PATH or --pack-root/-R C:\SOME_PATH, bash automatically removes the backwards slash \ because it is considered to be a escape character, so it becomes C:SOME_PATH, which is a valid directory name. It's wrong, but valid.
I wrongly assumed that this would throw an error when cpackget tried to check if it existed. The colon : character is not allowed in directory names on Windows. But in reality, that does not fail in neither bash nor cmd:

in bash, this becomes a valid name C:SOME_PATH which Windows later translates it to the unicode character 61498 which is valid.
in cmd, this becomes SOME_PATH (C: is removed) only, which is still valid.

To fix that, users should either escape all backwards slashes C:\\SOME_PATH\\SUB_DIRor wrap it around single/double quotes cpackget -R "C:\SOME_PATH".
Another finding is that bash paths formatted as /c/SOME_PATH are always translated to C:/SOME_PATH with the forward slash before passing it to cpackget. And this is where things get confusing, because when cpackget lists installed packs, and they all start with C:\SOME_PATH. This means that cpackget will never receive paths in /c/SOME_PATH format. It seems this is just a visualization feature of bash.
By using the correct approach to set the path, cpackget works just fine.
So I created this patch to fix cases where paths like C:/SOME_PATH are sent to cpackget. The following paths are now cleared before being used by cpackget:

C:/SOME-PATH becomes C:\SOME_PATH
C:////SOME_PATH becomes C:\SOME_PATH
C:\\\\SOME_PATH becomes C:\SOME_PATH
C:/SOME_PATH\SUBDIR becomes C:\SOME_PATH\SUBDIR, and so on