Try to convert compiler warnings to errors on Jenkins

Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

This PR makes sure that this rule only applies to source files in the Stan universe? We would not be able to guarantee no errors in things like boost::mpi, for example.

It doesn't change that and I noticed that there are some compiler warnings
from compiling boost. I'm not sure how to fix that, actually...
On Mon, Jun 11, 2018 at 09:48 wds15<EMAIL_ADDRESS>wrote:

This PR makes sure that this rule only applies to source files in the Stan
universe? We would not be able to guarantee no errors in things like
boost::mpi, for example.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-396139367, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAxJ7F-MPGe3eYhts3uoVNPQQ8AslLtKks5t7hLYgaJpZM4Ue9jP
.

--
Via smol keyboard

Someone should fix the problem this is aimed at and there is some code here
for it but it needs work.
On Wed, Sep 12, 2018 at 11:24 PM Daniel Lee<EMAIL_ADDRESS>wrote:

@seantalts https://github.com/seantalts, it looks like this can just be
closed? Is that right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420871030, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAxJ7JUBCjPaR8fOqFoQob_lcePm4_54ks5uac_cgaJpZM4Ue9jP
.

--
📲

What is the problem it's trying to fix? Mind making that an issue? I read
through it and couldn't pick out the actual problem.
On Thu, Sep 13, 2018 at 7:32 AM seantalts<EMAIL_ADDRESS>wrote:

Someone should fix the problem this is aimed at and there is some code here
for it but it needs work.
On Wed, Sep 12, 2018 at 11:24 PM Daniel Lee<EMAIL_ADDRESS>wrote:

@seantalts https://github.com/seantalts, it looks like this can just
be
closed? Is that right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420871030, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AAxJ7JUBCjPaR8fOqFoQob_lcePm4_54ks5uac_cgaJpZM4Ue9jP
.

--
📲
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420975320, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAZ_F-D39WYQssIhAVisPjDtXQncEs4xks5uakJAgaJpZM4Ue9jP
.

Compiler Warnings do not fail the build
On Thu, Sep 13, 2018 at 08:30 Daniel Lee<EMAIL_ADDRESS>wrote:

What is the problem it's trying to fix? Mind making that an issue? I read
through it and couldn't pick out the actual problem.
On Thu, Sep 13, 2018 at 7:32 AM seantalts<EMAIL_ADDRESS>wrote:

Someone should fix the problem this is aimed at and there is some code
here
for it but it needs work.
On Wed, Sep 12, 2018 at 11:24 PM Daniel Lee<EMAIL_ADDRESS>wrote:

@seantalts https://github.com/seantalts, it looks like this can just
be
closed? Is that right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420871030, or
mute
the thread
<

https://github.com/notifications/unsubscribe-auth/AAxJ7JUBCjPaR8fOqFoQob_lcePm4_54ks5uac_cgaJpZM4Ue9jP

.

--
📲
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420975320, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AAZ_F-D39WYQssIhAVisPjDtXQncEs4xks5uakJAgaJpZM4Ue9jP
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420989596, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAxJ7CYR8N-koN1hnT6gwFiDOBV_HLmyks5uak_-gaJpZM4Ue9jP
.

--
📲

Thanks. Mind creating an issue? I searched math and didn't find one that's
related.
I started writing one, but didn't have enough details. Was this just for
Jenkins? Or also for Travis? Is it across all the Jenkins tests or is it
correctly failing builds for a subset? It seems like you'd know these
things off the top of your head.
If you don't want to write one, I'll write a vague one later.
On Thu, Sep 13, 2018 at 8:32 AM seantalts<EMAIL_ADDRESS>wrote:

Compiler Warnings do not fail the build
On Thu, Sep 13, 2018 at 08:30 Daniel Lee<EMAIL_ADDRESS>wrote:

What is the problem it's trying to fix? Mind making that an issue? I read
through it and couldn't pick out the actual problem.
On Thu, Sep 13, 2018 at 7:32 AM seantalts<EMAIL_ADDRESS>wrote:

Someone should fix the problem this is aimed at and there is some code
here
for it but it needs work.
On Wed, Sep 12, 2018 at 11:24 PM Daniel Lee<EMAIL_ADDRESS>wrote:

@seantalts https://github.com/seantalts, it looks like this can
just
be
closed? Is that right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420871030,
or
mute
the thread
<

https://github.com/notifications/unsubscribe-auth/AAxJ7JUBCjPaR8fOqFoQob_lcePm4_54ks5uac_cgaJpZM4Ue9jP

.

--
📲
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420975320, or
mute
the thread
<

https://github.com/notifications/unsubscribe-auth/AAZ_F-D39WYQssIhAVisPjDtXQncEs4xks5uakJAgaJpZM4Ue9jP

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420989596, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AAxJ7CYR8N-koN1hnT6gwFiDOBV_HLmyks5uak_-gaJpZM4Ue9jP
.

--
📲
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420989983, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAZ_F1thQdM6Vtw7tZBfoPpYdl7AtafDks5ualBigaJpZM4Ue9jP
.

There is no Travis anymore on Math. I want to extend that to the other
repos as it still seems flaky. I'm a bit torn on the issue, but you asked
for me to do it. As far as I remember from what you told me, you just
wanted to make sure that compiler warnings failed the build. I can make
that into an issue if you want and try to make you the reporter so we can
go to you for more details?
On Thu, Sep 13, 2018 at 8:48 AM Daniel Lee<EMAIL_ADDRESS>wrote:

Thanks. Mind creating an issue? I searched math and didn't find one that's
related.
I started writing one, but didn't have enough details. Was this just for
Jenkins? Or also for Travis? Is it across all the Jenkins tests or is it
correctly failing builds for a subset? It seems like you'd know these
things off the top of your head.
If you don't want to write one, I'll write a vague one later.
On Thu, Sep 13, 2018 at 8:32 AM seantalts<EMAIL_ADDRESS>wrote:

Compiler Warnings do not fail the build
On Thu, Sep 13, 2018 at 08:30 Daniel Lee<EMAIL_ADDRESS>wrote:

What is the problem it's trying to fix? Mind making that an issue? I
read
through it and couldn't pick out the actual problem.
On Thu, Sep 13, 2018 at 7:32 AM seantalts<EMAIL_ADDRESS>wrote:

Someone should fix the problem this is aimed at and there is some
code
here
for it but it needs work.
On Wed, Sep 12, 2018 at 11:24 PM Daniel Lee <
<EMAIL_ADDRESS>wrote:

@seantalts https://github.com/seantalts, it looks like this can
just
be
closed? Is that right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/stan-dev/math/pull/891#issuecomment-420871030
,
or
mute
the thread
<

https://github.com/notifications/unsubscribe-auth/AAxJ7JUBCjPaR8fOqFoQob_lcePm4_54ks5uac_cgaJpZM4Ue9jP

.

--
📲
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420975320,
or
mute
the thread
<

https://github.com/notifications/unsubscribe-auth/AAZ_F-D39WYQssIhAVisPjDtXQncEs4xks5uakJAgaJpZM4Ue9jP

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420989596, or
mute
the thread
<

https://github.com/notifications/unsubscribe-auth/AAxJ7CYR8N-koN1hnT6gwFiDOBV_HLmyks5uak_-gaJpZM4Ue9jP

.

--
📲
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420989983, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AAZ_F1thQdM6Vtw7tZBfoPpYdl7AtafDks5ualBigaJpZM4Ue9jP
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420994077, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAxJ7CYQdMyHc1uwJvxyfdj6W2bQrzViks5ualP-gaJpZM4Ue9jP
.

--
📲

