Issue Reporter: UI/UX improvements

Testing #46595

I have a couple of UI change suggestions:

The paragraph left alignment is quite upsetting
Font sizes seem arbitrary. A good font size example usage is the feedback widget (smiley in the bottom right corner).

Let's not reduce the font size of explanatory paragraphs. They should not behave little small print, we want people to notice them

Escape should close the dialog
The form fields are strangely phrased. I feel being more direct here would help:

Type instead of This is a
Component instead of File on

The monospace tables look quite foreign:

Use a more complete family specifier: font-family: Monaco, Menlo, Consolas, "Droid Sans Mono", "Inconsolata", "Courier New", monospace, "Droid Sans Fallback";
Use a background color for code block areas (just like markdown in Github, code blocks have a grey background)
Decrease table row padding to make them much more compact. Or even better, get rid of the tables and just render everything as regular form fields

Drop the rounded corners and use a similar style to VS Code itself. Search, SCM and Extension viewlets or even the feedback widget have good text input field styles. Same applies to the button.

I just noticed the narrow UI:

This works much better IMO. Why not always drop the form field labels?

@joaomoreno Thanks for the feedback! I've incorporated some of it, let me know what you think:

I changed the paragraph left alignment, and the tables use the font and background color you suggested. I removed the rounded corners on everything except the selects, which is consistent with the rest of the VSCode UI - all of our selects have round corners on Mac. (This is why I made everything rounded to begin with, so that it looked consistent.) I think the labels are easier to understand as they currently are.

@RMacfarlane Much better! 🚀
I wonder if you should also prevent the user from selecting the labels using user-select: none?


[bug] supabase callback and deep link in windows

open-url is not triggering in win32.
Windows requiring additional code in order to open the contents of the protocol link within the same Electron instance
https://www.electronjs.org/docs/latest/api/app#apprequestsingleinstancelockadditionaldata
solution:
    app.on('second-instance', (_, commandLine) => {
        if (mainWindow) {
            if (mainWindow.isMinimized()) {mainWindow.restore()};
            mainWindow.focus();
        }
        const url = commandLine.find(arg => arg.startsWith(`${APP_SCHEMA}://`));
        if (url) {
            handleAuthCallback(url);
        }
    });

@hitaspdotnet thanks for opening the issue. Seems like in this case, we'd rather force single instance instead such that windows have the same expected behavior as MacOS?

we can check the platform too:
if (url && (process.platform === 'win32')) {
            handleAuthCallback(url);
        }

Merged a fix @hitaspdotnet thanks for reporting!


The Merge

Everything needed to ensure that Web3Signer will support Ethereum after the merge.

Is block v3 one item? 🤔 will you be adding a list of check items?


Adding Eduardo Pires profile and picture

Adding Eduardo Pires profile and picture 500x500px

:x: Validation status: errors

File
Status
Preview URL
Details

rd/sebastian-fyda.yml
:x:Error

Details

rd/joel-oleson.yml
:x:Error

Details

rd/jen-stirrup.yml
:x:Error

Details

rd/mamadou-bah.yml
:x:Error

Details

rd/amit-bansal.yml
:x:Error

Details

rd/dan-hermes.yml
:x:Error

Details

rd/paula-januszkiewicz.yml
:x:Error

Details

rd/stacia-varga.yml
:x:Error

Details

rd/igor-abade.yml
:x:Error

Details

rd/reza-rad.yml
:x:Error

Details

rd/nishith-pathak.yml
:x:Error

Details

rd/ritesh-modi.yml
:x:Error

Details

rd/udi-dahan.yml
:x:Error

Details

rd/mateusz-kozlowski.yml
:x:Error

Details

:x:Error

Details

rd/sebastian-fyda.yml

[Error] Unable to load file: sebastian-fyda.yml via processor: Profile: sebastian-fyda.yml is not in supported format: (Line: 18, Col: 18, Idx: 592) - (Line: 18, Col: 19, Idx: 593): While parsing a block mapping, did not find expected key.

rd/joel-oleson.yml

[Error] Unable to load file: joel-oleson.yml via processor: Profile: joel-oleson.yml is not in supported format: (Line: 18, Col: 18, Idx: 578) - (Line: 18, Col: 19, Idx: 579): While parsing a block mapping, did not find expected key.

rd/jen-stirrup.yml

[Error] Unable to load file: jen-stirrup.yml via processor: Profile: jen-stirrup.yml is not in supported format: (Line: 18, Col: 18, Idx: 577) - (Line: 18, Col: 19, Idx: 578): While parsing a block mapping, did not find expected key.

rd/mamadou-bah.yml

[Error] Unable to load file: mamadou-bah.yml via processor: Profile: mamadou-bah.yml is not in supported format: (Line: 18, Col: 18, Idx: 577) - (Line: 18, Col: 19, Idx: 578): While parsing a block mapping, did not find expected key.

rd/amit-bansal.yml

[Error] Unable to load file: amit-bansal.yml via processor: Profile: amit-bansal.yml is not in supported format: (Line: 18, Col: 18, Idx: 578) - (Line: 18, Col: 19, Idx: 579): While parsing a block mapping, did not find expected key.

rd/dan-hermes.yml

[Error] Unable to load file: dan-hermes.yml via processor: Profile: Validation against "https://dotnet.github.io/docfx/schemas/v1.0/schema.json#" failed:
Invalid type. Expected String but got Null. Path 'facebook'.
Invalid type. Expected String but got Null. Path 'twitter'.
Invalid type. Expected String but got Null.

[Error] sebastian-fyda.yml is not in supported format: (Line: 18, Col: 18, Idx: 592) - (Line: 18, Col: 19, Idx: 593): While parsing a block mapping, did not find expected key.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:x: Validation status: errors

File
Status
Preview URL
Details

rd/amit-bansal.yml
:x:Error

Details

rd/udi-dahan.yml
:x:Error

Details

rd/igor-abade.yml
:x:Error

Details

rd/paula-januszkiewicz.yml
:x:Error

Details

rd/mamadou-bah.yml
:x:Error

Details

rd/mateusz-kozlowski.yml
:x:Error

Details

rd/stacia-varga.yml
:x:Error

Details

rd/ritesh-modi.yml
:x:Error

Details

rd/joel-oleson.yml
:x:Error

Details

rd/dan-hermes.yml
:x:Error

Details

rd/nishith-pathak.yml
:x:Error

Details

rd/reza-rad.yml
:x:Error

Details

rd/jen-stirrup.yml
:x:Error

Details

rd/sebastian-fyda.yml
:x:Error

Details

:x:Error

Details

rd/amit-bansal.yml

[Error] Unable to load file: amit-bansal.yml via processor: Profile: amit-bansal.yml is not in supported format: (Line: 18, Col: 18, Idx: 578) - (Line: 18, Col: 19, Idx: 579): While parsing a block mapping, did not find expected key.

rd/udi-dahan.yml

[Error] Unable to load file: udi-dahan.yml via processor: Profile: udi-dahan.yml is not in supported format: (Line: 18, Col: 18, Idx: 567) - (Line: 18, Col: 19, Idx: 568): While parsing a block mapping, did not find expected key.

rd/igor-abade.yml

[Error] Unable to load file: igor-abade.yml via processor: Profile: igor-abade.yml is not in supported format: (Line: 18, Col: 18, Idx: 572) - (Line: 18, Col: 19, Idx: 573): While parsing a block mapping, did not find expected key.

rd/paula-januszkiewicz.yml

[Error] Unable to load file: paula-januszkiewicz.yml via processor: Profile: paula-januszkiewicz.yml is not in supported format: (Line: 18, Col: 18, Idx: 618) - (Line: 18, Col: 19, Idx: 619): While parsing a block mapping, did not find expected key.

rd/mamadou-bah.yml

[Error] Unable to load file: mamadou-bah.yml via processor: Profile: mamadou-bah.yml is not in supported format: (Line: 18, Col: 18, Idx: 577) - (Line: 18, Col: 19, Idx: 578): While parsing a block mapping, did not find expected key.

rd/mateusz-kozlowski.yml

[Error] Unable to load file: mateusz-kozlowski.yml via processor: Profile: mateusz-kozlowski.yml is not in supported format: (Line: 18, Col: 18, Idx: 607) - (Line: 18, Col: 19, Idx: 608): While parsing a block mapping, did not find expected key.

rd/stacia-varga.yml

[Error] Unable to load file: stacia-varga.yml via processor: Profile: stacia-varga.yml is not in supported format: (Line: 18, Col: 18, Idx: 583) - (Line: 18, Col: 19, Idx: 584): While parsing a block mapping, did not find expected key.

rd/ritesh-modi.yml

[Error] Unable to load file: ritesh-modi.yml via processor: Profile: ritesh-modi.yml is not in supported format: (Line: 18, Col: 18, Idx: 577) - (Line: 18, Col: 19, Idx: 578): While parsing a block mapping, did not find expected key.

rd/joel-oleson.yml

[Error] Unable to load file: joel-oleson.yml via processor: Profile: joel-oleson.yml is not in supported format: (Line: 18, Col: 18, Idx: 578) - (Line: 18, Col: 19, Idx: 579): While parsing a block mapping, did not find expected key.

rd/dan-hermes.yml

[Error] Unable to load file: dan-hermes.yml via processor: Profile: Validation against "https://dotnet.github.io/docfx/schemas/v1.0/schema.json#" failed:
Invalid type. Expected String but got Null. Path 'facebook'.
Invalid type. Expected String but got Null. Path 'twitter'.
Invalid type. Expected String but got Null. Path 'linkedin'.

rd/nishith-pathak.yml

[Error] Unable to load file: nishith-pathak.yml via processor: Profile: nishith-pathak.yml is not in supported format: (Line: 37, Col: 19, Idx: 2524) - (Line: 37, Col: 20, Idx: 2525): While parsing a block mapping, did not find expected key.

rd/reza-rad.yml

[Error] Unable to load file: reza-rad.yml via processor: Profile: reza-rad.yml is not in supported format: (Line: 18, Col: 18, Idx: 562) - (Line: 18, Col: 19, Idx: 563): While parsing a block mapping, did not find expected key.

rd/jen-stirrup.yml

[Error] Unable to load file: jen-stirrup.yml via processor: Profile: jen-stirrup.yml is not in supported format: (Line: 18, Col: 18, Idx: 577) - (Line: 18, Col: 19, Idx: 578): While parsing a block mapping, did not find expected key.

rd/sebastian-fyda.yml

[Error] Unable to load file: sebastian-fyda.yml via processor: Profile: sebastian-fyda.yml is not in supported format: (Line: 18, Col: 18, Idx: 592) - (Line: 18, Col: 19, Idx: 593): While parsing a block mapping, did not find expected key.

