tour: New array? What about garbage collection?

Context: https://tour.golang.org/moretypes/15

If the backing array of s is too small to fit all the given values a bigger array will be allocated. The returned slice will point to the newly allocated array.

It would be useful to understand at what cost this newly created array would be.  What happens to the old array?
This might be outside of the scope of a tutorial, but I would find a sentence or two on what happens to the old array useful.
Cheers!

Thanks for the report.

This might be outside of the scope of a tutorial,

I also think it is. The tour is a very lightweight introduction to the language. AFAIK nowhere in the whole tour we venture in discussions on low-level details about memory-management, or the way runtime and the garbage collector work.
For this reason I'm going to close this issue, as I think what you ask is out of scope for the tour.


Add 3 custom sensor examples

Added 3 example custom sensors:

point lidar
naive radar
naive spinning radar

Please also take a look at the accompanying wiki page documentation:
https://github.com/osrf/mbzirc/wiki/Custom-Sensors
The documentation illustrates how new custom sensors can be added to the simulator using 3 approaches :

tweaking existing sensor param (point lidar),
implementing sensor from ground up (naive radar)
adapting existing sensor (naive spinning radar)

These example sensors should also be functional - just need to uncomment out the install cmds in the CMakeLists.txt in their respective folders as instructed in the wiki page.
Another change I made is that I bridged the laser scan topics for lidars (we were only bridging point cloud topics).

The approach looks good and extensible for teams.  I worked through the wiki instructions as well with no issues.


Add testing against NumPy nightlies

This description is a stub and will be updated soon.
This PR enables testing against NumPy nightlies, which were added in #632 and later disabled to be split out here, because some tests were failing on PyPy for Windows. I have yet to figure out a reproducer. It is to be noted that Windows was not tested with PyPy earlier, so it's probably okay to skip or xfail the test, too, if we can't fix it.
Related to #630

The great thing here is that the previous failing test for PyPy 3.9 on Windows does not fail anymore, because the NumPy development version available on https://anaconda.org/scientific-python-nightly-wheels/numpy/files does not support PyPy 3.9 anymore (so, it fails beforehand when installing). The PyPy 3.10 on Windows tests pass, as expected.
I think what's left to do here is to add some logic to determine when to run the nightly tests and to skip them if a wheel is not available for the desired Python implementation + version duo. This might require a bit of hacking and setting up a dynamic GHA matrix with some JSON – I should be able to figure that out.

Hey @agriyakhetarpal, sorry I got a bit lost and am trying to catch up. I had a look at the failing windows pypy test and this looks quite odd to me. To me it is still not clear to which degree we need to explicitly ensure that the code is also working properly with pypy considering the additional maintenance cost that comes with it. From my perspective it would probably be more valueable to focus on potential issues with future CPython and NumPy versions so I appreciate your work towards nightly tests. Is there any explict input you need for this PR?

Thanks for offering to see through this, @fjosw! As discussed on 14/10/2024, I'll rework this PR to make it simpler. Putting the plan out in public here:

as a pure Python package, we'll test the NumPy nightlies against just the latest Python version for now (3.13 at the time of writing) on all platforms instead of trying to devise a convoluted solution that tests all Python versions that are available for the nightlies. If that strains our CI (I don't think it would, since our CI is quite fast now), we'll test just Linux and Python 3.13. We'll avoid testing against PyPy for the nightlies, since we are not sure if we have enough time to fix and escalate bugs that can arise on a not-as-conventional Python implementation.
We'll test against free-threaded NumPy on Python 3.13t locally first (I'm yet to do it on my M-series macOS device). If there's a bug, that will be organised as a bug report for NumPy upstream and we can organise a CI job with free-threaded wheels for the three platforms that have wheels. https://py-free-threading.github.io/ is a very helpful resource about free-threading and offers advice on how to set up testing infrastructure related to this.
Since we are testing against the nightlies anyway, we should be fine with a slightly more relaxed upper bound on NumPy instead of the recommended N+3 minor version number (2.4). We can leave it as <3 or ~2, which helps us since we are in maintenance mode right now.

I'll split out the second and third points into their own, separate PRs. We should be good to go with this PR once I address the first point!

The test_logsumexp1 failure looks quite cryptic to me...

I fixed the tests, and all of them are passing now! Two things I observed:

the combo_check function predates many improvements made in testing infrastructure over recent years, and is quite dated now. I propose that we refactor the test suite to use pytest better and pave the way for it to be nicer for maintainers and new contributors.
when running the tests, the tracebacks do not go back into NumPy/SciPy APIs, but stop at our code itself (I haven't looked into the details as to why).

The combination of both of these things can sometimes restrict debugging (or so I had the experience of) when fixing the failing test – I had to check multiple angles (fwd vs rev modes, randn, keepdims set to True vs False, combo_check) before arriving to a conclusion. If we were to deprecate the combo_check function and change to a few simple pytest parametrization decorators, we would know exactly what sub-test fails in a test, and where – i.e., which would better isolate the failure instead of us having to do it ourselves.
Would something like this be a reasonable proposition?

the combo_check function predates many improvements made in testing infrastructure over recent years, and is quite dated now. I propose that we refactor the test suite to use pytest better and pave the way for it to be nicer for maintainers and new contributors.

I agree with this idea. Presumably we should be able to get roughly equivalent tests using pytest parameterized tests. If you do implement this it might be worth checking that it doesn't slow the tests down (I like the fact that our tests are really fast).

when running the tests, the tracebacks do not go back into NumPy/SciPy APIs, but stop at our code itself (I haven't looked into the details as to why).

This might be from an error occurring during the backward pass of reverse-mode autodiff. When an error occurs during the backward pass you can't get a traceback to the corresponding point in user code. It might be something else though - feel free to paste an example if you hit this again and I can look into it.


How to transfer data between Sessions?

Hi,
I have a login system where the logged in user's level (integer) is stored in the session with session.add_data("user_level", bytes([user_level])). But as the user logs in, it redirects to another served App, which creates a new session and the user_level data is lost. This is my Redirect class:
class Redirect(flx.JsComponent):
    def init(self, dest):
        super().init()
        self.dest = dest

    @flx.action
    def go(self):
        global window
        window.location.replace(self.dest)

How can I transfer the user data from one session to another?
Thanks

To clarify:
I'm trying to make a simple user system, that when a user logs into a session, the user's level is stored in the session, so that when various Widgets are instantiated, they are automatically hidden if the user's level is not high enough.

Ok, so I remembered that the user level data can be transferred by using a cookie and that works great.
But now I don't know how to access this information from a Widget(JS). I could add both the widget's level and user's level as arguments on instantiation, but is there a way to access the user level data from the JS side? Calling self.session.get_cookie inside a Widget doesn't seem to work.

Found that using the Central data store like in the example does the trick:
https://flexx.readthedocs.io/en/stable/guide/patterns.html#use-of-a-central-data-store

I now have a new problem:
When creating my main widget and store object like this:
class Store(flx.JsComponent):
    user_level = flx.IntProp(-1, settable=True)

class MyWidget(flx.PinboardLayout):
    def init(self):
        super().init()

        # Add some other widgets

        if self.root.store.user_level < 2: # <---- TypeError: this.root.store is null
            # do something
        else:
            # do something else

class MainPage(flx.PyWidget):
    store = flx.ComponentProp(None, settable=True)

    def init(self):
        super().init()
        self._mutate_store(Store())

        self.my_widget = MyWidget()

I get an error when MyWidget is being initialized in the init of MainPage.
Is there a way to get around this?

your code seems to work for me, what error do you get?

Hey @almarklein ,
The error is TypeError: this.root.store is null. This seems to happen only if self.root.store.user_level is used inside the init method.

I suspect that the proxy component of the root has no been updated to have the store property. You could delay the init to work around this:
    def init(self):
        super().init()

        # Add some other widgets
        window.setTimeout(self.init2, 0)
    def init2(self):

        if self.root.store.user_level < 2: # <---- TypeError: this.root.store is null
            with self:
                pass
        else:
            pass
        print("ok")

Excellent, that will do nicely 👍
Thanks!


Add kaolin-bubblegum for wezterm.

This adds color scheme toml and recommended retro tab bar configuration.
This part is totally optional, but provides some nice defaults if users
choose.

Hello @theherk,
That looks really good! Amazing job on styling the tab bar. I've tested in on Linux and colors look equally good.
The only thing that I would change; would you mind making the default cursor color white, to stay close to the original? And then keep the blue option keep commented out (as it really plays nice with the theme)?

Thank you for your contribution!


Preset: remove yandex preset

We want to remove yandex preset from jscs.  Can we make this changes in version 2.6.0?
/cc @markelog  @tarmolov

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27932413-preset-remove-yandex-preset?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

Explanation: now yandex codestyle isn't official, i.e. some teams in yandex doesn't use this preset or use it with overrides. Therefore my suggestion on creating repository for jscs-preset-yandex npm package in yandex organization on github was rejected.
In this situation we decided rename codestyle and move npm package for preset to another repo jscs-preset-awesome. Version 1.x of jscs-preset-awesome is backward compatible with yandex preset.
Migration guide for moving from yandex preset:

npm install --save-dev jscs-preset-awesome
Replace

{
    "preset": "yandex"
}

to
{
    "preset": "awesome"
}

in .jscsrc file.

Said news, but now surprise :/

Can we make this changes in version 2.6.0?

It would break backcompat, so only in 3.0

/сс @alt-j

Fixed via https://github.com/jscs-dev/node-jscs/commit/25b92408ae7f5410bf930e2497814538f034270f

Actual link to package with preset https://github.com/ymaps/codestyle/tree/master/packages/jscs-preset-loris


pool_size reset automatically to 25 if you set value more than 1000

Describe the bug
pool_size reset automatically to 25 if you set value more than 1000
To Reproduce
[pools.test.users.0]
username = "test"
password = "pass"
pool_size = 2000
statement_timeout = 0
Expected behavior
In this case auto reload should ignore value and keep previous or service should be in failed state with proper log message
Screenshots
n/a
Desktop (please complete the following information):

Ubuntu 22.04, but I'm sure that it is core problem and lack of documentation

weird. But it was eventually set to 5k. closed


Fixed clang compile error and some warnings.

Fixes the compiler error mentioned in #169

I see that errors that make a mac crash have supposedly been fixed... New to GitHub here and I'm wondering how to download the fixed version?


Minor coding style fixes in the RISCV64 code

@aenrbes I made some minor coding style fixing (formatting, whitespace etc.) to the RISCV64 code. I preferred making those fixes myself to avoid bothering you too many with very "little value" comments.
One change however is non-cosmetic: I moved the tilck.map to the build directory ${CMAKE_BINARY_DIR} because it was written to the root source directory: all the generated files should go in ${CMAKE_BINARY_DIR} because that could be anywhere. Supporting out-of-tree builds is very important.
I believe this small change shouldn't cause any issues as I didn't see any other references to tilck.map but I guess you might have some other code relying on that.

@vvaltchev Thank you so much for doing all the little fixes yourself! I will try my best to fix these problems myself in the next commit.
There is no problem to moved the tilck.map to the build directory.

Thank you for the review and for the positive approach, @aenrbes !


progress bars can be disabled

It would be nice to be able to disable the progress bars you see with lakectl local clone and other bulk operations.
Use case: I have some ML training jobs that run in a Kubernetes cluster. That cluster has logging set up so anything a container logs gets sent to an external logging service. Those services typically expect logs to be in a pretty simple format like 1 line per thing logged or in JSON format. However, the progress bars use special characters (e.g. \r) to update the cool little graphs and that confuses the heck out of the log parsers. It's still nice to have logs like downloaded some/file.txt but the progress bars are problematic. Ideally you'd detect that the lakectl stdout isn't a terminal and if that's the case convert the progress bar logs to "regular logs". If that's not possible a flag like --disable-progress would be great.

Thanks for doing this. It's an improvement but I don't think it really fixes the issue. If you pass --no-progress it doesn't display the ### animations anymore but it does still use \r and \n characters to update the output. For example, initially it might display something like:
download vet-video-labels/clljmlh8j0z5q~ ...
download vet-video-labels/clljmnw6f113m~ ...
download vet-video-labels/clljmdchw2ppv~ ...
download vet-video-labels/clljmjdop2qtg~ ...

But then, later, it moved back up a few lines to update the display to:
download vet-video-labels/clljmlh8j0z5q~ ... done!
download vet-video-labels/clljmnw6f113m~ ... done!
download vet-video-labels/clljmdchw2ppv~ ...
download vet-video-labels/clljmjdop2qtg~ ...

Most logging solutions (e.g. Google Cloud Logging, DataDog, SigNoz, etc.) assume a log message is a single line . That is, it expects a string of unicode characters terminated by a \n. Once it sees the \n that's the log message that gets stored. When the progress messages use meta-characters to move back upwards in the terminal to update existing lines those are new log lines and they're very hard for the likes of DataDog, Cloud Logging, etc. to parse.
I'd like to request that this get re-opened and --no-progress should mean no "updating display" - just log downloading X and then, when the download is complete, log X download complete (or something like that).

Thanks for doing this. It's an improvement but I don't think it really fixes the issue. If you pass --no-progress it doesn't display the ### animations anymore but it does still use \r and \n characters to update the output. For example, initially it might display something like:
download vet-video-labels/clljmlh8j0z5q~ ...
download vet-video-labels/clljmnw6f113m~ ...
download vet-video-labels/clljmdchw2ppv~ ...
download vet-video-labels/clljmjdop2qtg~ ...

But then, later, it moved back up a few lines to update the display to:
download vet-video-labels/clljmlh8j0z5q~ ... done!
download vet-video-labels/clljmnw6f113m~ ... done!
download vet-video-labels/clljmdchw2ppv~ ...
download vet-video-labels/clljmjdop2qtg~ ...

Most logging solutions (e.g. Google Cloud Logging, DataDog, SigNoz, etc.) assume a log message is a single line . That is, it expects a string of unicode characters terminated by a \n. Once it sees the \n that's the log message that gets stored. When the progress messages use meta-characters to move back upwards in the terminal to update existing lines those are new log lines and they're very hard for the likes of DataDog, Cloud Logging, etc. to parse.
I'd like to request that this get re-opened and --no-progress should mean no "updating display" - just log downloading X and then, when the download is complete, log X download complete (or something like that).

Hi @oliverdain thanks for the detailed information.
We don't usually reopen issues which are linked to a PR that was already merged.
Would you mind opening a new issue with this information and we'll take a look at it?
Thanks,

Thank you @oliverdain
@treeverse/product please take a look

I opened #8111


Failed to load native library on Windows with a Chinese user name

mvnd -v
Failed to load native library:jansi-2.4.0-16c474bda59fea77-jansi.dll. osinfo: Windows/x86_64
java.lang.UnsatisfiedLinkError: Can't load library: C:\Users\??\AppData\Local\Temp\jansi-2.4.0-16c474bda59fea77-jansi.dll
Exception in thread "main" java.lang.UnsatisfiedLinkError: org.fusesource.jansi.internal.CLibrary.isatty(I)I [symbol: Java_org_fusesource_jansi_internal_CLibrary_isatty or Java_org_fusesource_jansi_internal_CLibrary_isatty__I]
        at com.oracle.svm.jni.access.JNINativeLinkage.getOrFindEntryPoint(JNINativeLinkage.java:153)
        at com.oracle.svm.jni.JNIGeneratedMethodSupport.nativeCallAddress(JNIGeneratedMethodSupport.java:57)
        at org.fusesource.jansi.internal.CLibrary.isatty(CLibrary.java)
        at org.mvndaemon.mvnd.client.DefaultClient.main(DefaultClient.java:92)

java -version:
openjdk version "11.0.2" 2019-01-15
OpenJDK Runtime Environment 18.9 (build 11.0.2+9)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.2+9, mixed mode)

D:\mvnd-0.7.1-windows-amd64\conf\mvnd.properties :
detail:
         java.home=C:\\Program Files\\jdk-11.0.2
         maven.settings=D:\\apache-maven-3.6.1\\conf\\settings.xml

How should we deal with it?

Could you run java -jar D:\mvnd-0.7.1-windows-amd64\mvn\lib\ext\jansi-2.4.0.jar and report the output ?

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

This is very weird as your first report indicates the jansi library cannot be loaded or does not find the isatty function, but the second indicates that it works.  The mvnd invocation always fail and the jansi call always work or does it depend on the terminal used (cmd, powershelgl, git bash, ... )?

This is very weird as your first report indicates the jansi library cannot be loaded or does not find the isatty function, but the second indicates that it works. The mvnd invocation always fail and the jansi call always work or does it depend on the terminal used (cmd, powershell, git bash, ... )?

I found that the mvnd.exe startup did not recognize the path containing Chinese.
Use mvnd. CMD to start normally
I now use the scheme: rename mvnd.exe (CMD command will identify the MVND command execution file according to the environment variable as mvnd.cmd)

This is very weird as your first report indicates the jansi library cannot be loaded or does not find the isatty function, but the second indicates that it works. The mvnd invocation always fail and the jansi call always work or does it depend on the terminal used (cmd, powershell, git bash, ... )?

I found that the path containing Chinese was not recognized when mvnd.exe started. mvnd.cmd can be used normally.
The scenario I'm using now is to rename mvnd.exe to another name (the CMD command will identify the mvnd command execution file as mvnd.exe based on the environment variable)

So the problem is when the mvnd.exe is in a directory which contains Chinese characters or when the pom.xml is in a directory which contains Chinese characters ?

This is very weird as your first report indicates the jansi library cannot be loaded or does not find the isatty function, but the second indicates that it works. The mvnd invocation always fail and the jansi call always work or does it depend on the terminal used (cmd, powershell, git bash, ... )?

I found that the path containing Chinese was not recognized when mvnd.exe started. mvnd.cmd can be used normally.
The scenario I'm using now is to rename mvnd.exe to another name (the CMD command will identify the mvnd command execution file as mvnd.exe based on the environment variable)

So the problem is when the mvnd.exe is in a directory which contains Chinese characters or when the pom.xml is in a directory which contains Chinese characters ?
It would be nice to be able to reproduce the problem.

This is very weird as your first report indicates the jansi library cannot be loaded or does not find the isatty function, but the second indicates that it works. The mvnd invocation always fail and the jansi call always work or does it depend on the terminal used (cmd, powershell, git bash, ... )?

I found that the path containing Chinese was not recognized when mvnd.exe started. mvnd.cmd can be used normally.
The scenario I'm using now is to rename mvnd.exe to another name (the CMD command will identify the mvnd command execution file as mvnd.exe based on the environment variable)

So the problem is when the mvnd.exe is in a directory which contains Chinese characters or when the pom.xml is in a directory which contains Chinese characters ?
It would be nice to be able to reproduce the problem.

Reproducible conditions are:
The Window OS login user name is in Chinese

I have the same problem，run java -jar D:\mvnd-0.7.1-windows-amd64\mvn\lib\ext\jansi-2.4.0.jar That's the same output

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

Looks like it's caused by https://github.com/adoptium/temurin-build/issues/1496 and has been fixed with https://bugs.openjdk.java.net/browse/JDK-8240823 in JDK 11.0.8
Please upgrade the JDK.

When I encountered this problem, I found the installation directory of Maven. Replace D:\apache-maven-3.9.5\lib\jansi-2.4.0.jar with the lower version of jansi-1.18.jar.then it works.
将maven的安装目录中\lib\jansi-2.4.0.jar替换成较低版本的jansi-1.18.jar.然后就不报错了.

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

After replacement, no error was reported.


Upgrade slf4j

We're currency using a pretty old version of slf4j. I think we should update it, to reduce the probability of conflicts.

I have upgraded several dependencies to more recent versions.


Enlarge droplet to 64GB RAM to avoid memory error

When running the /contrib/crontab/update task, the TraveledSpeedsClassifier was raising MemoryError in it's predict method (line 70).
This is a temporary solve. We should investigate if this memory consumption can be reduced.
This issue is further described in #560.
Fixes:
#556

As these droplets (virtual machines, for those not familiar with DigitalOcean jargon) are short-lived, this might not be only a temporary fix. I think it worth it to deploy and check how much the bill raises at the end of the month. If that is a considerable amount, then we could try to find a better way…
Maybe finally it is time for Dask ✨


Add placeholder option / deprecate title option

A new option - placeholder will function the same as title does currently. placeholder will be available and title will be deprecated in the next minor release.
title will be removed in the next major release. The title option will then behave as it would be expected (a tooltip with the title text will be displayed when hovered over).
Addresses #977.

This change will be a part of the v1.14.0 release. This issue will be updated after it's made available.

Released in v1.14.0-beta!

Hi @caseyjhol,
I tried adding placeholder to option elements, but it is not displayed. Instead whole text of options is shown when it's selected. I'm using regular single select, not the multiple one.


revise-code

LGTM, please just test these scenarios in your queries to make sure there's no issue (as we have 2+ namespace matcher in the query)

metric{namespace=""}
metric{namespace="XXX", namespace="YYY"}
metric{namespace=~".*"}
metric{namespace!~""}

@m-yosefpor do you have any idea? I don't still understand if our label is injected all the time, why did this part of the code exist from the beginning?

namespace!~""
we've tested these scenarios and they're ok.


New Blank File Repair

What problem does this PR solve?
In local and official website testing, it was found that creating a new blank file cannot be parsed. The changes made to address this issue #3762
Type of change

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

Sorry, it's not designed in that way.
Physical file creation is well supported for OS.
Virtual file is just a name, then all the chunks in that files are added and edited manually.


Add Format-Coverage -PesterResults

Following on from https://github.com/dahlbyk/posh-git/pull/461, it would be nice if PR builds could report coverage in their build log even if it's not pushed to Coveralls. That's not currently possible because the Format-Coverage call requires -CoverallsApiToken.
My proposal is pretty simple: add a second parameter set that accepts an existing set of Pester results.

This one was on my list as well, I just added these lines to validate whether or not the provided results contain the CodeCoverage object
if (!$PesterResults.CodeCoverage) {
    Write-Error 'Please provide pester results with code coverage using the -CodeCoverage parameter'
    exit 1;
}

Would you mind if I add them to your commit and merge it like that? Other than that, everything seems in tiptop shape.
Also, funny how a module about code coverage has 0 tests (I'll have to fix that).

Rebased and merged. Thanks for contributing!


Call to undefined function GuzzleHttp\Psr7\build_query()

Following error is produced when running the test.
\sendinblue\api-v3-sdk\lib\Api\ListsApi.php:2185
Call to undefined function GuzzleHttp\Psr7\build_query()
$headers = array_merge(
            $defaultHeaders,
            $headerParams,
            $headers
        );

        $query = \GuzzleHttp\Psr7\build_query($queryParams);

        return new Request(
            'GET',
            $this->config->getHost() . $resourcePath . ($query ? "?{$query}" : ''),
            $headers,
            $httpBody
        );

After updating to 7.4.2 of the SDK, this was resolved.
Note Guzzle & psr7 had to be downgraded to the following
"guzzlehttp/guzzle": "^7.0",
"guzzlehttp/psr7": "^1.6.1",
https://github.com/sendinblue/APIv3-php-library/issues/156

Thanks for reporting this issue
We have released a patch for this issue . Please use the latest version (v7.4.2) of the PHP SDK.
Regards
Sendinblue Team


Overview page should show volume and transfers data

Problem
Overview page needs to show volume and transfers data.

This should be working now


Fix unlinkSync call in runtests.js

I noticed that the call to unlink on L47 was actually asynchronous--you want unlinkSync.   I noticed this because with newer versions of Node.js (8 or newer), calling an async function in a core module without the callback results in a deprecation notice.  Well, more than one...
Executing 1122 testcase(s) with 1 engine(s) using 4 thread(s), total 1085 task(s)
(node:2027) [DEP0013] DeprecationWarning: Calling an asynchronous function without callback is deprecated.
(node:2027) [DEP0013] DeprecationWarning: Calling an asynchronous function without callback is deprecated.
(node:2027) [DEP0013] DeprecationWarning: Calling an asynchronous function without callback is deprecated.
(node:2027) [DEP0013] DeprecationWarning: Calling an asynchronous function without callback is deprecated.
... etc ...

This PR eliminates the torrent of warnings.

Thanks! Please update AUTHORS.rst (first list in the file), see CONTRIBUTING.

updated AUTHORS.rst

Thanks!


Find the 2003 GAO audit of the WARN Act

The Congressional Research Service reports:

In 2003, the Government Accountability Office (GAO; then the General Accounting Office) released a report in which it obtained information on WARN notices from state dislocated worker units and then matched them to mass layoff and plant closure data from the Bureau of Labor Statistics. The report found that of the 5,349 WARN notices issued, only 717 (13%) could be matched to a specific mass layoff or plant closing. The report also identified 1,257 additional events that appeared to require notification under the WARN Act in which it could not find a corresponding layoff notice filed with the state.

That is sourced to:

See Government Accountability Office (GAO), The Worker Adjustment and Retraining Notification Act: Revising the Act and Educational Materials Could Clarify Employer Responsibilities and Employee Rights, GAO-03-1003, September 19, 2003, http://www.gao.gov/products/GAO-03-1003.

Let's find that report and add it to our documentation site.

This is done.


added Intersection observer options root

The element that is used as the viewport for checking visiblity of the target. Must be the ancestor of the target. Defaults to the browser viewport if not specified or if null

possible to add a test case regarding the same?


Vertical mode bug.

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 4.2.0.

Platform/Target and Browser Versions: windows google chrome 64.0.3282.186.

https://jsfiddle.net/Geminis/8hzogg76/

What you did
I copypasted the html from Getting Started With Swiper page and changed the slides to simple colored divs with different heights. I also set the width and the height if the slider itself as it has been done in the getting started guide. so the html is:

The js is:
var mySwiper = new Swiper ('.swiper-container', {
    // Optional parameters
    direction: 'vertical',
    loop: true,

  })

the css is:
.swiper-container {
    height: 300px;
    width: 300px;
}
.box-1 {
	height: 100px;
	background-color: green;
}
.box-2 {
	height: 200px;
	background-color: red;
}

Here is my jsFiddle https://jsfiddle.net/Geminis/8hzogg76/
Expected Behavior
1)I expected the slider to be the same height as I had set in my css.
Actual Behavior
1)The slider has different height and it's larger than I need.