That's perfect! Yeah... sorry, I didn't remember that we remove travis from
math. (And I think that's a god thing.) I just didn't have the details off
the top of my head.
Yes, create the issue and tag me.
On Thu, Sep 13, 2018 at 9:37 AM seantalts<EMAIL_ADDRESS>wrote:

There is no Travis anymore on Math. I want to extend that to the other
repos as it still seems flaky. I'm a bit torn on the issue, but you asked
for me to do it. As far as I remember from what you told me, you just
wanted to make sure that compiler warnings failed the build. I can make
that into an issue if you want and try to make you the reporter so we can
go to you for more details?
On Thu, Sep 13, 2018 at 8:48 AM Daniel Lee<EMAIL_ADDRESS>wrote:

Thanks. Mind creating an issue? I searched math and didn't find one
that's
related.
I started writing one, but didn't have enough details. Was this just for
Jenkins? Or also for Travis? Is it across all the Jenkins tests or is it
correctly failing builds for a subset? It seems like you'd know these
things off the top of your head.
If you don't want to write one, I'll write a vague one later.
On Thu, Sep 13, 2018 at 8:32 AM seantalts<EMAIL_ADDRESS>wrote:

Compiler Warnings do not fail the build
On Thu, Sep 13, 2018 at 08:30 Daniel Lee<EMAIL_ADDRESS>wrote:

What is the problem it's trying to fix? Mind making that an issue? I
read
through it and couldn't pick out the actual problem.
On Thu, Sep 13, 2018 at 7:32 AM seantalts<EMAIL_ADDRESS>wrote:

Someone should fix the problem this is aimed at and there is some
code
here
for it but it needs work.
On Wed, Sep 12, 2018 at 11:24 PM Daniel Lee <
<EMAIL_ADDRESS>wrote:

@seantalts https://github.com/seantalts, it looks like this
can
just
be
closed? Is that right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/stan-dev/math/pull/891#issuecomment-420871030
,
or
mute
the thread
<

https://github.com/notifications/unsubscribe-auth/AAxJ7JUBCjPaR8fOqFoQob_lcePm4_54ks5uac_cgaJpZM4Ue9jP

.

--
📲
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<https://github.com/stan-dev/math/pull/891#issuecomment-420975320
,
or
mute
the thread
<

https://github.com/notifications/unsubscribe-auth/AAZ_F-D39WYQssIhAVisPjDtXQncEs4xks5uakJAgaJpZM4Ue9jP

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420989596,
or
mute
the thread
<

https://github.com/notifications/unsubscribe-auth/AAxJ7CYR8N-koN1hnT6gwFiDOBV_HLmyks5uak_-gaJpZM4Ue9jP

.

--
📲
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420989983, or
mute
the thread
<

https://github.com/notifications/unsubscribe-auth/AAZ_F1thQdM6Vtw7tZBfoPpYdl7AtafDks5ualBigaJpZM4Ue9jP

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-420994077, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AAxJ7CYQdMyHc1uwJvxyfdj6W2bQrzViks5ualP-gaJpZM4Ue9jP
.

--
📲
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/891#issuecomment-421009306, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAZ_F8sXdmlyl19FPBjj7-mVfHwNSD1yks5ual9NgaJpZM4Ue9jP
.

@seantalts, I added issue #1077.
I'm closing this issue since this PR hasn't been kept up-to-date with develop (and it's never really passed tests) and... it looks like Jenkins has moved to a different warnings plugin? https://github.com/jenkinsci/warnings-plugin (says it's deprecated and points to https://github.com/jenkinsci/warnings-ng-plugin).
The changes on this PR look sound and should only be the ones on Jenkinsfile, so it's not a lot of work. But I don't know if we want to switch to the new warnings plugin.


Setting empty string to language will throw an exception

When using playground, the language prop passed to Web Chat is empty string instead of undefined, which caused Web Chat throw an exception.

Today WebChat will blow up if language = "", it should not blow up/

@corinagum did we fixed this in #2226?
https://github.com/microsoft/BotFramework-WebChat/commit/f5b8db54380cf0059bc0fc55fc580a2aebba0cee#diff-d77ef25d8b6a6b71d83356cce84eb046R148

@compulim haha I thought you said that was a partial fix but I'm not sure what else needs to be done. One of us should test this on the Playground and report back


api: cleanup types

Just a general cleanup before I went modifying some of the types.  Didn't want this messing up the diff.

/lgtm


Multisampling?

First off, thanks for the excellent library. Is there a way to enable multisampling without having to write some opengl boilerplate?

Never mind, I'm dumb, I used SDL vanilla to create a window with multisampling a gave it to setinitwindow and everything is nice and smooth now. Cheers.


Fix spinning Yarn

Coverage remained the same at 75.491% when pulling be0426985633bbddcd0e4b8eb99a50c1fef843b9 on achensetyawati:dev into aa3b9fac4b7b00b197e18332c9ef11b6bd29ef0f on danliris:dev.


[WIP] Added the GuzzleHttp adapter

This adds the guzzle Adapter.
Currently working on fixing tests.
Should this be opened against next_version?

By the way, I'm having issues with fopen(path/to/file.jpg): failed to open stream: No such file or directory
How can I mock this? :/

@Ninir Guzzle have a MockSubsciber, its the easiest way.
http://guzzle.readthedocs.org/en/latest/testing.html

@Baachi The MockSubscriber is meant to queue responses or exceptions. Is there something I am missing?

Yes, its the easies/best way to unit test it. There is also a MockHandler but i never used it.

Thanks for your work on this :)


BatchNormalization mode=2

How to get BatchNormalization mode=2 which was in the previous versions of keras?
According to the documentation of keras==1.2.2 mode=2 does :  ' feature-wise normalization, like mode 0, but using per-batch statistics to normalize the data during both testing and training.'
Why was mode removed? and is there any way to get that in the latest version of keras.

Hi!
This issue isn't related to a bug/enhancement/feature request or other accepted types of issue.
Request you to post these type of questions in the following resources :
https://keras.io/
https://keras.io/#support
https://gitter.im/Keras-io/Lobby
Thanks!
If you think I made a mistake, please open another issue.


redirect on first time installer

is there a way to make redirect to 'install' route on first time installer? i try with checking pdo connection on appserviceprovider, but there's something i need that appserviceprovider can't handle

Why not create a middleware that checks the presence of the installation file generate in the storage folder ? and redirect to /install if it is not present ?

    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request $request
     * @param  \Closure $next
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        $filename = storage_path("installed");
        if (file_exists($filename)) {
            return $next($request);
            echo "Le fichier $filename existe.";
        } else {
            return redirect('/install');
        }
    }


Use compression in insert_arrow, with example ArrowStream

I noticed insert_arrow is about 10x slower that this equivalent code which does the insert over HTTP using zstandard compression. Maybe something else is going on, but from looking at network usage on my desktop, I think it's because insert_arrow is sending uncompressed data.
Try something like this as an example. This method uses Arrow to do the compression, which is nice and fast.
import pyarrow as pa
import requests
import orjson

def insert_arrow_http(arrow_table_to_insert, ...):
    if arrow_table_to_insert.shape[0] == 0:
        print("arrow_table_to_insert is empty - skipped")
        return

    ch_host = ...
    ch_http_port = ...
    ch_user = ...
    ch_password = ...
    ch_url = f"http://{ch_host}:{ch_http_port}"

    sink = pa.BufferOutputStream()
    arrow_file_writer = pa.ipc.new_file(
        sink,
        arrow_table_to_insert.schema,
        options=pa.ipc.IpcWriteOptions(compression=pa.Codec(compression='zstd', compression_level=3))
    )
    arrow_file_writer.write_table(arrow_table_to_insert)
    arrow_file_writer.close()

    compressed_bytes= sink.getvalue()
    compressed_bytes_length = len(compressed_bytes)

    response = requests.post(
        url=ch_url,
        auth=requests.auth.HTTPBasicAuth(ch_user, ch_password),
        params={
            "query": "insert into the_db.the_table format Arrow",
        },
        data=compressed_bytes
    )

    if "X-ClickHouse-Exception-Code" in response.headers:
        error_string = f"""X-ClickHouse-Exception-Code {response.headers['X-ClickHouse-Exception-Code']}"""
        raise RuntimeError(error_string)

    ch_summary = orjson.loads(response.headers['X-ClickHouse-Summary'])
    written_rows = int(ch_summary['written_rows'])
    elapsed_s = float(ch_summary['elapsed_ns']) /<PHONE_NUMBER>
    print(f"{written_rows:,} rows in {elapsed_s:.3f}s @ {written_rows/elapsed_s:,.0f} rows/sec, {compressed_bytes_length /written_rows:,.1f} compressed bytes/row")

Something similar works for inserting using ArrowStream, with generators that yield RecordBatches. For ArrowStream, the payload needs to be an IPC-serialized schema followed by IPC-serialized data batches.
def arrow_stream_generic_generator(schema, arrow_stream_reader, codec="zstd"):
    uncompressed = schema.serialize()
    compressed = pyarrow.compress(uncompressed, codec=codec)
    yield (compressed)
    for chunk in arrow_stream_reader:
        uncompressed = chunk.serialize()
        compressed = pyarrow.compress(uncompressed, codec=codec)
        yield (compressed)

Then an example using that with DuckDB to read from a folder of parquet files (could be CSV, whatever) and insert using ArrowStream.
DuckDB has a fetch_record_batch which yields Arrow batches.
ddb = duckdb.connect()
sql = "SELECT * FROM 'test_file.parquet'"
arrow_stream_reader = ddb.execute(sql).fetch_record_batch(rows_per_batch=100000)

response = requests.post(
    url=ch_url,
    auth=HTTPBasicAuth(ch_user, ch_password),
    # Don't use send_progress_in_http_headers here - breaks with long-running inserts (too many headers)
    params={
        "enable_http_compression": "1",
        "query": "insert into the_db.the_table format ArrowStream",
    },
    headers={"content-encoding": "zstd"},
    data=arrow_stream_generic_generator(
        arrow_stream_reader.schema, arrow_stream_reader
    ),
)

It should be possible to do ArrowStream in the same style as the first Arrow example, with pa.ipc.new_stream used like pa.ipc.new_file, I just couldn't figure out how to wire it up correctly to do this kind of lower-footprint streaming pipeline with Requests.
arrow-odbc is another example of a package with support for streaming results as arrow batches (which can then be used with the arrow_stream_generic_generator above for similar streaming inserts from a streaming source). Might be helpful to look at if you'd like to add support for a read_arrow_batches in clickhouse_connect.

Thanks for the analysis!  Adding arrow streams support has been discussed in other issues but having more code examples is much appreciated.  I like the idea of using the Arrow compression as well, since that's tricky to implement otherwise.  Of course if you do get this working with clickhouse_connect a PR would be awesome.

@chriscomeau79 have you observed similar problem with fetching data from client as "arrow" format? or the library implementation of fetching data as arrow is good enough? (talking about python client)

@chriscomeau79

It should be possible to do ArrowStream in the same style as the first Arrow example, with pa.ipc.new_stream used like pa.ipc.new_file, I just couldn't figure out how to wire it up correctly to do this kind of lower-footprint streaming pipeline with Requests.

Yes. It's possible:
import pyarrow as pa
import requests
from requests.auth import HTTPBasicAuth

def send_batch(batch: pa.RecordBatch) -> None:
    sink = pa.BufferOutputStream()
    with pa.ipc.new_stream(
            sink,
            batch.schema,
            options=pa.ipc.IpcWriteOptions(compression=pa.Codec(compression='zstd', compression_level=3)),
    ) as writer:
        writer.write(batch)
    writer.close()
    response = requests.post(
        url="http://localhost:8123",
        auth=HTTPBasicAuth("default", ""),
        params={
            "query": "insert into default.table1 format ArrowStream",
        },
        data=sink.getvalue()
    )
    if "X-ClickHouse-Exception-Code" in response.headers:
        error_string = f"X-ClickHouse-Exception-Code {response.headers['X-ClickHouse-Exception-Code']}"
        raise RuntimeError(error_string)

There's no point in serializing the whole Arrow format when you have a RecordBatch. It meant to be written batch-by-batch per HTTP request.
You can always write multiple batches and then send it to ClickHouse too.


⚠ Bofrost.de has degraded performance

In 07dd19c, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1013 ms

Resolved: Bofrost.de performance has improved in ac4b078.


Better ratio trend

New page with a different kind of visualisation of the overall trend. Heavily inspired by this video by MinutePhysics

Fix typo in #13


3.2. Particles disappearing when using the scrollwheel outside of running window

Godot version: 3.2 beta4
OS/device including version: Win7
Issue description:
When using the mouse scrollwheel outside of the running window, particles disappear. Moving the mouse back into the window area causes the particles to reappear. The scrollwheel has no affect if mouse is inside the running window. There is no input map assigned to the mouse scrollwheel. No code using the scrollwheel scancode. The issue is present when running the project from the editor as well as when running the exported project.

