Fix layout of  in messages

Here's what a message with an unordered list looks like, in my emulator for a Pixel 2 XL running Android 8.1:

There are two issues here:

The bullet glyphs are clipped at the left.
There's excess space between the preceding graf and the first list item.

For comparison, here's the webapp on the same message, if I narrow the window to make the text flow the same way:

I think even that might be improved by slightly less space before the first list item; but we're showing way more than that.
I noticed this while reviewing #2665 , but it was already the same way before that change.

(just a note) The bullet glyphs are similarly cut off in iOS.

@gnprice I think this issue can be closed.

This was indeed fixed by #2687. Closing.


launch: add px4.launch script that only starts the px4 binary

This provides a clean ROS interface for external projects which is independent from the px4 CLI interface.
It can be used with:
        <include file="$(find px4)/launch/px4.launch">
        </include>

@mhkabir this should fix your ROS issues.

Does this need docs?

Probably, I don't know if we already have some on how to use PX4 with ROS.

Well, we have some brief mention of launch files here: http://dev.px4.io/en/simulation/ros_interface.html - something to hand over to @TSC21 ?

Thanks a lot!


Health Check which require dependencies via DI

Hi
I am new to App Metrics and have a question. I have a project on .net core 2.1.0.
I have a health check which has external dependencies which need to be loaded by the dependency injector. How do I go about adding this Health Check? See below.
I have been able to register it by using RegisterFromAssembly but would prefer to be more specific about the health check I want loaded (I dont want all health checks in all assemblies to be loaded, but only the ones specified).
Thanks
Andrew
using App.Metrics.Health;
using System;
using System.Threading;
using System.Threading.Tasks;

namespace Aerium.Velocity.Infrastructure.HealthChecks
{
    public class MassTransitHealthCheck : HealthCheck
    {
        private readonly MassTransitReceiveEndpointObserver receiveEndpointObserver;

        public MassTransitHealthCheck(MassTransitReceiveEndpointObserver receiveEndpointObserver) : base("MassTransit Health Check")
        {
            this.receiveEndpointObserver = receiveEndpointObserver ?? throw new ArgumentNullException(nameof(receiveEndpointObserver));
        }

        protected override ValueTask<HealthCheckResult> CheckAsync(CancellationToken cancellationToken = default)
        {
            var faultedEnpoints = receiveEndpointObserver.GetFaultedEndpoints(60);
            if (faultedEnpoints.Count > 0)
            {
                var faultedEnpoints3Minutes = receiveEndpointObserver.GetFaultedEndpoints(180);
                if (faultedEnpoints3Minutes.Count > 0)
                {
                    return new ValueTask<HealthCheckResult>(HealthCheckResult.Unhealthy());
                }
                else
                {
                    return new ValueTask<HealthCheckResult>(HealthCheckResult.Degraded());
                }
            }
            else
            {
                return new ValueTask<HealthCheckResult>(HealthCheckResult.Healthy());
            }
        }
    }
}

I have been able to register is as follows:
namespace Aerium.Velocity.Instance.Worker
{
    public class Startup
    {
        public Startup(IConfiguration configuration)
        {
            Configuration = configuration;
        }

        public IConfiguration Configuration { get; }

        // This method gets called by the runtime. Use this method to add services to the container.
        public void ConfigureServices(IServiceCollection services)
        {
            services.RegisterWorkerService(Configuration);

            var metrics = AppMetricsHealth.CreateDefaultBuilder()
               .HealthChecks.RegisterFromAssembly(services)
               .BuildAndAddTo(services);

            services.AddHealth(metrics);
            services.AddHealthEndpoints();
        }

        // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
        public void Configure(IApplicationBuilder app, Microsoft.AspNetCore.Hosting.IHostingEnvironment env)
        {
            app.UseHealthAllEndpoints();
        }
    }
}

Remove all, and only put :
`
app.UseHealthAllEndpoints();
services.AddHealth(builder =>
            {
                builder.HealthChecks.AddCheck(new MyHealthCheckWithParamaters(70, 90));
                builder.HealthChecks.AddCheck<MyHealthCheck>();
            });
```

How do I add a health check (MassTransitHealthCheck in by example in the post above) if the health check has external dependencies which require loading via DI?

@afmuller adding a health check by type will inject dependencies, health checks are singletons, if you need transient dependencies you'd have to go service locator and inject IServiceProvider for example

Thank you very much @alhardy .
I have tried the following:
            services.AddHealth(builder =>
            {
                builder.HealthChecks.AddCheck<MassTransitHealthCheck>();
            });

            services.AddHealthEndpoints();

but I get this compile error:
MassTransitHealthCheck must be a non-abstract type with a public parameterless constructor in order to use it as parameter 'THealthCheck' in the generic type or method 'IHealthCheckBuilder.AddCheck()'. I would appreciate your assistance, I am not sure what I am missing...
This is the health check - which has the dependencies to be injected in the constructor.
    public class MassTransitHealthCheck : HealthCheck
    {
        private readonly MassTransitReceiveEndpointObserver receiveEndpointObserver;

        public MassTransitHealthCheck(MassTransitReceiveEndpointObserver receiveEndpointObserver) : base("MassTransit Health Check")
        {
            this.receiveEndpointObserver = receiveEndpointObserver ?? throw new ArgumentNullException(nameof(receiveEndpointObserver));
        }

        protected override ValueTask<HealthCheckResult> CheckAsync(CancellationToken cancellationToken = default)
        {
            var faultedEnpoints = receiveEndpointObserver.GetFaultedEndpoints(60);
            if (faultedEnpoints.Count > 0)
            {
                var faultedEnpoints3Minutes = receiveEndpointObserver.GetFaultedEndpoints(180);
                if (faultedEnpoints3Minutes.Count > 0)
                {
                    return new ValueTask<HealthCheckResult>(HealthCheckResult.Unhealthy());
                }
                else
                {
                    return new ValueTask<HealthCheckResult>(HealthCheckResult.Degraded());
                }
            }
            else
            {
                return new ValueTask<HealthCheckResult>(HealthCheckResult.Healthy());
            }
        }
    }

I'm running into the same issue. To work around the two proposed workarounds, I think we'd need access to the internal HealthChecksAsServices provided by the App.Metrics.Health.Extensions.DependencyInjection package so that we can add types ourselves manually. Right now, that's done for us automatically when calling RegisterFromAssembly.
Would you accept a PR that allows for the registration of HealthCheck types into IHealthCheckTypeProvider? It would be a bit tricky, since I won't be able to call e.g. AddHealthChecksAsServices multiple times because internally it instantiates StaticHealthCheckTypeProvider and replaces the IHealthCheckTypeProvider singleton registration with that new instance. But I think it should be doable and rather useful.
My use case scenario is that I'm building a set of packages providing health check classes. My consuming apps should be able to choose which health check types to register. But as @afmuller pointed out, we can't just use .AddCheck<T>, since it requires a type with a default ctor, so we should register the types with the StaticHealthCheckTypeProvider.
Come to think of it, perhaps I should just provide my own implementation of that. Then no need to change App.Metrics.Health.Extensions.DependencyInjection at all. 🤔
I'll give that a go to give it a try. If it works, I can still submit a PR here so others can benefit from it too, if you think it's useful.

Looks like there's no way to do anything without making some types public - right now it's pretty much all internal.

After some more investigation, it turns out I don't need to change anything at all. All I need to do to register my own HealthCheck types using DI is the following:
services
    .AddSingleton<HealthCheck, CosmosDbHealthCheck>()
    .AddSingleton<HealthCheck, RedisHealthCheck>()
    .AddHealthAndMetrics(health => health.BuildAndAddTo(services))

Notice that I'm simply registering the HealthCheck types with the service collection and then invoking BuildAndAddToServices. That's all you need :)


wai-aria: Fix overly-enthusiastic escaping of "_"

The escaping accidentally included test names breaking links and  resulting in duplicate tests in the tests list.

@halindrome: Sorry, but one more pull request. We need the ATTAs or ATTAcomm.js to escape "_" because failing to do so makes the generated results remove that character and italicize parts of constant names. :-/ I had attempted a quick fix, but as the commit message indicates, it was a tad too quick.


Pip installation issue

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version:
Python version:
Installed using virtualenv? pip? conda?:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

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

@shyamdsundar
Please refer to this resolved issues: #47365, #47140, #44656 and verify your system and pip requirements.
We see that the issue template has not been filled, could you please do so as it helps us analyse the issue [tf version, steps followed before you ran into this error or stand alone code to reproduce the issue faced]


docs(CodeSandbox): fix paths for images in exported examples

Fixes #3735.
Also picks latest updates from Stardust.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3758   +/-   ##
=======================================
  Coverage   99.84%   99.84%
=======================================
  Files         175      175
  Lines        3125     3125
=======================================
  Hits         3120     3120
  Misses          5        5

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 a6e5824...1e6b58b. Read the comment docs.


Unit tests

This is unit test cases for ms tail spin game test

Successfully unit test cases are merged


Additional validation when preparing policies containing encapsulated assertions, specifically when the NO_OP_IF_CONFIG_MISSING tag is present. Essentially makes it more forgiving if the NO_OP_IF_CONFIG_MISSING property is present
Adds encoding for policy names that include '/' and '\'. They get encoded to '¯' and '¯'
No longer forcing JDBCConnections to have secure passwords. Allowing JDBCConnections to be loaded with plain text passwords.
filtering out the cluster.hostname cluster property
sorting encapsulated arguments and parameters by name.
making the private key file optional during deployment bundle build time

During the build it shouldn't be possible for a file to contain either of the slashes. Because of this there should be no reason to check. Where were you thinking of adding that check?

I was thinking if someone did some weird stuff in their project like make folders with _¯, which translates to the Gateway as slashes. Then during export, it would complain that there are forbidden characters in the folder names and prevent them from exporting. Although this is a really edge case scenario

if they create a folder name with _¯ in it, during build it would get decoded to / so that when you start a gateway you will see a / in the folder name and on export it will be exported at '_¯` again... at least thats how it is supposed to work

I see, but the FolderLoader#validateFolderName will stop the export because it has an invalid character /

Ya, you're right. Let's consider it an edge case


Remove member permissions on iam project when service account is deleted.

