Updating previously release api with missing properties within Microsoft.DBforPostgreSQL

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[X ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[ ] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[ ] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
Stable release 2024-08-01
 Added enum value to UserAssignedIdentity
Preview release 2024-11-01
Added property to ServerPropertiesForUpdate to update cluster size

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Microsoft.DBforPostgreSQL

/azp run


invalid password error

v1.6.3 on Debian 10
Was working ok for a couple weeks, but with usual JVM nonsense, where it bleeds RAM until the app hangs or crashes the machine.
This time when sparrow hanged like that, and had to be killed, on restart it doesn't recognize the wallet password anymore.
Does it have to be restored from seed?

Restored from seed, which worked, but had to increase JVM ram allotment to 6GB to get mixed UTXOs to appear.
Of course, lost all the labels that were in the original wallet.

Now I see also that Badbank txs did not recover, and have error:

[May 08 22:19:50] Failed to subscribe to paths: [../0/0-../0/19]

Was working ok for a couple weeks, but with usual JVM nonsense, where it bleeds RAM until the app hangs or crashes the machine.

It's not clear what you mean here - that Sparrow has a memory leak? Or the JVM itself? If so please open an issue with evidence of this - I have done long running memory monitoring tests on all 3 platforms to test this, and have found no evidence of it.

This time when sparrow hanged like that, and had to be killed, on restart it doesn't recognize the wallet password anymore.

What error did you get? Was the wallet file corrupted?

Restored from seed, which worked, but had to increase JVM ram allotment to 6GB to get mixed UTXOs to appear.

Are you running from source? Or are you introducing RAM limiting another way?

Now I see also that Badbank txs did not recover, and have error:

This means that the server you are connected to is failing, not Sparrow itself. If you are using Electrs, it may need a restart, or better upgrade to Fulcrum.

Thanks for response!
I think it's the JVM, where it consumes more and more resources until the app stops responding. This one is running in a Qubes VM by itself with 4 cores and 6GB available to it maximum. I'll try to get the exact numbers from top next time it happens.
The error was "invalid password", for password I get straight from file manager. I think the wallet file corrupted when it had to be killed, because it wasn't responding.
I'll try to reboot electrs. Fulcrum is not available for my node yet I think.

To get Badbank again was enough to just disconnect sparrow a few times with the button in lower right.


Viviali iotcentral new API onboarding

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

@ruowan do you know what kind of this error https://dev.azure.com/azure-sdk/public/_build/results?buildId=164674&view=logs&j=f10f5372-40ed-54fc-caa4-b1cb5bd1252b&t=d3b2b841-a93b-5fa5-47c3-d8c4f6eae6e3&l=258 and how to fix it?

@ruowan do you know what kind of this error https://dev.azure.com/azure-sdk/public/_build/results?buildId=164674&view=logs&j=f10f5372-40ed-54fc-caa4-b1cb5bd1252b&t=d3b2b841-a93b-5fa5-47c3-d8c4f6eae6e3&l=258 and how to fix it?

I add a comment. The input file path is not correct, so avocado can't find the input file and report these errors.

@qiaozha Could you please take a look at the breaking change error?

/azp run

We have approval from API review board @rysweet for public preview

@viv-liu merged


Adding nodes to the graph model without specifying the name

I often find that I need a graph model due to moving the input to 2 separate layers and merging them back down the track, despite most of the model being sequential in nature.
I think this leads to a lot of unnecessary naming of layers, and the syntax is more convoluted coming from lasagne. Would it be possible to just use the previously added node if the name is not given? Say the following model:
graph = Graph()
graph.add_input(name='input', input_shape=(32,))

graph.add_node(Dense(16), name='dense1', input='input')
graph.add_node(Dense(4), name='dense2', input='dense1')

graph.add_node(Dense(16), name='dense3', input='input')
graph.add_node(Dense(4), name='dense4', input='dense3')

graph.add_node(Dense(4), name='dense5', inputs=['dense2', 'dense4'])
graph.add_node(Dense(4), name='dense6', input='dense5')
graph.add_node(Dense(4), name='dense7', input='dense6')
graph.add_output(name='output', input='dense5')

Could be written instead as:
graph = Graph()
graph.add_input(name='input', input_shape=(32,))

graph.add_node(Dense(16), input='input')
graph.add_node(Dense(4), name='dense2')

graph.add_node(Dense(16), input='input')
graph.add_node(Dense(4), name='dense4')

graph.add_node(Dense(4), inputs=['dense2', 'dense4'])
graph.add_node(Dense(4))
graph.add_node(Dense(4))
graph.add_output(name='output')

This is more like the lasagne style where a layers which don't need to be referneced later on can just use a generic layer name l, but in keras perhaps some unique internal name could be used.

I often find that I need a graph model due to moving the input to 2 separate layers and merging them back down the track, despite most of the model being sequential in nature.

It's simple: what you want to do is use a keras.containers.Sequential instance for the sequential parts of your model, then add these instances to a Graph instance.


Repo sync for protected CLA branch

The pull request is created from master637347514381784972 to master to fix git push error for protected CLA branch

@ghogen : Thanks for your contribution! The author(s) have been notified to review your proposed change. @evanwindom


Fix segfault with multithreading

Uses https://github.com/oxfordcontrol/osqp/pull/216 to fix segfault issue.

It releases GIL when printing
It uses raw python memory management functions that do no require GIL

Fixes #33
@sbarratt could you try this branch in multithreaded mode? It fixes the segfault on my computer.

For some weird reason it is failing under python 2 on mac. I would not pay to much attention to it since we are planning to discontinue python 2.

Fixes the segfault on my computer!
import osqp
import numpy as np
import scipy as sp
from scipy import sparse
from multiprocessing.pool import ThreadPool
import time

data = []
for i in range(50):
    m = 1000
    n = 500
    Ad = sparse.random(m, n, density=0.3, format='csc')
    b = np.random.randn(m)

    # OSQP data
    P = sparse.block_diag([sparse.csc_matrix((n, n)), sparse.eye(m)], format='csc')
    q = np.zeros(n+m)
    A = sparse.vstack([
            sparse.hstack([Ad, -sparse.eye(m)]),
            sparse.hstack([sparse.eye(n), sparse.csc_matrix((n, m))])], format='csc')
    l = np.hstack([b, np.zeros(n)])
    u = np.hstack([b, np.ones(n)])

    data.append((P, q, A, l, u))

def f(i):
    P, q, A, l, u = data[i]
    # Setup workspace
    osqp.solve(P, q, A, l, u, verbose=False)

pool = ThreadPool(4)

tic = time.time()
for i in range(50):
    f(i)
toc = time.time()
print (toc-tic)

tic = time.time()
pool.map(f, range(50))
toc = time.time()
print (toc-tic)


remove TransportSqlAction from AlterTableOperation

Current coverage is 81.05% (diff: 81.81%)

Merging #4166 into master will decrease coverage by 0.06%

@@             master      #4166   diff @@
==========================================
  Files          1294       1294
  Lines         43978      44000    +22
  Methods           0          0
  Messages          0          0
  Branches       6994       6994
==========================================
- Hits          35676      35665    -11
- Misses         5672       5699    +27
- Partials       2630       2636     +6

Powered by Codecov. Last update 601c68e...5b6dc3b

@dobe please review

@bogensberger please prereview

@mfussenegger bogi has pre reviewed it, can you review it pls?


Add Motion Helper Objects

Create an object in java that you can pass to the driving subsystem which then moves accordingly (See Motion in SteamWorks2017).

Finished and merged with blessed


Upstream plutarch-quickcheck, plutarch-unit, plutarch-context-builder

Upstream testing utilities, make necessary changes to make overall experience smoother, write guilds and documentation over additions.

https://github.com/liqwid-labs/liqwid-libs/tree/main/plutarch-quickcheck
https://github.com/liqwid-labs/liqwid-libs/tree/main/plutarch-unit
https://github.com/liqwid-labs/liqwid-libs/tree/main/plutarch-context-builder

Some of the utilities are implemented again at plutarch, will close as it is not planned


Generate swagger specification for given action type

Currently, there is an option to generate swagger specification for given controller type. I am looking for generating swagger specification for given action type.
This is the thought:

Create IOperationFilter or OperationProcessor to add extension for code sample.

Generate swagger spec for each operation and inject in the x-code-sample property.

Implement javascript in swagger UI to get that values from swagger specs. I can inject custom javascript if I use Swashbuckle but could not inject custom javascript using NSwag.

@rsuter will there be any support provided

Sorry, I don't understand your issue?

`    public class AddCodeSamplesOperationFilter : IOperationFilter
{
    private readonly string _language;
    private readonly string _source;
    private const string ExtensionKey = "x-code-samples";

    public void Apply(Operation operation, OperationFilterContext context)
    {

        // Get Controller and action for which Swagger Spec is required
        var appAssembly = (context.ApiDescription.ActionDescriptor as ControllerActionDescriptor).ControllerTypeInfo.Assembly;
        var controllerName = (context.ApiDescription.ActionDescriptor as ControllerActionDescriptor).ControllerTypeInfo.FullName;

        var controllersList = appAssembly.GetTypes().Where(type => typeof(Microsoft.AspNetCore.Mvc.Controller)
                                .IsAssignableFrom(type) && type.FullName.Equals(controllerName));

        var actionName = (context.ApiDescription.ActionDescriptor as ControllerActionDescriptor).ActionName;

        // Generate Swagger Specification
        var swaggerGeneratorSettings = new WebApiToSwaggerGeneratorSettings
        {
            DefaultUrlTemplate = "api/{controller}/{action}/{id}"
        };
        var swaggerGenerator = new WebApiToSwaggerGenerator(swaggerGeneratorSettings);
        var swaggerSpec = swaggerGenerator.GenerateForControllersAsync(controllersList).GetAwaiter().GetResult();

        // Remove unnecessary paths from swagger spec
        var pathsToRemove = swaggerSpec.Paths.Keys.Where(path => !path.Equals($"/{context.ApiDescription.RelativePath}")).ToList();
        foreach (var path in pathsToRemove)
        {
            swaggerSpec.Paths.Remove(path);
        }

        // Generate Client Code
        var csharpClientSettings = new SwaggerToCSharpClientGeneratorSettings
        {
            ClassName = "{controller}Client",
            GenerateDtoTypes = false,
            GenerateExceptionClasses = false,
            GenerateResponseClasses = false,
            DisposeHttpClient = false,
            CSharpGeneratorSettings = {
                Namespace = "Ciam.Client"
            },
            InjectHttpClient = true
        };

        var csharpClientGenerator = new SwaggerToCSharpClientGenerator(swaggerSpec, csharpClientSettings);
        var csharpCode = csharpClientGenerator.GenerateFile();

        // Inject Generated Swagger Code in Swashbuckle Swagger Spec
        var codeSamples = new List<CodeSample>();
        codeSamples.Add(new CodeSample
        {
            Language = "csharp",
            Source = string.Format("C# Code for {0}", csharpCode)
        });
        codeSamples.Add(new CodeSample
        {
            Language = "angular",
            Source = string.Format("Angular Code for {0}", operation.OperationId)
        });
        operation.Extensions.Add(ExtensionKey, codeSamples);
    }

}

internal class CodeSample
{
    [JsonProperty("lang")]
    public string Language { get; set; }

    [JsonProperty("source")]
    public string Source { get; set; }
}`

I am using Swashbuckle and writing this operationfilter. So I want to generate Swagger Spec only for particular endpoint/action at one time.

@RSuter

Injecting custom javascript into Swagger UI should be simple to add... How is this solved/configured in Swashbuckle?

@Rsuter Swashbuckle gives the option to inject custom javascript in options like this:

The only thing is whether I user operation filter or document filter generating client code per action wise would be a great optimization for me.
Currently it is: swaggerGenerator.GenerateForControllersAsync(controllersList).GetAwaiter().GetResult();
If we can have something like : swaggerGenerator.GenerateForActionAsync(actionName).GetAwaiter().GetResult(); where action name is string, should generate client side method only.

I think you can easily implement this method yourself, with a custom operation processor... Adding the javascript injection settings should be simple. How are these injection js configured in Swagger UI?

I ended up creating this npm package for my requirement
https://www.npmjs.com/package/swaggerui-snippets


x11docker: 6.6.2 -> 6.9.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/x11docker/versions.
meta.description for x11docker is: "Run graphical applications with Docker"
meta.homepage for x11docker is: "https://github.com/mviereck/x11docker"
meta.changelog for x11docker is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/a46hgz1wnd8k74vlhaci17kqmlynhmsy-x11docker-6.9.0/bin/x11docker passed the binary check.

/nix/store/a46hgz1wnd8k74vlhaci17kqmlynhmsy-x11docker-6.9.0/bin/.x11docker-wrapped passed the binary check.

2 of 2 passed binary check by having a zero exit code.

2 of 2 passed binary check by having the new version present in output.

found 6.9.0 with grep in /nix/store/a46hgz1wnd8k74vlhaci17kqmlynhmsy-x11docker-6.9.0

found 6.9.0 in filename of file in /nix/store/a46hgz1wnd8k74vlhaci17kqmlynhmsy-x11docker-6.9.0

directory tree listing: https://gist.github.com/af59d15c800ac06acd526d119aaa2977

du listing: https://gist.github.com/69d6c5c480e2182f77b746ae53d23dad

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
x11docker

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/a46hgz1wnd8k74vlhaci17kqmlynhmsy-x11docker-6.9.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A x11docker https://github.com/r-ryantm/nixpkgs/archive/c3dcf58df24791321390683f3b9752d6a91743bb.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/a46hgz1wnd8k74vlhaci17kqmlynhmsy-x11docker-6.9.0
ls -la /nix/store/a46hgz1wnd8k74vlhaci17kqmlynhmsy-x11docker-6.9.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

x11docker

Maintainer pings

Result of nixpkgs-review pr 133908 at c3dcf58d run on x86_64-linux 1

1 package built successfully:

x11docker

2 suggestions:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/applications/virtualization/x11docker/default.nix:26:5:
   |
26 |     maintainers = with lib.maintainers; [ ];
   |     ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/virtualization/x11docker/default.nix:16:3:
   |
16 |   installPhase = ''
   |   ^


Not supported on darwin arm64

graphql-ppx does not support this platform :(

graphql-ppx comes prepacked as built binaries to avoid large
dependencies at build-time.

If you want graphql-ppx to support this platform natively,
please open an issue at our repository, linked above. Please
specify that you are on the darwin platform,
on the arm64 architecture.

🙂

Oh yes we should support it. We just have to figure out how to compile it on github actions. Any idea?

Oh yes we should support it. We just have to figure out how to compile it on github actions. Any idea?

From furious Googling, and seeing you don't use xcodebuild anywhere directly, I suppose something here might help: https://discuss.ocaml.org/t/ann-easy-cross-compilation-using-esy/6612

From furious Googling, and seeing you don't use xcodebuild anywhere directly, I suppose something here might help: https://discuss.ocaml.org/t/ann-easy-cross-compilation-using-esy/6612

I think cross-compilation is available in the latest point release of 4.10. (the example above is for cross-compile to mobile).

I think cross-compilation is available in the latest point release of 4.10. (the example above is for cross-compile to mobile).

BTW with Rosetta we could probably just run the intel version for now. So maybe you can edit the detection script and allow mac-os arm? That will probably work. If it works a PR is welcome!

BTW with Rosetta we could probably just run the intel version for now. So maybe you can edit the detection script and allow mac-os arm? That will probably work. If it works a PR is welcome!

Yes, with Rosetta it will definitely also work. I will try that later and if it does send a PR. :)

Yes, with Rosetta it will definitely also work. I will try that later and if it does send a PR. :)