Steps to reproduce:
Run the minimal project and scroll inside vs outside the running window.
Minimal reproduction project:
particle_test_scrollwheel.zip

I cannot reproduce this issue on 3.2.beta.custom_build.5fa190513 (commit 5fa190513)
OS: Arch Linux, kernel 5.4.6-arch3-1
Graphics: Intel Integrated Graphics UHD 620 (Whiskey Lake)

@FlashDaggerX The frame drop is just due to screen recording. When I don't screen record, then there is absolutely no frame dropping with this minimal project regardless of what I do.

Seems like particles visibility simply turns off when they go outside screen/viewport, that's an issue I'm experiencing for quite some time, for instance when a projectile with a smoke trail goes outside the screen. I think what is needed is calculating the bounding box of all particles before switching visibility completely, but since it's GPU accelerated not sure how feasible the solution might be.

@Xrayez  If that's the case, why does are the particles then only disappearing when I use the scrollwheel outside of the viewport?

I couldn't reproduce either in latest build btw, it just may be connected to the issue in some way, not sure how scrollwheel could affect this. 👀

I think what is needed is calculating the bounding box of all particles before switching visibility completely, but since it's GPU accelerated not sure how feasible the solution might be.

You can enlarge the AABB manually using Drawing > Visibility Rect in the inspector.
However, with CPUParticles2D, the AABB is supposed to be computed automatically since Godot 3.1.

Thanks, yes it did solve my issue, not sure if this was present back in 2.1 last time I checked! I might have also misinterpreted this property as something which clips the drawing instead so I neglected it in the past...

@Xrayez You got it!
Changing the Visibility Rect changes the margin after which using the scrollwheel hides visibility of the particles.
I still don't understand why this happens, especially why the scrollwheel thing, but as long as you guys know what going on I'm happy.

According to docs:

I wonder why it mentions editor specifically (misleading?). It may happen that editor input may affect particles in some way (with Sync Scene Changes option or similar), but I hardly see much connections in the source code.

Can anyone still reproduce this bug in Godot 3.2.3 or any later release?
I can't reproduce either btw

I can still reproduce it with the minimal project above.

I can't confirm this on 3.3.2 on Linux.

I can still reproduce it with the included MRP on 3.3.2 stabel on Win.
Increasing the Visibility Rect works as a workaround but does not eliminate the issue.

@golddotasksquestions Can you (or anyone else) still reproduce this bug in Godot 4.0.beta4 or any later release?
If yes, please ensure that a 4.0 up-to-date Minimal Reproduction Project (MRP) is included in this report (a MRP is a zipped Godot project with the minimal elements necessary to reliably trigger the bug). You can upload ZIP files in an issue comment with a drag and drop.

@Calinou
Yes, I can still reproduce this on the same hardware. Also in Godot4 Beta4. I have converted the original MRP to Godot 4 Beta4 and added it to the original post.
While testing this again, I noticed not only the particles are disappearing, but the whole KinematicBody2D/CharacterBody2D the Particles2D are added to. So maybe this is not related to the Particles? I checked the above gif I made years ago, and even there it seems like the Sprite is disappearing, indicating the KB/CB is totally disappearing when scrolling outside, not just the Particles. That something I apparently did not notice before.
So I made two more small test scenes (one with CharacterBody2D and Sprite2D and one with only the Sprite), and the issue was present in both as long as they where moved to the global_mouse_position()
extends Sprite2D

func _process(delta):
	global_position = get_global_mouse_position()


webpack doesn't handle babel while running from npm install flow

Bug report

What is the current behavior?
webpack doesn't run babel loader or babel loader doesn't work
If the current behavior is a bug, please provide the steps to reproduce.
Steps to reproduce:

mkdir my_test_project
cd my_test_project
npm init
npm i --save git+https://github.com/alexey-sh/test_webpack_with_npm.git
cat node_modules/testname/dist/index.js

It contains code with class declaration. but if I do

git clone https://github.com/alexey-sh/test_webpack_with_npm.git
cd test_webpack_with_npm
npm i
cat dist/index.js

I'll see a different code without class declaration (as expected)
There is also fix/install branch with working code
So the following steps works fine, and dist/index.js doesn't contain class declaration.

mkdir my_test_project
cd my_test_project
npm init
npm i --save git+https://github.com/alexey-sh/test_webpack_with_npm.git#fix/install
cat node_modules/testname/dist/index.js

What is the expected behavior?
It should work both from a repository and from package

Other relevant information:
webpack version:  latest
Node.js version:  v11.15.0, npm 6.13.0
Operating System: mac os x 10.14.6
Additional tools: babel

exclude: /(node_modules|bower_components)/,
You excluded node_modules. If your code is installed in node_modules babel-loader won't be applied.
Prefer include over exclude

If your code is installed in node_modules babel-loader won't be applied.

is it documented somewhere?


fix: Fix is_text_char.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 92.624%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
8978

Relevant Lines:
9693

💛  - Coveralls


chore(master): release 0.57.0-alpha

:robot: I have created a release beep boop
0.57.0-alpha (2024-03-04)
Features

new (8b09c82)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mafqla/yalis-blog/releases/tag/v0.57.0-alpha :sunflower:


proxy.golang.org: how to get rid of wrong version in the proxy

We have accidentally pushed dummy tag v6.7.8 to our repository several months ago, and even though it has been removed since then, proxy still remembers it. Our project is at 0.4.0 at the moment. How can we get get rid of that dummy version from golang proxy? People trying to use our client library are getting old v6.7.8 now. (github.com/grafana/loki v6.7.8+incompatible)
https://proxy.golang.org suggests that we need to publish new version, but that will only be 0.5.0 or perhaps 1.0.0.
Thanks.

The unfortunate news is, there is no way in general to get rid of a valid version in the proxy.
The good news is, CL 204440 has landed for Go 1.14, so as long as your latest compatible tagged version has a go.mod file, users on 1.14 (once it is released) will pick up the correct version instead of the erroneous +incompatible one.
CC @katiehockman @heschik @hyangah

Duplicate of #34189

I am trying to understand what can we do in this situation, except waiting for 1.14 and bumping version to v7. I guess we need to ask our users to use GOPROXY=direct when getting our module.

go get github.com/grafana/loki@v0 is a little better than that, at least.


Cannot read property 'removeItem' of undefined

My configs:
"test": "NODE_PATH=./frontend/src mocha  -r mock-local-storage   --compilers js:babel-core/register --require ./test/test_helper.jsx --recursive ./test",
1.When I try to remove item
localStorage.removeItem('term')
I keep getting error:
TypeError: Cannot read property 'removeItem' of undefined

When I try to set empty string get same error:

localStorage.setItem('term', '')

getItem and setItem(for truthy arguments) work well

Closed the issue. The problem was with browser history  https://github.com/ReactTraining/react-router/issues/3749


Initiating Login from a Third Party

In some cases, the login flow is initiated by an OpenID Provider or another party, rather than the Relying Party. In this case, the initiator redirects to the RP at its login initiation endpoint, which requests that the RP send an Authentication Request to a specified OP. This login initiation endpoint can be a deep link at the RP, rather than a default landing page. RPs supporting OpenID Connect Dynamic Client Registration 1.0 [OpenID.Registration] register this endpoint value using the initiate_login_uri Registration parameter.

The party initiating the login request does so by redirecting to the login initiation endpoint at the RP, passing the following parameters:

iss

REQUIRED. Issuer Identifier for the OP that the RP is to send the Authentication Request to. Its value MUST be a URL using the https scheme.

login_hint

OPTIONAL. Hint to the Authorization Server about the login identifier the End-User might use to log in. If the client receives a value for this string-valued parameter, it MUST include it in the Authentication Request as the login_hint parameter value.

target_link_uri

OPTIONAL. URL that the RP is requested to redirect to after authentication. RPs MUST verify the value of the target_link_uri to prevent being used as an open redirector to external sites.

The parameters can either be passed as query parameters using the HTTP GET method or be passed as HTML form values that are auto-submitted in the User Agent, and thus are transmitted via the HTTP POST method.

Other parameters MAY be sent, if defined by extensions. Any parameters used that are not understood MUST be ignored by the Client.

Clients SHOULD employ frame busting and other techniques to prevent End-Users from being logged in by third party sites without their knowledge through attacks such as Clickjacking. Refer to Section <IP_ADDRESS> of [RFC6819] for more details.

http://openid.net/specs/openid-connect-core-1_0.html#ThirdPartyInitiatedLogin

+1 for this feature.
Although this feature looks like it concerns only the protected apps (RP), it is a very nice addition to anvil.
Because during a social login, Anvil is indeed the RP for another OP (e.g. Google).
Therefore if a protected app already knows the final IdP that a given user wants to use, it is nice that anvil can find a way to bypass its own account chooser screen containg the list of supported OUTBOUND IdPs.
Adding an "iss" parameter on the /authorize request is an OIDC compliant way for a registered client to request an Outbound authentification to a registered idP.
Only Anvil will understand this parameter, because it is the only IdP (to my knowledge) that allows "Outbond" authN scenarios.
Other IdPs would discard this parameter, as they should.

Let me add an important point: in scenarios where the outbound IdP is in the same security realm as the interactive user, the outbound IdP can implement "auto-login" strategies with technologies such as Kerberos, NTLM, or even siteminder, thus providing to the interactive user a passwordless experience.

I would however recommend a configuration option that allows the admin to enable the iss parameter for "authenticated registered" clients only (let it be the default setting)
This would force that the iss parameter is only active for registered apps AND grant types that enforce client authentication (HTTP_BASIC for autorisation_code flow).
Knowing that the client-id can be spoofed in the implicit flow, this would allow an admin to say that he doesn't want any (JS or not) client to attempt that operation... thus preventing phishing attempts from unregistered apps...
Please note that the consequence of such a phishing attempt would only be that the user has proven is identity to the unregistered app (no password leakage, and no scopes other than "profile" could be added to the access_tokens)
But that's bad enough if you want my opinion.
However, it might not be so important to some very small IdPs (think university... Etc), especially knowing that these attempts could be made anyways, even if anvil shows an account chooser login form.