@Zver64 The demo link is not properly function because, instead of the Link to the css file, you've used the HTML Tag. also, the link is incorrect. That should resolve the issue you face.

@shashank1010 I changed it, but unfortunately nothing happened.

@Zver64 it works when you link to swiper properly like @shashank1010 mentionesd. Here's a fiddle with those things fixed: https://jsfiddle.net/8hzogg76/7/


Fixing CI

@TomasVotruba do you think new tag is needed for this change https://github.com/rectorphp/rector-laravel/commit/bcc9f60276a2fc4567a94f51e8ce5369d168b180#diff-2cd0c4d0c03ff28f9516f36a8c1602baee2c5ead80a9734ed1cedf67e2b24642 ?

The require-dev is used only here, it does not affect other packages, so not necessary to tag

@TomasVotruba what I meant is the NodeFactory change on
// decorate with namespaced names
        $nameResolver = new NameResolver(null, [
            'replaceNodes' => false,
            'preserveOriginalNames' => true,
        ]);
        $nodeTraverser = new NodeTraverser();
        $nodeTraverser->addVisitor($nameResolver);
        $nodeTraverser->traverse([$class]);

which doesn't has new tag yet on https://github.com/rectorphp/rector-laravel/commit/bcc9f60276a2fc4567a94f51e8ce5369d168b180#diff-2cd0c4d0c03ff28f9516f36a8c1602baee2c5ead80a9734ed1cedf67e2b24642

Ah, in that case yes :+1:

Can you manage the tagging?

Yes, I added rector-laravel tag 0.11.12 https://github.com/rectorphp/rector-laravel/releases/tag/0.11.12


Remove tab_options from reference (will implement later)

Let's implement tab_options() down the road, and for now remove from the reference, so it doesn't trip people up (There is some quartodoc work needed to get it displaying well in the docs).

Now removed in 951517662b8412597910c77ab3a1213f5aab3af5.


Prevent library from creating a new log file if writing to file is turned off in configuration

Apologies for posting this issue again, but I have not had a reply to my comment on a closed issue.
define ELPP_NO_DEFAULT_LOG_FILE
include "easylogging++.h"
INITIALIZE_EASYLOGGINGPP
int main(int argc, char** argv) {
el::Configurations conf;
conf.parseFromText("* GLOBAL:\n"
        "FORMAT               =  %datetime{%d/%M/%h:%m:%s} %msg\n"
        "FILENAME             =  Logger.log\n"
        "ENABLED              =  true\n"
        "TO_FILE              =  false\n"
        "TO_STANDARD_OUTPUT   =  true\n"
        "LOG_FLUSH_THRESHOLD  =  50 ## Flush after every 50 logs\n"
        "* TRACE:\n"
        "ENABLED             = false\n"
        "* FATAL:\n"
        "FORMAT               =  %datetime{%d/%M/%h:%m:%s} %msg\n"
        "TO_STANDARD_OUTPUT   =  true\n"
        "LOG_FLUSH_THRESHOLD  =  1\n"
        "* DEBUG:\n"
        "ENABLED              = false\n"
        "* INFO:\n"
        "FORMAT               = %msg\n"
        "* WARNING:\n"
        "FORMAT               =%msg\n");
el::Loggers::reconfigureAllLoggers(conf);
return 0;

}
When I compile this and run it, I expect the library not to create the Logger.log file, but it does. How can I prevent it from creating a new file when I set it to only write to std out.

