🐛 Bug: Re-run command suggestion includes second --mode create

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Expected
I ran npx create-typescript-app --mode create --base everything and filled in all the options. At the end, it should have suggested a re-run command that:

Included --base everything and --mode create
Filled in values I provided in the prompt...
...and not values implied by --base everything or --mode create

Actual
This was the suggestion it printed:
●  Tip: to run again with the same input values, use: npx create-typescript-app --mode create --base everything --access public --description "Hooray. Delete." --directory delete-me-please --email-github<EMAIL_ADDRESS>--email-npm<EMAIL_ADDRESS>--mode create --owner JoshuaKGoldberg --repository delete-me-please --title "Delete Me Please"

Note the two --mode creates. Bug!
Additional Info
No response

This isn't happening anymore.
node ~/repos/create-typescript-app/bin/index.js --mode create --base minimal --repository cta-cwd-test-1 --title "CTA cwd test 1" --description "Testing cwd in wip pr" --owner JoshuaKGoldberg

●  Tip: to run again with the same input values, use: npx create-typescript-app --base minimal --author "Josh Goldberg ✨" --description "Testing cwd in wip pr" --directory cta-cwd-test-1 --email-github<EMAIL_ADDRESS>--email-npm<EMAIL_ADDRESS>--mode create --owner JoshuaKGoldberg --preserve-generated-from --repository cta-cwd-test-1 --skip-restore --title "CTA cwd test 1"


NOJIRA fix ATs

This reverts "MTDSA-17206 Delete Historic FHL UK Property Business Annual Submission" #406
MTDSA-17206 will be re-opened and merged once the corresponding stubs PR is merged

Closed in favour of #411


Fixed #24 invalid enter_room and leave_room

Hi there!
Thanks for submitting this PR! 🎉
Have you tested it out before you've submitted the PR?

@pyropy Yes, I'm currently using it in my code and  it works :)


fix description message of command line options

Several command line options' descriptions were obviously copied and pasted, resulting in inconsistent description of these options.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: huang195
To complete the pull request process, please assign howardjohn
You can assign the PR to them by writing /assign @howardjohn in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@huang195: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
0b7ffa1f6083e28e1a59058c1e5036d26576435b
link
/test integ-new-install-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, huang195
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

feel free to remove the hold if you don't want to address the nitpick, just didn't want it automerged

New changes are detected. LGTM label has been removed.

@huang195: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
0b7ffa1f6083e28e1a59058c1e5036d26576435b
link
/test integ-new-install-k8s-presubmit-tests-master

e2e-mixer-no_auth-master
51e947577f5bbafe2bfb52417ba0e05b607f8a8a
link
/test e2e-mixer-no_auth-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@huang195: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e-mixer-no_auth-master
51e947577f5bbafe2bfb52417ba0e05b607f8a8a
link
/test e2e-mixer-no_auth-master

integ-new-install-k8s-presubmit-tests-master
51e947577f5bbafe2bfb52417ba0e05b607f8a8a
link
/test integ-new-install-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/hold cancel

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, huang195
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/retest


DeleteBehavior.ClientSetNull sets FKs to null on SaveChanges in some cases

I can't reopen #8633 so I'm posting a new issue.
The relationship is configured like this:
modelBuilder.Entity<Post>().HasOne(t => t.Blog).WithMany("PostBlogCollection").HasForeignKey("Blog_Seqnr").OnDelete(DeleteBehavior.ClientSetNull);

The following code:
var post = ctx.Posts.Find(1);
var blog = ctx.Blogs.Find(2);
// post.Blog == blog
post.Text = "asdf"; // Just to make post's State Modified
ctx.Blogs.Remove(blog);
ctx.SaveChanges();

Sets post.Blog_Seqnr to null and saves this to the DB successfully. I expected it to crash on SaveChanges with "ORA-02292: integrity constraint [...] violated - child record found".
On the other hand, the following code does exactly what I expected. The difference is only that post is now Unchanged instead of Modified:
var post = ctx.Posts.Find(1);
var blog = ctx.Blogs.Find(2);
// post.Blog == blog
ctx.Blogs.Remove(blog);
ctx.SaveChanges();

@Wain123 I am not able to reproduce this. In both cases, the blog is deleted from the database and the FK on the post is set to null. Here's my code:
public class Blog
{
    [DatabaseGenerated(DatabaseGeneratedOption.None)]
    public int Id { get; set; }

    public ICollection<Post> PostBlogCollection { get; set; }
}

public class Post
{
    [DatabaseGenerated(DatabaseGeneratedOption.None)]
    public int Id { get; set; }
    public string Text { get; set; }

    public int? Blog_Seqnr { get; set; }
    public Blog Blog { get; set; }

}

public class Context : DbContext
{
    public DbSet<Blog> Blogs { get; set; }
    public DbSet<Post> Posts { get; set; }

    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        => optionsBuilder.UseSqlServer(
            @"Server=(localdb)\mssqllocaldb;Database=Test;Trusted_Connection=True;ConnectRetryCount=0");

    protected override void OnModelCreating(ModelBuilder modelBuilder)
    {
        modelBuilder.Entity<Post>()
            .HasOne(t => t.Blog)
            .WithMany("PostBlogCollection")
            .HasForeignKey("Blog_Seqnr")
            .OnDelete(DeleteBehavior.ClientSetNull);

    }
}

public class Program
{
    public static void Main()
    {
        using (var context = new Context())
        {
            context.Database.EnsureDeleted();
            context.Database.EnsureCreated();

            context.Add(new Post { Id = 1, Blog = new Blog { Id = 2 } });
            context.SaveChanges();
        }

        using (var context = new Context())
        {
            var post = context.Posts.Find(1);
            var blog = context.Blogs.Find(2);

              post.Text = "asdf"; // Just to make post's State Modified

            context.Blogs.Remove(blog);
            context.SaveChanges();
        }

        using (var context = new Context())
        {
            var post = context.Posts.Find(1);
            var blog = context.Blogs.Find(2);
        }
    }
}

Apologies, I found a mistake in my code - indeed the FK is set to null in both cases.
However, if that is the intended behavior, why is #8633 marged as fixed by ClientSetNull?

@Wain123 ClientSetNull for non-nullable FKs (i.e. required relationships) is essentially the same as Restrict because it is not possible to set null. Therefore, an exception is thrown in SaveChanges instead. Nulls can only be set for nullable properties.


[Windows] Optimize the containerized OVS installation

Add logic to check the installed OVSext drivers, if only the desired version
of driver is already installed, skip the installation; otherwise, remove the
existing the drivers and re-install.
Add logic to check the installed VC redistributable files, if the existing
installd vc_redist version is equal to or higher than the lowest requirement,
skip the installation; otherwise re-install the files provided in the image.
Optimize the logic of maintaining the system paths by removing the duplicated
path.
Optimize Uninstall-OVS script by removing OVS bin paths from the system path
after it is deleted.

@XinShuYang @antoninbas I have introduced a ps module "OVSHelper.psm1" in this change, which is used to provide general functions independent from the exact versions of OVS. So I place it in the container image directly, rather than put the functions in Install-OVSDriver.ps1 packaged in the configmap. I didn't add the functions to the existing Helper.psm1, because I want to only provide the necessary function in the PS module file in container images.
I am thinking how to co-work with the existing script Install-OVS.ps1, which is trying to install OVS driver and/or userspaces on the Windows Node and register as native service. In theory, I would prefer to use the new implementations in Install-OVS.ps1, and I also hope to keep that script is independent as it used to work. I don't have a good idea to directly copy the functions in the script or import the new module. Copy functions may introduce troubles in the future code maintenance; while import-module may introduce file dependencies for the script.

Let's merge https://github.com/antrea-io/antrea/pull/6360 first, given that the mentorship program is coming to an end and I don't want to have @shikharish to resolve conflicts.

/test-windows-all

/test-windows-all

/test-windows-all

It feels like the script is still a bit too confusing, because of the 2 cases we are trying to support (services vs containerized). There are still references to "host paths" (e.g. hostDriverPath). IMO, it would be better if we could avoid such references. To me it seems that there is one install directory (OVSInstallDir, which defaults to C:\openvswitch). In the containerized case, it happens to be a host path, which is what we want. There are also 3 possible sources for OVS artifacts: local directory, local zip archive, web URL (pointing to archive). When we install the driver, we want to copy the driver file to $OVSInstallDir\driver. When we do that copy, the source and destination should not be the same, and it would be better if could ensure it, even if it means that in the web URL / archive cases, we use a temporary location first to unzip the files. Then we install the utility programs in the same way. Then based on the value of $InstallUserspace, we take care of services.
I hope that with this approach, we can handle all cases in a uniform and consistent way.

@antoninbas , now I got your point, updated accordingly in my latest change. Please help review again.

I have manually tested the script on my local testbed with both cases to install userspace services and not install services.

I have manually tested the script on my local testbed with both cases to install userspace services and not install services.

@wenyingd We have jenkins-windows-ovs in CI to verify related code changes. In the run_install_windows_ovs function, it tries to run Install-OVS.ps1 and perform a health check for OVS. With the current code changes in this PR, do you think we need to update or add some checks?

/test-windows-all

/test-windows-all


Hide Menu Button

Is there any way to hide the hamburger button in the navigation bar of a master detail page?

Hi @WilliamDumontRT
It can only be done by resorting to custom renderers. A quick search reveals some solutions out there for it.
However, you should consider whether its advisable to do so. A swipe will most likely still reveal the menu that the hamburger icon would reveal.


Set up access to SMS service at Link Mobility

Description
Link mobility will be used for sending SMS.
We need to set up access to both test and production environment
and store required connection secrets in key vault.
Tasks

[ ] Research ways to connect to service
[ ] Set up logic or practice for storing required secrets in KV
[ ] Document setup process (Altinn pedia?)
[ ] Manual verification

Acceptance Criteria

[ ] SMS micro serivce in AT, TT, YT has access to secrets enabling it to connect to the Link Mobility API (test env)
[ ] SMS micro serivce in production has access to secrets enabling it to connect to the Link Mobility API (prod env)