[Error] amit-bansal.yml is not in supported format: (Line: 18, Col: 18, Idx: 578) - (Line: 18, Col: 19, Idx: 579): While parsing a block mapping, did not find expected key.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

The changes in this PR is in live. Please review it from: https://rd.microsoft.com/en-us/

Checked! Everything is ok!


Wrong image weight on the server after uploading

I don't want to describe the problem here again, but it reproducable also with this library. You can read the problem at https://github.com/ivpusic/react-native-image-crop-picker/issues/1231. Maybe someone knows what is the reason of that issue?

+1

+1


get Window Manager working with Uno

The idea is that this could be incorporated into the platform - but for now this 'test' sample is tightly coupled to Android-specific code in the main activity (where Window Manager is 'wired up'). Need to think a bit more how to abstract that out...

related to https://github.com/unoplatform/uno/issues/6507


archivebox: refactor and fix

Motivation for this change
Archivebox didn't build due to conflicting dependency on django.
Possibly related to Issue https://github.com/NixOS/nixpkgs/issues/154950 that was closed by PR https://github.com/NixOS/nixpkgs/pull/154153
Things done
Resolves conflict by overriding django dependency for all dependencies.
Updated commit message and using packageOverrides as requested by reviewers on previous now closed PR https://github.com/NixOS/nixpkgs/pull/156520
Basic functionality tested and working.
Nothing else should depend on archivebox.

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

[ ] Tested, as applicable:

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

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage

[X] Tested basic functionality of all binary files (usually in ./result/bin/)

22.05 Release Notes (or backporting 21.11 Release notes)

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

[X] Fits CONTRIBUTING.md.

@dotlambda accepted your changes

Result of nixpkgs-review pr 158704 run on aarch64-darwin 1

1 package failed to build:

archivebox

1 package failed to build

@siraben Was that due to ipython, like for me on aarch64-linux?

@dotlambda yes, that's the cause of the failure.


Vue single file component multiple top level templates

What problem does this feature solve?
This will simplify having to put a wrapper around conditional templates.
Consider a table view, that will have a different layout whether there are any rows, or none at all.
<template v-if="results.length == 0">

Try running a search?

</template>

<template v-if="results.length > 0">

Search Results Block

{{ results }}

</template>

Currently, you have to wrap it all in a template, and then a different element within that (say, a div).
This is because a template can cause multiple top level elements to be found.
This has the side effect of putting a div on the page that may not be desired. For example, if results.length == 0 block wanted a different element, id, or class than results.length > 0 this would not be easily readible or necessarily even possible.
The way it works should be that the last template that is found, that evaluates to true, is drawn.
This is currently the case if multiple top level templates are created, just without executing the conditional checks.
What does the proposed API look like?
<template v-if="results.length == 0">

Try running a search?

</template>

<template v-if="results.length > 0">

Stock Results Block

{{ results }}

</template>

Would draw the Try running a search? on page load, and when results gets populated, it will draw the other template.

You can already do a series of v-if/v-else-if on the top element in the template to achieve this:
<template>

Try running a search?

Stock Results Block

{{ results }}

</template>

Your exact example will not work for other reason (template can't have multiple root elements, and your second template has h1 and pre), so I wrapped it in a div.

What you are actually asking for is multiple top-level elements for a component. While that's something useful in its own right, currently it requires non-trivial changes to the vdom implementation to support it. This is something that falls within "obviously nice to have but not an area of focus for now".
We'll revisit this when we are working on substantial improvements to the vdom in the future.


[5.7] WhereToday feature

In my and some other projects code often have a lot of queries that filter entries by today condition.
This PR introduces "whereToday" feature in Query Builder.

I agree with @ahinkle. This seems like it should be a scope you add per project that you need it, and not something added to the framework. Perhaps you could make a nice package to do this and include all the examples that @ahinkle gave.


Overwrite vue.util.isObject catch an error

Using babel
import Vue from 'vue';

const isObject = Vue.util.isObject;

Vue.util.isObject = function(obj){
    if ( obj._isVueClassic ){
        return true;
    }
    return isObject(obj);
}

Why can't overwrite isObject?

The internal util bindings are immutable. This is by design. And you should not be messing with Vue's internal utils anyway...


Tooltip keeps shaking when hover over and cannot be positioned to right

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Tooltip should be positioned to the right side of the drawer button instead of overlapping the button.
Steps to Reproduce (for bugs)

<Tooltip
  id={`tooltip-${category.key}`}
  title={category.title}
  placement="right-end"
>

    <Button
      id={category.key}
      className={classes.closedDrawerButton}
      component={Link}
      onClick={() => this.onClickSelectedItem(category.value)}
    >

    </Button>

</Tooltip>

Your Environment

Tech
Version

Material-UI
^1.0.0-beta.45

React
^15.6.1

browser
Chrome Version 66.0.3359.139

I'm closing as a duplicate of #10735

Add pointer-events: none;


Add a clarification where the user can get the default password for accessing from LAN

link

@laurijamsa could you please check UI and translation?

I thought that the remote configuration option would be on by default. Is it not? https://github.com/ruuvi/ruuvi.gwui.html/blob/9a43d02816f91cd4555a9fbf12bdfe22b3567956/src/index.html#L938-L939
https://github.com/ruuvi/ruuvi.gwui.html/blob/9a43d02816f91cd4555a9fbf12bdfe22b3567956/src/index.html#L923-L925
These lines need more info:
https://github.com/ruuvi/ruuvi.gwui.html/blob/9a43d02816f91cd4555a9fbf12bdfe22b3567956/src/index.html#L962-L963
On the bottom of the device we don't tell that the unique id (XX:XX:XX:XX:XX:XX:XX:XX) is a password. So we need to add more information here and also on the login page. Also, how the default password field shows only 8 "*" characters if default password is 48-bit id that has 16 hex characters + 7 semicolons? What's format of the password?

Also, how the default password field shows only 8 "*" characters if default password is 48-bit id that has 16 hex characters + 7 semicolons? What's format of the password?

It's for security reasons. Passwords never transmitted to the browser, so it doesn't know the length of the password and if password is set, then it is always displayed as eight "*" regardless of its length. If an attacker knows the length of the password, it makes the bruteforcing task much easier.
Although the browser has information that the default password is used, so we can show hint "XX:XX:XX:XX:XX:XX:XX:XX" instead of "********".

@laurijamsa could you please check updated UI and translation?

Translations are not perfect and the UI is currently a bit confusing because first it's told what the defaults are but to a new user it looks like that he should modify the defaults for some reason. And when doing it, I did run into another issue: https://github.com/ruuvi/ruuvi.gateway_esp.c/issues/442
This settings view could be more clear if we'd have a fourth radio button:

Use password protection with default password (default, safe)
Use custom password protection (advanced, safe if using a strong password)
No remote configuration access (safe)
Allow remote configuration without password (unsafe, not recommended)


Missing ~/.pwrc?

I've used pwy for a while to make a quick check on a sibling living in Japan. I tried it yesterday and got the following
Traceback (most recent call last):
  File "/home/user/.local/bin/pwy", line 8, in <module>
    sys.exit(main())
  File "/home/user/.local/lib/python3.9/site-packages/pwy/cli.py", line 202, in main
    info = get_weather_data(location, unit, lang)
  File "/home/user/.local/lib/python3.9/site-packages/pwy/cli.py", line 35, in get_weather_data
    f"&appid={get_key()}&units={unit}&lang={lang}")
  File "/home/user/.local/lib/python3.9/site-packages/pwy/cli.py", line 20, in get_key
    with open(f"{home}/.pwyrc") as f:
FileNotFoundError: [Errno 2] No such file or directory: '/home/user/.pwyrc'

Obviously missing a .pwyrc, but what should I put there? I don't see it on the github page. I tried uninstalling and reinstalling both via pip3 and cloning the repo. Same issue. Any help appreciated.