And of course, this has no consequences when IdPs are chained within the same secuity realm. E.g a global intranet with several local IdPs, or a federation of intranets through secure links.

So in case i was not clear, i'm still very very much in favor of that feature!!! ;)))


Unable to capture video/audio in Mac devices

Describe the bug
Unable to capture video/audio in Mac devices.
To Reproduce

Run the following snippet on Mac devices

from ffmpeg_streaming import Formats, input, Bitrate, Representation, Size

if __name__ == '__main__':
    video = input("0:0", capture=True)
    hls = video.hls(Formats.h264())
    _720p = Representation(Size(1280, 720), Bitrate(2048 * 1024, 320 * 1024))
    hls.representations(_720p)

    hls.output("hls.m3u8")

Error

RuntimeError: ('ffmpeg failed to execute command: ', "b'ffmpeg version 4.4.2 Copyright (c) 2000-2021 the FFmpeg developers\\n  built with clang version 13.0.1\\n  configuration: --prefix=/Users/runner/miniforge3/conda-bld/ffmpeg_1654969367107/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_pl --cc=x86_64-apple-darwin13.4.0-clang --disable-doc --disable-openssl --enable-avresample --enable-demuxer=dash --enable-gnutls --enable-gpl --enable-hardcoded-tables --enable-libfreetype --enable-libopenh264 --enable-libx264 --enable-libx265 --enable-libaom --enable-libsvtav1 --enable-libxml2 --enable-libvpx --enable-pic --enable-pthreads --enable-shared --disable-static --enable-version3 --enable-zlib --enable-libmp3lame --pkg-config=/Users/runner/miniforge3/conda-bld/ffmpeg_1654969367107/_build_env/bin/pkg-config\\n  libavutil      56. 70.100 / 56. 70.100\\n  libavcodec     58.134.100 / 58.134.100\\n  libavformat    58. 76.100 / 58. 76.100\\n  libavdevice    58. 13.100 / 58. 13.100\\n  libavfilter     7.110.100 /  7.110.100\\n  libavresample   4.  0.  0 /  4.  0.  0\\n  libswscale      5.  9.100 /  5.  9.100\\n  libswresample   3.  9.100 /  3.  9.100\\n  libpostproc    55.  9.100 / 55.  9.100\\n0:0: Protocol not found\\nDid you mean file:0:0?\\n'")

Expected behavior
The webcam records the video/audio and should store as "hls.m3u8".
Local machine (please complete the following information):

OS: [e.g. Mac OS]
FFMped [e.g. 4.4.2]

Since I unable to create a pull request, I will add my suggestion in this comment.
Findings:

The root cause for the above issue is due to the options are not properly transferred to initialisation classes.

As per the code here. The options are altered.

Suggestions/Fix:

Use get instead of pop. This will be applicable to capture and pre_opts.

Kindly verify the same and add your thoughts.  Thanks in advance.

Facing the same issue on my mac. Can't capture webcam video

It works for me when i add get instead for pop. How to add multiple representations because when I add _360p and _720p, I get error?


Vangelico hole

There's a large hole near vangelicos that causes you to fall through the map.

Where?

On the west side of the alleyway near the steps. Even locals fall through.
On Mon, Dec 16, 2019, 1:18 PM Bob74<EMAIL_ADDRESS>wrote:

Where?
[image: image]
https://user-images.githubusercontent.com/9498543/70931897-c561cd80-2038-11ea-8067-4bf6b7b270f2.png
[image: image]
https://user-images.githubusercontent.com/9498543/70931913-cc88db80-2038-11ea-9b5f-5554e5f8eca8.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Bob74/bob74_ipl/issues/21?email_source=notifications&email_token=AAKAVXYHNLRRBSUFGNUGZITQY7A6XA5CNFSM4JSQGICKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEG7TLMI#issuecomment-566179249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKAVX5SITLGYFSYQVAPXMLQY7A6XANCNFSM4JSQGICA
.

With the default provided ipl unloaded the hole isn't there so I'm sure it's related to the jewelry store ipl.

Is it this place?

It can be a conflict with another resource but I'm only using my resource to test and it works fine.

Yes that's the place. We don't have any resources other than yours that
deal with maps and whenever we walk around in that area we fall through.
On Mon, Dec 16, 2019, 1:25 PM Bob74<EMAIL_ADDRESS>wrote:

Is it this place?
[image: image]
https://user-images.githubusercontent.com/9498543/70932291-ae6fab00-2039-11ea-9ae6-6ef5a09933d0.png
It can be a conflict with another resource but I'm only using my resource
to test and it works fine.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Bob74/bob74_ipl/issues/21?email_source=notifications&email_token=AAKAVX5QAIM25UHQMXNEZV3QY7BZZA5CNFSM4JSQGICKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEG7UB6I#issuecomment-566182137,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKAVXZ3PGH3CD3WHZ2YXVLQY7BZZANCNFSM4JSQGICA
.

Did you try disabling some of your resources to find which one might conflict?

Can you verify which IPL is the correct one to be loaded? Maybe somebody else configured the wrong one.

This is the cut off. Once we go past this spot there is no collision and we fall through the map.

The only IPL I load in this area are these ones:
    RequestIpl("post_hiest_unload") -- Jewelry
    RequestIpl("refit_unload") -- Max Renda

You can try removing them manually to see if it does something:
    RemoveIpl("post_hiest_unload") -- Jewelry
    RemoveIpl("refit_unload") -- Max Renda

Also, I'm interested in your resources list if you don't mind.

Okay interesting. Removing them doesn't appear to be affecting the hole so it must be something else. All of the maps we load are on the east side of the map. Looks like it's not the IPL loader after all! Guess I've got some investigation to do.

I suggest you try without loading any other maps since it can use quite a lot of memory and lead to this kind of issue (if it is custom ymap).
I'm really interested if you find out which resource cause this since it seems to be quite common ;)

So I found the issue in my particular case. I am using a custom map thst opens a couple magazine shops in town, which conflict with the vangelico map for some reason.

Thanks, I think most people having these issues use custom maps also.
Good to know 😉


Update Errror

Hello, I'm noticing that when I try to update through XivAlex itself that an error pops up where while it does download to an UPDATE_ folder the process fails because it tries to pull the update content from the wrong path, from the look of it.

It definitely has downloaded properly but instead of checking for it there it doesn't check for it in the subfolder it creates, in this case, 'XivAlexander_1_14_8_54_df4e86a'

The self updater didnt work for me either. You'll have to manually uninstall the old version and reapply the new one through the files it had downloaded.

https://github.com/Soreepeong/XivAlexander/releases/tag/v<IP_ADDRESS>
The problem should have been fixed, please download the latest relase from the link above.


fix(core): fix require_literal_leading_dot flipped behavior

Regression introduced in: https://github.com/tauri-apps/tauri/pull/6969 see https://github.com/tauri-apps/tauri/pull/6969#discussion_r1232018347

What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[X] No

Checklist

[X] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[X] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[X] I have added a convincing reason for adding this feature, if necessary

Other information

Shouldn't we just PR against 1.x and just cherry pick to dev or merge 1.x to dev later?

yeah I will just open another PR


Translate Reference Docs Overview section into Russian

/auto-cc

/cc @kubernetes/sig-docs-ru-owners

@zacharysarah please approve this PR :smiley: (as the bot requires it)

/lgtm
/approve


Does this project have a license?

Hey there! I've been going through some roguelike tutorials and I came across your sample code and am doing some exercises based off of it. Sometimes I like to use the results of such studies in future projects, but I wanted to check with you first if that's okay as I didn't notice any particular license on this repo. Can you confirm what terms this code is available under?
Thanks for writing and sharing this!

Code is available under MIT license. It was in the package.json file, but I just explicitly added a LICENSE file to clarify.


TextField, when center aligned, shows caret in the middle when pasting text

Steps to Reproduce
I have a TextField like this:
child: TextField(
        onChanged: handleInputChange,
        enabled: !disabled,
        textAlign: TextAlign.center,
        style: TextStyle(color: Colors.black, fontSize: 16.0),
        obscureText: obscureText,
        decoration: new InputDecoration(
          contentPadding: const EdgeInsets.all(7.0),
          hintText: hintText,
          border: new OutlineInputBorder(
              borderRadius: new BorderRadius.all(const Radius.circular(5.0))
          )
        ),
      )

As I type, the caret is correctly shown at the end, but if I paste text from my keyboard, like my email which it remembers, the caret is shown in the middle in a weird position, not really in between characters. BTW, this does not happen when left aligned, in which case the caret is correctly advanced to the end.