Tried accessing using LinkMobility's SDK, but endpoint is hard coded.
Have followed up with an issue in their repository:
https://github.com/PSWinCom/LinkMobility.PSWin.Client/issues/3


Correct social share icon

Right now it's a Ukraine design for GDG Lviv

Theoretically fixed, but I don't know where to test


Java 11 EOL

Depending  on the platform, Java 11 has already or will reach EOL soon, e.g. Oracle JDK, EOL Sep 2023, OpenJDK, EOL Oct 2024.
I suggest we move to the next LTS versions, Java 17 or Java 21.

Firstly, and as far as I understand, security updates will be available for Java 11 until 2032, right? So, there is no real need to change the Java version now.
Secondly, do you want to use Java 17 or 21 for the Docker image running the command-line tool or as basis for the source code? The difference is that we can easily use any Java version >= 11 to run the compiled citydb-tool on the command line. But this repository also provides a Java API and library which can be used by others to build their own software with 3DCityDB support. If we use Java 17 or 21 as basis for our source code, this will also be a mandatory dependency for users of our API. On the other hand, having a first stable release based on Java 11 and changing the Java version later will mean a breaking change for users. So maybe we should switch to a newer Java version before the first stable release?
@yaozhihang, what do you think? Java 17 and 21 introduce nice language features such as records, sealed classes and virtual threads that might be interesting to use. Should we switch now?

I think we could switch java 17 or 21. It looks more strict to the java environment requirements. But there are also nice solutions e.g. we can build our citydb-tool together with a light-weight java runtime.

There is no specific reason for me to use Java 17 or 21. I just saw that some distributors are starting to EOL Java 11, so I figured it's best to move to a newer version now and avoid making the change in the near future. As you do the dev work, you should decide this.

One more thing I came accross. I tried building the tool with a Java 17 base image, but that fail. README.md currently says Java 11 or higher is requried. I believe this is related to this line: https://github.com/3dcitydb/citydb-tool/blob/48e32cad98916d24cf4738135a3b09e377bf8804/build.gradle#L24

No issue on my local machine. Can build with Java 17 without issues. Can you post the error message?

Der Fehler tritt auf, wenn ich mit dem JDK 17 als baseimage bauen möchte:
docker build --progress=plain \
  --build-arg RUNTIME_IMAGE_TAG=17-jdk \
  --build-arg BUILDER_IMAGE_TAG=17-jdk -t test .

#8 [builder 4/4] RUN chmod u+x ./gradlew && ./gradlew installDist
#8 0.324 Downloading https://services.gradle.org/distributions/gradle-8.4-bin.zip
#8 1.254 ............10%............20%.............30%............40%.............50%............60%.............70%............80%.............90%............100%
#8 3.409
#8 3.409 Welcome to Gradle 8.4!
#8 3.409
#8 3.409 Here are the highlights of this release:
#8 3.409  - Compiling and testing with Java 21
#8 3.409  - Faster Java compilation on Windows
#8 3.410  - Role focused dependency configurations creation
#8 3.410
#8 3.410 For more details see https://docs.gradle.org/8.4/release-notes.html
#8 3.410
#8 3.508 Starting a Gradle Daemon (subsequent builds will be faster)
#8 15.11
#8 15.11 FAILURE: Build failed with an exception.
#8 15.11
#8 15.11 * What went wrong:
#8 15.11 Could not determine the dependencies of task ':citydb-cli:installDist'.
#8 15.11 > Could not resolve all dependencies for configuration ':citydb-cli:runtimeClasspath'.
#8 15.11    > Failed to calculate the value of task ':citydb-cli:compileJava' property 'javaCompiler'.
#8 15.21       > No matching toolchains found for requested specification: {languageVersion=11, vendor=any, implementation=vendor-specific} for LINUX on x86_64.
#8 15.21          > No locally installed toolchains match and toolchain download repositories have not been configured.
#8 15.21
#8 15.21 * Try:
#8 15.21 > Learn more about toolchain auto-detection at https://docs.gradle.org/8.4/userguide/toolchains.html#sec:auto_detection.
#8 15.21 > Learn more about toolchain repositories at https://docs.gradle.org/8.4/userguide/toolchains.html#sub:download_repositories.
#8 15.21 > Run with --stacktrace option to get the stack trace.
#8 15.21 > Run with --info or --debug option to get more log output.
#8 15.21 > Run with --scan to get full insights.
#8 15.21 > Get more help at https://help.gradle.org.
#8 15.21
#8 15.21 BUILD FAILED in 14s
#8 ERROR: process "/bin/sh -c chmod u+x ./gradlew && ./gradlew installDist" did not complete successfully: exit code: 1

Anscheinend fehlt die toolchain, bzw. das repo dafür.

Right. Seems that there has been a change in Gradle 8. Before 8, Gradle automatically downloaded a Java version matching the configured toolchain in case the locally installed version didn't match. Since 8, a download repository must be configured which we haven't done so far.
Let's first decide whether we move the source code to Java 17. Java 21 is not supported by Gradle at the moment, so I would skip this option.

I would first keep Java 11 at the moment. As soon as the Gradle supports java 21, we should test if we can get significant performance benefit from the Java Virtual Threads. If yes, Switching to java 21 is nice.

Switching to java 17 now is also not bad.

moved to Java 17 with cca0f64821887fc916940468ef61cfcb4d4b1408


[Cattle] Add 'inspect' action to container to get the full docker inspect output

Hi! Will this feature add to the rancher?
I need this one because there is no way to show containers environment variables in rancher UI.
Maybe is there feature-request fot adding new tab with environment variables in container overview?

@QIvan - We are working on showing all container information in the container overview.  Should be seeing that soon.

@deniseschannon @tfiduccia
Hi again! Did you close this issue because it's implemented? Where can I see inspect of a container (I didn't find this info in the release notes)?

[Almost] all the fields are shown in the UI in 1.2.x:


Notification service should watch for Network Fee changes

Expected Behavior
The notification service keeps track of InflationFactorUpdated events emitted from the StableToken smart contract so app users can see a Network Fee in their feeds.
Current Behavior
No tracking of Network Fee

Setting this on the back burner for now cc @nityas

Do you mean blockchain api instead of notification service?
https://github.com/celo-org/celo-monorepo/blob/master/packages/blockchain-api/src/blockscout.ts#L176

Yes, updated thanks!


[Help] Mutable Struct Example

I understand that you can't pass mutable values as arguments, but there aren't any example of modifying structs.
If a simple one could be provided, it would be appreciated.
Thank you

If you modify struct in rust, the value in dart will not be changed.
But you are free to declare as mut, say, pub fn hello(mut a: MyStruct){}

So if the class/struct from dart is passed into the above function, it would move the struct but the dart object will remain. Correct?

yes

Thanks for the clarification.

You are welcome


[BUG] error for saving final .pth model file

Describe the bug
Models are not saving because (see error below).
To Reproduce
It happens with model training process at the saving part, changing settings doesn't matter.
Expected behavior
To find the final .pth files in the weights directory.
Assets
here is the error from the console:
Saved model 'C:\...\Applio-RVC-Fork\logs\pop_smoke\G_2333333.pth' (epoch 100)
Saved model 'C:\...\GitHub\Applio-RVC-Fork\logs\pop_smoke\D_2333333.pth' (epoch 100)
Process Process-1:
Traceback (most recent call last):
  File "C:\...\Applio-RVC-Fork\env\lib\multiprocessing\process.py", line 315, in _bootstrap
    self.run()
  File "C:\...\Applio-RVC-Fork\env\lib\multiprocessing\process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "C:\...\Applio-RVC-Fork\rvc\train\train.py", line 261, in run
    train_and_evaluate(
  File "C:\...\Applio-RVC-Fork\rvc\train\train.py", line 562, in train_and_evaluate
    ckpt,
UnboundLocalError: local variable 'ckpt' referenced before assignment
Index training finished!

Desktop:

OS: Windows 11
Browser: Librewolf (Firefox)

Additional context
My previous installation broke and did a complete reinstallation to the newest version (commit 35f15830b4a7ff00cc541e6c1ca862b0cda0f16b).
Reinstalling the newest version doesn't work either.
If there are any further questions, I'll do my best to answer them.

Fixed, you can get it cloning the repo or waiting to the next release!

I think this is the same bug https://github.com/IAHispano/Applio/issues/293


Wired results from IMAP

Hello,
first, thank you for your great work on the package!
Now, my question ;)
After update from v1 to v2.4.4 I have wired problems when getting emails from IMAP,
I'm using query like ~ folder->query->since(now->subMin(10))->get() and the details of emails is random, I mean for example - I had one email content on few others emails,
I've changed config - and remove attachments: 'dispositions' => ['inline'] instead of 'dispositions' => ['attachments', 'inline'] but still
when fetching a lot of emails and for example just dump uid & hasXXXBody & hasAttachemnts
results are random, the same UID sometimes has true sometimes has false ..
For now, I changed my code, and fetch emails each minute - because if there are fewer emails in the package there were no problems.
If I try to get one email byUid - then always has correct content/attachments etc...
Maybe you can help me or give some advice on how to fix this issue?
(I use your package for more than a year probably :) and works perfectly 👍  )

Hi @PrzemekTkaczyk ,
thanks for your report.
Never think you've seen it all - well this one is new to me. I try to make sense of it but I can't imagine a scenario which might cause this.
However to get some clues:

Please provide the entire used config
Does updating to v2.5.0 solve this?
Can you disclose the provider you are using? Is it a Microsoft product?

This should not work:
$folder->query()->since(now()->subMin(10))->get()

Please try to use:
$folder->query()->since(now()->subDays(1))->get();
// ..or
$folder->query()->unseen()->get();

Here is an other alternative:
$now = now();