https://github.com/reasonml-community/graphql-ppx/blob/master/copyPlatformBinaryInPlace.js

https://github.com/reasonml-community/graphql-ppx/blob/master/copyPlatformBinaryInPlace.js

Yes, with Rosetta it will definitely also work. I will try that later and if it does send a PR. :) Also the performance of Rosetta is ridiculously good, I'm still amazed lol.

Cool, I am tempted to sell my 16" maxed out mbp for a M1 Air lol :D

Yes, with Rosetta it will definitely also work. I will try that later and if it does send a PR. :) Also the performance of Rosetta is ridiculously good, I'm still amazed lol.

Cool, I am tempted to sell my 16" maxed out mbp for a M1 Air lol :D

I did get the Air, just to really test it. 12 core Ryzen 5 vs MacBook on an intense Rust release build and the air was only 20% slower... whereas my last generation 15in MacBook Pro is about twice as slow as the Air. And it never gets warmer than 80%! Just unbelievable stuff, especially since I am not a big fan of Apple, heh.

I did get the Air, just to really test it. 12 core Ryzen 5 vs MacBook on an intense Rust release build and the air was only 20% slower... whereas my last generation 15in MacBook Pro is about twice as slow as the Air. And it never gets warmer than 80%! Just unbelievable stuff, especially since I am not a big fan of Apple, heh.

@jfrolich Maybe for MacOS arm64 users the ppx could build from source?

@jfrolich Maybe for MacOS arm64 users the ppx could build from source?

I recall some other PPX doing this (cant recall)

I recall some other PPX doing this (cant recall)


Added vmc-vcenter-stats execution module and corresponding unit tests

Added execution module and corresponding unit tests for vm_vcenter_stats

Added integration tests and also updated testscraper to populate required configuration


Aggregated Player Career Stats

create ncaa_career_aggregated to pull a player's career stats as a single row. ID columns like season, school_id, and division are left as 0 to main int consistency with the original functions. The function is very similar to the original ncaa_career_stats function, but tweaked enough that I figured it was easier to have a new function for rather than conditionals

updated add_batting_metrics and add_pitching_metrics with a season = True default arg to account for generating the career advanced stats. When season = False, the function leaves the advanced stats that are dependent on season/team/division as NA

added df['season'] = df['season'] + 1 for the season-by-season stats from the ncaa_career_stats function because it would default 2021-2022 as the 2021 season despite games being played in Spring 2022

Future todo: add linear weight based stats (i.e. wOBA, wRC, wRAA) for career aggregated stats. Not included in this PR.


Fix CWE-117

Signed-off-by: Taras Drozdovskyi<EMAIL_ADDRESS>Description
Fix CWE-117. I would like to note that the task is more difficult than it seems at first glance. The reason is that the LGTM(CodeQL) does not have the correct template for solving this problem. It can be argued that the vulnerability has been fixed, but the LGTM cannot detect this fix. Hence each line is marked as excluded. I have not excluded this type of error for the entire project, since the vulnerability is serious. And we will have to fix it every time it appears. Sometimes the lines of code became out very long, I didn't want to do that, but I had to.
I hope this is a temporary solution, and a nicer solution will be found.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. Below is just an example and you can delete it when you contribute.
Test Configuration:

OS type & version: Ubuntu 18.04
Hardware: x86-64
Toolchain: Docker v20.04 and Go v1.16
Edge Orchestration Release: v1.1.2

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

@tdrozdovsky Let us take an action completed for fixing CWE-117 as v1.1.4.

Please see Github Code scanning alerts and share your thought with us.

Recommendation from the Github Code scanning alerts does not work:

also the current implementation allows you to see the attack attempt by checking the log

Also, how about assigning log level to debug instead of info for CWE-117 logs?

Sorry, I did not understand. Please explain what do you mean?

Also, how about assigning log level to debug instead of info for CWE-117 logs?

Sorry, I did not understand. Please explain what do you mean?