Is this the right issue to say "maybe you should not delete the service account key? (at least until everything else is deleted?)" We were using bbl cleanup-leftovers today to clear out an account and it stopped mid-way because we deleted our service account key and then deleting buckets failed

Is this due to the bump in leftovers? Also, the bbl ci user should be able to delete a test account and delete a test bucket separately?

What we are seeing is a service account/key get deleted, but the email is still a member with permissions on the project.

The project's iam policy will have bindings that are a combination of a set of members and some role. In order to remove a single member from a binding (or more than one binding), it seems that the google terraform provider does something like: https://github.com/terraform-providers/terraform-provider-google/blob/master/google/resource_iam_member.go#L157-L212
Plus some other stuff because the google sdk doesn't seem to have an equal API for this cli command:
gcloud projects remove-iam-policy-binding PROJECT_ID --member=MEMBER --role=ROLE

TODO: Retry logic

Will add retry logic if a separate issue is opened regarding this feature!


Search button doesn't appear

Running visualize_amr_corpus on the little prince dev set, when I first open the seaerch interface, there's no searh button. WHen I close the search and re-open it, the search button is present, and it tries to run a search when I click it.

Fixed (this was due to there being default search filters being loaded, a leftover from debugging, which caused the program to crash before the search button was drawn when using a corpus that did not match the default search filters)


Can't install @ngrx/devtools

Command:

npm install @ngrx/devtools --save-dev

Result:
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm WARN optional Skipping failed optional dependency /gulp-watch/chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

I believe this is normal. It should have actually installed it still no?

On their github, they say that devtools are not yet compatible with ngrx/store v2.0, which is the version used in the project. Maybe is there a link ?

Ah I see, yes I believe you could link against Mike's github repo which contains the refactor to work with v2: https://github.com/mikeryan52/devtools/tree/v2

@pocmanu I use Mike's version via github personally and it works. Give me a shout here on on slack if you have trouble with it.

Should I clone https://github.com/MikeRyan52/devtools.git directly and do npm install after it? Or do I need to put some git flag to access /tree/v2?

@d3viant0ne I'm still just playing with the seed right now, trying to learn ngrx & redux, I don't really need the devtools right now, I don't have much time to dev, but thanks for the help offer (and all the great work you're doing all :))

https://github.com/ngrx/store-devtools are now updated 👍  and you can use them via npm.


[EventHubs] Bug fix in ep, hide last enqueued info api, sample update

Can one of the admins verify this patch?


Submit for Qt 6

Have you considered submitting this to Qt codereview? It would be very nice to have this officially in Qt.
Qt 6 will require at least C++11 (maybe even higher), so this implementation should be good to go.
Related Qt bug: https://bugreports.qt.io/browse/QTBUG-44219

I have indeed thought about it, because this would be a nice Qt::ConnectionType flag. All the template nightmare already exists in some form in QtCore. Since C++11 was not a requirement, I dropped the idea. If C++11 is a requirement in Qt 6, then perhaps I should look into it. I am surprised no one has already done it.

Just an FYI: Qt 6 will require C++17.

This has just been implemented in Qt 6.0: https://codereview.qt-project.org/c/qt/qtbase/+/311098
Instead of using template magic it uses internal QObject code.
Still props to you though for coming up with this template implementation. :)

I am happy to see this! I still haven't found the time or energy to look at implementing it myself. Yes, my code needs to do a lot of the logic that Qt already does (i.e. Qt's own "template magic"). Implementing it in qtbase makes a lot of sense.
However, Qt 6 is far away for many of us, so I hope this project will remain useful.


Runtime error when opening WAAPI Transfer!

Hi, I've just setup the Reaper to Wwise workflow. It worked a few time and now I'm getting to following when running the WAAPI Transfer extension.

Reaper version: 6.05
WAAPI Extension version: 1.10 Beta
Wwise version: 2019.1.5 Build 7093
Thanks in advance.