$folder->query()->since(now()->subDays(1))->setFetchBody(false)->get()->each(function($message) use($now){
    if ($message->date->toDate()->diffInMinutes($now) <= 5) {
        $message->parseBody();
        var_dump($message);
    }
}

Checkout https://www.php-imap.com/api/query for additional methods such as Query::chunked().
Best regards,

Thank you for your advices
I update to 2.5
and change to chunked query
I will check how it will works :)


Bug 1912565: update moby/term dependency

Description of the change:
Updates the indirect moby/term dependency to a newer version due to the following error on macOS only when running commands like make run-local and make unit
vendor/github.com/docker/docker/pkg/term/tc.go:13:28: undefined: unix.SYS_IOCTL
vendor/github.com/docker/docker/pkg/term/tc.go:18:28: undefined: unix.SYS_IOCTL
vendor/github.com/docker/docker/pkg/term/termios_bsd.go:24:31: undefined: unix.SYS_IOCTL
vendor/github.com/docker/docker/pkg/term/termios_bsd.go:37:31: undefined: unix.SYS_IOCTL
# github.com/moby/term
vendor/github.com/moby/term/tc.go:13:28: undefined: unix.SYS_IOCTL
vendor/github.com/moby/term/tc.go:18:28: undefined: unix.SYS_IOCTL
vendor/github.com/moby/term/termios_bsd.go:24:31: undefined: unix.SYS_IOCTL
vendor/github.com/moby/term/termios_bsd.go:37:31: undefined: unix.SYS_IOCTL

The docker term library attempts to make syscalls directly, which is not supported in recent versions of macOS.
See https://github.com/ory/dockertest/issues/212 for more info
Note: you can update a transitive dependency without using replaces so long as the version is compatible via the go tool.

As part of maintaining the require statements in go.mod, the go command tracks which ones provide packages imported directly by the current module and which ones provide packages only used indirectly by other module dependencies. Requirements needed only for indirect uses are marked with a "// indirect" comment in the go.mod file. Indirect requirements are automatically removed from the go.mod file once they are implied by other direct requirements. Indirect requirements only arise when using modules that fail to state some of their own dependencies or when explicitly upgrading a module's dependencies ahead of its own stated requirements.

Motivation for the change:
Fixes broken local testing on macOS
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /docs
[ ] Commit messages sensible and descriptive

rebased off 1.20 bump

/approve

/hold
Can we pin github.com/moby/term instead?

/hold cancel
/lgtm

/bugzilla refresh

/bugzilla refresh


Add CI/CD and PDM to new indexer worker

Description

Patch for @stacimc's base branch to use PDM and add indexer worker tests to CI
Testing Instructions

./ov just indexer_worker/test should run the tests inside the container. CI/CD additions need to pass in the base branch.
Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[N/A] I ran the DAG documentation generator (just catalog/generate-docs for catalog
PRs) or the media properties generator (just catalog/generate-docs media-props
for the catalog or just api/generate-docs for the API) where applicable.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@stacimc If you'd like this merged into your branch, please go ahead (it doesn't need any further reviews). If you'd prefer it to stay isolated and as a fast-follow, that's fine, too, just let me know so I know which to expect :slightly_smiling_face:


lsyncd: 2.2.3 -> 2.3.1