Duplicate of https://github.com/easylogging/easyloggingpp/issues/367

Closing this. see related issue (#367)
Basically it's fixed in v9.85


webpack depenancy uglifyjs-webpack-plugin causing build errors - AWS

Bug report
I am deploying my app to AWS using docker and it has started failing on npm install due to the following error.
aws_beanstalk/staging-app: s/webpack/node_modules/uglifyjs-webpack-plugin > node lib/post_install.js [91m [0mThe command '/bin/sh -c npm install' returned a non-zero code: 1.
Upon further investigation it looks like it is being caused by webpack 3.1.0's dependency on uglifyjs-webpack-plugin 0.4.6. It seems to be a known issue
webpack removed its dependency on uglifyjs-webpack-plugin in webpack 3.4.0. Updating webpack seems like a possible fix but I'm not sure what the repercussions would be in relation to next
System information
Docker image:  64bit Amazon Linux/2.10
node 8.11.2
npm 6.1.0
next 6.0.3

I don't think this is an issue with Next.js: https://github.com/zeit/next.js/blob/canary/package.json#L105
We're already on 3.10.0, and uglifyjs-webpack-plugin is a dependency.

Yes I'm not sure if I was clear but the issue is being caused by the fact that next is using webpack 3.1.0

Oh I had written originally 3.1.0 instead of 3.10.0 sorry for the confusion


Assertion failure in tr_find_character

If mruby is built with enable_debug in build_config.rb, then each of the following inputs can trigger an assertion failure in tr_find_character:
'a'.count('')
'a'.delete('')
'a'.delete!('')
'a'.squeeze('')
'a'.squeeze!('')
'a'.tr('', '')
'a'.tr!('', '')

It looks like tr_find_character does not expect the pattern type to be TR_UNINITIALIZED.
Assertion failed: (0), function tr_find_character, file /Users/clayton/git/mruby/mrbgems/mruby-string-ext/src/string.c, line 371.
Abort trap: 6

This issue was reported by Dinko Galetic & Denis Kasak (https://hackerone.com/dgaletic).

/cc @take-cheeze

Seems it needs specialization for empty pattern.

I fixed it. Soon to be committed.


Prettifies JSON output by default. Closes #1339

Prettifies JSON output by default. Closes #1339

Coverage remained the same at 100.0% when pulling 82ebe0309f0ea4aa70223606b1d6e5f385b7c8e7 on waldekmastykarz:prettify-json-default into 3cd2138030fb26f354403ceaaa489cc4bd11f849 on pnp:v3.

Merged manually. Thank you 👏


fix Error handling for URLs

closes #21
when parse_url() returns false we should throw InvalidArgumentException

Awesome. One quick thing. Could you strengthen the test with an expectation on the error message as well.

Awesome. One quick thing. Could you strengthen the test with an expectation on the error message as well.

done 🚀


starting directory support with XDG Desktop Portal backend

https://github.com/PolyMeilex/rfd/blob/e331eff49a95b095e2e82750af114cf0c81c3dfd/src/backend/xdg_desktop_portal.rs#L199

This was fixed in ashpd 0.3 btw

But it is still not supported for OpenFileOptions

But it is still not supported for OpenFileOptions

That needs to be implemented in the portals themselves, so in https://github.com/flatpak/xdg-desktop-portal/. ashpd is just a rust wrapper for the dbus interfaces.

Issue was opened at https://github.com/flatpak/xdg-desktop-portal/issues/796, i will look into fixing it in the upcoming days

Waiting on https://github.com/bilelmoussaoui/ashpd/pull/90 to get merged.

The replacement PR https://github.com/bilelmoussaoui/ashpd/pull/173 was merged, so this is waiting on a new release of ashpd now.

https://docs.rs/ashpd/latest/ashpd/desktop/file_chooser/struct.OpenFileRequest.html#method.current_folder
Support for default directories has been added in ashpd 0.7.0
I'll try to create a PR for this since

Support for default directories has been added in ashpd 0.7.0

According to https://github.com/bilelmoussaoui/ashpd/releases current_folder support has been added to OpenFile in ashpd 0.6.8, just in case that is relevant in terms of compatibility.
Thanks for contributing to this functionality!

Hmm I can't compile zbus_names 2.6.0 on linux because zvariant::Str<'_>: From<Cow<'_, str>>, is this a known issue? Also occurs with the rfd 0.14.0 crates.io release, no problems on 0.13

Doesn't seem to work. No errors or anything, the picker just doesn't open the given directory.

Doesn't seem to work. No errors or anything, the picker just doesn't open the given directory.

Edit: FileDialog.save_file() somehow still works despite using the same code

Sounds like a KDE portal implementation error/issue

Yup I can confirm it works on GNOME. Although it's weird that the save file works properly cos that means it's only partially implemented.

Would be great to get a new version with this fix, as it currently is a regression when running rfd version 0.14.0 with default features:

with versions < 0.14.0 the the set_directory function is working correctly
with version 0.14.0 the set_directory function is not working
with master the set_directory function is working correctly again


on_message broken.

Description of the problem
Client does not receive any messages. I can't get any output, even with debug logging on.
Code to reproduce
from fbchat import Client, ThreadType, Message
import asyncio

import logging
logging.basicConfig(level=logging.DEBUG)

# Subclass fbchat.Client and override required methods
class EchoBot(Client):
    async def on_message(self, author_id, message_object, thread_id, thread_type, **kwargs):
        await self.mark_as_delivered(thread_id, message_object.uid)
        await self.mark_as_read(thread_id)

        # If you're not the author, echo
        if author_id == self.uid:
            await self.send(message_object, thread_id=thread_id, thread_type=thread_type)

client = Client()

async def main():
    await client.start("[..]")
    print(f"Own ID: {client.uid}")
    await client.send(Message(text="Hi me!"), thread_id=client.uid, thread_type=ThreadType.USER)
    client.listen()

client.loop.run_until_complete(main())
client.loop.run_forever()

Environment information

Python version: 3.7.4
fbchat-asyncio version: 0.2.4

Notes
Maybe this issue has something to do with the original fbchat repo? https://github.com/carpedm20/fbchat/issues/483

version: 0.2.4

use 0.3.0

Using version 0.3.0, I recieve the messages in the debug output, however the client won't echo those messages:

Using Version 0.3.0, I can receive the messages in the debug output, however, the client won't echo them:
from fbchat import Client, ThreadType, Message
import asyncio

import logging
logging.basicConfig(level=logging.DEBUG)

# Subclass fbchat.Client and override required methods
class EchoBot(Client):
    async def on_message(self, author_id, message_object, thread_id, thread_type, **kwargs):
        await self.send(message_object, thread_id=thread_id, thread_type=thread_type)

client = Client()

async def main():
    await client.start("[...]")
    print(f"Own ID: {client.uid}")
    client.listen()

client.loop.run_until_complete(main())
client.loop.run_forever()

Am I doing something wrong?

Do the logs say something related to failing to send and does sending something else (like that "Hi me!") work?

Here are the logs from when I sent a message:
DEBUG:websockets.protocol:client - event = data_received(<360 bytes>)
DEBUG:websockets.protocol:client < Frame(fin=True, opcode=2, data=b'3\xe7\x04\x00\x05/t_ms\x1f {"deltas":[{"attachments":[],"body":"test","irisSeqId":"339047","irisTags":["DeltaNewMessage"],"messageMetadata":{"actorFbId":"100013896135225","folderId":{"systemFolderId":"INBOX"},"messageId":"mid.$cAABa9kzAcjl02lGTOFuqQxHVrwmC","offlineThreadingId":"6605165547370776962","skipBumpThread":false,"skipSnippetUpdate":false,"tags":["source:messenger:web"],"threadKey":{"otherUserFbId":"100013896135225"},"threadReadStateEffect":"KEEP_AS_IS","timestamp":"1574794185528"},"requestContext":{"apiArgs":{}},"class":"NewMessage"}],"firstDeltaSeqId":339047,"lastIssuedSeqId":339047,"queueEntityId":100013896135225}', rsv1=False, rsv2=False, rsv3=False)
DEBUG:hbmqtt.client.plugins.packet_logger_plugin:mqttwsclient <-in-- PublishPacket(ts=2019-11-26 20:49:45.073334, fixed=MQTTFixedHeader(length=615, flags=0x3), variable=PublishVariableHeader(topic=/t_ms, packet_id=7968), **payload=PublishPayload(data='bytearray(b\'{"deltas":[{"attachments":[],"body":"test","irisSeqId":"339047","irisTags":["DeltaNewMessage"],"messageMetadata":{"actorFbId":"100013896135225","folderId":{"systemFolderId":"INBOX"},"messageId":"mid.$cAABa9kzAcjl02lGTOFuqQxHVrwmC","offlineThreadingId":"6605165547370776962","skipBumpThread":false,"skipSnippetUpdate":false,"tags":["source:messenger:web"],"threadKey":{"otherUserFbId":"100013896135225"},"threadReadStateEffect":"KEEP_AS_IS","timestamp":"1574794185528"},"requestContext":{"apiArgs":{}},"class":"NewMessage"}],"firstDeltaSeqId":339047,"lastIssuedSeqId":339047,"queueEntityId":100013896135225}\')'))**
DEBUG:hbmqtt.mqtt.protocol.handler:Add message to delivery
DEBUG:websockets.protocol:client > Frame(fin=True, opcode=2, data=b'@\x02\x1f ', rsv1=False, rsv2=False, rsv3=False)
DEBUG:hbmqtt.mqtt.protocol.handler:Message queue size: 1
DEBUG:hbmqtt.mqtt.protocol.handler:Delivering message <hbmqtt.session.IncomingApplicationMessage object at 0x000001B11DF74A48>
DEBUG:hbmqtt.client.plugins.packet_logger_plugin:mqttwsclient -out-> PubackPacket(ts=2019-11-26 20:49:45.185040, fixed=MQTTFixedHeader(length=2, flags=0x0), variable=PacketIdVariableHeader(packet_id=7968), payload=None)
DEBUG:hbmqtt.client:Waiting message delivery
**INFO:fbchat.client:Message(text='test', mentions=[], emoji_size=None, uid='mid.$cAABa9kzAcjl02lGTOFuqQxHVrwmC', author='100013896135225', created_at=datetime.datetime(2019, 11, 26, 18, 49, 45, 528000, tzinfo=datetime.timezone.utc), is_read=None, read_by=[], reactions={}, sticker=None, attachments=[], quick_replies=[], u**nsent=False, reply_to_id=None, replied_to=None, forwarded=False) from<PHONE_NUMBER>35225 in USER
DEBUG:hbmqtt.mqtt.protocol.handler:0 message(s) available for delivery

The message however did not echo.

@tulir Have you got any updates?


#120 Change hover link of OTH Regensburg logo

Summary
Update the OTH Regensburg logo URL:

German (de): Change to www.oth-regensburg.de
English (en): Change to www.oth-regensburg.de/en

@rdodiya thank you for the contribution!


OrderBy when property name is known at runtime

I want to an AsyncTableQuery at runtime that includes sorted data as requested by the client. The client knows the names of the properties on the pocos, so I figure I could build an expression and pass the expression to OrderBy. I'm missing a step. How can I get from the lambda expression to the Expression<Func<T, U>> that the OrderBy method requires?
I have something like this so far, which I extracted from some similar code I wrote that works with LinqToSql.
        public static LambdaExpression GetOrderByExpression<T>(string propertyName)
        {
            var type = typeof(T);
            var property = type.GetProperty(propertyName, BindingFlags.IgnoreCase | BindingFlags.Public | BindingFlags.Instance);
            if (property == null)
            {
                throw new MissingMemberException($"Class {type.FullName} does not contain a property with name '{propertyName}'");
            }

            var parameter = Expression.Parameter(type, "p");
            var propertyAccess = Expression.MakeMemberAccess(parameter, property);
            return Expression.Lambda(propertyAccess, parameter);
        }

This is the working LinqToSql code that works with IQueryable
   ```

public static IOrderedQueryable OrderBy(
this IQueryable source,
SortExpression sortExpression)
{
if (sortExpression != null)
{
var type = typeof(TEntity);
var property = type.GetProperty(sortExpression.PropertyName, BindingFlags.IgnoreCase | BindingFlags.Public | BindingFlags.Instance);
if (property == null)
{
throw new MissingMemberException($"Class {type.FullName} does not contain a property with name '{sortExpression.PropertyName}'");
}
            var command = sortExpression.Direction == ListSortDirection.Ascending
                ? "OrderBy"
                : "OrderByDescending";
            var parameter = Expression.Parameter(type, "p");
            var propertyAccess = Expression.MakeMemberAccess(parameter, property);
            var orderByExpression = Expression.Lambda(propertyAccess, parameter);
            var resultExpression = Expression.Call(
                typeof(Queryable),
                command,
                new Type[] { type, property.PropertyType },
                source.Expression,
                Expression.Quote(orderByExpression));

            source = source
                .Provider
                .CreateQuery<TEntity>(resultExpression);
        }

        return (IOrderedQueryable<TEntity>)source;
    }


Add ability to trap right-arrow key press on a Quickpick item

VSCode Version: 1.10.0-insider 256d76fed7947e7289f921df10658d7a69233b65
OS Version: macOS Sierra 10.12.3 (16D32)

I would like to be able to trap a right-arrow key press on a Quickpick item, to provide similar behavior to the built-in open files picker.

@bpasero Is the quick select command something we have documented and that can be bound to a different key?

No, the behaviour for right-arrow is still hardcoded and not expressible as command.
However I am not sure @eamodio asks for that, I think as an extension he wants to define the command that runs when pressing right-arrow in the picker his extension owns.

@bpasero yes, exactly that.

I would like to be able to trap a right-arrow key press on a Quickpick item, to provide similar behavior to the built-in open files picker.

By that you mean "select the item without closing the picker", right? We don't abstract on the level of keys, unless you wanna implement a keybinding, so it's unlikely that we would expose the pure keypress. Also people usually want to configure keybindings.
To get the current arrow right behaviour you could maybe try a combination of onDidSelectItem, a custom context key, and a command that binds to ->

Right, perform some action but leave the picker open.
Oh that is quite interesting on the command with custom context -- there is a lot of power in that. Is there a command to get the value of a custom context? Like a GetContext corollary to SetContext? Then it would be much easier to coordinate - as onDidSelectItem could SetContext and then the conditional command could read from that context to perform its action

FYI, I have this working in GitLens now, but I am unable to trap just the right arrow (well I think I can most of the time, but then the user can't use the arrows in the input box -- ick), so I am currently trapping alt+right arrow. It would be great to have this feature, so it feels more natural with the other vscode behavior. :)

@jrieken @chrmarti instead of actually supporting this -- can we set a context value for which commands can then use to support right-arrow. Something like canUseQuickArrow or some better name -- which would get set when the cursor in the quick pick input was all the way to the right. With that context value, extensions can then easily support this behavior, without breaking arrow key navigation in the quick pick input.


integration with moti

Description
It would be interesting to include standard animations in the View, Box and presseable
Problem Statement
It would be interesting to include standard animations in the View, Box and presseable
Proposed Solution or API
It would be interesting to include standard animations in the View, Box and presseable
Alternatives
No response
Additional Information
No response

@Brunowilliang We appreciate your feedback. We will pass it to the concerned team so that they check the possibilities of adding the same in future updates. Thank you.

Hi @Brunowilliang, We have a Transition component for simple animations with a recipe of Fade, SlideFade, ScaleFade etc. So Users can easily create simple animations. Adding extra prop for animation will require component level computation which affects the component's performance. Box, Pressable, and View are primitive components from which composite components are created so it affects a lot more components.
Although we are open to discussion. let us know. What's your take on this.
I am moving this to the discussion section.


body was not decodable (unable to check for Status): Object 'Kind' is missing

What happened: No issues in use but the logs of the federation controller manager repeatedly an error.
What you expected to happen: No errors
How to reproduce it (as minimally and precisely as possible): Deploy a federated namespace cluster and then run kubectl logs -f federation-controller-manager-specific-pod
Anything else we need to know?:
E0328 15:25:15.215813       1 reflector.go:134] github.com/kubernetes-sigs/federation-v2/pkg/controller/sync/accessor.go:164: Failed to list <nil>: the server could not find the requested resource
I0328 15:25:15.454506       1 leaderelection.go:245] successfully renewed lease pacman/federation-controller-manager
I0328 15:25:16.220615       1 reflector.go:169] Listing and watching <nil> from github.com/kubernetes-sigs/federation-v2/pkg/controller/sync/accessor.go:164
I0328 15:25:16.222713       1 request.go:1144] body was not decodable (unable to check for Status): Object 'Kind' is missing in '{
  "paths": [
    "/apis",
    "/metrics",
    "/swagger-ui/",
    "/version"
  ]
}'
E0328 15:25:16.222743       1 reflector.go:134] github.com/kubernetes-sigs/federation-v2/pkg/controller/sync/accessor.go:164: Failed to list <nil>: the server could not find the requested resource

gist of deployment and federatedtypeconfig
https://gist.github.com/cooktheryan/83109f825cd4c62145c07c03e1d422a2
Environment:

Kubernetes version (use kubectl version): v1.12.4
Federation version quay.io/kubernetes-multicluster/federation-v2:v0.0.7
Install method (helm vs script): Operator script
Install configuration (namespaced vs cluster-scoped) namespace
Others

/kind bug

@cooktheryan The error here indicates the failure of the sync controller to launch an informer.  That can occur when a FederatedTypeConfig refers to an API type that does not exist.  Given that the only supported deployment method for this repo is via the helm chart, and I can't reproduce the problem deploying with a chart, I'm left with the assumption that the cause of the problem you're seeing is related to the operator script mode of deployment rather than a bug with federation.  I've relabeled the issue accordingly.
That said, I'm happy to try to help diagnose the problem.  Please post (e.g. via gist, not on the issue directly) the federated type configs and summary of installed federated type crds (e.g via kubectl api-resources) for the deployment when you have a chance.

It seems like the issue has gone away on it's own


[Bug]: MVE2 fix: redirect after remove keychain

Is there an existing issue for this?

[x] I have searched the existing issues

Current Behavior
GIVEN a user wants to remove keychain from my account page
WHEN confirm remove keychain
THEN user see "keychain removed" confirmation
AND WHEN user click "close"
THEN user stays in my account
AND IF user clicks "remove keychain" again
THEN user infinite loop

Expected Behavior
After confirming keychain removal, then...

Redirect user to discovery page
Put app into no keychain / visitor state

Project
Frontend
What operating system are you seeing the problem on?
Windows
What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
Anything else?
No response
Code of Conduct

[x] I agree to follow this project's Code of Conduct

Consider converting to a refinement. Probably want to have mockups to show exact desired intent, and to think more broadly about how my account connects with the rest of the application. @nielskijf


Navbar is Not working Properly [Issue]

Please do not report security vulnerabilities here. The Responsible Disclosure Program details the procedure for disclosing security issues.
Thank you in advance for helping us to improve this library! Please read through the template below and answer all relevant questions. Your additional work here is greatly appreciated and will help us respond as quickly as possible. For general support or usage questions, use the Auth0 Community or Auth0 Support. Finally, to avoid duplicates, please search existing Issues before submitting one here.
By submitting an Issue to this repository, you agree to the terms within the Auth0 Code of Conduct.
Description

Provide a clear and concise description of the issue, including what you expected to happen.

Reproduction

Detail the steps taken to reproduce this error, what was expected, and whether this issue can be reproduced consistently or if it is intermittent.
Where applicable, please include:

Code sample to reproduce the issue
Log files (redact/remove sensitive information)
Application settings (redact/remove sensitive information)
Screenshots

Environment
https://github.com/auth0/node-jsonwebtoken/assets/119070053/e68b53ee-4ada-4610-ba1b-76fc73eea1c

8

Please provide the following:

Version of this library used:
Version of the platform or framework used, if applicable:
Other relevant versions (language, server software, OS, browser):
Other modules/plugins/libraries that might be involved:

https://github.com/auth0/node-jsonwebtoken/assets/119070053/784f14c7-354d-4b0e-b996-001ce72dad59


Brian patch 8

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

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

#sign-off

@bharney: I'm sorry - only the author of this article, @bharney0, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@bharney: I'm sorry - only the author of this article, @bharney0, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off

#hold-off and #sign-off but why?

@lamebrain: I'm sorry - only the author of this article, @bharney0, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off

@lamebrain: I'm sorry - only the author of this article, @bharney0, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@lamebrain: I'm sorry - only the author of this article, @bharney0, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@lamebrain: I'm sorry - only the author of this article, @bharney0, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#hold-off and #sign-off but why?

@lamebrain: I'm sorry - only the author of this article, @bharney0, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

You can't sign off and hold off at the same time.

#hold-off and #sign-off but why?

@lamebrain: I'm sorry - only the author of this article, @bharney0, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Data Type Exception

Using the supplied flow and 2 images I chose (Image Processor Node)
Cannot handle this data type: (1, 1, 439, 3), |u1

File "D:\From Git\ComfyUI\execution.py", line 154, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "D:\From Git\ComfyUI\execution.py", line 84, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "D:\From Git\ComfyUI\execution.py", line 77, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "D:\From Git\ComfyUI\custom_nodes\ComfyUI-PickScore-Nodes\pickscore.py", line 46, in process
processor(
File "D:\From Git\ComfyUI\venv\lib\site-packages\transformers\models\clip\processing_clip.py", line 102, in __call__
image_features = self.image_processor(images, return_tensors=return_tensors, **kwargs)
File "D:\From Git\ComfyUI\venv\lib\site-packages\transformers\image_processing_utils.py", line 446, in __call__
return self.preprocess(images, **kwargs)
File "D:\From Git\ComfyUI\venv\lib\site-packages\transformers\models\clip\image_processing_clip.py", line 318, in preprocess
images = [self.resize(image=image, size=size, resample=resample) for image in images]
File "D:\From Git\ComfyUI\venv\lib\site-packages\transformers\models\clip\image_processing_clip.py", line 318, in
images = [self.resize(image=image, size=size, resample=resample) for image in images]
File "D:\From Git\ComfyUI\venv\lib\site-packages\transformers\models\clip\image_processing_clip.py", line 144, in resize
return resize(image, size=output_size, resample=resample, data_format=data_format, **kwargs)
File "D:\From Git\ComfyUI\venv\lib\site-packages\transformers\image_transforms.py", line 275, in resize
image = to_pil_image(image)
File "D:\From Git\ComfyUI\venv\lib\site-packages\transformers\image_transforms.py", line 163, in to_pil_image
return PIL.Image.fromarray(image)
File "D:\From Git\ComfyUI\venv\lib\site-packages\PIL\Image.py", line 3092, in fromarray
raise TypeError(msg) from e

Are you using the portable version of ComfyUI? Do you have any other custom nodes enabled? What dimensions are your input images?

Are you using the portable version of ComfyUI?

Yes, in a 3.11.x virtual environment

Do you have any other custom nodes enabled?

Yes, See below as I will attach my comfyui.log

What dimensions are your input images?

Doesn't matter. I tried

Source size images (different sizes)
Scaling image A to 512px on the longest side and resizing image B to be the same size
Scaling image A to 512px on the shortest side and resizing image B to be the same size
Resizing image A to 512x512 resizing image B to be the same size
Resizing image A to 1024x1024 resizing image B to be the same size

comfyui.log

Same situation, not working with most up to date version of ComfyUI , transformers possibly related error?
Error: Cannot handle data type (1, 1, 768, 3), |u1
ZuellniPickScoreImageProcessor: Unsupported data type
File "execution.py", line 155: recursive_execute function
File "execution.py", line 85: get_output_data function
File "execution.py", line 78: map_node_over_list function
File "pickscore.py", line 46: process function in pickscore module
File "processing_clip.py", line 102: call method in processing_clip module
File "image_processing_utils.py", line 446: call method in image_processing_utils module
File "image_processing_clip.py", line 318: preprocess method in image_processing_clip module
File "image_processing_clip.py", line 318: resize method in image_processing_clip module
File "resize" method in image_transforms module: TypeError, invalid array conversion

Sorry, I can't quite figure this one out, the issue being that it Just Works For Me™ no matter what I throw at it. It could be some other node interfering like in #2, but I couldn't find anything special about the ones in the log above. Still, you can try running it without any other nodes enabled. Also, are you both able to run the inference code from the PickScore repo without any errors?

Closing for now since I still can't reproduce.


[Sockets] bot.send loops indefinitely if client disconnected

https://github.com/howdyai/botkit/blob/dev/webchannel/lib/Sockets.js#L135-L156
I think that retry limit would be a good solution.

Looks cool!


Improve documentation based on TUDelft and CWI review

[X] Add os.chdir('.')
[X] Also store the code in the html
[X]  Make seperate Methodology page on openstef

Add complete overview of features. And add description of what they mean.
Add overview of methodology; what does the input and output look like. How do we use 1 model for multiple timestep prediction

Result: https://openstef.github.io/openstef/concepts.html#high-level-methodology-openstef


[FEA] Support all CTK-supported SM arches in cuda_init_architectures

Is your feature request related to a problem? Please describe.
Thrust supports all architectures supported by CTK, including deprecated ones. This means we currently need to build against SM35+ in our benchmarking library.
However, rapids_cuda_init_architectures only adds architectures supported by RAPIDS (sm60+) when CMAKE_CUDA_ARCHITECTURES=ALL.
Describe the solution you'd like
Ideally, these special values would be something like:

ALL: non deprecated arches supported by CTK (e.g. sm53+)
ALL_LEGACY: all arches supported by CTK, including deprecated (e.g. sm35+)
RAPIDS: all arches supported by RAPIDS (e.g. sm60+)

Since the current convention around ALL is already in use by rapids-cmake users, maybe the following values would be more agreeable and less disruptive:

CTK: non deprecated arches supported by CTK (e.g. sm53+)
CTK_LEGACY: all arches supported by CTK, including deprecated (e.g. sm35+)
ALL / RAPIDS: all arches supported by RAPIDS (e.g. sm60+)

Also -- I'm not sure if NATIVE will add <sm60 if such a card is present. If not, adding NATIVE_CTK + NATIVE_CTK_LEGACY (or similar) would be useful for us.
Describe alternatives you've considered
We can still just list all arches explicitly, so this is a low-priority issue with an easy workaround. Most users will want NATIVE anyway.

Closing this issue as https://github.com/rapidsai/rapids-cmake/issues/314 better represents what rapids-cmake will do.
If you need to build with against all support archs, using the new CMAKE_CUDA_ARCHITECTURES=all/all-major is the best approach.


Beta: Text and Icons not always showing correctly in dialog

Text and icons do not show correctly about 50% of the time in all of my dialogs, but only on Flutter Beta Channel.
Issues are intermittent and include:

Some of the text showing and some not
Icons not showing
All of the text not showing

There's nothing else that's problematic - all sizes and styling look fine.
It's a relatively new issue - I haven't seen it before and I generally stick to the Beta Channel. I haven't been able to reproduce it on Stable.
There's nothing indicating an issue and there's no difference in the console between Beta and Stable.
Beta:

Stable:

UPDATE
Here's the output from flutter doctor -v:

I'm only seeing the issue on Android currently - I haven't tested it yet on IOS, but will post the results as soon as I have the ability to.
I'm seeing the issue on Samsung Galaxy S10e. I've had someone tell me that there is a similar problem (screen is blank) that extends beyond Dialogs on Pixel 4 XL. Reverting back to standard also fixed their issue.
I'm using ScreenUtil to set the size of the Text Widget's TextStyle like so:
Text(
  "Here, have a dialog",
  style: TextStyle(
      color: Colors.black,
      fontWeight: FontWeight.bold,
      fontSize: ScreenUtil().setSp(20.0)),
),

Perhaps it's a race condition between the Dialog opening time and calculating the text size?
This reproduces it on the Samsung Galaxy S10e:
import 'dart:math';

import 'package:flutter/material.dart';
import 'package:flutter_screenutil/flutter_screenutil.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatefulWidget {
  @override
  _MyAppState createState() => _MyAppState();
}

class _MyAppState extends State<MyApp> {
  @override
  void initState() {
    ScreenUtil.init();
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Sometimes Sad Dialog',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key}) : super(key: key);

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Container(
          child: RaisedButton(
            color: Colors.purple,
            onPressed: () {
              showBouncyDialog();
            },
            child: Text(
              "Give me a dialog",
              style: TextStyle(
                color: Colors.white,
                fontSize: ScreenUtil().setSp(60.0),
              ),
            ),
          ),
        ),
      ),
    );
  }

  void showBouncyDialog() async {
    await showGeneralDialog<bool>(
      pageBuilder: (context, anim1, anim2) {},
      barrierLabel: '',
      barrierColor: Colors.black.withOpacity(0.3),
      transitionDuration: Duration(milliseconds: 400),
      context: context,
      barrierDismissible: true,
      transitionBuilder: (
        BuildContext context,
        Animation<double> animation,
        Animation<double> anotherAnimation,
        Widget child,
      ) {
        final double curvedValue =
            max(0, Curves.elasticInOut.transform(animation.value));

        return Transform.scale(
          scale: curvedValue,
          child: Dialog(
            child: Padding(
              padding: EdgeInsets.all(50.0),
              child: Column(
                mainAxisSize: MainAxisSize.min,
                crossAxisAlignment: CrossAxisAlignment.start,
                children: [
                  Text(
                    "Here, have a dialog",
                    style: TextStyle(
                        color: Colors.black,
                        fontWeight: FontWeight.bold,
                        fontSize: ScreenUtil().setSp(20.0)),
                  ),
                  SizedBox(
                    height: 20.0,
                  ),
                  Text(
                    "Here, have some more test that is of a longer length to see if wrapping is working.",
                    style: TextStyle(
                        color: Colors.blue, fontSize: ScreenUtil().setSp(40.0)),
                  ),
                  SizedBox(
                    height: 20.0,
                  ),
                  Center(
                    child: RaisedButton(
                      onPressed: () {
                        Navigator.of(context).pop();
                      },
                      color: Colors.red,
                      child: Text(
                        "Pop my cherry",
                        style: TextStyle(
                          color: Colors.white,
                          fontSize: ScreenUtil().setSp(60.0),
                        ),
                      ),
                    ),
                  )
                ],
              ),
            ),
          ),
        );
      },
    );
  }
}

And here are screenshots of it acting up with the code above:
expected

nothing showing

Topmost Text not showing

Hi @DustyLamp,
Please provide flutter doctor -v and a minimal complete reproducible code sample that shows the issue.
Also specify platform (Android, iOS) and devices on which you see this issue.
Thanks.

Thanks @darshankawar - I've made an update to the original comment

Hi @DustyLamp, thanks for the code sample.
I see that you are using flutter_screen_util plugin which is 3rd party. Just to segregate the issue, is it possible for you to check by removing this plugin related code and try again with core code and see if the issue still occurs ?

Yes - good question @darshankawar.
I can reproduce the issue while text sizes 11 or smaller.
I've edited the code to remove flutter_screen_util plugin

Thanks @DustyLamp for the code sample.
I tried it on latest beta and on Samsung S10+ device, but didn't see the notice.
64936.mp4.zip

flutter doctor -v
Flutter 1.21.0-9.2.pre • channel beta • https://github.com/flutter/flutter.git
Framework • revision 81a45ec2e5 (5 days ago) • 2020-08-27 14:14:33 -0700
Engine • revision 20a9531835
Tools • Dart 2.10.0 (build 2.10.0-7.3.beta)

Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, 1.21.0-9.2.pre, on Mac OS X 10.15.2 19C57, locale en-US)
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.
[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.6)
[✓] Connected device (4 available)

! Doctor found issues in 1 category.

Since you said, it's intermittent, I'd suggest you keep an eye on it and see if there's any pattern that causes the issue.
Thanks.

When I say it's intermittent, it's happening every second time that I push the button in my code so it's not rare. It happens more often than not.
As for patterns that cause it:

Usually, when I first open the application it works without a problem
The second time I start seeing the issue and then the issue usually persists from that point onwards.

See gif below:

I am unable to replicate this issue on latest beta and on the devices I tried, but per OP, it's happening at their end. Will need further analysis on it,so keeping this issue open.

I'm experiencing something similar. Text/Icon widgets are rarely shown after animations. Happens both on android and ios platforms. As far as i can tell, it affects debug and release builds.

Here's some example code:
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: Center(
        child: MicroAnimationBounceIn(
          child: RaisedButton(
            onPressed: () {},
            child: Text("This Text disappears in release builds"),
          ),
        ),
      ),
    );
  }
}

class MicroAnimationBounceIn extends StatefulWidget {
  final Widget child;
  final Duration delayBounce;
  final Duration duration;

  MicroAnimationBounceIn({
    Key key,
    this.child,
    this.delayBounce = const Duration(milliseconds: 100),
    this.duration = const Duration(seconds: 1),
  }) : super(key: key);

  @override
  _MicroAnimationBounceInState createState() => _MicroAnimationBounceInState();
}

class _MicroAnimationBounceInState extends State<MicroAnimationBounceIn>
    with SingleTickerProviderStateMixin {
  AnimationController _animationController;
  Animation<double> _animation;

  @override
  void initState() {
    super.initState();

    _animationController = AnimationController(
      duration: widget.duration,
      vsync: this,
      animationBehavior: AnimationBehavior.preserve,
    );

    _animation = Tween<double>(begin: 0.0, end: 1.0).animate(CurvedAnimation(
        parent: _animationController, curve: Curves.elasticInOut))
      ..addListener(() {
        setState(() {});
      });

    Future.delayed(widget.delayBounce)
        .then((value) => _animationController.forward());
  }

  @override
  void dispose() {
    _animationController.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return Transform.scale(
      scale: _animation.value,
      child: widget.child,
    );
  }
}

Another important note: This problem appears on physical devices more often than on simulators/emulators.
My flutter doctor output:
Flutter 1.21.0-9.2.pre • channel beta • https://github.com/flutter/flutter.git
Framework • revision 81a45ec2e5 (11 days ago) • 2020-08-27 14:14:33 -0700
Engine • revision 20a9531835
Tools • Dart 2.10.0 (build 2.10.0-7.3.beta)

Could you reproduce it with my example code @darshankawar ?

I reproduced this issue using the sample code from https://github.com/flutter/flutter/issues/64936#issuecomment-688661111
It happens reliably on a Moto G4 if I run the app in release mode, hit the back button, and resume the app.  The animation will run but the text will not be rendered.
Bisected this to https://github.com/google/skia/commit/a5b986087102f71357d4cf3055f237e14c31a0f7.  The text shows if I remove the !fInitialMatrix.rectStaysRect() check from GrTextBlob::canReuse.
@herbderby @bungeman

Can you print the values of fInitialMatrix when the problem happens?

The last fInitialMatrix seen during the animation is:
m0=0.000065 m1=0.000000 m2=540.004028
m3=0.000000 m4=0.000065 m5=888.019592
m6=0.000000 m7=0.000000 m8=1.000000

The problem stems from building the initial text blob with a size of
1/650,000. This creates an initial blob with no viewable text. What is the
text size on SkFont when you initially draw it, and what is the text size
on SkFont when you expect it to draw? Can you provide a repro case in c++?
Thanks,
Herb
On Mon, Sep 21, 2020 at 3:42 PM Jason Simmons<EMAIL_ADDRESS>wrote:

The last fInitialMatrix seen during the animation is:
m0=0.000065 m1=0.000000 m2=540.004028
m3=0.000000 m4=0.000065 m5=888.019592
m6=0.000000 m7=0.000000 m8=1.000000
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/64936#issuecomment-696330897,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANUMHGVSE6KUYJAIYG5DFLSG6UDHANCNFSM4QQGWA4A
.

The SkFont in this example always has font size 14.
The 0.000065 value in the matrix happens because the text is wrapped in an animation that applies a scale transform.  When the animation works properly, the fInitialMatrix starts at 0.000065 and gradually increases as the animation progresses, eventually reaching a value of 3.0.
If I remove the !fInitialMatrix.rectStaysRect() check then I see that behavior consistently.
With the rectStaysRect check I sometimes see the fInitialMatrix scale factor stuck at 0.000065 even though the animation should have increased the scale.
I don't have a standalone C++ reproduction.

xref https://skia-review.googlesource.com/c/skia/+/319031

The change was reverted in Skia and rolled into the engine.


Order lists in alphabetical order

As it is right now, the lists are quite messy. What about ordering them alphabetically?
I don't mind creating another PR for that, but I thought I'd ask first because there may be some logic behind the current order.

am working on that, but it is time consuming, almost there! thanks for your feedback.

Ok, great! Just FYI, in Sublime Text it's a built-in feature to order lists with a click of a button. :slightly_smiling_face:


Add French layout FR-BEPO for Pulse 15

This follow the french norm NF ZF1-300, bépo model.
Bépo is a french ergonomic layout.
More information on:

AFNOR french standardization national organization :
https://normalisation.afnor.org/actualites/faq-clavier-francais/
Bépo association:
http://bepo.fr/wiki/Accueil

Thanks for your contribution! :partying_face:

Thanks :)


Add support of Italian language

Looks good! Three small things:

You can ignore the @houndci warnings.
Please add tests for Italian. See the other examples in test/definitions.
Credit yourself in the readme and in package.json.


redundant geographical elements

Hi,All,
This code maybe simply concatenate all the geoJSON data.
However, many redundant geographical elements  also appear in the result.
How to solve this problem? Could you give me some advice? Thanks.
Kind regards,
Xun

Hi Xun,
If you're looking for something that combines overlapping features, you'll want a module with a union method like Turf.


Get some basic unit test coverage in back end

Write at least a few unit tests, and ideally a functional test as well to make sure that the API will simply run.

Changed to writing functional tests for remaining flask resources (book.py and listing.py).


Improve Documentation

I updated the Figma file to add a german language with a BlueWallet. But now, I stumbled on how to create the PDF file from it and contribute it to the project.
It would be great to get a bit more insights into the project so that more people can contribute.

[ ] Create a proper readme.md what this project is about. Example from makeareadme
[ ] A contributer.md allows people to help improve the project and add guidelines for new contributors to help them get started with the project
[ ] Add a license for the project, which is important for legal parts because, without a license, there are uncertainties that the maintainer could avoid the project.

Hi @md-weber thanks!
Working on writing up some instructions on the readme.
Unfortunately right now the process to create a PDF is pretty annoying as figma only creates loose pages from each artboard.
Also added a MIT license.


Is this even working?

I don't understand how do you guys run this? I get all sorts of errors, from .env (not included) not found to index.html template not found (wrong folder?).

I don't understand how do you guys run this? I get all sorts of errors, from .env (not included) not found to index.html template not found (wrong folder?).

if you want to test it, go to this https://chatgpt-prompt-splitter.jjdiaz.dev/


Overwriting the app_id

Hello
Passing a dynamic app_id in the intercom_script_tag makes the helper ignoring all IntercomRails config data.
We do not set the app_id in the intercom.rb settings file since the tag is dynamically loaded with ajax through cached pages and a rails app serving multiple websites and using multiple intercom apps.
Thanks !

Hi @jerome could you explain this bug a little more? The intercom_script_tag functions ignores the global config?

In the contrary: you cannot pass an app_id in the intercom_script_tag. It is ignored. Only the global config prevails.

up ?


Api/endpoint bug fixes

fixes registration endpoint. checks for username duplicates. fixed delete endpoint misspell and other bugs.

You have merge conflicts you might wanna look at


Styled input is broken Cannot set properties of undefined (setting 'value')

For component
<Input type="email" />

On input typing following error is shown:
Cannot set properties of undefined (setting 'value')

Reason is this line:
bind:value={valueSig}

Hey @ziimakc! There appears to be an issue with core and this prop with the optimizer. As a result, I've added a workaround for now that should get this up and functional after the next release 👍

You can also feel free to try the changes now with the pkg pr new on the issue mentioned above.


make primary branch name configurable

We should separate out the repo name and main branch as config options. NMA's course-content repo still uses master as the primary branch name, while the github default is now main, and so the steps used in process_notebooks.py to rename branches will break on newer repos.

Makes sense.
My suggestion would be to define it as a global variable at the top of process_notebooks.py and configure it by reading from os.environ.
Then the workflow yamls would need to be updated to set that. (I guess you could be clever and set it dynamically in the workflow scripts by using metadata about the branch the pull request is against. Not sure if that's necessary.)
Implementing this will introduce conflicts with the work in #3.


Use junit

this will publish the results from the e2e run if they happened for easy viewing.

retest this please

LGTM. Note that tests are only passing on pipelet.


QoS ThrottleRate "MBytes/sec" -> "Mbits/sec" documentation errors

Examples of ThrottleRateActionBitsPerSecond are listed in the output item "ThrottleRate" as 'Bytes/sec' whereas in fact they are in Bits/sec.
Please also correct '1MB' to be '1Mb' in all locations to correctly indicate 'bits' instead of 'Bytes' - the application itself is case insensitive so the outcome is the same. '1MB' input will be the same as '1Mb' input, and is output as '1.049Mbits/sec'.
Documented example:
PS C:\> New-NetQosPolicy -Name "FTP" -AppPathNameMatchCondition "ftp.exe" -ThrottleRateActionBitsPerSecond 1MB -PolicyStore ActiveStore
Name           : FTP
Owner          : PowerShell / WMI
NetworkProfile : All
Precedence     : 127
AppPathName    : ftp.exe
ThrottleRate   : 1 MBytes/sec

Actual output:
Name           : ftp
Owner          : PowerShell / WMI
NetworkProfile : All
Precedence     : 127
AppPathName    : ftp.exe
JobObject      :
ThrottleRate   : 1.049 MBits/sec

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

ID: ace9d8fc-967d-e9ca-44ae-dc26f2fb6703
Version Independent ID: fd6ec687-d40a-8534-0d33-a4d5cd498d5a
Content: New-NetQosPolicy (NetQoS)
Content Source: docset/winserver2019-ps/netqos/New-NetQosPolicy.md
Product: w10
Technology: windows
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

@stampycode , Thanks for pointing out. I am fixing these errors.


Installation issue

Hi,
I'm trying to install this toolbox on colab, but got the following error message:
Do you know any solutions to this?
Thanks a lot.
BW,
Tong

e

It seems to me that your environment was not clean. Since I believe the torchtext was not required in this repository.
Could you please try to install s3prl with another clean colab environment?
If you find the same issue, please share the link with me! Thanks!

Dear Shu-wen,
I tried to use conda in colab and it seems that the installation is OK.
However, when I tried to install it on the local cluster (not google’s colab), I got the following error:
[cid:FFD270AE-4673-4463-810F-E3B15F5D94C2]
I had a check, the .whl file for dtw-python was collected. All other packages were installed normally via their wheels.
Appreciate very much if you know how to solve this problem.
Thanks a lot for your quick reply.
On 27 Sep 2021, at 02:42, Shu-wen Yang<EMAIL_ADDRESS>wrote:
It seems to me that your environment was not clean. Since I believe the torchtext was not required in this repository.
Could you please try to install s3prl with another clean colab environment?
If you find the same issue, please share the link with me! Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/s3prl/s3prl/issues/208#issuecomment-927439594, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHTA4PG7SC37U2NQBTU7ZZDUD7DYDANCNFSM5EXTGWTQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Can you please share you python version and pip version?
Perhaps I will try to first upgrade pip to the latest version :)

Python 3.9.6
Pip  21.2.4
Hope upgrading will solve this problem.Thanks a lot!
On 29 Sep 2021, at 01:00, Shu-wen Yang<EMAIL_ADDRESS>wrote:
Can you please share you python version and pip version?
Perhaps I will try to first upgrade pip to the latest version :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/s3prl/s3prl/issues/208#issuecomment-929708363, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHTA4PEXY7LHKM4WQHOS6ILUEJJIRANCNFSM5EXTGWTQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I am closing it now. Free feel to re-open it!


Cequel still alive

Just wondering, since there hasn't been any activity the last couple of months, and the official cassandra-driver is several versions further already. Is Cequel still being maintained? Is this just a small dip, or is it bleeding to death?

Looks unmaintained. That said, there are forks which bump Cassandra driver:
https://github.com/Galeria-Kaufhof/cequel/commit/51afb3cf9b302ce805fa6fca6be458b56cb98c91
That does seem to work, up until Cassandra 2.1.8, but Cequel breaks with Cassandra 2.2.0 because it tries to use system columns which no longer exist to get schema info.
Ubuntu (14.04) upgraded the Cassandra package from 2.1.8 to 2.2.0 on August 2 which broke my local installation so I had to hold the package at 2.1.8.
In any case if you use Cequel beyond its sell-by date you're advised to test well and I for one would appreciate news of any gotchas.

@lesliev I made the same mistake of upgrading to 2.2.0 and saw that migrations failed.
@outoftime are you actively working on a 2.2.0 fix? Or should we start working on it ourselves?

If you do fork Cequel, be aware of this bug we fixed a few days ago:
https://github.com/visfleet/cequel/commit/006dac39cfcd929745d55eab889ed57b73441bc9
Cequel was leaking threads when an error caused it to reconnect.
I've not made a PR because I've not yet gotten the tests properly working.

This does not seem to work with Rails 3.2.22 - was that intentional?
Is there a version somewhere that works with cassandra-driver 2.1.3 (or 2.1.4) and Rails 3.2.22?

Greg


Allow ddev config to specify a particular drupal sub-version and retain that configuration

In working on https://github.com/ddev/ddev-drupal-contrib/pull/87 I have an issue
Imagine the following scenario:

I have a contrib module
I download the addon
I start the project with php 8.2 because the module was already setup for php 8.2
I run ddev poser fails because it needs php 8.3

So the rest doesn't work, and ddev config --update doesn't work either.
I know php 8.3 is the recommended command, but this could fail on newer versions.
So personally I would suggest one of two things:

Be verbose on the addos on the settings that ddev needs, forcefully adding corepack and php 8.3 for D11 and php 8.2 for D10 if those are the minimum requirements.
Better yet, as ddev already handles that provide a way to pass this drupal version through a command line override.

Would something like this be accepted?
Something like ddev config --project-type=drupal --drupal-version=11?

I think what I'd like best is for ddev describe to say drupal(11) or something like that. Should be pretty easy. So make clear that it's project type drupal with the detected subtype 11

This is what DDEV does:
https://github.com/ddev/ddev/blob/fddf8c8df8e38ffd9214d6220140f9275640f636/pkg/ddevapp/drupal.go#L326-L342

Yeah I noted that on issue summary. I am trying to address the alternative to configure a project type with a Drupal version without anything to look the version for

DDEV doesn't track that, so that would be going back to the older way. There was definitely a significant conversation before doing it the way we did.

I understand. I guess I was seeing if there was a way away to bring the old way back but only through an optional override or something. Like if want to force it for any reason

So the title should be something like "Allow ddev config to specify a particular drupal subversion and retain that configuration" ?

Something like that, yes, or "use the default configuration for that drupal subversion"

Related conversation in

https://github.com/ddev/ddev/issues/6306

Let's solve this in

https://github.com/ddev/ddev/issues/6635

Looking forward to your ideas (and maybe implementation), thanks.


feat(): Possibility to use key events on MultiSelect control

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe the problem.
I have edit form. And user wants to go though controls by clicking 'tab' on keyboard. so when user is clicking on MultiSelect control, last one should open.
Another situation: user's selecting one option in MultiSelect control and clicking 'Enter' on keyboard, after that form should submit.
Describe the solution you'd like
It would be nice if MultiSelect /Select control have method 'OnTabAsync' and 'OnEnterAsync'
as you see I'm actively using these components from your fantastic Blazor Zone library.
Thank you so much for this incredible library.
Additional context
No response

@anna777x could you give a star for this project? thanks

@anna777x hi. thank you for contacting us. let us make your requirements clear.

I guess you mean that auto expands the dropdown when the user tabs Tab keyboard when the MultiSelect component has focus.
in EditorForm component the user press Enter auto submit the form

is right?

you understand me absolutely correct!


Unwanted steps are shown in tracing (Jaeger) output

There should be only ballerina/http/Client.post

To reproduce the issue use the below code
https://github.com/shafreenAnfar/springboot-and-ballerina/tree/main/ballerina-user-rest-service

We may have to discuss further whether to include the client invokes inside the ballerina/http module or not.