Logs
N/A
Flutter doctor
$ flutter doctor -v
[✓] Flutter (Channel beta, v0.8.2, on Mac OS X 10.13.6 17G65, locale en-US)
    • Flutter version 0.8.2 at /Users/pablo/Documents/work/projects/forks/flutter
    • Framework revision 5ab9e70727 (3 weeks ago), 2018-09-07 12:33:05 -0700
    • Engine revision 58a1894a1c
    • Dart version 2.1.0-dev.3.1.flutter-760a9690c2

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.2)
    • Android SDK at /Users/pablo/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.2
    • ANDROID_HOME = /Users/pablo/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.0, Build version 10A255
    • ios-deploy 2.0.0
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 28.0.2
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[✓] Connected devices (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 7.1.1 (API 25) (emulator)

• No issues found!

This also happens when writing text if there's a space at the end.

I made a workaround to fix this problem, you can try it. @JediBurrell
_controller.addListener(() {
  if (_controller.text.length != _controller.text.trim().length) {
    _controller.text = _controller.text.trim();
  }
  _controller.selection =
    TextSelection.collapsed(offset: _controller.text.length);
});

This seems fixed for me starting with v0.10.2, can you guys verify it got fixed from 0.9.x to 0.10.x?

My flutter version is 0.11.3, and I met same problem with @JediBurrell 's situation(text if there's a space at the end), so I found a workaround. This is my flutter doctor:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v0.11.3, on Mac OS X 10.14.1 18B75, locale en-CN)
[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
[✓] Android Studio (version 3.2)
[!] IntelliJ IDEA Ultimate Edition (version 2018.2.6)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
[✓] VS Code (version 1.28.2)

I'm unable to reproduce this on the iOS simulator.  I'll try on a real device soon if I can get set up.  There have been several improvements to the TextField cursor recently that hopefully have fixed this.

With the example code given I cannot reproduce this anymore on neither a physical Android nor iOS device. I am tentatively closing this. If you still see this, please do not hesitate to reopen with additional details of how you reproduced it.

Looks like a dup of #24456 that was fixed recently https://github.com/flutter/flutter/issues/24456#issuecomment-446707682

I made a workaround to fix this problem, you can try it. @JediBurrell
_controller.addListener(() {
  if (_controller.text.length != _controller.text.trim().length) {
    _controller.text = _controller.text.trim();
  }
  _controller.selection =
    TextSelection.collapsed(offset: _controller.text.length);
});

hello, but i wanna show the last white space, do you have any idea? your method will remove the last white space.


Allow users to choose the base image

I have abstracted the Dockerfile and Makefile to allow users to start with a base image other than the plain ubuntu image.  In my case, I need both Nvidia/CUDA as well as VNC/noVNC functionality.

Hi @chipkent Can you please provide your Dockerfile with nvidia support. I tried your way, but its not working for me.


Make maven agent version agnostic

Version agnostic maven agent already exists in agent-maven folder
CI has been updated on prow side as well.
This PR tests if those updates are still working properly.

/test e2e-aws-jenkins

/retest

/test e2e-aws-jenkins

/retest

/retest


Improve README

The README is not really clear and should be updated (https://docs.logdna.com/docs/cloudwatch too)
What wasn't clear is:

1 - Create a new Lambda function and skip to the Configure function section I guess this was about an old interface? It's impossible to find the Configure Function section see screenshot below

Before I could click on create, I had to create a role (this wasn't mentioned)
Only after that I was able to click on Upload .zip from the Code entry type select input, and edit the handler see screenshot

And the environment vars below

This is good to know! We'll work on updating and making README better. Thanks for suggestions!

Still the same README 👎

@ishi-geru @Jolg42
We have improved and just updated our lambda function and docs - please, let us know if you have any other question! So sorry for being late! Thanks!


nodes created via autoscaling group fail to mount /home and then "instance was taken out of service in response to a user health-check." and repeat ad infinitum

Environment:
aws-parallelcluster-2.7.0
Bug description and how to reproduce:
I am not clear on reproduction. I have a running parallelcluster instance with an autoscaling group and Slurm. When a job is submitted the watcher sees no nodes and tells the ASG to add a node and it goes away and provisions a node but that fails so that node is terminated and then the process starts again.
The problem is that if a compete node fails it writes the logs to /home/logs but it is failing as it cannot mount \home so no log files.
- if a compute node was terminated due to failure, there will be a directory
`/home/logs/compute`. Attach one of the `instance-id.tar.gz` from that directory

So it was working
i-0b895fa88a19fd610.cfn-init.txt
and then failed
i-0ac380e114a0c5b5e.cfn-init.txt

I just found out the error. The compute nodes belong to a security group and that needs to be in the security group for the master node. It was removed (in error) so the compute nodes could come up but not access the master node when the mount happens. I assume the mount is the first time the compute node tries to get to the master node.

Yes correct, mount /home shared from master is one of the first action done by the compute during startup.
Thank you for reporting the resolution of your issue

@lukeseawalker is there any way to add diagnostics for this? It would help greatly if the compute node reported that it could not access the master node.


Implement ability to control the sorting

It would be great if we could specify a sort, rather than automatically having the list of cards be reversed. This would be particularly helpful when the data backing the cards is used elsewhere on the page.

@mattblang can't you just sort your data?


reactTraverser.js:29 Uncaught TypeError: Converting circular structure to JSON

For some reason this is happening..

i am also caught in this error for a few days now


CAF-3538: Add Document Worker and Composite Document Worker policy ty…

https://jira.autonomy.com/browse/CAF-3538
…pes to database base data so they are available for chained worker logic.

This needs updated to specify an entry for type 'ChainedAction', using same settings as CompositeDocumentWorkerHandler. Remove entries for Composite and DocumentWorkerHandler.


vip_axilite fails in newer modelsim versions

The v2 2020.10.06 release which adds axilite_channel_handler_pkg fails with a segmentation error in newer versions of Modelsim DE.
This is the error message
# Attempting stack trace sig 11
# Signal caught: signo [11]
# vsim_stacktrace.vstf written
# Current time Fri Oct 23 15:20:42 2020
# ModelSim DE Stack Trace
# Program = vsim
# Id = "2020.4"
# Version = "2020.10"
# Date = "Oct 13 2020"
# Platform = "linuxpe"
# Signature = a791d2643986943217c290b2b32443f4
# 0    0x08304195: '<unknown (@0x8304195)>'
# 1    0x0830465f: '<unknown (@0x830465f)>'
# 2    0x08305a99: '<unknown (@0x8305a99)>'
# 3    0xf6ac81e4: '/disk1/jenkins/workspace/kspt_tb_axiLiteRegisters_master/UVVM/bitvis_vip_axilite/src/axilite_vvc.vhd:461'
# 4    0x08379139: '<unknown (@0x8379139)>'
# 5    0x085a416a: '<unknown (@0x85a416a)>'
# 6    0xf6ac0193: '/disk1/jenkins/workspace/kspt_tb_axiLiteRegisters_master/kspt/src/support/tb_axiLiteRegisters/Vhdl/tb_axiLiteRegisters.testBench.vhd:87'
# 7    0x08379139: '<unknown (@0x8379139)>'
# 8    0x085a416a: '<unknown (@0x85a416a)>'
# 9    0x08652c05: '<unknown (@0x8652c05)>'
# 10   0x0863ca81: '<unknown (@0x863ca81)>'
# 11   0x0863dbe1: '<unknown (@0x863dbe1)>'
# 12   0x08b293bc: '<unknown (@0x8b293bc)>'
# 13   0x08b1eec9: '<unknown (@0x8b1eec9)>'
# 14   0x092d34eb: '<unknown (@0x92d34eb)>'
# 15   0x092d2ada: '<unknown (@0x92d2ada)>'
# 16   0x092d8063: '<unknown (@0x92d8063)>'
# 17   0x092d817a: '<unknown (@0x92d817a)>'
# 18   0x08816bc9: '<unknown (@0x8816bc9)>'
# 19   0x086a6519: '<unknown (@0x86a6519)>'
# End of Stack Trace

/.. Lines deleted for brevity ../

# ** Fatal: (SIGSEGV) Bad handle or reference.
#    Time: 0 ps  Iteration: 0  Process: /tb_axiliteregisters/i_axilite_vvc/read_address_channel_executor File: /disk1/jenkins/workspace/kspt_tb_axiLiteRegisters_master/UVVM/bitvis_vip_axilite/src/axilite_vvc.vhd
# FATAL ERROR while loading design

Tested modelsim DE versions:
2020.4 (fails)
2020.1 (fails)
2019.3 (works)
When we checkout v2020.08.21 the testbench works in all the versions listed.
We didn't change anything in the testbench except update our filelist to include axilite_channel_handler_pkg.

Hi,
Yes, this seems to be a bug in the simulator related to the unconstrained arrays used in the Axilite VVC when calling the procedures from the axilite_channel_handler_pkg. We will contact Mentor about this.
In the meanwhile, what you can do is to map the signals from the record interfaces directly in each procedure call in the VVC, e.g.
write_response_channel_check(msg                => format_msg(v_cmd),
                              clk                => clk,
                              bready             => axilite_vvc_master_if.write_response_channel.bready,
                              bresp              => axilite_vvc_master_if.write_response_channel.bresp,
                              bvalid             => axilite_vvc_master_if.write_response_channel.bvalid,
                              alert_level        => error,
                              scope              => C_CHANNEL_SCOPE,
                              msg_id_panel       => v_msg_id_panel,
                              config             => vvc_config.bfm_config);

Which means you have to update the procedures in the axilite_channel_handler_pkg.
You can find the updated files I used for testing this fix in the zip file below to save you some time.
axilite_modelsim_2020_fix.zip
Best regards,
Erick

I found the same issue with axi_channel_handler_pkg on modelsim2021.1 .
Workaround is  the same, you can find bellow the files that fix the issue.
axi_src_corrected.zip
JulienH

Fixed in v2021.04.08


chore: remove espree dependency

Looking at the codebase, we're never using espree anywhere.
Looking at #2, tests are still passing when using the espree dependency of ESLint.
So I think it's safe to remove espree from our devDependencies

Never mind, tests are failing on this 😢
Will probably be possible to remove it once we remove ESLint v7 support I think 🤔


feat: Add Python analogues of Java templates

Globally do not use ~[BODY]~ in python templates And add empty_line to each file if not added

What does it mean? Should templates with ~[BODY]~ be removed?
Also replied in review discussion. Unreplied ones are fixed

Globally do not use ~[BODY]~ in python templates And add empty_line to each file if not added

What does it mean? Should templates with ~[BODY]~ be removed? Are imports from typing are needed? Why not just from typing import * for almost every injected file?
Also replied in review discussions. Unreplied ones are fixed

No, just delete ~[BODY]~ constructions
Imports from typing are needed, because we are not modifying helper classes


Update Readme to reflect new api

Since the sample app was changed to use GitHub api, we should update the Readme to have a corresponding sample.

I can help with this and #297, let me know if/when the RC1 api is stable enough for me to update the readme.

Its stable update away ☺ I'll make changes if anything else needs it

closed by #378.


ui: update npm dependencies to latest

Run commands to update dependencies:
npm outdated
npm update

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

@blueorangutan ui

@rohityadavcloud a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6091 (SL-JID-1240)

@blueorangutan package

@rohityadavcloud a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 2835

@blueorangutan ui
@blueorangutan package

@rohityadavcloud a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@rohityadavcloud a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6091 (SL-JID-1250)

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 2857

@davidjumani great, agree. Let's merge this only after running some manual tests.


Add ripgrep in documentation to test with just ci

After compiling just on OpenBSD/adJ 7.6beta1 and trying to test with just ci  I obtained:
 % just ci
./bin/forbid
+ which rg
which: rg: Command not found.
error: Recipe `forbid` failed on line 123 with exit code 2

I discovered that rg is  ripgrep.  On OpenBSD/adJ I installed with doas pkg_add ripgrep and after that I could run the tests.
By the way only 3 tests failed:
...
failures:

---- attributes::multiple_attributes_one_line stdout ----
Bad stderr: Diff < left / right > :
<error: Justfile contains no recipes.
>exit 1

thread 'attributes::multiple_attributes_one_line' panicked at tests/attributes.rs:59:6:
Output mismatch.
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- constants::constants_are_not_exported stdout ----
thread 'constants::constants_are_not_exported' panicked at tests/constants.rs:60:6:
Stderr regex mismatch:
"echo $HEXUPPER\nbash: línea 1: HEXUPPER: variable sin asignar\nerror: Recipe `foo` failed on line 4 with exit code 127\n"
!~=
/Regex("^(?s).*HEXUPPER: unbound variable.*$")/

---- os_attributes::os stdout ----
thread 'os_attributes::os' panicked at tests/os_attributes.rs:59:7:
  unexpected os family

failures:
    attributes::multiple_attributes_one_line
    constants::constants_are_not_exported
    os_attributes::os

test result: FAILED. 849 passed; 3 failed; 7 ignored; 0 measured; 0 filtered out; finished in 17.29s

error: test failed, to rerun pass `-p just --test integration`
error: Recipe `test` failed on line 17 with exit code 101

for the constants_are_not_exported function
would it be possible to replace
fn constants_are_not_exported() {
 Test::new()
   .justfile(
     "
       set export

       foo:
         echo $HEXUPPER
     ",
   )
   .stderr_regex(".*HEXUPPER: unbound variable.*")
   .status(127)
   .run();
}

by
fn constants_are_not_exported() {
 Test::new()
   .justfile(
     "
       foo:
          echo "source $HOME/$HEXUPPER" >> ~/.bashrc
     ",
   )
   .stderr_regex(".*HEXUPPER: unbound variable.*")
   .status(127)
   .run();
}

inspired from https://github.com/casey/just/discussions/1808

Thanks @olfa-arfani and @laniakea64
Running RUST_BACKTRACE=1 cargo test I obtain:
---- attributes::multiple_attributes_one_line stdout ----
Bad stderr: Diff < left / right > :
<error: Justfile contains no recipes.
>exit 1

thread 'attributes::multiple_attributes_one_line' panicked at tests/attributes.rs:59:6:
Output mismatch.
stack backtrace:
   0: rust_begin_unwind
   1: core::panicking::panic_fmt
   2: integration::test::Test::run
             at ./tests/test.rs:293:7
   3: integration::attributes::multiple_attributes_one_line
             at ./tests/attributes.rs:48:3
   4: integration::attributes::multiple_attributes_one_line::{{closure}}
             at ./tests/attributes.rs:47:34
   5: core::ops::function::FnOnce::call_once
             at /usr/obj/ports/rust-1.81.0/rustc-1.81.0-src/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

---- constants::constants_are_not_exported stdout ----
thread 'constants::constants_are_not_exported' panicked at tests/constants.rs:60:6:
Stderr regex mismatch:
"echo $HEXUPPER\nbash: línea 1: HEXUPPER: variable sin asignar\nerror: Recipe `foo` failed on line 4 with exit code 127\n"
!~=
/Regex("^(?s).*HEXUPPER: unbound variable.*$")/
stack backtrace:
   0: rust_begin_unwind
   1: core::panicking::panic_fmt
   2: integration::test::Test::run
             at ./tests/test.rs:283:7
   3: integration::constants::constants_are_not_exported
             at ./tests/constants.rs:49:3
   4: integration::constants::constants_are_not_exported::{{closure}}
             at ./tests/constants.rs:48:32
   5: core::ops::function::FnOnce::call_once
             at /usr/obj/ports/rust-1.81.0/rustc-1.81.0-src/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

---- os_attributes::os stdout ----
thread 'os_attributes::os' panicked at tests/os_attributes.rs:59:7:
unexpected os family
stack backtrace:
   0: rust_begin_unwind
   1: core::panicking::panic_fmt
   2: integration::os_attributes::os
             at ./tests/os_attributes.rs:59:7
   3: integration::os_attributes::os::{{closure}}
             at ./tests/os_attributes.rs:35:8
   4: core::ops::function::FnOnce::call_once
             at /usr/obj/ports/rust-1.81.0/rustc-1.81.0-src/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

failures:
    attributes::multiple_attributes_one_line
    constants::constants_are_not_exported
    os_attributes::os

test result: FAILED. 849 passed; 3 failed; 7 ignored; 0 measured; 0 filtered out; finished in 21.45s

error: test failed, to rerun pass `--test integration`

Thanks for opening this! I just merged #2493, which prints out a better error message in case ripgrep is missing. I think the test failures should be separate issues.


[Bug] Translation Microya.ApiError<Microya.EmptyBodyResponse> error 3

Steps to Reproduce:

Installed BartyCrouch
Added "translate" to update tasks in config file
Ran bartycrouch update from command line

Expected Behavior:
Expected missing translations to be automatically translated by Microsoft or DeepL
Current Behavior:
2022-12-01 18:53:12.579: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:473:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:12.796: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:476:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:13.046: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:479:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:13.283: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:482:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:13.499: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:485:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:13.766: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:488:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:13.966: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:491:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:14.253: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:494:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:14.345: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:497:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:14.550: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:500:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:14.678: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:503:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:14.788: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:506:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:14.957: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:509:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
2022-12-01 18:53:15.121: ⚠  ./Everyday Language/es-MX.lproj/Localizable.strings:512:  Translation request failed with error: The operation couldn’t be completed. (Microya.ApiError<Microya.EmptyBodyResponse> error 3.)
I get the same error 3 if I use either Microsoft or DeepL. I can't figure out how to troubleshoot it; is it not sending the string to be translated?
Environment

Show environment details

Never mind! It just started working; it must have been that the Azure instance was still spinning up?

@tryconfident Sounds like your issue was fixed and not a problem with BartyCrouch, closing then! 🎉


Unify to /search

Reviewer: @suzinyou
Estimate: 30mins

Ticket
Fixes: 529, 629
Description
Goal

Unify /embeddings-search and /llm-response under a single endpoint called /search with an optional generate_llm_response parameter.

Changes

Create new endpoint and logic. Store user's llm response choice to DB.
Update pytests
Update docs and README
[cleanup] move feedback secret key generation to "save_user_query_to_db()" function
[cleanup] rename the endpoint's parameters to be cleaner
[bugfix] set state to final if embeddings search is successful (used to say in_progress)
added blogpost

Future Tasks

Update error behaviour to be consistent with this design

How has this been tested?

pytests
dev setup
docker compose

Checklist
Fill with x for completed.

[x] My code follows the style guidelines of this project
[x] I have reviewed my own code to ensure good quality
[x] I have tested the functionality of my code to ensure it works as intended
[x] I have resolved merge conflicts
[x] I have updated the automated tests
[x] I have updated the README file
[x] I have updated affected documentation
[x] I have added a blogpost in Latest Updates

Leaving some initial comments!

@suzinyou addressed comments and suggestions and rebased. Pls re-review / merge!


Why does registration always fail?

This error always appears during the final submission.

How to fix this?

Hello, check in the developer console you can get details

I had this error because I already have a Tesla business account associated to the company details I put to create the app settings.
So I ended up to use the same Tesla account to create the App settings.

🥇

I tried for the first time to create an app last night and got the instant "Rejected" notification

Hello, check in the developer console you can get details

@vbarrier How can we get into the developer console without being able to create an app?

Developer console of your browser and you will get the "real" http error with a text detail. In my case it was: "An account already exist with that company"

Same errors! I really don’t know how to register!
Help~

Has anyone solved this? I always get an instant rejection.

I got instant rejection if I include localhost in any of the links.

"Unable to Onboard" means tesla api is not support in some area! So I buy a American server, it work!

@vbarrier Sorry to bother you, Did you successfully registered the application from developer.tesla.com?
I don't know how to proceed with this step. Does it mean I have to develop on devices with Bluetooth？
./tesla-control -ble add-key-request public_key.pem owner cloud_key
Error: failed to find a BLE device: can't init hci: no devices available:

Do you have any suggestions? Thanks a lot!

whatever that means?

whatever that means?

I got the same response.
My region is Korea.

whatever that means?

Same response here, Belgium.
Would that be the reason?

whatever that means?

I got the same response.

FYI, it have been fixed now.

Not for me.


Toggle between play and pause icons fixed

fixes #76

@akshara-sun can you please update description to include which issue this closes

@akshara-sun can you please update description to include which issue this closes

whoops! I commented the issue instead of adding to the description! Let me do that now!

@akshara-sun can you please update description to include which issue this closes

done! the issue is also linked to the PR (I used the new "Linked Issues" feature to do this) so even I didn't add the issue number in the description and you closed this PR, it would close the issue automatically! I'll make sure to do both next time just in case 😉 Thanks!

Yea that's a pretty new github feature I'm not very used to. As a slight preference, I think keeping the issue number also in the description provides slightly better visibility


matching organisms name problem

hi ...
i can't match organisms name... i have a layout with {% include 'organisms-' ~ header_layout %} in data.json I set the variable hader_layout...
my list of organisms header is:
000-h-hn(nmain+sprofiles)
001-h-hf(sprofiles+cinfo+nmain)
002-h-ht(sprofiles+cinfo)hf(nmain)
003-h-ht(cinfo)hf(cinfo)hn(nmain)

but when set  header_layout: h-ht(cinfo)hf(cinfo)hn(nmain) this show the organisms 002 or anything that start with h-ht, always show me 002... i try full format but nothing is displayed and i print a
{{ dump(header_layout) }}  in my layout and show me the correct var set in data.json ...
any idea how to solve this ?? thanks for help

the problem occurs when i use () in the file name...


⚠ ClamAV has degraded performance

In 41106a5, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 76 ms

Resolved: ClamAV performance has improved in 56c2e14.


Increased required realpath_cache_size on Windows

Incorrectly set realpath_cache_size causes big speed degradation on Windows systems (like really big!).
The realpath_cache_size in the php.ini is total number of bytes in the path strings stored + metadata (see http://php.net/manual/en/ini.core.php#ini.realpath-cache-size).
So if the requirement is set to 1024 (which means 1024B), only 10-20 paths would fit there.
Just to get some idea about optimal value, I set really high limit in php.ini and added realpath_cache_size() call in the BlogController of  the Symfony demo app (it fetches the currently used cache size). The value fluctuated around 500 000. Homepage of Sylius consumed around 1 500 000. As I had those var_dumps in the controller, more was probably consumed during the request. Therefore, I suggest using 5000k as a recommended value. It should be enough for even big Symfony projects.
I did just a very basic performance testing with admin of Symfony demo app:

with realpath_cache_size set to 1000 takes 500-600ms to load
with realpath_cache_size set to 5000k takes 100-200ms to load
(tested only with browser, changed the value in php.ini, restarted Apache)

And with Sylius homepage:

with realpath_cache_size set to 1000 takes 16s to load (yeah, seconds)
with realpath_cache_size set to 5000k takes 2s to load

Furthermore, I suggest, that we change this check into "error" instead of warning, because now the requirements checker says that everything is fine and you might want to fine-tune other settings. (Or we can split the check into two, one as error with lower limit, like 500k and second will be recommendation with 5000k)
Related:

original PR https://github.com/sensiolabs/SensioDistributionBundle/pull/106
original issue https://github.com/symfony/symfony/issues/9189

@mhujer thanks for proposing this improvement and for your excellent explanation.
Although I don't use Windows, I did some tests on my computer. The default realpath_cache_size was 16K and I changed it to 5000K as you suggested. A quick test on a complex Symfony app gave me a 5% CPU time gain. I need to do more tests, but this change looks promising indeed.
It'd be great if some Windows users could share their thoughts (and tests) about this proposal. Ping @WouterJ

I did a quick test with the Symfony demo application using PHP 5.5.12 on Windows 8.1 using Wamp and I can't see any major performance changes when setting this value to 1024 or 5000k (did some very basic tests however).

This one looks good to me. @mhujer Could you please address @stof comment?

@HeahDude fixed

Thanks @mhujer. LGTM 👍

Thank you @mhujer.

The check should be >= and not just >, because if you set it to 5000k or even 5M, you still get notified about. Also remove the 5000k thing and just say 5M, because thats the value being checked for.

Thank you @backbone87 for noticing!
Refs:

issue: https://github.com/sensiolabs/SensioDistributionBundle/issues/286
PR with fix: https://github.com/sensiolabs/SensioDistributionBundle/pull/287


Swap/Warp to recent window in the same space

Yabai is an awesome tool and makes my mac much less painful. Thanks for writing this.
Currently, I've this mapping to warp to most recent window
alt - return : yabai -m window --swap recent; \
                     yabai -m window --focus recent

Although, when I switch to another space, come back to original space and hit this hotkey, it warps to a window in the space I switched from. Is there a way to restrict the recent to the space I am in ?

The recent window is just a variable that is set each time the focused window changes. What you can do is use the query command to get the most recent window. Just do yabai -m query --windows --space | jq '.[].id' | sed -n '2p'. This gives you the id of the most recent window in the space. Then you can substitute this for recent`.
I haven't tested this but it is similar to how I do my switch to recent window command. Hope this helps.


HTTP / REST API

From the ReadMe it looks like it has built-in client-server model support. But not clear if it supports the clients over HTTP. Would be great if it can be made to work with HTTP/REST API.

All the client-server communication goes through HTTP. However, as we're using gRPC, we rely on the HTTP/2 version of the protocol to support bidirectional streams of data between the client and the server. We use bidirectional streaming to simplify (and optimize) the exchange of multiple messages within a single proof or within protocols comprising anonymous authentication schemes, in a particular order.
Unfortunately, the usage of bidirectional streaming proved to be a problem when trying to use translator solutions like grpc-gateway, grpc-websocket-proxy and grpc-web (the latter is apparently the way to go); we experimented with all of these in an effort to enable RESTful APIs in front of our gRPC services, but the projects do not fully support (or even see it possible to truly support) bidirectional streaming.
So, the lack of a translator solution that would support bidirectional streams is the reason we decided to leave behind the idea of having RESTful APIs for our services.

Thanks for the very clear explanation @mancabizjak So, does that mean only go clients will be able to benefit from this library? or do you see any way others (such as JS from browser) can take advantage of this functionality?

Sure thing! In theory, clients in any of the languages supported by gRPC (see here) could benefit from the library. On the practical side however, client logic inevitably relies on math and crypto functionality implemented in the crypto packages, which means that not only communication layer, but also (at least some of) math/crypto functionality may need to be rewritten in the chosen language. The practicality of creating an emmy client in an arbitrary language would depend on the complexity of that particular client (since we have different clients for running different protocols/proofs  of varying complexity).
 We also have a client/compatibility package that can be used with the gomobile tool in order to create client code bindings for Java/Objective C, for usage within Android/iOS mobile apps. But it is a very limited version, and it's currently not up-to-date with the parent client package.
As for using the clients and crypto library from browser, I think our colleague @janhartman can elaborate more on the topic, as he was experimenting with all of the projects mentioned earlier, and more :slightly_smiling_face: AFAIK it's a bit tricky...

The best way to run clients in a browser would probably be to compile to WebAssembly since Go 1.11 just added experimental support for it. The math and crypto functionality would not need to be implemented again and I'm fairly confident it would work in a browser. However, using the Go implementation of gRPC in a browser would probably be problematic and you would likely have to implement the communication yourself by using grpc-web or something similar.


termination_by fails with dependent matching

The well-founded recursion module gives a surprising error with some dependent pattern matches:
def Option.attach (o : Option α) : Option { x : α // o = x } :=
  match o with
  | some x => some ⟨x, rfl⟩
  | none => none

def foo (xs : List α) : Option (List α) := do
  if h : xs.length < 10 then
    pure (xs.take 2)
  else
    have : sizeOf (xs.drop 5) < sizeOf xs := sorry
    let ⟨res, h⟩ ← .attach (foo (xs.drop 5))
    pure (res ++ res)
termination_by foo => xs

/-
type mismatch
  xs
has type
  List α : Type u_1
but is expected to have type
  ?β : Sort ?u.938
-/

Weirdly enough, it complains that the termination measure does not type check.  The crux is the let ⟨res, h⟩ ← .attach (foo (xs.drop 5)) line; it works just fine with let res ← .attach (foo (xs.drop 5)) or let res ← foo (xs.drop 5).

It also works with termination_by foo => sizeOf xs.

The following well-founded recursion on an indexed inductive type fails in a similar way:
inductive DNat : Nat → Type
  | zero : DNat 0
  | succ : DNat n → DNat (n+1)

@[reducible]
def DNat.proj : DNat i → Nat := fun _ => i

def DNat.size₁ : DNat i → Nat
  | zero => 0
  | succ n => size₁ n
  termination_by size₁ n => n.proj -- works

def DNat.size₂ : DNat i → Nat
  | zero => 0
  | succ n => size₂ n
  termination_by size₂ n => sizeOf n -- works

def DNat.size₃ : DNat i → Nat
  | zero => 0
  | succ n => size₃ n
  /- type mismatch
    n
  has type
    DNat i : Type
  but is expected to have type
    ?β : Sort ?u.5000 -/
  termination_by size₃ n => n

@nomeata Any ideas?

The termination argument’s type must not depend on the function parameters, as we need to run type inference to find a WellFoundedRelation β for it.
I wonder if we can detect this dependency at the suitable point and give an error message like

The type of the termination argument,
DNat i
depends on one or more of the function’s parameters.
Hint: Try using sizeOf.

Trying that in #3414.

BTW, the issue @gebner reported initially is a consequence of the combination of @Vtec234 ’s issue with #2102. If you change it to
def Option.attach (o : Option α) : Option { x : α // o = x } :=
  match o with
  | some x => some ⟨x, rfl⟩
  | none => none

def foo (xs : List α) : Option (List α) := do
  if h : xs.length < 10 then
    pure (xs.take 2)
  else
    let r := foo (xs.drop 5)
    have : sizeOf (xs.drop 5) < sizeOf xs := sorry
    let ⟨res, h⟩ ← .attach r
    pure (res ++ res)
termination_by xs
decreasing_by sorry

it works.
The problem is that due to #2102, the matcher abstracts over foo. This means that α is no longer a “fixed” parameter, and the termination argument may no longer depend on it.


Add HTTPS cloning commands to documentation and small improvements

Hello. First of all, thank you for this nice template. IMO most people prefer to use HTTPS over SSH for cloning. So I have added options for commands to clone the template in that way too. Please, have a look :)

Amazing) Thanks for the PR.


failed wasm compilaiton because of locals exceed maximum

Describe the Bug
I have a crate that works with game's save file, i parse everything in SaveFile struct that holds a few fields, one of them is forge, this struct has a method that uses a large hashmap that contains "definitions" for specific IDs that save file can have, SCHEMATICS is said hashmap:

Calling a function that uses that hashmap causes locals exceed maximum error when trying to build a wasm file.
Steps to Reproduce

Clone this repo (wasm_issue branch)
Uncomment this line
Change this line to forge: Forge::from_gvas(gvas)?,
Run cargo build in main directory
Build editor_ui with trunk build or wasm-bindgen

Expected Behavior
Wasm generating with no errors
Actual Behavior
throws an error:
error: failed getting Wasm module for 'drg-save-editor\target\wasm32-unknown-unknown\debug\editor_ui.wasm'

Caused by:
    0: failed to parse input as wasm
    1: failed to parse code section
    2: locals exceed maximum (at offset 1931704)
2023-05-24T16:54:38.126342Z ERROR ❌ error
error from HTML pipeline

Caused by:
    0: error from asset pipeline
    1: wasm-bindgen call returned a bad status

Additional Context
More info in original issue on trunk repo

This is an issue with wasmparser:
https://github.com/bytecodealliance/wasm-tools/blob/wasmparser-0.78.2/crates/wasmparser/src/operators_validator.rs#L157-L159
There isn't much that can be done about this in wasm-bindgen, as far as I understand it the parser doesn't allow for more then 50000 local variables, which somehow you have hit:
https://github.com/bytecodealliance/wasm-tools/blob/wasmparser-0.78.2/crates/wasmparser/src/limits.rs#L29
You can see how you can figure out which function it is here: https://github.com/bytecodealliance/wasm-tools/issues/665. You then have to split it into multiple functions to avoid hitting that limit.

one much simpler solution i found is it to use opt-level = "s" for debug builds since apparently this problem disappears in release mode


WARNING: Could not determine version of module x

Error example in detail
Serverless: Invoke webpack:package
Serverless: WARNING: Could not determine version of module arrify
Hi. We have an issue where our root hoisted dependencies (using Lerna) cannot be resolved correctly when we bundle and webpack:package runs. This is causing the bundled package.json file to include referenced version as an empty string, see below.
"dependencies": {
      "arrify": "",
    }
The general issue with this is that dependencies are automatically resolved to latest version which can cause issues if there are breaking changes and so on.
I noticed that there were a version published 8 hours ago by @j0k3r where this functionality was included;

However, this did not seem to do the trick. Anyone with similar issues using monorepo, serverless and serverless-webpack?

Sorry, this was posted in the wrong github repo. Please remove.

It's not published, it has been merged. If you use the master of serverless-webpack, the option is avalaible. Otherwise it's not.


I can install is x 10.10 ? This laptop.

I can install is x 10.10 ? This laptop.

I don't think 10.10 is a great idea, because it doesn't support SMBIOS MBP14,1. Also 10.10 doesn't naturally support NVME SSD.


feat: register cronos etherscan urls

I have read the CLA Document and I hereby sign the CLA

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
188 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-5.2%) to 84.841%

Files with Coverage Reduction
New Missed Lines
%

gnosis/eth/multicall.py
9
86.41%

gnosis/eth/ethereum_client.py
42
81.48%

gnosis/eth/oracles/oracles.py
137
49.44%

Totals

Change from base Build<PHONE_NUMBER>:
-5.2%

Covered Lines:
3179

Relevant Lines:
3747

💛  - Coveralls


Troubleshooting "CUDA Driver unavailable"

Description
We have a telsa T4 machine with driver 418.87.01. But when running trtserver with cuda 10.2, it reports CUDA Driver UNAVAILABLE.
Two questions:

We have CUDA 10.2 compat installed, for T4 card, there should not be compatibility problem.
In `nv_entrypoint.sh', there is _CUDA_COMPAT_STATUS env, but we cannot find where the env _CUDA_COMPAT_STATUS is set.

root@61563aa2c63e:/# bash /opt/tensorrtserver/nvidia_entrypoint.sh

===============================
== TensorRT Inference Server ==
===============================

NVIDIA Release 20.03dev (build 10247880)

Copyright (c) 2018-2019, NVIDIA CORPORATION.  All rights reserved.

Various files include modifications (c) NVIDIA CORPORATION.  All rights reserved.
NVIDIA modifications are covered by the license terms that apply to the underlying
project or file.
find: File system loop detected; '/usr/bin/X11' is part of the same file system loop as '/usr/bin'.

ERROR: This container was built for NVIDIA Driver Release 440.33 or later, but
       version 418.87.01 was detected and compatibility mode is UNAVAILABLE.

       [[CUDA Driver UNAVAILABLE (cuDevicePrimaryCtxRetain() returned 101)]]

NOTE: The SHMEM allocation limit is set to the default of 64MB.  This may be
   insufficient for the inference server.  NVIDIA recommends the use of the following flags:
   nvidia-docker run --shm-size=1g --ulimit memlock=-1 --ulimit stack=67108864 ...

root@61563aa2c63e:/# nvidia-smi
Mon Jun  1 08:04:09 2020
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 418.87.01    Driver Version: 418.87.01    CUDA Version: 10.1     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  Tesla T4            On   | 00000000:1A:00.0 Off |                    0 |
| N/A   48C    P0    28W /  70W |   9325MiB / 15079MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+
|   1  Tesla T4            On   | 00000000:1B:00.0 Off |                    0 |
| N/A   32C    P8     9W /  70W |      0MiB / 15079MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+
|   2  Tesla T4            On   | 00000000:1D:00.0 Off |                    0 |
| N/A   30C    P8    10W /  70W |      0MiB / 15079MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+
|   3  Tesla T4            On   | 00000000:1E:00.0 Off |                    0 |
| N/A   32C    P8    10W /  70W |      0MiB / 15079MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+
|   4  Tesla T4            On   | 00000000:3D:00.0 Off |                    0 |
| N/A   30C    P8    11W /  70W |      0MiB / 15079MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+
|   5  Tesla T4            On   | 00000000:3E:00.0 Off |                    0 |
| N/A   31C    P8     9W /  70W |      0MiB / 15079MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+
|   6  Tesla T4            On   | 00000000:40:00.0 Off |                    0 |
| N/A   30C    P8     9W /  70W |      0MiB / 15079MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+
|   7  Tesla T4            On   | 00000000:41:00.0 Off |                    0 |
| N/A   30C    P8     9W /  70W |      0MiB / 15079MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
+-----------------------------------------------------------------------------+

Triton Information
What version of Triton are you using?
20.03
Are you using the Triton container or did you build it yourself?
To Reproduce
Can only reproduce on some machines.
Expected behavior
The Trtserver should starts successfully on T4 machine.

How to troubleshoot this problem?

@clockfly you mentioned that you are trying to run the trtserver with CUDA version 10.2 but the nvidia-smi output shows that the your system has CUDA Version: 10.1.

@clockfly Please try updating to CUDA 10.2 on your host system and seeing if that fixes the issue. Reopen this issue if still having a problem.


How to add charger/ charging stations ?

Hi, im sorry if this sounds absurd , but how can I add charger in my dashboard ? I have read the similar issue someone faced before this but the issue was closed due to inactivity . Can someone help me pls ?
I read from the previous issue that I need to go to onboard new station and then create new charging registration tokens and copy the URL as shown below

Once I have copied the URL , what are the next steps ? I'm sorry I'm a totally beginner , I hope someone can help me step by step , thank you , these might be a long issue T_T .
btw I don't have any EV charger yet so I cant really tested it properly, I just don't know how to connect the charger since there's no guidelines or steps in which I can follow . If no direct solution can be provided can someone at least brief to me how are the process of connecting the actual EV charger to the software ? Thank you so much , your help will be greatly appreciated T_T .

now I have successfully connected to EV simulator thanks to https://github.com/sap-labs-france/ev-server/issues/3048 and https://github.com/sap-labs-france/ev-server/issues/3442
The charging station on the dashboard can be seen below

after successfully connected to the simulator, May I know if there is a method in which we can send random values of energy consumptions to the dashboard so that we can see the graph for energy consumptions ? your help will be greatly appreciated

hi , now im able to change parameters of the charging station and assign the charging station site area . however I cannot start transaction to simulate the data . can someone help me ? the logs are showing the error

am I missing something ? your help will be greatly appreciated

by the way , what url should I placed in the URL section when changing the charger parameters ? is it the websocket URL which I get from the token when onboard new station ?

Any news on what "URL" means?


Create docs folder, add architecture assets

Added a docs folder where we can store documentation resources.
I also added a couple of files resulting from the conversation I had with @bradhead this morning, that pick up on the idea I was mentioning of having a data contract enforced by an interface/abstract class, and leave the actual implementation details to each jurisdiction/entity that will be using the service to issue credentials of any kind. The xml file is a draw.io document that can be re-imported and updated, the svg file is a snapshot at the time of submission.
The proposed architecture has the benefit of being completely stateless (exception made for the information related to whether a credential has been issued or not, which is not really subject to privacy concerns), and removes the need to define ahead of time a specific API model between the VC controller and a data source (or potentially more than one) that is used to obtain the information that will be embedded in the credential.
The VC controller will rely on an interface + abstract class defining the APIs and objects that it will sue to perform its tasks. These APIs will include at least:

a method to perform discovery of possible data sources, and obtain a list of providers that can be used to fetch data.
a method to request the data source(s) for a list of records that could be used to compose a credential.
a method to request the data source for details on specific records - based on input parameters - that returns an the object the VC controller expects in order to successfully build and issue a VC.

The implementation of the abstract class will be provided by each entity using the service, and plugged-in at runtime for their instance: options for this are automatic discovery of implementations or simply specifying the name of the class implementing the abstract class, and having the controller try to find and load such binary on startup. As an example, for our demo the abstract class will also act as the data source itself, loading data form a file or maybe even just providing hard-coded data.
All of the business logic responsible for fetching the data from the data source(s), compiling an appropriate payload for the VC controller and implementing authentication/authorization procedures is delegated to the implementation, making it extremely flexible. In our idea, the JWT identifying the user would be passed along as a bearer token so that it can be provided to the data source (e.g.: Health Gateway) as a form of authentication and authorization to only scope requests to a specific user and their data.
Depending on how much information needs to be presented to the user, the frontend could be made smart enough to  adapt to different data models, or just rewritten quickly for a new credential structure by using the APIs exposed by the VC controller. It will not hold any information identifying the user, it will instead be provided by the authentication method (Identity Provider, token provided with an invite: it is up to each deployment to specify and potentially implement their authentication structure as required).
The VC controller will only need to have a configurable list of CORS origins injected at runtime (config file, environment variable) that specifies which domains it will accept requests from, so that existing frontends on separate domains could use the exposed APIs.
I think this is a relatively detailed picture, but at the same time provides a simple approach with the minimal possible number of components that need to be defined ahead of time. It also allows - as already mentioned - to easily repurpose the same code to issue credentials that come from data sources we have not really thought of yet (flat file, REST, SOAP, RPC, direct db read, etc.).

Agree @itchison .

@itchison agreed, mine is a suggestion/recommendation on how we can obtain a system that is scalable and flexible. Definitely open to other suggestions that may be better suited to the problem, my diagram is just meant to propose a solution I see fit - not meant to be the ultimate, perfect solution.
I also agree with you that issuing a credential may be higher on the backlog than run time dependency injection, at the same time structuring things from the beginning so that they can be updated/tweaked without having to deal with major technical debt (e.g.: major refactoring and rewrite to be able to change the data source) is valuable when designing components that must be - by nature - reusable (unless this is a one use case, one jurisdiction solution).


System Diagrams

System Architecture Block Diagram
I've made a block diagram outiling the possible system's architecture.
What do you think?

Object Counting Function Diagram

Motor Control Function Diagram

User Communication Function Diagram