Hello my friend,
I had the same issue as you.
Karl gave us a fix for it in this ticket : https://github.com/karltechno/Reaper-Waapi-Transfer/issues/14
Check if you're using port 8080 with another program.
In my case it was a web interface set with Reaper using port 8080.
So you have 2 options, you can erase the problematic Web Interface (or change his port), or change the WAMP port in your Wwise User Settings (my Wwise is now using port 8081 and everything is rolling again.
Have a nice day


Add pydantic 2 version of all code blocks in the documentation

FastAPI does a similar thing with Annotated usage and old/new generics style.

Unnecessary as our code blocks already work with both pydantic versions. The better approach would be to add it after writing tests for documentation.


t.func should handle optional arguments?

An example:
var t = require('tcomb');

var addWithMessage = t.func([t.Num, t.Num, t.maybe(t.Str)], t.Num).of(
    function(num1, num2, optionalMessage) {
        if (optionalMessage) {
            console.log(optionalMessage);
        }
        return num1 + num2;
    }
);

console.log(addWithMessage(1, 2)); // => returns a function (since the auto curry is in place), should execute the function instead?

cc @beckyconning

No I think it shouldn't. I think of maybe as a type of the provided type and null rather than as an optional argument flag. There are lots of situations where you need a curried function with arguments that can handle null. Very interesting idea though!! : )

This would also break the type signature paradigm of
addWithMessage :: Num -> Num -> Maybe(Str)
Instead it would be quite complex like
addWithMessage :: Num -> (Num, Maybe(Str))
Or something

I mean
addWithMessage :: Num -> Num -> Maybe(Str) -> Num
And
addWithMessage :: Num -> (Num, Maybe(Str)) -> Num
Sorry my laptop is dead so I'm typing on my phone lol!

Good remarks, thanks.
I can add:

I try to avoid variadic functions when possible
I like regular behaviours: rule out exceptions when possible and keep the code clean (as it is now)
the only benefit seems writing addWithMessage(1, 2) instead of addWithMessage(1, 2, null), not a big deal (in a type checking context)

It's reasonable trying to handle optional args, but given the drawbacks maybe it doesn't worth it.
cc @gavacho (you had a use case, what do you thing?)

I think If it's really important (I don't think I'd use it personally) then there should be an extra array at the end saying that arguments are optional. E.g
//
var f = T.func([T.Num, T.Num, T.Str], T.Str, [false, false, true])
.of(function (x, y, z)

But it does complicate things.

As a thought experiment, let's take a look at lodash.  Many of that library's functions have optional arguments (e.g. https://lodash.com/docs#map)
I wanted to use tcomb to express that kind of function signature (for the sake of using tcomb to type-check arguments at runtime and avoid having to write that code myself all the time).  Perhaps this is not something tcomb should be concerned with (e.g. what happens when the optional argument is the first param?)
Maybe authors should be doing something like:
var map = function(collection, callback, thisArg) {
    // handle all the different ways arguments could be provided
    return mapCore(collection, callback, null);
}

var mapCore = func(Collection, Callback, maybe(Obj)).of(
    function(collection, callback, thisArg) {
        ...
    }
);

I'd definitely welcome a better pattern tho!

Sorry to revive a closed issue, but this is exactly what  I'm looking for and couldn't find another reference to this on the web.
So, using the example from the Little guide:
const SumType = t.func([t.Number, t.Number], t.Number);
const sum = SumType.of((a, b) => a + b);

I need to make a function whit an optional second parameter. So, before getting here I've tried in the REPL:
> var SumType = t.func([t.Number, t.maybe(t.Number)], t.Number);
> var sum = SumType.of((a, b) => a + b);
> sum(1)
TypeError: [tcomb] Invalid value null supplied to Number

Then tried:
> var SumType = t.func([t.Number, t.maybe], t.Number);
> var sum = SumType.of((a, b) => a + b);
> sum(1)
TypeError: [tcomb] Invalid value [
  1
] supplied to arguments of function (Number, maybe) => Number (expected an array of length 2)

So I decide to avoid the .func combinator:
> var sum = (a, b) => { t.Number(a); t.maybe(t.Number(b)); return a + b; }
> sum(1)
TypeError: [tcomb] Invalid value undefined supplied to Number
// now it doesn't even work with the proper parameters :scream:
> sum(1,2)
TypeError: [tcomb] Invalid argument type 2 supplied to maybe(type, [name]) combinator (expected a type)

What would be the proper way to build a function with and optional parameter that gets checked by tcomb and throws either if input or output (that was the main reason to use the func combinator) are not the expected ones? (Considering I want to have optional parameters)

Sorry, I'm new to tcomb and I thought t.maybe combinator could be used directly. Now I see that I need to build a new type first, so I tried:
var OptionalNumber = t.maybe(t.Number, 'OptionalNumber')
var SumType = t.func([t.Number, OptionalNumber], t.Number)
var sum = SumType.of((a, b) => a + b)
sum(1)
TypeError: [tcomb] Invalid value null supplied to Number

I see that if for instance I define sum like:
var sum = SumType.of((a, b) => b ? a + b : a)
// Then I can perfectly `sum` a single paramenter:
sum(1) // 1

But now I'm checking for b, what's the point of using tcomb?
Well, I understand it covers other cases, but I think it should cover this one too.
What am I missing?
Extra questions:
Is like I'm assuming and one cannot use t.maybe combinator directly on a t.func definiton? Or is it that I've used it wrong in my examples? (I'm referring in particular to this now:
var SumType = t.func([t.Number, t.maybe(t.Number)], t.Number)
var sum = SumType.of((a, b) => a + b)
sum(1)
TypeError: [tcomb] Invalid value null supplied to Number

By directly I mean: without defining a new type first.
Another consideration: doesn't it get too verbose? Imagine we would need optional values for every type, then there's a lot of types to type.
I hope someone can shed some light on this.

@fernandocanizo It's the codomain check that (correctly) fails
sum(1) = 1 + undefined = NaN

JSON.stringify(NaN) = 'null' explains the weird error message "Invalid value null supplied to Number" (should be "Invalid value NaN supplied to Number")

Oh! Thanks, I was so puzzled about that error message. So it's ok to use directly t.maybe(t.Number) without defining a new type, great.
Now I tried:
var SumType = t.func([t.Number, t.maybe(t.Number)], t.Nil)
var sum = SumType.of((a, b) => a + b)
sum(1)
TypeError: [tcomb] Invalid value null supplied to Nil

Which got me puzzled again cause docs say:

There are 2 additional built-in types: t.Nil: null or undefined

So I went for t.Any:
var SumType = t.func([t.Number, t.maybe(t.Number)], t.Any)
var sum = SumType.of((a, b) => a + b)
sum(1)
NaN

@fernandocanizo I am no expert, but I think that you need to handle the logic of making the output correct yourself. tcomb can only validate that you have done so. Saying that, I would expect your function to have looked like this:
> var SumType = t.func([t.Number, t.maybe(t.Number)], t.Number)
undefined
> var sum = SumType.of((a, b) => a + (b||0))
undefined
> sum(1)
1
> sum(1,2)
3

Sounds logical. It's true I wasn't handling the case of second parameter not coming. Thanks :)


Uncaught Error: Cannot read property 'object' of undefinedTypeError: Cannot read property 'objec...

[Enter steps to reproduce:]

...
...

Atom: 1.17.0 x64
Electron: 1.3.15
OS: Mac OS X 10.11.6
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
at ObjectsRegistry.get (/Applications/Atom.app/Contents/Resources/electron.asar/browser/objects-registry.js:41:28)
at EventEmitter. (/Applications/Atom.app/Contents/Resources/electron.asar/browser/rpc-server.js:306:31)
at emitMany (events.js:127:13)
at EventEmitter.emit (events.js:201:7)
at WebContents. (/Applications/Atom.app/Contents/Resources/electron.asar/browser/api/web-contents.js:222:13)
at emitTwo (events.js:106:13)
at WebContents.emit (events.js:191:7)
At /Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:217

Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
    at ObjectsRegistry.get (/Applications/Atom.app/Contents/Resources/electron.asar/browser/objects-registry.js:41:28)
    at EventEmitter.<anonymous> (/Applications/Atom.app/Contents/Resources/electron.asar/browser/rpc-server.js:306:31)
    at emitMany (events.js:127:13)
    at EventEmitter.emit (events.js:201:7)
    at WebContents.<anonymous> (/Applications/Atom.app/Contents/Resources/electron.asar/browser/api/web-contents.js:222:13)
    at emitTwo (events.js:106:13)
    at WebContents.emit (events.js:191:7)
    at metaToValue (/Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:217:13)
    at undefined.remoteMemberFunction (/Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:113:18)
    at AtomEnvironment.module.exports.AtomEnvironment.getWindowDimensions (/Applications/Atom.app/Contents/Resources/app/src/atom-environment.js:663:1)
    at AtomEnvironment.module.exports.AtomEnvironment.storeWindowDimensions (/Applications/Atom.app/Contents/Resources/app/src/atom-environment.js:697:1)
    at WindowEventHandler.module.exports.WindowEventHandler.handleWindowBlur (/Applications/Atom.app/Contents/Resources/app/src/window-event-handler.js:184:1)
    at /Applications/Atom.app/Contents/Resources/app/src/window-event-handler.js:3:1

Commands
     -5:40.1.0 core:save (input.hidden-input)
 13x -3:26.5.0 core:undo (input.hidden-input)
  2x -3:21.2.0 core:redo (input.hidden-input)
  9x -3:19.4.0 core:undo (input.hidden-input)
  2x -3:12.7.0 core:redo (input.hidden-input)
  5x -3:11.3.0 core:undo (input.hidden-input)
  6x -3:09.3.0 core:redo (input.hidden-input)
  2x -3:07.1.0 core:undo (input.hidden-input)
  3x -3:06.2.0 core:redo (input.hidden-input)
  2x -3:05.2.0 core:undo (input.hidden-input)
  2x -3:04.6.0 core:save (input.hidden-input)

Non-Core Packages
autocomplete-modules 1.6.10
busy-signal 1.4.1
color-picker 2.2.5
editorconfig 2.2.2
emmet 2.4.3
file-icons 2.1.5
flow-ide 1.4.2
highlight-selected 0.13.1
intentions 1.1.2
language-babel 2.61.0
language-vue 0.21.2
linter 2.1.4
linter-eslint 8.1.7
linter-stylelint 3.10.0
linter-ui-default 1.5.0
markdown-preview-plus 2.4.9
merge-conflicts 1.4.5
pigments 0.39.1
Stylus 3.1.0
vue-autocomplete 0.1.1

Thanks for taking the time to contribute!
We noticed that this is a duplicate of #11692. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


chore: change FOXY_APY config to 'num' instead of 'str'

Description
chore: change REACT_APP_FOXY_APY config to num instead of str and set a default value of 0.15.
This will make sure that any value we put in the environment variable is at least a valid number
Notice

[X] Have you followed the guidelines in our Contributing guide?
[X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[X] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
N/A
Risk
Very very low. It actually reduces risk by catching mistakes in the environment variable.
Testing
Best to test locally.

In .env add REACT_APP_FOXY_APY
Set the value to a valid number
Start the app yarn dev
Change the value to not a number
Start the app

Screenshots (if applicable)

@cjthompson should this update the sample.env value to 0.15 too? i've got them set as .12 in production environments as of right now.

@0xdef1cafe note that .15 is being shown in prod at the moment because of https://github.com/shapeshift/lib/blob/5404919d10bf19285315665b1d89c2a1c5256a4c/packages/investor-foxy/src/api/api.ts#L971

@0xdef1cafe note that .15 is being shown in prod at the moment because of https://github.com/shapeshift/lib/blob/5404919d10bf19285315665b1d89c2a1c5256a4c/packages/investor-foxy/src/api/api.ts#L971

thanks - the PR to show the APY from the env var was only added yesterday and is not in production yet
https://github.com/shapeshift/web/pull/1470

@0xdef1cafe updated sample.env


feat(sdk): extract kwil defaults

Just because I want to be able to use them in the example dApp's fake backend.

Things still seem to work. Confident enough to turn this into a show PR.
@ditoglez Let's iterate later if you're less than happy about this.


Funnel zigzag

For a specific set of scenarios the funnels produce zigzag.
Example debug_finance-uncertainty

that's most likely due to some of those scenarios have 5 year intervals until 2060, while others have not… i'll come up with some way to interpolate those values later or tomorrow

fix is a bit hacky and might not cover some edge cases. should be rewritten at some point → #32

would it be easier if you just ignore all years that end at a 5 (instead of interpolating)? or maybe you did exactly this.
in any case, i would have no problem if we just drop this information and you only work steps of 10 or more years (very rarely it is only 2050 2060 2070 2100)


Add support for swagger.io validator

The OpenAPI Specification (formerly known as the Swagger RESTful API Documentation Specification) defines a format for describing RESTful APIs.  The Swagger project provides a set of tools for working with this format, including a hosted validator which provides a validation badge and JSON result.  This commit adds shields.io support for this badge.
The service currently only provides validation of files conforming to version 2.0 of the OpenAPI Specification.  This commit adds a path component for specifying the version under the assumption that the validator may support version 3.0 or later as they are released.
It accepts the URL to validate as two path components, a scheme followed by the rest of the URL, to match the convention used for the JIRA host and webpage online shields.
I hope you find it useful.  Thanks for considering,
Kevin

This is lovely, thanks! Would love to get it merged. I requested a couple minor changes.

Thanks for reviewing it!  I've fixed two of the issues you mentioned.  The remaining issue, about SSL support, requires some thought/discussion.  Let me know what you think.

Thank you!


Elapsed time includes absolute path to $pwd

The elapsed time portion if my prompt includes the entire path to pwd plus a couple of directories:

Here is my config.fish
# ~/.config/fish/config.fish: DO NOT EDIT -- this file has been generated
# automatically by home-manager.

# Only execute this file once per shell.
set -q __fish_home_manager_config_sourced; and exit
set -g __fish_home_manager_config_sourced 1

source /nix/store/jvdphk4ax16yxzj2b8hcpm2j8k1waf7l-hm-session-vars.fish

fish_config theme choose "Catppuccin Mocha"

eval (/nix/store/php4qidg2bxzmm79vpri025bqi0fa889-coreutils-9.5/bin/dircolors -c ~/.dir_colors)

status is-login; and begin

    # Login shell initialisation

end

status is-interactive; and begin

    # Interactive shell initialisation
    /nix/store/0qygwgixvg42d5lhfkj18b1zvclmcvc4-fzf-0.52.1/bin/fzf --fish | source

    set fish_greeting # Disable greeting
    set hydro_color_pwd $fish_color_cwd
    set hydro_color_git $fish_color_host
    set hydro_color_prompt $fish_color_host_remote

    if set -q KITTY_INSTALLATION_DIR
        set --global KITTY_SHELL_INTEGRATION no-rc
        source "$KITTY_INSTALLATION_DIR/shell-integration/fish/vendor_conf.d/kitty-shell-integration.fish"
        set --prepend fish_complete_path "$KITTY_INSTALLATION_DIR/shell-integration/fish/vendor_completions.d"
    end

    # add completions generated by Home Manager to $fish_complete_path
    begin
        set -l joined (string join " " $fish_complete_path)
        set -l prev_joined (string replace --regex "[^\s]*generated_completions.*" "" $joined)
        set -l post_joined (string replace $prev_joined "" $joined)
        set -l prev (string split " " (string trim $prev_joined))
        set -l post (string split " " (string trim $post_joined))
        set fish_complete_path $prev "/home/walawren/.local/share/fish/home-manager_generated_completions" $post
    end

    /nix/store/35klgzald67mkslqb9kkv01gn98zfbza-direnv-2.34.0/bin/direnv hook fish | source

end

Here is my plugin-hydro.fish
# Plugin hydro
set -l plugin_dir /nix/store/h44v3fqpf2pcrqjqmrr8y3dj6xqd699f-source

# Set paths to import plugin components
if test -d $plugin_dir/functions
    set fish_function_path $fish_function_path[1] $plugin_dir/functions $fish_function_path[2..-1]
end

if test -d $plugin_dir/completions
    set fish_complete_path $fish_complete_path[1] $plugin_dir/completions $fish_complete_path[2..-1]
end

# Source initialization code if it exists.
if test -d $plugin_dir/conf.d
    for f in $plugin_dir/conf.d/*.fish
        source $f
    end
end

if test -f $plugin_dir/key_bindings.fish
    source $plugin_dir/key_bindings.fish
end

if test -f $plugin_dir/init.fish
    source $plugin_dir/init.fish
end

I'm not sure if this is intentional, but I have discovered that this only happens when I am in a devshell using direnv. Maybe because of one of these variables that gets exported?
+AR +AS +CC +CLASSPATH +CONFIG_SHELL +CXX +DETERMINISTIC_BUILD +HOST_PATH +IN_NIX_SHELL +JAVA_HOME +LD +NIX_BINTOOLS +NIX_BINTOOLS_WRAPPER_TARGET_HOST_x86_64_unknown_linux_gnu +NIX_BUILD_CORES +NIX_CC +NIX_CC_WRAPPER_TARGET_HOST_x86_64_unknown_linux_gnu +NIX_CFLAGS_COMPILE +NIX_ENFORCE_NO_NATIVE +NIX_HARDENING_ENABLE +NIX_LDFLAGS +NIX_STORE +NM +NODE_PATH +OBJCOPY +OBJDUMP +PYTHONHASHSEED +PYTHONNOUSERSITE +PYTHONPATH +RANLIB +READELF +SIZE +SOURCE_DATE_EPOCH +STRINGS +STRIP +_PYTHON_HOST_PLATFORM +_PYTHON_SYSCONFIGDATA_NAME +__structuredAttrs +buildInputs +buildPhase +builder +cmakeFlags +configureFlags +depsBuildBuild +depsBuildBuildPropagated +depsBuildTarget +depsBuildTargetPropagated +depsHostHost +depsHostHostPropagated +depsTargetTarget +depsTargetTargetPropagated +doCheck +doInstallCheck +dontAddDisableDepTrack +mesonFlags +name +nativeBuildInputs +out +outputs +patches +phases +preferLocalBuild +propagatedBuildInputs +propagatedNativeBuildInputs +shell +shellHook +stdenv +strictDeps +system ~PATH ~XDG_DATA_DIRS

If this isn't worth your time, I get it. I'm just hoping the fix might be something simple.

I also run Hydro in a system where we are supposed to use direnv and haven't had any problems yet. First, try commenting out your config.fish and Hydro plugins file to see if the problem persists. If I could get a repro of your system where the problem shows up, I would probably brute-force it by debugging this the old way, editing conf.d/hydro.fish directly.

I am new to both fish and hydro. My config.fish and hydro.fish are in the description as dropdowns. Is there something missing for reproducibility? Or are you asking me to slim them down?
I'll try slimming it down and updating the description this week.

Hey, I was having this same issue and the cause seems to be that NixOS's packages include an outdated version of hydro from 2022, where the function _hydro_postexec does not specify the out variable to be local.
This has the consequence that it also gets the global value set to this variable by nix when evaluating a derivation, which in the case of a devshell is the devshell's path.
I'll try to get nixpkgs's hydro package updated, but if I'm not mistaken, that should fix it.

Thanks for that! You are awesome, @spiwocoal!

The update hasn't made its way into nixos-unstable yet. I think that will happen within the next 1-2 weeks. I'll test when it does and then close this issue.

Tested and confirmed fix.


[K32W] Add commissioning instructions

Describe how to setup the topology for NXP E-Lock Demo App.

@saurabhst ?

@doru91 can you restyle please?

@doru91 can you restyle please?

Restyled.


ModuleNotFoundError: No module named 'lottery_ticket'

when I run "python mnist_fc/download_data.py", a error reported:
File "mnist_fc/download_data.py", line 23, in
from lottery_ticket.foundations import save_restore
ModuleNotFoundError: No module named 'lottery_ticket'
I had run "python setup.py install" according https://github.com/google-research/lottery-ticket-hypothesis/pull/3

Same error here. Did you found something?

Same error here. Did you found something?

Do some guys solve it?

@ActonMartin @joaolcaas @snowzjy
Duplicate of #2 . Remove lottery_ticket from the setup.py file.

Hello, I have the same issue.
I have already remove lottery_ticket from setup.py , but still get:
    Traceback (most recent call last):
    File "mnist_fc/download_data.py", line 23, in <module>
    from lottery_ticket.foundations import save_restore
    ModuleNotFoundError: No module named 'lottery_ticket'

on
    python download_data.py

my catalogs structure are:
   /home/johny/lottery-ticket/mnist_fc

Im run python as
   python

At the same time, commands
   python setup.py install
   pip3 install .

works successfully,
Is there any way to fix issue ?
Thank you!

You can do this:

Create folder: "lottery_ticket"
Copy folder "foundations" and "mnist_fc" into folder "lottery_ticket"
Move file "download_data.py" to out of "mnist_fc" folder
Run cmd: python download_data.py


Add rubyMinimal

For small scripts that don’t need the full interpreter with all dependencies (especially runtime dependencies)
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Btw this is only a staging PR because of the whitespace change m(

@Mic92 Feel free to propose a set of options we should enable by default even for rubyMinimal. I’m not a ruby user, just a grumpy devops guy who wants minimal CI overhead. :)

Here’s a short breakdown:
useRailsExpress: unofficial patchset
rubygemsSupport: small scripts, no gems required
zlibSupport: zlib module (pretty specific)
opensslSupport: https/ssl; important for webclients, but imposes a runtime dependency openssl
gdbmSupport: age-old key-value database
cursesSupport: curses is pretty specific, usually not required for simple scripts
fiddleSupport: libffi runtime dependency
yamlSupport: libyaml runtime dependency
docSupport: not required for small scripts

I use gems a lot in small scripts -- how big a difference does
Rubygems make?  I imagine it's pretty tiny.
Also, how big a difference does OpenSSL make?  I'm torn here, but
leaning towards keeping it because I think ever user of rubyMinimal will
have at least one other package that uses OpenSSL anyway.

I think ever user of rubyMinimal will have at least one other package that uses OpenSSL anyway.

That doesn’t necessarily hold on restricted systems or on CI systems which get redeployed on every run.

I use gems a lot in small scripts -- how big a difference does
Rubygems make?  I imagine it's pretty tiny.

Then let’s enable gems

Okay, from my side this is ready to merge. Should we push it to staging or is master fine?

I think ever user of rubyMinimal will have at least one other package
that uses OpenSSL anyway.

That doesn’t necessarily hold on restricted systems or on CI systems
which get redeployed on every run.

Do the machines you want to use this on otherwise have OpenSSL
installed?

Do the machines you want to use this on otherwise have OpenSSL installed?

I don’t know, I also don’t know whether they have necessarily the same version of openssl that would come with rubyMinimal.

Okay, I rebased and got rid of the space changes in the ruby script, so no mass rebuild required. Merging.

Well, once I figure out the build failure …


IBC-related governance proposals

We don't want to forget to have governance proposals for modifying / mutating IBC state:

[ ] Performing unplanned upgrades of an IBC connection
[ ] Freezing IBC clients
[ ] Unfreezing IBC clients

We're going to go ahead with having governance handles to freeze/thaw clients. However, we are not going to allow arbitrarily amending client state


Add GetTimestampAtHeight to client state interface

Summary
The solo machine cannot currently process timeouts because it does not have a consensus state associated with the timeout height until after processing the timeout. Core IBC assumes a consensus state exists in state for the timeout height, but solo machines stores the consensus state in the client state.
We should add a function GetTimestampAtHeight to the client state function which returns the timestamp at a given consensus state height.
Splitting PR recommendation
PR 1
Add the GetTimetstampAtHeight to each client state + unit tests

[ ] 06-solomachine
[ ] 07-tendermint
[ ] 09-localhost

PR 2
Modify connection keeper GetTimestampAtHeight to use the client state interface function. See if it is possible to write a test for solo machines to send a successful timeout msg
For Admin Use

[x] Not duplicate issue
[x] Appropriate labels applied
[x] Appropriate contributors tagged/assigned

I have asked today if it's possible to merge Colin's PR to move forward with PR1 of this issue.


Manual part change

This is a working PR
TODO:

[x] Actually implement UI for manual part changes
[x] Check on staging

It's annoying that the existing part isn't the first option in the select

It's annoying that the existing part isn't the first option in the select

It was annoying

Looks good on staging. I'm going to sleep on reviewing this code though

Also fixed bug in the change password backend flow


add script for nightlywikibackup

see https://github.com/SmartDataAnalytics/OpenResearch/issues/156 for the initial need - make publicly available here.

see https://github.com/WolfgangFahl/py-3rdparty-mediawiki/blob/master/scripts/nightlywikibackup


CSS and Style element tests failing with correct code.

This will be the central issue for tracking this problem. So far it appears to be effecting tests that look for the style element or that test for specific style elements on body. Please let us know if you see cases that break from this pattern.

Examples of this problem can be found on Waypoint: Use JQuery to Modify the Entire Page and Waypoint: Use CSS Selectors to Style Elements

An example test would be something like $("body").css("background-color") === "rgb(0, 0, 0)". The test will work correctly when logged from a script in the editor, but it will not work when run in the test suite.

Has this been resolved?

@catfish68 do you have the same issue described in https://github.com/FreeCodeCamp/FreeCodeCamp/issues/4865?

I have the same issue, I have refreshed, cleared my cache and evened used 3 different devices all with the same end results that I can not get passed this waypoint
Sent from my iPhone

On Dec 1, 2015, at 12:28 PM, Arsen Melikyan<EMAIL_ADDRESS>wrote:
@catfish68 do you have the same issue described in #4865?
—
Reply to this email directly or view it on GitHub.

same issue here in waypoint: inherit style from the body element
coded like:

Hello World

but run test fails, hard refreshed many time, but not working

Try a different browser
Sent from my iPhone

On Dec 10, 2015, at 5:50 AM, rajsoyantar<EMAIL_ADDRESS>wrote:
same issue here in waypoint: inherit style from the body element
coded like:
body { background-color: green; font-family: Monospace; }
Hello World
but run test fails, hard refreshed many time, but not working
—
Reply to this email directly or view it on GitHub.

I have same issue too

Put it all on the same line of code just passed it :)

Had the same issue but I just completed as i placed the font before the color as that was the issue


On import animation optimization doesn't work because of node renaming

Operating system or device - Godot version:
Windows 8 - Bleeding Edge (454b210242ec15a9928324757f27989216bc5a59)
Issue description (what happened, and what was expected):
Okay, so a recent update (0db02f26b66ba4e3ac671607ba85fc794ae698fa) introduced some changes to the scene import, but one thing it broke was the running the animation optimizer on scenes as they're imported. The reason specifically for this breaking is because the update made it so that the AnimationPlayer node which is created automatically on scene import had its named changed to 'animations', whereas the method for method for preprocessing animation optimization search for a node with the default name 'AnimationPlayer'. This is obviously incredibly trivial to fix, and I would have already fixed it myself, but I wanted to know whether there was a reason for renaming the AnimationPlayer node, and whether it should be fixed by either reverting the name or have the optimizer method simply search for the node with a new name.
Steps to reproduce:
Import any Collada file with animation using the optimization flag.
Link to minimal example project (optional but very welcome):

Close?

Yeah, I'll close it now.


API documentation: Shorter string representation of types, faster code, and other smaller changes

Description
Changed string representation of types from .NET format to C# format
Examples:   Int32 changed to int, Nullable<DateTime> changed to DateTime?, and Converter<Nullable<DateTime>, String> changed to Converter<DateTime?, string>. More info: https://github.com/MudBlazor/MudBlazor/discussions/3520.
The code uses https://github.com/benaadams/Ben.Demystifier/blob/main/src/Ben.Demystifier/TypeNameHelper.cs (Apache License). The speed of new and old method is the same. I tested automatically each page, and then I tested manually the differences: all changes are correct except of type of MudDialogInstance.Close(T returnValue) method parameter, but it isn't important.
The code is two times faster now

The Properties, EventCallbacks, and Methods properties were called respectively: 4, 6, and 6 times per one page, but should be only once which will speeds up the code about two times. Generally, C# properties are too easy to call, so I changed them to these methods: getProperties(), getEventCallbacks(), getMethods(). Each of these methods is called only once and the result is stored in a list. I wanted to call them in the OnParametersSet() method, but I doesn't work because the properties table depends not only on the Type parameter but also on the value of the "CATEGORIES - INHERITANCE - NONE" grouping switch. I don't know Blazor well, so I ended with calling these methods in the Razor markup code.
The DocStrings.GetMemberDescription(type, member) method is about 10 ms faster because now the GetSaveTypename() method is run only once per table, and not for every table row.
The DocsTypeInfo component is about 10 ms faster because now the _nonnullableType is computed only once in the OnInitialized() method.

Other changes

By default, properties are grouped by a category (temporarily until it will be set in user settings).
I changed "CATEGORY" label to "CATEGORIES" in the "CATEGORY - INHERITANCE - NONE" switch.
New line characters in a text copied to clipboard make more sense now.
I refactored (simplified) some code.

How Has This Been Tested?
I tested each page automatically using the https://github.com/MudBlazor/MudBlazor/discussions/3595 code, and additionally performed manual tests.
Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

@iwis  Great work on the perf.
Regarding using the type aliases its common for Documentation to use the Net Type Names instead of aliases.
e.g https://docs.microsoft.com/en-us/dotnet/api/system.filestyleuriparser?view=net-6.0
@MudBlazor/core-team
Regarding the switch I think of it as being being prefixed  with the word "By".
By Inheritence By Category By Class
But I am not precious about this change
@MudBlazor/core-team
In any case great work 🙏

Regarding using the type aliases its common for Documentation to use the Net Type Names instead of aliases.

Sure, but in the end it is a question of taste and how it looks. I think we had this discussion already and decided to use the aliases.

Regarding using the type aliases its common for Documentation to use the Net Type Names instead of aliases.
e.g https://docs.microsoft.com/en-us/dotnet/api/system.filestyleuriparser?view=net-6.0 @MudBlazor/core-team

The .NET documentation uses aliases and ? instead of Nullable for properties (See SortedSet.Count and SortedSet.Max detail pages. It doesn't display types for properties at overview page at all - see SortedSet).
MudBlazor already uses aliases for methods (see https://mudblazor.com/api/colorpicker#methods), although inconsistently because it doesn't cope with more difficult cases (see https://mudblazor.com/api/messagebox#methods). When it comes to methods, I don't think we should follow the .NET documentation overview pages because they are incomplete: properties don't have types, methods don't have result type and parameter names. The .NET documentation overview pages have always been hard to read for me - I even wrote a script that changes them: https://github.com/iwis/NET-Framework-documentation-signatures.

Regarding the switch I think of it as being being prefixed with the word "By".
By Inheritence By Category By Class
But I am not precious about this change @MudBlazor/core-team

Currently, in the new doc v5 it is: CATEGORY | INHERITANCE | NONE on https://dev.mudblazor.com/ without the "group by:" label before the switch:

I am not sure if it is clear now, that is why I proposed: CATEGORIES | INHERITANCE | NONE:

What do you think is the best? The "By Inheritence By Category By Class" isn't correct because the label of the 3rd button is "None" now - no grouping. Pay attention that on mobile we have only 360 pixels, so for longer labels we will probably need to use MudMenu/MudSelect.

Regarding using the type aliases its common for Documentation to use the Net Type Names instead of aliases.
e.g https://docs.microsoft.com/en-us/dotnet/api/system.filestyleuriparser?view=net-6.0 @MudBlazor/core-team

The .NET documentation uses also aliases and ? instead of Nullable for properties (See SortedSet.Count and SortedSet.Max detail pages. It doesn't display types for properties at overview page at all - see SortedSet).
MudBlazor already uses aliases for methods (see https://mudblazor.com/api/colorpicker#methods), although inconsistently because it doesn't cope with more difficult cases (see https://mudblazor.com/api/messagebox#methods). When it comes to methods, I don't think we should follow the .NET documentation overview pages because they are incomplete: properties don't have types, methods don't have result type and parameter names. The .NET documentation overview pages have always been hard to read for me - I even wrote a script that changes them: https://github.com/iwis/NET-Framework-documentation-signatures.

Regarding the switch I think of it as being being prefixed with the word "By".
By Inheritence By Category By Class
But I am not precious about this change @MudBlazor/core-team

Currently, in the new doc v5 it is: CATEGORY | INHERITANCE | NONE on https://dev.mudblazor.com/ without the "group by:" label before the switch:

I am not sure if it is clear now, that is why I proposed: CATEGORIES | INHERITANCE | NONE:

What do you think is the best? The "By Inheritence By Category By Class" isn't correct because the label of the 3rd button is "None" now - no grouping. Pay attention that on mobile we have only 360 pixels, so for longer labels we will probably need to use MudMenu, MudSelect, or MudRadioGroup.

I dont mind.  I think you thought about it so I say go for your feeling.

@henon. For me this is good to go.

Thanks @iwis !


Fix #2596

Fixes #2596 .

Pull Request Test Coverage Report for Build 8611

8 of 12 (66.67%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 61.25%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-spring/sharding-jdbc-orchestration-spring/sharding-jdbc-orchestration-spring-boot-starter/src/main/java/org/apache/shardingsphere/shardingjdbc/orchestration/spring/boot/util/PropertyUtil.java
2
4
50.0%

sharding-spring/sharding-jdbc-spring/sharding-jdbc-spring-boot-starter/src/main/java/org/apache/shardingsphere/shardingjdbc/spring/boot/util/PropertyUtil.java
4
6
66.67%

Totals

Change from base Build 8610:
-0.01%

Covered Lines:
8518

Relevant Lines:
13907

💛  - Coveralls


Using Protractor to run tests against multiple user roles

I am trying to run the same test for different roles in the application.
The problem is since there is no logout/login page(SSO magic). I can't logout and login is a different user on the same browser.
I have been trying to get around this and look up a way to pull up a new browser instance or anything.
The only thing that got kind of close was if I tried to use multiCapabilities:
multiCapabilities: [{
    browserName: 'chrome',
    acceptSslCerts: true,
    trustAllSSLCertificates: true,
    specs: ['claimsoverview/login-claims-analyst.spec.js']
}, {
    browserName: 'chrome',
    acceptSslCerts: true,
    trustAllSSLCertificates: true,
    specs: ['claimsoverview/login-supervisior.spec.js']
}],

Each individual spec works, but when I run them together they do not and I get Unknown error unsupported protocol. And also  Error: Error while waiting for Protractor to sync with the page: {}
Before each test I have to hit an Oracle SSO page which is non-angular then I can finally be re-routed to the url that is angular.
//Here is the beginning of one spec..
it('should load homePage from Oracle Sign in Page', function() {

   var dvr = browser.driver;

    //To get around the untrusted cert issue per browser
    dvr.get('https://url');
    UntrustedCertOverride(dvr);

    //Get SSO PageObject and run login()
    var oracleSsoPage = new OracleSsoPage(dvr);
    oracleSsoPage.login(browser.params.claimsAnalyst.userName, browser.params.claimsAnalyst.password);

    //Expect the title to be ConDOR
    expect(browser.getTitle()).toEqual("homePage");
});

@jpetraws Thanks for posting this.
I too am attempting to perform some tests with differing specs for each required role.  Surprisingly, this does not seem to be supported in any other manner than what you have demonstrated.  Please let me know if you may have found other avenues as I'm still searching.
While your approach accomplishes the main goal of being able to have different specs for each role, I believe it has one drawback.  Perhaps you may have a way around this as well?  The drawback that I see with this approach, is that we can no longer use the suites capabilities and consequently invoke protractor to only run a subset of suites (--suite=suite1,suite2).  Any thoughts on this?
TIA!

I want to add multiple credential at run time while running my automation test for Login screen.
How can i do that?

Write one function for login.
2)Add multiple credentials in json file
3)Include json file using  [ var myjsonData = require("../myjson.json"); ]
4)Iterate using loop and call login function in it and pass parameters to function as data in  json file  randomly.


trying to take a look travis. ignore it

Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Run the unit tests with gradlew app:connectedGmsDebugAndroidTest to make sure you didn't break anything (with an emulator running or phone connected)

[x] If you have multiple commits please combine them into one commit by squashing them.

@sherlockbeard Note that you can enable Travis on your fork and test Travis builds on PRs for your own repo.


Feature flag for thread pool executor

With the futures API having landed in libcore in Rust 1.36 and async/.await possibly landing in 1.38, the futures crate will need to eventually transition from alpha status into something more stable. Not all its parts have the same degree of maturity, though. In order to make the crate stable, the stable parts need to be separated from the unstable parts. This is obviously a long process. As one further small step towards this, I'd like to suggest to make the thread pool optional.
AFAIK the thread pool executor in futures-rs was always intended to serve more as an example of how a multi-threaded executor could work, not as an actual production-grade implementation that people should use. There are other implementations like @withoutboats's juliex and probably others that I've just not read about. Am I up-to-date with this? Is this still the purpose of the thread pool executor? If its purpose is still mainly to serve an example implementation, then I think we should make this clear by assigning it a feature flag and making it a non-default feature.

Why not replace the futures threadpool impl with juliex? I think it would be nice if the futures library had a good threadpool and that way it's less scattered in different crates? AFAICT juliex has seen a lot of work and it sees much more use than the futures threadpool.
It would still be an "example implementation", but also be the one we can use in actual products.
@withoutboats @cramertj ? I could be motivated to try and PR that if there is consensus.


debugging: 'disconnect' doesn't work when there are subprocesses

Type: Bug

Behaviour
Expected vs. Actual
Expected: clicking 'disconnect' disconnects from both sessions, debugging toolbar disappears.
Actual: clicking 'disconnect' hides information in 'call stack' panel, but the debugging toolbar is still visible.
Steps to reproduce:

a.py

import time
print("a start")
time.sleep(10000)
print("a end")

b.py

import subprocess

print("b start")
subprocess.run(["python", "a.py"])
print("b end")

.vscode/launch.json

{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Python: attach",
            "type": "python",
            "request": "attach",
            "connect": {
                "host": "localhost",
                "port": 5678
            },
            "pathMappings": [
                {
                    "localRoot": "${workspaceFolder}",
                    "remoteRoot": "."
                }
            ],
            "justMyCode": true
        }
    ]
}

Start debugpy

python -m debugpy --log-to logs --listen 5678 --wait-for-client -m b

Ensure that 'Debug: Show Sub Sessions In Tool Bar' is disabled. Start debugging (F5).
Click the 'disconnect' button. The 'call stack' panel becomes empty.
Click 'disconnect' again. Nothing happens.
Click 'disconnect' again. Nothing happens, error stack trace is printed to logs.

Python version (& distribution if applicable, e.g. Anaconda): 3.10.1
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Global
Value of the python.languageServer setting: Default

Investigation
Line
Because of that line, debugging session for the subprocess doesn't stop, the session for the parent process stops instead.
However I couldn't reproduce the same bug when debugging JS so it is probably a bug in python extension.
Details

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

Connecting to DAP Server at:  localhost:5678
Connecting to DAP Server at:  localhost:5678

User Settings

languageServer: "Pylance"

formatting
• provider: "black"

Extension version: 2022.13.12301015
VS Code version: Code 1.70.2 (e4503b30fc78200f846c62cf8091b76ff5547662, 2022-08-16T05:35:13.448Z)
OS version: Windows_NT x64 10.0.22000
Modes:

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonvspyt551cf:30345471
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30542924
vsaa593:30376534
pythonvs932:30410667
wslgetstarted:30449410
cppdebug:30492333
pylanb8912:30545647
vsclangdc:30486549
c4g48928:30535728
d7fjc500:30543455
dsvsc012:30540252
azure-dev_surveyone:30548225

We don't do anything special when we detach, just unregister the tracing hooks.
If VSCode only sends one "disconnect" request to the parent process, then that's the one that'll be disconnected (but will continue running - processes are only killed on disconnect for "launch"). I suspect the reason why the stack window stops showing it is because the process being debugged is still treated as a child session by VSCode, but its parent session is now gone, so there's no node in the tree to be a parent.
The repro steps also mention that "error stack trace is printed to logs" - @gwisp2, can you share that stack trace? If this is indeed due to confusion about the debug session parent, it should validate that.

The repro steps also mention that "error stack trace is printed to logs" - @gwisp2, can you share that stack trace? If this is indeed due to confusion about the debug session parent, it should validate that.

 ERR cannot find session: Error: cannot find session
	at ExtHostDebugService.getSession (d:\dev\vscode\out\vs\workbench\api\common\extHostDebugService.js:666:19)
	at ExtHostDebugService.$acceptDebugSessionTerminated (d:\dev\vscode\out\vs\workbench\api\common\extHostDebugService.js:484:40)
	at RPCProtocol._doInvokeHandler (d:\dev\vscode\out\vs\workbench\services\extensions\common\rpcProtocol.js:381:27)
	at RPCProtocol._invokeHandler (d:\dev\vscode\out\vs\workbench\services\extensions\common\rpcProtocol.js:366:45)
	at RPCProtocol._receiveRequest (d:\dev\vscode\out\vs\workbench\services\extensions\common\rpcProtocol.js:307:32)
	at RPCProtocol._receiveOneMessage (d:\dev\vscode\out\vs\workbench\services\extensions\common\rpcProtocol.js:234:26)
	at d:\dev\vscode\out\vs\workbench\services\extensions\common\rpcProtocol.js:115:52
	at Listener.invoke (d:\dev\vscode\out\vs\base\common\event.js:457:27)
	at PrivateEventDeliveryQueue.deliver (d:\dev\vscode\out\vs\base\common\event.js:627:38)
	at Emitter.fire (d:\dev\vscode\out\vs\base\common\event.js:592:37)
	at BufferedEmitter.fire (d:\dev\vscode\out\vs\base\parts\ipc\common\ipc.net.js:478:35)
	at d:\dev\vscode\out\vs\workbench\api\node\extensionHostProcess.js:188:41
	at Listener.invoke (d:\dev\vscode\out\vs\base\common\event.js:457:27)
	at PrivateEventDeliveryQueue.deliver (d:\dev\vscode\out\vs\base\common\event.js:627:38)
	at Emitter.fire (d:\dev\vscode\out\vs\base\common\event.js:592:37)
	at BufferedEmitter.fire (d:\dev\vscode\out\vs\base\parts\ipc\common\ipc.net.js:478:35)
	at PersistentProtocol._receiveMessage (d:\dev\vscode\out\vs\base\parts\ipc\common\ipc.net.js:736:45)
	at d:\dev\vscode\out\vs\base\parts\ipc\common\ipc.net.js:622:83
	at Listener.invoke (d:\dev\vscode\out\vs\base\common\event.js:457:27)
	at PrivateEventDeliveryQueue.deliver (d:\dev\vscode\out\vs\base\common\event.js:627:38)
	at Emitter.fire (d:\dev\vscode\out\vs\base\common\event.js:592:37)
	at ProtocolReader.acceptChunk (d:\dev\vscode\out\vs\base\parts\ipc\common\ipc.net.js:269:37)
	at d:\dev\vscode\out\vs\base\parts\ipc\common\ipc.net.js:236:61
	at Socket.listener (d:\dev\vscode\out\vs\base\parts\ipc\node\ipc.net.js:50:17)
	at Socket.emit (node:events:526:28)
	at addChunk (node:internal/streams/readable:315:12)
	at readableAddChunk (node:internal/streams/readable:289:9)
	at Socket.push (node:internal/streams/readable:228:10)
	at Pipe.onStreamRead (node:internal/stream_base_commons:190:23)

That stracktrace is printed on 3rd, 5th, 7th, 9th... click on the 'disconnect' button.

Is the expectation that in this case we would then show just the subsession in the callstack view and let you disconnect from that manually, or that we would disconnect from both sessions?

I would expect there to be a distinction between disconnecting using the floating toolbar vs disconnecting using the inline buttons on processes in the Call Stack view. For the former, the users reasonably expect it to represent the entire logical debug session, so clicking "Disconnect" there should probably disconnect everything. On the other hand, per-process "Disconnect" buttons in the Call Stack feel like they should apply only to the targeted process (and it's not an uncommon pattern for the parent process to be of no interest in a debug session once it spawns the child).

I think this is fixed in https://github.com/microsoft/vscode/pull/169074, since

On the other hand, per-process "Disconnect" buttons in the Call Stack feel like they should apply only to the targeted process (and it's not an uncommon pattern for the parent process to be of no interest in a debug session once it spawns the child).

is exactly what happens with that PR


add resultset footer to plain text representation

Describe your changes
Issue number
Closes #682
Checklist before requesting a review

[ ] Performed a self-review of my code
[ ] Formatted my code with pkgmt format
[ ] Added tests (when necessary).
[ ] Added docstring documentation and update the changelog (when needed)

:books: Documentation preview :books:: https://jupysql--802.org.readthedocs.build/en/802/

This looks like a bug:


broader or configurable permissions for zip file

we need this if root generates a ZIP file that should be downloaded from another user.
currently it has 600 permissions (i.e. not obeying the generating user umask).

fixed in 0.7.2


Make the testsuite green again

Unit

[x] LowLevelFileIO run again https://github.com/quadre-code/quadre/pull/9
[x] Fix NativeMenu tests https://github.com/quadre-code/quadre/pull/16/files
[x] Use the correct message in test assertion https://github.com/quadre-code/quadre/pull/19
[x] Use key instead of keyIdentifier also in tests https://github.com/quadre-code/quadre/pull/38
[x] Linux: fix keyKeyBindingManager tests https://github.com/quadre-code/quadre/pull/45

Integration

[x] Let run integration tests again https://github.com/quadre-code/quadre/pull/7
[x] Add a setTimeout in a FindInFile test https://github.com/quadre-code/quadre/pull/26
[x] Fix DocumentCommandHandlers test on macOS https://github.com/quadre-code/quadre/pull/53

Extension

[x] Fix MDN tests: Quick Edit is also avaible in style attributes https://github.com/quadre-code/quadre/pull/33
[x] Fix InlineColorEditor tests https://github.com/quadre-code/quadre/pull/37
[x] Fix CSSCodeHints tests https://github.com/quadre-code/quadre/pull/59

General

[x] Force "en" locale on test windows https://github.com/quadre-code/quadre/pull/11
[x] Expose again NodeConnection._getConnectionTimeout https://github.com/quadre-code/quadre/pull/18
[x] Make sure also dot files are copied for tests https://github.com/quadre-code/quadre/pull/20
[x] Fix html menu tests https://github.com/quadre-code/quadre/pull/24
[x] Make sure also dot files are copied for src https://github.com/quadre-code/quadre/pull/27
[x] Add back extension title for legacy extensions https://github.com/quadre-code/quadre/pull/42

The testsuite is not all green, but I defer the remaining work to https://github.com/quadre-code/quadre/issues/55


Light Level not showing

Currently playing Valhelsia enhanced vanilla with betterf3 enabled, the light level doesn't display any information. When betterf3 is disabled, the vanilla minecraft displays the light level.

l

I had the same problem and I found that immersive portals mod caused this behavior for me at least.

Closed in 4.0.0


Add Jmeter Prometheus plugin

Add Jmeter prometheus plugin as suggested here: https://github.com/johrstrom/jmeter-prometheus-plugin/issues/82
I've removed markerClass and componentClasses from the same as I'm not sure what they should refer to.

It's almost good. Please provide in vendor whatever name you like - it may be your company name, or your full name, or your nickname.
Also, you have to provide markerClass - it's a full Java class name of your main component, used to identify presence of plugin in Java classpath. In your plugin, com.github.johrstrom.collector.SuccessRatioCollector seem to be a valid option.

componentClasses list all of test elements in your plugin. This field is used for nice feature when use opens JMX file on a machine without your plugin, and plugin manager offers to install missing plugin. This field is optional, you can omit it.

I've added the vendor, marker and component classes

Looks good now. I'm merging it.
When you have new releases of your plugin, please consider creating another PR, adding new version to versions section.
Thanks for contributing!


TodoApplicationTest seems to have a bug cause build to fail

The class TodoApplicationTest seems to have a bug, and is causing the maven build to fail.
this line looks incorrect.
}).given(this.todoItemRepository).deleteById(any(String.class));
the fix for me was
}).given(this.todoItemRepository).deleteById(any(String.class),any());

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

ID: 987d4207-7979-b16b-9f07-8e619fc94ffb
Version Independent ID: 042135b7-854f-8b57-c6f0-e29ba840b5d4
Content: Tutorial: Linux Java app with MongoDB - Azure App Service
Content Source: articles/app-service/tutorial-java-spring-cosmosdb.md
Service: app-service
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@pmcgregoraus, thank you for sharing this feedback. We will review it and get back to you.

@pmcgregoraus Thank you for brining this to our attention. Upon further review, the doc author will need to take a look as we lack the necessary permissions to make the change you pointed out. As a result, we have assigned this issue to the doc author.

#reassign:cephalin

Internal item created: https://dev.azure.com/msft-skilling/Content/_workitems/edit/288580
#please-close


App crashes while opening ar camera

While loading a simple AR screen in pixel2/one plus nord devices, facing below crash issue in ar camera. Issue not reproducible few of our test devices ( Samsung galexy s9, s9+, F64 ). Facing issue in pixel and one plus nord devices only.
Failed to populate image metadata for CPU image. camera_stream_id={camera_id = "0", stream_index = 0}, status=UNAVAILABLE:      ARCoreError: third_party/arcore/ar/infrastructure/android_data_source.cc:896     IMU-camera offset for camera not initialized. camera_id=0
We are Using updated are ar-core services only ( 1.30.22). Could anyone help us to solve the issue ?
Thank you.
Full crash log:
ar_crash_pixel_test.txt

This may not be ARCore related; looking at the tombstone--
#00 pc<PHONE_NUMBER>08246c  /apex/com.android.runtime/lib64/bionic/libc.so (abort+160) (BuildId:<PHONE_NUMBER>147338b8a9538321d4c456)
#01 pc<PHONE_NUMBER>16bed0  /data/app/com.test.ar.a-_TbEHP0UHv9fIi-HzDa9oA==/lib/arm64/libfilament-jni.so (utils::TPanic<utils::PreconditionPanic>::panic(char const*, char const*, int, char const*, ...)+192) (BuildId: f970bc55c73f5308754d01f0a832b184139854ea)

Could you also include the bits above the tombstone? I would think that the libfilament panic would be accompanied by an error message which was not included in the log; thanks.


Remove formatting errors

Codecov Report
Base: 77.21% // Head: 77.21% // No change to project coverage :thumbsup:

Coverage data is based on head (ac749a9) compared to base (e87f0df).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #143   +/-   ##
=========================================
  Coverage     77.21%   77.21%
  Complexity      759      759
=========================================
  Files           117      117
  Lines          2291     2291
  Branches        287      287
=========================================
  Hits           1769     1769
  Misses          447      447
  Partials         75       75

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


site-container.yml fails on 'restart ceph osds daemon(s)'

Bug Report
What happened:
Tried to deploy a new ceph cluster using site-container.yml from ceph-ansible stable-5.0 branch. Ansible hit a timeout during the "ceph-handler : restart ceph osds daemon(s)" stage whilst restarting the first OSD.

Error while running 'ceph --name client.bootstrap-osd --keyring /var/lib/ceph/bootstrap-osd/ceph.keyring --cluster ceph -s', PGs were not reported as active+clean

After some amount of time has passed, the cluster does eventually report as all pgs active+clean. Re-running the site-container,yml playbook at this point completes successfully.
What you expected to happen:
site-container.yml will run to completion
How to reproduce it (minimal and precise):
We've seen this several times now, but it is intermittent so possibly relies on some timing window? Happens maybe 1 in 10 times.
Environment:

OS (e.g. from /etc/os-release): Centos 7
Kernel (e.g. uname -a): Linux condor_sc0 5.4.119-1.el7.elrepo.x86_64 #1 SMP Fri May 14 06:13:51 EDT 2021 x86_64 x86_64 x86_64 GNU/Linux
Docker version if applicable (e.g. docker version): 19.03.15
Ansible version (e.g. ansible-playbook --version):  2.9.25
ceph-ansible version (e.g. git head or tag or stable branch): stable-5.0
ansible.log
ansible.log
Ceph version (e.g. ceph -v): ceph version 15.2.13 (c44bc49e7a57a87d84dfff2a077a2058aa2172e2) octopus (stable)
all_yml.txt
hosts.txt

@DavePiperMicrosoft you might need to adapt handler_health_osd_check_delay and/or handler_health_osd_check_retries


Modifying GlobalTransform instead of Transform

Is there a reason why GlobalTransform is modified rather than Transform?
It prevents children from moving alongside their parents

It is mainly a choice here... tho i might just misunderstanding how the transform system works in bevy, i just used GlobalTransform to see the position relative to world and modified it directly to not overlap the 2 and complicate things(assuming moving the GlobalTransform of an entity will affect its children)

It's preventing my child entities to move with their parent; I guess the GlobalTransform of the parent doesn't affect its children. Only its Transform does. Replacing GlobalTransform with Transform in the source code fixes that problem for me.
In the Bevy cheatbook:

The Transform represents the relative position. You can manipulate it directly.
The GlobalTransform represents the absolute position. It is managed by bevy internally; do not manipulate it yourself.

Also in the transforms page

GlobalTransform represents the absolute global position in the world. If the entity does not have a parent, then this will have the same value as the Transform. The value of GlobalTransform is calculated/managed internally by Bevy. You should treat it as read-only; do not mutate it.

Yeah i have ran some checks now, seems really weird for me and i will replace it in the necessary places, thanks for the clarification

it pretty much solves it(tho it will be a while until i actually finish stuff for a next release


x64 ?

how can i run it in x64?

@nsulikowski What's not working? The library is working fine for me in x64

These should all be compiled AnyCPU so should work fine on x64.


completed challenge 1 - added priyalweb custom profife card

pr request for hacktoberfest 2020

Love the black-hole thing & your profile cell looks great!
Good work & thanks for your contributions!


UDP: Memory leak when allocated packet is smaller than requested

Preface
Bug already got resolved in main by https://github.com/zephyrproject-rtos/zephyr/pull/50212, but the fix should be backported to older Zephyr versions.
I have verified that this bug also happens on current Zephyr v3.1-branch.
Describe the bug
When trying to send a package, it may happen that not all requested memory can be allocated. In case of UDP, the (smaller) package can not be used at all and must be discharged.
Pre https://github.com/zephyrproject-rtos/zephyr/pull/50212, such a packet gets logged, but not unreferenced before returning an error. This causes a memory leak.
To Reproduce
Steps to reproduce the behavior:

Check out 85e7818ff2af3df6751373bb0ed42bb96ea2381c (last commit in main before the fix got applied) and apply https://github.com/zephyrproject-rtos/zephyr/pull/50213 on top
west build -b native_posix -d build-native_posix-zperf zephyr/samples/net/zperf/ -- -DCONFIG_NET_DEBUG_NET_PKT_ALLOC=y -DCONFIG_NET_ZPERF_MAX_PACKET_SIZE=2048
build-native_posix-zperf/zephyr/zephyr.exe
Connect to the Zephyr shell (e.g. picocom /dev/pts/17)
Run command zperf udp upload ::1 5001 1 1500
Run command net allocs, observe the memory locations which are used instead of free
Optionally: Repeat 5 and 6 until all memory is wasted

Expected behavior
Free memory when package is too small.
Impact
Network connectivity (eventually) breaks down.
Logs and console output
uart:~$ zperf udp upload ::1 5001 1 1500

Setting IP address 2001:db8::1
Setting destination IP address 2001:db8::2
Setting IP address <IP_ADDRESS>
Setting destination IP address <IP_ADDRESS>

Remote port is 5001
Connecting to ::1
Duration:       1.00 s
Packet size:    1500 bytes
Rate:           10 kbps
Starting...
Rate:           10 Kbps
Packet duration 1171 ms
[00:00:11.940,000] <err> net_ctx: Available payload buffer (1452) is not enough for requested DGRAM (1500)
Failed to send the packet (12)
[00:00:11.940,000] <err> net_ctx: Available payload buffer (1452) is not enough for requested DGRAM (1500)
Failed to send the packet (12)
[00:00:12.950,000] <err> net_pkt: Data buffer (1500) allocation failed (context_alloc_pkt:1411)
[00:00:12.950,000] <err> net_ctx: Failed to allocate net_pkt
[00:00:13.980,000] <err> net_pkt: Data buffer (1500) allocation failed (context_alloc_pkt:1411)
[00:00:13.980,000] <err> net_ctx: Failed to allocate net_pkt
[00:00:15.020,000] <err> net_pkt: Data buffer (1500) allocation failed (context_alloc_pkt:1411)
[00:00:15.020,000] <err> net_ctx: Failed to allocate net_pkt
[00:00:16.080,000] <err> net_pkt: Data buffer (1500) allocation failed (context_alloc_pkt:1411)
[00:00:16.080,000] <err> net_ctx: Failed to allocate net_pkt
[00:00:17.180,000] <err> net_pkt: Data buffer (1500) allocation failed (context_alloc_pkt:1411)
[00:00:17.180,000] <err> net_ctx: Failed to allocate net_pkt
[00:00:18.300,000] <err> net_pkt: Data buffer (1500) allocation failed (context_alloc_pkt:1411)
[00:00:18.300,000] <err> net_ctx: Failed to allocate net_pkt
[00:00:19.420,000] <err> net_pkt: Data buffer (1500) allocation failed (context_alloc_pkt:1411)
[00:00:19.420,000] <err> net_ctx: Failed to allocate net_pkt
[00:00:20.540,000] <err> net_pkt: Data buffer (1500) allocation failed (context_alloc_pkt:1411)
[00:00:20.540,000] <err> net_ctx: Failed to allocate net_pkt
[00:00:21.660,000] <err> net_pkt: Data buffer (1500) allocation failed (context_alloc_pkt:1411)
[00:00:21.660,000] <err> net_ctx: Failed to allocate net_pkt
[00:00:22.780,000] <err> net_pkt: Data buffer (1500) allocation failed (context_alloc_pkt:1411)
[00:00:22.780,000] <err> net_ctx: Failed to allocate net_pkt
Failed to send the packet (12)
-
Upload completed!
LAST PACKET NOT RECEIVED!!!
Statistics:             server  (client)
Duration:               0 us    (0 us)
Num packets:            0       (0)
Num packets out order:  0
Num packets lost:       0
Jitter:                 0 us
Rate:                   0 Kbps  (0 Kbps)

uart:~$ net allocs
Network memory allocations

memory          Status  Pool    Function alloc -> freed
0x807e848/1      used      TX   context_alloc_pkt():1411
0x807e800/1      used      TX   context_alloc_pkt():1411
0x807e7b8        free      TX   context_alloc_pkt():1411 -> pkt_alloc_with_buffer():1447

Environment (please complete the following information):

OS: Linux
Toolchain: Zephyr SDK
Commit: 85e7818ff2af3df6751373bb0ed42bb96ea2381c + https://github.com/zephyrproject-rtos/zephyr/pull/50213

Thanks @rettichschnidi. Closing as this is already resolved on main. The issue will be reused when backport PRs are open.


Always on top not working in Remote full screen

Microsoft PowerToys version
0.68.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Always on Top
Steps to reproduce
Open "Connessione Desktop remoto" (.rdp) in full screen , and always on top not working
only work if i minimize window and next full screen
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

This isn't working for me either. A workaround is to deactivate "Always on top" on local pc and then try the shortcut on the remote pc.


ACM-13433 - Cluster Sets > User Management Tab > Fixes no users in group bug

Description
This PR is for ticket
Acceptance Criteria

[x] ClusterSetAccessManagement - fixes the issue when no users are in the group, dialog displays No user in group

Before

After

Unit Tests

[x] displays the user name in the AcmLabel when users are available in the group
[x] displays "No users in group" in the AcmLabel when no users are available in the group

/retest

/retest-required

/retest

/lgtm


V6 theme re org

Overview
New organization of theming. Default styles are now created using a method so that they are theme aware (currently only applied to buttons). Passing custom styles and custom variants is still allowed/available and now token overwrites work as expected.
Passing custom variant:
// Theme object
const customTheme = {
  componentName: {
    variantName: {
      base: {
        backgroundColor: 'red'
      }
    }
  }
}

<ThemeProvider value={customTheme}>

Updating token
// Theme object
const customTheme = {
  tokens: {
    primary: {
      base: 'green'
    }
  }
}

<ThemeProvider value={customTheme}>

Screenshots (if applicable)
n/a
Testing

[ ] Updated Typescript types and/or component PropTypes
[ ] Added / modified component docs
[ ] Added / modified Storybook stories

Cool! I think this looks good 👍 . Can't wait till we're able to get rid of the other cruft here too

Btw - for disabled buttons, can we add a cursor: not-allowed? In addition to just making the background <color>100

@akleiner2 that's actually there, coming from shared.js disabled styles

may not be visible though, due to the pointer-events: none


Composite Mod Packages

The possibility to package multiple mods into a single zip file with a manifest describing the contents. This way it would be easier to package multiple small mods that work together well. For example a "platformer composite mod package" with mods like bounce blocks, wall climbing mod etc.

This is on main


multiple small issues

[x] piwik statistics link is not shown for unauthenticated users!
[x]  do not make the login button fixed do not make it transparent, make it slightly bigger
[x]  make the beta tag more visible (also bigger)
[x]  iRODS is not showing configuration and the buttons are not nice as they were (but you are working on that one probably)
[x]  https://github.com/ufal/lindat-dspace/issues/272

question: are licenses automatically uploaded to license selector? (if I create a new license, will it be visible in license selector?)

question: are licenses automatically uploaded to license selector? (if I create a new license, will it be visible in license selector?)
the answer is no

piwik statistics link is not shown for unauthenticated users!

@vidiecan the piwik stats link is shown now, but you are immediately asked to login. Is this the desired behavior?

yes

#292 not fixed by this

the requirement for #272 is:
short text about license selector
BUTTON
-or-
short text describing the listing
small select box

where -or- is either some ui element or the word "or", if it looks reasonable


Question-Service fabric securing with onprem active directory

Is there way to secure SFx with on-prem active directory group ?
Thanks
Naresh

<EMAIL_ADDRESS>please respond.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Naresh Khatri<EMAIL_ADDRESS>Sent: Wednesday, June 27, 2018 3:53:35 PM
To: Microsoft/service-fabric
Cc: Subscribed
Subject: [Microsoft/service-fabric] Question-Service fabric securing with onprem active directory (#164)
How does service fabric connects with onprem active directory ? Does it uses ldap protocol or oauth ?
Is there way to secure SFx with active directory group ?
Thanks
Naresh
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fservice-fabric%2Fissues%2F164&data=02|01|rsinha%40microsoft.com|ba57fa1be0bc4e31a60808d5dc180ac7|72f988bf86f141af91ab2d7cd011db47|1|0|636656918177920325&sdata=WEBSmri4oxYhEg0PW7SaMfDERDdOuF3vwyYklaT8UnU%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKzNrH6ybGgix1S1Sql90TZpfemw5DTdks5uA10ngaJpZM4U5cxD&data=02|01|rsinha%40microsoft.com|ba57fa1be0bc4e31a60808d5dc180ac7|72f988bf86f141af91ab2d7cd011db47|1|0|636656918177920325&sdata=r7x59obsfTJ76lpI%2FOe8aTJ2FQYzbQfgAY4PNOeSjto%3D&reserved=0.

@rishirsinha  @rajeet
i was able to setup individual client identities and it worked with windows integrated authentication with SFx.
My next question is does it support AD group for authentication/authorization ?
so instead of putting list of users during cluster creation in configuration file.
we specify AD group which would be having list of users and machine accounts/service accounts/gmsa etc.
Thanks,
Naresh Khatri

Client identities can be any AD object. The auth is handled by windows security providers.

@rishirsinha
when doing lookup for AD Groups, SF seems to be looking at domain which SF servers are in.
for example
(1) worker1.domain1.com
(2) worker2.domain1.com
(3) worker3.domain1.com
so in this case SF will do lookup for group at domain1.com in AD.
is it possible to specify lookup to be done at different domain like domain2.com.
Correct me if my understanding of SF is wrong.
In SF cluster inside one of VM i can do lookup of groups inside domain2.com if i change location to domain2.com.
by default it do search in domain1.com.
Thanks,
Naresh Khatri


Notebook saving doesn't trim code cell whitespace when files.trimTrailingWhitespace setting is true

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
When I save a Jupyter notebook which has trailing whitespace in it's code cells, I expect it to be trimmed if files.trimTrailingWhitespace is true
VS Code Version
Version: 1.68.0 (user setup) Commit: 4af164ea3a06f701fe3e89a2bcbb421d2026b68f Date: 2022-06-08T11:44:16.822Z Electron: 17.4.7 Chromium: 98.0.4758.141 Node.js: 16.13.0 V8: <IP_ADDRESS>-electron.0 OS: Windows_NT x64 10.0.19044
Jupyter Extension Version
v2022.5.1001601848
Jupyter logs
No response
Coding Language and Runtime Version
No response
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
No response

I don't think a new setting is needed for this scenario.
If you look at the screenshot, from the user point of view it seems to me that the highlighted trailing spaces should be removed when the notebook is saved if files.trimTrailingWhitespace is true.
I'm not sure why notebooks should ignore that setting, even if the saving mechanics are more complicated. The notebook respects most other settings and it's one of the stated goals to make the notebook experience as similar to regular editing.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Malformed Mach-o file

Checks

[x] I have checked that this issue has not already been reported.

[x] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
Don't have one :(
Issue description
On macOS:
❯ rm -rf .pixi/
❯ pixi add ...
  × failed to solve the pypi requirements of 'default' 'linux-64'
  ├─▶ failed to query python interpreter
  ├─▶ Failed to query Python interpreter at `.../.pixi/envs/default/bin/python3.12`
  ╰─▶ Malformed Mach-o file (os error 88)

Expected behavior
?

So you are running on macOS and solving (only) for Linux-64?


Network problem script

Added a script that checks for important networking events. This script will be run by the custom plugin monitor.
Currently the script only checks if the conntrack table is full and sends a temporary event if it is. More checks will be added in the future.

This change is

/assign @Random-Liu

@rramkumar1 @Random-Liu
I am not sure we need to add all available custom plugins in node-problem-detector repo. But, imo, i am fine with it.

@andyxning Addressed your comments, PTAL. Also I agree that it would make sense to add all available custom plugins to the repo since they should be useful for anyone.

lgtm
@Random-Liu PTAL.

ping @Random-Liu

30s interval is too often, and I don't think any remedy system will take an action every minute. Can we increase the interval to 5 mins at least? cc/ @Random-Liu