Thank you for the issue.
Did you set your OWM API key? If not, you need to run the command pwy --config <YOUR_OWM_API_KEY>, after that you should be good to go. (;
Note: You need to wait for your OWM API key to be activated, it's an OWM thing.
If the issue still persists, please let me know.

Well, damn. That was probably on me. I've been moving a ton of stuff around trying to...well, it doesn't really matter.
All good now. Thanks.

Thank you! (;


Configure tasks synchronously

Targets #52.
Test build.gradle:
distribution {
    serviceName "my-service"
    mainClass "dummy.service.MainClass"
    args "hello"
}

dependencies {
    compile "com.palantir.gradle.javadist:gradle-java-distribution:$javaDistributionVersion"
}

println "Class-Path: ${manifestClasspathJar.manifest.attributes["Class-Path"]}"
println "distTar output: ${distTar.outputs.files.singleFile}"

afterEvaluate {
    println "Class-Path (afterEvaluate): ${manifestClasspathJar.manifest.attributes["Class-Path"]}"
    println "distTar output (afterEvaluate): ${distTar.outputs.files.singleFile}"
}

With 0.5.0:
Class-Path: null
distTar output: /home/htung/dev/sandbox/build/distributions/sandbox.tgz
Class-Path (afterEvaluate): gradle-java-distribution-0.5.0.jar sandbox.jar
distTar output (afterEvaluate): /home/htung/dev/sandbox/build/distributions/my-service.tgz

After patch:
Class-Path: gradle-java-distribution-0.5.0-4-gd49c1de.dirty-sources.jar gradle-java-distribution-0.5.0-4-gd49c1de.dirty.jar sandbox.jar
distTar output: /home/htung/dev/sandbox/build/distributions/my-service.tgz
Class-Path (afterEvaluate): gradle-java-distribution-0.5.0-4-gd49c1de.dirty-sources.jar gradle-java-distribution-0.5.0-4-gd49c1de.dirty.jar sandbox.jar
distTar output (afterEvaluate): /home/htung/dev/sandbox/build/distributions/my-service.tgz

According to https://discuss.gradle.org/t/whats-the-difference-between-plugin-conventions-vs-plugin-extensions/5523 conventions are no longer recommended. Perhaps there's a fix here which doesn't involve such a large change.

e.g. I suspect this whole thing is fixable by making the classpath computation lazy (by deferring it as a lambda)

Er, the classpath computation is lazy in the new impl, but that's not the primary problem here; the configuration of the distTar name is.
The reason conventions were deprecated is namespacing concerns, where method names could collide between plugins. In this case, the distribution extension already made distribution a closure-taking method, which could collide with other plugins; this implementation should have no more API collision surface area than the original.

why isn't this solved by just moving distTar.configure(ext) then?

Where would you suggest moving it in this case? I tried looking for a reasonable place to put it, but plugin application is too early (the DistributionExtension is not configured yet), and given the current API, I'm not sure you can schedule anything to run after the distribution { } closure automatically.
I chose instead to move code in bulk; there's very little actual change to the code; it just moves task creation somewhat later (plugin application time -> post-distribution { } time) and moves task configuration earlier (afterEvaluate -> post-distribution { } time), to minimize behavior change.

From a more abstract POV, Gradle configuration is moving towards rule-based model configuration. That model seems to be based around immutable state, with a dependency resolver that schedules execution of configuration blocks based on their state dependencies.
But without that resolver/scheduler (which doesn't seem to exist yet), there's not a way to automagically schedule my almanacPublish task configuration after distTar task configuration; at least, if distTar task configuration is done as early as possible (during project evaluation), I can just move my task declaration below that point, instead of moving it into afterEvaluate (which in turn would force anything depending on that task into afterEvaluate, etc.).

I'm still a bit puzzled why this shows up for you, in my almanacPublish blocks I'm just referencing artifact distTar, and I get the right names.

I'm going to close this in favor of #56

...if you want to contribute the configurable classpath thing you're welcome to do so, but obv. doesn't need the rest of the change on this PR.


Created 'internal.provider' package for our provider implementations

Created the 'internal.provider' package which holds the implementation of our provider interfaces. Also cleaned up the JavaDoc in AndroidClientLogger to no longer use '@inheritDoc', as it is not necessary.

@bsiegel Just a note related to package visibility: if we put the provider classes inside an provider.implementation folder we cannot make them package-private, they must be public. To limit access I would suggest keeping them under provider and making them package-private.


在sql语句中，可以使用：

LIMIT来返回第N条记录（SQLCMD LIMIT 4, 1）
COUNT来返回结果的数量

可以新增一个driver api

already done lazy creation

I can't do it!
Follow this example:

call query() --- using lazy strategy, the cmd is not yet executed
modified some data --- data been modified
call fetchone() --- execute the cmd now.

BUT THE DATA HAS CHANGED!!!


Added a timeout condition which will throw a timeout error whenever the server of backplane-api is down

What type of PR is this?
feature
What this PR does / Why we need it?
This PR will throw a timeout error whenever the backplane-api server is down for 10 seconds or more
Which Jira/Github issue(s) does this PR fix?
Resolves #OSD-17197
Special notes for your reviewer
You can test it locally by decreasing the timeout duration to a few milliseconds. It will throw a timeout error.
Pre-checks (if applicable)

[X] Ran unit tests locally
[X] Validated the changes in a cluster
[ ] Included documentation changes with PR

Codecov Report

Merging #158 (6804b5a) into main (75b8c6e) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #158   +/-   ##
=======================================
  Coverage   49.18%   49.18%
=======================================
  Files          51       51
  Lines        3302     3302
=======================================
  Hits         1624     1624
  Misses       1401     1401
  Partials      277      277

Impacted Files
Coverage Δ

internal/github/github.go
59.18% <100.00%> (ø)

/lgtm

/label tide/merge-method-squash

/approve


S3FileSystem ls returning outdated list of keys after S3 bucket contents changed

I've been working on a project that has an S3 bucket that uploads and deletes files frequently (once a day), and just as frequently the contents need to be checked. These files are not managed using versions, and all have their timestamp in the filename. I've been using s3fs.S3FileSystem for checking the contents of this bucket, creating a new instance each time and using the ls method (code snippet at bottom).
What I've noticed is that after about a day or two of running the program, ls will have stopped returning current keys in the S3 bucket, and instead start to return only an old version of this list of keys. It seems this begins to happen after the contents of the S3 bucket have changed while the program is still running.
I've included a screenshot from our logging platform showing the discrepancy between what is returned using s3fs.S3FileSystem's ls, vs boto3.client("s3")'s list_objects. What's included is the timestamp of the log messages, as well as the timestamps attached to each file to show that there's quite a bit of difference from the two requests. These requests were not made while the contents of the S3 bucket were being changed.

I've blanked out project related information, so apologies for the readability. The last two keys have static names, and as you may be able to see, the timestamps on the other keys are about a day out of date using s3fs compared to boto3 when listing objects in the same S3 bucket.
The code used for fetching the list of keys using s3fs (excluding keys and some logic) was
def fetch_datasets():
  s3 = s3fs.S3FileSystem(key=..., secret=..., client_kwargs=...)
  filenames = s3.ls(...)
  return filenames

The code used for fetching the list of keys using boto3 to compare with (excluding keys and some logic) was
def fetch_datasets():
  s3 = boto3.client("s3", aws_secret_key_id=..., aws_secret_access_key=...)
  query = s3.list_objects(Bucket=..., Prefix=..., Delimiter=...)
  filenames = [<list comprehension for getting CommonPrefixes>]
  return filenames

As I'm creating a new instance of the S3FileSystem class each time, I wouldn't expect ls to return an outdated list of keys. This seems only to happen when the S3 bucket contents have been updated since program start, and the keys have been fetched using ls at least once before.
If I'm missing something about this behaviour, or if there's anything I can add, please let me know!

s3fs by default caches filename listings, since fetching them is generally an expensive operation. You can do filenames = s3.ls(..., refresh=True) or fs.invalidate_vache() to explicitly prevent this at each call.
Furthermore, you can control the directory caching behaviour via the kwargs use_listings_cache and listings_expiry_time (see fsspec.dircache.DirCache).

Apologies for the late response. That helps, thank you! I must have missed that when googling


Revision Fetch Failure

On my node I noticed that I am sometimes unable to fetch a revision because of the following error "timeout while blocking for price table update", which essentially means that a timeout occurred while we're waiting on another thread to update the price table.
The timeout can occur for multiple reasons but one of those reasons might be the withdrawal timeout. In any case, we don't want a revision fetch to fail at the fetchByAccount stage because it was unable to fetch a price table (likely also attempting to fetch it using an ephemeral account).
We have to ignore the timeout error, but also have to swap the ongoing update with one that updates using a revision, because otherwise we might run into the same error when trying to fetch a revision using contract payments since we still don't have a valid price table.
df160ce2: unable to fetch revision with account: LatestRevision: failed to fetch pricetable, err: context deadline exceeded; timeout while blocking for pricetable update
06917fbc: unable to fetch revision with account: dial tcp <IP_ADDRESS>:9983: connect: no route to host
77607c85: unable to fetch revision with account: dial tcp <IP_ADDRESS>:9983: connect: no route to host
288b7811: unable to fetch revision with contract: dial tcp: lookup seafile.tolbxela.com: i/o timeout
90775f8d: unable to fetch revision with account: dial tcp <IP_ADDRESS>:9983: connect: no route to host
4f41278f: unable to fetch revision with contract: dial tcp: lookup mysiaworker.ddns.net: i/o timeout

not comfortable doing the pointer swap, closing for now


Deprecate extensions/v1beta1.Job

Moved to 1.5 from #1634.
@erictune fyi
@kubernetes/docs ptal

This change is

I'm applying tech approval label based on previous PR.


Make MigrationHop MSONable again

Taking over a previous pull request that was abandoned: https://github.com/materialsvirtuallab/pymatgen-analysis-diffusion/pull/264

Pls fix the test errors. The linting errors can be dealt with later, but the unittest must pass.

Pls merge the latest master. A lot of the problems will disappear.

The tests fail because there are changes in your PR with host_symm_structure. These need to be fixed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
136 unchanged lines in 6 files lost coverage.
Overall coverage remained the same at 81.363%

Files with Coverage Reduction
New Missed Lines
%

pymatgen/analysis/diffusion/utils/edge_data_from_sc.py
4
94.81%

pymatgen/analysis/diffusion/utils/supercells.py
4
90.38%

pymatgen/analysis/diffusion/neb/pathfinder.py
10
96.18%

pymatgen/analysis/diffusion/aimd/rdf.py
16
81.91%

pymatgen/analysis/diffusion/neb/full_path_mapper.py
47
81.01%

pymatgen/analysis/diffusion/analyzer.py
55
61.42%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1659

Relevant Lines:
2039

💛  - Coveralls

@shyuep would it be possible to release an updated version of pymatgen-analysis-diffusion with this change on PyPi? I am working on code in emmet that depends on this patch.

Done.

Great! Thank you so much @shyuep for your responsiveness in getting this fix addressed in a timely manner.


Topic usage

Hi @hzwirnmann. Just a short note that when/if you're ready, you can add the "ome-tiff" topic to be listed with other repos like these: https://github.com/topics/ome-tiff
All the best,
~Josh

Hi @joshmoore and thanks for the hint. I added it as you suggested,


Recommit - badphotography

@proofconstruction Could you try again on this ? I reforked the repo and it shouldn't having any issue now.


container.attach writes attachOptions into container's stdin

When writing into container's stdin sometimes dockerode automatically writes attachOptions infront of the actual input.
Does anybody have the same issue and a possible solution?
Input
{"stream":true,"stdin":true,"stdout":true,"stderr":true,"hijack":true,"logs":false}say hello
Code
const container = docker.getContainer('720736bad7ecc2f1ced7364366a69e240d1a6ad5dea02f9a77f3c2b0bb7990ae')

container.attach({
  stream: true,
  stdin: true,
  stdout: true,
  stderr: true,
  hijack: true,
  logs: false
}, (_, stream) => {
  // Map output to console
  stream.pipe(process.stdout)

  const string = 'say hello\n'
  // Write data to the container's stdin
  stream.write(string)
})```

Hi! We are also having the same issue. Any updates in the past 4 months? Are you using a Mac by any chance?

@Luluno01 No unfortunately not.
I ended up writing it myself from scratch with the Docker API
Are you trying to connect to a Minecraft Server in a docker container?
I think the problem only occurs in this specific case.

@Luluno01 No unfortunately not.
I ended up writing it myself from scratch with the Docker API
Are you trying to connect to a Minecraft Server in a docker container?
I think the problem only occurs in this specific case.

@Luluno01 No unfortunately not. I ended up writing it myself from scratch with the Docker API
Are you trying to connect to a Minecraft Server in a docker container? I think the problem only occurs in this specific case.

Thanks for the information. No, I'm just spinning up a container running a Node.js application and writing some inputs via stdin. I would not call my case as some "specific" case, though.

Hi, I find this problem might relate to docker version. Would you mind sharing your docker version? Thanks!


[BACKEND][TRANSFORM] add a pass to prefetch around a loop

This pass add prefetch in the loop pre-header( 3 stages in advance) and add prefetch in the loop.

@whitneywhtsang I am done with the refactoring of the pass. Please take a look.


Anchor tag not working in zoho mail

How can we reproduce this bug?
Write out the HTML (or Inky code) that causes the issue.
What did you expect to happen?
What happened instead?
What email clients does this happen in?

+91 ${loggedUserNumber}


apk add --update py-pip fails

I'm running through the Docker tutorials. When building the flask image the dockerfile command
RUN apk add --update py-pip
fails with
ERROR: unsatisfiable constraints:
py-pip (virtual):
provided by: py2-pip
required by: world[py-pip]
If I log into the container and update py2-pip explicitly then it all works fine.
It looks like there is something going wrong with the way py-pip is resolving to version 2.
Once py2-pip is installed the update py-pip goes through fine.
Since this command forms part of the official Docker self-paced training, this issue does need looking at as I have lost several hours tracking it down and not learning about Docker :(

Meh - looks like this was fixed in the tutorial code base but it hasn't been pushed live!
https://github.com/docker/labs/commit/23025ea977a32256ac0ffa4d6e5507b502baf169


Minor fixes  (improved)

Following previous suggestions, I'm no longer applying -moz-win-glass to all windows (no longer works on PiP as of 105), and drag space is now an option.
So:

Added - userChrome.DragSpaceAboveTabsWindowedMode-Decreased option
Fixed - Rounded corners appear on fullscreen windows (with mica enabled)
Fixed - Findbar (control+f) has an incorrect background color (with mica enabled)

You can now control the size of the drag space.
I didn't want to change the findbar. This is a bug. I'll fix it soon. It should display as Mozilla intended.

Thanks for the suggestions and feedback.


NodeID

What is the recommended method for assigning NodeID? If we use a deterministic method, and a node is removed, then it cannot be added back with the same NodeID (rej exist error in log). Our nodes are managed using Kubernetes and Helm, which doesn't provide much mechanism for manually assigning NodeID's.

You are free to choose any NodeID you like. NodeID is just a numeric value picked by your code, I don't think it has anything to do with software like Kubernetes.
Note that in the context of raft/dragonboat, removing a node means using membership change operation to permanently remove a node from the raft cluster. If you have a crashed node that can be restarted, that node is not "removed" when it is offline.
Not allowing removed NodeID to be reused by a new Node is not a restriction introduced by Dragonboat, it is a fundamental safety requirement of the Raft protocol. Imagining that you have a node with ID 100 and let's say that node has all entries up to index 200, if you remove that node and then add another new node with the same ID 100, that will confuse all other nodes in the same Raft as node with ID 100 previously confirmed to the cluster that it has all entries up to index 200, they are now all gone on the new node with the same ID 100.

Thanks!

Now I am trying to set up automatic deployment of the cluster and I need to somehow automatically set the identifier for the nodes
So far identifiers are set when adding each node manually, but due to the fact that some node could be removed, it may turn out that the specified identifier of the node may already exist in the cluster, then to the addition will be completed with an error
There is an idea to increment the identifier at the level of the cluster logic and not set it manually, but in order for such logic to be consistent and the identifiers do not intersect, a quorum condition is necessary, which is undesirable when changing the composition of the cluster, are there any ways to achieve automatic assignment of cluster identifiers?


About Activate power mode

The Activate power mode plugin is popular in China this days:100: . Look at this:http://weibo.com/1627825392/D6ejNvxVf?type=comment#_rnd1449050912727
a little advice: I think it's better if the window won't shake. because I feel really dizzy. :100:

Fixed in 0.4.0.


Some minor tweaks

A few other questions/comments:

Is the PC scheme Hydrax-PC? I think it would be good to mention it explicitly.

Have you thought about using VartimeEdwardsPrecomputation for commitments? I think most users would be happy with that tradeoff (faster proofs, slower precomputation, a bit more memory), though maybe not 100%.

To answer your questions:

Is the PC scheme Hydrax-PC? I think it would be good to mention it explicitly.

Yes, I'll add it to README.md

Have you thought about using VartimeEdwardsPrecomputation for commitments? I think most users would be happy with that > tradeoff (faster proofs, slower precomputation, a bit more memory), though maybe not 100%.

I did try this. But, I didn't notice any improvement in proof generation times. Maybe I can push what I did on a separate branch?

One thing to note is that VartimeEdwardsPrecomputation trades space for time pretty aggressively, but for very large multiscalar multiplications the precomputation may be too large to help much.

Yes, thanks @hdevalence! I had forgotten about this from our email exchange a few months ago.


How to change doughnut segment color on click?

Hello!
I need change dougnut segment color on click. I try to use onClick event for this purpose. But settting of new backgroundColor has no effect on chart. I'm not sure, that my way is correct. But I guess, there is no such feature in chart.js.
There is my event handler:
options: {
	onClick: function(evt, elements) {
        if (elements && elements.length) {
			console.log(elements[0]._model.backgroundColor);
			//elements[0]._model.backgroundColor = "#ff9933" - it doesn't work
        	}
	}
}

Here is my plunk  for convenience.
Thanks!

@greatromul here's how to do it: http://plnkr.co/edit/ESLfihjIYRNLFjFpk5Jd?p=preview

@etimberg Great Job, with it!!
But there is some issue, unable to remove label from legends which works fine otherwise


Fix the phpdoc blocks

Here are the unfixed/unfixable errors
|  ------ -----------------------------------------------------------------
|   Line   src/Core/Manifest/Module.php
|  ------ -----------------------------------------------------------------
|   287    The method "requireFile" has "1" @param tags but only "0" where
|          expected. on "SilverStripe\Core\Manifest\requireFile"
|  ------ -----------------------------------------------------------------
|
|  ------ ---------------------------------------------
|   Line   src/Forms/FormField.php
|  ------ ---------------------------------------------
|   1038   "\FieldHolder.ss" is not a valid Fqsen. on
|          "SilverStripe\Forms\FormField::FieldHolder"
|   1301   "\{@link" is not a valid Fqsen. on
|          "SilverStripe\Forms\FormField::Type"
|  ------ ---------------------------------------------
|
|  ------ ---------------------------------------------------------------------
|   Line   src/Forms/GridField/GridFieldFilterHeader.php
|  ------ ---------------------------------------------------------------------
|   172    The "arguments" parameter of the method "handleAction" is missing a
|          @param tag on
|          "SilverStripe\Forms\GridField\GridFieldFilterHeader::handleAction"
|  ------ ---------------------------------------------------------------------
|
|  ------ -----------------------------------------------------------------------
|   Line   src/Forms/GridField/GridField_ActionMenuItem.php
|  ------ -----------------------------------------------------------------------
|   28     "\{@link" is not a valid Fqsen. on
|          "SilverStripe\Forms\GridField\GridField_ActionMenuItem::getTitle"
|   40     "\{@link" is not a valid Fqsen. on
|          "SilverStripe\Forms\GridField\GridField_ActionMenuItem::getExtraData"
|   53     "\{@link" is not a valid Fqsen. on
|          "SilverStripe\Forms\GridField\GridField_ActionMenuItem::getGroup"
|  ------ -----------------------------------------------------------------------
|
|  ------ -----------------------------------------------------------------
|   Line   src/Forms/GridField/GridField_ActionMenuLink.php
|  ------ -----------------------------------------------------------------
|   20     "\{@link" is not a valid Fqsen. on
|          "SilverStripe\Forms\GridField\GridField_ActionMenuLink::getUrl"
|  ------ -----------------------------------------------------------------
|
|  ------ ---------------------------------------------------------
|   Line   src/Forms/GridField/GridField_ActionProvider.php
|  ------ ---------------------------------------------------------
|   17     "\{@link" is not a valid Fqsen. on
|          "SilverStripe\Forms\GridField\GridField_ActionProvider"
|  ------ ---------------------------------------------------------
|
|  ------ -----------------------------------------------------------------------
|   Line   src/Forms/GridField/GridField_ColumnProvider.php
|  ------ -----------------------------------------------------------------------
|   23     "\{@link" is not a valid Fqsen. on
|          "SilverStripe\Forms\GridField\GridField_ColumnProvider::augmentColumn
|          s"
|  ------ -----------------------------------------------------------------------
|
|  ------ --------------------------------------------------------------------
|   Line   src/Forms/UploadReceiver.php
|  ------ --------------------------------------------------------------------
|   103    The method "setAllowedFileCategories" has "2" @param tags but only
|          "1" where expected. on
|          "SilverStripe\Forms\UploadReceiver::setAllowedFileCategories"
|  ------ --------------------------------------------------------------------
|
|  ------ ---------------------------------------------------------------
|   Line   src/ORM/ArrayList.php
|  ------ ---------------------------------------------------------------
|   661    The method "filterAny" has "1" @param tags but only "0" where
|          expected. on "SilverStripe\ORM\ArrayList::filterAny"
|  ------ ---------------------------------------------------------------
|
|  ------ -----------------------------------------------------------------------
|   Line   src/ORM/DataList.php
|  ------ -----------------------------------------------------------------------
|   315    The method "sort" has "1" @param tags but only "0" where expected. on
|          "SilverStripe\ORM\DataList::sort"
|   386    The method "filter" has "1" @param tags but only "0" where expected.
|          on "SilverStripe\ORM\DataList::filter"
|   448    The method "filterAny" has "1" @param tags but only "0" where
|          expected. on "SilverStripe\ORM\DataList::filterAny"
|   616    The method "exclude" has "2" @param tags but only "0" where expected.
|          on "SilverStripe\ORM\DataList::exclude"
|   654    The method "excludeAny" has "2" @param tags but only "0" where
|          expected. on "SilverStripe\ORM\DataList::excludeAny"
|   789    "\Generator&DataObject" is not a valid Fqsen. on
|          "SilverStripe\ORM\DataList::getGenerator"
|  ------ -----------------------------------------------------------------------
|
|  ------ -----------------------------------------------------
|   Line   src/ORM/DataObject.php
|  ------ -----------------------------------------------------
|   1215   "\{@link" is not a valid Fqsen. on
|          "SilverStripe\ORM\DataObject::validate"
|   3474   "\$sourceQueryParams" is not a valid Fqsen. on
|          "SilverStripe\ORM\DataObject::getSourceQueryParams"
|   3495   "\$sourceQueryParams" is not a valid Fqsen. on
|          "SilverStripe\ORM\DataObject::setSourceQueryParams"
|   3505   "\$sourceQueryParams" is not a valid Fqsen. on
|          "SilverStripe\ORM\DataObject::setSourceQueryParam"
|   3515   "\$sourceQueryParams" is not a valid Fqsen. on
|          "SilverStripe\ORM\DataObject::getSourceQueryParam"
|   3750   "\$field_labels" is not a valid Fqsen. on
|          "SilverStripe\ORM\DataObject::fieldLabels"
|  ------ -----------------------------------------------------
|
|  ------ ----------------------------------------------------------------------
|   Line   src/ORM/FieldType/DBDate.php
|  ------ ----------------------------------------------------------------------
|   268    The method "Format" has "2" @param tags but only "1" where expected.
|          on "SilverStripe\ORM\FieldType\DBDate::Format"
|  ------ ----------------------------------------------------------------------
|
|  ------ ---------------------------------------------------------------
|   Line   src/ORM/Filterable.php
|  ------ ---------------------------------------------------------------
|   61     The method "filterAny" has "1" @param tags but only "0" where
|          expected. on "SilverStripe\ORM\Filterable::filterAny"
|  ------ ---------------------------------------------------------------
|
|  ------ ---------------------------------------------------------------
|   Line   src/ORM/ListDecorator.php
|  ------ ---------------------------------------------------------------
|   219    The method "filterAny" has "1" @param tags but only "0" where
|          expected. on "SilverStripe\ORM\ListDecorator::filterAny"
|  ------ ---------------------------------------------------------------
|
|  [ERROR] Found 27 errors
|

One of the core issues we have here is that some of the methods in question make use of func_get_args to handle arguments. Our doc blocks in such cases have attempted to document the behaviour implemented by those.
Is it possible to ignore the errors & warnings only for a specific list of functions? That would be better than an across the board "ignore all errors" setting.
Note that amending these to use the new-ish variadic syntax would be nice, but it's a breaking change as it causes errors in overridden methods.

but it's a breaking change

Do you mean changing the actual php method declaration ?
I agree that I need to find a way to have ignore lines for "magic" codebases

Yeah, I meant for the actual code declaration. Tidying up semantics in the docblock for the messier parts of the real declaration is our trade off here... but it means that some docblocks may be an inexact match to the declaration, Hence the errors.

Do I have something to do here ?

Do I have something to do here ?

Thanks William! I've fixed a build failure that you couldn't have predicted or really known how to fix: We've got this *.peg parser thing that generates a PHP file, and the build checks if they're the same (git diff is empty) to ensure we don't forget to commit the resulting generated file. Since you changed the PHPDoc in that generated file, it triggered the build error.

Thanks William! I've fixed a build failure that you couldn't have predicted or really known how to fix: We've got this *.peg parser thing that generates a PHP file, and the build checks if they're the same (git diff is empty) to ensure we don't forget to commit the resulting generated file. Since you changed the PHPDoc in that generated file, it triggered the build error.


fix: ie11 failing

move promises to the execution of the javascript variant
RKT-7590

hotfix-v1.0/RKT-7590-promise-variants


kodi.packages.netflix: 1.14.1 -> 1.15.0

Motivation for this change

version bump
changes in netflix broke the plugin, this version bump fixes it

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 nixpkgs-review --run "nixpkgs-review wip"
[x] 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.

ofborg could build them.


image_types_ota.bbclass: Improve cmdline handling

Enhance the handling of special characters in the cmdline such as semicolons, which bash cannot handle correctly during build time.

We pass a cmdline parameter "slub_debug=FZP,zs_handle,zspage;FZPU"

Since the argument has no blank spaces you can pass it using a single string without the double-quotes. Have you tried this and what error do you get?
The special chars in the kernel command line are --, . and =.
https://www.kernel.org/doc/html/latest/admin-guide/kernel-parameters.html

This is not an issue with bash itself, since this would be a valid scenario for bash, it is actually an issue inside bitbake's shell parser:
Added the following at conf/local.conf:
OSTREE_KERNEL_ARGS:append = " slub_debug=FZP,zs_handle,zspage;FZPU test=foo"
Then bitbake fails during parsing:
    >    return yacc.parse(lexer=lexer, debug=debug), remaining

  File "/home/rsalveti/build/qualcomm/bitbake/lib/ply/yacc.py", line 267, in LRParser.parse(input=None, lexer=<bb.pysh.pyshlex.PLYLexer object at 0x743addc3acb0>, debug=False, tracking=0, tokenfunc=None):
             else:
    >            return self.parseopt_notrack(input,lexer,debug,tracking,tokenfunc)

  File "/home/rsalveti/build/qualcomm/bitbake/lib/ply/yacc.py", line 1049, in LRParser.parseopt_notrack(input=None, lexer=<bb.pysh.pyshlex.PLYLexer object at 0x743addc3acb0>, debug=False, tracking=0, tokenfunc=None):
                                 errtoken.lexer = lexer
    >                        tok = self.errorfunc(errtoken)
                             del errok, token, restart   # Delete special functions
  File "/home/rsalveti/build/qualcomm/bitbake/lib/bb/pysh/pyshyacc.py", line 650, in p_error(p=LexToken(TOKEN,'FZPU',0,0)):
             w('Unexpected EOF')
    >    raise sherrors.ShellSyntaxError(''.join(msg))

bb.pysh.sherrors.ShellSyntaxError: LexToken(TOKEN,'FZPU',0,0)
followed by:
  LexToken(TOKEN,'test=foo',0,0)
  LexToken(COMMA,';',0,0)
  LexToken(Do,'do',0,0)
  LexToken(NEWLINE,'\n',0,0)
  LexToken(TOKEN,'kargs_list="${kargs_list} --karg-append=$arg"',0,0)

ERROR: Parsing halted due to errors, see error messages above

Just the following is enough to avoid the parsing error when tested locally:
-       for arg in ${OSTREE_KERNEL_ARGS}; do
+       for arg in "${OSTREE_KERNEL_ARGS}"; do

@quic-raghuvar can you check if that fixes your issue?

After checking with a full build locally, adding quotes won't really fix the issue as we do indeed require every argument to be specified via karg-append, so we need to iterate over every command line argument, and ; will indeed cause parsing errors.
Can you please just extend the commit message pointing the command line argument used which caused the issue for you? Please describe better the issue and how your change is fixing the problem.

Yes. Using "${OSTREE_KERNEL_ARGS}" in place of ${OSTREE_KERNEL_ARGS} will not recognize each argument individually. It will take all of them as a single string. Also, it is a bash issue. I was able to reproduce the behavior even with a bash script that was ran locally.
I will update the commit message.

I have updated the commit message

Can the same change be propagated to kirkstone branch?

https://github.com/uptane/meta-updater/pull/121/commits/e50b8b0fc4f279bee5ba6a5cbb73aff536cf1416 is the PR on kirkstone


ENH: Add TubeTK as a remote module for ITK

This pull request adds ITKTubeTK as a remote module to ITK.  Compliance level is 3.
The github for ITKTubeTK is at:
https://github.com/KitwareMedical/ITKTubeTK
The dashboard for ITKTubeTK at:
https://open.cdash.org/index.php?project=TubeTK
There are numerous publications and such that cover its capabilities.
The major features are demonstrated in Jupyter Notebooks in the ITKTubeTK/examples directory.
PR Checklist

[X] No [API changes](https://github.com/InsightSoftwareConsortium/ITK/blob/master
[X] No [major design changes](https://github.com/InsightSoftwareConsortium/ITK/blob/master
[X] Added test (or behavior not changed)
[X] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[X] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

@aylward Could this be moved to the InsightSoftwareConsortium?

Of course!    Is that what is required / preferred?
On Wed, Mar 4, 2020 at 8:10 AM Hans Johnson<EMAIL_ADDRESS>wrote:

@aylward https://github.com/aylward Could this be moved to the
InsightSoftwareConsortium?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/InsightSoftwareConsortium/ITK/pull/1676?email_source=notifications&email_token=AACEJL5BLAP4QWNTEKKZSFLRFZHKTA5CNFSM4LA3RUB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENXYROY#issuecomment-594512059,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACEJLYMCFBZ76LHOLPR7FTRFZHKTANCNFSM4LA3RUBQ
.

--
Stephen R. Aylward, Ph.D.
Senior Director of Strategic Initiatives
Kitware: Advancing the frontiers of understanding
by developing innovative open-source software platforms
and integrating them into research, processes, and products.

@aylward It is preferred (at least by me :)) when there is no strong reason to the contrary.   Rather than only Kitware employees, the ITK developer community can also address pull requests/merges/issues.  Small things like compiler warnings or code style changes become more cumbersome for the ITK developers/contributers to address quickly when a kitware employee must be in the loop.  Not a big deal, but a minor process simplification.


Issue with ImportError for pycrreach in compute_reachable_set.py

When I run the compute_reachable_set.py file, I encounter an ImportError: cannot import name 'pycrreach' from 'commonroad_reach' error. Afterward, I tried running pip install -v . in the main directory, but the error persists. I couldn't find the pycreach file in the commonroad_reach folder. Has it been deleted?

The pycrreach shared library will be installed to the CommonRoad-Reach installation in your site-packages directory. It looks like you are not running the installed version, but rather the package as contained in the cloned repo (which does not come with the shared library).
If you want to modify the tool, please check the README_FOR_DEVS on how to do an editable install. This should also solve your issue.

sorry, I checked my site-packages but I didn't see pycrreach. Can I download it directly?

Please provide more information on what you did. Did pip install -v . complete successfully or were there any errors? After successful installation, the shared library should be in site-packages/commonroad_reach.

Can I download it directly?
No, but you can install it as part of our PyPI package as described in the README. If you do not need to modify the tool, this is the preferred way. Note that we currently only provide pre-built wheels for Linux (and not for Windows or MacOS).

Hello Sir,
I used the command pip install -v . to download, but it showed that the environment variable CRDC_DIR could not be found. So I changed the statement and was able to download.

However, after the download was successful, I also could not find it in site-packages/commonroad_reach. Using pip install commonroad-reach did not work either, and I am unable to run compute_reachable_set.py.

Judging by the output, it looks like the installation completed successfully. I cannot think of a reason why the shared library is not installed in this case. Could you please paste the output of ls /home/xxx/anaconda3/envs/commonroad_reach/lib/python3.8/site-packages/commonroad_reach here (where xxx is replaced by your user of course)?
From which directory are you running compute_reachable_set.py? If you are running the script from the root directory of this repo. The Python interpreter will prefer importing the local version of comonroad_reach (from the directory commonroad_reach in this repo) rather than the installed version from site-packages. You can try renaming this directory to something unrelated (e.g. _commonroad_reach) and check if the script works then. If this is the case, you have the problem that I mentioned in my first post. Then I would recommend doing an editable install as described above.

Thak you sir，i rename this directory and it works，but there is still a question，I still have two questions. The first is that when I use compute_reachable_set, it shows that I haven't provided the lanelet network parameter. How should I modify it? The second is that when I use test_collision_checker, it shows that I don't have the collision_checker_cpp parameter. Can you tell me how to resolve this?

I'm happy that the tool now works for you. Renaming the directory is just a temporary solution; please consider doing an editable install, in particular, if you plan to modify the tool.
It looks like you are using an older version of commonroad-reach that is not compatible with the latest release of our route planner (we released a new version that changes its interface ~2 weeks ago). Please do git pull to update to the latest version, where we fixed this issue.

Thank you, sir. I am now able to run.
I have provided two tips for others who may encounter the same issue:
First, modify the path to ensure that the commonroad in site-packages is called when the file is running.
Second, after creating the conda environment, follow the instructions for "Building the Code" to install it.

Hello, sir, thank you for your tools and assistance. I am very interested in your project. Currently, I am most interested in the collision_checker part, but I still have two questions: First, I found that in the test_collision_checker.py file I am looking for, I do not know how to run it, how should I modify the collision_checker_cpp part within the function? Second, when I am reviewing the collision_checker class, what I want to know is whether the source code of the pycrcc.cpython-38-x86_64-linux-gnu.so library is open source, and can I view it?


当时序不存在的时候能否直接返回空

IoTDB> select variable,number,save,value from root.product.1_data.57 where   time >=1664146800000 and time <=1664153923031 and variable in  ( 'Location') order by time  asc
Msg: 411: Error occurred in query process: Unknown time series root.product.1_data.57.variable in where clause
因为设备未报数据过来所以 表内是没有数据的，如果要过滤某个值就会报错，这里能不能处理成跟time一样 没有值就返回为空

@JackieTien97

IMO, it is different between "no data" and "no schema".. Users may need to do some other action when they meet  the latter case.

Thanks for reporting, we will optimize this
https://issues.apache.org/jira/browse/IOTDB-4576

Has implemented in this PR.


Determine whether entered environment closely matches an already existing one

When the user enters details of an environment they'd like to create, we should be able to tell them beforehand that there already exist x environments which are close matches.

There is now a list of matching environments shown on the Create Environment screen.


Make generate_guid faster

Overview
generate_guid can be called many times during a single Rails request - in our app, it's called 70 times just in middleware, and over 1000 in some heavy Rails actions. This might shave a millisecond or two off those requests.
require 'bundler/inline'

gemfile do
  source 'https://rubygems.org'
  gem 'benchmark-ips', require: 'benchmark/ips'
end

module GuidGenerator
  HEX_DIGITS = (0..15).map{|i| i.to_s(16)}

  module_function

  # generate a random 64 bit uuid
  def generate_guid length = 16
    guid = ''
    length.times do |a|
      guid << HEX_DIGITS[rand(16)]
    end
    guid
  end

  def generate_guid2 length = 16
    guid = rand(16**length).to_s(16)
    guid.length < length ? guid.rjust(length, "0") : guid
  end
end

Benchmark.ips do |x|
  x.report("old version") { GuidGenerator.generate_guid }
  x.report("new version") { GuidGenerator.generate_guid2 }
  x.compare!
end

Warming up --------------------------------------
         old version    50.760k i/100ms
         new version   278.462k i/100ms
Calculating -------------------------------------
         old version    507.072k (± 0.3%) i/s -      2.538M in   5.005250s
         new version      2.785M (± 0.2%) i/s -     14.202M in   5.099937s

Comparison:
         new version:  2784661.4 i/s
         old version:   507071.9 i/s - 5.49x  (± 0.00) slower

Reviewer Checklist

[ ] Perform code review
[ ] Add performance label
[ ] Perform appropriate level of performance testing
[ ] Confirm all checks passed
[ ] Add version label prior to acceptance

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

Thank you for making this pr to contribute to the ruby agent. I see that the target of this PR is the main branch, but that branch is only used for releasing new versions. Could you update this pr to target the dev branch please? Thank you!

OK, that's updated.

We've merged in a branch to dev that resolves the errors related to rails 7, so if you could merge dev back into your branch to get those tests passing again, that would be great! Thank you.

Hi @jdelStrother, this improvement has now made it out to RubyGems in the v8.4.0 release. Thanks again for your contribution!


Update the nightly toolchain to 2022-06-10.

This contains 3 categories of changes to avoid warnings and failures:

clippy now warns if you derive PartialEq without also deriving Eq. All of the types that triggered this warning can implement Eq, so I added Eq derives.
Miri no longer allows integer-to-function-pointer casts. I tweaked the subscribe implementations to avoid casting through usize.
clippy added a warning for expressions of the form let var: () = expr();, but we did that intentionally as part of a compile test. I disabled that warning for the relevant test case.

bors r+

So clap_derive deprecated things, not clap. Rather than adding clap_derive to Cargo.toml to pin it as well, I'm going to try to fix it forward (migrate code to the non-deprecated things).

bors r+


Drop markusthoemmes and warrenvw from OWNERS

As per title.
/hold
Until next week.

/unhold
This is actually fine.


Bigtable: 08. Resource Prefixes

Regenerate Bigtable GAPIC stubs to enable resource headers.
The googleapis gapic changes were sync in this commit:
https://github.com/googleapis/googleapis/commit/de4a5ff6534892029e083d7c0651e06e98c51729
The related api-client-staging PR is:
https://github.com/googleapis/api-client-staging/pull/613

@andreamlin or @garrettjonesgoogle please merge this when you have a chance. It's a simple regen


Add new mode "WINDOW" for compressible agent

Add a mode, with the possible name "WINDOW", that provide a sliding windowing view of the chat history, for example, only the last 5 messages is reserved in the chat.
@bioshazard Are you still interested?

TY for the tag, definitely. If someone doesn't beat me to it, I'll probably get around to it after Thanksgiving.

Compressible agent will be deprecated.


Choppy scrolling in scm view with many files

Issue Type: Bug
repo

With a lot of files in the scm view
On a tall monitor so that many files are visible at a time
Scroll though the scm file list

Bug
Scrolling is noticeably choppy. When profiling, it looks like the creation of context keys is taking a lot of time for each scroll update

Here's my profile
Profile-20181206T173038.json.txt
VS Code version: code-oss-dev 1.30.0 (Commit unknown, Date unknown)
OS version: Darwin x64 18.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
16.00GB (0.09GB free)

Process Argv
.

Screen Reader
no

VM
18%

Will be fixed by https://github.com/Microsoft/vscode/issues/64503


just like this?

running on Android: examples/Basic 0.8.0
just like this:

How can I get a result like this effect？：
https://github.com/500px/500px-android-blur

cc @cmcewen

@cdmalcl try changing the overlay color alpha or the blur radius to change the effect

I did, not just like the right effect. And the blur radius, no change after the change.

Hey guys! Seems that nothing happens in this issue for more than two months. I'm about to close it for now. Feel free to re-open it if needed.


[vue3.x] Control the value in two different inputs

Similar to #408 and #592.When I move the slider around the input values update but if I change the inputs values the slider doesn't update.
Here's my code:
    <script setup>
    import { ref} from 'vue'
    import VueSlider from 'vue-slider-component'
    import 'vue-slider-component/theme/material.css'
    const value = ref([0, 100])
    </script>

    <template>
      <input type="text" v-model.number="value[0]">
      <vue-slider ref="slider" v-model="value" :contained="true" />
      <input type="text" v-model.number="value[1]">
    </template>

Here'e what it looks like:

But it should look like this:

What's also strange is if I check the Vue devtools the value updates:

As #408  issue says, you can't change value[0] responsively by modifying it directly - that's a problem with the vue framework itself.

As #408 issue says, you can't change value[0] responsively by modifying it directly - that's a problem with the vue framework itself.

Since that was on an older version of Vue I was hoping they might have fixed it.
Regardless do you have a workaround for Vue 3?
Thanks.

@charle-connoringold You can make a new copy of the value, like this:
<input :value="value[0]" @change="value = [+$event.target.value || 0, value[1]]">
<input :value="value[1]" @change="value = [value[0], +$event.target.value || 0]">

@NightCatSama Thanks for that!
I wanted to prohibit slider crossing so I turned it into a function. If anyone else runs into this here's the code:
<script setup>
const handleChange = (val) => {
  if (Number(val[0]) >= Number(val[1])) {
    return
  } else if (Number(val[1]) <= Number(val[0])) {
    return
  }
  value.value = val
}
</script>

<template>
  <vue-slider v-model="value"  :enable-cross="false" />
<input class="max-w-[100px] text-center py-0 md:py-1" type="text" :value="value[0]"
  @change="handleChange([$event.target.value || 0, value[1]])">

<input class="max-w-[100px] text-center py-0 md:py-1" type="text" :value="value[1]"
  @change="handleChange([value[0], $event.target.value || 0])">

</template>


EmfMetaModelExporter should check the registry for existing eclasses before trying to create duplicates

Code where problem occurs.

How urgent is this? Do you need solved for LangDev?

We have a workaround, by manually adding the appropriate mappings (source code)
Would be nicer for the demo if it worked.


Problem with installing netifaces on MacOS

I have a problem installing netifaces 0.11.0 on MacOS. This problem is solved when I changed to netifaces2 0.0.16 (https://pypi.org/project/netifaces2/).

good to know I will try out netifaces2 and if it looks good change the requirement for Hio

removed dependencies on netificaces


all: fix typos

Signed-off-by: Pierre Curto<EMAIL_ADDRESS>
What issue is this addressing?

What type of issue is this addressing?

What this PR does | solves

Yes sorry, my IDE messed up. Resending a PR.
Le sam. 28 janv. 2023 à 10:25, Hajime Hoshi @.***> a
écrit :

@.**** requested changes on this pull request.
Please check the test results.
https://github.com/hajimehoshi/ebiten/actions/runs/4031142660/jobs/6930260852#step:6:34
—
Reply to this email directly, view it on GitHub
https://github.com/hajimehoshi/ebiten/pull/2558#pullrequestreview-1273876833,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACZ4ZCGZP7F3MUF3FZAUWDWUTQZLANCNFSM6AAAAAAUJRLSGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

What's the difference between this and #2559?

Same. I may have broken things since I amended the previous commit.
Le sam. 28 janv. 2023 à 10:38, Hajime Hoshi @.***> a écrit :

What's the difference between this and #2559?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


[1.4.0-M1] does not properly handle source files containing special characters

Have a scala file named "`packageTable`.scala" (the filename includes the back ticks; it is generated by some Slick Codegen tool for a table named package).  The back-tick is encoded as %60, but then not properly matched against the filesystem:
[error] IO error while decoding /Users/.../project/com.kariusdx.lis-db/main/com.kariusdx.lisdb/slickdb/table/%60packageTable%60.scala with UTF-8: /Users/.../project/com.kariusdx.lis-db/main/com.kariusdx.lisdb/slickdb/table/%60packageTable%60.scala (No such file or directory)

@jeffrey-aguilera Thanks for the report.


Custom Controls in Mat Table headers with matSort support

Bug, feature request, or proposal:
Proposal
What is the expected behavior?
Be able to add custom controls to  headers along with matSort

What is the current behavior?
It is possible to add custom controls e.g.
<ng-container matColumnDef="to">
    <mat-header-cell *matHeaderCellDef>
      <div class="vbox">
        To
        <input [(ngModel)]="filter.to">
      </div>
    </mat-header-cell>
    <mat-cell *matCellDef="let element">{{element.to}}</mat-cell>
  </ng-container>

However this does not work with matSort
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://goo.gl/wwnhMV
What is the use-case or motivation for changing an existing behavior?
I need individual filter controls for each column in the table, all the examples use a single filter field.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Is there anything else we should know?

This is possible today, it's just not documented. You just need to:

Have your custom control implement MatSortable
Inject MatSort into your custom control and call register
Subscribe to the sort state subjects to respond the other control instances changing the sort state


Testing Issue (Test passes even when it shouldn’t)

Coding Interview Prep - Data Structures;
The testing of challenges in the aforementioned section always passes (all the test-cases pass and I am able to move ahead to next challenge) even if the code is clearly wrong.
I have observed this issue for the past couple of days.
Thanks

Can you be more specific about which challenges and which code you are using for the challenges?

I have observed this issue in a number of challenges in the Data Structures section under the Coding Interview Prep category.
The issue mentioned above starts (from what I can see) from the 'Perform a Union on Two Sets' challenge and ends at 'Use Spread and Notes for ES5 Set() Integration' challenge.
I am using the Google Chrome.
And in regards to my code; simply typing anything into the editor gives me a 'Pass'. It even passes when the editor is completely empty.
Thank you :)

Can you post the actual code which passes for the Perform a Union on Two Sets challenge which you feel should not pass?  Thank you.

I'm sorry for the miscommunication.
The issue starts at 'Perform and Intersection on Two Sets of Data' challenge.
So, the below are the codes that I used for the 'Perform and Intersection on Two Sets of Data' challenge:
First example:
//There is no code here at all. The editor is completely empty in this example. Test still passes.

Second example:
let x = 1;

Both the examples above allow me to proceed to the next challenge.
Thanks

@Edwinyms I see what you mean on the production site.  There were some changes made a while back on the master branch of the repo which fix this issue, but the changes have yet to be deployed.  I am closing this issue for now, because I have validated your code fails the tests as it should using the latest master branch code.


[Fix #12061] Support regex in StringLiteralsInInterpolation

Interpolation can occur in strings, symbols, and regular expressions, so update code to support regex and extend code examples in documentation comments.

Thanks!


ENH: stats.Mixture: add mixture distributions for new infrastructure

What does this implement/fix?
This adds scipy.stats.Mixture, a class for mixture distributions.
import numpy as np
import matplotlib.pyplot as plt
from scipy.stats import Mixture, Normal

vars = (Normal(mu=-1.2, sigma=1.1),
        Normal(mu=1.3, sigma=0.9))
weights = (0.4, 0.6)
X = Mixture(vars, weights=weights)
x = np.linspace(-5, 5, 300)
plt.plot(x, X.pdf(x))
plt.hist(X.sample(10000), density=True, bins=30)
plt.show()

Additional information
Questions:

What should the class be called?
What should the arguments be called? Should the random variables and their probabilities be separate lists (as they are now) or, say, a list of tuples?
To avoid copy-pasting all the method documentation, I was thinking that this and ContinuousDistribution should be a subclass of a new superclass StatisticalDistribution, and method documentation would go there. Of course, there will be several other subclasses, like DiscreteDistribution, etc. Should that superclass be created in this PR or shall we do a separate PR and rebase this after that merges?

Thoughts:

There are many ways of making this more efficient at the cost of extra complexity. Let's keep this simple.
Currently this is only for distributions with scalar shapes (not arrays). Separate PR can extend to array shapes.
I was thinking of creating a wrapper class for the old distributions so that they could work with Mixture. Separate PR, though.
I remember a request for zero-inflated distributions. I'd like to enable this by creating a degenerate/Dirac continuous distribution in a follow-up PR. The obvious alternative is to wait until there is a discrete uniform distribution (which could have all its mass at zero).
When there are discrete distributions in the mixture, I suppose the PDF at the support values of the discrete distributions will be infinite? And perhaps this class should also have a PMF method, and the PMF at values that are not in the support of a discrete distribution would be zero?
I'll probably want to test edge behavior with distributions that have finite support (but it doesn't really matter until these are public).
mode may not return the global maximum of the PDF. This is already documented.
See gh-21707, which would allow mixtures to be composed of most existing continuous distributions.

Very nice addition!

What should the class be called?

Mixture sounds good to me.

What should the arguments be called?

No strong opinion, but vars is a built-in function so might as well avoid that. I think the easiest choice could be to make vars and weights position only, so that a nice name could be picked later if deemed necessary.

Should the random variables and their probabilities be separate lists (as they are now) or, say, a list of tuples?

Separate lists as is done now sounds good to me. That also makes the code easier to extend if the weights are extended to broadcastable.

When there are discrete distributions in the mixture, I suppose the PDF at the support values of the discrete distributions will be infinite? And perhaps this class should also have a PMF method, and the PMF at values that are not in the support of a discrete distribution would be zero?

I think that’s a very good idea. With the new framework comes new possibilities, and I’d be very excited to see it support more general distributions in addition to continuous and discrete.
This is somewhat out of the scope of this PR, but allow me to elaborate a bit about being “more general”. A discussion here states that any probability distribution is a mixture of three types of distributions: continuous, discrete, and singular. This is a result of the Lebesgue decomposition theorem. This means a base ProbabilityDistribution (or StatisticalDistribution) class would contain three “basic” methods: pdf for the continuous component, pmf for the discrete component, and cdf as a “catch-all” that includes the singular component if any.
So in this regard, a fully general Mixture class would mix a list of ProbabilityDistributions to produce another ProbabilityDistribution. (The current PR mixes a list of ContinuousDistributions to produce a ContinuousDistribution, if I read correctly.) But I think current one is in good shape and those kind of extensions could come in follow-ups.

vars is a built-in function so might as well avoid that. A easy choice could be making vars and weights position only, so that a nice name could be picked later if deemed necessary.

vars is positional only right now, but I will change the name. (Weird that this isn't caught by the linter.) I thought the weights/probabilities argument should be keyword-only, actually, so I'd prefer to pick a good name now. I went with weights for now because it seems common enough to call the probabilities "weights" in this context, and it's shorter than "probabilities".

The current PR mixes a list of ContinuousDistributions to produce a ContinuousDistribution, if I read correctly.

Yes, because DiscreteDistribution doesn't exist yet. But as mentioned, I plan to extend this to support discrete distributions when the new discrete distribution infrastructure class exists (and that is part of work coming soon).

One thing that could be worth considering is how the mixture distribution works with data fitting.

I can demonstrate how the proposed fit can be set up with these in their current state in documentation, if need be.

import numpy as np
import matplotlib.pyplot as plt
from scipy.stats import Mixture, Normal
from scipy import optimize

# Generate distribution and data
rng = np.random.default_rng(834589345982345)
vars = (Normal(mu=-1.2, sigma=1.1),
        Normal(mu=1.3, sigma=0.9))
weights = (0.4, 0.6)
X = Mixture(vars, weights=weights)
data = X.sample(10000, rng=rng)

# Fit distribution to data
def get_mixture(x):
    u1, s1, u2, s2, w1 = x
    vars = (Normal(mu=u1, sigma=s1),
            Normal(mu=u2, sigma=s2))
    weights = (w1, 1-w1)
    return Mixture(vars, weights=weights)

def f(x, data):
    X = get_mixture(x)
    return -X.logpdf(data).sum()

eps = 1e-3
lb = [-np.inf, eps, -np.inf, eps, eps]
ub = [np.inf, np.inf, np.inf, np.inf, 1-eps]
guess = [0, 1, eps, 1, 0.5]
bounds = optimize.Bounds(lb, ub)
res = optimize.minimize(f, x0=guess, bounds=bounds, args=(data,))

# Plot
Y = get_mixture(res.x)
x = np.linspace(-5, 5, 300)
plt.plot(x, X.pdf(x), label='original')
plt.plot(x, Y.pdf(x), '--', label='fit')
plt.hist(data, density=True, bins=30, label='histogram')
plt.legend()
plt.show()

Features to make this easier and/or faster can be added later. So I would suggest that this doesn't need to wait for all that to settle (since it hasn't even been settled for ContinuousDistribution).
Fitting is important, and I've worked on it quite a bit (e.g. writing scipy.stats.fit, several enhancements to rv_continuous.fit, and the overrides of many distribution fit methods). It is pretty easy to make the scope of fitting about as big as all the rest of all the distribution infrastructure and distributions. One thing I'd rather not do is make it too easy - as it is in the old infrastructure - to set up fitting problems that optimizers can't solve, at least not without documentation that admits that fitting is done on a best effort basis. There have been many bug reports about rv_continuous.fit over the years, and many of them boil down to either shortcomings of optimizers or MLE itself. Requiring a tiny bit of user effort goes a long way for these problems.

Made the forum post. We'll run tests after gh-21760 is resolved.


Epoc.js on Windows?

I apologize if this is an ignorant question based on the parameters/scope of this project, but is it possible to install the plugin via npm on windows?
Thanks in advance :)

Hey!
Sorry for the late reply!
I don't have access to a windows machine so I'm not 100% but it might not work because I only added configurations for MacOS in my binding.gyp file.
I should totally add something for windows though, you're right, i'll need to find a way to test it.

Just a thought,(dot, dot, dot)
I was thinking of recommending Emotiv to a Dentist.
I envision, small dental devices that are like miniature CnC machines that
machinists and mechanical engineers use, except they fit in the mouth(<--
Shabang! Mind-blown! Those little machines use Emotiv to read the brain for
their input control system)
Pain can be read by the Emotiv device I'm certain, it should kick up an EEG
storm, lots of high-amplitude noise I guess???.
Now a dental machine can either avoid pain, or hone-in
https://www.google.ca/search?q=honing-in&spell=1&sa=X&ved=2ahUKEwi22cDJ2fLaAhVE3mMKHYzEA5kQkeECKAB6BAgAECo
on the pain(that is: abscess search and destroy)
A device that is conceptually like a modern water-pick.  It can blast
abscess's and drain them with enough time.  To find them, ---> follow the
pain.
If we now point the high energy water-stream at the most painful spot, then
the Emotiv device will register a powerful jolt of activity because of the
pain in the mouth.
This means that we have located a large volume of infection. The tissue
hurts because it is sick.  The computer can now register a position and
rotation for the water-pick based on the sensation of pain.
Now the device can direct a powerful jet of water directly into the source
of infection below the gum-line because the Emotiv device is able to locate
where the pain is coming from and focus in on it by directing the
water-pick.
It will burst the abscess by flooding it with a high-pressure water stream.
By using pain response as a beacon, water-picks can be guided using Emotiv
and now abscesses that are hiding deep around the roots of our teeth can be
located, flushed-out and disinfected.
Now how do we do that?  How do we use the patient's brain to control a
device that will be used to heal the patient?
On Sun, May 6, 2018 at 7:11 PM, Charlie<EMAIL_ADDRESS>wrote:

Hey!
Sorry for the late reply!
I don't have access to a windows machine so I'm not 100% but it might not
work because I only added configurations for MacOS in my binding.gyp file.
I should totally add something for windows though, you're right, i'll need
to find a way to test it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/charliegerard/Epoc.js/issues/3#issuecomment-386938484,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFcPwx6J2tcTMG2m_fKsNunhf23naly5ks5tv606gaJpZM4RUw2r
.

@marcclintdion I think this is not the correct place for your message...

Hey!
quite interested if this project still has plans for this to works on windows

@yotsandev @kinghaseo If you want to have a framework to read raw eeg data from the Emotiv Insight / EPOC / EPOC+, I'd suggest you to have a look at this CyKit repository.


|play command

ytdl
Music stops before end of song
server.dispatcher = connection.playStream(ytdl(server.queue[0]), {
                filter: "audioonly",
                quality: 'highestaudio',
                volume: 1,
                highWaterMark: 1<<25
            });

quality: 'highestaudio'
setting seemed to be what was stopping the video from playing all the way through :)


⚠ Avatar API Endpoint has degraded performance

In 9978f7b, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1688 ms

Resolved: Avatar API Endpoint performance has improved in e417ea4.


Remove relref

Description
Remove a relref to unblock the website build: https://github.com/pulumi/docs/actions/runs/8076715071/job/22065631439?pr=10896#step:9:73
Checklist:

[ ] I have reviewed the style guide.
[ ] If blogging, I have reviewed the blogging guide.
[ ] I have manually confirmed that all new links work.
[ ] I added aliases (i.e., redirects) for all filename changes.
[ ] If making css changes, I rebuilt the bundle.

@cnunciato Looks like @zchase is running into the same issue I was running into---the PR is updating extra code-related files that weren't necessarily intended to be updated.
Zack, did you intend to update the files in static/programs? If not, can you back those changes out?


Error

What is the output link of /viaversion dump?
https://gist.github.com/anonymous/bc3b98316d78c73860b55a7eaf9525d1
Are you using any additional software like BungeeCord? If so, what software and version? (Please list your plugins as well)
PaperSpigot 1.11.2 (build 1104)
plugins: https://gist.github.com/Nishh/af751e1e52cf2536446fa7a01a730b80
How does this error happen? login? Using an item?
On server start (server.properties and bukkit.yml has debug = true)
Is there an error in the console? Use pastebin.com. Is there a kick message?
https://gist.github.com/Nishh/adf63b160e79e331c195fd7b006dbc9f

Should be fixed in the latest devbuild.


add extra_kwargs property

This is a POC at this point. requires comunity input.
Adds a new  extra_kwargs property to allow passing extra settings to kafka clients.

merging to use it internally


Implement JSON handling in separate component

Actual Behaviour
MainActivity is implementing JSON handling
Expected Behaviour
It should be handled in a separate component
Parent Issue : #2152

@ParthivMangukiya Simply make a constructor of your download handler with one required parameter - Context. Then you won't have to depend on any activity and application context
Furthermore, there are some issues in your implementation. I'll point those out and we'll iterate on those. The best way to do that would be to send a PR of WIP status and I'll review and collaborate on that


Collisions are not resolved properly in some cases

Look at calendar of peckato1 in week 19. 10. There are two overlapping events – one starts at 11.00, the next one at 11.45. – rendered in half size, but in the same row. I don’t see anything unusual about these events, it’s actually a typical example of overlapping events.

Another problem with collisions is #76.

Isn't this the same issue?

I don’t think so, see screenshots…

I believe the "special case" is when it doesn't start at the same time.

Yeah, but then it’s really lame mistake. :( I should take a look at this algorithm.

The "algorithm" is a tangled piece of spaghetti, relevant components are due for refactoring.

Uh, it’s even worse, it doesn’t work even when two events starts at the same time. Take a look at #76.
And now see what really happened.

Magic!
@MMajko, shame on you…

The horizontal layout was totally ok, but the vertical had some missing styles, now everything looks good.


class org.springframework.boot.actuate.endpoint.mvc.MvcEndpoint not found in spring boot 2.7.1

When used latest version 2.0.1_2.8.3 with Spring Boot 2.7.1, the following error is reported. A closer look reveals that the missing class  org.springframework.boot.actuate.endpoint.mvc.MvcEndpoint has been removed in Spring boot 2.7.1:

Thanks @iamcaoxudong ,
This was resolved with
https://github.com/taboola/async-profiler-actuator-endpoint/pull/22


Uncaught error in MS Edge due to missing PerformanceObserver

Why
We are seeing errors on our Gatsby based website because this plugin is
making use of PerformanceObserver without checking whether the API is
actually existing. Note that this issue only exists in the non-Chromium versions
of Edge.
Error Report in Instana

What
Check for existence of PermanceObserver for those web browsers that do
support window.performance, but not window.PerformanceObserver.
Ideally this plugin should also document the need for a PerformanceObserver
polyfill and its dependency to the first-contentful paint event. Unfortunately
the exact reasoning behind this dependency to PerformanceObserver is not
clear to me and hence I didn't start to establish documentation around this
(would be nice if this documentation stated the reason).
Fixes https://github.com/JaeYeopHan/gatsby-remark-images-medium-zoom/issues/11

Sorry to late. thanks pr!


Resolve error in custom job output path variable substitution

At the bottom of the page, there is a provided example on how to specify a customised job output path expression. This example includes the following path expression:
azureml://datastores/${{default_datastore}}/paths/{{$name}}/${{output_name}}
This expression uses a variable substitution of name variable {{$name}} which includes a syntax error. It instead should be ${{name}}, like so:
azureml://datastores/${{default_datastore}}/paths/{{$name}}/${{output_name}}

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

ID: 229c897e-59a1-224e-2e9a-35d2653c65e5
Version Independent ID: ce528b1e-6fe4-f6db-d6d8-b811ade2bb7d
Content: CLI (v2) core YAML syntax - Azure Machine Learning
Content Source: articles/machine-learning/reference-yaml-core-syntax.md
Service: machine-learning
Sub-service: core
GitHub Login: @balapv
Microsoft Alias: balapv

@Liam2032
Thanks for your feedback! We will investigate and update as appropriate.

@Liam2032
Thanks for reporting this! We have created a PR for this issue, and the changes should go live soon.


Fix key name for split names

Split steps were not visible on dataset state

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


[Docs] mesheryctl system update page missing

Current State
No page exists for this command in the mesheryctl Command Reference.
Desired State
A docs page for this subcommand should be available.
Implementation

Create an entry for this subcommand in https://github.com/layer5io/meshery/tree/master/docs/_data/mesheryctlcommands/

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

Hey! @leecalcote I want to work on this. Could you assign this to me?

Sounds great, @Blakelist7 👍

@Nikhil-Ladha I guess we can close this issue as PR #2974 fixes this

@Nikhil-Ladha I guess we can close this issue as PR #2974 fixes this

Yes. Thank you @Blakelist7 .


netopeer2-cli hangs when yes/no is not given when prompted

Hi,
when listen is initiated, it prompts user to supply yes/no for the host key verification part.
If user enters y or any characters other than yes/no it hangs for ever.
After multiple enters, if we supply yes it works, but there is unwanted message it prints (Please type 'yes' or 'no':)
> listen
Waiting 60s for an SSH Call Home connection on port 4334...
The authenticity of the host '::ffff:<IP_ADDRESS>' cannot be established.
ssh-rsa key fingerprint is 27:b6:38:df:94:24:d1:9b:16:53:01:5b:ed:c9:05:6a:39:0c:e0:b2.
Are you sure you want to continue connecting (yes/no)? y

^C
[root@fedora /]# netopeer2-cli

# netopeer2-cli
> listen
Waiting 60s for an SSH Call Home connection on port 4334...
The authenticity of the host '::ffff:<IP_ADDRESS>' cannot be established.
ssh-rsa key fingerprint is 27:b6:38:df:94:24:d1:9b:16:53:01:5b:ed:c9:05:6a:39:0c:e0:b2.
Are you sure you want to continue connecting (yes/no)? y

yes
Please type 'yes' or 'no': root@::ffff:<IP_ADDRESS> password:
Interactive SSH Authentication
Type your password:
Password:

Right, if you update libnetconf2, it should work.


Build docker images for ARM

Note: After supporting running with QEMU, building turms-admin publish-turms-admin becomes extremely slow.

Note: After supporting running with QEMU, building turms-admin publish-turms-admin becomes extremely slow.


feat(cli): separate errors into 'user errors' and 'internal errors'

Context

Improves CLI error reporting
Description

This PR:

Introduces an error.rs file that defines the CLI's result + error type (anyhow w/ internal constructors such as UserError).
Updates the rest of the CLI to use this error module.
Improves various error messages in the CLI

Manually testing the PR

Running a selection of commands with erroneous outputs

I think long term It would be better to have interactive recoveries eg "you are already logged in as {current_alias}, logout {current_alias} and log in as {new_alias}? Y/n". But lets keep this for now.

Dw this is coming


Backward compatible upgrade to fix deprecation warnings concerning LooseVersion and pkg_resources for Python3.8+

Description
distutils LooseVersion and pkg_resources are deprecated and not available in later python versions.
This introduces two commits fixing these deprecation for Python3.8+ while maintaining functionality for Python3.7 and before.
NOTE: Reverting the second commit removes the backward compatibility to Python3.7, as importlib.metadata is introduced in Python3.8.
Fixes #
Where has this been tested?

Platform(s): Ubuntu 22.04
Python version(s): 3.7, 3.8, 3.10
Unreal Engine version(s): 4.26
CARLA version: 0.9.14 (fails as expected), 0.9.15

Possible Drawbacks
The second commit bloats the code a bit and should be reverted when python3.7 should not be supported anymore removing pkg_resources entirely.

This change is

Approved, nice deprecation check