Description of changes
Version bump.
The patch (https://github.com/lsyncd/lsyncd/commit/0af99d8d5ba35118e8799684a2d4a8ea4b0c6957.patch) failed to apply, because it is now available in lsyncd already (https://github.com/lsyncd/lsyncd/commit/0af99d8d5ba35118e8799684a2d4a8ea4b0c6957).
Things done

Built on platform(s)

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

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

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

[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 211939 run on x86_64-linux 1

1 package built:

lsyncd


Add more targets for save-agent

Currently, save-agent is built only for linuxX64.
We can add more targets like jvm. It should simplify debugging and local running

Let's maybe narrow it down to only JVM, because previously we tried to build agent for mingX64 but with no real benefits. JVM can be good though.

JVM can be good though.

Fully agree, Also we can add tests on JVM.
Currently we have a single test for linuxX64 which expects windows as launch system. Looks like it doesn't work


pcl build failure

Host Environment

OS: Windows 11
Compiler: Visual Studio 2022, Microsoft (R) C/C++ Optimizing Compiler Version 19.29.30133 for x64
executed in "x64 Native Tools Command Prompt for VS 2019" as administrator

To Reproduce
Steps to reproduce the behavior:
./vcpkg install pcl
Failure logs
...
Starting package 95/95: pcl:x64-windows
Building package pcl[core]:x64-windows...
-- Using cached PointCloudLibrary-pcl-f36a69a5e89953708990c4669317f989d532cf08.tar.gz.
-- Extracting source C:/data/vcpkg/downloads/PointCloudLibrary-pcl-f36a69a5e89953708990c4669317f989d532cf08.tar.gz
-- Applying patch add-gcc-version-check.patch
-- Applying patch fix-check-sse.patch
-- Applying patch fix-find-qhull.patch
-- Applying patch fix-numeric-literals-flag.patch
-- Applying patch pcl_config.patch
-- Applying patch pcl_utils.patch
-- Applying patch remove-broken-targets.patch
-- Applying patch fix-cmake_find_library_suffixes.patch
-- Using source at C:/data/vcpkg/buildtrees/pcl/src/89d532cf08-fd49a85837.clean
-- Found external ninja('1.10.2').
-- Configuring x64-windows
CMake Warning at installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_configure.cmake:485 (message):
  The following variables are not used in CMakeLists.txt:

      PCL_BUILD_WITH_FLANN_DYNAMIC_LINKING_WIN32

  Please recheck them and remove the unnecessary options from the
  `vcpkg_cmake_configure` call.

  If these options should still be passed for whatever reason, please use the
  `MAYBE_UNUSED_VARIABLES` argument.
Call Stack (most recent call first):

  ports/pcl/portfile.cmake:42 (vcpkg_cmake_configure)
  scripts/ports.cmake:142 (include)

-- Building x64-windows-dbg
-- Building x64-windows-rel
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:155 (message):
    Command failed: C:/data/cmake/bin/cmake.exe --build . --config Release --target install -- -v -j9
    Working Directory: C:/data/vcpkg/buildtrees/pcl/x64-windows-rel
    See logs for more information:
      C:\data\vcpkg\buildtrees\pcl\install-x64-windows-rel-out.log

Call Stack (most recent call first):
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake:102 (vcpkg_execute_build_process)
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake:41 (vcpkg_cmake_build)
  ports/pcl/portfile.cmake:60 (vcpkg_cmake_install)
  scripts/ports.cmake:142 (include)

Error: Building package pcl:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `git pull` and `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  package: pcl[core]:x64-windows -> 1.12.0
  vcpkg version: a4b5cde7f 2021-11-16 (4 hours ago)
  vcpkg-tool version: 2021-11-15-ec76affcc987efbf172b7da0b05df711b48fa535

Please also see attached log file.
install-x64-windows-rel-out.log

Duplicate of #21327 ,we track in pr #21327 ,Temporarily close this pr


Interconnect Issue + NaN Error

When I run python train.py (attached here) it does not give a Y next to the GPU when Interconnect Matrix is checked. Following which it runs the CPU version, and NaN error is attained after a few steps. i have not seen any such issue when training on NVIDIA Tesla K40c (attached as sm25epc.log) or Tesla T4 (attached as nohup.txt
nohup.txt
sm25epc.log
train.txt
)
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 20.04.2 LTS

TensorFlow installed from (source or binary): Anaconda (GPU)

TensorFlow version: latest version (I'm guessing 2.5 above) I installed using conda create -n tf-gpu or something similar last week

Python version: 3.9.7

Installed using virtualenv? pip? conda?: Anaconda (2021.05 release)

Bazel version (if compiling from source):

GCC/Compiler version (if compiling from source):

CUDA/cuDNN version: NVIDIA-SMI 450.142.00   Driver Version: 450.142.00   CUDA Version: 11.0

GPU model and memory: A100-SXM4-40GB

Describe the problem
Provide the exact sequence of commands / steps that you executed before running into the problem
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

job.21631.txt
Error file attached here

This is submitted in a GPU cluster (there's a SLURM script and all)
I find that when submitting to a local A100 it does not give a N error but it still has a CUDA initialisation error and reverts to Tensorflow CPU Libary (There I use CUDA 11.5) Remaining is the same

@AnishKumarNayak,
Can you confirm if you're trying the code in latest stable version i.e 2.7.0? Also from the template, I can see that you mentioned the CUDA version as 11.0 whereas as per tested build config for 2.7.0, cuda version 11.2 is used. You can refer this link and make sure you have corresponding versions of compiler, CUDA and cuDNN are used. Thanks!

Hi,
It should be 2.7.0 as I have installed the default latest version available in Anaconda3 just a week back.
My bad, I was a little careless. I am using a GPU cluster at the moment, and we submit our job using SLURM script. I have used source to call upon CUDA 11.0.2 and CUDA 11.2 on separate occasions with the same issue.
As far as compilers are concerned, I am not familiar with that as we did not build Tensorflow from source, but rather installed using Anaconda3 as a virtual environment. CUDA and cuDNN have been confirmed to be compatible with each other.

@AnishKumarNayak,
I took a look at your code in train.txt and I am not able to get the core library that you've used in import section. Can you share a standalone reproducible code or a colab gist to expedite the trouble-shooting process? Thanks!


DS-3549: Encode bitstreams with UTF-8 in XMLUI

When viewing a bitstream in the browser the file is encoded as ISO-8859 instead of UTF-8. This change forces the UTF-8 encoding.
https://jira.duraspace.org/browse/DS-3549
See also #1693

This obviously breaks all ISO-8859 content. Some browsers (Chrome) will get the right encoding from the data if no encoding is transmitted in the response header:

Without this patch both UTF-8 and ISO-8859 are displayed correct in Chrome, but only ISO-8859 in Firefox.
With this patch only UTF-8 works in Chrome and Firefox.

Additionally I do not think, that application/pdf;charset=UTF-8 is a valid Content-Type.

This PR seems to need further discussion, and the travis report isn't a pass yet. Shifting to 6.4 until we can resolve those issues (if we get some action in the next week I'll move back to 6.3)


Separate EventDataProvider into different components

Describe the feature you'd like to request
Now that we have a general ProviderView, we should make different forms of data providers separate and explicit.
Describe the solution you'd like
Currently EventDataProvider will switch based on (entityType, chartType)
We can leave chartType, but I think we should expose separate
ArtifactEventDataProvider and ProjectEventDataProvider
Describe alternatives you've considered
Leave as is

Gonna deprioritize this for now

turns out this is important

Done here


Enhance KV logs.

What does this PR do?
Give more context when an error occurs.
Motivation
Enhance KV logs.
More

[ ] ~Added/updated tests~
[ ] ~Added/updated documentation~

I would say this is not only an enhancement, but also a bug fix (Line 804)

The change to L804 is related to a L642 and it was not needed before this change.

The change to L804 is related to the change L642, it was not needed before.


Add BRepFill_Filling based constrained surface creation

Currently the surface construction methods in build123d do not provide great control over adjacent face continuity requirements. I propose to add BRepFill_Filling as a build123d direct API Face method.
Further reading: https://neweopencascade.wordpress.com/2019/03/15/build-constrained-surfaces-with-brepoffsetapi_makefilling-part-i/
I was able to create a rough proof of concept, and it does function as expected:

Working prototype:
from typing import Iterable, Union, Tuple
from enum import Enum, auto
from OCP.BRepFill import BRepFill_Filling
from OCP.GeomAbs import *  # import specific

# put in build_enums.py
class Continuity(Enum):
    """geometric continuity"""

    C0 = auto()
    G1 = auto()
    G2 = auto()

    def __repr__(self):
        return f"<{self.__class__.__name__}.{self.name}>"

# put in topology.py ?
_continuity_dict = {
    Continuity.C0: GeomAbs_C0,
    Continuity.G1: GeomAbs_G1,
    Continuity.G2: GeomAbs_G2,
}

# put in topology.py as a classmethod of Face
@classmethod
def make_surface_patch(
    cls,
    edge_face_constraint: Iterable[
        Union[
            Tuple[Edge, Face, Continuity],
            Tuple[Edge, Continuity],
        ]
    ],
) -> Face:
    # input validation
    for efc in edge_face_constraint:
        if not isinstance(efc[0], Edge):
            raise ValueError(f"Provided {type(efc[0])} is not an Edge or Wire")

    patch = BRepFill_Filling()
    for efc in edge_face_constraint:
        if isinstance(efc[1], Continuity) and len(efc) == 2:
            patch.Add(efc[0].wrapped, _continuity_dict[efc[1]])
        else:
            patch.Add(efc[0].wrapped, efc[1].wrapped, _continuity_dict[efc[2]])
    patch.Build()
    result = patch.Face()

    return cls(result)

This looks good to me.  At some point we're going to need some good surface modelling documentation with all of these different methods to choose from.

@gumyr a few more question for you -- do you think the "UI" makes sense? The tuples supply the e.g. edge/face/continuity pairing. Is this consistent with other parts of build123d? I wanted it to be very explicit which parts "go together". I also decided against any kind of topology exploration (to automatically pair edges/faces) to avoid complexity and not to mention that it might not even be possible.
There are a few other things I have noticed:

According to the OCCT docs it is possible to add a punctual constraint via a gp_Pnt. I currently think it is probably worth adding support for this. This does not require a constraint.
As far as I can tell (and I could be very wrong) I do not see the point for the user to be able to supply the continuity for unsupported edges, they seem to only use C0 continuity.

Would you mind sharing the code you used to create the heart value example?  It would be easier to discuss the API by looking at a real example.
One thing that might help is to rename the Continuity Enum to terms more commonly used but this is easier said than done.

IMO the ability to specify continuity is very useful.

@gumyr Here is a version of the example that you referenced (combine with the earlier prototype code):
r = 30
angle = 40
offs = 100

plnA = Plane.XY
with BuildSketch() as s:
    Circle(r)
    split(bisect_by=Plane.XZ)

plnB = Plane.XY.rotated((0, -angle, 0)).offset(offs + 20)
with BuildSketch(plnB) as s2:
    Circle(r)
    split(bisect_by=Plane.XZ)

plnC = Plane.XY.rotated((0, angle + 20, 0)).offset(offs - 10)
with BuildSketch(plnC) as s3:
    Circle(r)
    split(bisect_by=Plane.XZ)

selA = s.sketch.edges().sort_by(Axis.Y)[0]
selB = s2.sketch.edges().sort_by(Axis.Y)[0]
selC = s3.sketch.edges().sort_by(Axis.Y)[0]

with BuildLine(Plane.XZ) as l:
    m1 = Spline([selA @ 0, selB @ 0], tangents=[plnA.z_dir, plnB.z_dir])
    m2 = Spline([selA @ 1, selC @ 1], tangents=[plnA.z_dir, plnC.z_dir])
    m3 = Spline([selB @ 1, selC @ 0], tangents=[-plnB.z_dir, plnC.z_dir])

# create construction geometry (support faces)
ex1 = Edge.extrude(m1.edge(), (0, 10, 0))
ex2 = Edge.extrude(m2.edge(), (0, 10, 0))
ex3 = Edge.extrude(m3.edge(), (0, 10, 0))

exA = Edge.extrude(selA, -10 * plnA.z_dir)
exB = Edge.extrude(selB, 10 * plnB.z_dir)
exC = Edge.extrude(selC, 10 * plnC.z_dir)

# position constraints on surface:
n1 = Spline([selA @ 0.5, selB @ 0.5], tangents=[(0, 0, 1), plnB.z_dir])
n2 = Spline([selA @ 0.5, selC @ 0.5], tangents=[(0, 0, 1), plnC.z_dir])
n3 = Spline(
    [selB @ 0.5, selC @ 0.5],
    tangents=[-plnB.z_dir, plnC.z_dir],
    tangent_scalars=[1.1, 1.1],
)

top_face = Face.make_surface_patch(
    edge_face_constraint=[
        (m1.edge(), ex1, Continuity.G1),
        (m2.edge(), ex2, Continuity.G1),
        (m3.edge(), ex3, Continuity.G1),
        (selA.edge(), exA, Continuity.G1),
        (selB.edge(), exB, Continuity.G1),
        (selC.edge(), exC, Continuity.G1),
        (n1.edge(), Continuity.C0),
        (n2.edge(), Continuity.C0),
        (n3.edge(), Continuity.C0),
    ]
)

@MatthiasJ1 as you can see I AM specifying continuity above, the question is if "unsupported edges" need a continuity mode input because they only seem to work with C0 continuity.

What about adding a couple methods to make the interface a little more user friendly?
top_face = Face.make_surface_patch(
    edge_face_constraint=[
        m1.tangent_continuous(ex1),
        m2.curature_continuous(ex2),
        ...
        n1,
        n2,
        n3,
    ]
)

The m1.tangent_continuous(ex1) method could return a tuple of (m1.edge(), ex1, Continuity.G1) so the make_surface_patch API wouldn't actually change. The C0 constraint doesn't actually provide any more information so I don't think it's required - am I missing something here?

@gumyr I agree that the C0 constraint is not providing anything, so that can be default behavior without the user needing to specify. Regarding the helper functions, I am not sure they are adding enough value to be worth it -- so I prefer to not add them for now.
Here is the new proposed method signature:
# put in topology.py as a classmethod of Face
@classmethod
def make_surface_patch(
    cls,
    edge_face_constraint: Iterable[
        Union[
            Tuple[Edge, Face, Continuity],
            Edge,
            VectorLike,
        ]
    ],
) -> Face:

Does VectorLike make sense in this application (will be converted to gp_Pnt)

Looks fine to me. This is an advanced feature so I expect users will have more trouble with defining their surface than figuring out the syntax.  VectorLike is more consistent than Vector across all the build123d APIs.  Tuple isn't really required, tuple is the recommendation IIRC.


Thread varargs through rust-analyzer

This adds a varargs flag to various data structures and fills it from the AST.
Fixes https://github.com/rust-analyzer/rust-analyzer/issues/5374
cc @flodiebold for the typesystem/chalk changes

bors r=flodiebold


Plans on porting to C/C++ ?

Any plans on porting this to C/C++ so that other language bindings (I.E. SWIG) can be made available?

The NFP algo would be faster in C, but I'm not sure how much. It's essentially a massive amount of floating point operations, which I assume would optimize pretty well in V8.
Given that my C is pretty rusty, I'm going to say no for now. Might be a project for someone with better knowledge of C perhaps.

Speed isn't my main concern in this case because I understand that JS is a beast in its own way (Your demo impressed me, and quite a few others obviously). This should be ported to a library. Leave this issue open perhaps for others to collaborate, express interest and/or make it happen.

I agree its very cool. Rather than C++ though, I might attempt a python port using numpy for the matrix work.

+1 for Python port. Would be willing to help on that.

i agree a python or c++ port would be great

:+1: for CUDA/OpenCL port :smile:

golang anyone? : ]
The only thing unique to this implementation is the No Fit Polygon algo. In C++/Python/Java etc there are much better, mature libraries for path clipping/svg parsing/random geometric functions. I think if someone wanted to write a *lang version of this they'd be better off reading the papers linked in the readme and using the NFP algo in geometryutils.js as inspiration rather than a straight port.

Thanks for the pointers Jack.
So I've started a related github project at https://github.com/Neon22/inkscape-nest
My idea being to integrate it into inkscape as a python extension (plugin) for use in a dedicated SVG environment. I've got a couple of oteh rplugins in the top level rope that I'v ewritten before.
If anyone wants to help out - lets discuss in the Issues over there.

@Neon22 Great!

Looking forward to it!

@servant74, @crushedice2000, @antimodular, @rmelly - care to come over and join me ?

+1 for some sort of command-line implementation.

i stopped trying to port the javascript and implement the approach from bottom up following the papers. i started with the implementation of no-fit polygons: https://github.com/kallaballa/libnfp

@kallaballa   , I'm interested in the translation of svgnest with C++

Recently, I implemented multi threaded rectangular layout in the C++environment, but there are several defects on the motherboard. It can be arranged once in about 100 seconds
C++wrote the core layout section, and used Python for the genetic algorithm section
I am planning to write a C++version of polygon layout, preferably to support the presence of voids in the board and the presence of voids on the motherboard. I have a very clever idea of dropping parts into the motherboard from above at a specific angle and position

Have their been any developments in this space? I have not had any success with the CLI forks for deepnest.


Can't Install

Im getting this error :(

Hi Kevin
Do you have the mysql and mcrypt modules installed for PHP?
What operating system are you on?

Windows, i uploaded it on my hosting server and this error appear.

Then the issue would be related to missing php modules.
If you are sure the modules are installed, let me know and i can see where we can go from there

Closing this ticket. Reopen it if you still have an issue


Italian: adapt character disposition to letter frequency

Actually the Italian keyboard disposition adds the accented chars to the standard ones, but does not take account of the different frequency of the letters in italian texts.
To facilitate a fast typing, if a letter is more frequent IMO it should be obtained with a shorter move: so it should appear in a inner position in the branches.
The Y letter is first (nearer) in the top-left branch, but it does not appear at all in the Italian alphabet (except in foreign words of course). On the other side, letter R is the 7th most used letter but it appears as the most distant in the branch.
eg see there
Wikipedia
So I suggest to swap Y and R position in the Italian keyboard map.
Other possible useful swaps according with letter frequencies in Italian are Q - X (X is another non-Italian letter) and S - L
I have tried these swaps with the "load local map" function and I find them useful.

Would you be willing to do the research and come up with a better layout
for italian language? we can make the change and would be able to defend
the layout if it's backed by data.
On Tue, 8 Feb 2022, 23:11 flauta, @.***> wrote:

Actually the Italian keyboard disposition adds the accented chars to the
standard ones, but does not take account of the different frequency of the
letters in italian texts.
To facilitate a fast typing, if a letter is more frequent IMO it should be
obtained with a shorter move: so it should appear in a inner position in
the branches.
The Y letter is first (nearer) in the top-left branch, but it does not
appear at all in the Italian alphabet (except in foreign words of
course). On the other side, letter R is the 7th most used letter but it
appears as the most distant in the branch.
eg see there https://www.sttmedia.com/characterfrequency-italian
Wikipedia https://it.wikipedia.org/wiki/Analisi_delle_frequenze
So I suggest to swap Y and R position in the Italian keyboard map.
Other possible useful swaps according with letter frequencies in Italian
are Q - X (X is another non-Italian letter) and S - L
I have tried these swaps with the "load local map" function and I find
them useful.
—
Reply to this email directly, view it on GitHub
https://github.com/flide/8VIM/issues/252, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABX5WR6BM72OKVQBS2SLPKTU2FIUFANCNFSM5N3E2BVQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

More frequent letters are important, however also frequent sequences of letters are important.
I remember that @Glitchy-Tozier developed some software to optimize the keyboard layout taking those in consideration as well.
If I'm not mistaken the mechanism of this software was explained in #90 you might take a look at that to see if the generated layout is better.

If you want, I can see what I get when optimizing for the Italian language!
The main problem with my script is that it ignores diacritics. It'll most likely still provide you with way better results compared to the default layout though.
You can find a post regarding my script here:   https://github.com/flide/8VIM/discussions/138#discussion-64700

Great, diacritics should not matter as those are added always in the same way, thus you simply have to assign the most frequent to the easiest combination.
The problem now would be finding (or generating) the most common Italian bigrams.

Right, this is a problem. The website I used for other langues doesn't seem to feature italian bigrams, so I'm not sure where to find them.

Here is one italian digrams table (the only I found):
http://www.crittologia.eu/critto/php/frequenze_bigrammi_lingua.phtml
as for the diacritics: the actual solution ("à" beginning on the same side/place of "a" and so on) is fine for me.

Okay, so with some formatting, the list looks like this. Would you guys agree that this is representative of Italian?

The words and text source was the PAISA repertory http://www.corpusitaliano.it/
As you can see the results are similar.

Sorry the N-grams list from PAISA is this
lista_paisa_ngrams.txt
(the other one was done on a smaller sample)

Okay, so when doing a really quick search, this is the result:
Tomorrow I'll improve the parameters and do a more thorough search.
 I'm going to generate a new XML and start immediately to esperiment your new disposition.
I want to put in another consideration: In the italian language the letter Q is always followed by U. It is possible to add a 'diacritic' gesture to get this combination in a single move? The same could be for the combination CH (that is nearly frequent as C + vowel).

Thanks for your work. I'm going to generate a new XML and start immediately to esperiment your new disposition.

If you wait one more hour, I can give you my final result. :)

I want to put in another consideration: In the italian language the letter Q is always followed by U. It is possible to add a 'diacritic' gesture to get this combination in a single move? The same could be for the combination CH (that is nearly frequent as C + vowel).

I have no idea, unfortunately.

I noticed that in your script output the order of the letters in the result isn't the same of the XML config file.
eotlnrai dcsgumpv whqzfybk ---x--j- (first round, second round etc)
in the XML must be converted to
aipvbkj- eodcwh-- tlsgqz-x nrumfy-- (lower right arm, lower left arm etc)
Could you adapt the output of the script, so that the result can be directly inserted in the XML file?

Should/can the spaces be included?

It's not important, since the string must be completed with the missing symbol chars. Let's say that your output is a reference.
But no, in the XML file there are no spaces between the groups. And the string must be duplicated for the capital letters.
See above an example
https://github.com/flide/8VIM/issues/252#issuecomment-1035439896

There you go.

On Sun, 20 Feb 2022, 18:42 Glitchy-Tozier, @.***> wrote:

There you go.

You are receiving this because you commented.Message ID:
@.***>

I don't think this makes sense at the moment.

My script doesn't include special characters (.,!?@')
There already is a script that does just that: https://github.com/sslater11/8vim_keyboard_layout_file_generator

Thanks for your work! I'm working to the new XML file and I will try it ASAP.

it_regular_italian_NEW.xml.zip
This is the new Italian XML file. Besides the remapping of the keys (according to layout #1 https://github.com/flide/8VIM/issues/252#issuecomment-1046235243 ) I deleted the accented letters that don't exist in Italian and reordered the keys to follow strictly the order of the branches.
@flide I propose you to push this to the source code, if you think it's the right time. Please note: I tested it only via the "Load local file" function, since I'm not able to recompile the project.

I find this much better than the previous layout to write in Italian.
It also works decently for English.


Cannot install local wheel

Pipenv fails to install local wheel requirements
$ python -m pipenv.help output
Pipenv version: '2018.6.25'
Pipenv location: '/home/kieran/.local/lib/python3.6/site-packages/pipenv'
Python location: '/usr/bin/python3'
Other Python installations in PATH:

2.7: /usr/bin/python2.7

2.7: /usr/bin/python2.7

3.6: /usr/bin/python3.6m

3.6: /usr/bin/python3.6

2.7.14: /usr/bin/python

2.7.14: /usr/bin/python2

3.6.3: /usr/bin/python3

PEP 508 Information:
{'implementation_name': 'cpython',
'implementation_version': '3.6.3',
'os_name': 'posix',
'platform_machine': 'x86_64',
'platform_python_implementation': 'CPython',
'platform_release': '4.13.0-45-generic',
'platform_system': 'Linux',
'platform_version': '#50-Ubuntu SMP Wed May 30 08:23:18 UTC 2018',
'python_full_version': '3.6.3',
'python_version': '3.6',
'sys_platform': 'linux'}
System environment variables:

CLUTTER_IM_MODULE
LC_ALL
LS_COLORS
LESSCLOSE
XDG_MENU_PREFIX
LANG
GDM_LANG
MANAGERPID
DISPLAY
INVOCATION_ID
UNITY_DEFAULT_PROFILE
COMPIZ_CONFIG_PROFILE
GTK2_MODULES
COLORTERM
SSH_AUTH_SOCK
MANDATORY_PATH
XDG_GREETER_DATA_DIR
USER
DESKTOP_SESSION
QT4_IM_MODULE
TEXTDOMAINDIR
DEFAULTS_PATH
NIX_PATH
PWD
HOME
JOURNAL_STREAM
TEXTDOMAIN
QT_ACCESSIBILITY
XDG_SESSION_TYPE
COMPIZ_BIN_PATH
XDG_DATA_DIRS
XDG_SESSION_DESKTOP
SSH_AGENT_LAUNCHER
GTK_MODULES
GNOME_SESSION_XDG_SESSION_PATH
SHELL
TERM
VTE_VERSION
XDG_SEAT_PATH
QT_IM_MODULE
XMODIFIERS
IM_CONFIG_PHASE
XDG_CURRENT_DESKTOP
UNITY_HAS_3D_SUPPORT
NIX_SSL_CERT_FILE
SHLVL
LANGUAGE
WINDOWID
GDMSESSION
GNOME_DESKTOP_SESSION_ID
LOGNAME
DBUS_SESSION_BUS_ADDRESS
XDG_RUNTIME_DIR
XAUTHORITY
XDG_SESSION_PATH
XDG_CONFIG_DIRS
PATH
SESSION_MANAGER
LESSOPEN
GTK_IM_MODULE
OLDPWD
_
PYTHONDONTWRITEBYTECODE
PIP_PYTHON_PATH

Pipenv–specific environment variables:
Debug–specific environment variables:

PATH: /home/kieran/.nix-profile/bin:/home/kieran/.nix-profile/sbin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin
SHELL: /bin/bash
LANG: en_US.UTF-8
PWD: /home/kieran/Git/pyUmbral

Contents of Pipfile ('/home/kieran/Git/pyUmbral/Pipfile'):
[[source]]
url = "https://pypi.python.org/simple"
verify_ssl = true
name = "pypi"

[packages]
# Wheels for cryptography until the next release.
cryptography35_linux = { file="./wheelhouse/cryptography-2.3.dev1-cp35-cp35m-linux_x86_64.whl", markers="python_version >= '3.5' and python_version < '3.6' and platform_system == 'Linux'"}
cryptography36_linux = { file="./wheelhouse/cryptography-2.3.dev1-cp36-cp36m-linux_x86_64.whl", markers="python_version >= '3.6' and python_version < '3.7' and platform_system == 'Linux'"}
cryptography36_darwin = { file="./wheelhouse/cryptography-2.3.dev1-cp36-cp36m-macosx_10_13_x86_64.whl", markers="python_version >= '3.6' and python_version < '3.7' and platform_system == 'Darwin'"}

# Use bytestringSplitter from github until we release it on PyPi.
bytestringSplitter = {git = "https://github.com/nucypher/bytestringSplitter.git", ref = "nucypher-depend"}

pynacl = "*"

# Since wheel-specified dependencies aren't locked...
idna = ">=2.1"
asn1crypto = ">=0.21.0"
six = ">=1.4.1"
cffi = ">=1.7"

[dev-packages]
pytest = "*"
coverage = "*"
pytest-cov = "*"
pytest-mock = "*"
codecov = "*"
sphinx = "*"
sphinx-autobuild = "*"
</summary>
</details>

##### Expected result

Expected a typical installation of specified requirements from local wheel files.

##### Actual result

Here is the full traceback

Traceback (most recent call last):
File "/usr/local/bin/pipenv", line 11, in
sys.exit(cli())
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 722, in call
return self.main(*args, **kwargs)
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 697, in main
rv = self.invoke(ctx)
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 1066, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 895, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/cli.py", line 416, in install
selective_upgrade=selective_upgrade,
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/core.py", line 1972, in do_install
pypi_mirror=pypi_mirror,
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/core.py", line 1346, in do_init
pypi_mirror=pypi_mirror,
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/core.py", line 1070, in do_lock
settings['packages'], project, r=False, include_index=True
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/utils.py", line 514, in convert_deps_to_pip
new_dep = Requirement.from_pipfile(dep_name, dep)
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 754, in from_pipfile
r = FileRequirement.from_pipfile(name, pipfile)
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 384, in from_pipfile
return cls(**arg_dict)
File "", line 20, in init
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 281, in get_requirement
req = first(requirements.parse(line))
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/vendor/first.py", line 70, in first
for el in iterable:
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/vendor/requirements/parser.py", line 50, in parse
yield Requirement.parse(line)
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/vendor/requirements/requirement.py", line 220, in parse
return cls.parse_line(line)
File "/home/kieran/.local/lib/python3.6/site-packages/pipenv/vendor/requirements/requirement.py", line 198, in parse_line
pkg_req = Req.parse(line)
File "/home/kieran/.local/lib/python3.6/site-packages/pkg_resources/init.py", line 2988, in parse
req, = parse_requirements(s)
File "/home/kieran/.local/lib/python3.6/site-packages/pkg_resources/init.py", line 2935, in parse_requirements
yield Requirement(line)
File "/home/kieran/.local/lib/python3.6/site-packages/pkg_resources/init.py", line 2944, in init
raise RequirementParseError(str(e))
pkg_resources.RequirementParseError: Invalid requirement, parse error at "'./wheelh'"

##### Steps to replicate

0. Install pipenv 2018.6.25
1. Create a pipfile that references a local wheel file as a requirement
2. run `pipenv install`

hi @KPrasch, thanks for using pipenv!  I'm looking into this issue.  Hold tight!

Ok @KPrasch I think I figured out the issue, the syntax to specify a wheel in a pipfile looks like this:
[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[packages]
requests = {path = "./wheelhouse/requests-2.19.1-py2.py3-none-any.whl"}

[dev-packages]

You'll notice that instead if file we have used path.   Try changing it and then doing a pipenv lock!

Ah, okay! Thanks for finding this!  For future reference, where can I find this documented?

I'm honestly not sure, I got the syntax from doing pipenv install ./wheelhouse/requests-2.19.1-py2.py3-none-any.whl and then looking at the generated Pipfile.  If I don't find it in the docs a section may need to be created for it, feel free to make a contribution!

Ok!
For the record, something must've changed with version 2018.6.25, since we were installing .whl files using "file" as the the inline table key with prior versions of pipenv without issues; downgrading pipenv also fixes the issue.
Thanks again for your help!

I'm checking to see if this is a regression, in the meantime I have this explanation:
path is for relative paths to packages or wheels
file is for uris

I can confirm this is not a regression and was added in #540

hey @KPrasch sorry for the confusion, the file key may have worked in the prior release, but if it did it was an accident :( file is meant for URIs only -- the path key is for relative paths.  That's why we have two separate keys
If this was possible in the last release I expect we're going to hear more about it so we're going to have to be prepared for this a bit more in the coming days.  Sorry for the confusion and thanks for filing this issue!

Thanks for the quick responses and all your great work! :-)


CLI prints prompts twice

Describe the bug
NodeJsMongo template prompted to enter location region to store deployment metadata
"Please enter an Azure region to use to store deployment metadata"
To Reproduce

Open a terminal and create new directory named "testnodemg"
run the command az dev up --template todo-nodejs-mongo
enter environment name as tempnodetest, azureregion as eastus, select a subscription, enter azure region to store metadata as eastus

Expected behavior
Expected cli to prompt user to enter only 3 information Environment Name, Azure Location, Azure Subscription
Environment
Information on your environment:
* Nodejs v16.14.0
* azd version 0.0.1-beta.368 (commit 18e5243e994490b797c8e9b29aecbdf9c8a8c98a)
* IDE and version : VS Code

Additional context
Add any other context about the problem here.

With all the recent UX changes. I believe the above issues are fixed. cc: @vhvb1989 @Austinauth
Please re-open if you think otherwise.


Installation issue on raspberry pi

Summary of Problem

What are you trying to do?
installation
What happens?
installation fail
What should have happened?

`sudo npm install serialport --unsafe-perm --build-from-source

<EMAIL_ADDRESS>install /home/osmc/node_modules/@serialport/bindings
prebuild-install --tag-prefix @serialport/bindings@ || node-gyp rebuild

gyp ERR! build error
gyp ERR! stack Error: not found: make
gyp ERR! stack     at getNotFoundError (/home/osmc/.nvm/versions/node/v7.10.1/lib/node_modules/npm/node_modules/which/which.js:13:12)
gyp ERR! stack     at F (/home/osmc/.nvm/versions/node/v7.10.1/lib/node_modules/npm/node_modules/which/which.js:68:19)
gyp ERR! stack     at E (/home/osmc/.nvm/versions/node/v7.10.1/lib/node_modules/npm/node_modules/which/which.js:80:29)
gyp ERR! stack     at /home/osmc/.nvm/versions/node/v7.10.1/lib/node_modules/npm/node_modules/which/which.js:89:16
gyp ERR! stack     at /home/osmc/.nvm/versions/node/v7.10.1/lib/node_modules/npm/node_modules/which/node_modules/isexe/index.js:44:5
gyp ERR! stack     at /home/osmc/.nvm/versions/node/v7.10.1/lib/node_modules/npm/node_modules/which/node_modules/isexe/access.js:8:5
gyp ERR! stack     at FSReqWrap.oncomplete (fs.js:114:15)
gyp ERR! System Linux 4.14.34-4-osmc
gyp ERR! command "/home/osmc/.nvm/versions/node/v7.10.1/bin/node" "/home/osmc/.nvm/versions/node/v7.10.1/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/osmc/node_modules/@serialport/bindings
gyp ERR! node -v v7.10.1
gyp ERR! node-gyp -v v3.5.0
gyp ERR! not ok
npm WARN enoent ENOENT: no such file or directory, open '/home/osmc/package.json'
npm WARN osmc No description
npm WARN osmc No repository field.
npm WARN osmc No README data
npm WARN osmc No license field.
npm ERR! Linux 4.14.34-4-osmc
npm ERR! argv "/home/osmc/.nvm/versions/node/v7.10.1/bin/node" "/home/osmc/.nvm/versions/node/v7.10.1/bin/npm" "install" "serialport" "--unsafe-perm" "--build-from-source"
npm ERR! node v7.10.1
npm ERR! npm  v4.2.0
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: prebuild-install --tag-prefix @serialport/bindings@ || node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'prebuild-install --tag-prefix @serialport/bindings@ || node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the @serialport/bindings package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     prebuild-install --tag-prefix @serialport/bindings@ || node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs @serialport/bindings
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls @serialport/bindings
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /root/.npm/_logs/2018-09-24T15_38_26_055Z-debug.log
`
Versions, Operating System and Hardware

SerialPort@?
latest
Node.js v?
7.10.1
Windows? Linux? Mac?
linux raspberry pi (OSMC)
Hardware and chipset? (Prolific/FTDI/Other)

gyp ERR! stack Error: not found: make

Do you have make installed on your system?

Oh my bad, it's a fresh install and yes missing make and g++
thank you very much


Vector.svg_object isolated + test

Hi,
I just tried to isolate the vector.js file to make it usable on the client side and minimize the code weight.
To use it one the client side :
var qrVector = require('qr-image/lib/vector');
var qrObject = qrVector.svg_object('text');
What do you think ?

Well, I got 38.2ko of minified code, but is it possible to get a lighter client library ?
What do we really need and what is useless to generate a svg object ?
Thanks


Update db dropdown in create project from database dialog to not be editable

This PR fixes #14141. This dropdown shouldn't be editable since an existing database needs to be chosen.

Pull Request Test Coverage Report for Build 532330013

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 43.384%

Totals

Change from base Build 529879093:
0.003%

Covered Lines:
24746

Relevant Lines:
52252

💛  - Coveralls


Spendable income calculation is weirdly wrong

[ ] The spendable income is calculated wrong from round 1 for two of the three players of this testing session that I was running to recreate the bug of the IHE-Delft session:
t2p5 (IHE-Delft): t1p1 (Test) -> U10 (R1: Sand bags)
t2p4 (IHE-Delft): t1p2 (Test) -> N07, I cound not retrieve from the facilitator side where was that player living at the IHE-Delft round 1
t2p6 (IHE-Delft): t1p6 (Test) -> U04 (R1: Green garden and rise level)
[ ] The error was not present when testing the player alone
[ ] The error appears when trying to buy improvements
[ ] At last, the website broke when I advanced the game steps with some of the players not all on the table, upon approval from the facilitator. When I tried to continue with this player, instead of a warning the website broke up

@vjcortesa : Question: did you run the different players at the same time on the same computer in different tabs / screens of the browser? If so, the results of the players can get mixed and overwritten. Only one player should play at a given time on a computer.
If not, I will have to try to reproduce the error.

Dear Alexander, I got this error by playing two players in different browsers (one in chrome, one in explorer) and one player on my phone (Test t1p2) was on my phone and was wrong

Strange. I have not been able to reproduce any error like this. The bottom left player looks as if there was a timeout. When I log in as t1p1 in that session now, I get normal screens:

The wrong calculation at the bottom right is because it is incomplete -- the taxes, mortgage, and bought measures are missing. I will repair that.
The two screens in the top row have a difference of 20k. That is indeed strange, and looks as if the taxes have been deducted twice. I looked in the database, and also there the calculation wold end up being 20 k more positive. Did you do something special with the players in the test session, e.g., advancing the facilitator far ahead of the player, like closing the buying and then buying afterward? Or did everything go normal?

The income looks correct in the last sessions that were played, and the calculation that is shown on the screen in the Budget tab is complete.


Network Configuration URL for protocol server during installation

Problem:
Currently during installation of protocol server we ask for a network configuration URL. Nothing happens to this input.
Solution
The user should give the address of the Layer 2 Config at this place. Once it is provided, it should be copied into the schemas folder within the container for both the client and network parts of the Protocol Server being installed.

To establish clarity and work with Venkatesh.

Duplicate of #18

Actually #18
is the parent of
#132
#133
#134

This is a duplicate ticket for #18, hence marking this as closed.

@viraj89 , the understanding was that 18 is a parent ticket and it has three child tickets - protocol-server, registry and gateway. You have left the other two as active and for discussion. Wouldnt it make sense to also have this as active and track the protocol server implementation independently?
Actually for protocol server we have a potential solution already available (i.e. copy the given file to the schemas folder within the container).


Validate CreatePipeline request's input commit and error accordingly

I'm really unconvinced that this is the right fix.  We read from and write to this field internally, and if it's only for internal use, I guess that should be documented.   For example, it's read to create some internal structures for the S3 gateway, so there must have been some intent to use it, right?

I'm really unconvinced that this is the right fix. We read from and write to this field internally, and if it's only for internal use, I guess that should be documented. For example, it's read to create some internal structures for the S3 gateway, so there must have been some intent to use it, right?

Great point, we seem to be leaking private fields. However, the validation applied here only runs at pipeline creation time, so shouldn't affect how we use these field internally.
That said, the long term solution could be to either remove this field from the PPS spec, or actually implement the feature that actually keeps the input at a fixed commit as the user expects.


Can't use some components

Hi, I hope this is the right place to post. I'm using Tekla 2021 and 2022 and I've encountered this issue for both versions. The same errors showed up for Arrow/ Font/ Mark/ Text Attributes. Other Attributes definitions work fine. Is this a known problem?

Hello, nice to see you :)
This is the right place to raise this kind of issue. You're right - there are missing definitions for specified attributes.
I will try to fix asap and release new version.

@dandad36 please check the newest release 2.3.0 with hotfix:
https://github.com/LetsConstructIT/Grasshopper-TeklaDrawingLink/releases/tag/2.3.0

Thank you for the quick solution. That seems to fix it :)


data-generator version not available

The data-generator version 2.4.0 is not available on npmjs. I can see the latest version is only 1.0.0 ( as can be seen here. Hence, it is prompting an error while compiling.

I'm having the same issue. I was looking to install the demo to see how things are under the hood and had no luck, because of this mismatch version. I believe it is not even the same package.

data-generator is a private package. It's not published to npm at all, in any version.
{
    "name": "data-generator",
    "version": "2.7.1",
    "private": true,
    ...
}

The data-generator package you're seing on npm isn't the same as the one we're using, and it was authored by someone else.
To install the demo, clone the react-admin repository and run make install.


Recently Added Intents are not being triggered

Hi there --
My operating system is MacOS High Sierra 10.13.3. Not sure how to re-create this issue (I think if I were able to, I'd be able to fix it).
I've been trying to debug a problem with our agent for a while now, and have been running up against a mysterious error. I'm interfacing to DF using the NodeJS client, which is why I've posted the issue here, though I believe the issue might lie with the platform itself (if there's a better place to post this, please advise).
None of the intents that I have added in the past week are being triggered when a text that matches a training phrase is entered. Currently, I am creating new intents using the web console interface, and then communicating with the bot using the nodeJS API.
I have tried modifying the lifespans of the context to make sure they are not interfering with one another, and also have ensured that the text on each intent is sufficiently specific as to prevent interference.
The most recent test I have tried is to run a query in the Dialogflow console itself, having reset all contexts. (I also notice the mysterious appearence of a context called system_counters, which I don't believe was there before) The results are shown below: despite the query being a verbatim training phrase (the training phrases are specific, and definitely dissimilar to training phrases for other intents for this agent).

I do not believe that this project has exceeded a quota limit. There are currently around 25 intents, and fewer than 20 training phrases per agent.
All previously written intents continue to work fine, and I am able to change and update training and response phrases without issue.
The only issue I can think of that has changed the account in the past week or so is that we created a service account that can manage multiple agents simultaneously. This is necessary for our application, as we're running a serverless site which can only store one private key as a process variable. This itself was discussed on the Google forums, and was a solution that others had successfully used. I can't think why that would affect the adding of intents, though, especially as everything else continues to work as expected.
Thanks!
agnes

@agnescameron this repository is specifically for the JavaScript client libraries used to interact with dialogflow, we don't work on the dialogflow UI.
It might be worth opening an issue here, which is the issue tracker for the product :+1:
Let me know if I'm misunderstanding your problem.

sure, thanks! I mean, I think it's not a UI problem specifically, but it's not a Node problem either. Thanks for the tip!


Force custom lockscreen wallpaper does not work on windows 10 pro

I believe this content only applies to education and enterprise?

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

ID: 0286c17f-aca8-c9bc-597c-86b70165c72e
Version Independent ID: be2bdcde-8667-4ff5-c4e5-bd50696d9341
Content: Configure Windows Spotlight on the lock screen (Windows 10)
Content Source: windows/configuration/windows-spotlight.md
Product: w10
GitHub Login: @jdeckerMS
Microsoft Alias: jdecker

You should probably also include a link on how to manually change it.
So it's no longer possible to use Gpo to change lockscreen?

From: Jeanie Decker<EMAIL_ADDRESS>Sent: Thursday, October 25, 2018 6:27:34 PM
To: MicrosoftDocs/windows-itpro-docs
Cc: Rasmus Eeg Møller; Mention
Subject: Re: [MicrosoftDocs/windows-itpro-docs] Force custom lockscreen wallpaper does not work on windows 10 pro (#1946)
@rasmuseeghttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frasmuseeg&data=02|01||4386b0fb52bd480d884708d63a96c4d7|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636760816546718941&sdata=W8uPlYhvVlwA1UmTsBt%2B%2BzkYTFTdDO8Si%2BC7UsaDs8k%3D&reserved=0 The setting "Force a specific default lock screen image" only applies to ENT and EDU, correct. I'll add that to the sentence to make it clearer.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fissues%2F1946%23issuecomment-433117934&data=02|01||4386b0fb52bd480d884708d63a96c4d7|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636760816546718941&sdata=VZKzAGBCWYYzxBPTDFYp%2F1I3F4Mw3PLlaZ5qtSqg74w%3D&reserved=0, or mute the threadhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABcPKNxG2KFz7gLt3pphpnRvF_tIEzDgks5uoeZ2gaJpZM4X6omH&data=02|01||4386b0fb52bd480d884708d63a96c4d7|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636760816546875198&sdata=Aog6SeewLE638J5mdXbHLB7LeCpxFEyZis3a0sJq8oA%3D&reserved=0.

@rasmuseeg The lock screen Group Policy setting is for Windows 10 Enterprise or Education only. You can configure a lock screen image in a provisioning package (https://docs.microsoft.com/en-us/windows/configuration/wcd/wcd-personalization), however for it to work on Windows 10 Pro, you also need the SetEduPolicies settings (https://docs.microsoft.com/en-us/windows/client-management/mdm/personalization-csp).

Well that sounds durable. Yet unnecessary limitation? :-/

From: Jeanie Decker<EMAIL_ADDRESS>Sent: Thursday, October 25, 2018 7:00:54 PM
To: MicrosoftDocs/windows-itpro-docs
Cc: Rasmus Eeg Møller; Mention
Subject: Re: [MicrosoftDocs/windows-itpro-docs] Force custom lockscreen wallpaper does not work on windows 10 pro (#1946)
@rasmuseeghttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frasmuseeg&data=02|01||bdcf8c217c0f4369853708d63a9b6d2a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636760836550219912&sdata=oF3yHzIiuKkvSctCmvNOFcUJ2vKV%2B9Ld74Dw2%2FdX%2BAw%3D&reserved=0 The lock screen Group Policy setting is for Windows 10 Enterprise or Education only. You can configure a lock screen image in a provisioning package (https://docs.microsoft.com/en-us/windows/configuration/wcd/wcd-personalizationhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fwindows%2Fconfiguration%2Fwcd%2Fwcd-personalization&data=02|01||bdcf8c217c0f4369853708d63a9b6d2a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636760836550219912&sdata=%2FsC2uq%2BCUQH7cNxsaY00ltF%2FZW9zQCPBenr9d5SKN2Y%3D&reserved=0), however for it to work on Windows 10 Pro, you also need the SetEduPolicies settings (https://docs.microsoft.com/en-us/windows/client-management/mdm/personalization-csphttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fwindows%2Fclient-management%2Fmdm%2Fpersonalization-csp&data=02|01||bdcf8c217c0f4369853708d63a9b6d2a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636760836550219912&sdata=6SJRMCPtbFd0QRPa6tPuHsu6bzIITSvUSPXZvB12ubo%3D&reserved=0).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fissues%2F1946%23issuecomment-433129023&data=02|01||bdcf8c217c0f4369853708d63a9b6d2a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636760836550219912&sdata=SvEWBMDo5IlAvFzc7sctoD7ViqaCAbGhKg%2BoLhYhaSk%3D&reserved=0, or mute the threadhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABcPKPOR-Fe1JFYQIAPBZzfnxC0Op2LKks5uoe5GgaJpZM4X6omH&data=02|01||bdcf8c217c0f4369853708d63a9b6d2a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636760836550219912&sdata=VjdWsiC0tUq800H50u3ASKFn%2FtH3%2FJqQ7V21CiPTIGA%3D&reserved=0.

@officedocsbot assign @e0i

@rasmuseeg
This issue has been addressed with the following commit and content has been updated according to your feedback.
[fdd6437]   Merged PR 12353: clarify lockscreen image group policy
We are thus closing this issue since it seems to be resolved. Feel free to re-open it if you feel it hasn't been resolved.
Thank you for taking the time to provide feedback.

@officedocsbot close


Migrate your impacted OAuth out-of-band flow to an alternative method before Oct. 3, 2022

I received an email from Google saying I "have OAuth clients that used the OAuth OOB flow in the past."
Is this something that has been or can be fixed in gdrive?

See https://github.com/msfjarvis/gdrive  or https://github.com/carstentrink/gdrive

@carstentrink Thanks for the suggestion!
I managed to compile and install https://github.com/msfjarvis/gdrive. When I run gdrive it asks to authorize by visiting a Google URL (I suppose my old .gdrive/token_v2.json doesn't work anymore). Visiting the URL I get:
Authorization Error
Error 400: invalid_request
Missing required parameter: client_id

In msfjarvis/gdrive you need to place the json file with clientId and clientSecret from https://console.cloud.google.com/ in some place (I also do not know which location to place it). Either way it will NOT work as it is NOT in production.
self advertising
Just see the README in https://github.com/carstentrink/gdrive

You need to edit the clientId and clientSecret in the file handlers_drive.go.
Then compile.

@carstentrink THANK YOU! 🙌🏻
For other dummies like myself, here's what I did:

Followed these instructions to install the latest Go (min. 1.17 required) on Ubuntu 20.04
Downloaded https://github.com/carstentrink/gdrive/archive/refs/heads/main.zip
Edited handlers_drive.go to include my credentials, as instructed in README
Executed go build
Moved the generated gdrive binary to /usr/local/bin/
Executed gdrive version for the first time to get a Verification URL and Authorization Code prompt
Visited the verification URL. Accepted the fact that gdrive isn't a 'Google approved' app. Pasted the provided Authorization Code to the prompt

@carstentrink
Please clarify. I was looking at
https://developers.google.com/identity/protocols/oauth2/resources/oob-migration#inspect-your-application-code
but the 'RedirectURL' line in auth/oauth.go is still type 'oob'.
As far as I understood this type will stop working after 3 days.
Thanks

As long as your are in Testing all is good. Do not make it to production.

See (3) in the section https://github.com/carstentrink/gdrive#important
Alternatively, you can compile https://github.com/msfjarvis/gdrive that already uses Add support for OAuth callback flow

One small change to hansgv's comment. In step 6 I executed gdrive about for the Verification URL.

Someone messed with @carstentrink code and we can't compile now.
# github.com/msfjarvis/gdrive
./handlers_drive.go:369:74: undefined: auth.AuthCodeHTTP

Please try again. I have fixed it. I am using go1.19.1 version.

Hi, I'm using a headless linux box but can't sign in to google with my user using browsh.
I tried compiling on MacOS without any errors, but got the following when trying to run the ./gdrive about command:
Failed to get about: Get "https://www.googleapis.com/drive/v3/about?alt=json&fields=maxImportSizes%2CmaxUploadSize%2CstorageQuota%2Cuser&prettyPrint=false": oauth2: cannot fetch token: 400 Bad Request
Response: {
  "error": "invalid_grant",
  "error_description": "Token has been expired or revoked."
}

Any ideas on how to sign in?
I would like to use this in a CI system, but I think the token will keep expiring…

@juliusspencer
Do it locally then copy both linux binary and .config/gdrive/something.jsonto the remote box.

Thanks, yeah I did just that, I just need to update the variable that stores the token as it updates as the docker container overwrites it. I'm almost there.

May be this repo has some actions that are useful. https://github.com/marufshidiq/gdrive-cli-builder/blob/master/.github/workflows/build.yml

This is great, and everything works, except ~/.config/gdrive/USERNAME_v2.json is not created. I created the folder structure ~/.config/gdrive/ and then the file USERNAME_v2.json was created correctly.


when use lazyload components    Can't bind to 'ngModel' since it isn't a known property of 'nz-input'.

Version
1.0.0
Environment
window   node  v8.11.2  ng.an 0.7.0
Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-start-4zzcef
Steps to reproduce
i have read (https://github.com/NG-ZORRO/ng-zorro-antd/issues/130)
but the solution :
If you want to use ngModel, you have to import FormsModule in your module
import { FormsModule } from '@angular/forms';） not work;
when use lazyload components
template  notice
Can't bind to 'ngModel' since it isn't a known property of 'nz-input'.
<nz-input [(ngModel)]="text" nzSize="large">
What is expected?
正常编译
What is actually happening?
core.js:1598 ERROR Error: Uncaught (in promise): Error: Template parse errors:
Can't bind to 'ngModel' since it isn't a known property of 'nz-input'.

If 'nz-input' is an Angular component and it has 'ngModel' input, then verify that it is part of this module.
If 'nz-input' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.
To allow any property add 'NO_ERRORS_SCHEMA' to the<EMAIL_ADDRESS>of this component. ("
Lz works!
<nz-input [ERROR ->][(ngModel)]="text" nzSize="large">

"): ng:///LzModule/LzComponent.html@2:12
'nz-input' is not a known element:
1. If 'nz-input' is an Angular component, then verify that it is part of this module.
2. If 'nz-input' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message. ("
  Lz works!
  [ERROR ->]

"): ng:///LzModule/LzComponent.html@2:2
Error: Template parse errors:
## Other?

在子模块中import NgZorroAntdModule

Correct way
shoud be  like this
<input nz-input placeholder="Basic usage" [(ngModel)]="LzText">


Check date pattern match for world wide languages

Currently we support 24 hours and AM PM. We need to check if there are time formats not covered by this.

I checked the node behavior new Date().toLocaleTimeString() which is used by tsc and the only two formats it prints is 24 hours and am/pm. Closing since these two patterns are matched


The picked value on QuickPickItems is not being honored

From https://code.visualstudio.com/api/references/vscode-api#QuickPickItem:

picked?: boolean
Optional flag indicating if this item is picked initially. This is only honored when using the widnow.showQuickPick() API. To do the same thing with the window.createQuickPick() API, simply set the selectedItems to the items you want picked initially. (Note: This is only honored when the picker allows multiple selections.)
see - canPickMany
Note: this property is ignored when kind is set to Separator

I guess isPickSelected is meant to be used for this.


Removing both 3rd party prompt known issues

We are deploying a "1st party AppId" fix, so customers will no longer see the 3rd party permissions prompt the first-time they use the web app.

Will do


GPLayer doesn't seem to support multiple input units

Using the "Hybrid Deep GP models: ..." tutorial, when I changed
tf.keras.layers.Dense(1, activation="linear"),
to
tf.keras.layers.Dense(2, activation="linear"),
I got an error, same as reported in #27.
I then set a Zero mean function, as suggested in #27:
gp_layer = gpflux.layers.GPLayer(
kernel, inducing_variable, num_data=num_data, num_latent_gps=output_dim,
mean_function=gpflow.mean_functions.Zero()
),
I got a different error.

Hi @dtchang, it seems like you want to increase the input dimensionality of the GP by increasing the number of outputs of the underlying neural network, is that correct? note that the InducingPoints inducing_variable defined in the previous cell [3] is constructed to live in a 1D space, so this won't be compatible with the 2D output from the neural network. You could construct a 2D grid of inducing points instead. Does that resolve your issue? If not, please provide more details & error messages!

In general, one only needs to specify the output dimension for a Keras layer, not the input dimension since the input dimension is automatically determined by the output dimension of the previous layer. TFP layers honor this. Shouldn't GPLayer honor this to be useful in building hybrid models, or maybe even DGP models? (There is no documentation on the usage expectation, patterns and constraints for GPLayer.)

Hi @dtchang, it would indeed be possible to determine the input dimensionality of the GP and automatically create inducing points with the right dimension. However, in this first version of GPflux we decided to leave as much initialisation as possible to the user. I appreciate that this makes the code more verbose and less automatic, but it does give a user more control to construct non-standard (deep) GP models using, for example, interdomain inducing features. I would very much welcome a PR that addresses this problem though.


VScroll

Using your demo https://swimlane.gitbooks.io/angular2-data-table/content/examples/client-paging.html
If I try to enable scrollbarV: true,
I receive no data.
It's mandatory to do in the html?

Works when rowHeight not auto


Swift package manager support

Uh. Thanks for submitting a PR with an empty comment Michael. 😅
No moving the headers please. Please use symlinks instead. Thanks!

@TimOliver oh sorry, forgot to include a description 🙇🏻♂ .I will update this. Thanks

@TimOliver It's now updated. please check when you have time. thank you 🙇🏻♂

@TimOliver thanks