Currently many lines of logs are used at the information level. I suggest that the log which can cause problems with external input run in debug mode as below. What do you think?
log.Println(  > log.Debug(

Also, how about assigning log level to debug instead of info for CWE-117 logs?

Sorry, I did not understand. Please explain what do you mean?

Currently many lines of logs are used at the information level. I suggest that the log which can cause problems with external input run in debug mode as below. What do you think? log.Println(  > log.Debug(

Also, how about assigning log level to debug instead of info for CWE-117 logs?

Sorry, I did not understand. Please explain what do you mean?

Currently many lines of logs are used at the information level. I suggest that the log which can cause problems with external input run in debug mode as below. What do you think? log.Println(  > log.Debug(

It doesn't depend on the method. And for this, changing the method is not correct.

This idea is not intended to resolve CWE-117. I thought of this idea in order to reduce the frequency of printing!


if you don't need the closure, don't do the closure

that's why god invented commas babe

I think most of this was taken care of in https://github.com/marrus-sh/laboratory/commit/6dfb82bcb95b2f9aafede440798c52797b92bdb1 although there might be a few stragglers to take care of in Constructors


New version: RoyQu.RedPanda-C++ version 3.2

Created by 🥟 Dumplings in workflow run #9850.

Log
Updated: 3.1 → 3.2
[session-c44555aa] Repository or file not found
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

URL: https://sourceforge.net/projects/redpanda-cpp/files/v3.2/RedPanda.C++.3.2.win32.MinGW32_11.5.Setup.exe/download
Status Code: 200
(Automated message - build 899)


ref: dvc data status

Report
See https://github.com/iterative/dvc/pull/7943.
@skshetry Let's start on this ASAP so that if there are concerns about the UI, they can be raised early.

That would be neat! You can make it a draft PR for now and we can have product/ QA discussions on top of it. Thanks!

@jorgeorpinel Saugat is out but the core PR should be nearly ready to merge. If you have time, please play around with it or even just check out the comments in the PR to see some example output.

After looking into it I'm not too worried about docs. Mainly we need the new command reference and to review existing references of dvc status to decide which ones to replace. As for the reference, probably the more complicated part will be to decide which useful examples to include. On the long run the issue I see is the overlap with dvc status but that's more of a product question.

On the long run the issue I see is the overlap with dvc status but that's more of a product question.

Even for the command ref for the initial PR, this is probably the biggest question that we need to address. My take is that we recommend dvc data status as a better option whenever you are focused on the status of the data and not the pipeline stages. dvc status is still needed if you have want the status of those pipeline stages (we had plans to have a dvc stage status command or similar but it's nor prioritized right now).

Wrt the command namespaces, since the basic commands like add and push relate more to data management, it would make sense for dvc status to give the new data status IMO. And to move the existing pipeline status command into dvc stage status. But that could break backward compatibility so it's probably tricky... Anyway, just a thought.

Yeah, maybe for 3.0. Ideally, we would have an improved dvc stage status and can deprecate the existing dvc status.


Added build of all targets to Travis so we know when something breaks

This is probably why the watchOS change that wouldn't build was passing
Added xcpretty to the end 😎

Nice, didn't know that xcpretty was installed on travis 👍

Yeah its cool.


Tech Grid Theme

Please do not modify files in the template folder. Thank you!

I was trying to create a merge from my forjed repository.
On Mon, Feb 27, 2023, 9:44 PM TapiocaFox @.***> wrote:

Please do not modify files in the template folder. Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/magneticchen/Daijishou/pull/267#issuecomment-1447485465,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6EOFHP52IIDXEHDQCI62WLWZVRBZANCNFSM6AAAAAAVKE6YJU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ok. I'm sorry about that as I'm am a noob at GitHub. I'm better at making a
theme than using GitHub. Will the merge be denied again?
On Tue, Feb 28, 2023, 2:17 AM UnbelievableFlavour @.***>
wrote:

@Galvy80 https://github.com/Galvy80 and you did, which is fine. But you
also edited the template index.json
—
Reply to this email directly, view it on GitHub
https://github.com/magneticchen/Daijishou/pull/267#issuecomment-1447695199,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6EOFHLE25Q27KATN7QN5WDWZWQ7PANCNFSM6AAAAAAVKE6YJU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@magneticchen I don't think this merge request should be closed. It should only be edited before merging. The only change that is necessary is reverting the default index.json changes.

Is there anything else I need to do on my end? Also again very sorry about
changing the file.
On Tue, Feb 28, 2023, 3:32 AM UnbelievableFlavour @.***>
wrote:

@magneticchen https://github.com/magneticchen I don't think this merge
request should be closed. It should only be edited before merging. The only
change that is necessary is reverting the default index.json changes.
—
Reply to this email directly, view it on GitHub
https://github.com/magneticchen/Daijishou/pull/267#issuecomment-1447771967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6EOFHLH57PUNZMVBHERZEDWZWZ3RANCNFSM6AAAAAAVKE6YJU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Galvy80 you should probably push the commit. Since it is not yet changed in this MR.

@magneticchen I don't think this merge request should be closed. It should only be edited before merging. The only change that is necessary is reverting the default index.json changes to this: https://github.com/magneticchen/Daijishou/blob/main/themes/templates/platform_wallpapers_pack/index.json

I am doing this because there was a person in the past pushing almost 10 problematic requests while I kept reminding him there was errors. I found that I need to make people aware of the situation and improve by trial and error.

I'm sorry about editing the file. Do I need to do anything else to merge
this so my theme is part of Daijisho?
On Tue, Feb 28, 2023, 5:04 AM TapiocaFox @.***> wrote:

@magneticchen https://github.com/magneticchen I don't think this merge
request should be closed. It should only be edited before merging. The only
change that is necessary is reverting the default index.json changes to
this:
https://github.com/magneticchen/Daijishou/blob/main/themes/templates/platform_wallpapers_pack/index.json
I am doing this because there was a person in the past pushing almost 10
problematic requests while I kept reminding him there was errors. I found
that I need to make people aware of the situation and improve by trial and
error.
—
Reply to this email directly, view it on GitHub
https://github.com/magneticchen/Daijishou/pull/267#issuecomment-1447897950,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6EOFHPVMLQPGY2HAW3UFDLWZXESNANCNFSM6AAAAAAVKE6YJU
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm sorry about editing the file. Do I need to do anything else to merge this so my theme is part of Daijisho?
…
On Tue, Feb 28, 2023, 5:04 AM TapiocaFox @.> wrote: @magneticchen https://github.com/magneticchen I don't think this merge request should be closed. It should only be edited before merging. The only change that is necessary is reverting the default index.json changes to this: https://github.com/magneticchen/Daijishou/blob/main/themes/templates/platform_wallpapers_pack/index.json I am doing this because there was a person in the past pushing almost 10 problematic requests while I kept reminding him there was errors. I found that I need to make people aware of the situation and improve by trial and error. — Reply to this email directly, view it on GitHub <#267 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6EOFHPVMLQPGY2HAW3UFDLWZXESNANCNFSM6AAAAAAVKE6YJU . You are receiving this because you were mentioned.Message ID: @.>

You can abandon entire fork. Then re-fork a new branch and add all your item again in the correct method (Not touching the template folder).


Android Auto

Would it be posible to use this app in android auto?

Sorrry, no idea. Don't use Android Auto.


Extend the graph teams app publish command to allow updates

Right now, graph teams app publish and graph teams app update are separate commands.
There should be an update option to the publish command which will allow updating of apps already deployed to the teams app catalog:
graph teams app publish -filePath <filePath> --u|--update
This is similar to how the CLI handles updates to SPFx packages:
o365 spo app add -p somepackage.sppkg --overwrite

@waldekmastykarz @estruyf I can pick this one up if the syntax if Ok with you.

In spo app add you can retrieve the original package name to detect which app to update. Can you do the same here? Otherwise, it won't be possible to update the app without specifying its id which justifies doing it all in a separate command.

Ok I have had some time to investigate this now and looks like we are better off not having this in the CLI. I was originally thinking if the --update option was specified, then we check if the app exists in the teams app catalog and if it does, we would update it (This can be checked by filtering on the externalId property of the app)
But on second thought, it's better if we leave it out of the CLI and do this from the calling script instead. Something along the lines of this: https://pnp.github.io/office365-cli/user-guide/cli-output-mode/#sample-script
Will close this one now.


Always null package name

I am using this snippet of yours inside of a service, but I always get null as a package name. Testing on Google Pixel, Android version 7.1
AppChecker appChecker = new AppChecker();
String packageName = appChecker.getForegroundApp(context);

My permissions look like this:
<uses-permission android:name="android.permission.GET_TASKS"/>
    <uses-permission android:name="android.permission.PACKAGE_USAGE_STATS" tools:ignore="ProtectedPermissions"/>

Hi arenaq,
Are you requesting the Usage Stats permission to the user? Even though you have it set in the android manifest, unfortunately, it still requires a manual acceptance from the user. In the sample code you may find code to call the relevant activity.
Here is what it looks like, taken from this file:
private void requestUsageStatsPermission() {
    if(!hasUsageStatsPermission(this)) {
        startActivity(new Intent(Settings.ACTION_USAGE_ACCESS_SETTINGS));
    }
}

Cheers,
Ricardo

Yes, that was it. Thank you.
Only needsUsageStatsPermission does not work, because hasUsageStatsPermission returns always false, but I managed to get around this by calling requestUsageStatsPermission everytime.
Or do you have an idea how to fix needsUsageStatsPermission?

That's odd. Let me take a look at it and I will come back to you. It might take a couple of days, sorry, a bit tight this week in time.
Cheers,
Ricardo

Have you fixed this? I would like to use this library.

Hi,
Sorry for the delay. I tried to reproduce, but it is working correctly. In any case, I have released a new version, 1.1.0. This problem was not addressed but I recommend upgrading.
Will keep digging on this issue.
Cheers,
Ricardo


hyper-util auto version detection breaks the ability to use Upgraded::downcast

Version
hyper = 1.2.0
hyper-util = 0.1.3
Description
When using: hyper_util::server::conn::auto::Builder::new(..).serve_connection_with_upgrades the io stream is wrapped with Rewind which is private and therefore cannot be used to call Upgraded::downcast.
To reproduce the issue I'v modified the upgrades example here: https://github.com/RyanAD/hyper-broken-upgrade
The shortened version is that handing a connection with:
hyper_util::server::conn::auto::Builder::new(TokioExecutor::new())
                    .serve_connection_with_upgrades(io, service_fn(server_upgrade)).await;

...

// downcast to the underlying TcpStream
let parts = upgraded
    .downcast::<TokioIo<TcpStream>>()
    .map_err(|_| "server: Unable to downcast Upgraded")?;

let mut conn: TcpStream = parts.io.into_inner();

Will fail, because the stream is actually wrapped in Rewind which is not visible externally and can't be used to call downcast.
Without using hyper_util::server::conn::auto::Builder::new it works as expected.  This also prevents downcasting Upgraded when using axum::serve, but that can be worked around by setting up the handling manually without using auto::Builder

Yea good point. Before, we were able to work around this explicitly since both types were in the same library. Now that they are separate, it will be trickier to do so. 🤔

bikeshedding/stopgap: hyper_util::server::conn::auto::upgrade::downcast::<T>(upgraded) (that's a horrible path but whatever).


error when run  python crawler.py posts

Error occurred during loading data. Trying to use cache server https://fake-useragent.herokuapp.com/browsers/0.1.11
Traceback (most recent call last):
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\site-packages\fake_useragent\utils.py", line 154, in load
for item in get_browsers(verify_ssl=verify_ssl):
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\site-packages\fake_useragent\utils.py", line 99, in get_browsers
html = html.split('')[1]
IndexError: list index out of range
Traceback (most recent call last):
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\site-packages\fake_useragent\utils.py", line 154, in load
for item in get_browsers(verify_ssl=verify_ssl):
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\site-packages\fake_useragent\utils.py", line 99, in get_browsers
html = html.split('')[1]
IndexError: list index out of range
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 1346, in do_open
h.request(req.get_method(), req.selector, req.data, headers,
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 1285, in request
self._send_request(method, url, body, headers, encode_chunked)
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 1331, in _send_request
self.endheaders(body, encode_chunked=encode_chunked)
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 1280, in endheaders
self._send_output(message_body, encode_chunked=encode_chunked)
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 1040, in _send_output
self.send(msg)
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 980, in send
self.connect()
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 1447, in connect
super().connect()
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 946, in connect
self.sock = self._create_connection(
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\socket.py", line 844, in create_connection
raise err
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\socket.py", line 832, in create_connection
sock.connect(sa)
socket.timeout: timed out
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\site-packages\fake_useragent\utils.py", line 64, in get
with contextlib.closing(urlopen(
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 214, in urlopen
return opener.open(url, data, timeout)
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 517, in open
response = self._open(req, data)
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 534, in _open
result = self._call_chain(self.handle_open, protocol, protocol +
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 494, in _call_chain
result = func(*args)
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 1389, in https_open
return self.do_open(http.client.HTTPSConnection, req,
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\urllib\request.py", line 1349, in do_open
raise URLError(err)
urllib.error.URLError:
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "E:\insta crawler\crawler.py", line 82, in
get_posts_by_user(
File "E:\insta crawler\crawler.py", line 27, in get_posts_by_user
ins_crawler = InsCrawler(has_screen=debug)
File "E:\insta crawler\inscrawler\crawler.py", line 68, in init
self.browser = Browser(has_screen)
File "E:\insta crawler\inscrawler\browser.py", line 25, in init
chrome_options.add_argument("user-agent="+UserAgent().random)
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\site-packages\fake_useragent\fake.py", line 69, in init
self.load()
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\site-packages\fake_useragent\fake.py", line 75, in load
self.data = load_cached(
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\site-packages\fake_useragent\utils.py", line 250, in load_cached
update(path, use_cache_server=use_cache_server, verify_ssl=verify_ssl)
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\site-packages\fake_useragent\utils.py", line 245, in update
write(path, load(use_cache_server=use_cache_server, verify_ssl=verify_ssl))
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\site-packages\fake_useragent\utils.py", line 187, in load
ret = json.loads(get(
File "C:\Users\8thstar\AppData\Local\Programs\Python\Python39\lib\site-packages\fake_useragent\utils.py", line 84, in get
raise FakeUserAgentError('Maximum amount of retries reached')
fake_useragent.errors.FakeUserAgentError: Maximum amount of retries reached

You can do this: pip install --upgrade fake-useragent


Augment H3Context to improve event.context typing

Describe the feature
With 1.8.0 can now have better typing for query parameters and body content using eventHandler generic.
We also have a proper way to define per event middlewares stack using onRequest
The issue is that it's difficult to type event context.
We can use typescript interface augmentation, but this impact all events in the project:
declare module 'h3' {
  interface H3EventContext {
    auth: { name: string }
  }
}

export default defineEventHandler(async (event) => {
  event.context.auth.name // string
}

Before 1.8.0, we could create a custom eventHandler wrapper and use an extended version of H3Event:
import type { H3Event, H3EventContext } from 'h3'

// custom event
interface H3CustomEvent extends H3Event {
  context: H3EventContext & {
    auth: { name: string }
  }
}

// custom event handler
function defineCustomEventHandler<T>(handler: (event: H3CustomEvent) => T) {
  return defineEventHandler(async (event) => {
    event.context.auth = { name: 'username' }
    eturn handler(event as H3CustomEvent)
  }
})

// usage
export default defineCustomEventHandler(async (event) => {
  event.context.auth.name // string
}

Same wrapper with `1.8.0`
// EventHandlerRequest, EventHandlerResponse, EventHandler are not exported
interface EventHandlerRequest {
  body?: any
  query?: QueryObject
}
type EventHandlerResponse<T = any> = T | Promise<T>

interface EventHandler<
  Request extends EventHandlerRequest = EventHandlerRequest,
  Response extends EventHandlerResponse = EventHandlerResponse,
> {
  __is_handler__?: true
  (event: H3Event<Request>): Response
}

// custom event
interface H3CustomEvent<T extends EventHandlerRequest = EventHandlerRequest>
  extends H3Event<T> {
  context: H3EventContext & {
    auth: { name: string }
  }
}

// custom event handler
export function customEventHandler<
  Request extends EventHandlerRequest = EventHandlerRequest,
  Response extends EventHandlerResponse = EventHandlerResponse,
>(
  handler: EventHandler<
    Request extends EventHandlerRequest ? Request : EventHandlerRequest,
    Request extends EventHandlerRequest ? Response : Request
  >,
): EventHandler<
  Request extends EventHandlerRequest ? Request : EventHandlerRequest,
  Request extends EventHandlerRequest ? Response : Request
> {
  //
  return eventHandler((event) => {
    event.context.auth = { name: 'username' }

    return handler(event as H3CustomEvent<Request>)
  })
}

// usage
export default customEventHandler((event) => {
  event.context.auth.name // string
})

Since usage of generics (and methods overloading), it's harder to wrap.
How about extending context with EventHandlerRequest, same as query and body like:
const auth = defineRequestMiddleware<{
  context: {
    auth: {
      name: string
    }
  }
}>((event) => {
  event.context.auth.name // string
})

export default eventHandler<{
  context: {
    auth: {
      name: string
    }
  }
}>({
  onRequest: [auth],
  async handler(event) {
    event.context.auth.name // string
  },
})

Additional information

[x] Would you be willing to help implement this feature?

Having context in request types is a good idea!
We had been thinking with @danielroe to consider doing this. (as part of #496 to also allow typing validated read body into event.context.body)


[FLINK-14972] Make Remote(Stream)Environment use Executors.

What is the purpose of the change
This PR makes the RemoteEnvironment (batch) and the RemoteStreamEnvironment use the newly introduced Executors.
Verifying this change
This change is already covered by existing tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@tweise this PR removes some unused methods from the RemoteStreamEnvironment that were introduced in the context of FLINK-11048.
I believe that with the newly introduced executors, these methods are obsolete. Please let me know what you think.

@aljoscha I integrated your comments.

I addressed the comments. Let me know if now it is good to merge @TisonKun and @aljoscha

I rebased to the master.

Thanks a lot @TisonKun and also I believe that now with the Executors, the reason why this method was introduced does not exist anymore, so it is safe to remove. Even if we re-add the method, it is going to simply forward the call to the adequate executor.

OK make sense. Possibly we can remote RemoteExecutor extends PlanExecutor since it is no more in used. I think eventually we have executors in this new inheritance hierarchy and don't need PlanExecutor any more.

@TisonKun The old RemoteExecutor is still used by the ScalaShell and that is why it is not removed in this PR. As soon as I refactor also the shell to use executors (work-in-progress), I will also remove that.

@tweise this PR removes some unused methods from the RemoteStreamEnvironment that were introduced in the context of FLINK-11048.
I believe that with the newly introduced executors, these methods are obsolete. Please let me know what you think.

LGTM. However you should probably tag the commits with a JIRA or squash them, changes of this type are IMO not "hotfix".

Hi @kl0u @aljoscha FLINK-14992 is waiting for this pull request. I don't see a blocker prevents this pr from merging. Could you take a look? I'm anyway start the work of FLINK-14992 tomorrow, which would conflict with the pull request.

Thanks for the comments @TisonKun @aljoscha and @tweise . Merged


Bump version to v0.24.0 + deprecations

What does this PR do?
Bumps to 0.24.0 and deprecates some code paths, notably passing process_group and device_mesh directly in the fsdp config object.
Dailies succeeded: https://github.com/mosaicml/composer/actions/runs/10528606007
Continuation of #3527

What issue(s) does this change relate to?

Before submitting

[ ] Have you read the contributor guidelines?
[ ] Is this change a documentation change or typo fix? If so, skip the rest of this checklist.
[ ] Was this change discussed/approved in a GitHub issue first? It is much more likely to be merged if so.
[ ] Did you update any related docs and document your change?
[ ] Did you update any related tests and add any new tests related to your change? (see testing)
[ ] Did you run the tests locally to make sure they pass?
[ ] Did you run pre-commit on your change? (see the pre-commit section of prerequisites)

Hello! The released was prepared in this MR but no tag was added. What would it take to complete the publication of this version?

@antoinebrl We'll be releasing the new version today -- just didn't want to release over the weekend :)


[executor-bench] make db-generator output more readable

Motivation
It is hard to track the slow progress when creating db, so use a progress bar instead of diem logger.
Have you read the Contributing Guidelines on pull requests?
Y
Test Plan
cargo run -- create-db --data-dir=/tmp/test --num-accounts=1000

/land


Method not found  ./omnicore-cli listunspent

Hi guys,
I got Method not found when ./omnicore-cli listunspent
$ ./omnicore-cli listunspent 0 999 '["mu3jCisS49s9GT8oBtrAUKpXMVBrFgjYWt"]'

error code: -32601
error message:
Method not found

omni_getinfo
$ ./omnicore-cli "omni_getinfo"

{
  "omnicoreversion_int": 30000000,
  "omnicoreversion": "0.3.0",
  "mastercoreversion": "0.3.0",
  "bitcoincoreversion": "0.13.2",
  "block": 536999,
  "blocktime":<PHONE_NUMBER>,
  "blocktransactions": 19,
  "totaltrades": 1689,
  "totaltransactions": 2848063,
  "alerts": [
  ]
}

Any advices?

How did you resolve this issue?

recompile & restart, done


Page anchors XHR request fails at least when field is used on DataObject via ModelAdmin

Fixed in 2.1.0


TransformAnimator: doesn't clear callback

https://github.com/ukonpower/ore-three-ts/commit/e0eeb0325ad6438b632a912b912e18ce4f8f6977


[Currency Bot] Bumped tedious from 15.1.3 to 18.2.0

Tada!

Closing because of https://github.com/instana/nodejs/commit/c99f0e02a4254f4d4ef558d9b0a6b5510384d862
We will manually update it soon


Order of elements in data

Hi, this looks very promising, and I am at the moment just playing around to evaluate its usefulness for my projects.
I looked at the TEI header demo and played around, for example removing and adding an .  The resulting  on the same demopage now show the following:  (sorry, the formatting seems to be off, but that is due to githubs mangling...)
`

Main title for the electronic version of the text
One or more additional titles, which may have a @type
Rudi Rüssel
            <editor>Heinz Hund</editor>
        <author>Christian Wittern</author></titleStmt>
        <publicationStmt>
            <publisher>Name of the institution publishing the electronic version</publisher>
            <date when="2021-09-24"/>
            <availability>
                <licence target="http://creativecommons.org/licenses/by/3.0/">Creative Commons Attribution
                    ShareAlike 3.0 License
                </licence>
            </availability>
        </publicationStmt>
        <sourceDesc>

A free text paragraph describing the manuscript used

        </sourceDesc>
    </fileDesc>
    <profileDesc>
        <langUsage>
            <language ident="de"/>
        <language ident="eng"/></langUsage>
        <textClass>
            <keywords>
                <term>XML</term>
                <term>TEI</term>
            </keywords>
        </textClass>
    </profileDesc>
    <revisionDesc>
        <change when="2021-05-31T09:04:25+02:00">teiHeader was automatically produced from the database</change>
    </revisionDesc>
</teiHeader>
<text>

Some text here.

</text>

`
The new author is appended to the end of the element, while the visual editing has the editors after the authors.  My question is:  Is this a bug or a feature? Can this be adjusted?
I would expect the order of the elements be the same in both cases.
All the best,
Chris

you're right Chris. This is still a glitch in the implementation. The demo uses the older fx-append action which is not accurate. There no if fx-insert action which allows much more flexibility and should handle this case. I leave this ticket open til i've re-written the example to use fx-insert

Ah, I see, thanks for pointing this out.  But this means that the code itself allows for this case, which is good news.  Keep up the good work, this looks very promising and has the potential to solve a lot of headaches for me:-)

will need to add a couple of insert tests.


consider OSC 777 and OSC 9, OSC 99

these are special escape sequences you output to the terminal
then your terminal emulator can handle the notification (either by itself, or forwarding to your system notifications)
wezterm, foot, rxvt support 777
kitty uses 99 https://sw.kovidgoyal.net/kitty/desktop-notifications/

Looks like they don't support icons or priorities so I don't know if it's worth implementing.

I can certainly add support for urgency level to OSC 99 however icons
are more problematic. How would you expect them to work? Paths wont work
over SSH for example.

I suppose SSH is a valid use case (will still be missing priorities though). I'll look into it today.

And I added support for urgency to OSC 99: https://github.com/kovidgoyal/kitty/commit/865f66221667117ab4d555bfd4bb81b4856f2a18

And I added support for urgency to OSC 99: kovidgoyal/kitty@865f662

Will it throw an error if the urgency is specified in an older version?

On Thu, May 16, 2024 at 09:15:49AM -0700, ObserverOfTime wrote:

And I added support for urgency to OSC 99: @.***(https://github.com/kovidgoyal/kitty/commit/865f66221667117ab4d555bfd4bb81b4856f2a18)

Will it throw an error if the urgency is specified in an older version?

No.

Implemented in 4ae7b6278f1699e3e3500acc4824ea3907660d08. Must be enabled by setting vim.g.nvimcord_use_osc = ... (replace ... with '777', '99', or '9') before calling nvimcord.setup. I will not add it to the setup opts or make any attempt to detect the terminal.

Typo.

all good


[iceberg] Introduce procedure to migrate table from iceberg to paimon

Purpose
【Not completed！！】

Linked issue: close #xxx

Tests

API and Format

Documentation

Please write [WIP] in PR title.


Option to share the circuit designed by the respective user

When anyone save the circuit he/she must have an option to share the circuit with read permission only.
I would like to work on the above mentioned project .

@iCoder-007, unless someone is a collaborator, he or she cannot edit the circuit.

But let say , i am the owner of the circuit and to share someone the circuit with read only permission

@iCoder-007 we can have a share button but I am not sure what exactly it will do. Right now unless the project is private, you can just share the URL of the project with your friend

Ya we can a share option were the user can share the circuit as image or something like that

@iCoder-007 if u just want to share the circuit as an image, just download it and share? Tools -> Download Image

ok ,sorry i didn't saw that option


UI Issue CSS Failure?

I follow all the guide here, and it seems the component is there but looks like hell, I imported all the requied styles in the scss file, but still failing.
Im using angular 5 with angular CLI 1.6.5, node ver 8.9.3
Any Ideas?

fixed, I added the import to the global scss instead of my-component.scss.
Thanks


Provide stronger wording for end retro consequences

Feature Request
Feedback has been provided stating that a user ended the retro not knowing the consequences which resulted in all feedback
Story
As a user of RetroQuest
I want to be clearly communicated of the implications of ending a retro
So that I do not accidentally loose the content for the team
Acceptance Criteria
Given a user select the end retro button
When the user is prompted with the message asking them to confirm the input
Then wording is provided indicating to the user that all thoughts will be cleared after ending the retro
Design
Current UI Design

Enhanced UI Design
The new text in the dialog should state Do you want to end the retro?  Ending a retro will clear away all thoughts!

Update merged and deployed.


Configuration drift

[ ] use terraform import
[ ] purposely cause confihuration drift via clickops, and correct state


Preserve RecordId's during import

There is a flag -preserveClustIDs (see http://orientdb.com/docs/2.1/Console-Command-Import.html). It would be nice if we could also preserve all ids within a cluster.

:+1: :+1:
This would be imperative for us for our planned database migration processes and I thought it would be possible. It would also be needed that the id sequence should start after the last (highest number) imported id again. Could that also be possible?
Scott

@smolinari  I know a workaround for this is to restore a backup instead of doing an import.

Our intentions though aren't to backup data or restore it, but rather to migrate portions of a database  to a new database.
Scott

This is not supported by design, sorry.


return the wrapped function from component decorators

fixes #1092

Codecov Report

Merging #1093 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1093      +/-   ##
==========================================
- Coverage   41.85%   41.85%   -0.01%
==========================================
  Files          69       69
  Lines       13202    13204       +2
  Branches     2204     2204
==========================================
  Hits         5526     5526
- Misses       7252     7254       +2
  Partials      424      424

Impacted Files
Coverage Δ

autobahn/wamp/component.py
38.84% <0%> (-0.21%)
:arrow_down:

Continue to review full report at Codecov.

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

@oberstet this is a simple fix, should go in next release


atan2 incorrect for some inputs

Dear maintainers, I am so happy to see Corman Common Lisp live on! I had an issue with atan2 a while ago (2012 if you'd believe me) - and I dug it up since I remembered it. I've attached a diff (zipped) for what worked well for me - would you incorporate/improve it please?
I haven't been able to build Corman Lisp yet, and would like to discuss that if possible too. Do you have an active mailing list or forum? I just posted on https://groups.google.com/forum/#!forum/cormanlisp
math-ops atan2.zip

Thank you for your interest in Corman Lisp!
I have replied to you in the Google Group. Unfortunately, we do not have an active mailing list or forum yet.
May I ask you to provide some code which demonstrates the problem? It would simplify the investigation.

Sure thing. This code loops around a unit circle from 0° to 360° in 15° steps. The last two numbers show the starting angle, and the reconstructed angle using atan. This is run on v3.1.2
(loop for ang from 0 to (* 2 pi) by (/ pi 12)       for deg = (round (* ang (/ 180 pi)))       for x = (cos ang)       and y = (sin ang)       for atan-ang = (atan y x)       for atan-deg = (round (* atan-ang (/ 180 pi)))   do (format t "~&Ang = ~,3Fr  pt = (~5,2F, ~5,2F) start deg = ~3D, atan = ~3D~%" ang x y deg atan-deg)) Ang = 0.000r  pt = ( 1.00,  0.00) start deg =   0, atan =   0 Ang = 0.262r  pt = ( 0.97,  0.26) start deg =  15, atan =  15 Ang = 0.524r  pt = ( 0.87,  0.50) start deg =  30, atan =  30 Ang = 0.785r  pt = ( 0.71,  0.71) start deg =  45, atan =  45 Ang = 1.047r  pt = ( 0.50,  0.87) start deg =  60, atan =  60 Ang = 1.309r  pt = ( 0.26,  0.97) start deg =  75, atan =  75 Ang = 1.571r  pt = ( 0.00,  1.00) start deg =  90, atan =  90 Ang = 1.833r  pt = (-0.26,  0.97) start deg = 105, atan = -75 Ang = 2.094r  pt = (-0.50,  0.87) start deg = 120, atan = -60 Ang = 2.356r  pt = (-0.71,  0.71) start deg = 135, atan = -45 Ang = 2.618r  pt = (-0.87,  0.50) start deg = 150, atan = -30 Ang = 2.880r  pt = (-0.97,  0.26) start deg = 165, atan = -15 Ang = 3.142r  pt = (-1.00, -0.00) start deg = 180, atan =   0 Ang = 3.403r  pt = (-0.97, -0.26) start deg = 195, atan =  15 Ang = 3.665r  pt = (-0.87, -0.50) start deg = 210, atan =  30 Ang = 3.927r  pt = (-0.71, -0.71) start deg = 225, atan =  45 Ang = 4.189r  pt = (-0.50, -0.87) start deg = 240, atan =  60 Ang = 4.451r  pt = (-0.26, -0.97) start deg = 255, atan =  75 Ang = 4.712r  pt = (-0.00, -1.00) start deg = 270, atan =  90 Ang = 4.974r  pt = ( 0.26, -0.97) start deg = 285, atan = -75 Ang = 5.236r  pt = ( 0.50, -0.87) start deg = 300, atan = -60 Ang = 5.498r  pt = ( 0.71, -0.71) start deg = 315, atan = -45 Ang = 5.760r  pt = ( 0.87, -0.50) start deg = 330, atan = -30 Ang = 6.021r  pt = ( 0.97, -0.26) start deg = 345, atan = -15 Ang = 6.283r  pt = ( 1.00, -0.00) start deg = 360, atan =   0 NIL
The existing atan only works in the first quadrant, and I think needs updating like in my diff.

Thanks, that is a good catch. With your fix Corman Lisp produces the same results as SBCL.


policy-compiler: add data_location field in egeria input asset json

This PR fixes #122. We have added a "data_location" field (which is mandatory) in the asset json which is input to Egeria during configuration time. This field needs to be populated with the correct data_location of the asset and will be in turned parsed by egeria connector and set in the geo field of the data catalog response https://github.com/IBM/the-mesh-for-data/blob/89a5359b4f4c911eb366bfaa8612f25b919b0354/pkg/connectors/protos/data_catalog_response.proto#L54 which is sent to the manager. The manager further will process the geo field accordingly.

Please modify this line as well and set geography to US:
https://github.com/IBM/the-mesh-for-data/blob/89a5359b4f4c911eb366bfaa8612f25b919b0354/samples/kubeflow/example_transactions.csv.json#L5

Fixed. Please verify. Thanks.


Validators self-delegate is over 100%

Currently some validators are showing more than 100% self delegate percentages:

@RiccardoM This might actually be an issue with bdjuno
Ex for blockscape desmosvalcons1d8ld00spt08rrfgnarar6kp6hwafwy2urnhs2l their voting power is at 119804 on height 15437

And their delegations is<PHONE_NUMBER>01 udaric on height 15442 and<PHONE_NUMBER>60 on height 15436 we didn't have a record for height 15437

even if they were both at height 119804
Voting power 119804 and delegations<PHONE_NUMBER>60 udaricwould've surpassed 100%.
What i can do is check if the percentage is over 100% i will just display the following as 100% instead.

update:
I'll just calculate within the delegations module. should be able to get 100%


Allow Adding high level links

Is your feature request related to a problem? Please describe.
It is currently not possible to add a link in the app for example to logout the admin user.
Describe the solution you'd like
Add a configuration that would allow dynamically adding links top most level again example ( sign out, settings, etc)
Describe alternatives you've considered
A way to override the default template ?
Express the feature either with a change to resource syntax, or with a change to the resource interface
For example
  admin do
    links do
       link :settings, "/settings"
    end
  end

On Second thoughts, I think there should be a specifc AshAdmin module with those configurations.

Yeah, so in the long run we need to create something like this:
defmodule MyApp.AdminUI do
  use AshAdmin.UI

  ...
end

Which we could put custom links in at that point. We would also use that to then explicitly configure the domains and resources. You could then have multiple admin UIs w/ different configurations.

@zachdaniel I am learning a bit of spark and will try to tackle this one along with file uploads in ash admin ;) .. hopefully will have a couple of PRs soon

That sounds great! I put it on the roadmap for "someday", but would be very happy to see it done much earlier <3

one of my clients need it ... can't think of a better reason to make it happen now .. will give it a shot .. if it wasn't for that it would have been at the top of my someday too as it looks like a good first ticket for me to try .. will check spark .. make an attempt and see your feedback .. I am sure I will learn something getting it working if i can ;)


estimator applying preprocessing standardisation to pytorch model causes broadcasting error

Describe the bug
Hi, I think I've found a bug where preprocessing won't work with a PyTorch model.

But if I instead permute the batch so the dimensions are in NHWC order, the attack fails because the batch can't be passed to the model: torch raises an error about invalid sizes
To Reproduce
I will paste a script below that's an attempt at a minimal viable example, along with tracebacks I get when I do or do not permute the batch
Expected behavior
Preprocessing standardisation to be applied without error
System information (please complete the following information):

OS: Ubuntu
Python version: 3.6
ART version or commit number: 1.3.1 from pypi
TensorFlow / Keras / PyTorch / MXNet version: torch 1.5.0 from pypi, torchvision 0.6.0

minimal viable example
first I will post a script to reproduce, then below that tracebacks I get with / without permuting
from art.estimators.classification import PyTorchClassifier
from art.attacks.evasion.carlini import CarliniL2Method
import numpy as np
import torch
from torch.optim import SGD
import torch.nn as nn
from torchvision import models, datasets, transforms

def get_default_device():
    if torch.cuda.is_available():
        return 'cuda'
    else:
        return 'cpu'

DEVICE = get_default_device()
print(f'device: {DEVICE}')

MEAN = [0.485, 0.456, 0.406]
STD = [0.229, 0.224, 0.225]
normalize = transforms.Normalize(mean=MEAN,
                                 std=STD)

transform = transforms.Compose([
        transforms.RandomResizedCrop(224),
        transforms.RandomHorizontalFlip(),
        transforms.ToTensor(),
        normalize,
    ])

dataset = datasets.ImageFolder(root='/home/ubuntu/data/imagenette2/train', transform=transform)  # fastai imagenette

dataloader = torch.utils.data.DataLoader(
    dataset, batch_size=64, shuffle=False,
    num_workers=4, pin_memory=True)

resnet = models.resnet50(pretrained=True)
resnet.to(DEVICE);

# run a few batches through with model in `train` mode to so `batchnorm` stats have a good value.
# see: https://discuss.pytorch.org/t/model-eval-gives-incorrect-loss-for-model-with-batchnorm-layers/7561/3
N_BATCH = 100

for batch_num, batch in enumerate(dataloader):
    if batch_num > N_BATCH:
        break
    with torch.no_grad():
        x, y = batch[0].to(DEVICE), batch[1].to(DEVICE)
        out = resnet(x)

resnet.eval();

# do Carlini Wagner attack
optimizer = SGD(resnet.parameters(), lr=0.001)
criterion = nn.CrossEntropyLoss()

clf_art = PyTorchClassifier(
    model=resnet,
    clip_values=(0., 1.),
    loss=criterion,
    optimizer=optimizer,
    input_shape=(3, 32, 32),
    nb_classes=10,
    preprocessing=(np.asarray(MEAN), np.asarray(STD))
)

cw_attack = CarliniL2Method(classifier=clf_art,
                            batch_size=64)

accs = []

for batch in dataloader:
    with torch.no_grad():
        x, y = batch[0], batch[1].to(DEVICE)
        # x = x.permute(0, 2, 3, 1)
        x_attacked = cw_attack.generate(x)
        out = resnet(x_attacked.to(DEVICE))
        y_pred = out.argmax(dim=1)
        accs.append(
            (y_pred == y).sum().cpu().numpy() / y.shape[0]
        )

without permuting this causes a broadcasting error
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-26-d6037ab10c77> in <module>
      5         x, y = batch[0], batch[1].to(DEVICE)
      6         # x = x.permute(0, 2, 3, 1)
----> 7         x_attacked = cw_attack.generate(x)
      8         out = resnet(x_attacked.to(DEVICE))
      9         y_pred = out.argmax(dim=1)

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/art/attacks/attack.py in replacement_function(self, *args, **kwargs)
     72                 if len(args) > 0:
     73                     args = tuple(lst)
---> 74                 return fdict[func_name](self, *args, **kwargs)
     75
     76             replacement_function.__doc__ = fdict[func_name].__doc__

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/art/attacks/evasion/carlini.py in generate(self, x, y, **kwargs)
    237         # No labels provided, use model prediction as correct class
    238         if y is None:
--> 239             y = get_labels_np_array(self.estimator.predict(x, batch_size=self.batch_size))
    240
    241         # Compute perturbation with implicit batching

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/art/estimators/classification/classifier.py in replacement_function(self, *args, **kwargs)
     69                 if len(args) > 0:
     70                     args = tuple(lst)
---> 71                 return fdict[func_name](self, *args, **kwargs)
     72
     73             replacement_function.__doc__ = fdict[func_name].__doc__

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/art/estimators/classification/pytorch.py in predict(self, x, batch_size, **kwargs)
    167
    168         # Apply preprocessing
--> 169         x_preprocessed, _ = self._apply_preprocessing(x, y=None, fit=False)
    170
    171         # Run prediction with batch processing

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/art/estimators/estimator.py in _apply_preprocessing(self, x, y, fit)
    230         # y = check_and_transform_label_format(y, self.nb_classes)
    231         x_preprocessed, y_preprocessed = self._apply_preprocessing_defences(x, y, fit=fit)
--> 232         x_preprocessed = self._apply_preprocessing_standardisation(x_preprocessed)
    233         return x_preprocessed, y_preprocessed
    234

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/art/estimators/estimator.py in _apply_preprocessing_standardisation(self, x)
    282                 div = np.asarray(div, dtype=x.dtype)
    283
--> 284             res = x - sub
    285             res = res / div
    286

ValueError: operands could not be broadcast together with shapes (64,3,224,224) (3,)

obviously caused by this line in Estimator._apply_preprocessing_standardisation
https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/4a74c887388cb428ab6a1c36752a7e13881cecaa/art/estimators/estimator.py#L284
which ideally would not have to check for channel order, but of course this is out of user's hand once everything is wrapped up in an estimator being used by an attack
with permuting, this causes torch to raise a RuntimeError and complain about channels
(i.e. to get the error below, uncomment the line # x = x.permute(0, 2, 3, 1))
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
<ipython-input-27-c536e8966ac0> in <module>
      5         x, y = batch[0], batch[1].to(DEVICE)
      6         x = x.permute(0, 2, 3, 1)
----> 7         x_attacked = cw_attack.generate(x)
      8         out = resnet(x_attacked.to(DEVICE))
      9         y_pred = out.argmax(dim=1)

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/art/attacks/attack.py in replacement_function(self, *args, **kwargs)
     72                 if len(args) > 0:
     73                     args = tuple(lst)
---> 74                 return fdict[func_name](self, *args, **kwargs)
     75
     76             replacement_function.__doc__ = fdict[func_name].__doc__

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/art/attacks/evasion/carlini.py in generate(self, x, y, **kwargs)
    237         # No labels provided, use model prediction as correct class
    238         if y is None:
--> 239             y = get_labels_np_array(self.estimator.predict(x, batch_size=self.batch_size))
    240
    241         # Compute perturbation with implicit batching

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/art/estimators/classification/classifier.py in replacement_function(self, *args, **kwargs)
     69                 if len(args) > 0:
     70                     args = tuple(lst)
---> 71                 return fdict[func_name](self, *args, **kwargs)
     72
     73             replacement_function.__doc__ = fdict[func_name].__doc__

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/art/estimators/classification/pytorch.py in predict(self, x, batch_size, **kwargs)
    180
    181             with torch.no_grad():
--> 182                 model_outputs = self._model(torch.from_numpy(x_preprocessed[begin:end]).to(self._device))
    183             output = model_outputs[-1]
    184             results[begin:end] = output.detach().cpu().numpy()

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/torch/nn/modules/module.py in __call__(self, *input, **kwargs)
    548             result = self._slow_forward(*input, **kwargs)
    549         else:
--> 550             result = self.forward(*input, **kwargs)
    551         for hook in self._forward_hooks.values():
    552             hook_result = hook(self, input, result)

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/art/estimators/classification/pytorch.py in forward(self, x)
    652
    653                         elif isinstance(self._model, nn.Module):
--> 654                             x = self._model(x)
    655                             result.append(x)
    656

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/torch/nn/modules/module.py in __call__(self, *input, **kwargs)
    548             result = self._slow_forward(*input, **kwargs)
    549         else:
--> 550             result = self.forward(*input, **kwargs)
    551         for hook in self._forward_hooks.values():
    552             hook_result = hook(self, input, result)

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/torchvision/models/resnet.py in forward(self, x)
    218
    219     def forward(self, x):
--> 220         return self._forward_impl(x)
    221
    222

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/torchvision/models/resnet.py in _forward_impl(self, x)
    201     def _forward_impl(self, x):
    202         # See note [TorchScript super()]
--> 203         x = self.conv1(x)
    204         x = self.bn1(x)
    205         x = self.relu(x)

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/torch/nn/modules/module.py in __call__(self, *input, **kwargs)
    548             result = self._slow_forward(*input, **kwargs)
    549         else:
--> 550             result = self.forward(*input, **kwargs)
    551         for hook in self._forward_hooks.values():
    552             hook_result = hook(self, input, result)

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/torch/nn/modules/conv.py in forward(self, input)
    347
    348     def forward(self, input):
--> 349         return self._conv_forward(input, self.weight)
    350
    351 class Conv3d(_ConvNd):

~/anaconda3/envs/pimeet2009/lib/python3.7/site-packages/torch/nn/modules/conv.py in _conv_forward(self, input, weight)
    344                             _pair(0), self.dilation, self.groups)
    345         return F.conv2d(input, weight, self.bias, self.stride,
--> 346                         self.padding, self.dilation, self.groups)
    347
    348     def forward(self, input):

RuntimeError: Given groups=1, weight of size [64, 3, 7, 7], expected input[64, 224, 224, 3] to have 3 channels, but got 224 channels instead

Thank you again @beat-buesser I confirmed that this does resolve my issue, so I will go ahead and close.
Need to survive getting results for a deadline but will be back in touch about implementing a solution involving the channels_first attribute


Dynamic FormArray with FormGroups inside

Hi,
I want to do something like this:
this.myForm = this.fb.group({
  'myArray': this.fb.array([
    this.fb.group({
      'name': ['', Validators.required],
      'age': ['', Validators.required],
    }),
    this.fb.group({
      'name': ['', Validators.required],
      'age': ['', Validators.required],
    }),
  ]),
});

And i want to add FormGroups to the array dynamically.
Is this possible? I have found some other issues that asked similar questions but I couldn't figure this out.
Any help would be very much appreciated!
Thanks!

The dynamic example in the example application shows exactly how to add and remove controls. Basically do something like this:

interface MyFormValue {
  myArray: MyArrayElementFormValue[];
}

interface MyArrayElementFormValue {
  name: string;
  age: number;
}

const initialFormState = createFormGroupState<MyFormValue>('form ID', {
  myArray: [],
});

// here is the validation function that applies the `required` validation function
// to the properties of each array element (
const validateForm = updateGroup<MyFormValue>({
  myArray: updateArray<MyArrayElementFormValue>(updateGroup<MyArrayElementFormValue>({
    name: validate<string>(required),
    age: validate<number>(required),
  })),
});

// then you can use actions or update functions to add/remove array elements
const formStateWithSingleControlThroughFunction = updateGroup<MyFormValue>(initialFormState, {
  myArray: addArrayControl<MyArrayElementFormValue>({
    name: 'Jon Doe',
    age: 42,
  }),
});

const formStateWithSingleControlThroughAction = formGroupReducer(
  initialFormState,
  new AddArrayControlAction<MyArrayElementFormValue>(initialFormState.controls.myArray.id, {
    name: 'Jon Doe',
    age: 42,
  }),
);

@tprjd did my example help you or do you require anything else? If not, please close this issue.

@MrWolfZ Yes, thanks you very much. You made it easy to understand.


kamelet: allow to add additional attributes to in/out

As today, a kamelet has the option to define the types it supports but that is limited to the following information:
type EventTypeSpec struct {
	// media type as expected for HTTP media types (ie, application/json)
	MediaType string `json:"mediaType,omitempty"`
	// the expected schema for the event
	Schema *JSONSchemaProps `json:"schema,omitempty"`
}

Given that a kamelet may be more complex and the tooling using them may need more information it would be nice to have the option to add additional metadata, as example, in some case it may be useful to add an information about what is the Java class the output of a camel is build upon:
spec:
  out:
    mediaType: "accplication/json"
    annotations:
        contentClass: com.acme.Foo

This may be useful for tools as they may compute the scheme of the payload.

I think partially this has been done already with #4100
The Java contentClass class information that is explicitly mentioned in this issue is missing though in the DataTypes section in Kamelet spec AFAIK.
Information such as the Java contentClass may be given on the individual schema resolver step using the resolve-pojo-schema-action Kamelet. The idea is to resolve the schema for a contentClass (delegating to Avro, Protobuf, ...) in a first step and the have the information available to serialize/deserialize the content using one of the serdes action Kamelets (e.g. avro-deserialize-action
@lburgazzoli WDYT?

If I properly recall the problem, here the idea was more at design time, so to provide hints for tooling  i.e. if you have a data mapper then you can use the schema in the mapper. But I think we can close this issue and eventually open a new and more detailed one.


Add runtime typecheck support for HTMLElement JSDoc type annotations

Currently the runtime typechecker will throw an error if HTMLElement is specified within a type annotation, eg:
/**
 * @param   {HTMLElement} element
 * @returns {void}
 */
function foo(element) {
    ...
}

We should allow specifying this as a type within annotations (and typechecking it at runtime). This probably means adding it as an enum value to Enum_Jsdoc_Types and doing some funky if...else logic in the typechecker.

Fixed in daddc8c


productivity: Create "Directory Navigation" subcategory

Hello! I noticed that the Command-Line Productivity list is quite large, which makes it a bit difficult to explore if you are looking for a specific set of utilities. As a first try, I created a subcategory for utilities related to directory navigation, which groups together 12 of the ~90 items in the list. Let me know if you have any comments/suggestions and I can rebase!

I agree with you there. Thanks for taking the time to sort through it. 🎉


Windows to Linux build warning to stdout

Signed-off-by: John Howard<EMAIL_ADDRESS>@thaJeztah Fixes https://github.com/docker/docker/issues/29209. Writes the warning when building from Windows to non-Windows go to stdout rather than stderr.

Oh, LOL, just opened https://github.com/docker/docker/pull/29857; GitHub didn't show yours

Let me know which one you like more :joy:

Haha. Beat you (just). But you know you had gofmt errors on yours....  Also there's some improvement to the logic in the tests which I did...

doh! let me close mine
would it make sense to use my commit message to give context to this change?
When building a Dockerfile from a Windows client
on a Linux daemon, a "security warning" is printed
on stderr.

Having this warning printed on stderr makes it
difficult to distinguish a failed build from
one that's succeeding, and the only way to
suppress the warning is through the `-q` option,
which also suppresses every output.

This change prints the warning on stdout,
instead of stderr, to resolve this situation.

@thaJeztah Updated

Oh, sorry, meant the actual commit message, as GitHub isn't always around to check 😇

as GitHub isn't always around to check

I really hope that isn't so - there would be a terrific amount of history lost in the repo if that were the case, it's certainly not the case for many, many PRs. Nonetheless done.

Yup, we need to pay more attention to that in general, as the actual commit message is what's in the repository's history. Sorry for being picky ☺


Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
ImageMagick.ImageMagick is not installed successfully
Steps to reproduce
winget install ImageMagick
Actual behavior
An error occurs during installation:

Found ImageMagick [ImageMagick.ImageMagick] Version <IP_ADDRESS>
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.imagemagick.org/ImageMagick/download/binaries/ImageMagick-7.1.0-37-Q16-HDRI-x64-dll.exe
An unexpected error occurred while executing the command:
Download request status is not success.
0x80190194 : Not found (404).

Expected behavior
The package is installed successfully
Environment
Windows Package Manager v1.2.11601
Windows: Windows.Desktop v10.0.19042.1766
Package: Microsoft.DesktopAppInstaller v1.17.11601.0

Screenshots and Logs
No response

I don't find this version on the official website either, but a newer version has been already released, namely ImageMagick-7.1.0-39-Q16-HDRI-x64-dll.exe


Add unsafe cpu::start_app_cpu() function

This requires https://github.com/sapir/xtensa-lx6-rt/tree/rom-symbols

This is already part of my pull request #15, which does not rely on the ROM routines.
@sapir I would propose to merge #15 and not use this one, does that make sense?

Yes, obviously I didn't notice your pull requests


Use throwIO instead of throw

When in IO, throwIO should be used, as per the documentation.

Sigh, I was unable to submit a separate pull request for this commit, which removes the need for issue 13. So it's in this one instead.

I merge the changes in develop branch. Kudos for the OpenCL headers.
p.s: I don't know how to set I already merged the changes in github. I close this one, but I have already get into develop branch.

Thanks :)


Doesn't work with minification.

For example, when using Closure Compiler, the code changes module.exports to module.c, and when this is run with exec (in loader.js), module.exports is an empty object, leading to errors. Is there a way to run the plugin before minification by ClosureCompilerPlugin?

+1
I been using https://github.com/roman01la/webpack-closure-compiler/
but after minification by gcc in ADVANCED mode, module.exports became module.c, since it wasn't in the form of module['exports'], it returns {} on https://github.com/webpack/extract-text-webpack-plugin/blob/master/loader.js#L97, which gives error

and when i run in SIMPLE mode, it does return the same string as UglifyJS (since module.exports is back now), but then an error occurs, i think with gcc internal code,
Caught error:  [Error: xxx.min.css from Closure Compiler
The compiler is waiting for input via stdin.
stdin:1: ERROR - Parse error. primary expression expected
body,blockquote,code,dd,div,dl,dt,fieldset,form,h1,h2,h3,h4,h5,h6,input,legend,li,ol,......

@quadrupleslap @Maxlufs problem still exists?

Closing due to inactivity. Please test with latest version and feel free to reopen if still regressions. Also please use official closure plugin https://github.com/webpack-contrib/closure-webpack-plugin. Feel free to feedback. Thanks!


Add a progressbar option

Hi,
Is it possible to add a progress bar to see the different part of a process ?
Example: to see the download of a file
Thanks

Can you elaborate? Just one progress bar for all the tasks, or as output of a certain task?

+1
I think that an option to enable progress bar for certain tasks would be great

+1
I use progressbar but listr gets a little crazy and begins to duplicate tasks (he executes them once, but he shows them many times)

Hi,
In my case, it would be to have to progress bar for a certain task.
Some of my jobs are slow and I need to send a feedback to the user. A progress bar would be perfect.
Thanks

In case anyone's looking to implement a progress bar without using observables, here's a basic implementation:
const repeat = (n, char) => [...new Array(Math.round(n))].map(() => char);
const progress = (percentage, size = 20) => {
  const track = repeat(size, " ");
  const completed = repeat((percentage / 100) * size, "=");
  return `${completed.join("")}${track.join("")}`.substr(0, size);
};

Then inside your task function somewhere:
const interval = setInterval(() => {
  task.output = `[${progress(ctx.percentage)}] ${ctx.percentage.toFixed(1)}%`;
  if (ctx.percentage >= 100) clearInterval(interval);
}, 200)

This assumes you're keeping track of progress in ctx.percentage. Of course you could do this without an interval, it totally depends on what you're tracking progress of. Usually there's an update handler of sorts, in which case you don't need the interval.


Fast softmax kernel

We need a better implementation (for CPU backend) for softmax.

runtime
softmax in gpt2 (ms)

onnxruntime
1.5

altius
2.4

The performance degradation was due to OpenMP. OMP_WAIT_POLICY=active solves this.


zone.record seems to be nil

this happens when calling .all
cloudflare-3.2.2/lib/cloudflare/zone.rb:192:in to_s': undefined method []' for nil:NilClass (NoMethodError)
works if preload = true

Can you submit a PR to fix this issue?

I'm refactoring the code, this issue should be mitigated due to the change In underlying code.


fetching branchName for FE2 URLs

This solves the Grasshopper Outdated Receive Component notification issue in FE2, because the branch.name of the commit created on the server should correspond to the StreamWrapper subscription. And so far it was "hacked" to store the branch id as a BranchName for FE2 URLs.
While this is a "proper" way to set the branchName, it might affect other methods using StreamWrapper constructor. Send/Receive was tested and works fine. But if this is too risky, we can switch to another hack checking for the branch.id here instead:

This still has the issue of branch 101+ won't work (@jsdbroughton I know you love this limitation), but I think we can live with this?

fixing the branch limits would be out of scope for this anyways..


MM-32783: header & footer for threads in search

Summary
Display "Thread in $channel" instead of the channel
name above a search result that belongs to a
thread (any message, root or child).
Also show the footer (same as for regular posts),
but only for thread roots.
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/18782
JIRA: https://mattermost.atlassian.net/browse/MM-32783
Screenshots

Release Note
Added thread replies to search results (when collapsible reply threads are enabled)

Thanks for participating in Hacktoberfest! You can claim your sticker set here: https://get.printfection.com/hacktober21/4144583267 @pascalhein

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich @emilyacook

Thanks @pascalhein
This looks great!
Question for UX: @matthewbirtch please let me know your thoughts on the following. Thanks 🙂

This is also added to pinned and saved rhs. Saved and Pinned search results are not specifically mentioned in the ticket, please let me know if this is expected.

Direct Message header:

0/5 I personally expected singular message , instead of messages in  "Thread in Direct Messages(with user-1)"
We should also remove the parenthesis around user name

We are now showing 2 reply arrows on a root post on the RHS, please note they do behave differently and may cause confusion.

This is also added to pinned and saved rhs. Saved and Pinned search results are not specifically mentioned in the ticket, please let me know if this is expected.

@jgilliam17 yes, this is expected

0/5 I personally expected singular message , instead of messages in  "Thread in Direct Messages(with user-1)"
We should also remove the parenthesis around user name

Agree with both of these

We are now showing 2 reply arrows on a root post on the RHS, please note they do behave differently and may cause confusion.

This is a good catch @jgilliam17. I agree this is confusing and we should fix that. if CRT is one, we should hide the second reply arrow on the right. However, we should keep the behavior from that arrow and apply it to the thread footer because it gives users a way back to the search results/mentions/saved list.

I'm not sure if the above is clear, so let me know. Really good catches @jgilliam17 - sorry I missed these in my review.

Hi @jgilliam17 and @matthewbirtch,
thanks for your feedback!
I've uploaded a new version to address your concerns. Please let me know if you have any comments.

/update-branch

/update-branch

Thanks @pascalhein
Manually tested, looks good.
Running E2Es here, will approve after report is posted.


feat: weekly summary

Resolves #45

I would like to know how to test this to be 100% confident of my review but overall it looks pretty good.

I have a small change request as a comment.
I also presume that this is now implemented without CRON, and instead with the post-event wildcard handler.
I also thought i paused whatthediff's access to the repo so I'm surprised that it seems to keep working fine.
Lastly it is not clear to me when is the time that it is able to run. Is it after Monday 0:00 UTC?

I'm using whatthediff's public repo analyzer, which should work regardless.
Use example.ts to test.
It's 00:00Z same as UTC (GitHub uses Z), any day.

I'm using whatthediff's public repo analyzer, which should work regardless.

How did you find this? I spent some time researching this and it never came across my radar!

Use example.ts to test.

I guess we should setup a tests folder and suffix with .t.ts etc.

It's 00:00Z same as UTC (GitHub uses Z), any day.

Its not clear to me then. Does it run daily? Does it have a check to run max once per week? Is it in the wildcard post event handler?

I'm using whatthediff's public repo analyzer, which should work regardless.

How did you find this? I spent some time researching this and it never came across my radar!

it's the public analyzer atwtd@repository-report

It's 00:00Z same as UTC (GitHub uses Z), any day.

Its not clear to me then. Does it run daily? Does it have a check to run max once per week? Is it in the wildcard post event handler?

it has a check to run max once per week, the supabase should provide this
export const getLastWeeklyTime = async (): Promise<number> => {
  const { supabase } = getAdapters();

  const { data } = await supabase.from("weekly").select("last_time").limit(1).single();
  if (data) {
    return Number(data.last_time);
  } else {
    return 0;
  }
};
//src/adapters/supabase/helpers/client.ts

@pavlovcik


dm4f: actions given wrong types after elaboration

Actions are currently given wrong types after elaboration. For instance, in ifc.fst
read : l:label -> Tot (ifc bool)

is elaborated to
IFC.read : l:label -> Tot (ifc bool)

This identity elaboration seems nonsense. I would expect to see
IFC.read : l:label -> IFC bool ...

Seems that at the moment actions are completely broken and can only be faked via reflection.

Guido points out that the actions do work in FStar.DM4F.IntST.fst and indeed for
get : unit -> Tot (st int)

the elaboration has the right type
STINT.get : unit -> STINT int ...

So why are read and get treated differently?

Just checked and in FStar.DM4F.ST.fst the problem does appear:
FStar.DM4F.ST.get : (s:Type -> unit -> Tot (FStar.DM4F.ST.st s s))

This confusion was caused by naming both the module and the effect IFC. Closing.


Variable issue permalink not enable

A have a variable which is a Dynamic permalink.
The template is good but when received there is no link, only text : the permalink is disabled :

Can you tell me what is wrong in the code or How to activate the permalink ?
Windows

MJML V3

<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

OS: Windows]

Browser  Chrome]

Please reach Mailjet support for that. That's not related at all to MJML repository.


docs: Issue templates

Issue #8

We now have labels 🎉 to find out what they are or if you need to add more https://github.com/eddiejaoude/stargate/blob/main/docs/labels.md

Looks good 👍 - yep labels can be added later as mentioned by @Vyvy-vi and already adds value as mentioned by @AllanRegush - great team work 💪

Thank you everyone 👍🏻


Create new TACLS netkan

Even if your submission tested fine locally there are several errors that only the bots will note and reject. This should be expected.
Working on #3960

Please delete TACLS.frozen.
This mod is has a CC-BY license as such you should add the mod's original author to the author field.
This netkan has no abstract and as such users will be unable to tell at-a-glance what this mod's purpose is.
This netkan has no name and as such users will be unable to tell what this mod is. http://puu.sh/oKiSz/fbdc25d9fe.png
Please delete branches after you merge a PR.
This merge is reverted via https://github.com/KSP-CKAN/NetKAN/pull/3963 as the .ckan generated is  grossly unacceptable.

Wow, whoops. Sorry about that.